-
Notifications
You must be signed in to change notification settings - Fork 194
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: fix upgrade tests #800
Conversation
/azp run pr-e2e-azure |
Azure Pipelines successfully started running 1 pipeline(s). |
a0bc071
to
d7c25b9
Compare
Signed-off-by: Anish Ramasekar <anish.ramasekar@gmail.com>
@@ -26,7 +26,7 @@ steps: | |||
|
|||
- script: | | |||
#Run test with latest on preview aks version available | |||
aksVersion=$(az aks get-versions -l $(AZURE_LOCATION) --query "max(orchestrators[?isPreview==null].orchestratorVersion)" -otsv) | |||
aksVersion=$(az aks get-versions -l $(AZURE_LOCATION) --query "max(orchestrators[?isPreview==null&&orchestratorVersion>'1.20'].orchestratorVersion)" -otsv) |
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.
1.19
is no longer supported in AKS: https://docs.microsoft.com/en-us/azure/aks/supported-kubernetes-versions?tabs=azure-cli#aks-kubernetes-release-calendar. Adding the orchestratorVersion
check here to make it more deterministic when picking the starting minor version.
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.
lgtm
Reason for Change:
Requirements
Issue Fixed:
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: