Bug #16664
closedRemove delete action under Config > Puppet Classes, Environments
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1163931
tPuppet classes and environments are created on import from Puppet server. When we delete them manually in Foreman,
they are recreated on the next import, effectively reverting the deletion. Because Puppet classes are meant
to be managed through import action anyway, we can remove the delete action.
Updated by Ondřej Pražák over 7 years ago
- Tracker changed from Feature to Bug
- Subject changed from Remove delete action under Config > Puppet Classes, Environments to Remove delete action under Config > Puppet Classes, Environments
- Category changed from Parameters to Puppet integration
- Target version set to 115
Updated by Dominic Cleal over 7 years ago
As Puppet environments can be created manually, it should be possible to delete them without relying on the import process. #2321 discussed removing manual environment creation and it was left in.
Deleting Puppet classes may still be a useful way to remove all custom settings and variables, then re-import it with defaults.
Updated by Marek Hulán over 7 years ago
Updated by Shlomi Zadok over 7 years ago
I agree with Dominic - deleting a puppet class may be useful at times. I think that removing the deletion functionality may be painful to some of the users.
Updated by Ondřej Pražák over 7 years ago
OK, lets keep it in. Since we do not want to introduce bulk deletion, I think we can close this if there are no objections.
Updated by Marek Hulán over 7 years ago
I think bulk deletion would be nice, but probably for all Foreman objects, not just hosts and puppet classes. So +1 for rejecting this and we'll see if someone asks for general bulk actions, I didn't find redmine issue for that yet. Ondra, please also propagate this into linked BZ after closing this issue.