- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-4.0]: core: determine the order of guest agent nic updates
by tnisan＠redhat.com 22 Aug '16

22 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: core: determine the order of guest agent nic updates ...................................................................... core: determine the order of guest agent nic updates Update the guest agent NIC information according to the order of the VM analyzers in VmsMonitoring. Change-Id: I8b3bdd11b97c133d3cfe09fe6f5c332031ae4e1c Bug-Url: https://bugzilla.redhat.com/1366786 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmsMonitoring.java 1 file changed, 17 insertions(+), 20 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/62517 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8b3bdd11b97c133d3cfe09fe6f5c332031ae4e1c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: core: update cluster to query only static vm data
by tnisan＠redhat.com 22 Aug '16

22 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: core: update cluster to query only static vm data ...................................................................... core: update cluster to query only static vm data In order to update VM, we need only its static data so no need to use the costly query of whole VM. Change-Id: If31cd57110eb73574ce2eb6e8e63e232c7c06dc9 Bug-Url: https://bugzilla.redhat.com/1366786 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateClusterCommand.java 1 file changed, 6 insertions(+), 4 deletions(-) Approvals: Jakub Niedermertl: Looks good to me, but someone else must approve Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/62515 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If31cd57110eb73574ce2eb6e8e63e232c7c06dc9 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: core: fix possible deadlock on update cluster version
by tnisan＠redhat.com 22 Aug '16

22 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: core: fix possible deadlock on update cluster version ...................................................................... core: fix possible deadlock on update cluster version Sort the VMs that are being updated according to the order that the monitoring updates VM related data to prevent a possible race when both are done within a transaction (with postgres lower or equal to 9.2 where updates of entries with foreign key locks the row of the foreign entity in the transaction scope). Change-Id: Iedceef969c8823200bbdb27a332831e9305aebcf Bug-Url: https://bugzilla.redhat.com/1366786 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ClusterOperationCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateClusterCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateClusterCommandTest.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmsMonitoring.java 4 files changed, 33 insertions(+), 17 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/62518 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iedceef969c8823200bbdb27a332831e9305aebcf Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[sdk_4.0]: [maven-release-plugin] prepare release 4.0.0.alpha15
by omachace＠redhat.com 22 Aug '16

22 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: [maven-release-plugin] prepare release 4.0.0.alpha15 ...................................................................... [maven-release-plugin] prepare release 4.0.0.alpha15 Change-Id: I884ed9547e4c0868a1daeeac7fcb8ad776d4142a Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M automation/build-artifacts.sh M generator/pom.xml M packaging/build.sh M pom.xml M sdk/pom.xml 5 files changed, 7 insertions(+), 7 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62659 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I884ed9547e4c0868a1daeeac7fcb8ad776d4142a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_4.0 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[sdk_4.0]: Fix typo in CHANGES.adoc
by juan.hernandez＠redhat.com 22 Aug '16

22 Aug '16

Juan Hernandez has submitted this change and it was merged. Change subject: Fix typo in CHANGES.adoc ...................................................................... Fix typo in CHANGES.adoc This patch fixes a typo in the CHANGES.adoc file. Change-Id: I549a3ba19a715bd38a96309010da15896cdfbfc9 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/CHANGES.adoc 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62661 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I549a3ba19a715bd38a96309010da15896cdfbfc9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.0 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[sdk_4.0]: Use POST method
by omachace＠redhat.com 22 Aug '16

22 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Use POST method ...................................................................... Use POST method This patch changes the method GET to POST to send data. Change-Id: I720e45918fa401a57dae3d52b35b74a784a1977b Signed-off-by: Ondra Machacek <omachace(a)redhat.com> (cherry picked from commit a337f06170f739fc4e322f8d1f76a6c9419f2489) --- M sdk/src/main/java/org/ovirt/engine/sdk4/internal/HttpConnection.java M sdk/src/main/java/org/ovirt/engine/sdk4/internal/SsoUtils.java 2 files changed, 33 insertions(+), 23 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62657 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I720e45918fa401a57dae3d52b35b74a784a1977b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_4.0 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[sdk_4.0]: Use POST method
by omachace＠redhat.com 22 Aug '16

22 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Use POST method ...................................................................... Use POST method This patch changes the method GET to POST to send data. Change-Id: I27c717e9d94e0371ca98c595c5525b676a355728 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> (cherry picked from commit ec889f70c36b06a2a4257a68c293dbf0b61234eb) --- M sdk/lib/ovirtsdk4/__init__.py 1 file changed, 26 insertions(+), 30 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62656 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I27c717e9d94e0371ca98c595c5525b676a355728 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_4.0 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[master]: Use POST method
by omachace＠redhat.com 22 Aug '16

22 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Use POST method ...................................................................... Use POST method This patch changes the method GET to POST to send data. Change-Id: I720e45918fa401a57dae3d52b35b74a784a1977b Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M sdk/src/main/java/org/ovirt/engine/sdk4/internal/HttpConnection.java M sdk/src/main/java/org/ovirt/engine/sdk4/internal/SsoUtils.java 2 files changed, 33 insertions(+), 23 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62655 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I720e45918fa401a57dae3d52b35b74a784a1977b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[master]: Use POST method
by omachace＠redhat.com 22 Aug '16

22 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Use POST method ...................................................................... Use POST method This patch changes the method GET to POST to send data. Change-Id: I27c717e9d94e0371ca98c595c5525b676a355728 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M sdk/lib/ovirtsdk4/__init__.py 1 file changed, 26 insertions(+), 30 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62651 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I27c717e9d94e0371ca98c595c5525b676a355728 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: fix possible deadlock on update cluster version
by ahadas＠redhat.com 22 Aug '16

22 Aug '16

Arik Hadas has submitted this change and it was merged. Change subject: core: fix possible deadlock on update cluster version ...................................................................... core: fix possible deadlock on update cluster version Sort the VMs that are being updated according to the order that the monitoring updates VM related data to prevent a possible race when both are done within a transaction (with postgres lower or equal to 9.2 where updates of entries with foreign key locks the row of the foreign entity in the transaction scope). Change-Id: Iedceef969c8823200bbdb27a332831e9305aebcf Bug-Url: https://bugzilla.redhat.com/1366786 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ClusterOperationCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateClusterCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateClusterCommandTest.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmsMonitoring.java 4 files changed, 36 insertions(+), 17 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Arik Hadas: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/62525 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iedceef969c8823200bbdb27a332831e9305aebcf Gerrit-PatchSet: 11 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