- Engine-commits - oVirt List Archives

Change in ovirt-guest-agent[ovirt-3.6]: packaging: remove autogenerated INSTALL
by vfeenstr＠redhat.com 02 Feb '16

02 Feb '16

Vinzenz Feenstra has submitted this change and it was merged. Change subject: packaging: remove autogenerated INSTALL ...................................................................... packaging: remove autogenerated INSTALL Change-Id: Ib499cdcf7d0ad5bdc77fe1992307c2477f9fc649 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- D INSTALL 1 file changed, 0 insertions(+), 370 deletions(-) Approvals: Sandro Bonazzola: Verified Vinzenz Feenstra: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52976 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib499cdcf7d0ad5bdc77fe1992307c2477f9fc649 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-guest-agent[ovirt-3.6]: packaging: added windows agent spec file
by vfeenstr＠redhat.com 02 Feb '16

02 Feb '16

Vinzenz Feenstra has submitted this change and it was merged. Change subject: packaging: added windows agent spec file ...................................................................... packaging: added windows agent spec file Change-Id: Ic1ecb0c9009c70bdafb5a22f78a85de1a32bb09a Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- A ovirt-guest-agent-windows.spec 1 file changed, 76 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified Vinzenz Feenstra: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52975 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic1ecb0c9009c70bdafb5a22f78a85de1a32bb09a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-guest-agent[ovirt-3.6]: Added git dependency
by vfeenstr＠redhat.com 02 Feb '16

02 Feb '16

Vinzenz Feenstra has submitted this change and it was merged. Change subject: Added git dependency ...................................................................... Added git dependency It turns out that the inner make commands don't work without it, but don't fail either Change-Id: Iff44a3270c8e2c18f852d9c282fc24f39a7453cb Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M automation/check-patch.packages 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified Vinzenz Feenstra: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52974 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iff44a3270c8e2c18f852d9c282fc24f39a7453cb Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-guest-agent[ovirt-3.6]: Added standard ci scripts
by vfeenstr＠redhat.com 02 Feb '16

02 Feb '16

Vinzenz Feenstra has submitted this change and it was merged. Change subject: Added standard ci scripts ...................................................................... Added standard ci scripts Only the check-patch, as that's the only job it has now on the ovirt ci Change-Id: I928b2a5c2ceff982475f9052acf96d610c0c04f7 Signed-off-by: David Caro <dcaroest(a)redhat.com> (cherry picked from commit dcff97c22fbbc9e2125c6e4edb2a707d83cccce3) --- M .gitignore A automation/check-patch.packages A automation/check-patch.sh 3 files changed, 18 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified Vinzenz Feenstra: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52964 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I928b2a5c2ceff982475f9052acf96d610c0c04f7 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: spagobi: add dependencies yum repo
by sbonazzo＠redhat.com 02 Feb '16

02 Feb '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: spagobi: add dependencies yum repo ...................................................................... spagobi: add dependencies yum repo Since we're not yet ready to publish in official repos, adding temporary nightly repo. Change-Id: I84c96761bd3abd58cbd012dc46a35bca22c6decd Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/spagobi/spagobi_create-rpms.yaml 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/52990 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I84c96761bd3abd58cbd012dc46a35bca22c6decd Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: cleanup in vm events refresher
by ahadas＠redhat.com 02 Feb '16

02 Feb '16

Arik Hadas has submitted this change and it was merged. Change subject: core: cleanup in vm events refresher ...................................................................... core: cleanup in vm events refresher 1. Inject DbFacade 2. Change 'convertEvent' to return the list of VMs instead of getting a list and fill it 3. Calculate fetch-time only once for both vms and devices 4. Simplify the 'convertEvent' method Change-Id: I1e24580ea017dd5a19b668c55d78c85f4950df24 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/EventVmStatsRefresher.java 1 file changed, 27 insertions(+), 31 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Arik Hadas: Verified; Passed CI tests Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/51058 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1e24580ea017dd5a19b668c55d78c85f4950df24 Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: cleanup in vm analyzer
by ahadas＠redhat.com 02 Feb '16

02 Feb '16

Arik Hadas has submitted this change and it was merged. Change subject: core: cleanup in vm analyzer ...................................................................... core: cleanup in vm analyzer 1. Extract calls to Destroy to separate methods 2. Replace calls to the depracated ResourceManager#getInstance 3. Extract code related to audit logs to separate methods 4. Use Early return to simplify updateVmRunTimeInfo 5. Rename variable props to changedFields 6. Use Stream API to simplify updateVmNumaNodeRuntimeInfo Change-Id: Ic6bf23080fd5e94d98c051296fe11f7524ea6c10 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmAnalyzer.java 1 file changed, 90 insertions(+), 79 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Arik Hadas: Verified; Passed CI tests Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/52835 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic6bf23080fd5e94d98c051296fe11f7524ea6c10 Gerrit-PatchSet: 13 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Add "/v4" prefix
by juan.hernandez＠redhat.com 02 Feb '16

02 Feb '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Add "/v4" prefix ...................................................................... restapi: Add "/v4" prefix This patch adds the "/v4" prefix to all the URIs used internally by the API. The version filter introduced in a previous patch will add this prefix if it hasn't been explicitly added to the URI by the user, or if the version has been specified using the "Version" header. Change-Id: I2abb2fbedaedf28f1e34c3e6ec7f8b14901f92d9 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/BackendApplication.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/invocation/VersionFilter.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendApiResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/util/LinkHelper.java 4 files changed, 21 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/52705 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2abb2fbedaedf28f1e34c3e6ec7f8b14901f92d9 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Use "current" object to build links
by juan.hernandez＠redhat.com 02 Feb '16

02 Feb '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Use "current" object to build links ...................................................................... restapi: Use "current" object to build links Currently the "LinkHelper" class receives an "UriInfo" object that it uses to extract the prefix of the application, in order to build links. This patch changes it so that it uses the "Current" object, as this will have support for multiple versions of the API in the future. Change-Id: Idb78608ed9f085000b6ff38ced48c6cb175abbdb Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendActionableResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendCollectionResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendCapabilitiesResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostNicsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendNicHelper.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmDiskResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmDisksResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmNicResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmNicsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmPoolResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterVolumeResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/util/ActionsBuilder.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/util/LinkHelper.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/AbstractBackendBaseTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/util/LinkHelperTest.java 19 files changed, 143 insertions(+), 162 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/52704 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idb78608ed9f085000b6ff38ced48c6cb175abbdb Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Move link helper to implementation module
by juan.hernandez＠redhat.com 02 Feb '16

02 Feb '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Move link helper to implementation module ...................................................................... restapi: Move link helper to implementation module This patch moves the "LinkHelper" class from the definition module to the implementation module, where it will be able to use the "current" object to build links that depend on the current request. Change-Id: I81d5323470ce463eb71809ec37474adbd0488353 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- R backend/manager/modules/restapi/interface/definition/src/test/java/org/ovirt/engine/api/utils/LinkCreatorTest.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendActionableResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendCollectionResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendCapabilitiesResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostNicsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendNicHelper.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmDiskResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmDisksResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmPoolResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterVolumeResource.java R backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/util/ActionsBuilder.java R backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/util/LinkHelper.java R backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/util/LinkHelperTest.java 17 files changed, 33 insertions(+), 35 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/52703 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I81d5323470ce463eb71809ec37474adbd0488353 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0