Skip to content

Conversation

@openshift-cherrypick-robot

This is an automated cherry-pick of #27486

/assign dgoodwin

This will be used to submit to a new sippy risk analysis API to
determine how risky the test failures in this job run are. As with junit
xmls, there can be multiple as we often invoke openshift-tests multiple
times in one job run.

Includes a change to make timestamp suffixes on junit artifacts more
consistent.
… and upload.

This merges test-flake files, submits them to sippy, and writes the
returned risk analysis as an additional artifact. It is intended to be
called from the workflow steps that run openshift-tests in the release
repo.
@dgoodwin
Copy link
Contributor

dgoodwin commented Nov 3, 2022

From TRT-652

@openshift-ci openshift-ci bot requested review from pweil- and spadgett November 3, 2022 12:23
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 3, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: openshift-cherrypick-robot
Once this PR has been reviewed and has the lgtm label, please assign mfojtik for approval by writing /assign @mfojtik in a comment. For more information see the Kubernetes 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

@dgoodwin
Copy link
Contributor

dgoodwin commented Nov 3, 2022

/hold

Concerns this will complicate the origin 1.25 rebase work, causing conflicts etc. We will revisit once that lands, or just leave this as a 4.13 feature.

@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 Nov 3, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 4, 2022

@openshift-cherrypick-robot: The following tests 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/e2e-aws-ovn-cgroupsv2 41b675f link false /test e2e-aws-ovn-cgroupsv2
ci/prow/e2e-aws-ovn-single-node-upgrade 41b675f link false /test e2e-aws-ovn-single-node-upgrade
ci/prow/e2e-gcp-ovn-rt-upgrade 41b675f link false /test e2e-gcp-ovn-rt-upgrade
ci/prow/e2e-agnostic-ovn-cmd 41b675f link false /test e2e-agnostic-ovn-cmd
ci/prow/e2e-gcp-ovn-upgrade 41b675f link true /test e2e-gcp-ovn-upgrade
ci/prow/e2e-openstack-ovn 41b675f link false /test e2e-openstack-ovn
ci/prow/e2e-gcp-ovn-image-ecosystem 41b675f link true /test e2e-gcp-ovn-image-ecosystem

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

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 10, 2022
@openshift-merge-robot
Copy link
Contributor

@openshift-cherrypick-robot: PR needs rebase.

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/test-infra repository.

@dgoodwin
Copy link
Contributor

/close

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 16, 2022

@dgoodwin: Closed this PR.

Details

In response to this:

/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/test-infra repository.

@openshift-ci openshift-ci bot closed this Nov 16, 2022
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. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants