Project

General

Profile

Bug #15559

Importing rhsm facts doesn't set the virtual flag

Added by Ivan Necas over 2 years ago. Updated 7 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Subscriptions
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

The rhsm_fact_parser doesn't properly set the the virtual flag. This is due to reason, that the
FactParser expects the virtual interface identifier in a form of 'eth0_1' while the rhsm
has 'eth0.1'

This leads to errors when importing the facts for a host, that was provisioned with virtual interfaces:

[ERROR 2016-06-30 03:23:04 main] rake aborted!
ActiveRecord::RecordInvalid: Validation failed: Identifier Can't add or remove `.` from identifier
/opt/rh/rh-ror41/root/usr/share/gems/gems/activerecord-4.1.5/lib/active_record/validations.rb:57:in `save!'
/opt/rh/rh-ror41/root/usr/share/gems/gems/activerecord-4.1.5/lib/active_record/attribute_methods/dirty.rb:29:in `save!'
/opt/rh/rh-ror41/root/usr/share/gems/gems/activerecord-4.1.5/lib/active_record/transactions.rb:273:in `block in save!'
/opt/rh/rh-ror41/root/usr/share/gems/gems/activerecord-4.1.5/lib/active_record/transactions.rb:329:in `block in with_transaction_returning_status'
/opt/rh/rh-ror41/root/usr/share/gems/gems/activerecord-4.1.5/lib/active_record/connection_adapters/abstract/database_statements.rb:201:in `block in transaction'

Associated revisions

Revision 7b2e6900 (diff)
Added by Ivan Necas over 2 years ago

Fixes #15559 - set interface virtual flag in rhsm_fact_parser

Revision 482545b8
Added by Brad Buckingham over 2 years ago

Merge pull request #6160 from iNecas/fix-virtual-iface

Fixes #15559 - set interface virtual flag in rhsm_fact_parser

History

#1 Updated by Ivan Necas over 2 years ago

  • Subject changed from Upgrade failed at step: update_subscription_facet_backend_data with error: Validation failed: Identifier Can't add or remove `.` from identifier to Importing rhsm facts doesn't set the virtual flag
  • Description updated (diff)
  • Category changed from Upgrades to Subscriptions

#2 Updated by Ivan Necas over 2 years ago

  • Legacy Backlogs Release (now unused) set to 168

Suggesting 3.0.2, as it generates upgrade issues.

#3 Updated by The Foreman Bot over 2 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Ivan Necas
  • Pull request https://github.com/Katello/katello/pull/6160 added

#4 Updated by Ivan Necas over 2 years ago

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

Also available in: Atom PDF