search

konflux-ci / qe-tools

Contains various useful commands for processing test data
Apache License 2.0
0 stars 9 forks source link

feat: add report-portal webhook command #43

Closed tnevrlka closed 7 months ago

tnevrlka commented 8 months ago

RHTAP-1526

This PR aims to move webhook functionality from e2e-tests repo into qe-tools. Pretty much copied from here: https://github.com/redhat-appstudio/e2e-tests/blob/main/magefiles/magefile.go#L619

Tested in https://github.com/openshift/release/pull/47771 -> https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_release/47771/rehearse-47771-periodic-ci-redhat-appstudio-infra-deployments-main-appstudio-e2e-tests-periodic/1748324512455200768 PipelineRun in the appstudio-qe OpenShift namespace finished successfully

Usage: ./qe-tools webhook report-portal

psturc commented 8 months ago

is this PR ready for a review?

tnevrlka commented 8 months ago

is this PR ready for a review?

I will implement the changes requested by @Dannyb48. It's ready otherwise

sonarcloud[bot] commented 7 months ago

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

srivickynesh commented 7 months ago

@tnevrlka I see a system error reported for all passed tests. As we discussed earlier could we contain it only for the failed tests?

tnevrlka commented 7 months ago

@tnevrlka I see a system error reported for all passed tests. As we discussed earlier could we contain it only for the failed tests?

Okay. Wouldn't it be better to address it in another PR? I feel like this one is focused on webhooks, not reporting