July 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: packaging: build: fix parallel make issues
by oschreib＠redhat.com 21 Jul '13

21 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: build: fix parallel make issues ...................................................................... packaging: build: fix parallel make issues Change-Id: I4eede91d86f42b3a6a784bc226a83f2368edfd82 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M Makefile 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17092 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4eede91d86f42b3a6a784bc226a83f2368edfd82 Gerrit-PatchSet: 2 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: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: handle selinux disabled
by oschreib＠redhat.com 21 Jul '13

21 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: handle selinux disabled ...................................................................... packaging: setup: handle selinux disabled In engine-setup restored previous behavior, exiting if selinux is disabled. In engine-setup-2 disable apache/selinux plugin if selinux is disabled. Bug-Url: https://bugzilla.redhat.com/979763 Change-Id: I4242c7aedd805f6d7e4ffcc7dc6dd77a31c63dee Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/fedora/setup/engine_validators.py M packaging/setup/plugins/ovirt-engine-setup/apache/selinux.py 2 files changed, 12 insertions(+), 2 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16771 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4242c7aedd805f6d7e4ffcc7dc6dd77a31c63dee Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: ignore sack cache populate errors
by oschreib＠redhat.com 21 Jul '13

21 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: ignore sack cache populate errors ...................................................................... packaging: setup: ignore sack cache populate errors for some reason yum when going to cache only mode tries to download online repositories, and if fails cause the entire transaction to fail. more over, when going online, it also tries to download online resources, and if fails cause the entire transaction to fail. any network issue during this stage will create an undesired failure. so just ignore these errors and hope for the best, as this sequence is only used when transaction is rollbacked. Change-Id: I6f6cba0071da862addcb438d47ac9401f746ee87 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/fedora/setup/miniyum.py 1 file changed, 23 insertions(+), 5 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17108 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6f6cba0071da862addcb438d47ac9401f746ee87 Gerrit-PatchSet: 2 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: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: services: sysv: add name to status
by oschreib＠redhat.com 21 Jul '13

21 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: services: sysv: add name to status ...................................................................... packaging: services: sysv: add name to status Change-Id: I8a5ffd98175ac28914d5e53f01e64048f1d121b2 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=985913 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/services/ovirt-engine-notifier.sysv.in M packaging/services/ovirt-engine.sysv.in M packaging/services/ovirt-websocket-proxy.sysv.in 3 files changed, 3 insertions(+), 3 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17088 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8a5ffd98175ac28914d5e53f01e64048f1d121b2 Gerrit-PatchSet: 2 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: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: remove 'installation' term from common
by oschreib＠redhat.com 21 Jul '13

21 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: remove 'installation' term from common ...................................................................... packaging: setup: remove 'installation' term from common Change-Id: I492d572a2ff71729dc677cae977450257b6846bf Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/core/engine.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17076 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I492d572a2ff71729dc677cae977450257b6846bf Gerrit-PatchSet: 2 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: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: iptables port range separator is ':'
by oschreib＠redhat.com 21 Jul '13

21 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: iptables port range separator is ':' ...................................................................... packaging: setup: iptables port range separator is ':' Change-Id: Ifcd76e93ab30145ca563a8f54eb2b88ea1e97f11 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> Reported-By: Hetz Ben Hamo <hetz(a)hetz.biz> --- M packaging/setup/plugins/ovirt-engine-setup/network/firewall_manager.py 1 file changed, 4 insertions(+), 3 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17036 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifcd76e93ab30145ca563a8f54eb2b88ea1e97f11 Gerrit-PatchSet: 2 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: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: move apache core to common
by oschreib＠redhat.com 21 Jul '13

21 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: move apache core to common ...................................................................... packaging: setup: move apache core to common Change-Id: Ide87c904733eaaffb1e0883987a5711baec285fc Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/system/__init__.py R packaging/setup/plugins/ovirt-engine-common/system/apache.py M packaging/setup/plugins/ovirt-engine-setup/apache/__init__.py 3 files changed, 2 insertions(+), 2 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16978 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ide87c904733eaaffb1e0883987a5711baec285fc Gerrit-PatchSet: 2 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: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: move uninstall to common
by oschreib＠redhat.com 21 Jul '13

21 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: move uninstall to common ...................................................................... packaging: setup: move uninstall to common this will enable us to reuse. Change-Id: I1abecb9afdbdaaf821d8ec03a7d9317f1fcd2675 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/core/__init__.py R packaging/setup/plugins/ovirt-engine-common/core/uninstall.py M packaging/setup/plugins/ovirt-engine-setup/core/__init__.py 3 files changed, 2 insertions(+), 2 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16977 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1abecb9afdbdaaf821d8ec03a7d9317f1fcd2675 Gerrit-PatchSet: 2 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: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: spec: avoid breaking firewalld
by oschreib＠redhat.com 21 Jul '13

21 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: spec: avoid breaking firewalld ...................................................................... packaging: spec: avoid breaking firewalld avoid breaking firewalld configuration when upgrading the -setup package. Change-Id: Icb96a6bf70a3784187838b51aceff3da6ee11e95 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Verified Ofer Schreiber: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17002 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icb96a6bf70a3784187838b51aceff3da6ee11e95 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-host-deploy[master]: vdsm: bridge: attempt to rollback if bridge creation failed
by Alon Bar-Lev 20 Jul '13

20 Jul '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: vdsm: bridge: attempt to rollback if bridge creation failed ...................................................................... vdsm: bridge: attempt to rollback if bridge creation failed Currently vdsm-bootstrap, host-deploy, vdsm-reg do not rollback network configuration if addNetwork fails. The sequence should be: - addNetwork - vdsm-restore-net-config -> rollback - vdsm-store-net-config -> commit Supporting non-node setup is not important to 3.3 as only legacy nodes still use the ovirt-host-deploy bridge plugin. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=979570 Change-Id: Id9e6b0d076f3a16f036e149258454da395569797 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M src/plugins/ovirt-host-deploy/vdsm/bridge.py 2 files changed, 47 insertions(+), 14 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17124 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id9e6b0d076f3a16f036e149258454da395569797 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