- Engine-commits - oVirt List Archives

Change in jenkins[master]: publisher: use 4.0 jobs for 4.0 publishing
by sbonazzo＠redhat.com 14 Jul '16

14 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: publisher: use 4.0 jobs for 4.0 publishing ...................................................................... publisher: use 4.0 jobs for 4.0 publishing - copycreate branched for supporting fc24 on master - use 4.0 jobs with 4.0 publisher where a 4.0 job exists - fix 3.6 publisher which was using master copy on 3.6 jobs Change-Id: I8640ea7962cf4f5502e0e63011ea2ae9f554258b Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt/publish-rpms-nightly-3.6.yaml M jobs/confs/projects/ovirt/publish-rpms-nightly-4.0.yaml M jobs/confs/yaml/builders/copy-create-job-artifact.yaml 3 files changed, 75 insertions(+), 48 deletions(-) Approvals: Sandro Bonazzola: Verified; Ready for merge Barak Korren: Looks good to me Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60696 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8640ea7962cf4f5502e0e63011ea2ae9f554258b Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Logging of VmPoolMonitor fixed
by ahadas＠redhat.com 14 Jul '16

14 Jul '16

Arik Hadas has submitted this change and it was merged. Change subject: core: Logging of VmPoolMonitor fixed ...................................................................... core: Logging of VmPoolMonitor fixed * Log message about failed VM prestarting was not always printed. E.g. when there was only one VM in the pool. Fixed. * Log level of failures during prestarting was changed from INFO to WARN. * Ids of VMs that failed to prestart were added to the log. Change-Id: I1384c893015b8abf91e2863d4181d14c807dee50 Bug-Url: https://bugzilla.redhat.com/1346848 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmPoolMonitor.java 1 file changed, 17 insertions(+), 18 deletions(-) Approvals: Jakub Niedermertl: Verified Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/60144 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1384c893015b8abf91e2863d4181d14c807dee50 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Don't try to build Java SDK 4 using standard CI
by eedri＠redhat.com 14 Jul '16

14 Jul '16

Eyal Edri has submitted this change and it was merged. Change subject: Don't try to build Java SDK 4 using standard CI ...................................................................... Don't try to build Java SDK 4 using standard CI There are currently two job configuration files for the Java SDK: * ovirt-engine-sdk-java_create-rpms.yaml This configuration file defines the jobs that build version 3 of the SDK, and those jobs don't use the "automation" directory used by the oVirt "standard CI" mechanism. Instead of that the job just calls "make" to do the build. * ovirt-engine-sdk-java_build-artifacts.yaml This configuration file defines the jobs that build version 4 of the SDK, using the "automation" directory. But currently we are trying to build version 4 of the SDK (from branch "sdk_4.0") from both job configuration files, and that will always fail because that branch does't contain a Makefile that can be executed just with the "make" command. This patch changes the job configuration files of the SDK so that we don't try to build version 4 of the Java SDK using the old approach based on the "make" command. Version 3 of the SDK will still be built the old way, as we don't expect big changes there and we don't consider it worth the effort. Change-Id: If4b8188ef5b68a1a94d5fdfa9375a7951cf49537 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M jobs/confs/projects/ovirt-engine-sdk-java/ovirt-engine-sdk-java_build-artifacts.yaml M jobs/confs/projects/ovirt-engine-sdk-java/ovirt-engine-sdk-java_create-rpms.yaml 2 files changed, 0 insertions(+), 8 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for review Eyal Edri: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60626 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If4b8188ef5b68a1a94d5fdfa9375a7951cf49537 Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(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]: ovirt-setup-lib: move from fc22 to fc24
by sbonazzo＠redhat.com 14 Jul '16

14 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-setup-lib: move from fc22 to fc24 ...................................................................... ovirt-setup-lib: move from fc22 to fc24 - Dropped fc22 since gone EOL - Moved master to fc24 Change-Id: Ifc2ce9555dffacc2dce3263fb4fb296240df2bef Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-setup-lib/ovirt-setup-lib_build-artifacts.yaml 1 file changed, 12 insertions(+), 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/60673 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifc2ce9555dffacc2dce3263fb4fb296240df2bef Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: replacing DataCenterWithoutSpm config value
by laravot＠redhat.com 13 Jul '16

13 Jul '16

Liron Aravot has submitted this change and it was merged. Change subject: core: replacing DataCenterWithoutSpm config value ...................................................................... core: replacing DataCenterWithoutSpm config value This patch replaces the DataCenterWithoutSpm with a new DataOperationsByHSM config value because of the change in the feature scope. Change-Id: Ie11ac48f0c501677ccc972af10d2cd2070f86532 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/AddImageFromScratchCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/BaseImagesCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/FeatureSupported.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsProxyData.java A packaging/dbscripts/upgrade/04_01_0150_delete_datacenterwithoutspm_conf_value.sql M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 7 files changed, 18 insertions(+), 17 deletions(-) Approvals: Jenkins CI: Passed CI tests Liron Aravot: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/60663 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie11ac48f0c501677ccc972af10d2cd2070f86532 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Move SDM checks to BaseImagesCommand
by laravot＠redhat.com 13 Jul '16

13 Jul '16

Liron Aravot has submitted this change and it was merged. Change subject: core: Move SDM checks to BaseImagesCommand ...................................................................... core: Move SDM checks to BaseImagesCommand This patch moves the isDataCenterWithSpm()/isDataCenterWithoutSpm() methods to BaseImagesCommand. Change-Id: I24ca127f873333368e20baaba2d0ac0a9fda3288 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/BaseImagesCommand.java 2 files changed, 14 insertions(+), 14 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Freddy Rolland: Looks good to me, but someone else must approve Liron Aravot: Verified -- To view, visit https://gerrit.ovirt.org/60662 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I24ca127f873333368e20baaba2d0ac0a9fda3288 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: ovirt-image-uploader: move from fc22 to fc24
by sbonazzo＠redhat.com 13 Jul '16

13 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-image-uploader: move from fc22 to fc24 ...................................................................... ovirt-image-uploader: move from fc22 to fc24 - Drop fc22 on 3.6 since fc22 gone EOL - Move master to fc24 - move 4.0 to its own branch Change-Id: I1f99f1e6d2787119288816d5fb90dcf21b919529 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-image-uploader/ovirt-image-uploader_standard.yaml 1 file changed, 11 insertions(+), 7 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60670 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1f99f1e6d2787119288816d5fb90dcf21b919529 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(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[ovirt-hosted-engine-setup-2.0]: build: post ovirt-hosted-engine-setup-2.0.1
by sbonazzo＠redhat.com 13 Jul '16

13 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-hosted-engine-setup-2.0.1 ...................................................................... build: post ovirt-hosted-engine-setup-2.0.1 Change-Id: I356786bf8bfa83266d2912f4357f00c1f642bdf9 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/build-artifacts.sh M configure.ac 2 files changed, 7 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60604 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I356786bf8bfa83266d2912f4357f00c1f642bdf9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-2.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>

1 0

Change in ovirt-hosted-engine-setup[refs/meta/config]: Allow everyone to push patches and mark them as verified.
by sbonazzo＠redhat.com 13 Jul '16

13 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Allow everyone to push patches and mark them as verified. ...................................................................... Allow everyone to push patches and mark them as verified. Change-Id: I58c4ccdaa37747d1a579e6a668d006fbb8823480 --- M project.config 1 file changed, 0 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/60665 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I58c4ccdaa37747d1a579e6a668d006fbb8823480 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: refs/meta/config 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[ovirt-engine-4.0]: restapi: Fix validation of pool when adding permission
by tnisan＠redhat.com 13 Jul '16

13 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: restapi: Fix validation of pool when adding permission ...................................................................... restapi: Fix validation of pool when adding permission When adding a permission we check that there is an object provided, a data center, a virtual machine, etc. But the validation for the pool uses an incorrect name, "vmpool" instead of "vmPool". As a result the validation always fail, and it is impossible to add a permission for a pool. This patch fixes that issue. Change-Id: I8be122dc6be608f5df85864be50d8faa0032806c Bug-Url: https://bugzilla.redhat.com/1354463 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit a6782813d66fecec7ff2f32e610375e4ac6527f9) --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendAssignedPermissionsResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendGroupAssignedPermissionsResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendUserAssignedPermissionsResourceTest.java 3 files changed, 3 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60645 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8be122dc6be608f5df85864be50d8faa0032806c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0