ansible/community.general
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2024-09-14 20:13:21 +02:00
Code Releases Activity

redhat_subscription: use CLI when using environments (#6319)

Browse source 
It turns out that the 'environments' that the D-Bus Register*() APIs
accept are the IDs of the environments, and not the user-facing names of
the environments (which is what the module has been accepting so far).

Since there is no easy way to do the mapping manually, for now use again
the subscription-manager CLI for registering when environments are
specified.
This commit is contained in:
Pino Toscano 2023-04-13 07:17:53 +02:00 committed by GitHub
parent bf780ea738
commit 83994c0a2d
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
changelogs/fragments/6319-redhat_subscription-fix-environment-parameter.yaml Normal file
View file

@ -0,0 +1,5 @@
bugfixes:
- redhat_subscription - do not use D-Bus for registering when ``environment`` is specified,
so it possible to specify again the environment names for registering,
as the D-Bus APIs work only with IDs
(https://github.com/ansible-collections/community.general/pull/6319).

6
plugins/modules/redhat_subscription.py
View file

@ -461,8 +461,10 @@ class Rhsm(RegistrationBase):
* Exception - if any error occurs during the registration * Exception - if any error occurs during the registration
''' '''
# There is no support for token-based registration in the D-Bus API # There is no support for token-based registration in the D-Bus API
# of rhsm, so always use the CLI in that case. # of rhsm, so always use the CLI in that case;
if not token and self._can_connect_to_dbus(): # also, since the specified environments are names, and the D-Bus APIs
# require IDs for the environments, use the CLI also in that case
if not token and not environment and self._can_connect_to_dbus():
self._register_using_dbus(was_registered, username, password, auto_attach, self._register_using_dbus(was_registered, username, password, auto_attach,
activationkey, org_id, consumer_type, activationkey, org_id, consumer_type,
consumer_name, consumer_id, consumer_name, consumer_id,