November 2012 - Engine-commits - oVirt List Archives

Change in otopi[master]: packaging: update location of sources after server move
by Alon Bar-Lev 30 Nov '12

30 Nov '12

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: update location of sources after server move ...................................................................... packaging: update location of sources after server move Change-Id: If2dbd31da63801094757c6e971512897f006c451 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M otopi.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9602 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If2dbd31da63801094757c6e971512897f006c451 Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-host-deploy[master]: packaging: update location of sources after server move
by Alon Bar-Lev 30 Nov '12

30 Nov '12

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: update location of sources after server move ...................................................................... packaging: update location of sources after server move Change-Id: I486f9fdedcbc903978a2bfcdb6dc5ef3e7d8119a Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ovirt-host-deploy.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9601 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I486f9fdedcbc903978a2bfcdb6dc5ef3e7d8119a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: adding engine Maintenance and Prepare mode.
by emesika＠redhat.com 29 Nov '12

29 Nov '12

Eli Mesika has submitted this change and it was merged. Change subject: core: adding engine Maintenance and Prepare mode. ...................................................................... core: adding engine Maintenance and Prepare mode. Support for that was already merged in http://gerrit.ovirt.org/#/c/8669/10 Solution above assumed that configuration values can be reloaded while the engine is still running. While this is true when working from API, it is not working from the engine-config, therefor, we must take the config values directly from the database and not rely on the config cache. Change-Id: Iae5d1148c9c2868abd0061700bb9988242dd8faf Signed-off-by: Eli Mesika <emesika(a)redhat.com> Bug-Url:https://bugzilla.redhat.com/868639 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java 1 file changed, 12 insertions(+), 9 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9568 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iae5d1148c9c2868abd0061700bb9988242dd8faf Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Fix new/edit pool dialogs for ja locale
by derez＠redhat.com 29 Nov '12

29 Nov '12

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: Fix new/edit pool dialogs for ja locale ...................................................................... webadmin: Fix new/edit pool dialogs for ja locale - Change the layout of some components in the new & edit pool dialogs to use relative values for their locations instead of absolute values that way, the dialog's layout supports long japanese terms - Rename newPoolPrestartedVmsIcon style to prestartedVmsIcon as this style is used for edit dialog also - Fix some typos (incrase -> increase) Change-Id: I5b9ead456e355f66eb4d8363fe0706ee4552c9fb Bug-Url: https://bugzilla.redhat.com/870251 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml 1 file changed, 18 insertions(+), 25 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9364 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5b9ead456e355f66eb4d8363fe0706ee4552c9fb Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>

1 0

Change in ovirt-iso-uploader[master]: cleanup: style by pep8
by kroberts＠redhat.com 29 Nov '12

29 Nov '12

Keith Robertson has submitted this change and it was merged. Change subject: cleanup: style by pep8 ...................................................................... cleanup: style by pep8 Change-Id: I0f5abd5dc6123937550300c132ffc1590a3e3851 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/engine-iso-uploader.py 1 file changed, 634 insertions(+), 294 deletions(-) Approvals: Keith Robertson: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8854 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0f5abd5dc6123937550300c132ffc1590a3e3851 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com>

1 0

Change in ovirt-iso-uploader[master]: cleanup: code warning by pyflakes
by kroberts＠redhat.com 29 Nov '12

29 Nov '12

Keith Robertson has submitted this change and it was merged. Change subject: cleanup: code warning by pyflakes ...................................................................... cleanup: code warning by pyflakes Change-Id: I012880df9e9d6b81b52cf311c24385342fe956c6 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/engine-iso-uploader.py 1 file changed, 7 insertions(+), 13 deletions(-) Approvals: Keith Robertson: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8853 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I012880df9e9d6b81b52cf311c24385342fe956c6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com>

1 0

Change in ovirt-host-deploy[master]: bridge: temporary workaround for firewalld behaviour
by Alon Bar-Lev 29 Nov '12

29 Nov '12

Alon Bar-Lev has submitted this change and it was merged. Change subject: bridge: temporary workaround for firewalld behaviour ...................................................................... bridge: temporary workaround for firewalld behaviour Currently, addNetwork just hangs if firewalld is up while creating the bridge. So as temporary workaround stop firewalld before creating the bridge. Change-Id: Ib6c248a7c49492f7889fc71f11ae1dad1e3434ae Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=869963 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/plugins/ovirt-host-deploy/vdsm/bridge.py 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9588 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib6c248a7c49492f7889fc71f11ae1dad1e3434ae Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in otopi[master]: iptables: disable and stop firewalld to avoid conflict
by Alon Bar-Lev 29 Nov '12

29 Nov '12

Alon Bar-Lev has submitted this change and it was merged. Change subject: iptables: disable and stop firewalld to avoid conflict ...................................................................... iptables: disable and stop firewalld to avoid conflict Change-Id: I423846092615a25352539cb0abc19088408ac93e Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/plugins/otopi/network/iptables.py 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9587 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I423846092615a25352539cb0abc19088408ac93e Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-engine[master]: bootstrap: ovirt-node upgrade: create destination directory
by yzaslavs＠redhat.com 29 Nov '12

29 Nov '12

Yair Zaslavsky has submitted this change and it was merged. Change subject: bootstrap: ovirt-node upgrade: create destination directory ...................................................................... bootstrap: ovirt-node upgrade: create destination directory One would expect that there will be an ovirt-node upgrade script, however there is none, and the /data/updates is vdsm specific path, which was created by vdsm init script, and then creation was moved to vdsm-reg service. vdsm-reg is going to be retired soon, so move the creation of this directory to the component that sends the iso image. Better solution would be a node upgrade script that receives the image via stdin. Change-Id: I5519bd63383feba9b3bf971543b0d630c24bbca0 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/OVirtNodeUpgrade.java 1 file changed, 31 insertions(+), 0 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9524 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5519bd63383feba9b3bf971543b0d630c24bbca0 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: frontend: VM template change shouldn't change selected clust...
by tjelinek＠redhat.com 29 Nov '12

29 Nov '12

Tomas Jelinek has submitted this change and it was merged. Change subject: frontend: VM template change shouldn't change selected cluster ...................................................................... frontend: VM template change shouldn't change selected cluster Changing VM template using combobox should not change selected cluster inside new virtual machine dialog when multiple clusters and templates are available inside one DS. Initially, the VM template has a binding to a DS and a cluster, but the VM template should be available across clusters of the same DS now. Change-Id: I23bf4aaa9f16d23996b2a28ef778748c1e622e38 Bug-Url: https://bugzilla.redhat.com/877981 Signed-off-by: Libor Spevak <lspevak(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewVmModelBehavior.java 1 file changed, 0 insertions(+), 7 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Libor Spevak: Verified -- To view, visit http://gerrit.ovirt.org/9512 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I23bf4aaa9f16d23996b2a28ef778748c1e622e38 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Libor Spevak <lspevak(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Libor Spevak <lspevak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>

1 0