Bug #3870

add a directory for modules to add config in

Added by Greg Sutcliffe almost 4 years ago. Updated over 3 years ago.

Status:Closed
Priority:Normal
Assigned To:Martin Bacovsky
Category:Hammer core
Target version:Foreman - Sprint 21
Difficulty: Bugzilla link:
Found in release: Pull request:
Story points-
Velocity based estimate-

Description

Instead of:

:modules:
- hammer_cli_foreman
etc

it would be nice to have something like $confdir/hammer.modules.d/ where a modules could add config of their own (e.g. enable foreman module by creating $confdir/hammer.modules.d/hammer_cli_foreman). This makes it much easier to enable modules from separate OS packages (as they can just drop a file in place, instead of sed'ing the main hammer config file)


Related issues

Related to Hammer CLI - Bug #5052: Documentation "Writing you own plugin" needs update Closed 04/03/2014

Associated revisions

Revision b73e2544
Added by Martin Bacovsky over 3 years ago

Fixes #3870 - add a directory for module's configs

Revision f73b380e
Added by mbacovsky over 3 years ago

Merge pull request #92 from mbacovsky/3870_modules_config

Fixes #3870 - add a directory for module's configs

History

#1 Updated by Dmitri Dolguikh over 3 years ago

  • Target version set to Sprint 21

#2 Updated by Martin Bacovsky over 3 years ago

  • Assigned To set to Martin Bacovsky
  • Category set to Hammer core
  • Status changed from New to Assigned

#3 Updated by Martin Bacovsky over 3 years ago

  • Status changed from Assigned to Ready For Testing

#4 Updated by Martin Bacovsky over 3 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Applied in changeset foreman:hammer-cli|commit:b73e2544052fae7e739be260a8d4fd1364a4f539.

#5 Updated by Tomáš Strachota over 3 years ago

  • Related to Bug #5052: Documentation "Writing you own plugin" needs update added

Also available in: Atom PDF