-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make containerd config configurable #830
Comments
This would also be useful to configure a registry mirror, which seems to be possible with docker only, currently. |
does #790 resolve this? |
@bryantbiggs I think it's not, because So the solution implemented in #790 does not support adding or overwriting any keys under tables that are already defined by the default configuration. A similar option like |
@ncsibra Sorry I am late here, but did you find the solution for it. I am facing a similar issue |
@rishabhToshniwal the options for this are better on AL2023-based EKS AMIs, see: https://awslabs.github.io/amazon-eks-ami/nodeadm/doc/examples/#configuring-containerd This doesn't rely on |
What would you like to be added:
Make containerd config configurable, similarly to
--docker-config-json
option.Why is this needed:
By default
max_container_log_line_size
set to16384
which means when a line is bigger than this size will be split into multiple log messages.Our problem with this, that it's based on byte size and sometimes it splits a UTF-8 character containing multiple bytes, which makes it invalid and unable to parse by fluent-bit/fluentd.
E.g.:
It's possible to modify the config in a custom userdata script, but would be nicer to do this through a parameter to the bootstrap script.
The text was updated successfully, but these errors were encountered: