From 4aa50962cb54b903c807b6a000cb41c28d4b1806 Mon Sep 17 00:00:00 2001 From: sgalea87 <43749726+sgalea87@users.noreply.github.com> Date: Thu, 27 May 2021 08:01:28 +0200 Subject: [PATCH] influxdb_user: Fix bug introduced by PR 2499 (#2614) * Update influxdb_user.py Fixed function name * Create 2614-influxdb_user-fix-issue-introduced-in-PR#2499 Added changelog * Rename 2614-influxdb_user-fix-issue-introduced-in-PR#2499 to 2614-influxdb_user-fix-issue-introduced-in-PR#2499.yml Fixed extension * Update changelogs/fragments/2614-influxdb_user-fix-issue-introduced-in-PR#2499.yml Co-authored-by: Amin Vakil Co-authored-by: Amin Vakil --- .../2614-influxdb_user-fix-issue-introduced-in-PR#2499.yml | 2 ++ plugins/modules/database/influxdb/influxdb_user.py | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) create mode 100644 changelogs/fragments/2614-influxdb_user-fix-issue-introduced-in-PR#2499.yml diff --git a/changelogs/fragments/2614-influxdb_user-fix-issue-introduced-in-PR#2499.yml b/changelogs/fragments/2614-influxdb_user-fix-issue-introduced-in-PR#2499.yml new file mode 100644 index 0000000000..dfae3f2bdf --- /dev/null +++ b/changelogs/fragments/2614-influxdb_user-fix-issue-introduced-in-PR#2499.yml @@ -0,0 +1,2 @@ +bugfixes: + - influxdb_user - fix bug which removed current privileges instead of appending them to existing ones (https://github.com/ansible-collections/community.general/issues/2609, https://github.com/ansible-collections/community.general/pull/2614). diff --git a/plugins/modules/database/influxdb/influxdb_user.py b/plugins/modules/database/influxdb/influxdb_user.py index d9e6b58051..cb35ea7ce6 100644 --- a/plugins/modules/database/influxdb/influxdb_user.py +++ b/plugins/modules/database/influxdb/influxdb_user.py @@ -174,7 +174,7 @@ def set_user_grants(module, client, user_name, grants): if v['privilege'] != 'NO PRIVILEGES': if v['privilege'] == 'ALL PRIVILEGES': v['privilege'] = 'ALL' - parsed_grants.add(v) + parsed_grants.append(v) # check if the current grants are included in the desired ones for current_grant in parsed_grants: