Red Hat Satellite engineering is moving the tracking of its product development work on Satellite to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "Satellite project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs will be migrated starting at the end of May. If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "Satellite project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/SAT-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.
Bug 2136231 - Fix ansible-collection-redhat-satellite requires of requests and pyyaml
Summary: Fix ansible-collection-redhat-satellite requires of requests and pyyaml
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Ansible Collection
Version: 6.12.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: 6.12.4
Assignee: satellite6-bugs
QA Contact: Satellite QE Team
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-10-19 16:53 UTC by Lukas Pramuk
Modified: 2023-04-27 14:30 UTC (History)
3 users (show)

Fixed In Version: ansible-collection-redhat-satellite-3.7.0-4
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2023-04-27 14:30:36 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker SAT-17301 0 None None None 2023-04-25 14:53:58 UTC
Red Hat Product Errata RHBA-2023:2052 0 None None None 2023-04-27 14:30:49 UTC

Description Lukas Pramuk 2022-10-19 16:53:06 UTC 
Description of problem:
On Sat 6.12 RHEL8.7  ansible-collection-redhat-satellite rpm needlessly pulls in  python38-* deps and misses to pull in python39-* deps

The error is there potentially. ansible-collection-redhat-satellite rpm is lucky that pulpcore 3.18 runs on the same python3.9 as ansible-core 2.13 does

This way the wrong requires are accidentaly satisfied by pulpcore pulling in same modules
(Running on the same python is not guaranteed in any way)

Version-Release number of selected component (if applicable):
6.12.0
ansible-collection-redhat-satellite 3.7.0

How reproducible:
deterministic

Steps to Reproduce:
1. Check Requires: of ansible-collection-redhat-satellite rpm 
# rpm -qR ansible-collection-redhat-satellite
(ansible >= 2.9 or ansible-core)
(python3-pyyaml if ansible)
(python3-requests if ansible)
(python38-pyyaml if ansible-core)    <<< wrong
(python38-requests if ansible-core)  <<< wrong
...

Actual results:
(python38-pyyaml if ansible-core)
(python38-requests if ansible-core)

Expected results:
(python38-pyyaml if ansible-core < 2.13)
(python39-pyyaml if ansible-core >= 2.13)
(python38-requests if ansible-core < 2.13)
(python39-requests if ansible-core >= 2.13)


Additional info:
We delivered these changes to 6.11.z by BZ#2132999
Comment 1 Brad Buckingham 2022-10-24 13:18:46 UTC 
Hi Lukas,

Bug 2132999 has a clone for 6.12 (bug 2125585).  Does this mean that this has also already been resolved on 6.12?
Comment 2 Evgeni Golov 2023-01-17 14:02:58 UTC 
This is fixed in stream: https://gitlab.cee.redhat.com/satellite/satellite-packaging/-/blob/SATELLITE-STREAM/packages/plugins/ansible-collection-redhat-satellite/ansible-collection-redhat-satellite.spec
And has been since 3.7.0-3: https://gitlab.cee.redhat.com/satellite/satellite-packaging/-/commit/71465aff6afd1075a7473df91b6b91e57d600362

If that has not landed in some Satellite branches, please fix accordingly ;)
Comment 7 Lukas Pramuk 2023-04-25 16:16:32 UTC 
VERIFIED.

@ansible-collection-redhat-satellite-3.7.0-4.el8sat.noarch.rpm

# rpm -qR ansible-collection-redhat-satellite | grep python
(python3-pyyaml if ansible)
(python3-requests if ansible)
(python3.11-pyyaml if ansible-core >= 2.14.2-3)
(python3.11-requests if ansible-core >= 2.14.2-3)
(python38-pyyaml if ansible-core < 2.13)
(python38-requests if ansible-core < 2.13)
(python39-pyyaml if (ansible-core >= 2.13 and ansible-core < 2.14.2-3))
(python39-requests if (ansible-core >= 2.13 and ansible-core < 2.14.2-3))

>>> the ansible-collection-redhat-satellite-3.7.0 rpm correctly requires python3.11 modules (and python39 modules)
Comment 13 errata-xmlrpc 2023-04-27 14:30:36 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Satellite 6.12.4 Async Update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2023:2052
Note You need to log in before you can comment on or make changes to this bug.