August 2015 - Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-setup[master]: packaging: add missing config for po/Makefile
by sbonazzo＠redhat.com 19 Aug '15

19 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: add missing config for po/Makefile ...................................................................... packaging: add missing config for po/Makefile Change-Id: I3d2f765b64b2e3ecf76cb206ee6fb0382c58e351 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/45067 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3d2f765b64b2e3ecf76cb206ee6fb0382c58e351 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup 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: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Add @Provides to watchdogs resource
by juan.hernandez＠redhat.com 19 Aug '15

19 Aug '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Add @Provides to watchdogs resource ...................................................................... restapi: Add @Provides to watchdogs resource Currently the watchdogs resource doesn't have a @Provides annotation. In theory this shouldn't be required as the annotation is in the base class. But in practice the application server reacts differently, in particular if the annotation isn't present the server won't use use the JAXB or JSON providers when the "Accept" header doesn't have a value. This means that requests like this: DELETE /vms/{vm:id}/watchdogs/{watchdog:id} Will fail with the following error message: Could not find MessageBodyWriter for response object of type: org.ovirt.engine.api.model.Action of media type: application/octet-stream This specific problem can be resolved adding the "Accept" header to the request, and that is good practice. For backwards compatibility sake this patch adds the annotation. Change-Id: I1bcb6263450be603b21e398196c6713ad599ce25 Bug-Url: https://bugzilla.redhat.com/1250046 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/WatchdogResource.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Ori Liel: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44910 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1bcb6263450be603b21e398196c6713ad599ce25 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: NPE on edit pool dialog
by tjelinek＠redhat.com 19 Aug '15

19 Aug '15

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: NPE on edit pool dialog ...................................................................... webadmin: NPE on edit pool dialog It has to be only in the templateWithVersion_SelectedItemChanged since the rest of the code is dependant on the selected template Change-Id: Ib16f40490ad7256e7f2e25c63283425eb2c2ae06 Bug-Url: https://bugzilla.redhat.com/1253258 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ExistingPoolModelBehavior.java 1 file changed, 0 insertions(+), 3 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Martin Betak: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/44944 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib16f40490ad7256e7f2e25c63283425eb2c2ae06 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-iso-uploader[master]: packaging: drop gcc reqirements
by sbonazzo＠redhat.com 19 Aug '15

19 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: drop gcc reqirements ...................................................................... packaging: drop gcc reqirements Dropping gcc requirements for gettext support See: http://git.savannah.gnu.org/cgit/gettext.git/tree/gettext-tools/examples/he… Change-Id: I626417f404d79a8160cd1f38690b1fbb605b1560 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/45063 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I626417f404d79a8160cd1f38690b1fbb605b1560 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader 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: automation(a)ovirt.org

1 0

Change in ovirt-iso-uploader[master]: nit: fix pep-8
by sbonazzo＠redhat.com 19 Aug '15

19 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: nit: fix pep-8 ...................................................................... nit: fix pep-8 Change-Id: Ic47ec5d7814a01672fa8261ee837d40a3d8db00b Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/__main__.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/45062 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic47ec5d7814a01672fa8261ee837d40a3d8db00b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader 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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: edit pool dialog not load when the DC is not up
by tjelinek＠redhat.com 19 Aug '15

19 Aug '15

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: edit pool dialog not load when the DC is not up ...................................................................... webadmin: edit pool dialog not load when the DC is not up The problem was that the PoolListModel started the progress by directly calling startProgress() wich is not good - it has to be handled by the infrastructure (e.g. AbstractModelBoundPopupPresenterWidget is handling the start/stop progress according to the AsyncOperation[Started/Completed]Event. Other fix needed was a null check on the spiceProxyEnabled since the model is not inited completely when the DC is not active. Change-Id: I8177155885dfb4c64785b905db614bc6b4ffe44a Bug-Url: https://bugzilla.redhat.com/1245599 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java 2 files changed, 1 insertion(+), 2 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Martin Betak: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/44998 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8177155885dfb4c64785b905db614bc6b4ffe44a Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-log-collector[master]: packaging: drop gcc reqirements
by sbonazzo＠redhat.com 19 Aug '15

19 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: drop gcc reqirements ...................................................................... packaging: drop gcc reqirements Dropping gcc requirements for gettext support See: http://git.savannah.gnu.org/cgit/gettext.git/tree/gettext-tools/examples/he… Change-Id: I1460bcbe463b5dae3210d2970124ce5e56414181 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/45039 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1460bcbe463b5dae3210d2970124ce5e56414181 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-log-collector 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: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: Fix permission issue after running repoman
by sbonazzo＠redhat.com 19 Aug '15

19 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: Fix permission issue after running repoman ...................................................................... Fix permission issue after running repoman If your scripts generated root-owned files in the inner tree but not in the root project dir, they would not be chowned to the user running mock Change-Id: Ib73b26803fc581704c56795cc30ff18efa1e4ab5 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M mock_configs/mock_runner.sh 1 file changed, 10 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge David Caro: Verified; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/45053 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib73b26803fc581704c56795cc30ff18efa1e4ab5 Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in jenkins[master]: ioprocess: Add 0.15 branch and drop el6 in master
by sbonazzo＠redhat.com 19 Aug '15

19 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: ioprocess: Add 0.15 branch and drop el6 in master ...................................................................... ioprocess: Add 0.15 branch and drop el6 in master Added also to the 3.5 publisher Change-Id: I730ca679faa401c084fedd1f0a47dc5957b7bca4 Signed-off-by: Dima Kuznetsov <dkuznets(a)redhat.com> Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/projects/ioprocess/ioprocess_create-rpms.yaml M jobs/confs/projects/ovirt/publish-rpms-nightly-3.5.yaml 2 files changed, 27 insertions(+), 10 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge Jenkins CI: Passed CI tests David Caro: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/44484 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I730ca679faa401c084fedd1f0a47dc5957b7bca4 Gerrit-PatchSet: 9 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Dima Kuznetsov <dkuznets(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Dima Kuznetsov <dkuznets(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yeela Kaplan <ykaplan(a)redhat.com>

1 0

Change in jenkins[master]: publish-rpms-nightly-3.5
by sbonazzo＠redhat.com 19 Aug '15

19 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: publish-rpms-nightly-3.5 ...................................................................... publish-rpms-nightly-3.5 Also adds some fc21/fc22 jobs that were missing. Ignore the ioprocess jobs, will be updated with the next patch Change-Id: Ie06056057f1a7361b84e9ed1c6c85f42e8fe8a6c Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> Signed-off-by: David Caro <dcaroest(a)redhat.com> --- A jobs/confs/projects/ovirt/publish-rpms-nightly-3.5.yaml 1 file changed, 139 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge Jenkins CI: Passed CI tests David Caro: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/44996 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie06056057f1a7361b84e9ed1c6c85f42e8fe8a6c Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0