Skip to content

OTA-1478 add 4.20 ota jobs - #65512

Closed
dis016 wants to merge 594 commits into
openshift:masterfrom
dis016:OTA-1478_add_ota_jobs_4.20
Closed

OTA-1478 add 4.20 ota jobs#65512
dis016 wants to merge 594 commits into
openshift:masterfrom
dis016:OTA-1478_add_ota_jobs_4.20

Conversation

@dis016

@dis016 dis016 commented May 30, 2025

Copy link
Copy Markdown
Contributor

OTA-1478 add 4.20 ota jobs

/cc @JianLi-RH @jiajliu @jhou1

@openshift-ci
openshift-ci Bot requested review from JianLi-RH, jhou1 and jiajliu May 30, 2025 09:22
@dis016

dis016 commented May 30, 2025

Copy link
Copy Markdown
Contributor Author

/hold for review and rehearse jobs

@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 May 30, 2025
release:
architecture: multi
channel: candidate
version: "4.21"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I guess this will break 4.20 ci tests in the job config, could you trigger some tests under openshift-verification-tests-main__ota-multi-stable-4.20-upgrade-from-stable-4.19.yaml to have a try?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@dis016 what are your follow up actions to address this?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@dis016 I think you can create a new JIRA card to track the work which requires OCP 4.21, and implement jobs only when there are OCP 4.21 payloads.

For this PR, you can just remove this version.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@jhou1 as @jiajliu suspected there is a problem in generating the release images for the jobs, i will work on it and correct the parameters to fix the same.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The issue was introduced by #58136. Dropping it when enable 4.20 test and then add it back someday sounds a workaround. But the issue will come back again when enable 4.21/4.22.. test.
Personally I think it will bring the team endless maintenance work. cc @JianLi-RH to consider fixing the test from code level. Actually 4.21 is not an upgrade target for path 4.20.0 -> 4.20.latest -> 4.21, which is not required to be configured in job level. We should rely on a long-term resolution to reduce team's maintenance work.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think we can monitor this for some releases, maybe 4.21, then disable this job.

@dis016

dis016 commented Jun 4, 2025

Copy link
Copy Markdown
Contributor Author

/pj-rehearse periodic-ci-openshift-verification-tests-main-ota-multi-stable-4.20-upgrade-from-stable-4.19-azure-ipi-amd-mixarch-f28

@openshift-ci-robot

Copy link
Copy Markdown
Contributor

@dis016: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@dis016

dis016 commented Jun 16, 2025

Copy link
Copy Markdown
Contributor Author

/pj-rehearse periodic-ci-openshift-verification-tests-main-ota-multi-stable-4.20-upgrade-from-stable-4.19-azure-ipi-amd-mixarch-f28

@openshift-ci-robot

Copy link
Copy Markdown
Contributor

@dis016: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@dis016

dis016 commented Jun 16, 2025

Copy link
Copy Markdown
Contributor Author

/pj-rehearse periodic-ci-openshift-verification-tests-main-ota-multi-stable-4.20-upgrade-from-stable-4.19-azure-ipi-amd-mixarch-f28

@openshift-ci-robot

Copy link
Copy Markdown
Contributor

@dis016: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@dis016

dis016 commented Jun 17, 2025

Copy link
Copy Markdown
Contributor Author

/pj-rehearse periodic-ci-openshift-verification-tests-main-ota-multi-stable-4.20-upgrade-from-stable-4.19-azure-ipi-amd-mixarch-f28

@openshift-ci-robot

Copy link
Copy Markdown
Contributor

@dis016: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

subhashkhileri and others added 16 commits June 27, 2025 11:14
* rhdh-test-instance time update

* rhdh-test-instance time update

* rhdh-test-instance time update

* rhdh-test-instance time update
Co-authored-by: Yang Yang <yanyang@yanyang-thinkpadx1carbongen11.raycom.csb>
…hift#66412)

Add more debugging information to verbose DHCP output.
* initial prow onboarding

Signed-off-by: sahare <sahare@redhat.com>

* make update successful

Signed-off-by: sahare <sahare@redhat.com>

* add sonar post submit

Signed-off-by: sahare <sahare@redhat.com>

* customize SONAR_GO_TEST_ARGS to exclude test/e2e

Signed-off-by: sahare <sahare@redhat.com>

* script syntax error during command evaluation

Signed-off-by: sahare <sahare@redhat.com>

* modify commands blocks for both sonar and sonar-post-submit

Signed-off-by: sahare <sahare@redhat.com>

* temp change to debug

Signed-off-by: sahare <sahare@redhat.com>

* trying to fix the command to exclude e2e

Signed-off-by: sahare <sahare@redhat.com>

* setting the user.home system property for the JVM

Signed-off-by: sahare <sahare@redhat.com>

---------

