Gilad Chaplik has uploaded a new change for review.
Change subject: core: add disk profile to newly created storage domain
......................................................................
core: add disk profile to newly created storage domain
Change-Id: I29aaf7b0b537f0b80b91b09e025e9487bbb69c2d
Signed-off-by: Gilad Chaplik <gchap...@redhat.com>
---
A
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/profiles/DiskProfileHelper.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStorageDomainCommand.java
2 files changed, 24 insertions(+), 2 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/37/29037/1
diff --git
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/profiles/DiskProfileHelper.java
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/profiles/DiskProfileHelper.java
new file mode 100644
index 0000000..4cdeac9
--- /dev/null
+++
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/profiles/DiskProfileHelper.java
@@ -0,0 +1,15 @@
+package org.ovirt.engine.core.bll.profiles;
+
+import org.ovirt.engine.core.common.businessentities.profiles.DiskProfile;
+import org.ovirt.engine.core.compat.Guid;
+
+public class DiskProfileHelper {
+
+ public static DiskProfile createDiskProfile(Guid storageDomainId, String
storageDomainName) {
+ DiskProfile profile = new DiskProfile();
+ profile.setId(Guid.newGuid());
+ profile.setName(storageDomainName);
+ profile.setStorageDomainId(storageDomainId);
+ return profile;
+ }
+}
diff --git
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStorageDomainCommand.java
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStorageDomainCommand.java
index 4e89420..7b32796 100644
---
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStorageDomainCommand.java
+++
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStorageDomainCommand.java
@@ -72,8 +72,15 @@
TransactionSupport.executeInNewTransaction(new
TransactionMethod<Void>() {
@Override
public Void runInTransaction() {
-
DbFacade.getInstance().getStorageDomainStaticDao().save(getStorageDomain().getStorageStaticData());
-
getCompensationContext().snapshotNewEntity(getStorageDomain().getStorageStaticData());
+ StorageDomainStatic storageStaticData =
getStorageDomain().getStorageStaticData();
+
DbFacade.getInstance().getStorageDomainStaticDao().save(storageStaticData);
+ // create default disk profile for type master or data storage
domains
+ if (storageStaticData.getStorageDomainType() ==
StorageDomainType.Master
+ || storageStaticData.getStorageDomainType() ==
StorageDomainType.Data) {
+
getDiskProfileDao().save(DiskProfileHelper.createDiskProfile(storageStaticData.getId(),
+ storageStaticData.getStorageName()));
+ }
+ getCompensationContext().snapshotNewEntity(storageStaticData);
StorageDomainDynamic newStorageDynamic =
new StorageDomainDynamic(null,
getStorageDomain().getId(), null);
getReturnValue().setActionReturnValue(getStorageDomain().getId());
--
To view, visit http://gerrit.ovirt.org/29037
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I29aaf7b0b537f0b80b91b09e025e9487bbb69c2d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Gilad Chaplik <gchap...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
