Skip to content

Add Arm64 ipi on Azure profiles for QE #33149

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.

Sign up for GitHub

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

Jump to bottom
Merged
openshift-merge-robot merged 1 commit into from
Oct 20, 2022
Merged

Add Arm64 ipi on Azure profiles for QE #33149

openshift-merge-robot merged 1 commit into from
Oct 20, 2022

Conversation

@lwan-wanglin
Copy link
Contributor

@lwan-wanglin lwan-wanglin commented Oct 17, 2022 •
edited
Loading

Add Arm64 ipi on Azure profiles for QE. refer ARMOCP-221 ARMOCP-222

  • Add avaliablity zones obtainment becasue not all the zones have arm64 type of machines.

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 17, 2022
@lwan-wanglin lwan-wanglin force-pushed the arm-azure branch 2 times, most recently from 668ec20 to ea949ad Compare October 17, 2022 08:46
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 17, 2022
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 17, 2022
@lwan-wanglin lwan-wanglin force-pushed the arm-azure branch 2 times, most recently from 8a68855 to af50c86 Compare October 18, 2022 03:53
@lwan-wanglin lwan-wanglin changed the title [WIP]Add Arm64 ipi on Azure profiles for QE Add Arm64 ipi on Azure profiles for QE Oct 18, 2022
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 18, 2022
@lwan-wanglin
Copy link
Contributor Author

lwan-wanglin commented Oct 18, 2022

/cc @aleskandro @jianlinliu @MayXuQQ @yunjiang29

aleskandro
aleskandro reviewed Oct 18, 2022
View reviewed changes
@lwan-wanglin lwan-wanglin force-pushed the arm-azure branch 2 times, most recently from 3b13102 to 470e66e Compare October 18, 2022 13:14
@aleskandro
Copy link
Member

aleskandro commented Oct 18, 2022

cc @deepsm007

@aleskandro
Copy link
Member

aleskandro commented Oct 18, 2022

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 18, 2022
@deepsm007
Copy link
Contributor

deepsm007 commented Oct 18, 2022

/assign @vrutkovs

vrutkovs
vrutkovs reviewed Oct 18, 2022
View reviewed changes
aleskandro
aleskandro reviewed Oct 18, 2022
View reviewed changes
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Oct 19, 2022
@deepsm007 deepsm007 mentioned this pull request Oct 19, 2022
Merged
@lwan-wanglin
Copy link
Contributor Author

lwan-wanglin commented Oct 19, 2022

@aleskandro @vrutkovs @jianlinliu, most jobs installation part are success, please help to review the pr again, thanks!

vrutkovs
vrutkovs approved these changes Oct 19, 2022
View reviewed changes
Copy link
Contributor

@vrutkovs vrutkovs left a comment

Choose a reason for hiding this comment

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

/lgtm

@vrutkovs
Copy link
Contributor

vrutkovs commented Oct 19, 2022

/hold

for others to review

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 19, 2022
@aleskandro
Copy link
Member

aleskandro commented Oct 19, 2022

/lgtm

Thanks @lwan-wanglin

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 19, 2022
@jianlinliu
Copy link
Contributor

jianlinliu commented Oct 20, 2022

/lgtm

@lwan-wanglin
Copy link
Contributor Author

lwan-wanglin commented Oct 20, 2022

/assign @kuiwang02
Could you help merge it? thanks

@pruan-rht
Copy link
Member

pruan-rht commented Oct 20, 2022

/lgtm
/approve

@lwan-wanglin
Copy link
Contributor Author

lwan-wanglin commented Oct 20, 2022

/unhold

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 20, 2022
@kuiwang02
Copy link
Contributor

kuiwang02 commented Oct 20, 2022

i am OK from golang's view.
@lwan-wanglin please request qe produtivity team to approve it.

@MayXuQQ
Copy link
Contributor

MayXuQQ commented Oct 20, 2022

/retest-required

@lwan-wanglin
Copy link
Contributor Author

lwan-wanglin commented Oct 20, 2022

/assign @jhou1 @liangxia
Could you help approve it when your are avaiable? thanks

@jhou1
Copy link
Member

jhou1 commented Oct 20, 2022

/lgtm

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 20, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aleskandro, jhou1, jianlinliu, lwan-wanglin, pruan-rht, vrutkovs

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 20, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 20, 2022

