- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: Changing OpenstackNetworkProviderProperties read-onl...
by alkaplan＠redhat.com 10 May '16

10 May '16

Alona Kaplan has submitted this change and it was merged. Change subject: engine: Changing OpenstackNetworkProviderProperties read-only from Boolean to boolean ...................................................................... engine: Changing OpenstackNetworkProviderProperties read-only from Boolean to boolean The value can either be true/false, and not undeterminded (null) Change-Id: I1830ac6cf51bb988852fde2f0cd5555989f0359b Bug-Url: https://bugzilla.redhat.com/1314375 Signed-off-by: mirecki <mmirecki(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/OpenstackNetworkProviderProperties.java 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Marcin Mirecki: Verified Martin Mucha: Looks good to me, but someone else must approve Alona Kaplan: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/56591 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1830ac6cf51bb988852fde2f0cd5555989f0359b Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Adding read only option to openstack providers
by alkaplan＠redhat.com 10 May '16

10 May '16

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: Adding read only option to openstack providers ...................................................................... webadmin: Adding read only option to openstack providers Openstack provider should also have the read only option, just like the external provider. Change-Id: Iba2a7e3232d14ef6462ee80273d9782ca701bab3 Bug-Url: https://bugzilla.redhat.com/1314375 Signed-off-by: mirecki <mmirecki(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/ProviderType.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/RemoveNetworksModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/networks/NetworkExternalSubnetListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/ProviderModel.java 5 files changed, 14 insertions(+), 22 deletions(-) Approvals: Marcin Mirecki: Verified Martin Mucha: Looks good to me, but someone else must approve Alona Kaplan: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/56595 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iba2a7e3232d14ef6462ee80273d9782ca701bab3 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Adding readOnly property to NetworkProvider
by alkaplan＠redhat.com 10 May '16

10 May '16

Alona Kaplan has submitted this change and it was merged. Change subject: restapi: Adding readOnly property to NetworkProvider ...................................................................... restapi: Adding readOnly property to NetworkProvider Change-Id: I0267f5818c7a47436ff02c9907a74592cdf72c62 Bug-Url: https://bugzilla.redhat.com/1314375 Signed-off-by: mirecki <mmirecki(a)redhat.com> --- M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/openstack/OpenStackNetworkProviderMapper.java 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Marcin Mirecki: Verified Alona Kaplan: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56612 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0267f5818c7a47436ff02c9907a74592cdf72c62 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: Add query fetching providers by multiple provider types
by alkaplan＠redhat.com 10 May '16

10 May '16

Alona Kaplan has submitted this change and it was merged. Change subject: engine: Add query fetching providers by multiple provider types ...................................................................... engine: Add query fetching providers by multiple provider types Change-Id: Idf5c9745d9fec15cd06b2db73e93e1fdc078aee7 Bug-Url: https://bugzilla.redhat.com/1314375 Signed-off-by: mirecki <mmirecki(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/provider/foreman/EngineForemanProviderFinder.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/GetAllProvidersQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/storage/CinderProviderValidator.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetAllProvidersParameters.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/provider/ProviderDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/provider/ProviderDaoImpl.java M packaging/dbscripts/providers_sp.sql 7 files changed, 26 insertions(+), 24 deletions(-) Approvals: Eli Mesika: Looks good to me, but someone else must approve Marcin Mirecki: Verified Martin Mucha: Looks good to me, but someone else must approve Alona Kaplan: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/56594 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idf5c9745d9fec15cd06b2db73e93e1fdc078aee7 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: Require cockpit-ovirt-dashboard, not cockpit-ovirt
by sbonazzo＠redhat.com 10 May '16

10 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Require cockpit-ovirt-dashboard, not cockpit-ovirt ...................................................................... Require cockpit-ovirt-dashboard, not cockpit-ovirt The package was renamed. Require the new package name instead Change-Id: I23d3b14d09680f1f57c1140bd8f6c0f2289e0505 Signed-off-by: Ryan Barry <rbarry(a)redhat.com> --- M ovirt-release40.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57281 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1958977cc241a96ffaa9800e22cfb7325ec87529 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Ryan Barry <rbarry(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-release[master]: Require cockpit-ovirt-dashboard, not cockpit-ovirt
by sbonazzo＠redhat.com 10 May '16

10 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Require cockpit-ovirt-dashboard, not cockpit-ovirt ...................................................................... Require cockpit-ovirt-dashboard, not cockpit-ovirt The package was renamed. Require the new package name instead Change-Id: I23d3b14d09680f1f57c1140bd8f6c0f2289e0505 Signed-off-by: Ryan Barry <rbarry(a)redhat.com> --- M ovirt-release-master.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57280 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I23d3b14d09680f1f57c1140bd8f6c0f2289e0505 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Ryan Barry <rbarry(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: show cluster compatibility of host in general subtab
by mperina＠redhat.com 09 May '16

09 May '16

Martin Peřina has submitted this change and it was merged. Change subject: webadmin: show cluster compatibility of host in general subtab ...................................................................... webadmin: show cluster compatibility of host in general subtab Change-Id: Ia2b524b27ce5b2785fc80235db308cc61a3544d5 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1083661 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostGeneralModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/host/SubTabHostGeneralInfoView.java 3 files changed, 19 insertions(+), 0 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Ondra Machacek: Verified Martin Peřina: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/57115 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia2b524b27ce5b2785fc80235db308cc61a3544d5 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Fix system permissions server
by juan.hernandez＠redhat.com 09 May '16

09 May '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Fix system permissions server ...................................................................... restapi: Fix system permissions server In version 3 of the engine the system permissions resource was implemented using an empty interface as a placeholder for the required JAX-RS annotations. But in version 4 of the engine the classes that implement the V3 compatibility layer don't use inheritance, and as a result this resource is actually empty. This patch fixes that, removing the empty class and replacing it with the generic assigned permissions implementation. Change-Id: I2f8d5ac0e0107538e4aae917b1f71fade0f670f4 Bug-Url: https://bugzilla.redhat.com/1334096 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- D backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/servers/V3SystemPermissionsServer.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/servers/V3SystemServer.java 2 files changed, 2 insertions(+), 31 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57237 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2f8d5ac0e0107538e4aae917b1f71fade0f670f4 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-host-deploy[master]: common: IDEA project files gitignored
by sbonazzo＠redhat.com 09 May '16

09 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: common: IDEA project files gitignored ...................................................................... common: IDEA project files gitignored Change-Id: I27e49cb1a3a5e4ca128e9c0fd557e26f2c234377 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M .gitignore 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57154 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I27e49cb1a3a5e4ca128e9c0fd557e26f2c234377 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(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]: dashboard: Serialize utilization trend as lowercase
by awels＠redhat.com 09 May '16

09 May '16

Alexander Wels has submitted this change and it was merged. Change subject: dashboard: Serialize utilization trend as lowercase ...................................................................... dashboard: Serialize utilization trend as lowercase Using @JsonValue to customize value serialization. Not adding @JsonCreator as we don't do deserialization. Change-Id: I266499a842102c16308c5756edf2222e95300bde Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/server/dashboard/UtilizedEntity.java 1 file changed, 10 insertions(+), 0 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Vojtech Szocs: Verified -- To view, visit https://gerrit.ovirt.org/57152 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I266499a842102c16308c5756edf2222e95300bde Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0