diff --git a/lib/ansible/modules/storage/netapp/na_elementsw_admin_users.py b/lib/ansible/modules/storage/netapp/na_elementsw_admin_users.py index c1744546334..a0e05c83a2f 100644 --- a/lib/ansible/modules/storage/netapp/na_elementsw_admin_users.py +++ b/lib/ansible/modules/storage/netapp/na_elementsw_admin_users.py @@ -163,20 +163,22 @@ class NetAppElementSWAdminUser(object): 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 """ changed = False admin_user = self.get_admin_user() - if not admin_user.access == self.access and self.access is not None: - changed = True - if self.element_password: - changed = True + if self.access is not None and len(self.access) > 0: + for access in self.access: + if access not in admin_user.access: + changed = True if changed: self.sfe.modify_cluster_admin(cluster_admin_id=admin_user.cluster_admin_id, access=self.access, password=self.element_password, attributes=self.attributes) + return changed def add_admin_user(self):