-
Notifications
You must be signed in to change notification settings - Fork 139
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
chore: rp-preproc cleanup #1023
chore: rp-preproc cleanup #1023
Conversation
/hold |
/retest |
c3a68ad
to
514d76f
Compare
/retest |
@tnevrlka are we good? What is left to drive this forward? |
@Dannyb48 There might be changes required due to KFLUXBUGS-16. It's in a Slack thread, but the gist is that something like CustomJUnitReporter might be needed in order to have correct timestamps for all the testcases |
514d76f
to
f500f7f
Compare
Quality Gate passedIssues Measures |
@tnevrlka: The following tests failed, say
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. |
f500f7f
to
f4d48e1
Compare
Quality Gate passedIssues Measures |
/unhold |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: psturc The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Description
CustomJUnitReporter
is no longer used and was moved to https://github.com/redhat-appstudio/qe-tools/rp_preproc
got replaced bydroute
This PR aims to cleanup code previously used for Report Portal reporting that is no longer used
Issue ticket number and link
KONFLUX-797
KONFLUX-784
Type of change
How Has This Been Tested?
Locally by
make local/test/e2e
.Should be also tested here by CI and looking afterwards if the upload to Report Portal is successful.
Checklist: