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.

Sign up for GitHub

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

Jump to bottom

[6.16.z]Packit testing for installed products report #16695

Closed
wants to merge 1 commit into from
Closed

[6.16.z]Packit testing for installed products report #16695

wants to merge 1 commit into from

Conversation

ColeHiggins2
Copy link
Member

Katello PR here Katello/katello#11182

@ColeHiggins2 ColeHiggins2 added No-CherryPick PR doesnt need CherryPick to previous branches Do Not Merge labels Oct 17, 2024
@ColeHiggins2 ColeHiggins2 self-assigned this Oct 17, 2024
@ColeHiggins2 ColeHiggins2 requested a review from a team as a code owner October 17, 2024 13:02
@ColeHiggins2
Copy link
Member Author

trigger: test-robottelo
pytest: tests/foreman/api/test_reporttemplates.py -k test_positive_installed_products
Katello:
katello: 11182

Gauravtalreja1
Gauravtalreja1 reviewed Oct 17, 2024
View reviewed changes
tests/foreman/api/test_reporttemplates.py
@@ -948,6 +948,7 @@ def test_positive_installed_products(
'hosts': rhel_contenthost.hostname,
},
}
# packit testing for this line here
Copy link
Collaborator

Choose a reason for hiding this comment

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

Just for awareness, If you would like to run an upstream PRT, the PRT comment can be added directly to the Jenkins job while build with parameters, without requiring robottelo PRs, and you can also post results on any GH PRs where you wish, such as here on upstream Katello PR

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8992
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_reporttemplates.py -k test_positive_installed_products --external-logging
Test Result : ========= 3 failed, 38 deselected, 180 warnings in 1592.06s (0:26:32) ==========

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Oct 17, 2024
@ColeHiggins2
Copy link
Member Author

Closing card.

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

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 left review comments

Assignees

@ColeHiggins2 ColeHiggins2

Labels
Do Not Merge No-CherryPick PR doesnt need CherryPick to previous branches PRT-Failed Indicates that latest PRT run is failed for the PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ColeHiggins2 @Satellite-QE @Gauravtalreja1