Bug #16664

Remove delete action under Config > Puppet Classes, Environments

Added by Ondřej Pražák about 1 year ago. Updated about 1 year ago.

Status:Rejected
Priority:Normal
Assigned To:-
Category:Puppet integration
Target version:Team Marek backlog
Difficulty: Bugzilla link:1163931
Found in release: Pull request:
Story points-
Velocity based estimate-

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.

History

#1 Updated by Ondřej Pražák about 1 year 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 Smart Variables to Puppet integration
  • Target version set to Team Marek backlog

#2 Updated by Dominic Cleal about 1 year 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.

#3 Updated by Marek Hulán about 1 year ago

Looking at original BZ, it seems that bulk actions (or at least bulk deletion) of puppet classes is the subject of this issue? That was also kind of rejected so did the course change? Potentially related #2047, possibly duplicates #387

#4 Updated by Shlomi Zadok about 1 year 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.

#5 Updated by Ondřej Pražák about 1 year 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.

#6 Updated by Marek Hulán about 1 year 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.

#7 Updated by Ondřej Pražák about 1 year ago

  • Status changed from New to Rejected

Also available in: Atom PDF