Project

General

Profile

Bug #17648

Attribute inheritance is not propagated correctly in UI/API workflows

Added by Shimon Shtein over 3 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

update method does not invoke #apply_inherited_attributes on the newly created host.

hg1.png View hg1.png 49.8 KB Jason Nance, 12/20/2016 01:40 PM
hg2.png View hg2.png 24.7 KB Jason Nance, 12/20/2016 01:40 PM
host.png View host.png 47.2 KB Jason Nance, 12/20/2016 01:40 PM
Hg1 Hg2 Host

Related issues

Related to Discovery - Refactor #14316: Host group fails to auto fill puppet ca & master in discovery provisioningNew
Related to Discovery - Bug #18106: Discovery v8 provisioning throws ActiveModel::ForbiddenAttributesError when Katello is installedClosed2017-01-17
Related to Discovery - Bug #20797: Provisioning errors with undefined method permit! for ActiveSupport::HashWithIndifferentAccessResolved2017-08-29

Associated revisions

Revision 044cd42f (diff)
Added by Shimon Shtein over 3 years ago

Fixes #17648 - Added support for attributes inheritance

Changed HostConverter.to_managed to automatically call attribute
inheritance mechanism when a new set of attributes is introduced.

History

#1 Updated by Lukas Zapletal over 3 years ago

  • Subject changed from PUT /discovered_hosts/:id is not inheriting attributes from selected hostgroup to Attribute inheritance is not propagated correctly in UI/API workflows

So after some chat it looks like we are not calling apply_inherited_attributes properly from API and also from UI (we call it in edit but not in update).

#2 Updated by The Foreman Bot over 3 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman_discovery/pull/316 added

#3 Updated by Lukas Zapletal over 3 years ago

  • Related to Refactor #14316: Host group fails to auto fill puppet ca & master in discovery provisioning added

#4 Updated by Jason Nance over 3 years ago

I believe this same issue exists with the realm as well. Please see attached screen shots for example. If I'm talking about something different let me know and I'll open a new bug.

In short, inherited Puppet CA, Puppet Master, and Realm are showing up in the host group but not on a newly created host.

#5 Updated by Shimon Shtein over 3 years ago

It should all be solved by this fix.
All inherited attributes are manged by those two methods: #apply_inherited_attributes and #set_hostgroup_defaults that are added by this PR.

Let us know, if it's not solving your case.

#6 Updated by Shimon Shtein over 3 years ago

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

#7 Updated by Simone Caldana over 3 years ago

  • Related to Bug #18106: Discovery v8 provisioning throws ActiveModel::ForbiddenAttributesError when Katello is installed added

#8 Updated by Lukas Zapletal almost 3 years ago

  • Related to Bug #20797: Provisioning errors with undefined method permit! for ActiveSupport::HashWithIndifferentAccess added

Also available in: Atom PDF