- Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.2]: Update engine's mgmt net to have the correct vlan id
by sbonazzo＠redhat.com 03 Sep '14

03 Sep '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: Update engine's mgmt net to have the correct vlan id ...................................................................... Update engine's mgmt net to have the correct vlan id Up until now, when setting the management network over a vlan, the logical network in the engine would not be automatically updated to match the real configuration. This addresses the issue. Change-Id: Ica4c5804395d364438f1e142387376cc7adc7172 Bug-Url: https://bugzilla.redhat.com/1124207 Signed-off-by: Antoni S. Puimedon <asegurap(a)redhat.com> (cherry picked from commit a6bd8963d3d82af57224227e5f2af7aef6cd3492) --- M src/ovirt_hosted_engine_setup/Makefile.am A src/ovirt_hosted_engine_setup/vds_info.py M src/plugins/ovirt-hosted-engine-setup/engine/add_host.py M src/plugins/ovirt-hosted-engine-setup/network/bridge.py 4 files changed, 109 insertions(+), 42 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32374 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ica4c5804395d364438f1e142387376cc7adc7172 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.2 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Antoni Segura Puimedon <asegurap(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: core: trailing spaces removed from dstSdUUID
by tnisan＠redhat.com 03 Sep '14

03 Sep '14

Tal Nisan has submitted this change and it was merged. Change subject: core: trailing spaces removed from dstSdUUID ...................................................................... core: trailing spaces removed from dstSdUUID Image.move fails due to trailing spaces in dstSdUUID. Removing them fixes marshalling issue. Bug-Url: https://bugzilla.redhat.com/1126531 Change-Id: I7d6b2b51a4f65fa23c1723e262a58deabaeaea83 Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcIIrsServer.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Piotr Kliczewski: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32359 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7d6b2b51a4f65fa23c1723e262a58deabaeaea83 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[master]: Update engine's mgmt net to have the correct vlan id
by sbonazzo＠redhat.com 03 Sep '14

03 Sep '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: Update engine's mgmt net to have the correct vlan id ...................................................................... Update engine's mgmt net to have the correct vlan id Up until now, when setting the management network over a vlan, the logical network in the engine would not be automatically updated to match the real configuration. This addresses the issue. Change-Id: Ica4c5804395d364438f1e142387376cc7adc7172 Bug-Url: https://bugzilla.redhat.com/1124207 Signed-off-by: Antoni S. Puimedon <asegurap(a)redhat.com> --- M src/ovirt_hosted_engine_setup/Makefile.am A src/ovirt_hosted_engine_setup/vds_info.py M src/plugins/ovirt-hosted-engine-setup/engine/add_host.py M src/plugins/ovirt-hosted-engine-setup/network/bridge.py 4 files changed, 109 insertions(+), 42 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/32269 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ica4c5804395d364438f1e142387376cc7adc7172 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Antoni Segura Puimedon <asegurap(a)redhat.com> Gerrit-Reviewer: Antoni Segura Puimedon <asegurap(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(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[master]: webadmin: Make reinstall dialog slightly bigger
by lvernia＠redhat.com 03 Sep '14

03 Sep '14

Lior Vernia has submitted this change and it was merged. Change subject: webadmin: Make reinstall dialog slightly bigger ...................................................................... webadmin: Make reinstall dialog slightly bigger This to obviate a scroll bar that appears when selecting a provider. Change-Id: Icdaba03bcefb5f65ca4bdecb7008b48c7a0758d1 Bug-Url: https://bugzilla.redhat.com/1117399 Signed-off-by: Lior Vernia <lvernia(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostInstallPopupView.ui.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alona Kaplan: Looks good to me, approved Lior Vernia: Verified -- To view, visit http://gerrit.ovirt.org/31863 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icdaba03bcefb5f65ca4bdecb7008b48c7a0758d1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-live[master]: packaging: refactored on original SL6.5 kickstart
by sbonazzo＠redhat.com 03 Sep '14