@lwan-wanglin: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/rehearse/periodic-ci-kata-containers-kata-containers-main-4.11-e2e-tests c798dcac71c4e90ec63c276aab25c6b78562f1bb link unknown /test pj-rehearse
ci/rehearse/openshift/openshift-controller-manager/master/e2e-gcp-builds 8a68855e42ed815a1b63ca3a5f2c86ec911f567f link unknown /test pj-rehearse
ci/rehearse/operator-framework/operator-marketplace/master/e2e-gcp-serial 8a68855e42ed815a1b63ca3a5f2c86ec911f567f link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-kube-apiserver-operator/master/e2e-aws-single-node 668ec20015b0d37fcaf08f7a6d0742bf8b760aa3 link unknown /test pj-rehearse
ci/rehearse/red-hat-storage/odf-console/master/odf-console-e2e-aws 8a68855e42ed815a1b63ca3a5f2c86ec911f567f link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-kube-apiserver-operator/master/e2e-aws-ovn 668ec20015b0d37fcaf08f7a6d0742bf8b760aa3 link unknown /test pj-rehearse
ci/rehearse/openshift/sdn/master/e2e-aws-sdn-serial af50c86de6ee4d68ca3d34e77f231251b6ac0c78 link unknown /test pj-rehearse
ci/rehearse/periodic-ci-che-incubator-che-test-harness-master-performance-tests af50c86de6ee4d68ca3d34e77f231251b6ac0c78 link unknown /test pj-rehearse
ci/rehearse/kubevirt/hyperconverged-cluster-operator/main/hco-e2e-image-index-sno-azure 470e66ef6d1d92a1a32e307947e209c16f23600c link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-network-operator/master/e2e-azure-ovn-dualstack 98d188ecf53b88169bb4706c2200c400bef9d419 link unknown /test pj-rehearse
ci/rehearse/openshift-cnv/cnv-ci/master/4.10-e2e-deploy 98d188ecf53b88169bb4706c2200c400bef9d419 link unknown /test pj-rehearse
ci/rehearse/openshift/machine-config-operator/master/e2e-aws-ovn-workers-rhel8 9864c48 link unknown /test pj-rehearse
ci/rehearse/periodic-ci-openshift-hypershift-main-periodics-conformance-proxy-aws-ovn-4-12 9864c48 link unknown /test pj-rehearse
ci/rehearse/openshift/machine-api-provider-ibmcloud/main/e2e-ibmcloud 9864c48 link unknown /test pj-rehearse
ci/rehearse/periodic-ci-openshift-openshift-tests-private-release-4.12-arm64-nightly-e2e-azure-ipi-sdn-p3 9864c48 link unknown /test pj-rehearse
ci/rehearse/periodic-ci-openshift-openshift-tests-private-release-4.12-arm64-nightly-e2e-azure-ipi-ovn-p1 9864c48 link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-kube-apiserver-operator/master/e2e-gcp-operator-single-node 9864c48 link unknown /test pj-rehearse
ci/rehearse/periodic-ci-openshift-hypershift-main-periodics-e2e-conformance-kubevirt 9864c48 link unknown /test pj-rehearse
ci/prow/pj-rehearse 9864c48 link false /test pj-rehearse
ci/rehearse/periodic-ci-openshift-hypershift-main-periodics-conformance-azure-ovn-4-12 9864c48 link unknown /test pj-rehearse

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@openshift-merge-robot openshift-merge-robot merged commit 680b677 into openshift:master Oct 20, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 20, 2022

@lwan-wanglin: Updated the following 2 configmaps:

  • ci-operator-4.12-configs configmap in namespace ci at cluster app.ci using the following files:
    • key openshift-openshift-tests-private-release-4.12__arm64-nightly.yaml using file ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.12__arm64-nightly.yaml
  • job-config-4.12 configmap in namespace ci at cluster app.ci using the following files:
    • key openshift-openshift-tests-private-release-4.12-periodics.yaml using file ci-operator/jobs/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.12-periodics.yaml
Details

In response to this:

Add Arm64 ipi on Azure profiles for QE. refer ARMOCP-221 ARMOCP-222

  • Add avaliablity zones obtainment becasue not all the zones have arm64 type of machines.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@lwan-wanglin lwan-wanglin deleted the arm-azure branch June 3, 2025 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bandrade bandrade Awaiting requested review from bandrade

@jianzhangbjz jianzhangbjz Awaiting requested review from jianzhangbjz

@MayXuQQ MayXuQQ Awaiting requested review from MayXuQQ

@yunjiang29 yunjiang29 Awaiting requested review from yunjiang29

3 more reviewers

@jianlinliu jianlinliu jianlinliu left review comments

@aleskandro aleskandro aleskandro left review comments

@vrutkovs vrutkovs vrutkovs approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@vrutkovs vrutkovs

@pruan-rht pruan-rht

@jianlinliu jianlinliu

@jhou1 jhou1

@liangxia liangxia

@aleskandro aleskandro

@kuiwang02 kuiwang02

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.