April 2016 - Engine-commits - oVirt List Archives

Change in ovirt-release[ovirt-3.6]: el6: move gluster to CentOS Storage SIG
by sbonazzo＠redhat.com 28 Apr '16

28 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: el6: move gluster to CentOS Storage SIG ...................................................................... el6: move gluster to CentOS Storage SIG Use Gluster RPMs provided by CentOS Storage SIG instead of those provided by Gluster community. We've been advised by Gluster community that their packaging effort is focused on CentOS Storage SIG and CentOS infrastructure is more reliable. Change-Id: Ib0d6ff00b0bce921519543e2bf0ebe24e91328c4 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-el6-deps.repo.in 1 file changed, 4 insertions(+), 13 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56809 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib0d6ff00b0bce921519543e2bf0ebe24e91328c4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: node: use default zone for firewalld services
by sbonazzo＠redhat.com 28 Apr '16

28 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: use default zone for firewalld services ...................................................................... node: use default zone for firewalld services In the past we ship a zone but it's not recommended as others daemons might also add their own zones. It's recommended to use the current zone and simplify the firewalld schema. Change-Id: Ib46992eb866f3b3dc6e4d4e4274fb86a810f4cc7 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M host-node/Makefile.am D host-node/firewalld/zones/ovirthostnode.xml M ovirt-release-master.spec.in 3 files changed, 9 insertions(+), 32 deletions(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Douglas Schilling Landgraf: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56652 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib46992eb866f3b3dc6e4d4e4274fb86a810f4cc7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: repo_closure_check: fix typo
by sbonazzo＠redhat.com 28 Apr '16

28 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: repo_closure_check: fix typo ...................................................................... repo_closure_check: fix typo Change-Id: I0647a406f3c8d331155b5e602f66e7869569dbb1 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/packaging/repo_closure_check.sh 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/56806 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0647a406f3c8d331155b5e602f66e7869569dbb1 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: repo_closure_check: use Storage SIG also on EL6
by sbonazzo＠redhat.com 28 Apr '16

28 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: repo_closure_check: use Storage SIG also on EL6 ...................................................................... repo_closure_check: use Storage SIG also on EL6 Change-Id: Ia579b4757eb1a7556464fe74f1315f69b34248c5 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/packaging/repo_closure_check.sh 1 file changed, 2 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/56801 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia579b4757eb1a7556464fe74f1315f69b34248c5 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: List and Map instead of ArrayList and HashMap in OsRep...
by ahadas＠redhat.com 28 Apr '16

28 Apr '16

Arik Hadas has submitted this change and it was merged. Change subject: core: List and Map instead of ArrayList and HashMap in OsRepository ...................................................................... core: List and Map instead of ArrayList and HashMap in OsRepository Use List and Map interfaces as return values of OsRepository methods instead of ArrayList and HashMap classes. Also removed redundant specifiers from interface's methods and fields. Change-Id: I50f2b9edc29625903fcd04ae72228e5e347daf69 Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/IconLoader.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/osinfo/OsRepository.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendOperatingSystemsResource.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/OsRepositoryImpl.java M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/OsRepositoryImplTest.java 5 files changed, 63 insertions(+), 67 deletions(-) Approvals: Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Shmuel Leib Melamud: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/56678 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I50f2b9edc29625903fcd04ae72228e5e347daf69 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Re-add gluster repos for FC22
by sbonazzo＠redhat.com 28 Apr '16

28 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Re-add gluster repos for FC22 ...................................................................... Re-add gluster repos for FC22 Fedora 22 still needs gluster repos for gluster 3.7 Change-Id: I227db2ba9aa464b7ec0981c84be57b6259b24abc Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/packaging/repo_closure_check.sh 1 file changed, 33 insertions(+), 12 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/56800 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I227db2ba9aa464b7ec0981c84be57b6259b24abc Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[master]: kickstart: move gluster to CentOS Storage SIG
by sbonazzo＠redhat.com 28 Apr '16

28 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: kickstart: move gluster to CentOS Storage SIG ...................................................................... kickstart: move gluster to CentOS Storage SIG Use Gluster RPMs provided by CentOS Storage SIG instead of those provided by Gluster community. We've been advised by Gluster community that their packaging effort is focused on CentOS Storage SIG and CentOS infrastructure is more reliable. Change-Id: Id8e6b4c6f4a9530265156858b708578249a1a561 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos-7/kickstart/ovirt-live-ovirt-custom.ks.in 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56795 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id8e6b4c6f4a9530265156858b708578249a1a561 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: qemu: update glusterfs repos
by sbonazzo＠redhat.com 28 Apr '16

28 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: qemu: update glusterfs repos ...................................................................... qemu: update glusterfs repos Moving automation to CentOS Storage SIG gluster repo as for oVirt default repositories. Change-Id: I14314b724f11e807b76e0851dcb44169e00e87a6 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/qemu/qemu_create-rpms.yaml 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56799 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I14314b724f11e807b76e0851dcb44169e00e87a6 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: repo_closure: update glusterfs repos
by sbonazzo＠redhat.com 28 Apr '16

28 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: repo_closure: update glusterfs repos ...................................................................... repo_closure: update glusterfs repos Moving automation to CentOS Storage SIG gluster repo as for oVirt default repositories. For Fedora, dropping repositories since they're empty: Fedora now always ships latest Gluster 3.7 packages. Change-Id: I36d72badd7572634f009e1f807973334a7443668 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/packaging/repo_closure_check.sh 1 file changed, 2 insertions(+), 9 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/56798 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I36d72badd7572634f009e1f807973334a7443668 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[master]: glusterfs: drop brick provisioning
by sbonazzo＠redhat.com 28 Apr '16

28 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: glusterfs: drop brick provisioning ...................................................................... glusterfs: drop brick provisioning brick provisioning was added for trying to support hyperconverged setup for Hosted Engine. The new approach is to use gdeploy (https://github.com/gluster/gdeploy) for the provisioning so the provisioning code is not useful anymore: this patch remove it. Change-Id: I9d44ef00c8721eb08d611b5a2901f74920605fdd Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-hosted-engine-setup.spec.in M po/POTFILES.in M po/pt_BR.po M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/engine/add_host.py M src/plugins/ovirt-hosted-engine-setup/storage/Makefile.am M src/plugins/ovirt-hosted-engine-setup/storage/__init__.py D src/plugins/ovirt-hosted-engine-setup/storage/glusterfs.py M src/plugins/ovirt-hosted-engine-setup/storage/nfs.py M src/plugins/ovirt-hosted-engine-setup/storage/storage.py M src/plugins/ovirt-hosted-engine-setup/system/packages.py 11 files changed, 345 insertions(+), 792 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified; Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56485 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9d44ef00c8721eb08d611b5a2901f74920605fdd Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0