Project

General

Profile

Actions

Bug #7849

closed

ceph-conf create empty log files

Added by Dmitry Smirnov about 10 years ago. Updated about 10 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Category:
-
Target version:
-
% Done:

0%

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

Description

`ceph-conf` creates empty log files in "/var/log/ceph". This is unexpected, undesireable (and undocumented) behaviour.
Even worse, it makes log files for non-existent config sections.

For example the following command:

/usr/bin/ceph-conf -c /etc/ceph/ceph.conf --name osd.999 --lookup "host" 

creates the following file:
/var/log/ceph/ceph-osd.999.log

This is not what config file query tool should do.

Actions #1

Updated by Sage Weil about 10 years ago

  • Status changed from New to 12
  • Priority changed from Normal to Urgent

This should be just a matter of passing the right flags to common_init or global_init

Actions #2

Updated by Sage Weil about 10 years ago

  • Status changed from 12 to Fix Under Review
  • Source changed from other to Community (user)
Actions #3

Updated by Josh Durgin about 10 years ago

  • Status changed from Fix Under Review to Resolved
Actions #4

Updated by Sage Weil about 10 years ago

  • Status changed from Resolved to In Progress

oops, the previous fix breaks ceph.py, which does ceph-conf --show-config-value to get admin_socket.

Actions #5

Updated by Sage Weil about 10 years ago

  • Target version deleted (0.79)
Actions #6

Updated by Dmitry Smirnov about 10 years ago

Socket part was introduced later -- perhaps only socket part shall be reversed?
Log-related change seems safe...

Actions #7

Updated by Sage Weil about 10 years ago

  • Status changed from In Progress to Fix Under Review
Actions #8

Updated by Ian Colle about 10 years ago

  • Assignee set to Josh Durgin

Josh - please review the wip branch

Actions #9

Updated by Josh Durgin about 10 years ago

  • Status changed from Fix Under Review to Resolved
  • Assignee changed from Josh Durgin to Sage Weil
Actions

Also available in: Atom PDF