Skip to content

Support fetching results from ReportPortal#1

Open
therazix wants to merge 1 commit into
mainfrom
fvagner-reportportal
Open

Support fetching results from ReportPortal#1
therazix wants to merge 1 commit into
mainfrom
fvagner-reportportal

Conversation

@therazix

@therazix therazix commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

Adds support for loading test results from ReportPortal. When --use-reportportal is passed to filter-tests, results are fetched from all reportportal report phases in the recipe instead of local results.yaml files. Plans without a reportportal report phase use the local results.yaml file as usual. The launch-uuid and related fields are stripped from the output recipe so that a subsequent rerun creates a new launch.

Resolves teemtee/tmt#4828

@psss psss added this to planning Jun 23, 2026
@github-project-automation github-project-automation Bot moved this to backlog in planning Jun 23, 2026
@psss psss moved this from backlog to review in planning Jun 23, 2026
@therazix therazix force-pushed the fvagner-reportportal branch from 70c96e8 to b6ee245 Compare June 24, 2026 08:24
@happz happz added the enhancement New feature or request label Jun 24, 2026
TMT_TO_RP_RESULT_STATUS = {
ResultOutcome.PASS.value: "PASSED",
ResultOutcome.FAIL.value: "FAILED",
ResultOutcome.INFO.value: "SKIPPED",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Info -> skipped looks weird, but I don't have the background on this

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I agree, but this is how tmt maps the result status to ReportPortal because ReportPortal does not have info or pending status. I kept it the same for consistency.

https://github.com/teemtee/tmt/blob/bbe0e5f65a0b0352dd89e6fd0decd1a102bbc014/tmt/steps/report/reportportal.py#L528-L536

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hmm, why don't you re-use that mapping? tmt is available that you can extract it right?

ResultOutcome.WARN.value: "FAILED",
ResultOutcome.ERROR.value: "FAILED",
ResultOutcome.SKIP.value: "SKIPPED",
ResultOutcome.PENDING.value: "SKIPPED",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

If the test is still pending, there's probably something wrong?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Same as #1 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: review

Development

Successfully merging this pull request may close these issues.

Add ReportPortal support to the recipe re-execution script

4 participants