-
Notifications
You must be signed in to change notification settings - Fork 2
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
Chart: Bump cluster
chart v0.18.0.
#564
Conversation
This comment has been minimized.
This comment has been minimized.
cluster
chart v0.18.0
This comment has been minimized.
This comment has been minimized.
/run cluster-test-suites |
This comment has been minimized.
This comment has been minimized.
/run cluster-test-suites |
This comment has been minimized.
This comment has been minimized.
/run cluster-test-suites |
This comment has been minimized.
This comment has been minimized.
cluster
chart v0.18.0cluster
chart v0.18.0.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
db611c3
to
c21873e
Compare
/run cluster-test-suites |
This comment has been minimized.
This comment has been minimized.
/run cluster-test-suites |
This comment has been minimized.
This comment has been minimized.
/run cluster-test-suites |
This comment has been minimized.
This comment has been minimized.
/run cluster-test-suites |
This comment has been minimized.
This comment has been minimized.
/run cluster-test-suites |
This comment has been minimized.
This comment has been minimized.
/run cluster-test-suites |
This comment has been minimized.
This comment has been minimized.
/run cluster-test-suites |
This comment has been minimized.
This comment has been minimized.
4b4dccf
to
013fc08
Compare
@tuladhar @Gacko I have rebased this PR so it now upgrades cluster chart from v0.17.0 to v0.18.0. When upgrading from v0.16.0 to v0.18.0 the e2e tests were failing, and now since v0.17.0 e2e tests were green, it seems that changes from v0.18.0 are breaking the e2e tests, but will run them now again, so we see how it goes. /run cluster-test-suites |
This comment has been minimized.
This comment has been minimized.
Before rebasing the error was about aws-pod-identity-webhook-app not having replicas:
Now it's another failure with hello-world replicas (details here):
|
Another try, if it fails differently again, something is flaky /run cluster-test-suites |
The upgrade was supposedly successful
but the current state of nodes is:
|
This comment has been minimized.
This comment has been minimized.
/run cluster-test-suites |
cluster-test-suites
📋 View full results in Tekton Dashboard Rerun trigger: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should state that this will roll all worker nodes.
There were differences in the rendered Helm template, please check! Output
|
/run cluster-test-suites |
cluster-test-suites
📋 View full results in Tekton Dashboard Rerun trigger: |
What this PR does / why we need it
Checklist
Trigger e2e tests
/run cluster-test-suites