Bug #14537: unable to register host with sub-man after puppet registration (bats failure) - Katello - Foreman

Actions

Copy link

Bug #14537

closed

unable to register host with sub-man after puppet registration (bats failure)

Added by Chris Duryee about 9 years ago. Updated almost 7 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Category:

Hosts

Target version:

Katello 3.0.0

Difficulty:

Triaged:

Bugzilla link:

Pull request:

https://github.com/Katello/katello/pull/5946, https://github.com/Katello/katello/pull/5951, https://github.com/Katello/katello/pull/5966

Fixed in Releases:

Found in Releases:

Red Hat JIRA:

Description

If a host is registered via puppet, it is unable to then register with subscription manager

stack:

2016-04-07 14:22:50 [katello/cp_proxy] [E] ActiveRecord::AssociationTypeMismatch: Taxonomy(#66886280) expected, got NilClass(#19624500)
 | /opt/rh/rh-ror41/root/usr/share/gems/gems/activerecord-4.1.5/lib/active_record/associations/association.rb:216:in `raise_on_type_mismatch!'
 | /opt/rh/rh-ror41/root/usr/share/gems/gems/activerecord-4.1.5/lib/active_record/associations/has_many_through_association.rb:32:in `block in concat'
 | /opt/rh/rh-ror41/root/usr/share/gems/gems/activerecord-4.1.5/lib/active_record/associations/has_many_through_association.rb:31:in `each'
 | /opt/rh/rh-ror41/root/usr/share/gems/gems/activerecord-4.1.5/lib/active_record/associations/has_many_through_association.rb:31:in `concat'
 | /opt/rh/rh-ror41/root/usr/share/gems/gems/activerecord-4.1.5/lib/active_record/associations/collection_proxy.rb:972:in `<<'
 | /opt/theforeman/tfm/root/usr/share/gems/gems/katello-3.1.0/app/lib/actions/katello/host/register.rb:69:in `connect_to_smart_proxy'
 | /opt/theforeman/tfm/root/usr/share/gems/gems/katello-3.1.0/app/lib/actions/katello/host/register.rb:32:in `block in plan'

Actions

Copy link

Updated by Chris Duryee about 9 years ago

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

Applied in changeset katello|8db03a71b353ec7633270a4131c865d606d00609.

Actions

Copy link

Updated by Chris Duryee about 9 years ago

Status changed from Closed to New
Assignee deleted (~~Chris Duryee~~)

After discussion with Justin and Stephen, the most correct way to fix this is to update https://github.com/Katello/katello/blob/master/app/lib/actions/katello/host/register.rb#L9-L12 to call host.save immediately before unregistering, so the org doesn't get lost. The other PR can be undone as part of the fix, since host.organization can be relied upon again.