Skip to content

[sandbox] Use options to pass PodSandboxConfig to shims#8100

Merged
kzys merged 1 commit intocontainerd:mainfrom
mxpv:options
Feb 13, 2023
Merged

[sandbox] Use options to pass PodSandboxConfig to shims#8100
kzys merged 1 commit intocontainerd:mainfrom
mxpv:options

Conversation

@mxpv
Copy link
Copy Markdown
Member

@mxpv mxpv commented Feb 13, 2023

Allow sandboxed shims access PodSandboxConfig when creating a sandbox, this way shims can configure many things - DNS, ports, etc.

/cc @dcantah this might be useful for your use case too.

Signed-off-by: Maksym Pavlenko [email protected]

@mxpv mxpv added this to the 1.7 milestone Feb 13, 2023
@mxpv mxpv added area/cri Container Runtime Interface (CRI) easy-to-review Easy to review labels Feb 13, 2023
@mxpv mxpv changed the title Use options to pass PodSandboxConfig to shims [sandbox] Use options to pass PodSandboxConfig to shims Feb 13, 2023
Copy link
Copy Markdown
Member

@dcantah dcantah left a comment

Choose a reason for hiding this comment

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

Makes sense, LGTM!

@kzys kzys merged commit 458fd48 into containerd:main Feb 13, 2023
@mxpv mxpv deleted the options branch February 13, 2023 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/cri Container Runtime Interface (CRI) easy-to-review Easy to review

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants