- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.5]: restapi: add use_sysprep and use_cloud_init for Vm creation
by oourfali＠redhat.com 22 Jul '15

22 Jul '15

Oved Ourfali has submitted this change and it was merged. Change subject: restapi: add use_sysprep and use_cloud_init for Vm creation ...................................................................... restapi: add use_sysprep and use_cloud_init for Vm creation By default we attach sysprep for new Windows VM - added new tags for the user to determine the behaviour, for backward compatibility the default is to attach sysprep for new Windows VMs. Change-Id: Iffc4c7ea50efd2ea332918e1b31594e3a949f4ee Bug-Url: https://bugzilla.redhat.com/1190663 Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/logging/Messages.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/resources/org/ovirt/engine/api/restapi/logging/Messages.properties 5 files changed, 28 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Looks good to me, approved Shahar Havivi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/43385 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iffc4c7ea50efd2ea332918e1b31594e3a949f4ee Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernández Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: Fixed issue when user group already exists
by dcaroest＠redhat.com 22 Jul '15

22 Jul '15

David Caro has submitted this change and it was merged. Change subject: Fixed issue when user group already exists ...................................................................... Fixed issue when user group already exists It was working until now because it did never exist on the vms Change-Id: I168d23f111c61aae1751d48deb342b08ff5e8b3b Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M mock_configs/mock_runner.sh 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Piotr Kliczewski: Looks good to me Barak Korren: Looks good to me David Caro: Verified; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/43865 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I168d23f111c61aae1751d48deb342b08ff5e8b3b Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>

1 0

Change in ovirt-engine[master]: Remove suppportedProtocols from getVdsCaps verification
by oourfali＠redhat.com 22 Jul '15

22 Jul '15

Oved Ourfali has submitted this change and it was merged. Change subject: Remove suppportedProtocols from getVdsCaps verification ...................................................................... Remove suppportedProtocols from getVdsCaps verification supportedProtocols parameters is not in used since ovirt-2.x https://gerrit.ovirt.org/43571 removes this value from vdsm report for getCaps. This following patch fix the test that handles it. Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com> Change-Id: I23c9bd7494a0d8daffc34ef5a883e0a455faf931 --- M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/jsonrpc/MarshallingTestCase.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Yaniv Bronhaim: Verified Oved Ourfali: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/43572 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I23c9bd7494a0d8daffc34ef5a883e0a455faf931 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: findbugs warning
by amureini＠redhat.com 22 Jul '15

22 Jul '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: findbugs warning ...................................................................... core: findbugs warning Fix findbugs warning resulting from a recent addition of exponential backoff to the CommandExecutor class. Change-Id: Ibae9da8ad479be440cfe96390349a07b4317e859 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandExecutor.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Ravi Nori: Looks good to me, but someone else must approve Greg Padgett: Verified Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43870 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibae9da8ad479be440cfe96390349a07b4317e859 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: decrease idle time for multi-phase CoCo commands
by amureini＠redhat.com 21 Jul '15

21 Jul '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: decrease idle time for multi-phase CoCo commands ...................................................................... core: decrease idle time for multi-phase CoCo commands Commands with multiple phases such as Live Merge can spend a lot of time idling, waiting on state transitions or child commands. This patch changes the CommandExecutor to lower the wait time between invoking callbacks when certain state changes occur such as a command status change to successful/failed or upon child command completion. Change-Id: I22ad7df3e6f6b85a72170db1ccb4ebac655eb576 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandExecutor.java 1 file changed, 10 insertions(+), 0 deletions(-) Approvals: Greg Padgett: Verified Ravi Nori: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43429 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I22ad7df3e6f6b85a72170db1ccb4ebac655eb576 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: capped exponential backoff for CoCo CommandExecutor
by amureini＠redhat.com 21 Jul '15

21 Jul '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: capped exponential backoff for CoCo CommandExecutor ...................................................................... core: capped exponential backoff for CoCo CommandExecutor Some commands run by CoCo complete quickly and would benefit from faster polling; however, some may take longer to complete and polling too often consumes unnecessary CPU. This is a good use case for capped exponential backoff, which is implemented here purely in-memory. New commands will poll 1 second, doubling until reaching the AsyncCommandPollingRateInSeconds configuration value. Change-Id: Id89929ba0d8aa8e80ae358acbe7cc763a92be135 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandExecutor.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/BackwardCompatibilityTaskCreationTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 4 files changed, 36 insertions(+), 8 deletions(-) Approvals: Greg Padgett: Verified Ravi Nori: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43428 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id89929ba0d8aa8e80ae358acbe7cc763a92be135 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: set description for cinder disk snapshot
by derez＠redhat.com 21 Jul '15

21 Jul '15

Daniel Erez has submitted this change and it was merged. Change subject: core: set description for cinder disk snapshot ...................................................................... core: set description for cinder disk snapshot Set description for snapshot in createSnapshot method (CinderBroker). Change-Id: I8554ffec9341f34e5eea8c1756490fab6699065c Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/CinderBroker.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/CreateCinderSnapshotCommand.java 3 files changed, 4 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Verified Maor Lipchuk: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43860 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8554ffec9341f34e5eea8c1756490fab6699065c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: vdsm-jsonrpc-java: new repo required by slf4j
by dcaroest＠redhat.com 21 Jul '15

21 Jul '15

David Caro has submitted this change and it was merged. Change subject: vdsm-jsonrpc-java: new repo required by slf4j ...................................................................... vdsm-jsonrpc-java: new repo required by slf4j When we build vdsm-jsonrpc-java using slf4j we need 1.7+ version which is available in jpackage repo. Bug-Url: https://bugzilla.redhat.com/1243339 Change-Id: Ib4bc76b3d02b0118362e6be8b48753b5a25d0d27 Required-For: https://gerrit.ovirt.org/#/c/43775/ Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> --- M jobs/confs/yaml/jobs/vdsm-jsonrpc-java/vdsm-jsonrpc-java_create-rpms.yaml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Piotr Kliczewski: Verified Martin Peřina: Looks good to me Jenkins CI: Passed CI tests David Caro: Ready for merge -- To view, visit https://gerrit.ovirt.org/43823 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib4bc76b3d02b0118362e6be8b48753b5a25d0d27 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: RadioboxCell - ensure onBrowserEvent invocation
by derez＠redhat.com 21 Jul '15

21 Jul '15

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: RadioboxCell - ensure onBrowserEvent invocation ...................................................................... webadmin: RadioboxCell - ensure onBrowserEvent invocation RadioboxCell- currently, onBrowserEvent method isn't invoked (which, for example, prevents FieldUpdater from updating...). Hence, added the default method signature of onBrowserEvent to ensure invocation. More specifically, this fix is required for the FieldUpdater of 'vmConfColumn' in VmSnapshotCustomPreviewPopupWidget. Change-Id: I9a3686a6513778e70c7029ff5c991304a0eee3e9 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/cell/RadioboxCell.java 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Verified Greg Sheremeta: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42724 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9a3686a6513778e70c7029ff5c991304a0eee3e9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: add frozen flag to VdsServer snapshot api
by derez＠redhat.com 21 Jul '15

21 Jul '15

Daniel Erez has submitted this change and it was merged. Change subject: core: add frozen flag to VdsServer snapshot api ...................................................................... core: add frozen flag to VdsServer snapshot api Added 'frozen' boolean to 'snapshot' api in order to indicate whether the VM has been frozen. VDSM uses that information to decide if 'quiesce' flag should be sent to libvirt (see https://gerrit.ovirt.org/43293) Change-Id: I979659a744351b65362274b39027616dabfa099c Bug-Url: https://bugzilla.redhat.com/1240636 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/SnapshotVDSCommandParameters.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SnapshotVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerConnector.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java 7 files changed, 51 insertions(+), 13 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Verified Maor Lipchuk: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43803 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I979659a744351b65362274b39027616dabfa099c Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0