-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Cleanup shim manager configuration #10243
base: main
Are you sure you want to change the base?
Conversation
389e06d
to
28bc851
Compare
core/runtime/v2/shim_manager.go
Outdated
switch vv := v.(type) { | ||
case bool: | ||
schedCore = vv | ||
case string: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If sched_core
is not a boolean type, containerd can not run in v1.7.12 and v2.0.0-rc.2. Do we need to handle the situation where sched_core
is not a boolean type?
version = 2
[plugins."io.containerd.runtime.v2.task"]
platforms = ["linux/amd64"]
sched_core = "true"
- containerd v1.7.12
containerd: (4, 3): Can't convert true(string) to bool
- containerd v2.0.0-rc.2
containerd: toml: cannot decode TOML string into struct field v2.Config.SchedCore of type bool
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, thanks for checking on this. I simplified to just check for boolean and warn if not boolean.
28bc851
to
40d4466
Compare
Keep platforms configuration on task manager and add environment config for shims. Signed-off-by: Derek McGowan <derek@mcg.dev>
40d4466
to
ca59fb0
Compare
LGTM |
Keep platforms configuration on task manager and add environment config for shims. The
sched_core
config is very runtime specific and does not need to be a top level configuration, rather just being able to add environment variables is much more useful. Renamed the shim manager plugin toio.containerd.shim.v1.manager
rather thanio.containerd.shim.v1.shim
, which was a recent change after config version 2.ctr plugins ls
now shows