Skip to content

cri: append envs from image config to empty slice to avoid env lost#5054

Merged
estesp merged 1 commit intocontainerd:masterfrom
jterry75:fix_env_windows
Feb 19, 2021
Merged

cri: append envs from image config to empty slice to avoid env lost#5054
estesp merged 1 commit intocontainerd:masterfrom
jterry75:fix_env_windows

Conversation

@jterry75
Copy link
Copy Markdown
Contributor

Signed-off-by: Justin Terry (SF) juterry@microsoft.com

Signed-off-by: Justin Terry (SF) <juterry@microsoft.com>
@jterry75 jterry75 requested a review from estesp February 19, 2021 00:39
@jterry75
Copy link
Copy Markdown
Contributor Author

Ref #5024 this fix is for Windows that is identical to the issue reference for Linux.

@dcantah
Copy link
Copy Markdown
Member

dcantah commented Feb 19, 2021

LGTM

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Feb 19, 2021

Build succeeded.

Copy link
Copy Markdown
Member

@kevpar kevpar left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

@estesp estesp merged commit e58be59 into containerd:master Feb 19, 2021
@jterry75 jterry75 deleted the fix_env_windows branch February 19, 2021 02:41
@mikebrow
Copy link
Copy Markdown
Member

Ref #5024 this fix is for Windows that is identical to the issue reference for Linux.

Thx @jterry75

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