]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/ansible: Replace deprecated <get_config> calls 25964/head
authorJuan Miguel Olmo Martínez <jolmomar@redhat.com>
Tue, 15 Jan 2019 17:04:06 +0000 (18:04 +0100)
committerJuan Miguel Olmo Martínez <jolmomar@redhat.com>
Tue, 15 Jan 2019 17:04:06 +0000 (18:04 +0100)
Changes needed to replace the obsolete get_config by the new get_module_option manager module method

Signed-off-by: Juan Miguel Olmo Martínez <jolmomar@redhat.com>
src/pybind/mgr/ansible/module.py

index 0d2210324a855a8fddbab783296d262a57376850..79684588a02056b1614eeb4eaf2fcfe84ebc6d5a 100644 (file)
@@ -179,13 +179,11 @@ class Module(MgrModule, orchestrator.Orchestrator):
     """An Orchestrator that an external Ansible runner service to perform
     operations
     """
-
-    OPTIONS = [
+    MODULE_OPTIONS = [
         {'name': 'server_url'},
         {'name': 'username'},
         {'name': 'password'},
         {'name': 'verify_server'} # Check server identity (Boolean/path to CA bundle)
-
     ]
 
     def __init__(self, *args, **kwargs):
@@ -234,10 +232,10 @@ class Module(MgrModule, orchestrator.Orchestrator):
 
         # Ansible runner service client
         try:
-            self.ar_client = Client(server_url = self.get_config('server_url', ''),
-                                    user = self.get_config('username', ''),
-                                    password = self.get_config('password', ''),
-                                    verify_server = self.get_config('verify_server', True),
+            self.ar_client = Client(server_url = self.get_module_option('server_url', ''),
+                                    user = self.get_module_option('username', ''),
+                                    password = self.get_module_option('password', ''),
+                                    verify_server = self.get_module_option('verify_server', True),
                                     logger = self.log)
         except Exception:
             self.log.exception("Ansible Runner Service not available. "
@@ -292,25 +290,25 @@ class Module(MgrModule, orchestrator.Orchestrator):
 
     def verify_config(self):
 
-        if not self.get_config('server_url', ''):
+        if not self.get_module_option('server_url', ''):
             self.log.error(
                 "No Ansible Runner Service base URL <server_name>:<port>"
                 "Try 'ceph config set mgr mgr/%s/server_url <server name/ip>:<port>'",
                 self.module_name)
 
-        if not self.get_config('username', ''):
+        if not self.get_module_option('username', ''):
             self.log.error(
                 "No Ansible Runner Service user. "
                 "Try 'ceph config set mgr mgr/%s/username <string value>'",
                 self.module_name)
 
-        if not self.get_config('password', ''):
+        if not self.get_module_option('password', ''):
             self.log.error(
                 "No Ansible Runner Service User password. "
                 "Try 'ceph config set mgr mgr/%s/password <string value>'",
                 self.module_name)
 
-        if not self.get_config('verify_server', ''):
+        if not self.get_module_option('verify_server', ''):
             self.log.error(
                 "TLS server identity verification is enabled by default."
                 "Use 'ceph config set mgr mgr/{0}/verify_server False' to disable it."