Skip to content

Conversation

@gavin-stackrox
Copy link
Contributor

@gavin-stackrox gavin-stackrox commented Jun 1, 2023

Description

Per title. By wrapping these failures with a junit report we get jira tracking via junit2jira. The offending log is also copied to the top level of openshift/release Artifacts to make it easier to examine.

Checklist

  • Investigated and inspected CI test results

Testing Performed

CI is sufficient.

  • with pod restart due to error during test (e.g. kubectl -n stackrox exec sensor-69d57d6467-vj9ks -- kill -ABRT 1)
    • the failed check appears in the JUnit panel
    • failure stands out in the prow UI (build log panel)
  • without
    • passing check can be found in the JUnit panel

@ghost
Copy link

ghost commented Jun 1, 2023

Images are ready for the commit at 3cbefea.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.0.x-495-g3cbefea077.

@gavin-stackrox gavin-stackrox changed the title ROX-16321: junitify the check for pod restarts ROX-16321: Junitify the check for pod restarts Jun 1, 2023
@gavin-stackrox gavin-stackrox requested a review from davdhacs June 2, 2023 15:59
@openshift-ci
Copy link

openshift-ci bot commented Jun 2, 2023

@gavin-stackrox: 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/ocp-4-10-qa-e2e-tests 3cbefea link false /test ocp-4-10-qa-e2e-tests

Full PR test history. Your PR dashboard.

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.

Copy link
Contributor

@davdhacs davdhacs left a comment

Choose a reason for hiding this comment

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

/lgtm

@gavin-stackrox gavin-stackrox merged commit 6e68d87 into master Jun 2, 2023
@gavin-stackrox gavin-stackrox deleted the gavin/ROX-16321/junitify-check-for-pod-restarts branch June 2, 2023 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants