OCPERT-331: Migrate Jenkins Stage pipeline to Prow jobs#76154
OCPERT-331: Migrate Jenkins Stage pipeline to Prow jobs#76154tomasdavidorg wants to merge 6 commits intoopenshift:mainfrom
Conversation
|
@tomasdavidorg: This pull request references OCPERT-331 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.22.0" version, but no target version was set. DetailsIn response to this: 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 openshift-eng/jira-lifecycle-plugin repository. |
|
Skipping CI for Draft Pull Request. |
|
/pj-rehearse periodic-ci-openshift-openshift-tests-private-release-4.21-stage-testing-e2e-aws-ipi |
|
@tomasdavidorg: 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.21-stage-testing-e2e-aws-ipi |
|
@tomasdavidorg: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
rioliu-rh
left a comment
There was a problem hiding this comment.
The build_root field is set incorrectly across all stage-testing configs — it uses a hardcoded name: release / namespace: openshift / tag: rhel-9-release-golang-1.22-openshift-4.17 instead of the release-specific builder image. Please update each config to match the corresponding main release config (see inline comment on 4.12 as an example). The correct values per release are:
| Release | tag |
|---|---|
| 4.12 | rhel-8-golang-1.19-openshift-4.12 |
| 4.13 | rhel-8-golang-1.19-openshift-4.13 |
| 4.14 | rhel-8-golang-1.20-openshift-4.14 |
| 4.15 | rhel-8-golang-1.20-openshift-4.15 |
| 4.16 | rhel-9-golang-1.21-openshift-4.16 |
| 4.17 | rhel-9-golang-1.22-openshift-4.17 |
| 4.18 | rhel-9-golang-1.22-openshift-4.18 |
| 4.19 | rhel-9-golang-1.23-openshift-4.19 |
| 4.20 | rhel-9-golang-1.24-openshift-4.20 |
| 4.21 | rhel-9-golang-1.24-openshift-4.21 |
| 4.22 | rhel-9-golang-1.24-openshift-4.22 |
All should use name: builder and namespace: ocp.
...t/openshift-tests-private/openshift-openshift-tests-private-release-4.12__stage-testing.yaml
Outdated
Show resolved
Hide resolved
rioliu-rh
left a comment
There was a problem hiding this comment.
Thanks for the PR. A few issues need to be addressed before merging — please see the inline comments for details. Summary:
build_rootshould be release-specific usingname: builder/namespace: ocpwith the correct per-release golang tag (see inline comment on 4.12 config and the table in the earlier review comment)- The workflow should be moved under the
cucushift/namespace to follow the convention used by all otheropenshift-tests-privateworkflows - The
envsection in the workflow should be simplified to onlyTEST_SCENARIOSandTEST_TIMEOUT—TEST_IMPORTANCE: allis a bug, andTEST_FILTERS/FILTERS_ADDITIONALare unnecessary - The
pre/postchains should usecucushift-installer-rehearse-aws-ipi-provisionandcucushift-installer-rehearse-aws-ipi-deprovisioninstead of the genericipi-aws-pre/ipi-aws-post, which are missing QE-specific steps and causeenable-qe-catalogsourceto run twice
...or/step-registry/release-qe-tests/stage-testing/release-qe-tests-stage-testing-workflow.yaml
Outdated
Show resolved
Hide resolved
...or/step-registry/release-qe-tests/stage-testing/release-qe-tests-stage-testing-workflow.yaml
Outdated
Show resolved
Hide resolved
...or/step-registry/release-qe-tests/stage-testing/release-qe-tests-stage-testing-workflow.yaml
Outdated
Show resolved
Hide resolved
...t/openshift-tests-private/openshift-openshift-tests-private-release-4.12__stage-testing.yaml
Outdated
Show resolved
Hide resolved
...t/openshift-tests-private/openshift-openshift-tests-private-release-4.17__stage-testing.yaml
Outdated
Show resolved
Hide resolved
...t/openshift-tests-private/openshift-openshift-tests-private-release-4.22__stage-testing.yaml
Outdated
Show resolved
Hide resolved
|
/cc @liangxia @jianlinliu |
...t/openshift-tests-private/openshift-openshift-tests-private-release-4.12__stage-testing.yaml
Show resolved
Hide resolved
...t/openshift-tests-private/openshift-openshift-tests-private-release-4.12__stage-testing.yaml
Outdated
Show resolved
Hide resolved
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: tomasdavidorg The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/pj-rehearse periodic-ci-openshift-openshift-tests-private-release-4.21-stage-testing-e2e-aws-ipi |
|
@tomasdavidorg: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
The rehearsal job is failing with: Root cause: Fix: Add
For example, 4.12–4.16 configs: tests:
- as: e2e-aws-ipi
cron: 1 * 31 2 *
steps:
cluster_profile: aws-sustaining-autorelease-412
env:
BASE_DOMAIN: sustaining-aws-412.devcluster.openshift.com
workflow: cucushift-installer-rehearse-aws-ipi-stage-testingAnd 4.17–4.22 configs: tests:
- as: e2e-aws-ipi
cron: 1 * 31 2 *
steps:
cluster_profile: aws-autorelease-qe
env:
BASE_DOMAIN: qe.devcluster.openshift.com
workflow: cucushift-installer-rehearse-aws-ipi-stage-testingPlease update all 11 configs and run |
|
/pj-rehearse periodic-ci-openshift-openshift-tests-private-release-4.21-stage-testing-e2e-aws-ipi |
|
@tomasdavidorg: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
Issue: Missing Stage CatalogSource SetupThe workflow
Proposed fix: create a new step
|
|
@tomasdavidorg It is great we are migrating the job from jenkins to prow. Have we thought of extending it to chat bot so that we can run this job at will? |
|
Hi @asood-rh, it will be part of the Do you mean any other specific chat bot? |
Issue: Conflicting CatalogSource Name in Stage Testing WorkflowThe new The Fix: The # Delete the QE catalogsource created by enable-qe-catalogsource
oc delete catalogsource qe-app-registry -n openshift-marketplace --ignore-not-found=true
oc wait --for=delete catalogsource/qe-app-registry -n openshift-marketplace --timeout=120s || true
# Then create the stage FBC catalogsource with the same name
oc create -f - <<EOF
apiVersion: operators.coreos.com/v1alpha1
kind: CatalogSource
metadata:
name: qe-app-registry
namespace: openshift-marketplace
...
EOFThis is safe because the deletion happens in |
|
@asood-rh The cluster-bot is not a good fit here. It is designed to provision ephemeral clusters on demand for developer testing/debugging, not to run full e2e workflows against a specific release payload. Stage-testing is an end-to-end workflow — it provisions a cluster, configures the stage catalog, runs the Stagerun test suite, and tears everything down. The right tools to trigger it on demand are:
Both support specifying the exact release version to test against, which is the key requirement for stage-testing. |
|
@tomasdavidorg presumbit job |
|
/test step-registry-shellcheck |
c670f1d to
e699310
Compare
|
Test passed but rather rebasing. Will take a look on the "Missing Stage CatalogSource Setup". |
|
[REHEARSALNOTIFIER]
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
@tomasdavidorg: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions 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. |
https://issues.redhat.com/browse/OCPERT-331