03 Sep '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: refactored on original SL6.5 kickstart ...................................................................... packaging: refactored on original SL6.5 kickstart Refactored kickstart files in order to be closer to the original kickstart files taken from SL6.5. This allow us to understand the changes needed for oVirt Live in order to ease the rebase for upcoming 6.6 and existing 7.0. Also restored some lines from the original kickstart after asking Ohad: they were removed accidentally Change-Id: I0eecc9a5f9fb730f94c56f85141c0a725105daf9 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos/Makefile R centos/kickstart/ovirt-live-base.ks M centos/kickstart/ovirt-live-gnome.ks A centos/kickstart/ovirt-live-ovirt-custom.ks.in 4 files changed, 198 insertions(+), 76 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32113 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0eecc9a5f9fb730f94c56f85141c0a725105daf9 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Ohad Basan <obasan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Update hibernate-validator version
by masayag＠redhat.com 03 Sep '14

03 Sep '14

Moti Asayag has submitted this change and it was merged. Change subject: core: Update hibernate-validator version ...................................................................... core: Update hibernate-validator version Update hibernate-validator version to 4.2.0.Final in pom.xml according to the run-time environment (JBoss-7.1.1.Final) Change-Id: I6138000eba51c132f3362b874469c31ba45d642d Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32198 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6138000eba51c132f3362b874469c31ba45d642d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: extmgr: respect enabled flag
by yzaslavs＠redhat.com 03 Sep '14

03 Sep '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: extmgr: respect enabled flag ...................................................................... extmgr: respect enabled flag Topic: AAA Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1136346 Change-Id: I4acfe5261a5cfb4312f6b55ce48a857266b6643b Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/extensions-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionsManager.java 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32332 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4acfe5261a5cfb4312f6b55ce48a857266b6643b Gerrit-PatchSet: 1 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: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: trailing spaces removed from dstSdUUID
by oourfali＠redhat.com 03 Sep '14

03 Sep '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: trailing spaces removed from dstSdUUID ...................................................................... core: trailing spaces removed from dstSdUUID Image.move fails due to trailing spaces in dstSdUUID. Removing them fixes marshalling issue. Bug-Url: https://bugzilla.redhat.com/1126531 Change-Id: I7d6b2b51a4f65fa23c1723e262a58deabaeaea83 Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcIIrsServer.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Piotr Kliczewski: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32358 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7d6b2b51a4f65fa23c1723e262a58deabaeaea83 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: webadmin: 'Add Users and Groups' dialog the column size shou...
by tnisan＠redhat.com 03 Sep '14

03 Sep '14

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: 'Add Users and Groups' dialog the column size should be limited ...................................................................... webadmin: 'Add Users and Groups' dialog the column size should be limited The column with the long name looks long and we need to scroll right/left to see the other columns Change-Id: Ia3009bffce854cc99031ad6571affea328c437d1 Bug-Url: https://bugzilla.redhat.com/1126283 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/permissions/AbstractPermissionsPopupView.java 1 file changed, 12 insertions(+), 6 deletions(-) Approvals: Ravi Nori: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32334 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia3009bffce854cc99031ad6571affea328c437d1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: webadmin: Add UI field for Neutron physical network
by tnisan＠redhat.com 03 Sep '14

03 Sep '14

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Add UI field for Neutron physical network ...................................................................... webadmin: Add UI field for Neutron physical network The same UI field was being used for both the oVirt label and the Neutron physical network, and only its interpretation changed according to context, which was confusing. As usual, had to change some styling for things to look decent (some of it is related to unresolved Look & Feel feature issues). Change-Id: I6b90800cf5948b35283bef03d8e2f95384799c80 Bug-Url: https://bugzilla.redhat.com/1108069 Bug-Url: https://bugzilla.redhat.com/1121546 Signed-off-by: Lior Vernia <lvernia(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/EditNetworkModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/AbstractNetworkPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/AbstractNetworkPopupView.ui.xml M packaging/branding/ovirt.brand/ovirt-patternfly-compat.css 6 files changed, 46 insertions(+), 53 deletions(-) Approvals: Alona Kaplan: Looks good to me, approved Lior Vernia: Verified -- To view, visit http://gerrit.ovirt.org/31695 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6b90800cf5948b35283bef03d8e2f95384799c80 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0