Skip to content

Conversation

tpapaioa
Copy link
Contributor

@tpapaioa tpapaioa commented Oct 15, 2025

Problem Statement

The module_satellite_iop fixture should use the new deploy-satellite-iop AAP workflow

Solution

Update the fixture to use the workflow defined in the server.deploy_workflows.iop setting.

Related Issues

@tpapaioa tpapaioa self-assigned this Oct 15, 2025
@tpapaioa tpapaioa added CherryPick PR needs CherryPick to previous branches 6.18.z Introduced in or relating directly to Satellite 6.18 labels Oct 15, 2025
@tpapaioa tpapaioa force-pushed the deploy_satellite_iop branch from a88f483 to 8d37b92 Compare October 16, 2025 18:03
@tpapaioa tpapaioa force-pushed the deploy_satellite_iop branch from 8d37b92 to ebfb0e3 Compare October 19, 2025 18:45
@tpapaioa
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_rhcloud_iop.py::test_iop_recommendations_e2e[rhel9-ipv4-local]
env:
  ROBOTTELO_SERVER__DEPLOY_WORKFLOWS__IOP: deploy-satellite-iop

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 13237
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_rhcloud_iop.py::test_iop_recommendations_e2e[rhel9-ipv4-local] --external-logging
Test Result : ================= 1 passed, 15 warnings in 2175.71s (0:36:15) ==================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Oct 20, 2025
@tpapaioa tpapaioa marked this pull request as ready for review October 20, 2025 16:13
@tpapaioa tpapaioa requested a review from a team as a code owner October 20, 2025 16:13
Copy link

@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 - here's some feedback:

  • Wrap your teardown and Broker.checkin calls in a finalizer (or try/finally) so cleanup always runs even if the IoP assertion or a downstream error fails.
  • Consider moving the IoP‐enabled assertion out of the fixture and into the tests themselves to keep fixtures focused on setup/teardown.
  • Verify that any tests relying on the old module_unconfigured_satellite fixture are updated to use the new satellite_factory signature to prevent fixture resolution errors.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- Wrap your teardown and Broker.checkin calls in a finalizer (or try/finally) so cleanup always runs even if the IoP assertion or a downstream error fails.
- Consider moving the IoP‐enabled assertion out of the fixture and into the tests themselves to keep fixtures focused on setup/teardown.
- Verify that any tests relying on the old module_unconfigured_satellite fixture are updated to use the new satellite_factory signature to prevent fixture resolution errors.

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.

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

Labels

6.18.z Introduced in or relating directly to Satellite 6.18 CherryPick PR needs CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants