Change in ovirt-engine[ovirt-engine-3.6]: core: Store the Vm ID rather then TEMPLATE ID
by piotr.kliczewski＠gmail.com
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
9 years, 5 months
Change in ovirt-engine[ovirt-engine-3.6]: webadmin: Handle null and empty list values in ListModelMult...
by piotr.kliczewski＠gmail.com
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
9 years, 5 months
Change in ovirt-engine[ovirt-engine-3.6]: frontend: Localize SerialNumberPolicy enum
by piotr.kliczewski＠gmail.com
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
9 years, 5 months
Change in ovirt-engine[ovirt-engine-3.6]: restapi: vmpools missed the type part of the display section
by piotr.kliczewski＠gmail.com
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
9 years, 5 months
Change in ovirt-engine[ovirt-engine-3.6]: engine: Fix LUN type for Fiber Channel
by piotr.kliczewski＠gmail.com
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
9 years, 5 months
Change in jenkins[master]: Set default number of artifacts to keep to 20.
by dcaroest＠redhat.com
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>
9 years, 5 months
Change in jenkins[master]: Added ovirt-guest-agent jobs
by dcaroest＠redhat.com
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>
9 years, 5 months
Change in jenkins[master]: drop FC20 on 3.5.z
by dcaroest＠redhat.com
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>
9 years, 5 months
Change in ovirt-guest-agent[master]: Added git dependency
by vfeenstr＠redhat.com
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
9 years, 5 months
Change in ovirt-hosted-engine-ha[master]: broker: Set date header in mails
by msivak＠redhat.com
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
9 years, 5 months