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

test: Un-Quarantine K8sUpdates on GKE #14464

Merged
merged 1 commit into from
Dec 21, 2020

Conversation

gandro
Copy link
Member

@gandro gandro commented Dec 21, 2020

This removes the quarantine check for the K8sUpdates test on GKE. The
issues with upgrade tests had on GKE were addressed in #14187, #14294,
#14295, and #14393.

The last fix was merged 6 days ago. The quarantined GKE pipeline had
since no failures that look related to the issues with DNS that we
previously observed.

Fixes: #13833
Fixes: #13898

This removes the quarantine check for the K8sUpdates test on GKE. The
issues with upgrade tests had on GKE were addressed in #14187, #14294,
 #14295, and #14393.

The last fix was merged 6 days ago. The quarantined GKE pipeline had
since no failures that look related to the issues with DNS that we
previously observed.

Fixes: #13833
Fixes: #13898

Signed-off-by: Sebastian Wicki <sebastian@isovalent.com>
@gandro gandro added release-note/ci This PR makes changes to the CI. ci/flake This is a known failure that occurs in the tree. Please investigate me! needs-backport/1.9 labels Dec 21, 2020
@gandro gandro requested a review from a team December 21, 2020 11:09
@gandro gandro requested a review from a team as a code owner December 21, 2020 11:09
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.10.0 Dec 21, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.9.2 Dec 21, 2020
@gandro
Copy link
Member Author

gandro commented Dec 21, 2020

test-me-please

Copy link
Member

@aanm aanm left a comment

Choose a reason for hiding this comment

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

🚀 🚀 🚀 🚀 🚀

@vadorovsky vadorovsky merged commit 699ca46 into master Dec 21, 2020
@vadorovsky vadorovsky deleted the pr/gandro/ci-gke-unquarantine-upgrade-test branch December 21, 2020 16:28
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.9 in 1.9.2 Jan 6, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.9 in 1.9.2 Jan 6, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.9 to Needs backport from master in 1.9.2 Jan 13, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.9 to Needs backport from master in 1.9.2 Jan 13, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed this from Needs backport from master in 1.9.2 Jan 14, 2021
@gandro
Copy link
Member Author

gandro commented Jan 14, 2021

I removed the "needs-backport" label for now, as this caused issues on the v1.9 branch. Needs investigation first as to why.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/flake This is a known failure that occurs in the tree. Please investigate me! release-note/ci This PR makes changes to the CI.
Projects
No open projects
6 participants