]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commit
qa/tasks/mgr: set mgr module option with --force
authorKefu Chai <kchai@redhat.com>
Fri, 10 Jan 2020 05:28:10 +0000 (13:28 +0800)
committerKefu Chai <kchai@redhat.com>
Fri, 10 Jan 2020 05:28:34 +0000 (13:28 +0800)
commit7210d7f951c479436147a63aef43a54e27eda70d
tree9e782f363100c0ae9f8b9168571661ccfa234d10
parentcf2d411251c5e92e1063bb6d9cc6aa9d501f33e3
qa/tasks/mgr: set mgr module option with --force

if mgr is not active, monitor will refuse to set any option consumed by
mgr modules.

the reason the tests pass somtimes is that, we have a racing here:

1. stop all mgr daemons
2. MgrMonitor gets updated and updates its mgr_module_options
accordingly.
3. in TestDashboard.setUp(), we reset the port number for dashboard
using "ceph config set mgr mgr/dashboard/y/ssl_server_port 7789"
4. restart all mgr daemons

but the 2nd step and 3rd step could race with each other, if the 2nd
step happens after 3rd step, the test passes. otherwise it fails.

in this change, "--force" is passed to the "ceph config set" command,
so ConfigMonitor can bypass the sanity test for the option, and just
set this option.

Signed-off-by: Kefu Chai <kchai@redhat.com>
qa/tasks/mgr/mgr_test_case.py