March 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: Call super to get Backend instance
by amureini＠redhat.com 14 Mar '16

14 Mar '16

Allon Mureinik has submitted this change and it was merged. Change subject: engine: Call super to get Backend instance ...................................................................... engine: Call super to get Backend instance Fix getBackend() to call super.getBackend(). This patch addresses Coverity CID 1348295. Change-Id: Ieb6bf7fe1cc0967da27af181c0ddd4f096fe9a75 Signed-off-by: Ala Hino <ahino(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GetAddedGlusterServersQuery.java 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Sahina Bose: Looks good to me, but someone else must approve Ala Hino: Verified -- To view, visit https://gerrit.ovirt.org/54704 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieb6bf7fe1cc0967da27af181c0ddd4f096fe9a75 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Add V3 action adapters
by juan.hernandez＠redhat.com 14 Mar '16

14 Mar '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Add V3 action adapters ...................................................................... restapi: Add V3 action adapters This patch adds the V3 input and output adapters for the "Action" type. Change-Id: Id295672e72a40959abb98cf20728ab21bec970ef Bug-Url: https://bugzilla.redhat.com/1317279 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3ActionInAdapter.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3ActionOutAdapter.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3FaultInAdapter.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3GracePeriodInAdapter.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3GracePeriodOutAdapter.java 5 files changed, 492 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54706 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id295672e72a40959abb98cf20728ab21bec970ef 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Release 4.0.0.alpha11
by juan.hernandez＠redhat.com 14 Mar '16

14 Mar '16

Juan Hernandez has submitted this change and it was merged. Change subject: Release 4.0.0.alpha11 ...................................................................... Release 4.0.0.alpha11 Change-Id: Ida7a416168c985d5bb348432a152f8a225c20fc7 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M packaging/build.sh M pom.xml 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54681 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ida7a416168c985d5bb348432a152f8a225c20fc7 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-guest-agent[ovirt-3.6]: windows: More defensive code for windows VirtIO handling
by vfeenstr＠redhat.com 14 Mar '16

14 Mar '16

Vinzenz Feenstra has submitted this change and it was merged. Change subject: windows: More defensive code for windows VirtIO handling ...................................................................... windows: More defensive code for windows VirtIO handling To avoid continous exceptions to eat 100% of the CPU we're now sleeping at least for 1 second after an exception and we're logging the exception in debug mode, so we can investigate potential problems. To avoid spamming the logs this is on debug log for now. Change-Id: I0cb7c1108a579bf55f9340cb1c211b65d6cd8f5b Bug-Url: https://bugzilla.redhat.com/1257835 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M ovirt-guest-agent/WinFile.py 1 file changed, 8 insertions(+), 1 deletion(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/54710 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0cb7c1108a579bf55f9340cb1c211b65d6cd8f5b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Vinzenz Feenstra <vfeenstr(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[master]: windows: More defensive code for windows VirtIO handling
by vfeenstr＠redhat.com 14 Mar '16

14 Mar '16

Vinzenz Feenstra has submitted this change and it was merged. Change subject: windows: More defensive code for windows VirtIO handling ...................................................................... windows: More defensive code for windows VirtIO handling To avoid continous exceptions to eat 100% of the CPU we're now sleeping at least for 1 second after an exception and we're logging the exception in debug mode, so we can investigate potential problems. To avoid spamming the logs this is on debug log for now. Change-Id: I0cb7c1108a579bf55f9340cb1c211b65d6cd8f5b Bug-Url: https://bugzilla.redhat.com/1257835 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M ovirt-guest-agent/WinFile.py 1 file changed, 8 insertions(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Vinzenz Feenstra: Verified; Looks good to me, approved Martin Polednik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/53909 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0cb7c1108a579bf55f9340cb1c211b65d6cd8f5b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: simplify the logic within vm-jobs monitoring
by ahadas＠redhat.com 14 Mar '16

14 Mar '16

Arik Hadas has submitted this change and it was merged. Change subject: core: simplify the logic within vm-jobs monitoring ...................................................................... core: simplify the logic within vm-jobs monitoring Instead of filtering the existing jobs from the reported jobs and then iterating the reported jobs to find the jobs to update and afterwards iterating the existing jobs to find the jobs to remove, we now iterating only the existing jobs and find the jobs to update and the jobs to remove in one cycle. Change-Id: I71d3a6be42193a717a3c68b7c4dbe9966e1e080d Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmJobsMonitoring.java 1 file changed, 18 insertions(+), 31 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/54314 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I71d3a6be42193a717a3c68b7c4dbe9966e1e080d Gerrit-PatchSet: 7 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: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: removing ovirt register jobs
by tlitovsk＠redhat.com 14 Mar '16

14 Mar '16

Anatoly Litovsky has submitted this change and it was merged. Change subject: removing ovirt register jobs ...................................................................... removing ovirt register jobs Change-Id: I9397e733aefae077925d5a41c7c196e21c4a57f0 Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- D jobs/confs/projects/ovirt-register/ovirt-register.yaml M jobs/confs/projects/ovirt/publish-rpms-nightly-3.6.yaml D jobs/confs/yaml/scms/ovirt-register-scms.yaml 3 files changed, 0 insertions(+), 29 deletions(-) Approvals: Sandro Bonazzola: Looks good to me Jenkins CI: Passed CI tests Anatoly Litovsky: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/54708 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9397e733aefae077925d5a41c7c196e21c4a57f0 Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(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 jenkins[master]: Reduced the amount of builds to keep in all ovirt-engine*bui...
by eedri＠redhat.com 13 Mar '16

13 Mar '16

Eyal Edri has submitted this change and it was merged. Change subject: Reduced the amount of builds to keep in all ovirt-engine*build-artifacts* jobs ...................................................................... Reduced the amount of builds to keep in all ovirt-engine*build-artifacts* jobs Change-Id: I0a2f92342e04e4056800e60d8b9a6ee1ae33a713 Signed-off-by: Gil Shinar <gshinar(a)redhat.com> --- M jobs/confs/yaml/templates/standard-stage.yaml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Eyal Edri: Looks good to me; Ready for merge Gil Shinar: Verified; Ready for review Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54683 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0a2f92342e04e4056800e60d8b9a6ee1ae33a713 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Gil Shinar <gil(a)tikalk.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gil(a)tikalk.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-sdk[master]: Add connection builder and automatic closing
by juan.hernandez＠redhat.com 11 Mar '16

11 Mar '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add connection builder and automatic closing ...................................................................... Add connection builder and automatic closing This patch adds a new "ConnectionBuilder" class that simplifies the task of creating multiple connections with the same parameters. It also adds implementations of the "__enter__" and "__exit__" methods to the "Connection" class, as this will be typically combined with the use of multiple connections. Change-Id: I0cd00228dcd1671b16edbefb1dd23c9c3932bd0f Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A sdk/examples/connection_builder.py M sdk/lib/ovirtsdk4/__init__.py M sdk/lib/ovirtsdk4/http.py 3 files changed, 101 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54662 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0cd00228dcd1671b16edbefb1dd23c9c3932bd0f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Update to version 4.0.9 of the model
by juan.hernandez＠redhat.com 11 Mar '16

11 Mar '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Update to version 4.0.9 of the model ...................................................................... restapi: Update to version 4.0.9 of the model This update is required in order to fix the URL of the OpenStack image providers service. Change-Id: Ia4226fbea48098de3347eba29a6dc08aa456321a 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/BackendApiResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostNicResource.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/v3/servers/V3HostNicServer.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/servers/V3HostNicsServer.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/servers/V3SystemServer.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostNicResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostNicsResourceTest.java M pom.xml 9 files changed, 6 insertions(+), 916 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests Martin Mucha: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/54659 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia4226fbea48098de3347eba29a6dc08aa456321a Gerrit-PatchSet: 2 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: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0