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

.github: do not useDigest in conformance tests #16836

Merged
merged 1 commit into from Jul 9, 2021
Merged

Conversation

aanm
Copy link
Member

@aanm aanm commented Jul 9, 2021

Conformance tests are executed against the code being submitted in a PR.
If useDigest is set to true it will use the default digest defined in
the helm charts, which in stable branches is the last stable release.

Fixes: 41f66a1 (".github: use quay.io images in smoke tests")
Signed-off-by: André Martins andre@cilium.io

Note for reviewers: <1.10 branches are not affected since useDigest was only set to true by default in 1.10

@aanm aanm added release-note/misc This PR makes changes that have no direct user impact. needs-backport/1.10 labels Jul 9, 2021
@aanm aanm requested review from a team as code owners July 9, 2021 00:09
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.10.3 Jul 9, 2021
.github/workflows/conformance-k8s-network-policies.yaml Outdated Show resolved Hide resolved
.github/workflows/tests-smoke.yaml Outdated Show resolved Hide resolved
Conformance tests are executed against the code being submitted in a PR.
If `useDigest` is set to true it will use the default digest defined in
the helm charts, which in stable branches is the last stable release.

Fixes: 41f66a1 (".github: use quay.io images in smoke tests")
Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm force-pushed the pr/fix-conformance-tests branch from 097c6e7 to 9e7d54f Compare July 9, 2021 00:10
Copy link
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

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

Ah.

@aanm aanm merged commit a4bdf97 into master Jul 9, 2021
@aanm aanm deleted the pr/fix-conformance-tests branch July 9, 2021 01:18
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport done to v1.10 in 1.10.3 Jul 9, 2021
@aanm aanm added release-note/ci This PR makes changes to the CI. and removed release-note/misc This PR makes changes that have no direct user impact. labels Jul 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/ci This PR makes changes to the CI.
Projects
No open projects
1.10.3
Backport done to v1.10
Development

Successfully merging this pull request may close these issues.

None yet

4 participants