September 2016 - Engine-commits - oVirt List Archives

Change in ovirt-release[ovirt-4.0]: build: post ovirt-release40-4.0.5
by sbonazzo＠redhat.com 28 Sep '16

28 Sep '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-release40-4.0.5 ...................................................................... build: post ovirt-release40-4.0.5 Change-Id: Ib6a93940e2112ef60d2e916ba8130119bd57f26c --- M automation/build-artifacts.sh M automation/check-patch.sh M configure.ac 3 files changed, 7 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/64932 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib6a93940e2112ef60d2e916ba8130119bd57f26c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> 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 ovirt-release[ovirt-4.0]: build: ovirt-release40-4.0.5
by sbonazzo＠redhat.com 28 Sep '16

28 Sep '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-release40-4.0.5 ...................................................................... build: ovirt-release40-4.0.5 Change-Id: I5aa1d1618b20074b208e7a614bca1db8a6ce5ba1 --- M automation/build-artifacts.sh M automation/check-patch.sh M configure.ac M ovirt-release40.spec.in 4 files changed, 5 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/64931 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5aa1d1618b20074b208e7a614bca1db8a6ce5ba1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> 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 ovirt-release[ovirt-4.0]: node: require ovirt-node-ng-nodectl
by sbonazzo＠redhat.com 28 Sep '16

28 Sep '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: require ovirt-node-ng-nodectl ...................................................................... node: require ovirt-node-ng-nodectl Nodectl provides an abstraction around imgbased functionality. Require it in NGN Change-Id: I88e88e824912b8bd9f86cd33e13d7bb7d57d09aa Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1372719 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1372717 Signed-off-by: Ryan Barry <rbarry(a)redhat.com> --- M ovirt-release40.spec.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/63426 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I88e88e824912b8bd9f86cd33e13d7bb7d57d09aa Gerrit-PatchSet: 5 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Ryan Barry <rbarry(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 ovirt-release[ovirt-4.0]: node: Add imgbase-clean-grub to the default services
by sbonazzo＠redhat.com 28 Sep '16

28 Sep '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: Add imgbase-clean-grub to the default services ...................................................................... node: Add imgbase-clean-grub to the default services Make sure it starts at boot time. Change-Id: Id10b0768c7b70f2c414c7add1a459b5777bbcd13 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1346872 Signed-off-by: Ryan Barry <rbarry(a)redhat.com> --- M ovirt-release40.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Sandro Bonazzola: Looks good to me, approved; Passed CI tests Ryan Barry: Verified -- To view, visit https://gerrit.ovirt.org/63506 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id10b0768c7b70f2c414c7add1a459b5777bbcd13 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Ryan Barry <rbarry(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 ovirt-engine[ovirt-engine-4.0.5]: core: Fix "succeeded" flag when retriving VM certificate
by tnisan＠redhat.com 28 Sep '16

28 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Fix "succeeded" flag when retriving VM certificate ...................................................................... core: Fix "succeeded" flag when retriving VM certificate Currently the command that retrieves the subject of the certificate of a virtual machine returns correctly the value, but doesn't set the "succeeded" flag to "true". As a result the API considers that operation failed, and doesn't populate the corresponding attributes of the VM object. This patch fixes that issue. Change-Id: Ibff406a5746fdecb6604ab2995ff468ad9ad74fc Bug-Url: https://bugzilla.redhat.com/1378933 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 308fa030d62649d75364a8d7d0a8b981b637034e) (cherry picked from commit 1d7fc77be9233c27c4a1069f257dc15993838e1f) --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetVdsCertificateSubjectByVmIdQuery.java 1 file changed, 8 insertions(+), 8 deletions(-) Approvals: Juan Hernandez: Verified Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/64925 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibff406a5746fdecb6604ab2995ff468ad9ad74fc Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: Move RemoveDiskSnapshots to CoCo
by laravot＠redhat.com 28 Sep '16

28 Sep '16

