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
ci: add cluster upgrade test #448
Conversation
Codecov Report
@@ Coverage Diff @@
## master #448 +/- ##
=======================================
Coverage 62.86% 62.86%
=======================================
Files 6 6
Lines 474 474
=======================================
Hits 298 298
Misses 144 144
Partials 32 32 |
/azp run pr-e2e-azure |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run pr-e2e-azure |
Azure Pipelines failed to run 1 pipeline(s). |
2b15c31
to
307146a
Compare
94536f1
to
947797d
Compare
Signed-off-by: Anish Ramasekar <anish.ramasekar@gmail.com>
az aks upgrade -g ${AZURE_CLUSTER_NAME} -n ${AZURE_CLUSTER_NAME} -k ${AKS_UPGRADE_VERSION} --yes | ||
|
||
# Sleep for 120 seconds to wait for nodes and pods to become ready | ||
sleep 2m |
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.
is 2m enough?
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.
The 2m sleep is after upgrade to check health of the components. az aks upgrade
is blocking until the upgrade is complete.
Signed-off-by: Anish Ramasekar anish.ramasekar@gmail.com
Reason for Change:
Requirements
Issue Fixed:
fixes #174
Does this change contain code from or inspired by another project?
If "Yes," did you notify that project's maintainers and provide attribution?
Special Notes for Reviewers: