December 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: userportal, webadmin: Custom dropdown chrome
by awels＠redhat.com 15 Dec '14

15 Dec '14

Alexander Wels has submitted this change and it was merged. Change subject: userportal,webadmin: Custom dropdown chrome ...................................................................... userportal,webadmin: Custom dropdown chrome - The height in the custom drop downs in chrome was too small, this patches fixes that issue Change-Id: I71b3cf9ef250647c18e08cccc8c83c21f7474477 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1173660 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/BaseListModelSuggestBox.java 1 file changed, 6 insertions(+), 4 deletions(-) Approvals: Alexander Wels: Verified Greg Sheremeta: Looks good to me, but someone else must approve Vojtech Szocs: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36134 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I71b3cf9ef250647c18e08cccc8c83c21f7474477 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: userportal, webadmin: tabs bleed through report menu
by awels＠redhat.com 15 Dec '14

15 Dec '14

Alexander Wels has submitted this change and it was merged. Change subject: userportal,webadmin: tabs bleed through report menu ...................................................................... userportal,webadmin: tabs bleed through report menu - Fixed issue where the sub tab tabs bleed through the reports menu. Change-Id: I5b3bfc89adcb05627fca4757e2931689936f8255 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1157300 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/ScrollableTabBarView.ui.xml 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Alexander Wels: Verified Greg Sheremeta: Looks good to me, but someone else must approve Vojtech Szocs: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36085 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5b3bfc89adcb05627fca4757e2931689936f8255 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Enforce maximal QoS commitment on interface
by lvernia＠redhat.com 15 Dec '14

15 Dec '14

Lior Vernia has submitted this change and it was merged. Change subject: engine: Enforce maximal QoS commitment on interface ...................................................................... engine: Enforce maximal QoS commitment on interface Made sure that no more than 75% of a an interface's capacity can be committed by QoS, as overcommitment may lead to unexpected and potentially destructive behavior. As part of this validation, also made sure all interface speeds are either reported by vdsm or can be computed in the engine (newly-created bonds); otherwise fail, as we cannot guarantee that the user hasn't over-committed. Change-Id: Icc89f4193d6b7b590c6e1e053af74ab23bc77a11 Signed-off-by: Lior Vernia <lvernia(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/SetupNetworksHelper.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/host/SetupNetworksHelperTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/network/NetworkInterface.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 7 files changed, 383 insertions(+), 24 deletions(-) Approvals: Lior Vernia: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34133 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icc89f4193d6b7b590c6e1e053af74ab23bc77a11 Gerrit-PatchSet: 20 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.2]: fixed long line to make pep8 happy
by msivak＠redhat.com 15 Dec '14

15 Dec '14

Martin Sivák has submitted this change and it was merged. Change subject: fixed long line to make pep8 happy ...................................................................... fixed long line to make pep8 happy Change-Id: Ia8474a3c97e600a285aa3f843e8fb96ae4ebcf37 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M ovirt_hosted_engine_ha/lib/storage_backends.py 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Martin Sivák: Looks good to me, approved Jiří Moskovčák: Verified -- To view, visit http://gerrit.ovirt.org/36180 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia8474a3c97e600a285aa3f843e8fb96ae4ebcf37 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.2 Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-ha[master]: fixed long line to make pep8 happy
by msivak＠redhat.com 15 Dec '14

15 Dec '14

Martin Sivák has submitted this change and it was merged. Change subject: fixed long line to make pep8 happy ...................................................................... fixed long line to make pep8 happy Change-Id: Ia8474a3c97e600a285aa3f843e8fb96ae4ebcf37 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M ovirt_hosted_engine_ha/lib/storage_backends.py 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Martin Sivák: Looks good to me, approved Jiří Moskovčák: Verified -- To view, visit http://gerrit.ovirt.org/36179 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia8474a3c97e600a285aa3f843e8fb96ae4ebcf37 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Don't validate missing cloud-init enums
by juan.hernandez＠redhat.com 15 Dec '14

15 Dec '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Don't validate missing cloud-init enums ...................................................................... restapi: Don't validate missing cloud-init enums Currently we blindly try to validate enums included in cloud-init specifications, but they may be missing. As they aren't declared as mandatory this patch changes the validator so it won't try to validate them when they are missing. Change-Id: If7c29a92909fb9f436b06d2ed534fe2cbc34a293 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/validation/CloudInitValidator.java 1 file changed, 14 insertions(+), 9 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36081 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If7c29a92909fb9f436b06d2ed534fe2cbc34a293 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: Avoid NPE when enum has no value
by juan.hernandez＠redhat.com 15 Dec '14

