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

tnisan at redhat.com tnisan at redhat.com
Thu Oct 15 09:24:26 UTC 2015


Tal Nisan 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/47352
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I44ee45ac9770238c8d353ae30bf18a280b92b7ea
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6.0
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: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list