Comment 4 for bug 1460801

Revision history for this message
Masaki Kanno (kanno-masaki) wrote :

Am I right?

--- ./a/solidfire.py 2017-10-19 17:07:47.290721165 +0900
+++ ./b/solidfire.py 2017-10-19 17:09:26.114723811 +0900
@@ -1000,8 +1000,8 @@ class SolidFireDriver(san.SanISCSIDriver
         # if it exists and return whichever one has count
         # available.
         for acc in accounts:
- if self._get_volumes_for_account(
- acc['accountID']) > self.max_volumes_per_account:
+ vols = self._get_volumes_for_account(acc['accountID'])
+ if len(vols) < self.max_volumes_per_account:
                 return acc
         if len(accounts) == 1:
             sfaccount = self._create_sfaccount(accounts[0]['name'] + '_')