Project

General

Profile

Actions

Bug #2907

closed

Add log dir default value and update loggign setup in the README

Added by Martin Bacovsky over 10 years ago. Updated almost 6 years ago.

Status:
Resolved
Priority:
Normal
Category:
-
Target version:
Difficulty:
Triaged:
Team Backlog:
Fixed in Releases:
Found in Releases:
In Kanboard:
Actions #1

Updated by Martin Bacovsky over 10 years ago

  • Subject changed from Update loggign setup in the README to Add log dir default value and update loggign setup in the README

when there is no log_dir set user is getting the following error:

$hammer
/usr/share/ruby/fileutils.rb:1544:in `path': no implicit conversion of nil into String (TypeError)
from /usr/share/ruby/fileutils.rb:1544:in `block in fu_list'
from /usr/share/ruby/fileutils.rb:1544:in `map'
from /usr/share/ruby/fileutils.rb:1544:in `fu_list'
from /usr/share/ruby/fileutils.rb:199:in `mkdir_p'
from /usr/local/share/gems/gems/hammer_cli-0.0.1/lib/hammer_cli/logger.rb:24:in `<module:Logger>'
from /usr/local/share/gems/gems/hammer_cli-0.0.1/lib/hammer_cli/logger.rb:5:in `<module:HammerCLI>'
from /usr/local/share/gems/gems/hammer_cli-0.0.1/lib/hammer_cli/logger.rb:4:in `<top (required)>'
from /usr/share/rubygems/rubygems/core_ext/kernel_require.rb:51:in `require'

set default to /var/log/foreman/ or ~/.foreman/log

Actions #2

Updated by Martin Bacovsky over 10 years ago

  • Status changed from New to Assigned
  • Assignee set to Martin Bacovsky
  • Target version set to 1.3.0
Actions #3

Updated by Martin Bacovsky over 10 years ago

  • Status changed from Assigned to Resolved

PR #9 https://github.com/theforeman/hammer-cli/pull/9 added
- default log_dir (/var/log/foreamn)
- support for ~ in :log_dir
- suggested to use config template in install instructions

Actions #4

Updated by Dominic Cleal over 10 years ago

  • Project changed from Foreman to Hammer CLI
  • Category deleted (44)
Actions

Also available in: Atom PDF