mirror of
https://github.com/saltstack/salt.git
synced 2025-04-17 10:10:20 +00:00
Don't use single character variables.
This commit is contained in:
parent
e712664bcf
commit
58f7fc3285
1 changed files with 14 additions and 14 deletions
|
@ -290,27 +290,27 @@ def find_credentials():
|
|||
if 'fallback_admin_username' in __pillar__.get('proxy'):
|
||||
usernames.append(__pillar__['proxy'].get('fallback_admin_username'))
|
||||
|
||||
for u in usernames:
|
||||
for p in __pillar__['proxy']['passwords']:
|
||||
for user in usernames:
|
||||
for pwd in __pillar__['proxy']['passwords']:
|
||||
r = __salt__['dracr.get_chassis_name'](host=__pillar__['proxy']['host'],
|
||||
admin_username=u,
|
||||
admin_password=p)
|
||||
admin_username=user,
|
||||
admin_password=pwd)
|
||||
# Retcode will be present if the chassis_name call failed
|
||||
try:
|
||||
if r.get('retcode', None) is None:
|
||||
DETAILS['admin_username'] = u
|
||||
DETAILS['admin_password'] = p
|
||||
__opts__['proxy']['admin_username'] = u
|
||||
__opts__['proxy']['admin_password'] = p
|
||||
return (u, p)
|
||||
DETAILS['admin_username'] = user
|
||||
DETAILS['admin_password'] = pwd
|
||||
__opts__['proxy']['admin_username'] = user
|
||||
__opts__['proxy']['admin_password'] = pwd
|
||||
return (user, pwd)
|
||||
except AttributeError:
|
||||
# Then the above was a string, and we can return the username
|
||||
# and password
|
||||
DETAILS['admin_username'] = u
|
||||
DETAILS['admin_password'] = p
|
||||
__opts__['proxy']['admin_username'] = u
|
||||
__opts__['proxy']['admin_password'] = p
|
||||
return (u, p)
|
||||
DETAILS['admin_username'] = user
|
||||
DETAILS['admin_password'] = pwd
|
||||
__opts__['proxy']['admin_username'] = user
|
||||
__opts__['proxy']['admin_password'] = pwd
|
||||
return (user, pwd)
|
||||
|
||||
log.debug('proxy fx2.find_credentials found no valid credentials, using Dell default')
|
||||
return ('root', 'calvin')
|
||||
|
|
Loading…
Add table
Reference in a new issue