Project

General

Profile

Actions

Bug #16664

closed

Remove delete action under Config > Puppet Classes, Environments

Added by Ondřej Pražák over 7 years ago. Updated over 5 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
-
Category:
Puppet integration
Target version:
-
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

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.
Actions #1

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
Actions #2

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.

Actions #3

Updated by Marek Hulán over 7 years 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

Actions #4

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.

Actions #5

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.

Actions #6

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.

Actions #7

Updated by Ondřej Pražák over 7 years ago

  • Status changed from New to Rejected
Actions

Also available in: Atom PDF