Skip to content

Windows CNI install script using lowercase "destdir"#4599

Merged
dmcgowan merged 1 commit intocontainerd:masterfrom
estesp:fix-script-var
Oct 3, 2020
Merged

Windows CNI install script using lowercase "destdir"#4599
dmcgowan merged 1 commit intocontainerd:masterfrom
estesp:fix-script-var

Conversation

@estesp
Copy link
Copy Markdown
Member

@estesp estesp commented Oct 2, 2020

Fixes packaging via GH Actions script, which sets DESTDIR and is used
across many scripts.

Found while bringing the CNI/CRI packaging work back to release/1.3. Will backport to release/1.4 to fix next point release of v1.4.x as well.

Signed-off-by: Phil Estes [email protected]

Fixes packaging via GH Actions script, which sets DESTDIR and is used
across many scripts.

Signed-off-by: Phil Estes <[email protected]>
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Oct 2, 2020

Build succeeded.

Copy link
Copy Markdown
Contributor

@Zyqsempai Zyqsempai 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

@dmcgowan dmcgowan left a comment

Choose a reason for hiding this comment

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

LGTM

@dmcgowan dmcgowan merged commit d852786 into containerd:master Oct 3, 2020
@estesp estesp added cherry-picked/1.4.x PR commits are cherry picked into the release/1.4 branch and removed cherry-pick/1.4.x labels Oct 5, 2020
@estesp estesp deleted the fix-script-var branch October 5, 2020 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-picked/1.4.x PR commits are cherry picked into the release/1.4 branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants