December 2013 - Engine-commits - oVirt List Archives

Change in ovirt-dwh[ovirt-3.3]: packaging: setup: detect existing DB in remote flows
by ydary＠redhat.com 18 Dec '13

18 Dec '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: detect existing DB in remote flows ...................................................................... packaging: setup: detect existing DB in remote flows Change-Id: Iee813572cab7e0afcf5f5dfa97e4121334c3aa5b Bug-Url: https://bugzilla.redhat.com/1026947 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/common_utils.py M packaging/ovirt-engine-dwh-setup.py 2 files changed, 60 insertions(+), 18 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22546 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iee813572cab7e0afcf5f5dfa97e4121334c3aa5b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: ovirt-3.3 Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-dwh[master]: packaging: setup: detect existing DB in remote flows
by ydary＠redhat.com 18 Dec '13

18 Dec '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: detect existing DB in remote flows ...................................................................... packaging: setup: detect existing DB in remote flows Change-Id: Iee813572cab7e0afcf5f5dfa97e4121334c3aa5b Bug-Url: https://bugzilla.redhat.com/1026947 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/common_utils.py M packaging/ovirt-engine-dwh-setup.py 2 files changed, 60 insertions(+), 18 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22538 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iee813572cab7e0afcf5f5dfa97e4121334c3aa5b Gerrit-PatchSet: 4 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Edit Network and apply to hosts at once
by masayag＠redhat.com 18 Dec '13

18 Dec '13

Moti Asayag has submitted this change and it was merged. Change subject: engine: Edit Network and apply to hosts at once ...................................................................... engine: Edit Network and apply to hosts at once Editing a network and applying its changes to all of the eligible hosts via a single action is enabled by sync-ing the network on the hosts (if the network wasn't renamed) See: http://www.ovirt.org/Features/MultiHostNetworkConfiguration Bug-Url: http://bugzilla.redhat.com/1040580 Change-Id: Iec04c7fb0c29ba6f61b7d788a9c3917f5d46554e Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/dc/UpdateNetworkCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/NetworkValidator.java 2 files changed, 148 insertions(+), 5 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22053 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iec04c7fb0c29ba6f61b7d788a9c3917f5d46554e Gerrit-PatchSet: 13 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-scheduler-proxy[master]: When all filters fail, return all hosts
by msivak＠redhat.com 18 Dec '13

18 Dec '13

Martin Sivák has submitted this change and it was merged. Change subject: When all filters fail, return all hosts ...................................................................... When all filters fail, return all hosts Change-Id: I3e88e22795d17dbd6c9dd03e7cc84da31feb483e Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1002444 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- A plugins/test_failing_plugin.py M src/ovirtscheduler/request_handler.py M src/ovirtscheduler/request_handler_test.py M tests/java/src/test/java/org/ovirt/schedulerproxy/SchedulerProxyTest.java 4 files changed, 64 insertions(+), 22 deletions(-) Approvals: Martin Sivák: Verified; Looks good to me, approved Jiří Moskovčák: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/22241 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3e88e22795d17dbd6c9dd03e7cc84da31feb483e Gerrit-PatchSet: 3 Gerrit-Project: ovirt-scheduler-proxy Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[ovirt-3.3]: packaging: setup: backup DB and WAR on upgrade
by ydary＠redhat.com 18 Dec '13

18 Dec '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: backup DB and WAR on upgrade ...................................................................... packaging: setup: backup DB and WAR on upgrade Change-Id: I97c4da7ff92619551515b652ec17734fced124af Bug-Url: https://bugzilla.redhat.com/1043314 Signed-off-by: Alex Lourie <alourie(a)redhat.com> Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/common_utils.py M packaging/ovirt-engine-reports-setup.py 2 files changed, 90 insertions(+), 55 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22532 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I97c4da7ff92619551515b652ec17734fced124af Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-3.3 Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: tools: Adds support for SMTP with STARTTLS to notifier
by emesika＠redhat.com 18 Dec '13

18 Dec '13

Eli Mesika has submitted this change and it was merged. Change subject: tools: Adds support for SMTP with STARTTLS to notifier ...................................................................... tools: Adds support for SMTP with STARTTLS to notifier Adds support for sending emails using SMTP with STARTTLS. Change-Id: I751a0db149cbd217728c036a65f7fadc6da4cdf9 Bug-Url: https://bugzilla.redhat.com/1020920 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/NotificationProperties.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/sender/mail/JavaMailSender.java M packaging/services/ovirt-engine-notifier/ovirt-engine-notifier.conf.in 3 files changed, 20 insertions(+), 8 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22297 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I751a0db149cbd217728c036a65f7fadc6da4cdf9 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(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: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[master]: packaging: setup: backup DB and WAR on upgrade
by ydary＠redhat.com 18 Dec '13

18 Dec '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: backup DB and WAR on upgrade ...................................................................... packaging: setup: backup DB and WAR on upgrade Change-Id: I97c4da7ff92619551515b652ec17734fced124af Bug-Url: https://bugzilla.redhat.com/1043314 Signed-off-by: Alex Lourie <alourie(a)redhat.com> Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/common_utils.py M packaging/ovirt-engine-reports-setup.py 2 files changed, 90 insertions(+), 55 deletions(-) Approvals: Alex Lourie: Verified Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22480 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I97c4da7ff92619551515b652ec17734fced124af Gerrit-PatchSet: 5 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: tools: Adds option MAIL_SMTP_ENCRYPTION to notifier
by emesika＠redhat.com 18 Dec '13

18 Dec '13

Eli Mesika has submitted this change and it was merged. Change subject: tools: Adds option MAIL_SMTP_ENCRYPTION to notifier ...................................................................... tools: Adds option MAIL_SMTP_ENCRYPTION to notifier Adds option MAIL_SMTP_ENCRYPTION to allow multiple types of SMTP encryption to be specified in notifier config. This new option also replaces old MAIL_ENABLE_SSL option. Change-Id: I4a914091060c0a86b2710ce20e666d67f28fa401 Bug-Url: https://bugzilla.redhat.com/1020920 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/NotificationProperties.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/sender/mail/JavaMailSender.java M backend/manager/tools/src/test/java/org/ovirt/engine/core/notifier/utils/mail/MailSenderTest.java M backend/manager/tools/src/test/resources/conf/connection-test-notifier.conf M backend/manager/tools/src/test/resources/conf/error-notifier.conf M backend/manager/tools/src/test/resources/conf/notifier-mail-test-plain.conf M backend/manager/tools/src/test/resources/conf/notifier-mail-test-secured.conf M backend/manager/tools/src/test/resources/conf/notifier-prop-test.conf M packaging/services/ovirt-engine-notifier/ovirt-engine-notifier.conf.in 9 files changed, 43 insertions(+), 10 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22296 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4a914091060c0a86b2710ce20e666d67f28fa401 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(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: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: tools: Removes option MAIL_PORT_SSL
by emesika＠redhat.com 18 Dec '13

18 Dec '13

Eli Mesika has submitted this change and it was merged. Change subject: tools: Removes option MAIL_PORT_SSL ...................................................................... tools: Removes option MAIL_PORT_SSL Removes option MAIL_PORT_SSL. IMO this option is redundant since currently MAIL_PORT or MAIL_PORT_SSL are used based on SSL turned on/off. So it's easier to specify port number only in one option (MAIL_PORT). Change-Id: I76cd77d7f7227e0ae81467189c06c8841ff7f775 Bug-Url: https://bugzilla.redhat.com/1020920 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/NotificationProperties.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/sender/mail/JavaMailSender.java M backend/manager/tools/src/test/resources/conf/notifier-mail-test-plain.conf M backend/manager/tools/src/test/resources/conf/notifier-mail-test-secured.conf M packaging/services/ovirt-engine-notifier/ovirt-engine-notifier.conf.in 5 files changed, 4 insertions(+), 10 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22295 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I76cd77d7f7227e0ae81467189c06c8841ff7f775 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(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: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: tools: Tests config options validity during notifier startup
by emesika＠redhat.com 18 Dec '13

18 Dec '13

Eli Mesika has submitted this change and it was merged. Change subject: tools: Tests config options validity during notifier startup ...................................................................... tools: Tests config options validity during notifier startup Moves all tests of configuration options to notifier startup code. Change-Id: I3d9f36aee42e51b4903ccf388ca4b328e1740507 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/Notifier.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/NotificationProperties.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/sender/mail/JavaMailSender.java 3 files changed, 70 insertions(+), 90 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22294 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3d9f36aee42e51b4903ccf388ca4b328e1740507 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(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: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0