Feature #3921

Facts should have the same terminus configurability as catalogs

Added by Luke Kanies almost 2 years ago. Updated almost 2 years ago.

Status:Closed Start date:06/01/2010
Priority:Normal Due date:
Assignee:Markus Roberts % Done:

0%
Category:compiler
Target version:2.6.0
Affected Puppet version:0.25.4 Branch:feature/master/3921-facts_terminus-setting
Keywords:facts routes
Votes: 0

Description

That is, we should have a ‘facts_terminus’ and ‘facts_cache’ setting.

This is getting ugly, and should really be done with a routes file, but that’s more complicated than I would have thought, so doing this is a reasonable short-term solution.

This will make it quite easy to get the master to upload its facts to any given external node tool.

It’s really easy, so this should easily fit into rowlf.

Related issues

related to Puppet - Bug #4413: Cannot save facts to the code store Closed 07/30/2010
related to Puppet - Bug #10635: puppet master does not work with 0.25.6 Rejected 11/08/2011

History

#1
Updated by Jesse Wolfe almost 2 years ago

  • Assignee set to Rein Henrichs

#2
Updated by Rein Henrichs almost 2 years ago

Per discussion with Luke, we won’t be implementing a facts_cache setting (yet).

#3
Updated by Rein Henrichs almost 2 years ago

  • Status changed from Accepted to In Topic Branch Pending Review
  • Branch set to feature/master/3921-facts_terminus-setting

Ready for testing in my feature/master/3921-facts_terminus-setting branch.

Git remote is git://github.com/reinh

#4
Updated by Markus Roberts almost 2 years ago

  • Status changed from In Topic Branch Pending Review to Closed

Added to master in:

commit:9d0a38ee1baa295fb8d2b69f13cbfc91b5de5800 [#3921] Add facts_terminus setting to Puppet settings commit:2874729efa22c56dde61654e7429bdbb4e41f9ef [#3921] Remove unnecessary require ‘puppet/resource’

#5
Updated by Rein Henrichs almost 2 years ago

  • Status changed from Closed to In Topic Branch Pending Review
  • Assignee changed from Rein Henrichs to Markus Roberts

The current implementation has a typo: “fact_terminus” should be “facts_terminus”. I have added a commit to the tip of my branch to fix this. Please update master.

#6
Updated by Nick Lewis almost 2 years ago

  • Status changed from In Topic Branch Pending Review to Closed

Pushed in commit:35636e9984f1f9b2c6fae3d1410b4b08b6311693 in master

Also available in: Atom PDF