Liron Aravot has submitted this change and it was merged. Change subject: engine: Move RemoveDiskSnapshots to CoCo ...................................................................... engine: Move RemoveDiskSnapshots to CoCo Move disks snapshot cold merge from taskhandlers to CoCo. Change-Id: I53d5423b42fe1dba5a1e3813b8409cd63c655748 Signed-off-by: Fred Rolland <frolland(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/RemoveDiskSnapshotTaskHandler.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/RemoveDiskSnapshotsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/RemoveSnapshotSingleDiskCommand.java 4 files changed, 96 insertions(+), 279 deletions(-) Approvals: Freddy Rolland: Verified Liron Aravot: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/62232 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I53d5423b42fe1dba5a1e3813b8409cd63c655748 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Fred Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(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]: Fixes ovirt-engine-extension-aaa-misc experimental flow
by eedri＠redhat.com 28 Sep '16

28 Sep '16

Eyal Edri has submitted this change and it was merged. Change subject: Fixes ovirt-engine-extension-aaa-misc experimental flow ...................................................................... Fixes ovirt-engine-extension-aaa-misc experimental flow It was missing reprovision param Change-Id: Ia40e0fcc9e31f6325c84b3734737f2f7d7cfa78e Signed-off-by: Gil Shinar <gshinar(a)redhat.com> --- M jobs/confs/projects/ovirt-engine-extension-aaa-misc/ovirt-engine-extension-aaa-misc_create-rpms.yaml 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Eyal Edri: Looks good to me; Ready for merge Gil Shinar: Verified; Ready for review Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64914 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia40e0fcc9e31f6325c84b3734737f2f7d7cfa78e Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(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: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Fix ovirt-engine-wildfly experimental flow
by eedri＠redhat.com 28 Sep '16

28 Sep '16

Eyal Edri has submitted this change and it was merged. Change subject: Fix ovirt-engine-wildfly experimental flow ...................................................................... Fix ovirt-engine-wildfly experimental flow It was missing reprovision parameter and was configured not to be part of deploy packages Change-Id: I1daf53fb63b78aeb1ffe5590a8a20de51db57f80 Signed-off-by: Gil Shinar <gshinar(a)redhat.com> --- M jobs/confs/projects/ovirt-engine-wildfly-overlay/ovirt-engine-wildfly-overlay.yaml M jobs/confs/projects/ovirt-engine-wildfly/ovirt-engine-wildfly.yaml M jobs/confs/yaml/templates/create-rpms_mock.yaml 3 files changed, 8 insertions(+), 2 deletions(-) Approvals: Eyal Edri: Looks good to me; Passed CI tests; Ready for merge Gil Shinar: Verified -- To view, visit https://gerrit.ovirt.org/64920 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1daf53fb63b78aeb1ffe5590a8a20de51db57f80 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(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: Pavel Zhukov <pavel(a)zhukoff.net> Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Fixed experimental flow for nsis-simple-service-plugin
by eedri＠redhat.com 28 Sep '16

28 Sep '16

Eyal Edri has submitted this change and it was merged. Change subject: Fixed experimental flow for nsis-simple-service-plugin ...................................................................... Fixed experimental flow for nsis-simple-service-plugin Create rpms of nsis-simple-service-plugin weren't executing experimental jobs. They were missing some parameters for that to work properly Change-Id: I617f156a76da7d0f4b63f79eae024698e2f52cd5 Signed-off-by: Gil Shinar <gshinar(a)redhat.com> --- M jobs/confs/yaml/templates/create-rpms_mock.yaml 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Eyal Edri: Looks good to me; Ready for merge Gil Shinar: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64525 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I617f156a76da7d0f4b63f79eae024698e2f52cd5 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(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: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in gerrit-admin[master]: Simplify get_bug_url function
by sbendavi＠redhat.com 28 Sep '16

28 Sep '16

Shlomo Ben David has submitted this change and it was merged. Change subject: Simplify get_bug_url function ...................................................................... Simplify get_bug_url function It uses 'bug-url.*\d\b' regex as a search pattern in the commit message. If it exist, it will return a list of all bug-urls. Change-Id: I208a468adc9370789616366f250e2e49ce9e472f Signed-off-by: Shlomi Ben-David <sbendavi(a)redhat.com> --- M hooks/lib/bz.py 1 file changed, 5 insertions(+), 7 deletions(-) Approvals: Shlomo Ben David: Verified; Looks good to me, approved Gil Shinar: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64510 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I208a468adc9370789616366f250e2e49ce9e472f Gerrit-PatchSet: 1 Gerrit-Project: gerrit-admin Gerrit-Branch: master Gerrit-Owner: Shlomo Ben David <sbendavi(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: Shlomo Ben David <sbendavi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0