Moti Asayag has uploaded a new change for review.
Change subject: engine: The wipe after delete should not be null
......................................................................
engine: The wipe after delete should not be null
When adding an image repository provider, a storage
domain is being created for it. However, the wipe-after-delete
option must be provided due its not-null constraint.
Change-Id: I89de7410aa25073dfa60c40b99589b164cdfe6ea
Signed-off-by: Moti Asayag <masa...@redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/storage/OpenStackImageProviderProxy.java
1 file changed, 10 insertions(+), 9 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/52/36852/1
diff --git
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/storage/OpenStackImageProviderProxy.java
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/storage/OpenStackImageProviderProxy.java
index 076ad0f..c64ae99 100644
---
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/storage/OpenStackImageProviderProxy.java
+++
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/storage/OpenStackImageProviderProxy.java
@@ -8,15 +8,6 @@
import java.util.List;
import java.util.Map;
-import com.woorea.openstack.base.client.HttpMethod;
-import com.woorea.openstack.base.client.OpenStackRequest;
-import com.woorea.openstack.base.client.OpenStackTokenProvider;
-import com.woorea.openstack.glance.Glance;
-import com.woorea.openstack.glance.model.Image;
-import com.woorea.openstack.glance.model.ImageDownload;
-import com.woorea.openstack.glance.model.Images;
-import com.woorea.openstack.keystone.utils.KeystoneTokenProvider;
-
import org.ovirt.engine.core.bll.provider.ProviderProxy;
import org.ovirt.engine.core.bll.provider.ProviderProxyFactory;
import org.ovirt.engine.core.common.businessentities.DiskImage;
@@ -35,6 +26,15 @@
import org.ovirt.engine.core.common.errors.VdcBllErrors;
import org.ovirt.engine.core.compat.Guid;
import org.ovirt.engine.core.dal.dbbroker.DbFacade;
+
+import com.woorea.openstack.base.client.HttpMethod;
+import com.woorea.openstack.base.client.OpenStackRequest;
+import com.woorea.openstack.base.client.OpenStackTokenProvider;
+import com.woorea.openstack.glance.Glance;
+import com.woorea.openstack.glance.model.Image;
+import com.woorea.openstack.glance.model.ImageDownload;
+import com.woorea.openstack.glance.model.Images;
+import com.woorea.openstack.keystone.utils.KeystoneTokenProvider;
public class OpenStackImageProviderProxy implements ProviderProxy {
@@ -116,6 +116,7 @@
domainStaticEntry.setStorageFormat(StorageFormatType.V1);
domainStaticEntry.setStorageType(StorageType.GLANCE);
domainStaticEntry.setStorageDomainType(StorageDomainType.Image);
+ domainStaticEntry.setWipeAfterDelete(false);
getDbFacade().getStorageDomainStaticDao().save(domainStaticEntry);
// Storage domain dynamic
StorageDomainDynamic domainDynamicEntry = new StorageDomainDynamic();
--
To view, visit http://gerrit.ovirt.org/36852
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I89de7410aa25073dfa60c40b99589b164cdfe6ea
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masa...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
