Project

General

Profile

Bug #2907

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

Added by Martin Bacovsky almost 7 years ago. Updated almost 2 years ago.

Status:
Resolved
Priority:
Normal
Category:
-
Target version:
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Team Backlog:
Fixed in Releases:
Found in Releases:
In Kanboard:

Associated revisions

Revision 74db54c2 (diff)
Added by Martin Bacovsky almost 7 years ago

fixes #2907 - Add log dir default value

Revision a729f529
Added by mbacovsky almost 7 years ago

Merge pull request #9 from mbacovsky/2907_default_log_dir

fixes #2907 - Add log dir default value

History

#1 Updated by Martin Bacovsky almost 7 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

#2 Updated by Martin Bacovsky almost 7 years ago

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

#3 Updated by Martin Bacovsky almost 7 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

#4 Updated by Dominic Cleal over 6 years ago

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

Also available in: Atom PDF