Resubmission of #50929 with fixes #51393
Conversation
openshift-ci
bot
added
the
do-not-merge/work-in-progress
|
Skipping CI for Draft Pull Request. |
|
/pj-rehearse periodic-ci-openshift-openshift-tests-private-release-4.16-amd64-nightly-aws-ipi-ovn-hypershift-private-mgmt-f7 periodic-ci-openshift-openshift-tests-private-release-4.16-amd64-nightly-aws-ipi-ovn-hypershift-private-guest-f7 periodic-ci-openshift-hypershift-release-4.15-periodics-e2e-aws-ovn-conformance periodic-ci-openshift-hypershift-release-4.16-periodics-e2e-aws-ovn-conformance-serial |
|
@fxierh: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/pj-rehearse periodic-ci-openshift-openshift-tests-private-release-4.16-amd64-nightly-aws-ipi-ovn-hypershift-private-mgmt-f7 periodic-ci-openshift-openshift-tests-private-release-4.16-amd64-nightly-aws-ipi-ovn-hypershift-private-guest-f7 |
|
@fxierh: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/pj-rehearse periodic-ci-openshift-hypershift-release-4.14-periodics-e2e-aws-ovn-proxy-conformance |
|
@fxierh: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/cc @csrwng Could you please review the sections you are responsible for in the resubmitted PR ? Thank you. |
openshift-ci
bot
requested review from
csrwng,
heliubj18,
jianlinliu and
liangxia
openshift-ci
bot
removed
the
do-not-merge/work-in-progress
|
/cc @jupierce This PR has been thoroughly rehearsed and should be okay, but could you please take a look to ensure the issues have been resolved appropriately? Thank you. |
|
One of the required job failed |
fxierh
force-pushed
the
hcp-aws-priv-resubmission
branch
from
df29ba4
to
4d286fe
Compare
|
Just a rebase. |
|
/pj-rehearse periodic-ci-openshift-hypershift-release-4.15-periodics-e2e-aws-ovn-conformance periodic-ci-openshift-hypershift-release-4.16-periodics-e2e-aws-ovn-conformance-serial For safety. |
|
@fxierh: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
Latest rehearsal results: |
...istry/cucushift/hypershift-extended/metadata/cucushift-hypershift-extended-metadata-ref.yaml
Show resolved
Hide resolved
ci-operator/step-registry/proxy/config-remove/proxy-config-remove-ref.yaml
Outdated
Show resolved
Hide resolved
fxierh
force-pushed
the
hcp-aws-priv-resubmission
branch
from
4d286fe
to
0b6319f
Compare
|
/lgtm |
...hypershift-extended/install-private/cucushift-hypershift-extended-install-private-chain.yaml
Outdated
Show resolved
Hide resolved
fxierh
force-pushed
the
hcp-aws-priv-resubmission
branch
from
0b6319f
to
ef48201
Compare
|
/lgtm |
|
/test ci-operator-registry |
|
/assign @liangxia Could you please take a look ? Thanks. |
|
/approve |
1 similar comment
|
/approve |
openshift-ci
bot
added
the
approved
|
/pj-rehearse ack |
|
@fxierh: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
openshift-ci-robot
added
the
rehearsals-ack
|
@fxierh: The following tests failed, say
Full PR test history. Your PR dashboard. 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-sigs/prow repository. I understand the commands that are listed here. |
Signed-off-by: Feilian Xie <fxie@redhat.com>
fxierh
force-pushed
the
hcp-aws-priv-resubmission
branch
from
ef48201
to
bbc50ca
Compare
|
Had to rebase due to #51645. |
openshift-ci-robot
removed
the
rehearsals-ack
|
[REHEARSALNOTIFIER]
A total of 245 jobs have been affected by this change. The above listing is non-exhaustive and limited to 25 jobs. A full list of affected jobs can be found here Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
/pj-rehearse ack |
|
@fxierh: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
openshift-ci-robot
added
the
rehearsals-ack
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fxierh, heliubj18, jianlinliu, liangxia, sjenning 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 |
#50929 was reverted by #51372 due to the fact that
HC_ZONESwas mistakenly written asHC_ZONEsin the following section:release/ci-operator/step-registry/hypershift/aws/create/hypershift-aws-create-chain.yaml
Lines 59 to 62 in 33f416a
causing
periodic-ci-openshift-hypershift-release-4.15-periodics-e2e-aws-ovn-conformanceto fail in the following way:This PR re-submits #50929 with fixes.
Rehearsal results:
periodic-ci-openshift-openshift-tests-private-release-4.16-amd64-nightly-aws-ipi-ovn-hypershift-private-mgmt-f7: ok
periodic-ci-openshift-hypershift-release-4.15-periodics-e2e-aws-ovn-conformance: ok
periodic-ci-openshift-openshift-tests-private-release-4.16-amd64-nightly-aws-ipi-ovn-hypershift-private-guest-f7: ok (step phase pre & post passed)
periodic-ci-openshift-hypershift-release-4.16-periodics-e2e-aws-ovn-conformance-serial: ok (step phase pre & post passed)
periodic-ci-openshift-hypershift-release-4.14-periodics-e2e-aws-ovn-proxy-conformance: failed due to an unrelated error (this job also failed in the same way)