]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commit
mon: should not take non-tell commands as tell ones
authorKefu Chai <kchai@redhat.com>
Tue, 7 Jan 2020 07:36:52 +0000 (15:36 +0800)
committerKefu Chai <kchai@redhat.com>
Tue, 7 Jan 2020 07:37:08 +0000 (15:37 +0800)
commit2a7b6f161f3ba0b5591fc2bd54e306327fd2be86
treea0e4523216f38dcb084e165a36afae0d1700a408
parent53066d09ab305f3b9ee3567509f8195bd00847f1
mon: should not take non-tell commands as tell ones

this change addresses a regression introduced by a2c34794dc. in which, a
new flag, 'FLAG_TELL' was added. and it's used to check if a command is
"TELL" command. if it is, it's added to the tell/asok command registry
and monitor will handle the commands in this registry using asok hooks.

but there are some commands whose flag is "HIDDEN". and after
a2c34794dc, is_tell() takes HIDDEN commands as TELL command. that's why
`ceph_test_admin_socket_output --all` fails. because, "mds freeze" is
now wrongly considered as a TELL command. but monitor is not able to
handle is using the asok hooks.

after this change, is_tell() will not mistake "mds freeze" as a TELL
command anymore.

Signed-off-by: Kefu Chai <kchai@redhat.com>
src/mon/MonCommand.h