Signed-off-by: sahare <sahare@redhat.com>
Signed-off-by: Todd Short <todd.short@me.com>
Co-authored-by: Anna Rania <asergien@asergien-thinkpadp1gen3.bos.csb>
Signed-off-by: Prashant Dhamdhere <prdhamdh@redhat.com>
Signed-off-by: Jose Castillo Lema <josecastillolema@gmail.com>
* Add CNV_SUBSCRIPTION_CHANNEL overwrite to cnv-tests step

Signed-off-by: Rabin Yasharzadehe <rabin@redhat.com>

* Set the right openshift-virtualization-tests image tag to use for the tests

Rename the 4.19 job name to use the master branch.
And we can use master from now on.

Signed-off-by: Rabin Yasharzadehe <rabin@redhat.com>

* Fix openshift-virtualization-tests step

It was never really running since we moved to the open-source version.

Signed-off-by: Rabin Yasharzadehe <rabin@redhat.com>

---------

Signed-off-by: Rabin Yasharzadehe <rabin@redhat.com>
* Release image in jetlag deployments!

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

* Test Jetlag PR

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

* Update pull-secret path

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

* Release info

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

* Release info

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

* Restore 4.19 job to avoid labeling required

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

* Update ping

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

* Registry login

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

* Scp prow's pull_secret

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

* Update configuration files

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

* Update jobs config

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

* Remove -dev jobs from jetlag ci

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

* Jetlag PR 655

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

* Replace env var by payload_url variable

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

* Undo manual PR pinning

* Jetlag PR 663

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

* Revert "Jetlag PR 663"

This reverts commit abf625a.

* Bump jetlag branch

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

---------

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>
Co-authored-by: José Castillo Lema <josecastillolema@gmail.com>
Signed-off-by: Adam D. Cornett <adc@redhat.com>
* Rename hypershift-e2e-azure to aks

This commit renames the informing test from hypershift-e2e-azure to hypershift-e2e-aks. This test is for managed Azure on HyperShift which uses AKS mgmt clusters. In the near future, we will support self-managed Azure which will need its own tests. We need to clean up the current naming so it's not confusing which test is for which Azure type on HyperShift.

* Remove Patryk-Stefanski from HyperShift approvers and reviewers
Co-authored-by: Anna Rania <asergien@asergien-thinkpadp1gen3.bos.csb>
openshift-bot and others added 13 commits July 4, 2025 04:03
….20 --skip-periodics --future-release 4.21 --confirm
…openshift-priv --only-org openshift --whitelist-file ./core-services/openshift-priv/_whitelist.yaml
…ncher

Automate config brancher by auto-config-brancher job at Fri, 04 Jul 2025 04:05:46 UTC
- Update maven binary versio to 3.9.9

Signed-off-by: Prashant Dhamdhere <prdhamdh@redhat.com>
…oint http://ghproxy --config-dir ./ci-operator/config --registry ./ci-operator/step-registry --prune-unused-replacements --prune-ocp-builder-replacements --prune-unused-base-images --current-release-minor=20 --ensure-correct-promotion-dockerfile-ignored-repos openshift/origin-aggregated-logging --ensure-correct-promotion-dockerfile-ignored-repos openshift/console --ensure-correct-promotion-dockerfile-ignored-repos openshift/linuxptp-daemon
….20 --skip-periodics --future-release 4.21 --confirm
…openshift-priv --only-org openshift --whitelist-file ./core-services/openshift-priv/_whitelist.yaml
…ncher

Automate config brancher by auto-config-brancher job at Fri, 04 Jul 2025 08:06:10 UTC
@dis016

dis016 commented Jul 4, 2025

Copy link
Copy Markdown
Contributor Author

/pj-rehearse periodic-ci-openshift-verification-tests-main-ota-multi-stable-4.20-upgrade-from-stable-4.19-azure-ipi-amd-mixarch-f28

@openshift-ci-robot

Copy link
Copy Markdown
Contributor

@dis016: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@dis016

dis016 commented Jul 4, 2025

Copy link
Copy Markdown
Contributor Author

PR is corrupted with rebase, i will reopen a new PR
/close

@openshift-ci openshift-ci Bot closed this Jul 4, 2025
@openshift-ci

openshift-ci Bot commented Jul 4, 2025

Copy link
Copy Markdown
Contributor

@dis016: Closed this PR.

Details

In response to this:

PR is corrupted with rebase, i will reopen a new PR
/close

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.

@openshift-ci-robot openshift-ci-robot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Jul 4, 2025
@openshift-ci-robot

Copy link
Copy Markdown
Contributor

[REHEARSALNOTIFIER]
@dis016: no rehearsable tests are affected by this change

Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

@openshift-ci

openshift-ci Bot commented Jul 4, 2025

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: dis016
Once this PR has been reviewed and has the lgtm label, please assign danilo-gemoli for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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 commented Jul 4, 2025

Copy link
Copy Markdown
Contributor

@dis016: The following test 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/prow/generated-config 9ef8123 link true /test generated-config

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-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. rehearsals-ack Signifies that rehearsal jobs have been acknowledged

Projects

None yet

Development

Successfully merging this pull request may close these issues.