-
Notifications
You must be signed in to change notification settings - Fork 4.8k
OTA-1585: Register the CVO tests extension #30316
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.
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
OTA-1585: Register the CVO tests extension #30316
Conversation
|
Skipping CI for Draft Pull Request. |
|
/testwith openshift/cluster-version-operator/main/e2e-agnostic-ovn openshift/cluster-version-operator#1236 |
|
@DavidHurta: This pull request references OTA-1585 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.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. |
|
/testwith openshift/cluster-version-operator/main/e2e-agnostic-ovn openshift/cluster-version-operator#1236 openshift/cluster-version-operator#1238 |
|
/testwith openshift/cluster-version-operator/main/e2e-agnostic-ovn openshift/cluster-version-operator#1236 openshift/cluster-version-operator#1238 |
7 similar comments
|
/testwith openshift/cluster-version-operator/main/e2e-agnostic-ovn openshift/cluster-version-operator#1236 openshift/cluster-version-operator#1238 |
|
/testwith openshift/cluster-version-operator/main/e2e-agnostic-ovn openshift/cluster-version-operator#1236 openshift/cluster-version-operator#1238 |
|
/testwith openshift/cluster-version-operator/main/e2e-agnostic-ovn openshift/cluster-version-operator#1236 openshift/cluster-version-operator#1238 |
|
/testwith openshift/cluster-version-operator/main/e2e-agnostic-ovn openshift/cluster-version-operator#1236 openshift/cluster-version-operator#1238 |
|
/testwith openshift/cluster-version-operator/main/e2e-agnostic-ovn openshift/cluster-version-operator#1236 openshift/cluster-version-operator#1238 |
|
/testwith openshift/cluster-version-operator/main/e2e-agnostic-ovn openshift/cluster-version-operator#1236 openshift/cluster-version-operator#1238 |
|
/testwith openshift/cluster-version-operator/main/e2e-agnostic-ovn openshift/cluster-version-operator#1236 openshift/cluster-version-operator#1238 |
|
/testwith openshift/cluster-version-operator/main/e2e-agnostic-ovn |
|
/test ? |
|
@DavidHurta: The following commands are available to trigger required jobs: 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. |
|
/test pull-ci-openshift-origin-main-e2e-gcp-ovn |
|
@DavidHurta: The specified target(s) for 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. |
|
/test e2e-aws-ovn |
c327f8e to
281c514
Compare
|
/test all |
|
@DavidHurta: This pull request references OTA-1585 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.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. |
|
@DavidHurta: This pull request references OTA-1585 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.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. |
|
@DavidHurta: This pull request references OTA-1585 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.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. |
|
/cc @hongkailiu |
|
Risk analysis has seen new tests most likely introduced by this PR. New tests seen in this PR at sha: 281c514
|
hongkailiu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/hold
https://github.com/openshift/cluster-version-operator/pull/1249/files#diff-af72828533c5a74651f4d542a30b2552e25fdbb62be6e03ac19a573231bc2348R3 is changes test name.
Should we merge ^^^ before registration (to avoid the process of renaming)?
|
/hold |
281c514 to
0e5a1c8
Compare
|
/unhold |
|
/retest-required |
|
Risk analysis has seen new tests most likely introduced by this PR. New Test Risks for sha: 0e5a1c8
New tests seen in this PR at sha: 0e5a1c8
|
|
/lgtm |
|
/assign @xueqzhan |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: DavidHurta, hongkailiu, petr-muller 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 |
|
/verified by CI The sole test, as of the moment, in the CVO extension is discoverable and is passing in the correct jobs. |
|
@DavidHurta: 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. |
|
Risk analysis has seen new tests most likely introduced by this PR. New Test Risks for sha: 0e5a1c8
New tests seen in this PR at sha: 0e5a1c8
|
1f741ac
into
openshift:main
Register the cluster version operator (CVO) tests extension in the origin's extension registry.
Testing
Confirming
infooutputs correct API version, metadataConfirming
listoutputs tests properly filtered by environmentConfirming
run-suitefunctions and produces valid JSON test resultsConfirming
run-testfunctions and produces valid JSON test resultsChecking an e2e run for the sole sanity test:
The sole CVO test
[Jira:Cluster Version Operator] cluster-version-operator-tests should support passing testsis visible in apull-ci-openshift-origin-main-e2e-gcp-ovnrun.