August 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: check active vds before activate storage domain
by amureini＠redhat.com 21 Aug '13

21 Aug '13

Allon Mureinik has submitted this change and it was merged. Change subject: engine: check active vds before activate storage domain ...................................................................... engine: check active vds before activate storage domain When activating a storage domain, at least one 'Up' host in DC is needed to do the job. Change-Id: Ia3e45b53d15b757237af8c76457f213c00a73909 Signed-off-by: plysan <plysab(a)gmail.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ActivateStorageDomainCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBase.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/ActivateStorageDomainCommandTest.java 3 files changed, 51 insertions(+), 9 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17940 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia3e45b53d15b757237af8c76457f213c00a73909 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: plysan <plysab(a)gmail.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: LiJiansheng <lijiangsheng1(a)gmail.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-Reviewer: plysan <plysab(a)gmail.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: fix yum rollbackability check
by oschreib＠redhat.com 21 Aug '13

21 Aug '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: fix yum rollbackability check ...................................................................... packaging: setup: fix yum rollbackability check The check to see if currently-installed packages will be able to be reinstalled if we need to rollback wasn't always working. Needs http://gerrit.ovirt.org/18310 (in otopi) to actually be able to know if these packages are available for reinstallation. Bug-URL: https://bugzilla.redhat.com/998921 Change-Id: I6449cf29771a6f9f6d165d6bc4dce2814fbc9a29 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/distro-rpm/packages.py 1 file changed, 20 insertions(+), 19 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18327 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6449cf29771a6f9f6d165d6bc4dce2814fbc9a29 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: text changes
by oschreib＠redhat.com 21 Aug '13

21 Aug '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: text changes ...................................................................... packaging: setup: text changes Change-Id: Ic2a50fdb2e3b2281fafa894b8e95c7902ba24380 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/all-in-one/vdsm.py M packaging/setup/plugins/ovirt-engine-setup/apache/root.py M packaging/setup/plugins/ovirt-engine-setup/apache/ssl.py M packaging/setup/plugins/ovirt-engine-setup/config/appmode.py M packaging/setup/plugins/ovirt-engine-setup/distro-rpm/packages.py M packaging/setup/plugins/ovirt-engine-setup/network/firewall_manager.py M packaging/setup/plugins/ovirt-engine-setup/provisioning/postgres.py M packaging/setup/plugins/ovirt-engine-setup/system/memcheck.py M packaging/setup/plugins/ovirt-engine-setup/system/nfs.py 9 files changed, 41 insertions(+), 41 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18320 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic2a50fdb2e3b2281fafa894b8e95c7902ba24380 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: write setup version into post install
by oschreib＠redhat.com 21 Aug '13

21 Aug '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: write setup version into post install ...................................................................... packaging: setup: write setup version into post install this will enable us to upgrade based on specific setup version if we ever require. Change-Id: I785942a238eb4085963928dc0d18e40d460ccb17 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/constants.py M packaging/setup/plugins/ovirt-engine-common/core/misc.py 2 files changed, 24 insertions(+), 0 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18319 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I785942a238eb4085963928dc0d18e40d460ccb17 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: move postinstall to common
by oschreib＠redhat.com 21 Aug '13

21 Aug '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: move postinstall to common ...................................................................... packaging: setup: move postinstall to common this will enable us to save the state after re-execution of setup for any reason. Change-Id: Ie9c67236ee02c5aa4d9bcb37c8e525f515aa9193 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/constants.py M packaging/setup/plugins/ovirt-engine-common/core/__init__.py R packaging/setup/plugins/ovirt-engine-common/core/postinstall.py M packaging/setup/plugins/ovirt-engine-remove/core/misc.py M packaging/setup/plugins/ovirt-engine-rename/core/__init__.py D packaging/setup/plugins/ovirt-engine-rename/core/postinstall.py M packaging/setup/plugins/ovirt-engine-setup/core/__init__.py 7 files changed, 18 insertions(+), 95 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18318 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie9c67236ee02c5aa4d9bcb37c8e525f515aa9193 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: allinone: non-standard ssh port
by oschreib＠redhat.com 21 Aug '13

21 Aug '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: allinone: non-standard ssh port ...................................................................... packaging: setup: allinone: non-standard ssh port Allow allinone to connect to localhost over non-standard ssh port. Bug-Url: https://bugzilla.redhat.com/994219 Change-Id: Ifd8a4cad5f0b480f04392933cd7c6e1b512c564d Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/constants.py M packaging/setup/plugins/ovirt-engine-setup/all-in-one/sshd.py M packaging/setup/plugins/ovirt-engine-setup/all-in-one/vdsm.py 3 files changed, 31 insertions(+), 0 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18317 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifd8a4cad5f0b480f04392933cd7c6e1b512c564d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.3]: packaging: build: minor fixups of dev targets
by oschreib＠redhat.com 21 Aug '13

21 Aug '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: build: minor fixups of dev targets ...................................................................... packaging: build: minor fixups of dev targets align install-dev and all-dev. fixups docs of dev targets. Change-Id: If05647b5508f5833783b3a966543dcbea2d34f56 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M Makefile M README.developer 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18316 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If05647b5508f5833783b3a966543dcbea2d34f56 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-image-uploader[ovirt-image-uploader-3.3]: sdk: aligning API initialization with iso-uploader
by sbonazzo＠redhat.com 21 Aug '13

21 Aug '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: sdk: aligning API initialization with iso-uploader ...................................................................... sdk: aligning API initialization with iso-uploader Aligning API initialization method with iso-uploader. This includes: - fixed creating objects in an insecure way - align imageuploader.conf with isouploader.conf - align sdk version requirement Excludes: - add the nossl option for the engine connection (doesn't work correctly) Change-Id: I59e9d95cd76a961265acc5475be6db7843b75121 Related-To: http://gerrit.ovirt.org/9288 Related-To: http://gerrit.ovirt.org/10815 Bug-Url: https://bugzilla.redhat.com/951100 Signed-off-by: Alex Lourie <alourie(a)redhat.com> Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-image-uploader.spec.in M src/__main__.py M src/imageuploader.conf 3 files changed, 73 insertions(+), 23 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/18349 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I59e9d95cd76a961265acc5475be6db7843b75121 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: ovirt-image-uploader-3.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(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-image-uploader[master]: sdk: aligning API initialization with iso-uploader
by sbonazzo＠redhat.com 21 Aug '13

21 Aug '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: sdk: aligning API initialization with iso-uploader ...................................................................... sdk: aligning API initialization with iso-uploader Aligning API initialization method with iso-uploader. This includes: - fixed creating objects in an insecure way - align imageuploader.conf with isouploader.conf - align sdk version requirement Excludes: - add the nossl option for the engine connection (doesn't work correctly) Change-Id: I59e9d95cd76a961265acc5475be6db7843b75121 Related-To: http://gerrit.ovirt.org/9288 Related-To: http://gerrit.ovirt.org/10815 Bug-Url: https://bugzilla.redhat.com/951100 Signed-off-by: Alex Lourie <alourie(a)redhat.com> Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-image-uploader.spec.in M src/__main__.py M src/imageuploader.conf 3 files changed, 73 insertions(+), 23 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Alex Lourie: Looks good to me, but someone else must approve Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/18309 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I59e9d95cd76a961265acc5475be6db7843b75121 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(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-log-collector[ovirt-log-collector-3.3]: log: default logging to timestamped files
by sbonazzo＠redhat.com 21 Aug '13

21 Aug '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: log: default logging to timestamped files ...................................................................... log: default logging to timestamped files Log messages to timestamped files as default, avoiding to overwrite existing ones. The default location of the logs was changed, to improve /var/log/ovirt-engine sanity. /var/log/ovirt-engine -> /var/log/ovirt-engine/ovirt-log-collector A log rotation was introduced in order to compress old logs. Bug-Url: https://bugzilla.redhat.com/998576 Change-Id: If180bb3c425d94449236147cddc17c504bf602b7 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-log-collector.spec.in M src/Makefile.am M src/__main__.py M src/config.py.in.in A src/logrotate.d/Makefile.am A src/logrotate.d/ovirt-log-collector.in 7 files changed, 83 insertions(+), 11 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Kiril Nesenko: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/18312 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If180bb3c425d94449236147cddc17c504bf602b7 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-3.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0