OCPQE-30673: migrate some clustercatalog cases to ote#525
Conversation
openshift-ci-robot
added
the
jira/valid-reference
|
@Xia-Zhao-rh: This pull request references OCPQE-30673 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 sub-task to target the "4.21.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. |
|
please run the cases with |
|
@Xia-Zhao-rh: This pull request references OCPQE-30673 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 sub-task to target the "4.21.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. |
|
/payload-aggregate periodic-ci-openshift-release-master-ci-4.21-e2e-gcp-ovn-techpreview 5 |
|
@Xia-Zhao-rh: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/3ad72d70-aa63-11f0-88de-a85375f57e42-0 |
|
|
/hold |
openshift-ci
bot
added
the
do-not-merge/hold
|
/test verify-commits |
|
/retest verify-commits |
|
@tmshort: The The following commands are available to trigger optional jobs: Use 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 kubernetes-sigs/prow repository. |
|
Interesting... verify-commits is passing‽‽ |
|
/test verify-commits |
1 similar comment
|
/test verify-commits |
|
|
||
| }) | ||
|
|
||
| g.It("PolarionID:69123-[Skipped:Disconnected]Catalogd clustercatalog offer the operator content through http server", g.Label("LEVEL0"), func() { |
There was a problem hiding this comment.
Why do we need this "PolarionID:69123"
Those tests are not sending a signal to the Sippy
Also, I understand that they are not using the old system anymore
So, could we remove it and keep the names meaningful?
There was a problem hiding this comment.
Why are we adding g.Label("LEVEL0") ?
Are we setting this one not to run those and produce data for Sippy?
|
Hi @Xia-Zhao-rh, You can ignore my comments on this PR — I understand you’re consolidating everything under the QE directory now.
Anyway, this is probably already part of your plan — I’m just putting it into words. 😊 |
|
Hi, @camilamacedo86 |
|
/test verify-commits |
|
/test verify-commits |
|
/test verify-commits |
Xia-Zhao-rh
force-pushed
the
add_clustercatalog_cases
branch
from
5596607 to
54cedd7
Compare
|
/retest |
Xia-Zhao-rh
force-pushed
the
add_clustercatalog_cases
branch
from
54cedd7 to
8263114
Compare
|
@Xia-Zhao-rh: This PR has been marked as verified by 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. |
|
Hi, @tmshort |
|
/payload-aggregate periodic-ci-openshift-operator-framework-operator-controller-release-4.21-periodics-e2e-aws-ovn-techpreview-extended-f1 5 |
|
@Xia-Zhao-rh: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/a1a2f040-c460-11f0-9f13-b4d7d500697c-0 |
|
/retest |
|
/payload-aggregate periodic-ci-openshift-operator-framework-operator-controller-release-4.21-periodics-e2e-aws-ovn-techpreview-extended-f1 5 |
|
@Xia-Zhao-rh: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/9a9234b0-c4f5-11f0-92ac-9019be6111c0-0 |
|
/payload-aggregate periodic-ci-openshift-operator-framework-operator-controller-release-4.21-periodics-e2e-aws-ovn-techpreview-extended-f1 |
|
@Xia-Zhao-rh: it appears that you have attempted to use some version of the payload command, but your comment was incorrectly formatted and cannot be acted upon. See the docs for usage info. |
|
/payload-job periodic-ci-openshift-operator-framework-operator-controller-release-4.21-periodics-e2e-aws-ovn-techpreview-extended-f1 |
|
@Xia-Zhao-rh: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/f28bef80-c509-11f0-9f7c-11784eb3d595-0 |
Hi, @kuiwang02 Above test pass, PTAL, thanks |
openshift/tests-extension/.openshift-tests-extension/openshift_payload_olmv1.json
Show resolved
Hide resolved
openshift/tests-extension/.openshift-tests-extension/openshift_payload_olmv1.json
Show resolved
Hide resolved
openshift/tests-extension/.openshift-tests-extension/openshift_payload_olmv1.json
Show resolved
Hide resolved
openshift/tests-extension/.openshift-tests-extension/openshift_payload_olmv1.json
Outdated
Show resolved
Hide resolved
openshift/tests-extension/.openshift-tests-extension/openshift_payload_olmv1.json
Outdated
Show resolved
Hide resolved
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Xia-Zhao-rh
force-pushed
the
add_clustercatalog_cases
branch
from
d7eb5a9 to
38b65b4
Compare
openshift-ci-robot
removed
the
verified
|
/lgtm |
|
/verified by @Xia-Zhao-rh |
openshift-ci-robot
added
the
verified
|
@Xia-Zhao-rh: This PR has been marked as verified by 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. |
|
/unhold |
openshift-ci
bot
removed
the
do-not-merge/hold
|
/approve |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: tmshort, Xia-Zhao-rh The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-ci
bot
added
the
approved
7 participants
This commit migrates 15 clustercatalog-related test cases from openshift-tests-private to the OpenShift Tests Extension (OTE) framework, following the migration guidelines in test/qe/README.md.
Files Modified:
Testing:
Assisted-by: Claude Code