From 20bd98af2476ed2ea5da5a4b59240dd16ba4d93d Mon Sep 17 00:00:00 2001
From: prachi <prachi@cloud.com>
Date: Thu, 2 Feb 2012 12:17:29 -0800
Subject: [PATCH] Bug 13418 Volumes resource limit is not checked during VM
 creation

Reviewed-By: Sam

- Added Missing resource limit check for volumes during VM creation
---
 server/src/com/cloud/vm/UserVmManagerImpl.java | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/server/src/com/cloud/vm/UserVmManagerImpl.java b/server/src/com/cloud/vm/UserVmManagerImpl.java
index 1cd111e34ff..63fcf6d4523 100755
--- a/server/src/com/cloud/vm/UserVmManagerImpl.java
+++ b/server/src/com/cloud/vm/UserVmManagerImpl.java
@@ -2428,6 +2428,14 @@ public class UserVmManagerImpl implements UserVmManager, UserVmService, Manager
             rae.setResourceType("vm");
             throw rae;
         }
+        // check if the volume can be created for the user
+        // Check that the resource limit for volumes won't be exceeded
+        if (_accountMgr.resourceLimitExceeded(owner, ResourceType.volume)) {
+            UserContext.current().setEventDetails("Maximum number of volumes for account: " + owner.getAccountName() + " has been exceeded.");
+            ResourceAllocationException rae = new ResourceAllocationException("Maximum number of volumes for account: " + owner.getAccountName() + " has been exceeded.");
+            rae.setResourceType("volume");
+            throw rae;
+        }
 
         //verify security group ids
         if (securityGroupIdList != null) {