Project

General

Profile

Actions

Bug #9913

closed

mon: audit log entires for forwarded requests lack info

Added by Sage Weil over 9 years ago. Updated about 9 years ago.

Status:
Resolved
Priority:
High
Category:
Monitor
Target version:
-
% Done:

0%

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

Description

2014-10-27 14:29:56.265787 mon.0 [INF] from='client.4114 :/0' entity='forwarded-request' cmd=[{"prefix": "auth add", "entity": "osd.2", "caps": ["osd", "allow *", "mon", "allow profile osd"]}]: dispatch

vs

2014-10-27 14:29:59.210533 mon.0 [INF] from='client.? 10.214.131.225:0/1017125' entity='client.admin' cmd=[{"prefix": "fs new", "data": "cephfs_data", "fs_name": "cephfs", "metadata": "cephfs_metadata"}]: dispatch

should just be a matter to calling the right method on the session?

Actions #1

Updated by Joao Eduardo Luis over 9 years ago

session is with the monitor that forwarded the request. there's no auth handler for the session as it is a monitor. We can get the entity_name_t for the message, but that will not give us the auth entitiy it was authenticated with. Nevertheless, it should make the 'from=' field a bit more verbose than it currently is, although I expected that info to be accurate as it is from the start.

Actions #2

Updated by Joao Eduardo Luis over 9 years ago

  • Category set to Monitor
  • Status changed from New to Fix Under Review
Actions #3

Updated by Sage Weil over 9 years ago

  • Status changed from Fix Under Review to Pending Backport
  • Source changed from other to Development
Actions #4

Updated by Loïc Dachary about 9 years ago

  • Status changed from Pending Backport to Resolved
  • Backport deleted (giant)

we only backport bug fixes to stable but not LTS releases

Actions

Also available in: Atom PDF