Change in ovirt-engine[master]: engine: Use UUID as content host id on Satellite

masayag at redhat.com masayag at redhat.com
Thu Oct 15 07:36:44 UTC 2015


Moti Asayag has submitted this change and it was merged.

Change subject: engine: Use UUID as content host id on Satellite
......................................................................


engine: Use UUID as content host id on Satellite

There is a difference in the semantic of the ID field under:
 /katello/api/v2/systems/:id resource between Katello to Satellite.

The :id field in Katello represents the UUID of the content host,
however under satellite it is a sequence number, which causes the
requests sent from the engine to fail with unknown resource identifier.
On Satellite (and also on Katello) the value of the UUID field of the
content host can be used to identify it @candlepin.

The solution will be to use the UUID field of the system resource
instead of the ID.

Change-Id: I44ee45ac9770238c8d353ae30bf18a280b92b7ea
Bug-Url: https://bugzilla.redhat.com/1271340
Signed-off-by: Moti Asayag <masayag at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/provider/foreman/ContentHost.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/provider/foreman/ForemanHostProviderProxy.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/provider/foreman/SystemProviderFinder.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/BaseProviderProxy.java
4 files changed, 14 insertions(+), 9 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Moti Asayag: Verified
  Oved Ourfali: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/47319
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I44ee45ac9770238c8d353ae30bf18a280b92b7ea
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list