Project

General

Profile

Actions

Bug #3551

closed

Foreman removes all puppet classes from a hostgroup when the environment is changed.

Added by Jim Bailey over 10 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Category:
Puppet integration
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

When changing the environment for a hostgroup, puppet removes all classes.


Related issues 2 (0 open2 closed)

Related to Foreman - Bug #18045: Puppet classes don't show up without hostgroupsClosedDaniel Lobato Garcia01/12/2017Actions
Related to Foreman - Bug #18427: Hostgroups don't have any puppet classes (or are removed upon editing a hostgroup)Resolved02/08/2017Actions
Actions #1

Updated by Dominic Cleal over 9 years ago

  • Category set to Puppet integration
Actions #2

Updated by Dominic Cleal over 9 years ago

  • Bugzilla link set to 1158991
Actions #3

Updated by Sebastian Gräßl over 7 years ago

The circumstances under which this occurs are when a class are not available in the environment the `HostGroup` will be changed to.
If the target environment does have the class available it will be kept. This does not happen instantly (anymore) but requires the form to be submitted.

Since Foreman does not create classes in puppet, but only reflect what is in puppet, keeping the classes in host groups as they are could lead to confusion, if they are not actually in the current environment. Keeping them, and marking them as "unavailable" if the environment does not include it and allow to remove them manually.

A simpler approach would be to simple warn in case classes are not available and will be purged on submit.

Thoughts on which route would be the more feasible way to go?

Actions #4

Updated by Marek Hulán over 7 years ago

I prefer warning about purging on submit.

Actions #5

Updated by Sebastian Gräßl over 7 years ago

  • Assignee set to Sebastian Gräßl
Actions #6

Updated by Marek Hulán over 7 years ago

  • Target version set to 1.4.4
Actions #7

Updated by The Foreman Bot over 7 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4048 added
Actions #8

Updated by Tomáš Strachota over 7 years ago

  • Target version deleted (1.4.4)
Actions #9

Updated by Daniel Lobato Garcia over 7 years ago

  • Target version set to 1.15.5
Actions #10

Updated by Anonymous over 7 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #11

Updated by Dominic Cleal over 7 years ago

  • translation missing: en.field_release set to 209
Actions #12

Updated by Dominic Cleal over 7 years ago

  • Related to Bug #18045: Puppet classes don't show up without hostgroups added
Actions #13

Updated by Daniel Lobato Garcia about 7 years ago

  • Target version changed from 1.15.5 to 169
Actions #14

Updated by Dominic Cleal about 7 years ago

  • Related to Bug #18427: Hostgroups don't have any puppet classes (or are removed upon editing a hostgroup) added
Actions #15

Updated by Brad Buckingham about 7 years ago

  • Target version deleted (169)
Actions #16

Updated by Daniel Lobato Garcia about 7 years ago

  • Target version set to 1.11.0
Actions

Also available in: Atom PDF