Skip to content

[WIP] Add configuration option 'merge_facts_later'#5689

Closed
nbarrientos wants to merge 1 commit intopuppetlabs:masterfrom
nbarrientos:pup7302
Closed

[WIP] Add configuration option 'merge_facts_later'#5689
nbarrientos wants to merge 1 commit intopuppetlabs:masterfrom
nbarrientos:pup7302

Conversation

@nbarrientos
Copy link
Contributor

@nbarrientos nbarrientos commented Mar 4, 2017

To be documented much better and tests looked at if the idea goes forward.

If enabled, it's not the node_terminus' responsability to merge facts
but the compiler's, saving queries to PuppetDB when agents invoke /node.
@puppetcla
Copy link

Waiting for CLA signature by @nbarrientos

@nbarrientos - We require a Contributor License Agreement (CLA) for people who contribute to Puppet, but we have an easy click-through license with instructions, which is available at https://cla.puppet.com/

Note: if your contribution is trivial and you think it may be exempt from the CLA, please post a short reply to this comment with details. http://docs.puppet.com/community/trivial_patch_exemption.html

@nbarrientos
Copy link
Contributor Author

Kind of superseded by the patch referenced in https://tickets.puppetlabs.com/browse/PUP-7302.

@nbarrientos nbarrientos closed this Jun 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants