July 2017 - Engine-commits - oVirt List Archives

Change in jenkins[master]: OVIRT-1331 - Enable fc26 updates mirror
by Code Review 17 Jul '17

17 Jul '17

>From Evgheni Dereveanchin <ederevea(a)redhat.com>: Evgheni Dereveanchin has submitted this change and it was merged. ( https://gerrit.ovirt.org/79479 ) Change subject: OVIRT-1331 - Enable fc26 updates mirror ...................................................................... OVIRT-1331 - Enable fc26 updates mirror this patch creates the related job to sync the FC26 update mirror Change-Id: Ifaec45ccb656e1bb2b4d47175004bd01a2f5a157 Jira-ticket: https://ovirt-jira.atlassian.net/browse/OVIRT-1331 Signed-off-by: Evgheni Dereveanchin <ederevea(a)redhat.com> --- M jobs/confs/projects/system/sync_mirrors.yaml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Eyal Edri: Looks good to me Barak Korren: Looks good to me Jenkins CI: Passed CI tests Evgheni Dereveanchin: Verified; Looks good to me, approved; Ready for merge -- To view, visit https://gerrit.ovirt.org/79479 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifaec45ccb656e1bb2b4d47175004bd01a2f5a157 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Evgheni Dereveanchin <ederevea(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Daniel Belenky <dbelenky(a)redhat.com> Gerrit-Reviewer: Evgheni Dereveanchin <ederevea(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com>

1 0

Change in jenkins[master]: Implement archive artifacts flow for containers
by Code Review 17 Jul '17

17 Jul '17

>From Eyal Edri <eedri(a)redhat.com>: Eyal Edri has submitted this change and it was merged. ( https://gerrit.ovirt.org/77511 ) Change subject: Implement archive artifacts flow for containers ...................................................................... Implement archive artifacts flow for containers As part of our efforts for implement std-ci flow for containers, I've added functionallity which is similar to 'archive artifacts' for containers. Every container which is tagged with 'exported-artifacts', will be re-tagged with the $BUILD_URL's sha1 for further tracking, and pushed to our intermediate phase repository in docker.io. Also refactored collect_artifacts.sh not to be parsed by JJB. Because the only reason we've passed collect_artifacts through JJB is to get the project name and since it's being injected by the standard template, we no longer need to do it. Jira: https://ovirt-jira.atlassian.net/browse/OVIRT-1390 Change-Id: I0e44f17a8c63326a13991e7655590a6caf0f6250 Signed-off-by: Daniel Belenky <dbelenky(a)redhat.com> --- M jobs/confs/shell-scripts/collect_artifacts.sh M jobs/confs/yaml/publishers/standard.yaml 2 files changed, 119 insertions(+), 21 deletions(-) Approvals: Eyal Edri: Looks good to me, approved; Ready for merge Jenkins CI: Passed CI tests Daniel Belenky: Verified -- To view, visit https://gerrit.ovirt.org/77511 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0e44f17a8c63326a13991e7655590a6caf0f6250 Gerrit-PatchSet: 34 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Daniel Belenky <dbelenky(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Daniel Belenky <dbelenky(a)redhat.com> Gerrit-Reviewer: Evgheni Dereveanchin <ederevea(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com>

1 0

Change in jenkins[master]: standard-stage: added docker credentials binding
by Code Review 17 Jul '17

17 Jul '17

>From Eyal Edri <eedri(a)redhat.com>: Eyal Edri has submitted this change and it was merged. ( https://gerrit.ovirt.org/77752 ) Change subject: standard-stage: added docker credentials binding ...................................................................... standard-stage: added docker credentials binding Jira: https://ovirt-jira.atlassian.net/browse/OVIRT-1390 As part of our efforts to implement 'std-ci' for containers, I've added the credentials of our 'intermediate-phase' docker repository. This repository will contain docker images we need to test. Change-Id: I4c58784eb30f47124960624c93aa7e844737e891 --- M jobs/confs/yaml/templates/standard-stage.yaml 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Eyal Edri: Looks good to me, approved; Ready for merge Jenkins CI: Passed CI tests Daniel Belenky: Verified -- To view, visit https://gerrit.ovirt.org/77752 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4c58784eb30f47124960624c93aa7e844737e891 Gerrit-PatchSet: 8 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Daniel Belenky <dbelenky(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Daniel Belenky <dbelenky(a)redhat.com> Gerrit-Reviewer: Evgheni Dereveanchin <ederevea(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com>

1 0

Change in ovirt-engine-sdk-ruby[sdk_4.1]: [maven-release-plugin] prepare release 4.1.8
by Code Review 17 Jul '17

17 Jul '17

>From Juan Hernandez <juan.hernandez(a)redhat.com>: Juan Hernandez has submitted this change and it was merged. ( https://gerrit.ovirt.org/79477 ) Change subject: [maven-release-plugin] prepare release 4.1.8 ...................................................................... [maven-release-plugin] prepare release 4.1.8 Change-Id: Ife5168e57b2ee8ff732c45c062f253e736135535 --- M generator/pom.xml M pom.xml M sdk/pom.xml 3 files changed, 4 insertions(+), 4 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/79477 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ife5168e57b2ee8ff732c45c062f253e736135535 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.1 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[sdk_4.1]: Update CHANGES.adoc for release 4.1.8
by Code Review 17 Jul '17

17 Jul '17

>From Juan Hernandez <juan.hernandez(a)redhat.com>: Juan Hernandez has submitted this change and it was merged. ( https://gerrit.ovirt.org/79476 ) Change subject: Update CHANGES.adoc for release 4.1.8 ...................................................................... Update CHANGES.adoc for release 4.1.8 This patch updates the CHANGES.adoc file, documenting the relevant changes in release 4.1.8. Change-Id: Icdc8a6ff8b24f0bd88f4684f8d5fd24acc2dcd0a Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/CHANGES.adoc 1 file changed, 34 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/79476 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icdc8a6ff8b24f0bd88f4684f8d5fd24acc2dcd0a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.1 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Boris Odnopozov <bodnopoz(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[sdk_4.1]: [maven-release-plugin] prepare for next development iteration
by Code Review 17 Jul '17

17 Jul '17

>From Juan Hernandez <juan.hernandez(a)redhat.com>: Juan Hernandez has submitted this change and it was merged. ( https://gerrit.ovirt.org/79478 ) Change subject: [maven-release-plugin] prepare for next development iteration ...................................................................... [maven-release-plugin] prepare for next development iteration Change-Id: Ib14ec0b88ec4887e673f634354582a9c3a964869 --- M generator/pom.xml M pom.xml M sdk/pom.xml 3 files changed, 4 insertions(+), 4 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/79478 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib14ec0b88ec4887e673f634354582a9c3a964869 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.1 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[sdk_4.1]: Update to model 4.1.37 and metamodel 1.1.14
by Code Review 17 Jul '17

17 Jul '17

>From Juan Hernandez <juan.hernandez(a)redhat.com>: Juan Hernandez has submitted this change and it was merged. ( https://gerrit.ovirt.org/79475 ) Change subject: Update to model 4.1.37 and metamodel 1.1.14 ...................................................................... Update to model 4.1.37 and metamodel 1.1.14 The more relevant changes in this new version of the model are the following: * Add the `all_content` parameter to `Host.Get` and `Hosts.List` https://bugzilla.redhat.com/1444081. * Fix the documentation of the method that lists events https://bugzilla.redhat.com/1447622. * Fix the direction of the main parameter of the method that adds a CDROM to a virtual machine. It should be input and output. * Improve the documentation explaining that in general the order of the results of _list_ methods isn't guaranteed. * Add the `index` attribute to the `Event` type https://bugzilla.redhat.com/1448511. The changes in the new version of the metamodel aren't relevant for the Ruby SDK. Change-Id: I9b5436625077174e419bfe05aefaaef5ab818943 Related-To: https://bugzilla.redhat.com/1444081 Related-To: https://bugzilla.redhat.com/1447622 Related-To: https://bugzilla.redhat.com/1448511 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M pom.xml 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/79475 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9b5436625077174e419bfe05aefaaef5ab818943 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.1 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Use docker start instead of restart in global setup
by Code Review 17 Jul '17

17 Jul '17

>From Eyal Edri <eedri(a)redhat.com>: Eyal Edri has submitted this change and it was merged. ( https://gerrit.ovirt.org/79283 ) Change subject: Use docker start instead of restart in global setup ...................................................................... Use docker start instead of restart in global setup Instead of restart docker with every job run, we'd better use docker start just to make sure it's running. This may reduce the noise from the docker-devicemapper combination bug. For more details on this issue: https://ovirt-jira.atlassian.net/browse/OVIRT-1516 Change-Id: I36e1b281e08b8483269ad4559198dad66667a17b Signed-off-by: Daniel Belenky <dbelenky(a)redhat.com> --- M jobs/confs/shell-scripts/global_setup.sh 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Barak Korren: Looks good to me, approved; Ready for merge Jenkins CI: Passed CI tests Daniel Belenky: Verified; Ready for review -- To view, visit https://gerrit.ovirt.org/79283 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I36e1b281e08b8483269ad4559198dad66667a17b Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Daniel Belenky <dbelenky(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Daniel Belenky <dbelenky(a)redhat.com> Gerrit-Reviewer: Evgheni Dereveanchin <ederevea(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: added utility methods for huge pages
by Code Review 17 Jul '17

17 Jul '17

>From Tomas Jelinek <tjelinek(a)redhat.com>: Tomas Jelinek has submitted this change and it was merged. ( https://gerrit.ovirt.org/79380 ) Change subject: core: added utility methods for huge pages ...................................................................... core: added utility methods for huge pages It will mostly be used by the scheduler to know where to schedule this VM to. Added two methods: - isBackedByHudepages: returns true if the VM has a custom property defined and it is a valid integer - getHugePages: returns a map "huge page size" -> "required amount". It returns it in this format and not as a HugePage object to easy the processing by the scheduler. Implemented in a separate utility class because it can not be part of the VMBase due to it's dependency on CustomPropertiesUtil and it's transitive dependencies. Change-Id: I66fec5a5b367ddadcee8033b4a8932de5be23e57 Bug-Url: https://bugzilla.redhat.com/?????? Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/HugePageUtils.java A backend/manager/modules/common/src/test/java/org/ovirt/engine/core/common/utils/HugePageUtilsTest.java 2 files changed, 126 insertions(+), 0 deletions(-) Approvals: Tomas Jelinek: Verified Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/79380 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I66fec5a5b367ddadcee8033b4a8932de5be23e57 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[v2.1.z]: Allow editing configuration on hosted-engine.conf file
by Code Review 17 Jul '17

17 Jul '17

>From Martin Sivák <msivak(a)redhat.com>: Martin Sivák has submitted this change and it was merged. ( https://gerrit.ovirt.org/79224 ) Change subject: Allow editing configuration on hosted-engine.conf file ...................................................................... Allow editing configuration on hosted-engine.conf file Until this fix the only editable configuration file on the shared storage was the broker.conf. This adds hosted-engine.conf to the editable files list and allows changing the configuration stored there. Change-Id: I99af23f37876c86a77940e2f5095f3157f8bfc20 Bug-Url: https://bugzilla.redhat.com/1457357 Signed-off-by: Jenny Tokar <jtokar(a)redhat.com> --- M ovirt_hosted_engine_ha/env/config.py 1 file changed, 8 insertions(+), 4 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Jenkins CI: Passed CI tests Jenny Tokar: Verified -- To view, visit https://gerrit.ovirt.org/79224 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I99af23f37876c86a77940e2f5095f3157f8bfc20 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: v2.1.z Gerrit-Owner: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Denis Chaplygin <dchaplyg(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0