Project

General

Profile

Bug #58192

mgr/rbd_support: flooding audit log with "config rm" entries

Added by Mykola Golub about 2 months ago. Updated about 2 months ago.

Status:
Duplicate
Priority:
Normal
Assignee:
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
pacific,quincy
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

% ceph log last 100 info audit 
2022-12-06T19:24:21.677117+0000 mon.a (mon.0) 1506 : audit [INF] from='mgr.4133 10.100.2.80:0/14606' entity='mgr.x' cmd=[{"prefix":"config rm","who":"mgr","name":"mgr/rbd_support/x/trash_purge_schedule"}]: dispatch
2022-12-06T19:24:21.677869+0000 mon.a (mon.0) 1507 : audit [INF] from='mgr.4133 10.100.2.80:0/14606' entity='mgr.x' cmd=[{"prefix":"config rm","who":"mgr","name":"mgr/rbd_support/x/mirror_snapshot_schedule"}]: dispatch
2022-12-06T19:25:21.685371+0000 mon.a (mon.0) 1508 : audit [INF] from='mgr.4133 10.100.2.80:0/14606' entity='mgr.x' cmd=[{"prefix":"config rm","who":"mgr","name":"mgr/rbd_support/x/mirror_snapshot_schedule"}]: dispatch
2022-12-06T19:25:21.686118+0000 mon.a (mon.0) 1509 : audit [INF] from='mgr.4133 10.100.2.80:0/14606' entity='mgr.x' cmd=[{"prefix":"config rm","who":"mgr","name":"mgr/rbd_support/x/trash_purge_schedule"}]: dispatch
2022-12-06T19:26:21.693005+0000 mon.a (mon.0) 1510 : audit [INF] from='mgr.4133 10.100.2.80:0/14606' entity='mgr.x' cmd=[{"prefix":"config rm","who":"mgr","name":"mgr/rbd_support/x/mirror_snapshot_schedule"}]: dispatch
2022-12-06T19:26:21.693745+0000 mon.a (mon.0) 1511 : audit [INF] from='mgr.4133 10.100.2.80:0/14606' entity='mgr.x' cmd=[{"prefix":"config rm","who":"mgr","name":"mgr/rbd_support/x/trash_purge_schedule"}]: dispatch
...

In 8e9e2a73f5b8abd743933e59a3b986598765a681 [1] we added a backward compatibility check, which moves the global schedule config from a localized module option to the global option. It was implemented in the way that the removal of the localized module option is called on every schedule load regardless of whether or not the option exists, causing the "config rm" log flood.

It may be easily changed to do "config rm" only if it exists. Additionally we may consider removing this compatibility code at all in new versions.

[1] https://github.com/ceph/ceph/pull/37864


Related issues

Duplicates rbd - Bug #57726: [rbd_support] set_localized_module_option(..., None) is spamming the audit channel Pending Backport

History

#1 Updated by Mykola Golub about 2 months ago

  • Status changed from In Progress to Fix Under Review
  • Pull request ID set to 49291

#2 Updated by Ilya Dryomov about 2 months ago

  • Status changed from Fix Under Review to Duplicate

#3 Updated by Ilya Dryomov about 2 months ago

  • Duplicates Bug #57726: [rbd_support] set_localized_module_option(..., None) is spamming the audit channel added

Also available in: Atom PDF