Project

General

Profile

Actions
Copy link

Bug #4632

closed

ndmp debug log belongs to /var/log

Added by Jan Kryl over 8 years ago. Updated about 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Jan Kryl
Category:
cmd - userland programs
Start date:
2014-02-26
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
NDMP
Gerrit CR:

Description

Following issues will be addressed on behalf of this ticket:

  • move default location of ndmp debug log from /var/ndmp to /var/log/ndmp
  • fix inaccurate information in NDMP man pages
  • introduce new -f option for ndmpd to run in foreground (for easier debugging)
  • fix double free when NDMP SMF property is read but isn't defined
  • modify ndmpd's startup code to print error message to stderr until it is daemonized
  • strictly use syslog for all messages after ndmpd's has been daemonized
  • set umask to safe value (0 -> 077)
  • avoid loading plugin if plugin path is empty string (eliminates spurious error message)
  • set plugin path to empty string by default (currently there isn't any plugin)
  • add existing debug-mode command line option among SMF properties
Actions
Copy link
 #1

Updated by Jan Kryl about 8 years ago

  • Status changed from New to In Progress
  • % Done changed from 0 to 100
Actions
Copy link
 #2

Updated by Jan Kryl about 8 years ago

  • Status changed from In Progress to Pending RTI
Actions
Copy link
 #3

Updated by Jan Kryl about 8 years ago

Issues with plugin path and double free were moved to separate tickets as they are fairly independent on other changes done on behalf of this ticket (#4645 and #4644).

Actions
Copy link
 #4

Updated by Robert Mustacchi about 8 years ago

  • Status changed from Pending RTI to Resolved
Actions
Copy link
 #5

Updated by Electric Monk about 8 years ago

git commit faac71c002f7c7a98741f991b25937b24f309df0

Author: Jan Kryl <jan.kryl@nexenta.com>

4632 ndmp debug log belongs to /var/log
4645 double free in ndmpd when undefined SMF property is read
4644 NDMP plugin-path configurable should be empty by default
Reviewed by: Albert Lee <trisk@nexenta.com>
Reviewed by: Marcel Telka <marcel@telka.sk>
Reviewed by: Dan McDonald <danmcd@omniti.com>
Approved by: Robert Mustacchi <rm@joyent.com>

Actions
Copy link

Also available in: Atom PDF