Bug 2029402

Summary:	[RFE] Add functionality in Hammer to Add/Delete a single Ansible role to Hostgroup without defining every role.
Product:	Red Hat Satellite	Reporter:	Ragesh <rkarimpa>
Component:	Ansible - Configuration Management	Assignee:	Oleh Fedorenko <ofedoren>
Status:	CLOSED ERRATA	QA Contact:	Gaurav Talreja <gtalreja>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	6.9.7	CC:	addubey, apatel, aruzicka, gtalreja, inecas, kgaikwad, momran, oezr, ofedoren, pcreech, rabajaj, rlavi
Target Milestone:	6.13.0	Keywords:	EasyFix, FutureFeature, Triaged
Target Release:	Unused
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:	rubygem-foreman_ansible-8.0.0, rubygem-hammer_cli_foreman_ansible-0.4.0	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	2164400 (view as bug list)		Environment:
Last Closed:	2023-05-03 13:21:03 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	2164400

Description Ragesh 2021-12-06 12:06:51 UTC

1. Description of Request:

Currently, there's no option to just Add/Delete a single Ansible role to Hostgroup without defining every role.

For adding a role to a hostgroup which already has few roles assigned to it, we need to follow the below process:

For Example: 

To add role3 to hostgroup test which already has role1 and role2 assigned to it, one needs to run:
~~~
# hammer hostgroup ansible-roles assign --name test --ansible-roles role1,role2,role3
~~~

Similarly, to remove only role3 from the above hostgroup, one has to run:
~~~
# hammer hostgroup ansible-roles assign --name test --ansible-roles role1,role2
~~~ 

2. Why does the customer need this? (List the business requirements here)  

Customer has problem with this as defining always every role is highly impractical/not user friendly.

Comment 1 Oleh Fedorenko 2021-12-09 14:47:42 UTC

Moving to Ansible component, since the changes must be done in hammer-cli-foreman-ansible plugin.

Comment 2 Oleh Fedorenko 2021-12-09 14:48:28 UTC

Created redmine issue https://projects.theforeman.org/issues/34121 from this bug

Comment 3 Bryan Kearney 2021-12-18 20:06:31 UTC

Upstream bug assigned to ofedoren

Comment 4 Bryan Kearney 2021-12-18 20:06:33 UTC

Upstream bug assigned to ofedoren

Comment 5 Bryan Kearney 2022-05-03 16:04:25 UTC

Moving this bug to POST for triage into Satellite since the upstream issue https://projects.theforeman.org/issues/34121 has been resolved.

Comment 10 Adam Ruzicka 2023-01-02 10:00:45 UTC

Wait, hammer "hostgroup ansible-roles assign" is meant to be "here's the full list, make it so". To fix this bz, we introduced "hammer hostgroup ansible-roles add" and "hammer hostgroup ansible-roles remove" to allow adding and removing of roles. Moving back to ON_QA

Comment 17 Gaurav Talreja 2023-01-29 15:29:32 UTC

Hi Mark, 

Thanks for updating and creating a separate BZ for API changes. We're OK with verifying this RFE BZ for 6.13 with only Hammer changes.

Thanks
Gaurav

Comment 21 errata-xmlrpc 2023-05-03 13:21:03 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 (Important: Satellite 6.13 Release), 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/RHSA-2023:2097