- Engine-commits - oVirt List Archives

Change in ovirt-engine-cli[master]: cli: introducing new state UNAUTHORIZED
by mpastern＠redhat.com 22 Oct '13

22 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: introducing new state UNAUTHORIZED ...................................................................... cli: introducing new state UNAUTHORIZED AUTHENTICATION_ERROR can occur when authentication session expires, this is yet another recoverable error that exposed via own state - UNAUTHORIZED Change-Id: I287743f00ba34997d7e2bbc1aaf5a65c86038ade Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/cli/context.py M src/ovirtcli/command/connect.py M src/ovirtcli/prompt.py M src/ovirtcli/settings.py M src/ovirtcli/shell/engineshell.py 5 files changed, 57 insertions(+), 29 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/20245 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I287743f00ba34997d7e2bbc1aaf5a65c86038ade Gerrit-PatchSet: 3 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 visualization effects
by mpastern＠redhat.com 22 Oct '13

22 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: implement visualization effects ...................................................................... cli: implement visualization effects Change-Id: I074e6d03f46f42261f96a71024fe28f871bd0c65 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/cli/command/command.py M src/cli/context.py M src/cli/error.py M src/cli/messages.py A src/cli/warning.py M src/ovirtcli/shell/engineshell.py A src/ovirtcli/utils/colorhelper.py 7 files changed, 182 insertions(+), 20 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/20147 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I074e6d03f46f42261f96a71024fe28f871bd0c65 Gerrit-PatchSet: 3 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: validate args in /connect command
by mpastern＠redhat.com 22 Oct '13

22 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: validate args in /connect command ...................................................................... cli: validate args in /connect command Change-Id: I785672abf85b317379604e9b30db0687b8a9653d Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/ovirtcli/command/connect.py 1 file changed, 14 insertions(+), 8 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/20141 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I785672abf85b317379604e9b30db0687b8a9653d 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: expose api capabilities in cli 2/2 #891227
by mpastern＠redhat.com 22 Oct '13

22 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: expose api capabilities in cli 2/2 #891227 ...................................................................... cli: expose api capabilities in cli 2/2 #891227 add indentation to command output Bug-Url: https://bugzilla.redhat.com/891227 Change-Id: I035a4a4f6bf45d237c5751473bb0a9b424b7c2d1 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/ovirtcli/command/capabilities.py 1 file changed, 2 insertions(+), 3 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/20139 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I035a4a4f6bf45d237c5751473bb0a9b424b7c2d1 Gerrit-PatchSet: 5 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[ovirt-engine-3.3]: engine: Prevent empty MAC Address range
by iheim＠redhat.com 22 Oct '13

22 Oct '13

Itamar Heim has submitted this change and it was merged. Change subject: engine: Prevent empty MAC Address range ...................................................................... engine: Prevent empty MAC Address range The patch prevents a user from entering a mac address range which might result in an empty pool, for example a range which contains multicast mac addresses only. Change-Id: I24113379e8fbbc15a63bdb1be4e6c719d7cb764f Bug-Url: https://bugzilla.redhat.com/1011912 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/MacPoolManager.java A backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/MacAddressRangeUtils.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/config/entity/helper/MacAddressPoolRangesValueHelper.java M backend/manager/tools/src/test/java/org/ovirt/engine/core/config/entity/helper/MacAddressPoolRangesValueHelperTest.java 4 files changed, 73 insertions(+), 73 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20177 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I24113379e8fbbc15a63bdb1be4e6c719d7cb764f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[node-3.0]: ovirt-node-plugin-vdsm.spec: Bump release
by dougsland＠redhat.com 21 Oct '13

21 Oct '13

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: ovirt-node-plugin-vdsm.spec: Bump release ...................................................................... ovirt-node-plugin-vdsm.spec: Bump release Changes: - engine_page: Add validation for cert remove Change-Id: If11c82702217245ec84b432e32f924f441ed849e Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M ovirt-node-plugin-vdsm.spec.in 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20373 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If11c82702217245ec84b432e32f924f441ed849e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: node-3.0 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)fedoraproject.org>

