]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/dashboard: dashboard help command showing wrong syntax for login-banner 46765/head
authorSarthak0702 <sarthak.dev.0702@gmail.com>
Tue, 21 Jun 2022 09:52:12 +0000 (15:22 +0530)
committerSarthak0702 <sarthak.dev.0702@gmail.com>
Tue, 21 Jun 2022 18:32:51 +0000 (00:02 +0530)
Signed-off-by: Sarthak0702 <sarthak.dev.0702@gmail.com>
src/pybind/mgr/dashboard/controllers/home.py
src/pybind/mgr/dashboard/module.py
src/pybind/mgr/dashboard/services/custom_banner.py [deleted file]

index b79b53ca86a3132e0d8092316d6a0d7f66515840..f911cf3888fa07b4263600dbce2815a639b05ee2 100644 (file)
@@ -14,7 +14,6 @@ import cherrypy
 from cherrypy.lib.static import serve_file
 
 from .. import mgr
-from ..services.custom_banner import get_login_banner_mgr
 from . import BaseController, Endpoint, Proxy, Router, UIRouter
 
 logger = logging.getLogger("controllers.home")
@@ -146,4 +145,4 @@ class LangsController(BaseController, LanguageMixin):
 class LoginController(BaseController):
     @Endpoint('GET', 'custom_banner')
     def __call__(self):
-        return get_login_banner_mgr()
+        return mgr.get_store('custom_login_banner')
index cefceb048622bcc24f2f3f21cd06f40c294eb872..5b2f90ab204d615efd7324d9590d14daeca8b69a 100644 (file)
@@ -13,9 +13,6 @@ import threading
 import time
 from typing import TYPE_CHECKING, Optional
 
-from .services.custom_banner import get_login_banner_mgr, \
-    set_login_banner_mgr, unset_login_banner_mgr
-
 if TYPE_CHECKING:
     if sys.version_info >= (3, 8):
         from typing import Literal
@@ -421,19 +418,25 @@ class Module(MgrModule, CherryPyConfig):
         return 0, 'RGW credentials configured', ''
 
     @CLIWriteCommand("dashboard set-login-banner")
-    def set_login_banner(self, mgr_id: Optional[str] = None, inbuf: Optional[str] = None):
+    def set_login_banner(self, inbuf: str):
+        '''
+        Set the custom login banner read from -i <file>
+        '''
         item_label = 'login banner file'
         if inbuf is None:
             return HandleCommandResult(
                 -errno.EINVAL,
                 stderr=f'Please specify the {item_label} with "-i" option'
             )
-        set_login_banner_mgr(inbuf, mgr_id)
+        mgr.set_store('custom_login_banner', inbuf)
         return HandleCommandResult(stdout=f'{item_label} added')
 
     @CLIReadCommand("dashboard get-login-banner")
     def get_login_banner(self):
-        banner_text = get_login_banner_mgr()
+        '''
+        Get the custom login banner text
+        '''
+        banner_text = mgr.get_store('custom_login_banner')
         if banner_text is None:
             return HandleCommandResult(stdout='No login banner set')
         else:
@@ -441,7 +444,10 @@ class Module(MgrModule, CherryPyConfig):
 
     @CLIWriteCommand("dashboard unset-login-banner")
     def unset_login_banner(self):
-        unset_login_banner_mgr()
+        '''
+        Unset the custom login banner
+        '''
+        mgr.set_store('custom_login_banner', None)
         return HandleCommandResult(stdout='Login banner removed')
 
     def handle_command(self, inbuf, cmd):
diff --git a/src/pybind/mgr/dashboard/services/custom_banner.py b/src/pybind/mgr/dashboard/services/custom_banner.py
deleted file mode 100644 (file)
index e28addd..0000000
+++ /dev/null
@@ -1,27 +0,0 @@
-import logging
-from typing import Optional
-
-from mgr_module import _get_localized_key
-
-from .. import mgr
-
-logger = logging.getLogger(__name__)
-
-
-def set_login_banner_mgr(inbuf: str, mgr_id: Optional[str] = None):
-    item_key = 'custom_login_banner'
-    if mgr_id is not None:
-        mgr.set_store(_get_localized_key(mgr_id, item_key), inbuf)
-    else:
-        mgr.set_store(item_key, inbuf)
-
-
-def get_login_banner_mgr():
-    banner_text = mgr.get_store('custom_login_banner')
-    logger.info('Reading custom login banner: %s', banner_text)
-    return banner_text
-
-
-def unset_login_banner_mgr():
-    mgr.set_store('custom_login_banner', None)
-    logger.info('Removing custom login banner')