Skip to content

Conversation

@shubhamsg199
Copy link
Contributor

Problem Statement
test_positive_all_options is failing as containerfile-install-command is added for hammer host package
related Katello PR: Katello/hammer-cli-katello#1017

Solution
Update hammer_commands.json

@shubhamsg199 shubhamsg199 self-assigned this Dec 19, 2025
@shubhamsg199 shubhamsg199 requested a review from a team as a code owner December 19, 2025 11:26
@shubhamsg199 shubhamsg199 added No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Dec 19, 2025
@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_hammer.py -k test_positive_all_options

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey there - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 13924
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_hammer.py -k test_positive_all_options --external-logging
Test Result : =========== 1 passed, 4 deselected, 5 warnings in 814.27s (0:13:34) ============

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Dec 19, 2025
@Gauravtalreja1 Gauravtalreja1 merged commit fb9d078 into SatelliteQE:master Dec 19, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR Stream Introduced in or relating directly to Satellite Stream/Master

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants