mgr/dashboard: fix issues from cherry-pick conflict resolution

src/test/mgr/mgr-dashboard-smoke.sh
src/vstart.sh
- Add 'tell mgr' in ac-user-create command.

src/pybind/mgr/dashboard/tests/test_access_control.py
qa/tasks/mgr/dashboard/test_auth.py
- Remove 'force_password' argument.

qa/tasks/mgr/dashboard/test_rgw.py
- Fix test_status test.

cherry-pick commit: b82478e1061bfc1475693b0910a7f14276b4d6ab

Fixes: https://tracker.ceph.com/issues/48355
Signed-off-by: Alfonso Martínez <almartin@redhat.com>

diff --git a/qa/tasks/mgr/dashboard/test_auth.py b/qa/tasks/mgr/dashboard/test_auth.py
index 6603b85928ba243938acd9f825acee57e797c81b..87bdd8771bd977a3d64186268bfa4fd603b54034 100644 (file)
--- a/qa/tasks/mgr/dashboard/test_auth.py
+++ b/qa/tasks/mgr/dashboard/test_auth.py
@@ -33,7 +33,7 @@ class AuthTest(DashboardTestCase):
 
     def test_login_without_password(self):
         with self.assertRaises(CommandFailedError):
-            self.create_user('admin2', '', ['administrator'], force_password=True)
+            self.create_user('admin2', '', ['administrator'])
 
     def test_a_set_login_credentials(self):
         self.create_user('admin2', 'admin2', ['administrator'])

diff --git a/qa/tasks/mgr/dashboard/test_rgw.py b/qa/tasks/mgr/dashboard/test_rgw.py
index 64c185291d3dc366027f6b375e3eb68dbed8ddef..9e781142069bdf03c2dcbb049cce87765b232e3f 100644 (file)
--- a/qa/tasks/mgr/dashboard/test_rgw.py
+++ b/qa/tasks/mgr/dashboard/test_rgw.py
@@ -332,8 +332,8 @@ class RgwDaemonTest(DashboardTestCase):
             '--system', '--access-key=admin', '--secret=admin'
         ])
         self._ceph_cmd(['dashboard', 'set-rgw-api-user-id', 'admin'])
-        self._ceph_cmd(['dashboard', 'set-rgw-api-secret-key', 'admin'])
-        self._ceph_cmd(['dashboard', 'set-rgw-api-access-key', 'admin'])
+        self._ceph_cmd_with_secret(['dashboard', 'set-rgw-api-secret-key'], 'admin')
+        self._ceph_cmd_with_secret(['dashboard', 'set-rgw-api-access-key'], 'admin')
 
         data = self._get('/api/rgw/status')
         self.assertStatus(200)

diff --git a/src/pybind/mgr/dashboard/tests/test_access_control.py b/src/pybind/mgr/dashboard/tests/test_access_control.py
index 268a4f1861e9b6c8db5d8d376f652d86fc30dde2..1a479aa67fdebfddd63f8ef931510562a8d15496 100644 (file)
--- a/src/pybind/mgr/dashboard/tests/test_access_control.py
+++ b/src/pybind/mgr/dashboard/tests/test_access_control.py
@@ -566,8 +566,7 @@ class AccessControlTest(unittest.TestCase, CLICommandTestMixin):
 
     def test_set_user_password_empty(self):
         with self.assertRaises(CmdException) as ctx:
-            self.exec_cmd('ac-user-set-password', username='admin', inbuf='\n',
-                          force_password=True)
+            self.exec_cmd('ac-user-set-password', username='admin', inbuf='\n')
 
         self.assertEqual(ctx.exception.retcode, -errno.EINVAL)
         self.assertEqual(str(ctx.exception), ERROR_MSG_EMPTY_INPUT_FILE)

diff --git a/src/test/mgr/mgr-dashboard-smoke.sh b/src/test/mgr/mgr-dashboard-smoke.sh
index 99dfb26b5e3a8a7f57f3b2926b2c68d171fa6da5..c3f2a822efc7177cd2aac91b3a4c4dd031933268 100755 (executable)
--- a/src/test/mgr/mgr-dashboard-smoke.sh
+++ b/src/test/mgr/mgr-dashboard-smoke.sh
@@ -54,7 +54,7 @@ function TEST_dashboard() {
 
     DASHBOARD_ADMIN_SECRET_FILE="/tmp/dashboard-admin-secret.txt"
     printf 'admin' > "${DASHBOARD_ADMIN_SECRET_FILE}"
-    ceph_adm dashboard ac-user-create admin -i "${DASHBOARD_ADMIN_SECRET_FILE}"
+    ceph_adm tell mgr dashboard ac-user-create admin -i "${DASHBOARD_ADMIN_SECRET_FILE}"
 
     tries=0
     while [[ $tries < 30 ]] ; do

diff --git a/src/vstart.sh b/src/vstart.sh
index f7dabd74c0b462f2ebcfebb70121e72d0ce60cd6..8174c3e5a848c355e2ae720e95158172554487a9 100755 (executable)
--- a/src/vstart.sh
+++ b/src/vstart.sh
@@ -830,7 +830,7 @@ EOF
         if $with_mgr_dashboard; then
             DASHBOARD_ADMIN_SECRET_FILE="${CEPH_CONF_PATH}/dashboard-admin-secret.txt"
             printf 'admin' > "${DASHBOARD_ADMIN_SECRET_FILE}"
-            ceph_adm dashboard ac-user-create admin -i "${DASHBOARD_ADMIN_SECRET_FILE}" \
+            ceph_adm tell mgr dashboard ac-user-create admin -i "${DASHBOARD_ADMIN_SECRET_FILE}" \
                 administrator
             if [ "$ssl" != "0" ]; then
                 if ! ceph_adm tell mgr dashboard create-self-signed-cert;  then