Uploaded image for project: 'Subscription Watch'
  1. Subscription Watch
  2. SWATCH-1929

Support for hypervisor generic 'virtual_host_uuid' fact

XMLWordPrintable

      Add support for the recently added system profile fact virtual_host_uuid (https://github.com/RedHatInsights/inventory-schemas/pull/121) which replaces legacy facts
      rhsm.VM_HOST_UUID and satellite.virtual_host_uuid We actually still need to consult the 'satellite'->'virtual_host_uuid' fact, because the reporting pipeline for Satellite doesn't yet use the canonical field (more info in https://github.com/RedHatInsights/yuptoo doesn't have any special handling of virtual_host_uuid, and the satellite plugin reports virtual_host_uuid as a Satellite-specific fact).

       

      satellite.virtual_host_uuid references

      • InventoryRepository
      • InventoryHost

      VM_HOST_UUID references

      • insert-mock-hosts script [1]
      • conduit api spec [2]
      • ConsumerInventory
      • InventoryHost
      • InventoryRepository
      • InventoryService

      [1] https://github.com/RedHatInsights/rhsm-subscriptions/blob/1059075a30d65777801695419e6b2be174d13b3e/bin/insert-mock-hosts
      [2] https://github.com/RedHatInsights/rhsm-subscriptions/blob/0ae6b5c7973d9bec1f06a5cb0fd7ef74885dafc9/swatch-system-conduit/src/main/spec/internal-organizations-sync-api-spec.yaml#L260

       

      Done Criteria

      • Conduit is reporting virtual_host_uuid
      • Tally is consuming virtual_host_uuid instead of rhsm.VM_HOST_UUID
      • IQE Note- Ephemeral environment tests are updated to use the virtual_host_uuid fact. 

              jcarvaja@redhat.com Jose Carvajal Hilario
              bciconel@redhat.com Bruno Ciconelle
              Sanket Jagtap Sanket Jagtap
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: