- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-4.0]: restapi: Move Quota from Vm to VmBase
by piotr.kliczewski＠gmail.com 11 Oct '16

11 Oct '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: restapi: Move Quota from Vm to VmBase ...................................................................... restapi: Move Quota from Vm to VmBase Moving Quota from Vm to VmBase in the REST API model adds quota parameter to the Template in the API. Change-Id: I82a207be205e0e4f6c5740b7c9eaa54caafbc838 Bug-Url: https://bugzilla.redhat.com/1379802 Signed-off-by: Andrej Krejcir <akrejcir(a)redhat.com> --- M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmBaseMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 2 files changed, 12 insertions(+), 9 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Roman Mohr: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Andrej Krejcir: Verified -- To view, visit https://gerrit.ovirt.org/65168 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I82a207be205e0e4f6c5740b7c9eaa54caafbc838 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Automatic openstack host deploy warning
by piotr.kliczewski＠gmail.com 11 Oct '16

11 Oct '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: webadmin: Automatic openstack host deploy warning ...................................................................... webadmin: Automatic openstack host deploy warning Choosing to deploy Neutron provider on host should warn user, this is may not work and it is recommended to manually deploy the OVS agent on the host. Change-Id: I414100a6902a1374c9cc5a74c2c9487bee67aba9 Bug-Url: https://bugzilla.redhat.com/1377783 Signed-off-by: mirecki <mmirecki(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/CommonApplicationConstants.properties M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/provider/HostNetworkProviderWidget.ui.xml 3 files changed, 23 insertions(+), 0 deletions(-) Approvals: Jakub Niedermertl: Looks good to me, but someone else must approve Marcin Mirecki: Verified Martin Mucha: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Vojtech Szocs: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/65315 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I414100a6902a1374c9cc5a74c2c9487bee67aba9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[sdk_4.0]: Accept 202 code for 'Add' methods
by omachace＠redhat.com 11 Oct '16

11 Oct '16

Ondra Machacek has submitted this change and it was merged. Change subject: Accept 202 code for 'Add' methods ...................................................................... Accept 202 code for 'Add' methods Currently the SDK doesn't accept the HTTP 202 code as a valid result of 'Add' methods, it is valid return code, so SDK needs to accept it. Change-Id: I8b9054848f47e29fe71d645025f047d13de93c39 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/java/ServicesImplGenerator.java A sdk/src/test/java/org/ovirt/engine/sdk4/ResponseCodeTest.java 2 files changed, 72 insertions(+), 1 deletion(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65349 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8b9054848f47e29fe71d645025f047d13de93c39 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-java[master]: Accept 202 code for 'Add' methods
by omachace＠redhat.com 11 Oct '16

11 Oct '16

Ondra Machacek has submitted this change and it was merged. Change subject: Accept 202 code for 'Add' methods ...................................................................... Accept 202 code for 'Add' methods Currently the SDK doesn't accept the HTTP 202 code as a valid result of 'Add' methods, it is valid return code, so SDK needs to accept it. Change-Id: I8b9054848f47e29fe71d645025f047d13de93c39 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/java/ServicesImplGenerator.java A sdk/src/test/java/org/ovirt/engine/sdk4/ResponseCodeTest.java M sdk/src/test/java/org/ovirt/engine/sdk4/VmsServiceTest.java 3 files changed, 72 insertions(+), 2 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/65345 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8b9054848f47e29fe71d645025f047d13de93c39 Gerrit-PatchSet: 3 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[sdk_4.0]: Accept 200 code for 'Add' methods
by omachace＠redhat.com 11 Oct '16

11 Oct '16

Ondra Machacek has submitted this change and it was merged. Change subject: Accept 200 code for 'Add' methods ...................................................................... Accept 200 code for 'Add' methods Currently the SDK doesn't accept the HTTP 200 code as a valid result of 'Add' methods, but some methods in the server return this code, for example the method that adds a cluster quota limit. This patch changes the SDK so that accepts it. Change-Id: I59e2a5748bd4fc914a33d0431eacca6cdae543b9 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> (cherry picked from commit 33ec13f1bb1124674b4007f91b0c57cc0b97bb8f) --- M generator/src/main/java/org/ovirt/sdk/python/ServicesGenerator.java M sdk/tests/test_vm_service.py 2 files changed, 25 insertions(+), 1 deletion(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65346 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I59e2a5748bd4fc914a33d0431eacca6cdae543b9 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[master]: Accept 200 code for 'Add' methods
by omachace＠redhat.com 11 Oct '16

