Skip to content

[release/1.6] Allow git commands in Vagrantfile#6941

Merged
mikebrow merged 1 commit intocontainerd:release/1.6from
estesp:backport-vagrant-fix
May 12, 2022
Merged

[release/1.6] Allow git commands in Vagrantfile#6941
mikebrow merged 1 commit intocontainerd:release/1.6from
estesp:backport-vagrant-fix

Conversation

@estesp
Copy link
Copy Markdown
Member

@estesp estesp commented May 12, 2022

Due to a git CVE, we need to mark /vagrant as a "safe" directory
otherwise building containerd inside Vagrant fails with git command
failures.

Backport of #6915 for release/1.6

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

Due to a git CVE, we need to mark /vagrant as a "safe" directory
otherwise building containerd inside Vagrant fails with git command
failures.

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

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

LGTM

@mikebrow mikebrow merged commit 74c1e96 into containerd:release/1.6 May 12, 2022
@mikebrow
Copy link
Copy Markdown
Member

nod build fails and .. the version of containerd in that vagrant doesn't report version correctly to the cri api..

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.

5 participants