Project

General

Profile

Actions

Bug #37290

open

Hostgroup inheritance for LCE and CV does not work for grandchildren hostgroups

Added by Nadja Heitmann about 2 months ago. Updated 29 days ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Hosts
Target version:
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

  1. 1) create a hostgroup 'parent' with LCE and CV
  2. 2) create a child hostgroup 'child' and choose 'parent' as as parent hostgroup
  • It takes the correct values from 'parent' for LCE and CV, but does not add 'Inherit parent (...)'
  • Change LCE and CV to something new
  1. 3) create a grandchild hostgroup 'grandchild' and choose 'child' as parent hostgroup
  • CV and LCE are taken from 'parent' hostgroup rather than from 'child' hostgroup

In my opinion, there are multiple thing that are going wrong here:
1) https://github.com/Katello/katello/blob/master/app/helpers/katello/hosts_and_hostgroups_helper.rb#L134 and https://github.com/Katello/katello/blob/master/app/helpers/katello/hosts_and_hostgroups_helper.rb#L163 methods do not pick the `Inherit parent (...)` option, it is added to the beginning of the list only
2) The parent host group is not resolved correctly for CV and LCE. This can either be a UI problem during saving of the new hostgroup or a problem that is hidden deeper.

Actions #1

Updated by Jeremy Lenz about 2 months ago

  • Status changed from New to Need more information

Hi Nadja

We've had a lot of improvements in the hostgroup area lately. Does this issue also occur on nightly / master ?

Actions #2

Updated by Nadja Heitmann about 2 months ago

  • Found in Releases Katello Next added
  • Found in Releases deleted (Katello 4.11.1)
Actions #3

Updated by Nadja Heitmann about 1 month ago

Jeremy Lenz wrote in #note-1:

We've had a lot of improvements in the hostgroup area lately. Does this issue also occur on nightly / master ?

Hi Jeremy,
thanks for the feedback. I have tested on a nightly Forklift and there is the same problem. I cannot pin down why the inherited CL/LCE is not resolved correctly for grandchildren.

Actions #4

Updated by Samir Jha about 1 month ago

  • Status changed from Need more information to New
  • Target version set to Katello 4.13.0
  • Triaged changed from No to Yes
Actions #5

Updated by Samir Jha about 1 month ago

  • Category set to Hosts
Actions #6

Updated by Partha Aji 29 days ago

https://projects.theforeman.org/issues/37290#note-3

Hey Nadja,

While I am able to reproduce 2 i.e.

2) create a child hostgroup 'child' and choose 'parent' as as parent hostgroup
It takes the correct values from 'parent' for LCE and CV, but does not add 'Inherit parent (...)'
Change LCE and CV to something new

I am not able to reproduce 3

3) create a grandchild hostgroup 'grandchild' and choose 'child' as parent hostgroup
CV and LCE are taken from 'parent' hostgroup rather than from 'child' hostgroup

It clearly shows me the child hostgroups LCE and CV for the grandchild.
We can try to resolve the inherit but I dont think the other is a problem.

Can you please verify again.

Actions #7

Updated by Nadja Heitmann 29 days ago

Hey Partha, thanks a lot for testing. I tried again, and indeed, it is exactly as you describe. Looks like there was an issue on my nightly Forklift deploy, sorry for that!

2) create a child hostgroup 'child' and choose 'parent' as as parent hostgroup
It takes the correct values from 'parent' for LCE and CV, but does not add 'Inherit parent (...)'
Change LCE and CV to something new

This is a minor issue then, I would say. Would be nice to fix though to have the UI consistent.

Actions

Also available in: Atom PDF