Merge pull request #611 from cdosborn/fix-quota-volume-update

Fix volume quota update
cyverse · Apr 18, 2018 · 0ebcfc5 · 0ebcfc5
2 parents 86cc6d7 + 7cd6bca
commit 0ebcfc5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -25,6 +25,9 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 -->
 
 ## [Unreleased](https://github.com/cyverse/atmosphere/compare/v32-1...HEAD)
+### Fixed
+ - Quota updates concerning volumes would silently fail ([#611](https://github.com/cyverse/atmosphere/pull/611))
+
 ## [v32-1](https://github.com/cyverse/atmosphere/compare/v32-0...v32-1) - 2018-04-17
 ### Added
  - Support multiple hostnames for Atmosphere(1) server ([#602](https://github.com/cyverse/atmosphere/pull/602))

diff --git a/service/quota.py b/service/quota.py
@@ -209,10 +209,10 @@ def _set_volume_quota(user_quota, identity):
     username = identity.created_by.username
     logger.info("Updating quota for %s to %s" % (username, volume_values))
     driver = get_cached_driver(identity=identity)
-    username = driver._connection._get_username()
     ad = get_account_driver(identity.provider)
     admin_driver = ad.admin_driver
-    result = admin_driver._connection._cinder_update_quota(username, volume_values)
+    tenant = ad.get_project(username)
+    result = admin_driver._connection._cinder_update_quota(tenant.id, volume_values)
     logger.info("Updated quota for %s to %s" % (username, result))
     return result