Skip to content

[20.10 backport] Add shim config for custom runtimes for plugins#41974

Merged
tiborvass merged 1 commit into
moby:20.10from
thaJeztah:20.10_backport_for_linux_1169_plugins_custom_runtime-panic
Feb 18, 2021
Merged

[20.10 backport] Add shim config for custom runtimes for plugins#41974
tiborvass merged 1 commit into
moby:20.10from
thaJeztah:20.10_backport_for_linux_1169_plugins_custom_runtime-panic

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

backport of #41854

fixes docker/for-linux#1169 [v20.10.1] dockerd panics when configuring default runtime
fixes docker/for-linux#1185 [20.10.2] start docker failed

This fixes a panic when an admin specifies a custom default runtime,
when a plugin is started the shim config is nil.

- Description for the changelog

Fix a daemon panic on setups with a custom default runtime configured

@cpuguy83 cpuguy83 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

This fixes a panic when an admin specifies a custom default runtime,
when a plugin is started the shim config is nil.

Signed-off-by: Brian Goff <[email protected]>
(cherry picked from commit 2903863)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah
thaJeztah force-pushed the 20.10_backport_for_linux_1169_plugins_custom_runtime-panic branch from 89d22bc to df2a989 Compare February 17, 2021 20:20
@thaJeztah

Copy link
Copy Markdown
Member Author

rebased to trigger CI with test-fixes that were merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants