Comments (10)
would this be a good time to bring up logging to syslog?
from crun.
@dmolik if you need that, I can try to add it in the current mess :-)
Is it anything runc already does and we need to keep some CLI compatibility? If not, would something like --log=syslog:IDENTITY
be good for you?
from crun.
@dmolik I've opened https://github.com/giuseppe/crun/pull/28
It is still work in progress, but you can start using syslog, like crun --log=syslog:
and journald as crun --log=journald:
. If not specified, they both use the container name as the identifier, otherwise they use whatever is specified after :
. So for example if you want to override an identifier in the logging, you can do: crun --log=syslog:FOO
.
What do you think?
from crun.
Setting the facility name would be cool.
from crun.
I've implemented that functionality in the PR.
You can use --log=syslog:IDENTITY
to override the identifier, otherwise it defaults to the container name. Similarly for --log=journald:
from crun.
is that enough for you? :-) Can I merge the PR?
from crun.
LGTM. Thanks! Sorry for the slow response.
from crun.
@giuseppe Should this issue be closed?
from crun.
@giuseppe Can we close?
from crun.
yes, let's close it as there is no work going on and it is not really a specific issue
from crun.
Related Issues (20)
- Discrepancy between crun and runc when disallowing access by default to devices with cgroups v1 HOT 5
- Does not ignore SIGWINCH if no terminal is allocated
- container create failed: sd-bus call: Cannot set property CPUQuotaPeriodUSec, or unknown propert HOT 8
- Compilation error with clang++ 17 HOT 3
- Missing libsystemd headers on Alpine Linux v3.19 HOT 2
- Version 1.14.5 HOT 1
- [packit] Propose downstream failed for release 1.15 HOT 2
- Container creation fails with read-only bind mount HOT 3
- what is the purpose and future of crun-wasm - asking for (a friend) Debian packaging :) ? HOT 5
- Different CRUN/RUNC cgroup hierarchy HOT 6
- weird seccomp caching bug, causes bud-runtime-flag test to flake HOT 4
- The flag `O_PATH` could be added in more places HOT 1
- Pod cannot be deleted due to missing container startup command when using crun HOT 8
- Cannot run any container if storage location is owned by a group that is not the users main group (rootless mode) HOT 5
- OCI runtime create failed: unknown version specified: unknown. HOT 2
- Warning `Unit uses MemoryLimit=;` HOT 5
- Systemd not supported on crun - s390x HOT 14
- Support for wasm-micro-runtime handler HOT 1
- Why fork the current process on create when libcrun_container_create() is called by python process? HOT 5
- can't see poststart running. HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from crun.