Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dockerfiles: quote FROM images if they contain 'sha256' #14887

Merged
merged 1 commit into from
Feb 8, 2021

Conversation

aanm
Copy link
Member

@aanm aanm commented Feb 8, 2021

This prevents quay.io trigger builds from being considered invalid.

Signed-off-by: André Martins andre@cilium.io

This prevents quay.io trigger builds from being considered invalid.

Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm added release-note/misc This PR makes changes that have no direct user impact. area/build Anything to do with the build, more general then area/CI labels Feb 8, 2021
@aanm aanm requested a review from a team February 8, 2021 12:53
@aanm aanm requested a review from a team as a code owner February 8, 2021 12:53
@aanm aanm requested a review from a team February 8, 2021 12:53
@aanm aanm requested a review from a team as a code owner February 8, 2021 12:53
@aanm aanm requested a review from jrfastab February 8, 2021 12:53
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.10.0 Feb 8, 2021
@aanm
Copy link
Member Author

aanm commented Feb 8, 2021

test-me-please

@joestringer joestringer merged commit 77838f7 into cilium:master Feb 8, 2021
@joestringer joestringer moved this from In progress to Done in 1.10.0 Feb 12, 2021
@joestringer joestringer moved this from In progress to Done in 1.10.0 Feb 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build Anything to do with the build, more general then area/CI release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants