February 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: Switch documentation to AsciiDoc
by juan.hernandez＠redhat.com 11 Feb '16

11 Feb '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Switch documentation to AsciiDoc ...................................................................... restapi: Switch documentation to AsciiDoc This patch changes the documentation (the README.md file) to use AscciDoc instead of Markdown (thus renaming it to README.adoc). Change-Id: I4e534039332ccbcac4555ca62c87e7263b9a5a05 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A backend/manager/modules/restapi/README.adoc D backend/manager/modules/restapi/README.md 2 files changed, 1,436 insertions(+), 1,153 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/53397 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4e534039332ccbcac4555ca62c87e7263b9a5a05 Gerrit-PatchSet: 1 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: packaging: fix ovirt-engine-setup-plugin-ovirt-engine depend...
by sbonazzo＠redhat.com 11 Feb '16

11 Feb '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: fix ovirt-engine-setup-plugin-ovirt-engine dependencies ...................................................................... packaging: fix ovirt-engine-setup-plugin-ovirt-engine dependencies ovirt-engine-setup-plugin-ovirt-engine package should depend on the vmconsole-proxy-helper setup plugin. Change-Id: Ie3602519ba1d3cf71bf6a5732a1890b948c04038 Bug-Url: https://bugzilla.redhat.com/1306178 Signed-off-by: Rafael Martins <rmartins(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved; Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve Rafael Martins: Verified -- To view, visit https://gerrit.ovirt.org/53339 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie3602519ba1d3cf71bf6a5732a1890b948c04038 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: V3 compatibility for add VM parameters
by juan.hernandez＠redhat.com 11 Feb '16

11 Feb '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: V3 compatibility for add VM parameters ...................................................................... restapi: V3 compatibility for add VM parameters In version 4 of the API the "force", "clone", and "detach_only" parameters used to add and remove virtual machines have been replaced by matrix parameters. Now that the API supports multiple versions this parameter can be rea-added without modifying the V4 implementation. Change-Id: I8324cd116d53dcc58b8fbc85eb79901d9f79b4b8 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/V3Adapter.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/servers/V3VmServer.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/servers/V3VmsServer.java 3 files changed, 44 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/53055 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8324cd116d53dcc58b8fbc85eb79901d9f79b4b8 Gerrit-PatchSet: 4 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: V3 compatibility for "force" parameter to add host
by juan.hernandez＠redhat.com 11 Feb '16

11 Feb '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: V3 compatibility for "force" parameter to add host ...................................................................... restapi: V3 compatibility for "force" parameter to add host In version 4 of the API the "force" parameter of the operation to add a host was removed in favour of the "refresh" operation. Now that the API supports multiple versions this parameter can be rea-added without modifying the V4 implementation. Change-Id: I0bf7146532466ac87c9859902e0dc8413af7e747 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/servers/V3HostServer.java 1 file changed, 16 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/53048 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0bf7146532466ac87c9859902e0dc8413af7e747 Gerrit-PatchSet: 5 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Improve logging of VM prestart errors
by ahadas＠redhat.com 11 Feb '16

11 Feb '16

Arik Hadas has submitted this change and it was merged. Change subject: core: Improve logging of VM prestart errors ...................................................................... core: Improve logging of VM prestart errors Checks performed inside VmPoolCommandBase.isVmFree() produce messages in the case of an error that give more details about the cause of the error. Currently these messages are simply discarded making debugging of VM prestart errors more difficult. This patch directs these messages into the engine log. The failure messages are logged only if VmPoolManager failed to prestart the required number of VMs. To avoid excessive logging, similar messages are grouped together and variable substitutions are ignored. Change-Id: Ib7a79c757d253bc484e9b18c18da6f918c9b41eb Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmPoolCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmPoolMonitor.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/ErrorMessageUtils.java 3 files changed, 65 insertions(+), 18 deletions(-) Approvals: Jenkins CI: Passed CI tests Shmuel Leib Melamud: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/51005 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib7a79c757d253bc484e9b18c18da6f918c9b41eb Gerrit-PatchSet: 7 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: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: epel7: Update python-crypto dependency
by sbonazzo＠redhat.com 11 Feb '16

11 Feb '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: epel7: Update python-crypto dependency ...................................................................... epel7: Update python-crypto dependency Change-Id: I84393d2a399dc026a2aaeb48d9d9740206e74c63 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M ovirt-release-3.6/ovirt-el7-deps.repo.in M ovirt-release-master/ovirt-el7-deps.repo.in 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/53387 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I84393d2a399dc026a2aaeb48d9d9740206e74c63 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(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]: jsonrpc: version bump
by sbonazzo＠redhat.com 11 Feb '16

11 Feb '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: jsonrpc: version bump ...................................................................... jsonrpc: version bump Branch: master (only) Bug-Url: https://bugzilla.redhat.com/1296930 Change-Id: I65f12645ba7bf143992f4136cf8440fad8d15224 Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> --- M ovirt-engine.spec.in M pom.xml 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Piotr Kliczewski: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/53274 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I65f12645ba7bf143992f4136cf8440fad8d15224 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Added copy-out option to mock_all
by sbonazzo＠redhat.com 11 Feb '16

11 Feb '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Added copy-out option to mock_all ...................................................................... Added copy-out option to mock_all Using it also on ovirt-wgt job to extract the iso from the chroot Change-Id: I3e24f1a7420f5f583c820b5806efd1bf4b8f0f73 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/projects/ovirt-wgt/ovirt-wgt_create-rpms.yaml M jobs/confs/shell-scripts/mock_build_onlyrpm.sh M jobs/confs/yaml/templates/create-rpms_mock.yaml 3 files changed, 23 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge Jenkins CI: Passed CI tests David Caro: Verified; Ready for review -- To view, visit https://gerrit.ovirt.org/53369 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3e24f1a7420f5f583c820b5806efd1bf4b8f0f73 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: added help for Storage > ISO > DC > Attach
by gshereme＠redhat.com 10 Feb '16

10 Feb '16

Greg Sheremeta has submitted this change and it was merged. Change subject: webadmin: added help for Storage > ISO > DC > Attach ...................................................................... webadmin: added help for Storage > ISO > DC > Attach Change-Id: Id7310bf2a623dd049d66f88425ee7cbcc81a9270 Bug-Url: https://bugzilla.redhat.com/1174045 Signed-off-by: Greg Sheremeta <gshereme(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageDataCenterListModel.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Greg Sheremeta: Verified -- To view, visit https://gerrit.ovirt.org/53321 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id7310bf2a623dd049d66f88425ee7cbcc81a9270 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Fix name of V3 SELinux adapter
by juan.hernandez＠redhat.com 10 Feb '16

10 Feb '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Fix name of V3 SELinux adapter ...................................................................... restapi: Fix name of V3 SELinux adapter It needs to be named *SE*, not *Se*, otherwise it can't be found. Change-Id: Ib2d80b377a721e2ea9d5e5fea7b1be33a083b011 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- R backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3SELinuxOutAdapter.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Idan Shaby: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/53354 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib2d80b377a721e2ea9d5e5fea7b1be33a083b011 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(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