1 0

Change in ovirt-node-plugin-vdsm[master]: engine_page: replace log import
by dougsland＠redhat.com 21 Oct '13

21 Oct '13

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: engine_page: replace log import ...................................................................... engine_page: replace log import Due http://gerrit.ovirt.org/#/c/18927/ that add a log module we must update engine_page. Change-Id: Ie8e07b94ae18e1e7fd0c89b9b005f8eb5c0fe7eb Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1019859 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/engine_page.py 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved Ryan Barry: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/20244 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie8e07b94ae18e1e7fd0c89b9b005f8eb5c0fe7eb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)fedoraproject.org> Gerrit-Reviewer: Michael Burns <mburns(a)redhat.com> Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Add vm name to quota message
by dfediuck＠redhat.com 21 Oct '13

21 Oct '13

Doron Fediuck has submitted this change and it was merged. Change subject: webadmin: Add vm name to quota message ...................................................................... webadmin: Add vm name to quota message When a VM was created with no quota in an audit/enforcing DC (using REST for instance), the error message did not include the name of the offending vm Change-Id: I5a55b549f0127fde8b0afd6daae2ac06e0218e3d Bug-Url: https://bugzilla.redhat.com/949576 Signed-off-by: Noam Slomianko <nslomian(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/quota/QuotaManager.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 2 files changed, 4 insertions(+), 1 deletion(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19469 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5a55b549f0127fde8b0afd6daae2ac06e0218e3d Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-scheduler-proxy[master]: Samples: add memory based load-balancing sample module
by dfediuck＠redhat.com 21 Oct '13

21 Oct '13

Doron Fediuck has submitted this change and it was merged. Change subject: Samples: add memory based load-balancing sample module ...................................................................... Samples: add memory based load-balancing sample module Adds a comprehensive example of a memory filter. Logic stages: - find the host with the least amount of free memory - check what is the maximum vm size it can pass to the other hosts without overloading them - find a vm it can pass that is small enough to run on one of the other servers. If safe_selection = true, return the smallest vm else try and find one that will solve the overutilization in one migration Change-Id: I1b4a41064631912f5624723327a2ff5ebf158d5c Signed-off-by: Noam Slomianko <nslomian(a)redhat.com> --- A plugins/examples/host_memory_balance.py 1 file changed, 165 insertions(+), 0 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19934 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1b4a41064631912f5624723327a2ff5ebf158d5c Gerrit-PatchSet: 8 Gerrit-Project: ovirt-scheduler-proxy Gerrit-Branch: master Gerrit-Owner: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: core:Upgrade script missing field in vm_template
by iheim＠redhat.com 21 Oct '13

21 Oct '13

Itamar Heim has submitted this change and it was merged. Change subject: core:Upgrade script missing field in vm_template ...................................................................... core:Upgrade script missing field in vm_template There are 2 updates that have wrong numbering in 3.3.1 Beta : 1) 03_03_0650_add_signle_qxl_pci_column.sql => this change will not be installed in the upgrade since the latest change was 0680 2) 03_03_0680_add_cpu_shares_to_vm.sql => this script will also be skipped The fix to that is to add the following scripts to 3.3.1.Beta which are exact copies of 1) and 2) from above as : 03_03_0681_add_cpu_shares_to_vm.sql 03_03_0682_add_signle_qxl_pci_column.sql Change-Id: I5e30b76c4ade4706e5340eb1d7da0275eb8afb43 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1005533 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- A packaging/dbscripts/upgrade/03_03_0681_add_cpu_shares_to_vm.sql A packaging/dbscripts/upgrade/03_03_0682_add_signle_qxl_pci_column.sql 2 files changed, 2 insertions(+), 0 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20348 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5e30b76c4ade4706e5340eb1d7da0275eb8afb43 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(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: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0