March 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: Adding config entry for VDSM 4.14
by yzaslavs＠redhat.com 05 Mar '14

05 Mar '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: Adding config entry for VDSM 4.14 ...................................................................... Adding config entry for VDSM 4.14 Update SupportedVDSMVersions config value to support VDSM 4.14 Change-Id: I4e661ad31afb5599d4d7dd6370e5c9fb98e31863 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1067096 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Eli Mesika: Verified Ravi Nori: Looks good to me, but someone else must approve Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25077 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4e661ad31afb5599d4d7dd6370e5c9fb98e31863 Gerrit-PatchSet: 2 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: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: tools: snmp: fix 'minimal snmp configuration' section in con...
by emesika＠redhat.com 05 Mar '14

05 Mar '14

Eli Mesika has submitted this change and it was merged. Change subject: tools: snmp: fix 'minimal snmp configuration' section in conf file ...................................................................... tools: snmp: fix 'minimal snmp configuration' section in conf file key name has to be FILTER and not FILTERS. users using current example will not get any snmp notification. Change-Id: I0202abfd9c2d26e41d2073a293e27d983f7578f9 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1072849 Signed-off-by: Mooli Tayer <mtayer(a)redhat.com> --- M packaging/services/ovirt-engine-notifier/ovirt-engine-notifier.conf.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Eli Mesika: Verified; Looks good to me, approved mooli tayer: Verified -- To view, visit http://gerrit.ovirt.org/25394 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0202abfd9c2d26e41d2073a293e27d983f7578f9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: adding missing default to a non-null column
by emesika＠redhat.com 05 Mar '14

05 Mar '14

Eli Mesika has submitted this change and it was merged. Change subject: core: adding missing default to a non-null column ...................................................................... core: adding missing default to a non-null column This patch fixes an issue in event_subscriber table that was added a notification_method column which does not allow nulls. If the table has any records before the upgrade, this will fail since the existing records will have NULL for the new added column. Change-Id: Id3c7ce30c32b9b38f08ff6c8c0eb8bfa02cf5066 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1072867 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M packaging/dbscripts/upgrade/03_05_0050_event_notification_methods.sql 1 file changed, 1 insertion(+), 3 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved mooli tayer: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/25376 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id3c7ce30c32b9b38f08ff6c8c0eb8bfa02cf5066 Gerrit-PatchSet: 3 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: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: set domain for vm-init
by juan.hernandez＠redhat.com 05 Mar '14

05 Mar '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: set domain for vm-init ...................................................................... restapi: set domain for vm-init Wrong check for setting domain for VM Change-Id: Ia3ee9738b4003b1e094d59a21313f6f84e73ce7f Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1069201 Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Shahar Havivi: Verified Juan Hernandez: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25373 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia3ee9738b4003b1e094d59a21313f6f84e73ce7f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: Use static repos for a new layout
by knesenko＠redhat.com 05 Mar '14

05 Mar '14

Kiril Nesenko has submitted this change and it was merged. Change subject: Use static repos for a new layout ...................................................................... Use static repos for a new layout Change-Id: I44087b1d6f5aafa965ad8c0b9b23e8f8d4b949b2 Signed-off-by: Kiril Nesenko <knesenko(a)redhat.com> --- M jobs/packaging/repo_closure_check.sh 1 file changed, 16 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Ewoud Kohl van Wijngaarden: Looks good to me, but someone else must approve Kiril Nesenko: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25295 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I44087b1d6f5aafa965ad8c0b9b23e8f8d4b949b2 Gerrit-PatchSet: 6 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ewoud(a)kohlvanwijngaarden.nl> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Edit vNic Profile- disable pm if the profile is us...
by alkaplan＠redhat.com 05 Mar '14

05 Mar '14

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: Edit vNic Profile- disable pm if the profile is used by a vm ...................................................................... webadmin: Edit vNic Profile- disable pm if the profile is used by a vm Port mirroring checkbox should be disabled if the VNIC profile is used by a VM. Bug-Url: https://bugzilla.redhat.com/1031553 Change-Id: I06b851dcfbae9cd1ba48fda20aa0d0db6c3cb6cc Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/EditVnicProfileModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java 2 files changed, 39 insertions(+), 0 deletions(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved Lior Vernia: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24896 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I06b851dcfbae9cd1ba48fda20aa0d0db6c3cb6cc Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(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-engine[master]: webadmin: Manage network dialog- replace VM network checkbox...
by alkaplan＠redhat.com 05 Mar '14

05 Mar '14

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: Manage network dialog- replace VM network checkbox with icon ...................................................................... webadmin: Manage network dialog- replace VM network checkbox with icon VM network = True => green VM icon is shown VM network = False => no icon is show Change-Id: Ie96e3f173b0b510f1b8f20c9e9649eb937054d1a Bug-Url: https://bugzilla.redhat.com/1016636 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/column/SafeHtmlWithSafeHtmlTooltipColumn.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterManageNetworkPopupView.java 2 files changed, 44 insertions(+), 26 deletions(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24830 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie96e3f173b0b510f1b8f20c9e9649eb937054d1a Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(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-engine[master]: webadmin: label group in setup networks shouldn't have tooltip
by alkaplan＠redhat.com 05 Mar '14

05 Mar '14

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: label group in setup networks shouldn't have tooltip ...................................................................... webadmin: label group in setup networks shouldn't have tooltip Mouse over label group on setup networks displays the last displayed tooltip. Shouldn't display anything. Change-Id: I6c46d850f5452b7318e52c6ea5494a3ab2c6a3da Bug-Url: https://bugzilla.redhat.com/1064432 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/ItemInfoPopup.java 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved Lior Vernia: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25011 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6c46d850f5452b7318e52c6ea5494a3ab2c6a3da Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(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-engine[master]: packaging: setup: database: suppress password based on env key
by sbonazzo＠redhat.com 05 Mar '14

05 Mar '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: database: suppress password based on env key ...................................................................... packaging: setup: database: suppress password based on env key Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1072476 Change-Id: Ia4d1ad9b0acda23a59f55d1653a542c9ea038754 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ovirt-engine.spec.in M packaging/setup/plugins/ovirt-engine-common/base/db/connection.py M packaging/setup/plugins/ovirt-engine-rename/core/pki.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/db/connection.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/options.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/pki/ca.py 6 files changed, 23 insertions(+), 24 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/25347 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia4d1ad9b0acda23a59f55d1653a542c9ea038754 Gerrit-PatchSet: 5 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: Sandro Bonazzola <sbonazzo(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]: webadmin: Setup Networks- unused space in unattached network...
by alkaplan＠redhat.com 05 Mar '14

05 Mar '14

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: Setup Networks- unused space in unattached networks panel ...................................................................... webadmin: Setup Networks- unused space in unattached networks panel Unattahced networks scrolling area should fill all available space. Change-Id: I630c2599eceef7e4165a8728ed8e4faa52c67d22 Bug-Url: https://bugzilla.redhat.com/1068614 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostSetupNetworksPopupView.ui.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved Lior Vernia: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25002 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I630c2599eceef7e4165a8728ed8e4faa52c67d22 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(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