11 Oct '16

Ondra Machacek has submitted this change and it was merged. Change subject: Accept 200 code for 'Add' methods ...................................................................... Accept 200 code for 'Add' methods Currently the SDK doesn't accept the HTTP 200 code as a valid result of 'Add' methods, but some methods in the server return this code, for example the method that adds a cluster quota limit. This patch changes the SDK so that accepts it. Change-Id: I59e2a5748bd4fc914a33d0431eacca6cdae543b9 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/python/ServicesGenerator.java M sdk/tests/test_vm_service.py 2 files changed, 25 insertions(+), 1 deletion(-) 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/65344 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I59e2a5748bd4fc914a33d0431eacca6cdae543b9 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-guest-agent[ovirt-4.0]: Report session start and stop on all Guest OSes
by vfeenstr＠redhat.com 11 Oct '16

11 Oct '16

Vinzenz Feenstra has submitted this change and it was merged. Change subject: Report session start and stop on all Guest OSes ...................................................................... Report session start and stop on all Guest OSes Previously the session shutdown was reported on Windows only and the session startup wasn't reported at all. This patch fixes this missing feature. Change-Id: Iddf478c6d961dad60c2095762654d1bf4c33c25e Backport-To: 4.0 Bug-Url: https://bugzilla.redhat.com/1341106 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> (cherry picked from commit d89d9619f537fa446ae5b5e0f1cc794b0f2ba243) --- M ovirt-guest-agent/OVirtAgentLogic.py M ovirt-guest-agent/OVirtGuestService.py 2 files changed, 19 insertions(+), 7 deletions(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/65342 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iddf478c6d961dad60c2095762654d1bf4c33c25e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-guest-agent[master]: Report session start and stop on all Guest OSes
by vfeenstr＠redhat.com 11 Oct '16

11 Oct '16

Vinzenz Feenstra has submitted this change and it was merged. Change subject: Report session start and stop on all Guest OSes ...................................................................... Report session start and stop on all Guest OSes Previously the session shutdown was reported on Windows only and the session startup wasn't reported at all. This patch fixes this missing feature. Change-Id: Iddf478c6d961dad60c2095762654d1bf4c33c25e Backport-To: 4.0 Bug-Url: https://bugzilla.redhat.com/1341106 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M ovirt-guest-agent/OVirtAgentLogic.py M ovirt-guest-agent/OVirtGuestService.py 2 files changed, 19 insertions(+), 7 deletions(-) Approvals: Jenkins CI: Passed CI tests Vinzenz Feenstra: Verified; Looks good to me, approved Milan Zamazal: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/64994 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iddf478c6d961dad60c2095762654d1bf4c33c25e Gerrit-PatchSet: 6 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Milan Zamazal <mzamazal(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-sdk-ruby[sdk_4.0]: Install required gems before Travis build
by juan.hernandez＠redhat.com 11 Oct '16

11 Oct '16

Juan Hernandez has submitted this change and it was merged. Change subject: Install required gems before Travis build ...................................................................... Install required gems before Travis build This patch changes the .travis.yml file so that the required Ruby gems (rake-compiler and rspec) are installed before startig the build. Change-Id: Ie98693afd35e1e581a2d987417b969197ec24f46 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit c8d1f1b094ab322a008dad921d734d272bf124c6) --- M .travis.yml 1 file changed, 5 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/65338 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie98693afd35e1e581a2d987417b969197ec24f46 Gerrit-PatchSet: 1 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-ruby[sdk_4.0]: Add Travis CI for Mac OS
by juan.hernandez＠redhat.com 11 Oct '16

11 Oct '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add Travis CI for Mac OS ...................................................................... Add Travis CI for Mac OS This patch adds a configuration file that enables running the build in the Travis CI environment and Mac OS. Change-Id: Ib8ffd2f6966bc75db02561fb57e056677629b3c5 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 47d3ea5489c6225313f5dc839a4972570af2728e) --- A .travis.yml 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/65337 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib8ffd2f6966bc75db02561fb57e056677629b3c5 Gerrit-PatchSet: 1 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