-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[azure] support new marketpalce image 413.92.2023101700 #44693
Conversation
@jinyunma do we have a conclusion on if 4.13 should also use the new published "413.92.2023101700"? Or we just want to guarantee 4.14 is working fine currently |
offer: ${imageInfo[1]} | ||
sku: ${imageInfo[2]} | ||
version: ${imageInfo[3]} | ||
${os_image_plan} |
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 need to set os_image_plan
with an empty value in previous lines when OS_IMAGE_PLAN
is empty?
And when OS_IMAGE_PLAN
is set to WithPurchasePlan
, we also assemble os_image_plan
, right?
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.
yes, fixed.
not yet, didn't get response from EXDSP- 2164.
yes, since manual testing is passed, so I plan to add it into CI, if there is any change for image version published in azure marketplace, I think we can just update ENV |
56e4161
to
3cee214
Compare
|
||
#set plan for os image, default value is WithPurchasePlan | ||
os_image_plan="" | ||
if [[ "${OS_IMAGE_PLAN}" != "" ]] && [[ ${ocp_minor_version} -ge 14 ]]; then |
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.
I guess we do not need to judge ocp_minor_version
here, if someone set OS_IMAGE_PLAN on an unsupported version, that should be a job config issue.
TAG_VERSION: '@4.15' | ||
test: | ||
- chain: openshift-e2e-test-qe-destructive | ||
workflow: cucushift-installer-rehearse-azure-ipi-marketplace | ||
- as: azure-ipi-marketplace-noplan-f28 |
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.
How about merge the coverage into the above destructive ci? non-destructive cover 413.92.2023101700
, destructive one covers azureopenshift:aro4:413_v2:413.92.20230614
?
8fbd189
to
580c1b7
Compare
steps: | ||
cluster_profile: azure-qe | ||
env: | ||
BASE_DOMAIN: qe.azure.devcluster.openshift.com | ||
E2E_RUN_TAGS: '@amd64 and @azure-ipi and @network-ovnkubernetes and not @fips' | ||
OS_IMAGE_MASTERS: azureopenshift:aro4:413_v2:413.92.20230614 | ||
OS_IMAGE_MASTERS_PLAN: NoPurchasePlan | ||
OS_IMAGE_VERSION: 413.92.2023101700 |
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 cover azureopenshift:aro4:413_v2:413.92.20230614
on this job, right?
OS_IMAGE_MASTERS_PLAN: NoPurchasePlan | ||
OS_IMAGE_PLAN: NoPurchasePlan | ||
OS_IMAGE_URN: azureopenshift:aro4:413_v2:413.92.20230614 | ||
OS_IMAGE_VERSION: 413.92.2023101700 |
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.
From the codes, seem like we do not need OS_IMAGE_VERSION
setting here?
/lgtm |
/pj-rehearse periodic-ci-openshift-openshift-tests-private-release-4.14-amd64-nightly-azure-ipi-marketplace-f28 |
/pj-rehearse periodic-ci-openshift-openshift-tests-private-release-4.14-amd64-nightly-azure-ipi-marketplace-f28 |
/pj-rehearse periodic-ci-openshift-openshift-tests-private-release-4.14-amd64-nightly-azure-ipi-marketplace-f28 |
/pj-rehearse periodic-ci-openshift-openshift-tests-private-release-4.14-amd64-nightly-azure-ipi-marketplace-noplan-f28-destructive periodic-ci-openshift-openshift-tests-private-release-4.13-amd64-nightly-azure-ipi-workers-marketplace-f28 |
[REHEARSALNOTIFIER]
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
/lgtm |
/pj-rehearse periodic-ci-openshift-openshift-tests-private-release-4.14-amd64-nightly-azure-ipi-marketplace-f28 |
/pj-rehearse ack cc @liangxia to review |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jianlinliu, jinyunma, liangxia The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
https://issues.redhat.com/browse/CORS-2725
cucushift-installer-rehearse-azure-ipi-marketplace
to test image413.92.2023101700
to provision all nodes on 4.14+.cucushift-installer-check-azure-osimage