Skip to content

sandbox: create sandbox with network namespace path#8149

Merged
mxpv merged 1 commit intocontainerd:mainfrom
Burning1020:sb-netns
Mar 8, 2023
Merged

sandbox: create sandbox with network namespace path#8149
mxpv merged 1 commit intocontainerd:mainfrom
Burning1020:sb-netns

Conversation

@Burning1020
Copy link
Copy Markdown
Member

follow up #8129

@dcantah dcantah requested a review from mxpv February 22, 2023 14:02
Copy link
Copy Markdown
Member

@mxpv mxpv left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread sandbox/controller.go Outdated
@fuweid fuweid added this to the 2.0 milestone Feb 25, 2023
Comment thread sandbox/controller.go Outdated
@Burning1020 Burning1020 force-pushed the sb-netns branch 3 times, most recently from 79c4730 to 3fb3c11 Compare February 27, 2023 13:12
@Burning1020
Copy link
Copy Markdown
Member Author

@samuelkarp PTAL

Comment thread api/runtime/sandbox/v1/sandbox.proto
@fuweid fuweid modified the milestones: 2.0, 1.7 Mar 6, 2023
@Burning1020 Burning1020 force-pushed the sb-netns branch 2 times, most recently from 7be98f7 to 36b7c27 Compare March 6, 2023 09:13
Comment thread sandbox/proxy/controller.go Outdated
Comment thread sandbox/controller.go
Comment thread plugins/sandbox/controller.go Outdated
@Burning1020 Burning1020 force-pushed the sb-netns branch 4 times, most recently from c78e453 to 9d2eceb Compare March 6, 2023 13:13
@dmcgowan
Copy link
Copy Markdown
Member

dmcgowan commented Mar 7, 2023

I'm in favor of switching back to the version with the proto update. There was already approval from those API owners to make that change before the final 1.7 release.

@mxpv mxpv merged commit 48a1350 into containerd:main Mar 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants