Gilad Chaplik has uploaded a new change for review.

Change subject: core: create default disk profile for attached SD
......................................................................

core: create default disk profile for attached SD

Disk profile wasn't created for 'old' storage domain that
is attached to 3.5 data center.

Change-Id: Ic59d6f38096fb5e20fee5b9b4705b4603fdf0d4f
Bug-Url: https://bugzilla.redhat.com/1146073
Signed-off-by: Gilad Chaplik <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AttachStorageDomainToPoolCommand.java
1 file changed, 9 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/28/33428/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AttachStorageDomainToPoolCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AttachStorageDomainToPoolCommand.java
index 9a57721..9803e5e 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AttachStorageDomainToPoolCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AttachStorageDomainToPoolCommand.java
@@ -12,6 +12,7 @@
 import org.ovirt.engine.core.bll.NonTransactiveCommandAttribute;
 import org.ovirt.engine.core.bll.RetrieveImageDataParameters;
 import org.ovirt.engine.core.bll.context.CommandContext;
+import org.ovirt.engine.core.bll.profiles.DiskProfileHelper;
 import org.ovirt.engine.core.common.AuditLogType;
 import org.ovirt.engine.core.common.FeatureSupported;
 import org.ovirt.engine.core.common.action.AttachStorageDomainToPoolParameters;
@@ -150,6 +151,14 @@
                                 }
                             }
                         }
+                        // Creating default disk profile for existing storage 
domain.
+                        if 
(FeatureSupported.storageQoS(getStoragePool().getcompatibility_version())
+                                && getDbFacade().getDiskProfileDao()
+                                        
.getAllForStorageDomain(getStorageDomain().getId())
+                                        .isEmpty()) {
+                            
getDiskProfileDao().save(DiskProfileHelper.createDiskProfile(getStorageDomain().getId(),
+                                    getStorageDomainName()));
+                        }
                     }
 
                     runVdsCommand(VDSCommandType.AttachStorageDomain,


-- 
To view, visit http://gerrit.ovirt.org/33428
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic59d6f38096fb5e20fee5b9b4705b4603fdf0d4f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Gilad Chaplik <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to