15 Dec '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Avoid NPE when enum has no value ...................................................................... restapi: Avoid NPE when enum has no value The mechanism that we currently use to validate enum values donsn't take into account that the values may be null. When this happens a NPE is triggered and logged, instead of generating a user friendly message. This patch changes the validation logic so that it will generate a reasonable error message. Change-Id: Ie8e2ba41729bcbbf5b24a64e51dd258c039bed4e Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/common/jaxrs/src/main/java/org/ovirt/engine/api/common/util/EnumValidator.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36080 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie8e2ba41729bcbbf5b24a64e51dd258c039bed4e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: OpenStack network provider
by juan.hernandez＠redhat.com 15 Dec '14

15 Dec '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: OpenStack network provider ...................................................................... restapi: OpenStack network provider This patch adds the resources for the OpenStack neetwork providers. The representation of the resource will look like this: GET /openstackneetworkproviders <openstack_network_providers> <openstack_network_provider id="{provider:id}">...</openstack_network_provider> ... </openstack_network_providers> GET /openstacknetworkproviders/{provider:id} <openstack_network_provider id="{provider:id}"> <name>myprovider</name> <description>My provider</description> <url>http://neutron.example.com</url> <username>myuser</username> <password>mypass</password> <tenant_name>mytenant</tenant_name> <plugin_type>open_vswitch<plugin_type> <agent_configuration> <network_mappings>...</network_mappings> <broker_type>qpid|rabbit_mq<broker_type> <address>...</address> <port>...</port> <username>...</username> <password>...</password> </agent_configuration> <properties> <property> <name>prop1</name> <value>value1</myvalue> </property> <property> <name>prop2</name> <value>value2</myvalue> </property> </properties> </openstack_network_provider> The providers collection will support listing, getting, adding and removing providers, with the usual methods. The provider resource will support getting, deleting and updating the provider, with the usual methods. In addition it will support the "testconnectivity" and "importcertificates" operations. The first used to check the connectivity with the external provider: POST /openstacknetworkproviders/{provider:id}/testconnectivity <action/> The second is used to import the chain of certificates of the external provider: POST /openstacknetworkproviders/{provider:id}/importcertificates <action/> The provider resource includes sub-collections for certificates and networks. For certificates it will look like this: GET /openstacknetworkproviders/{provider:id}/certificates <certificates> <certificate id="{host:id}">...</certificate> ... </certificates> GET /openstacknetworkproviders/{provider:id}/certificates/{certificate:id} <certificate id="{certificate:id}"> <subject>CN=neutron.example.com</subject> <content>...</content> </certificate> For networks: GET /openstacknetworkproviders/{provider:id}/networks <openstack_networks> <openstack_network id="{network:id}">...</openstack_network> ... </openstack_networks> GET /openstacknetworkproviders/{provider:id}/networks/{network:id} <openstack_network id="{networkd:id}"> <name>mynetwork</name> </openstack_network> The network resource will in turn include a sub-collection for sub-networks: GET /openstacknetworproviders/{provider:id}/networks/{network:id}/subnets <openstack_subnets> <openstack_subnet id="{subnet:id}">...<openstack_subnet> ... </openstack_subnets> GET /openstacknetworproviders/{provider:id}/networks/{network:id}/subnets/{subnet:id} <openstack_subnet id="{subnet:id}"> <cidr>...</cidr> <ip_version>v4|v6<ip_version> <gateway>192.168.122.1</gateway> <dns_servers> <dns_server>192.168.122.1</dns_server> <dns_server>192.168.122.2</dns_server> ... </dns_servers> </openstack_subnet> Change-Id: I0ff73ad3dfc3c220716cd6ec8ca635f5c3bb348d Bug-Url: https://bugzilla.redhat.com/1132259 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/network/AddSubnetToProviderCommand.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/network/GetExternalSubnetsOnProviderByExternalNetworkQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/AddExternalSubnetParameters.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetExternalSubnetsOnProviderByExternalNetworkQueryParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/MessageBrokerType.java A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/NetworkPluginType.java A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/openstack/OpenStackNetworkProviderResource.java A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/openstack/OpenStackNetworkProvidersResource.java A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/openstack/OpenStackNetworkResource.java A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/openstack/OpenStackNetworksResource.java A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/openstack/OpenStackSubnetResource.java A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/openstack/OpenStackSubnetsResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/utils/ApiRootLinksCreator.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/utils/LinkHelper.java M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/BackendApplication.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendCapabilitiesResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/openstack/BackendOpenStackNetworkProviderResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/openstack/BackendOpenStackNetworkProvidersResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/openstack/BackendOpenStackNetworkResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/openstack/BackendOpenStackNetworksResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/openstack/BackendOpenStackSubnetResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/openstack/BackendOpenStackSubnetsResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/OpenStackNetworkProviderValidator.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/openstack/OpenStackSubnetValidator.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendApiResourceTest.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/openstack/BackendOpenStackNetworkProviderResourceTest.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/openstack/BackendOpenStackNetworkProvidersResourceTest.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/openstack/BackendOpenStackNetworkResourceTest.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/openstack/BackendOpenStackNetworksResourceTest.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/openstack/BackendOpenStackSubnetResourceTest.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/openstack/BackendOpenStackSubnetsResourceTest.java A backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/IpVersionMapper.java A backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/openstack/OpenStackNetworkMapper.java A backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/openstack/OpenStackNetworkProviderMapper.java A backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/openstack/OpenStackSubnetMapper.java A backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/openstack/OpenStackNetworkMapperTest.java A backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/openstack/OpenStackNetworkProviderMapperTest.java A backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/openstack/OpenStackSubnetMapperTest.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NewNetworkModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/NewExternalSubnetModel.java 43 files changed, 2,550 insertions(+), 37 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33087 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0ff73ad3dfc3c220716cd6ec8ca635f5c3bb348d Gerrit-PatchSet: 22 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Use Jackson 1.9.9
by juan.hernandez＠redhat.com 15 Dec '14

15 Dec '14

Juan Hernandez has submitted this change and it was merged. Change subject: core: Use Jackson 1.9.9 ...................................................................... core: Use Jackson 1.9.9 The version of the OpenStack Java SDK that we use requires at least Jackson 1.9.9 in order to work correctly, otherwise responses received from Neutron aren't parsed correctly. This patch changes the dependencies module so that the correct version of Jackson will be bundled with the engine, overriding the one provided by the application server. Change-Id: I55d02ecd385c86a32a69f154610d4093aea9b74e Bug-Url: https://bugzilla.redhat.com/1064231 Related-To: https://bugzilla.redhat.com/1132259 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/dependencies/common/pom.xml A backend/manager/dependencies/common/src/main/modules/org/codehaus/jackson/jackson-core-asl/main/module.xml A backend/manager/dependencies/common/src/main/modules/org/codehaus/jackson/jackson-jaxrs/main/module.xml A backend/manager/dependencies/common/src/main/modules/org/codehaus/jackson/jackson-mapper-asl/main/module.xml A backend/manager/dependencies/common/src/main/modules/org/codehaus/jackson/jackson-xc/main/module.xml M backend/manager/modules/restapi/interface/definition/pom.xml M backend/manager/modules/vdsbroker/pom.xml M pom.xml 8 files changed, 142 insertions(+), 8 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Moti Asayag: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/33959 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I55d02ecd385c86a32a69f154610d4093aea9b74e Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.2]: don't use vdsm backend when there is not valid vol or imgUUID
by msivak＠redhat.com 15 Dec '14

15 Dec '14

Martin Sivák has submitted this change and it was merged. Change subject: don't use vdsm backend when there is not valid vol or imgUUID ...................................................................... don't use vdsm backend when there is not valid vol or imgUUID Change-Id: Idcd181d260f4f20676a4124b899575b42935753c Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1171452 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/hosted_engine.py M ovirt_hosted_engine_ha/lib/storage_backends.py 2 files changed, 3 insertions(+), 1 deletion(-) Approvals: Martin Sivák: Looks good to me, approved Jiří Moskovčák: Verified -- To view, visit http://gerrit.ovirt.org/36174 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idcd181d260f4f20676a4124b899575b42935753c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.2 Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0