Change in ovirt-engine[ovirt-engine-3.6]: engine: Use UUID as content host id on Satellite
Oved Ourfali 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(a)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/47350
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I44ee45ac9770238c8d353ae30bf18a280b92b7ea
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org