From b48e2c121cdebee07326d5546f4eb890b0b468e7 Mon Sep 17 00:00:00 2001
From: Lenka Segura <lsegura@redhat.com>
Date: Feb 15 2023 12:35:03 +0000
Subject: Change the retrieved discourse groups from list to dict


---

diff --git a/roles/fas2discourseconfig/defaults/main.yml b/roles/fas2discourseconfig/defaults/main.yml
index 78d7adf..f6b8124 100644
--- a/roles/fas2discourseconfig/defaults/main.yml
+++ b/roles/fas2discourseconfig/defaults/main.yml
@@ -2,6 +2,5 @@
 # defaults file for Fas2discourseConfig
 fas2discourse_keytab_path: "/etc/fas2discourse"
 fas2discourse_principal: "fas2discourse/fas2discourse.hostname@FEDORAPROJECT.ORG"
-discourse_groups: {{ discourse_groups }}
 f2d_namespace: "fas2discourse-operator"
 f2d_secret: "fas2discourse-operator-k8s-secret"
diff --git a/roles/fas2discourseconfig/library/fas2discourse_retrieve_discourse_data.py b/roles/fas2discourseconfig/library/fas2discourse_retrieve_discourse_data.py
index 6678ebe..63a686f 100644
--- a/roles/fas2discourseconfig/library/fas2discourse_retrieve_discourse_data.py
+++ b/roles/fas2discourseconfig/library/fas2discourse_retrieve_discourse_data.py
@@ -37,16 +37,21 @@ author:
 EXAMPLES = r"""
 - name: Retrieve discourse groups.
   fas2discourse_retrieve_discourse_data:
-    discourse_groups: "{{ discourse_groups }}"
   register: fas2discourse_discourse_response
 """
 
 RETURN = r"""
 # These are examples of possible return values, and in general should use other names for return
 # values.
-
-["group1", "group2", "group3"]
-
+discourse_groups:
+    description: All the discourse groups.
+    type: str
+    returned: always
+    sample:
+{
+        "groups":[{"group": "design"},
+                  {"group": "editors"}]
+}
 
 msg:
     description: The output message that the module generates.
@@ -72,7 +77,7 @@ def get_groups(http_client):
 def run_module():
     # define available arguments/parameters a user can pass to the module
     module_args = dict(
-        discourse_groups=list(type="str", required=True),
+        discourse_groups=dict(type="str", required=True),
     )
 
     # seed the result dict in the object