October 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.3]: provider: glance images listing size and limits
by iheim＠redhat.com 28 Oct '13

28 Oct '13

Itamar Heim has submitted this change and it was merged. Change subject: provider: glance images listing size and limits ...................................................................... provider: glance images listing size and limits This patch adds two configuration parameters to control the glance image listing. In fact OpenStack glance servers by default return a limited number of images (limit_param_default) and it's up to the client to either request a longer list (using the "limit" parameter) or to iterate with multiple requests. GlanceImageListSize defines the number of images that should be returned in each request (20 as the default in glance client). GlanceImageTotalListSize defines the total number of images that should be transferred from the server (with multiple requests if necessary). Change-Id: I7b534caccbdcf121b81571384e3420f1a6efde00 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1013619 Signed-off-by: Federico Simoncelli <fsimonce(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/IsoDomainListSyncronizer.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/OpenStackImageProviderProxy.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 7 files changed, 54 insertions(+), 7 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20626 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7b534caccbdcf121b81571384e3420f1a6efde00 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: packaging: engine-backup: Safer password input
by iheim＠redhat.com 28 Oct '13

28 Oct '13

Itamar Heim has submitted this change and it was merged. Change subject: packaging: engine-backup: Safer password input ...................................................................... packaging: engine-backup: Safer password input Allow passing db password also interactively and using an environment variable, in addition to using a file and using a command line option. Bug-Url: https://bugzilla.redhat.com/1002401 Change-Id: Icbc497af2653417beab54a9be307332cdf69a48a Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/bin/engine-backup.sh 1 file changed, 36 insertions(+), 2 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20606 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icbc497af2653417beab54a9be307332cdf69a48a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-cli[master]: cli: ovirt-shell does not exit when using -f option(#854519)
by mpastern＠redhat.com 28 Oct '13

28 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: ovirt-shell does not exit when using -f option(#854519) ...................................................................... cli: ovirt-shell does not exit when using -f option(#854519) ovirt-shell does not exit when -f option is used. This option is used by system scripts and it would be useful to exit the ovirt-shell upon completion of the command execution. Change-Id: I5441a401ed39fc7148b6001fb826b145ba66d6c7 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=854519 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M src/ovirtcli/shell/engineshell.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12212 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5441a401ed39fc7148b6001fb826b145ba66d6c7 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>

1 0

Change in ovirt-hosted-engine-ha[master]: packaging: distribute doc in tarball
by gpadgett＠redhat.com 28 Oct '13

28 Oct '13

Greg Padgett has submitted this change and it was merged. Change subject: packaging: distribute doc in tarball ...................................................................... packaging: distribute doc in tarball Change-Id: I55e893f3e9d25edaa79bcaeff74f6dc8c9bb079a Bug-Url: https://bugzilla.redhat.com/1021064 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M Makefile.am M configure.ac A doc/Makefile.am 3 files changed, 30 insertions(+), 1 deletion(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20306 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I55e893f3e9d25edaa79bcaeff74f6dc8c9bb079a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-dwh[master]: packaging: setup: clear instructions for creating remote DB
by ydary＠redhat.com 28 Oct '13

28 Oct '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: clear instructions for creating remote DB ...................................................................... packaging: setup: clear instructions for creating remote DB Change-Id: I67822260ebb963e6be1fd94921a1456b5a1ecf3b Bug-Url: https://bugzilla.redhat.com/1021560 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/ovirt-engine-dwh-setup.py 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Alex Lourie: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20576 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I67822260ebb963e6be1fd94921a1456b5a1ecf3b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: implement CLI invocation capabilities #895559
by mpastern＠redhat.com 28 Oct '13

28 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: implement CLI invocation capabilities #895559 ...................................................................... cli: implement CLI invocation capabilities #895559 implemented by -E/--execute-command option, e.g: [mpastern@laptop ~]$ ovirt-shell --execute-command "list vms;list clusters" [oVirt shell (connected)]# list vms id : 77fd5b88-bcd6-4296-bf20-f3d857f7f0d2 name : test [oVirt shell (connected)]# list clusters id : 99408929-82cf-4dc7-a532-9d998063fa95 name : Default description: The default server cluster [mpastern@laptop ~]$ [mpastern@laptop ~]$ [mpastern@laptop ~]$ Change-Id: Ie1d31b3321441b7efef6a27f6223ef96c803f1e5 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=895559 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/cli/context.py M src/ovirtcli/infrastructure/options.py M src/ovirtcli/infrastructure/settings.py M src/ovirtcli/shell/engineshell.py 4 files changed, 67 insertions(+), 21 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ravi Nori: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/20618 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie1d31b3321441b7efef6a27f6223ef96c803f1e5 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>

1 0

Change in ovirt-image-uploader[master]: packaging: spec: depends on python-lxml
by sbonazzo＠redhat.com 28 Oct '13

28 Oct '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: spec: depends on python-lxml ...................................................................... packaging: spec: depends on python-lxml Added dep on python-lxml: it's needed also for unit testing. Change-Id: I2a356c1dacfd1ca9c366ba45eddf117abdbeed44 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-image-uploader.spec.in 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Kiril Nesenko: Looks good to me, but someone else must approve Keith Robertson: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/20609 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2a356c1dacfd1ca9c366ba45eddf117abdbeed44 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.0]: bin: add --vm-start-paused to hosted-engine
by sbonazzo＠redhat.com 28 Oct '13

28 Oct '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: bin: add --vm-start-paused to hosted-engine ...................................................................... bin: add --vm-start-paused to hosted-engine Allow to start the hosted engine VM qith qemu paused. Change-Id: I749e877fe1fb259b99f1dfa46f580bcb1a5ec724 Bug-Url: https://bugzilla.redhat.com/1020858 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M man/hosted-engine.8 M src/bin/hosted-engine.in 2 files changed, 17 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20612 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I749e877fe1fb259b99f1dfa46f580bcb1a5ec724 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: core: ReconstructMasterParameters - return removed initializ...
by iheim＠redhat.com 28 Oct '13

28 Oct '13

Itamar Heim has submitted this change and it was merged. Change subject: core: ReconstructMasterParameters - return removed initialization ...................................................................... core: ReconstructMasterParameters - return removed initialization In change I3bb06f63b6fae8d4bc0c2b931cbfeb6b71a023a9, the initialization of the member privateNewMasterDomainId has been moved to the ctor. The issue is that the initialization was placed only in some of the ctors leading the unexpected behaviour when reconstruct is executed. Change-Id: I752c8065ee15faaeb6ffd41909d349297b8a3a58 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1023741 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/ReconstructMasterParameters.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20584 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I752c8065ee15faaeb6ffd41909d349297b8a3a58 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: agent: extend timeout for domain monitor acquisition
by gpadgett＠redhat.com 28 Oct '13

28 Oct '13

Greg Padgett has submitted this change and it was merged. Change subject: agent: extend timeout for domain monitor acquisition ...................................................................... agent: extend timeout for domain monitor acquisition Acquisition may take ~3 minutes, so extend timeout to 4. Change-Id: Ic586b1f11374632724cf71da5d3cac72eb83ca19 Bug-Url: https://bugzilla.redhat.com/1018849 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/constants.py.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20305 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic586b1f11374632724cf71da5d3cac72eb83ca19 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0