August 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.6]: core: Store the Vm ID rather then TEMPLATE ID
by piotr.kliczewski＠gmail.com 07 Aug '15

07 Aug '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: core: Store the Vm ID rather then TEMPLATE ID ...................................................................... core: Store the Vm ID rather then TEMPLATE ID The insertion of dedicated vm to host record took the template instead of vm id. Change-Id: I1c9fe6c69b127434a2c037264253cd61d7d30770 Bug-Url: https://bugzilla.redhat.com/1249631 Signed-off-by: Roy Golan <rgolan(a)redhat.com> (cherry picked from commit de1c5540b4a91db25a401b32bca2b10e760cc44e) --- M packaging/dbscripts/vms_sp.sql 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Roy Golan: Verified Omer Frenkel: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/44458 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1c9fe6c69b127434a2c037264253cd61d7d30770 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: webadmin: Handle null and empty list values in ListModelMult...
by piotr.kliczewski＠gmail.com 07 Aug '15

07 Aug '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: webadmin: Handle null and empty list values in ListModelMultipleSelectListBox ...................................................................... webadmin: Handle null and empty list values in ListModelMultipleSelectListBox The code assumed the values are always passed which isn't the case for new clusters and that caused NPE in GWT and dialogs to appear stuck on loading. Change-Id: I77fd69736080eb253f0c3259bda6e3108a2493a2 Bug-Url: https://bugzilla.redhat.com/1248509 Signed-off-by: Roy Golan <rgolan(a)redhat.com> (cherry picked from commit b697f2846a21027899202023b235c095f1333edb) --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/ListModelMultipleSelectListBox.java 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Roy Golan: Verified -- To view, visit https://gerrit.ovirt.org/44457 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I77fd69736080eb253f0c3259bda6e3108a2493a2 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: frontend: Localize SerialNumberPolicy enum
by piotr.kliczewski＠gmail.com 07 Aug '15

07 Aug '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: frontend: Localize SerialNumberPolicy enum ...................................................................... frontend: Localize SerialNumberPolicy enum Moved SerialNumberPolicy values from Enums -> LocalizedEnums so they could be properly translated. Change-Id: I6b0a1c6d23c6df77ddece6de0590a143fa3d3909 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1247889 Signed-off-by: Martin Betak <mbetak(a)redhat.com> --- M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Enums.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/LocalizedEnums.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Enums.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties 4 files changed, 9 insertions(+), 9 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Martin Betak: Verified -- To view, visit https://gerrit.ovirt.org/44402 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6b0a1c6d23c6df77ddece6de0590a143fa3d3909 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: restapi: vmpools missed the type part of the display section
by piotr.kliczewski＠gmail.com 07 Aug '15

07 Aug '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: restapi: vmpools missed the type part of the display section ...................................................................... restapi: vmpools missed the type part of the display section Change-Id: I0d7dc8196557ad4862e299c740a377ec36a48ce2 Bug-Url: https://bugzilla.redhat.com/1236497 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmPoolsResource.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Tomas Jelinek: Verified Ori Liel: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44397 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0d7dc8196557ad4862e299c740a377ec36a48ce2 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: engine: Fix LUN type for Fiber Channel
by piotr.kliczewski＠gmail.com 07 Aug '15

07 Aug '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: engine: Fix LUN type for Fiber Channel ...................................................................... engine: Fix LUN type for Fiber Channel Populate LUN LunType field for Fiber Channel with correct type instead of "Unknown". Change-Id: I51ab4dcd0d3ee87269d515544dd6caaba62cb1d1 Bug-Url: https://bugzilla.redhat.com/1179921 Signed-off-by: Fred Rolland <frolland(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/GetDeviceListVDSCommand.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/vdsbroker/GetDeviceListVDSCommandTest.java 2 files changed, 5 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Freddy Rolland: Verified -- To view, visit https://gerrit.ovirt.org/44472 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I51ab4dcd0d3ee87269d515544dd6caaba62cb1d1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: Set default number of artifacts to keep to 20.
by dcaroest＠redhat.com 07 Aug '15

07 Aug '15

David Caro has submitted this change and it was merged. Change subject: Set default number of artifacts to keep to 20. ...................................................................... Set default number of artifacts to keep to 20. Change-Id: I6efd2f203daf8cb31d8c08854d34e95c14eddeb9 Signed-off-by: Anton Marchukov <amarchuk(a)redhat.com> --- M jobs/confs/yaml/defaults/defaults.yaml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Anton Marchukov: Verified; Ready for review Jenkins CI: Passed CI tests David Caro: Looks good to me; Ready for merge -- To view, visit https://gerrit.ovirt.org/44541 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6efd2f203daf8cb31d8c08854d34e95c14eddeb9 Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(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

Change in jenkins[master]: Added ovirt-guest-agent jobs
by dcaroest＠redhat.com 07 Aug '15

07 Aug '15

David Caro has submitted this change and it was merged. Change subject: Added ovirt-guest-agent jobs ...................................................................... Added ovirt-guest-agent jobs Change-Id: I0a692654728af91ac9f300a0b07a6462f1a13cbc Signed-off-by: David Caro <dcaroest(a)redhat.com> --- A jobs/confs/projects/ovirt-guest-agent/ovirt-guest-agent_standard.yaml A jobs/confs/yaml/scms/ovirt-guest-agent.yaml 2 files changed, 20 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/44547 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0a692654728af91ac9f300a0b07a6462f1a13cbc Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(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: Jenkins CI 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]: drop FC20 on 3.5.z
by dcaroest＠redhat.com 07 Aug '15

07 Aug '15

David Caro has submitted this change and it was merged. Change subject: drop FC20 on 3.5.z ...................................................................... drop FC20 on 3.5.z 3.5.4 was the last 3.5.z release supporting fc20. Fedora users should wait for 3.6.0 while upgrading to fc22. Change-Id: I261ed154fa2c481218db16741282749a6efd17c2 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/otopi/otopi_create-rpms.yaml M jobs/confs/projects/ovirt-dwh/ovirt-dwh_create-rpms.yaml M jobs/confs/projects/ovirt-engine-extension-aaa-ldap/ovirt-engine-extension-aaa-ldap_create-rpms.yaml M jobs/confs/projects/ovirt-engine-sdk-java/ovirt-engine-sdk-java_create-rpms.yaml M jobs/confs/projects/ovirt-engine-sdk/ovirt-engine-sdk_build-artifacts.yaml M jobs/confs/projects/ovirt-host-deploy/ovirt-host-deploy_create-rpms.yaml M jobs/confs/projects/ovirt-hosted-engine-ha/ovirt-hosted-engine-ha_create-rpms.yaml M jobs/confs/projects/ovirt-hosted-engine-setup/ovirt-hosted-engine-setup_create-rpms.yaml M jobs/confs/projects/ovirt-image-uploader/ovirt-image-uploader_create-rpms.yaml M jobs/confs/projects/ovirt-image-uploader/ovirt-image-uploader_standard.yaml M jobs/confs/projects/ovirt-iso-uploader/ovirt-iso-uploader_create-rpms.yaml M jobs/confs/projects/ovirt-log-collector/ovirt-log-collector_create-rpms.yaml M jobs/confs/projects/ovirt-reports/ovirt-reports_create-rpms.yaml M jobs/confs/projects/vdsm-jsonrpc-java/vdsm-jsonrpc-java_create-rpms.yaml M jobs/confs/projects/vdsm/vdsm_create-rpms.yaml M jobs/confs/projects/vdsm/vdsm_install-rpm-sanity.yaml 16 files changed, 1 insertion(+), 43 deletions(-) Approvals: Sandro Bonazzola: Verified; Ready for review Jenkins CI: Passed CI tests David Caro: Verified; Looks good to me; Ready for merge -- To view, visit https://gerrit.ovirt.org/44546 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I261ed154fa2c481218db16741282749a6efd17c2 Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(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

Change in ovirt-guest-agent[master]: Added git dependency
by vfeenstr＠redhat.com 07 Aug '15

07 Aug '15

Vinzenz Feenstra has submitted this change and it was merged. Change subject: Added git dependency ...................................................................... Added git dependency It turns out that the inner make commands don't work without it, but don't fail either Change-Id: Iff44a3270c8e2c18f852d9c282fc24f39a7453cb Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M automation/check-patch.packages 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Vinzenz Feenstra: Looks good to me, approved David Caro: Verified -- To view, visit https://gerrit.ovirt.org/44549 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iff44a3270c8e2c18f852d9c282fc24f39a7453cb Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-ha[master]: broker: Set date header in mails
by msivak＠redhat.com 07 Aug '15

07 Aug '15

Martin Sivák has submitted this change and it was merged. Change subject: broker: Set date header in mails ...................................................................... broker: Set date header in mails RFC5322 requires a 'Date' header to be set. If it is not provided, mail clients might display strange send dates. Change-Id: I84c03008657992174bf2bce84b9ac7b5de37a94f Bug-Url: https://bugzilla.redhat.com/1229774 Signed-off-by: Roman Mohr <rmohr(a)redhat.com> --- M ovirt_hosted_engine_ha/broker/notifications.py 1 file changed, 6 insertions(+), 2 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Roman Mohr: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44505 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I84c03008657992174bf2bce84b9ac7b5de37a94f Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0