Project

General

Profile

Refactor #7438

Move facter dependency loading into the facts plugin descriptor

Added by Dmitri Dolguikh almost 5 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Category:
Plugins
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

- this cleans up logging -- we no longer include Proxy::Log into the Object
- it's more appropriate to include module dependencies after the module descriptor has been parsed and configuration loaded: dependencies shouldn't be loaded if the module is disabled, for example.


Related issues

Related to Smart Proxy - Bug #7062: Wrong requires in smart-proxy's 'facts' moduleClosed2014-08-13

Associated revisions

Revision f655a4ed (diff)
Added by Dmitri Dolguikh almost 5 years ago

Fixes #7438: moved facter loading into the facter plugin descriptor

History

#1 Updated by Dmitri Dolguikh almost 5 years ago

  • Project changed from Foreman to Smart Proxy

#2 Updated by Dmitri Dolguikh almost 5 years ago

  • Related to Bug #7062: Wrong requires in smart-proxy's 'facts' module added

#3 Updated by Dominic Cleal almost 5 years ago

  • Category set to Plugins
  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/smart-proxy/pull/205 added
  • Pull request deleted ()

#4 Updated by Anonymous almost 5 years ago

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

#5 Updated by Dominic Cleal almost 5 years ago

  • Legacy Backlogs Release (now unused) set to 21

Also available in: Atom PDF