- Engine-commits - oVirt List Archives

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

Change in ovirt-engine[master]: webadmin: 'remove' confirmation dialog message cleanup
by ecohen＠redhat.com 28 Oct '13

28 Oct '13

Einav Cohen has submitted this change and it was merged. Change subject: webadmin: 'remove' confirmation dialog message cleanup ...................................................................... webadmin: 'remove' confirmation dialog message cleanup Before this patch, 'remove' confirmation dialog message contained a default message that could have been overridden only if the dialog's HashName didn't start with "remove_". In this patch: * 'remove' confirmation dialog still contains a default message, but now it can always be overridden, regardless of the dialog's hashName value. * Cleaned up (now unnecessary) HashName manipulation in RemoveNetworkQoSModel. * Cleaned up useless calls to set dialog's message (which were probably useful up until http://gerrit.ovirt.org/#/c/14621/ has been merged) in order to prevent regressions such as displaying "Virtual Machines" as the dialog's content instead of the default "Are you sure you want to remove the following items?" message. * Removed the related (now unused) UI constants. Change-Id: I7411dec0eaf8d60cefff563988618f7c548eecce Bug-Url: https://bugzilla.redhat.com/1021584 Signed-off-by: Greg Sheremeta <gshereme(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/RemoveConfirmationPopupView.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/bookmarks/BookmarkListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/PermissionListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/SystemPermissionListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/roles_ui/RoleListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/roles_ui/RolePermissionListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/scheduling/ClusterPolicyListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/RemoveNetworkQoSModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/RemoveNetworksModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/disks/DiskListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/RemoveProvidersModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/quota/QuotaListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/quota/QuotaUserListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageDiskListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/TemplateBackupModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/VmBackupModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/tags/TagListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateStorageListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/users/UserListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/users/UserPermissionListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/RemoveVmInterfaceModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/RemoveVnicProfileModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmDiskListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java 32 files changed, 3 insertions(+), 114 deletions(-) Approvals: Einav Cohen: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20471 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7411dec0eaf8d60cefff563988618f7c548eecce Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-cli[master]: cli: /disconnect on exit only if not already disconnected
by mpastern＠redhat.com 28 Oct '13

28 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: /disconnect on exit only if not already disconnected ...................................................................... cli: /disconnect on exit only if not already disconnected Change-Id: I7fa77a1fdd4db73c84c398d8c4ab7499cd967250 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/ovirtcli/shell/engineshell.py 1 file changed, 6 insertions(+), 2 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/20608 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7fa77a1fdd4db73c84c398d8c4ab7499cd967250 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-engine-cli[master]: cli: implement origin state in FiniteStateMachine
by mpastern＠redhat.com 28 Oct '13

28 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: implement origin state in FiniteStateMachine ...................................................................... cli: implement origin state in FiniteStateMachine Change-Id: If2234549d553db1fe52d681da99de67451e7a125 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/ovirtcli/state/finitestatemachine.py 1 file changed, 8 insertions(+), 0 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/20607 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If2234549d553db1fe52d681da99de67451e7a125 Gerrit-PatchSet: 1 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-engine-cli[master]: cli: cleanup
by mpastern＠redhat.com 28 Oct '13

28 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: cleanup ...................................................................... cli: cleanup Change-Id: Ia26a327492c4b45f6fba4ade55abb8bb821c21e5 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/cli/messages.py M src/cli/settings.py M src/ovirtcli/command/help.py M src/ovirtcli/command/ping.py A src/ovirtcli/infrastructure/__init__.py R src/ovirtcli/infrastructure/context.py R src/ovirtcli/infrastructure/historymanager.py R src/ovirtcli/infrastructure/object.py R src/ovirtcli/infrastructure/options.py R src/ovirtcli/infrastructure/settings.py M src/ovirtcli/main.py M src/ovirtcli/shell/cmdshell.py M src/ovirtcli/shell/connectcmdshell.py M src/ovirtcli/shell/engineshell.py M src/ovirtcli/shell/promptmanager.py R src/ovirtcli/shell/promptmode.py 16 files changed, 21 insertions(+), 23 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/20577 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia26a327492c4b45f6fba4ade55abb8bb821c21e5 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-engine-cli[master]: cli: do not recover prompt while still in error
by mpastern＠redhat.com 28 Oct '13

28 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: do not recover prompt while still in error ...................................................................... cli: do not recover prompt while still in error Change-Id: Idd3ec528be0c82c4f7b04f234554210329c10d3e Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/ovirtcli/shell/engineshell.py 1 file changed, 10 insertions(+), 4 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/20575 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idd3ec528be0c82c4f7b04f234554210329c10d3e Gerrit-PatchSet: 1 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