Merge pull request #19 from lisongmin/set-options-before-start

fix: some option e.g. client.ssl, server.ssl need to be enabled before start the volume
gluster · Sep 17, 2021 · 6d2bf25 · 6d2bf25
2 parents bc0982a + 9d4d616
commit 6d2bf25
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/plugins/modules/gluster_volume.py b/plugins/modules/gluster_volume.py
@@ -541,6 +541,12 @@ def main():
             changed = True
 
         if volume_name in volumes:
+            # set options
+            for option in options.keys():
+                if option not in volumes[volume_name]['options'] or volumes[volume_name]['options'][option] != options[option]:
+                    set_volume_option(volume_name, option, options[option])
+                    changed = True
+
             if volumes[volume_name]['status'].lower() != 'started' and start_on_create:
                 start_volume(volume_name)
                 changed = True
@@ -584,12 +590,6 @@ def main():
                     set_quota(volume_name, directory, quota)
                     changed = True
 
-            # set options
-            for option in options.keys():
-                if option not in volumes[volume_name]['options'] or volumes[volume_name]['options'][option] != options[option]:
-                    set_volume_option(volume_name, option, options[option])
-                    changed = True
-
         else:
             module.fail_json(msg='failed to create volume %s' % volume_name)