Change in ovirt-engine[ovirt-engine-3.6]: userportal, webadmin: default refresh
by oourfali@redhat.com
Oved Ourfali has submitted this change and it was merged.
Change subject: userportal,webadmin: default refresh
......................................................................
userportal,webadmin: default refresh
- Defaulted refresh to false
Change-Id: I8bf4eef17f82bf1552f89b6cf450533d3837921b
Signed-off-by: Alexander Wels <awels(a)redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryParametersBase.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendPermissionResourceTest.java
M frontend/webadmin/modules/frontend/src/test/java/org/ovirt/engine/ui/frontend/communication/VdcOperationManagerTest.java
3 files changed, 14 insertions(+), 15 deletions(-)
Approvals:
Alexander Wels: Verified
Jenkins CI: Passed CI tests
Oved Ourfali: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/63229
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8bf4eef17f82bf1552f89b6cf450533d3837921b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Alexander Wels <awels(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: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-engine[ovirt-engine-4.0]: userportal, webadmin: default refresh
by oourfali@redhat.com
Oved Ourfali has submitted this change and it was merged.
Change subject: userportal,webadmin: default refresh
......................................................................
userportal,webadmin: default refresh
- Defaulted refresh to false
Change-Id: I8bf4eef17f82bf1552f89b6cf450533d3837921b
Signed-off-by: Alexander Wels <awels(a)redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryParametersBase.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendPermissionResourceTest.java
M frontend/webadmin/modules/frontend/src/test/java/org/ovirt/engine/ui/frontend/communication/VdcOperationManagerTest.java
3 files changed, 10 insertions(+), 6 deletions(-)
Approvals:
Alexander Wels: Verified
Jenkins CI: Passed CI tests
Oved Ourfali: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/63225
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8bf4eef17f82bf1552f89b6cf450533d3837921b
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Alexander Wels <awels(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: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-engine[master]: core: Fix upgrade issue
by oourfali@redhat.com
Oved Ourfali has submitted this change and it was merged.
Change subject: core: Fix upgrade issue
......................................................................
core: Fix upgrade issue
Fixes upgrade issue by copying original
03_06_1790_add_vm_auto_login_column_to_user_profiles_table.sql (which
was by mistake not backported into ovirt-engine-3.6 branch during 3.6
version development) into
04_01_0270_add_vm_auto_login_column_to_user_profiles_table.sql. This
does not make any harm on upgrade from 4.0 to 4.1 and it will fix
direct upgrade from 3.6 to 4.1 if we will allow it in future.
Change-Id: I2e7f13a7a30aefde903c30ad5453201e53fe80e9
Bug-Url: https://bugzilla.redhat.com/1371515
Signed-off-by: Martin Perina <mperina(a)redhat.com>
---
A packaging/dbscripts/upgrade/04_01_0270_add_vm_auto_login_column_to_user_profiles_table.sql
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Martin Peřina: Verified
Eli Mesika: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Oved Ourfali: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/63200
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2e7f13a7a30aefde903c30ad5453201e53fe80e9
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-engine[master]: core: Always fix VMs status during fencing if host if off
by oourfali@redhat.com
Oved Ourfali has submitted this change and it was merged.
Change subject: core: Always fix VMs status during fencing if host if off
......................................................................
core: Always fix VMs status during fencing if host if off
Always fix VMs status during fencing flow when host is off (detected by
power management status command) and don't rely on successful execution
of power management start of the host.
Change-Id: I439d5a323092978a6cd58ceef93aa7f340e6dcf6
Bug-Url: https://bugzilla.redhat.com/1367557
Signed-off-by: Martin Perina <mperina(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/pm/RestartVdsCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Martin Peřina: Verified
Eli Mesika: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Arik Hadas: Looks good to me, but someone else must approve
Oved Ourfali: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/63224
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I439d5a323092978a6cd58ceef93aa7f340e6dcf6
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Peřina <mperina(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: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in jenkins[master]: restore check-patch for master on el7
by eedri@redhat.com
Eyal Edri has submitted this change and it was merged.
Change subject: restore check-patch for master on el7
......................................................................
restore check-patch for master on el7
Change-Id: I1cf8bed6066f4dec29b596ca5ed26421815d356d
Signed-off-by: Eyal Edri <eedri(a)redhat.com>
---
M jobs/confs/projects/vdsm/vdsm_standard.yaml
1 file changed, 0 insertions(+), 2 deletions(-)
Approvals:
Eyal Edri: Verified; Looks good to me; Ready for merge
Jenkins CI: Passed CI tests
Irit Goihman: Looks good to me
--
To view, visit https://gerrit.ovirt.org/63235
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1cf8bed6066f4dec29b596ca5ed26421815d356d
Gerrit-PatchSet: 2
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in jenkins[master]: Changing repos_check_closure jobs email recipients
by eedri@redhat.com
Eyal Edri has submitted this change and it was merged.
Change subject: Changing repos_check_closure jobs email recipients
......................................................................
Changing repos_check_closure jobs email recipients
Change-Id: I7a754be0630cfef2248f90be27db0fed5cdb8168
Signed-off-by: Sharon Naftaly <snaftaly(a)redhat.com>
---
M jobs/confs/yaml/templates/repos-check-closure.yaml
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Eyal Edri: Verified; Looks good to me; Passed CI tests; Ready for merge
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/63238
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7a754be0630cfef2248f90be27db0fed5cdb8168
Gerrit-PatchSet: 1
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Sharon Naftaly <snaftaly(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in otopi[otopi-1.5]: packaging: spec: exclude debug plugins
by didi@redhat.com
Yedidyah Bar David has submitted this change and it was merged.
Change subject: packaging: spec: exclude debug plugins
......................................................................
packaging: spec: exclude debug plugins
Without this, they are packaged both in 'otopi' and in
'otopi-debug-plugins'.
Bug-Url: https://bugzilla.redhat.com/1365752
Change-Id: I723a191f85c6c64d6f19d215c0f4597dd3305b51
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
---
M otopi.spec.in
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Yedidyah Bar David: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/63203
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I723a191f85c6c64d6f19d215c0f4597dd3305b51
Gerrit-PatchSet: 1
Gerrit-Project: otopi
Gerrit-Branch: otopi-1.5
Gerrit-Owner: Sandro Bonazzola <sbonazzo(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>
8 years, 3 months
Change in otopi[master]: packaging: spec: exclude debug plugins
by didi@redhat.com
Yedidyah Bar David has submitted this change and it was merged.
Change subject: packaging: spec: exclude debug plugins
......................................................................
packaging: spec: exclude debug plugins
Without this, they are packaged both in 'otopi' and in
'otopi-debug-plugins'.
Bug-Url: https://bugzilla.redhat.com/1365752
Change-Id: I723a191f85c6c64d6f19d215c0f4597dd3305b51
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
---
M otopi.spec.in
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Yedidyah Bar David: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/62183
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I723a191f85c6c64d6f19d215c0f4597dd3305b51
Gerrit-PatchSet: 3
Gerrit-Project: otopi
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>
8 years, 3 months
Change in ovirt-engine[ovirt-engine-4.0.4]: webadmin: don't cast from Guid to StorageDomain
by tnisan@redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: webadmin: don't cast from Guid to StorageDomain
......................................................................
webadmin: don't cast from Guid to StorageDomain
This patch removes a wrong cast from the return type of getEntity()
(Guid) to StorageDomain, and replaces it with a cast to Guid.
Change-Id: Icb41d6aa1f415414d8bc404e7288a5fe317df8f8
Bug-Url: https://bugzilla.redhat.com/1371304
Signed-off-by: Idan Shaby <ishaby(a)redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ImportVmFromExportDomainModel.java
1 file changed, 1 insertion(+), 2 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Allon Mureinik: Looks good to me, approved
Idan Shaby: Verified
--
To view, visit https://gerrit.ovirt.org/63118
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Icb41d6aa1f415414d8bc404e7288a5fe317df8f8
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0.4
Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-engine[ovirt-engine-4.0]: restapi: Update to model 4.0.33
by tnisan@redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: restapi: Update to model 4.0.33
......................................................................
restapi: Update to model 4.0.33
The more relevant changes included in the new version of the model are
the following:
* Add the `custom_scheduling_policy_properties` attribute to the
`Cluster` type.
* Add services and types to support transfer of images.
* Remove the `add` and `remove` operations of virtual machine
CDROMs.
* Fix the type of the `usages` attribute of the `Network` type, it
should be a list of values of the `NetworkUsage` enum, not a list
of strings.
Change-Id: I934f27cbb78fa73fe06df85b1673a94d2880b896
Related-To: https://bugzilla.redhat.com/1351576
Related-To: https://bugzilla.redhat.com/1337077
Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendApiResource.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmCdromResource.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmCdromsResource.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3NetworkInAdapter.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3NetworkOutAdapter.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/servers/V3VmCdromServer.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/servers/V3VmCdromsServer.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendClusterNetworkResourceTest.java
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/NetworkMapper.java
D backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/NetworkUsage.java
M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/NetworkMapperTest.java
M pom.xml
12 files changed, 50 insertions(+), 49 deletions(-)
Approvals:
Juan Hernandez: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
Amit Aviram: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/63121
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I934f27cbb78fa73fe06df85b1673a94d2880b896
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Amit Aviram <aaviram(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
8 years, 3 months