Bug fix for na_elementsw_admin_users (#52377)

* Revert "changes to clusteR"

This reverts commit 33ee1b71e4bc8435fb315762a871f8c4cb6c5f80.

* Revert "Revert "changes to clusteR""

This reverts commit f1104a37b4.

* Updates

* bug fix

* Bug fix

* Revert "Bug fix"

This reverts commit 5cbea70a65.
pull/53285/head
Chris Archibald 5 years ago committed by John R Barker
parent 725654f9d3
commit 11a058fa39

@ -163,20 +163,22 @@ class NetAppElementSWAdminUser(object):
def modify_admin_user(self): def modify_admin_user(self):
""" """
Modify a admin user. If a password is set the user will be modified as there is no way to compare a new password with an existing one Modify a admin user. If a password is set the user will be modified as there is no way to
compare a new password with an existing one
:return: if a user was modified or not :return: if a user was modified or not
""" """
changed = False changed = False
admin_user = self.get_admin_user() admin_user = self.get_admin_user()
if not admin_user.access == self.access and self.access is not None: if self.access is not None and len(self.access) > 0:
changed = True for access in self.access:
if self.element_password: if access not in admin_user.access:
changed = True changed = True
if changed: if changed:
self.sfe.modify_cluster_admin(cluster_admin_id=admin_user.cluster_admin_id, self.sfe.modify_cluster_admin(cluster_admin_id=admin_user.cluster_admin_id,
access=self.access, access=self.access,
password=self.element_password, password=self.element_password,
attributes=self.attributes) attributes=self.attributes)
return changed return changed
def add_admin_user(self): def add_admin_user(self):

Loading…
Cancel
Save