Bug 1751727 - Async hypervisor update doesn't override nil hypervisorId
Summary: Async hypervisor update doesn't override nil hypervisorId
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Candlepin
Classification: Community
Component: candlepin
Version: 2.6
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: ---
Assignee: Abhishek Kumar
QA Contact:
URL:
Whiteboard:
Depends On: 1746912
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-09-12 13:09 UTC by Barnaby Court
Modified: 2019-09-24 17:27 UTC (History)
6 users (show)

Fixed In Version: candlepin-2.9.0-1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1746912
Environment:
Last Closed: 2019-09-24 17:27:47 UTC
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github candlepin candlepin pull 2435 0 None None None 2019-09-17 10:32:35 UTC

Description Barnaby Court 2019-09-12 13:09:35 UTC 
+++ This bug was initially created as a clone of Bug #1746912 +++

Description of problem:

If a hypervisor is registered with subscription-manager it won't have a hypervisorId at that time. Subsequent virt-who checkins won't set the hypervisorId to the value in the virt-who report (The host will be reported as Unchanged)

Version-Release number of selected component (if applicable): candlepin-2.6.8-1.el7sat.noarch


How reproducible: Always


Steps to Reproduce:
1. Register a hypervisor with subman
2. Install virt-who on the hypervisor and initiate a checkin (ensure the DMI UUID is present so it finds the existing hypervisor)

Actual results: hypervisorId doesn't get updated to what's in the virt-who report


Expected results: it should be updated


Additional info:

Related code: https://github.com/candlepin/candlepin/blob/bf4b1f18d045dcec3136a1b0f4e94452c80728c9/server/src/main/java/org/candlepin/service/impl/HypervisorUpdateAction.java#L153-L154
Note You need to log in before you can comment on or make changes to this bug.