Skip to content
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

Reportportal should note guest hostname for investigation #3209

Merged
merged 2 commits into from
Sep 20, 2024

Conversation

happz
Copy link
Collaborator

@happz happz commented Sep 12, 2024

It may happen the guest name or role are not set, yet there are multiple guests reported into the same launch, when reporting multiple tmt runs into the same RP launch, e.g. when those runs run the same tests but on different hosts. This is a common case for those running tests on multiple architectures, e.g. when testing a Koji build.

Pull Request Checklist

  • implement the feature
  • extend the test coverage
  • include a release note

@happz happz added the plugin | reportportal The reportportal report plugin label Sep 12, 2024
@happz happz added this to the 1.37 milestone Sep 12, 2024
@happz
Copy link
Collaborator Author

happz commented Sep 12, 2024

@kkaarreell @4N0body5 what do you think about this addition?

@kkaarreell
Copy link
Collaborator

@kkaarreell @4N0body5 what do you think about this addition?

Ack

@happz happz added the ci | full test Pull request is ready for the full test execution label Sep 16, 2024
@happz happz force-pushed the reportportal-record-guest-hostname branch from da338e9 to ef86b4c Compare September 16, 2024 07:18
@happz happz force-pushed the reportportal-record-guest-hostname branch from ef86b4c to 0985617 Compare September 19, 2024 15:09
@happz happz added the status | ready for merge The only missing piece is to do the rebase the current 'main' and let the CI finish. label Sep 20, 2024
It may happen the guest name or role are not set, yet there are multiple
guests reported into the same launch, when reporting multiple tmt runs
into the same RP launch, e.g. when those runs run the same tests but on
different hosts. This is a common case for those running tests on
multiple architectures, e.g. when testing a Koji build.
@happz happz force-pushed the reportportal-record-guest-hostname branch from 0985617 to fa084bd Compare September 20, 2024 06:48
@happz
Copy link
Collaborator Author

happz commented Sep 20, 2024

Unrelated failures, merging.

@happz happz merged commit 1025ff6 into main Sep 20, 2024
21 of 23 checks passed
@happz happz deleted the reportportal-record-guest-hostname branch September 20, 2024 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci | full test Pull request is ready for the full test execution plugin | reportportal The reportportal report plugin status | ready for merge The only missing piece is to do the rebase the current 'main' and let the CI finish.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants