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

Update CI infrastructure for v1.10 release #15947

Merged

Conversation

christarazi
Copy link
Member

@christarazi christarazi commented Apr 29, 2021

  • test: Avoid using image digests in CI

@christarazi christarazi requested a review from a team as a code owner April 29, 2021 21:37
@christarazi christarazi added area/CI Continuous Integration testing issue or flake needs-backport/1.10 labels Apr 29, 2021
@christarazi christarazi added the release-note/misc This PR makes changes that have no direct user impact. label Apr 29, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Apr 29, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.10.0-rc2 Apr 29, 2021
@christarazi christarazi changed the title pr/christarazi/misc 1.10 changes Update CI infastructure for v1.10 release Apr 29, 2021
@christarazi

This comment has been minimized.

@christarazi christarazi changed the title Update CI infastructure for v1.10 release Update CI infrastructure for v1.10 release Apr 29, 2021
test/helpers/cons.go Outdated Show resolved Hide resolved
test/helpers/kubectl.go Show resolved Hide resolved
@christarazi christarazi added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. and removed ready-to-merge This PR has passed all tests and received consensus from code owners to merge. labels Apr 30, 2021
@christarazi

This comment has been minimized.

CI runs don't need to use image digests, as there's no real benefit.
Additionally, the CI infrastructure is not set up to properly parse the
digests from the image tag, which causes the upgrade test
(test/k8sT/Updates.go).

Signed-off-by: Chris Tarazi <chris@isovalent.com>
@christarazi christarazi force-pushed the pr/christarazi/misc-1.10-changes branch from 3a9894f to 70dd06f Compare May 4, 2021 17:12
@christarazi
Copy link
Member Author

Dropped the commit which bumped the stable Helm releases to v1.10 as we don't have the v1.10-dev chart ready yet. A new PR will include it once we have that chart.

@christarazi
Copy link
Member Author

test-me-please

@christarazi christarazi added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label May 6, 2021
@jrajahalme jrajahalme merged commit 1ac77fc into cilium:master May 6, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.10 in 1.10.0-rc2 May 6, 2021
@christarazi christarazi deleted the pr/christarazi/misc-1.10-changes branch May 6, 2021 18:45
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.10 to Backport done to v1.10 in 1.10.0-rc2 May 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
1.10.0-rc2
Backport done to v1.10
Development

Successfully merging this pull request may close these issues.

None yet

5 participants