Use attribute instead of passing parameter
Start using self.user_creds_id attribute instead of passing it as parameter in the fucntion _try_get_user_creds. Change-Id: I5a0bd3a1d6734cccce4c15c9b34ef3aeb89a577d
This commit is contained in:
parent
df3cd52c38
commit
136903bf08
|
@ -1343,7 +1343,7 @@ class Stack(collections.Mapping):
|
|||
|
||||
stack.delete(backup=True)
|
||||
|
||||
def _try_get_user_creds(self, user_creds_id):
|
||||
def _try_get_user_creds(self):
|
||||
# There are cases where the user_creds cannot be returned
|
||||
# due to credentials truncated when being saved to DB.
|
||||
# Ignore this error instead of blocking stack deletion.
|
||||
|
@ -1361,7 +1361,7 @@ class Stack(collections.Mapping):
|
|||
# The stack_status and reason passed in are current values, which
|
||||
# may get rewritten and returned from this method
|
||||
if self.user_creds_id:
|
||||
user_creds = self._try_get_user_creds(self.user_creds_id)
|
||||
user_creds = self._try_get_user_creds()
|
||||
# If we created a trust, delete it
|
||||
if user_creds is not None:
|
||||
trust_id = user_creds.get('trust_id')
|
||||
|
|
Loading…
Reference in New Issue