Skip to content

[Refactor] Move generateSharingPIDOpts from cmd to pkg/containerutil#1893

Merged
AkihiroSuda merged 1 commit into
containerd:mainfrom
Laitr0n:refactor_generateSharingPIDOpts
Jan 22, 2023
Merged

[Refactor] Move generateSharingPIDOpts from cmd to pkg/containerutil#1893
AkihiroSuda merged 1 commit into
containerd:mainfrom
Laitr0n:refactor_generateSharingPIDOpts

Conversation

@Laitr0n
Copy link
Copy Markdown
Contributor

@Laitr0n Laitr0n commented Jan 20, 2023

Splitting pr #1864
Same with pr #1857, prepare for refactor the container start command.

Checklist:

  • Move generateSharingPIDOpts from cmd to pkg/containerutil

Signed-off-by: Laitron meetlq@outlook.com

@Laitr0n Laitr0n marked this pull request as ready for review January 20, 2023 15:59
Comment thread pkg/containerutil/containerutil.go Outdated
@Laitr0n Laitr0n force-pushed the refactor_generateSharingPIDOpts branch 4 times, most recently from 20ff38c to 1a3d437 Compare January 20, 2023 17:01
@Laitr0n Laitr0n requested a review from AkihiroSuda January 20, 2023 17:04
@Laitr0n Laitr0n force-pushed the refactor_generateSharingPIDOpts branch from 1a3d437 to cc1f9d1 Compare January 20, 2023 17:08
Comment thread pkg/containerutil/containerutil.go Outdated
@Laitr0n Laitr0n force-pushed the refactor_generateSharingPIDOpts branch from cc1f9d1 to 8cb887a Compare January 22, 2023 02:32
@Laitr0n Laitr0n requested review from djdongjin and removed request for AkihiroSuda January 22, 2023 02:34
Copy link
Copy Markdown
Member

@djdongjin djdongjin left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

Copy link
Copy Markdown
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Thanks

@AkihiroSuda AkihiroSuda merged commit ac44ddb into containerd:main Jan 22, 2023
@AkihiroSuda AkihiroSuda added this to the v1.2.0 milestone Jan 22, 2023
@Laitr0n Laitr0n deleted the refactor_generateSharingPIDOpts branch January 23, 2023 16:50
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