The Puppet Labs Issue Tracker has Moved: https://tickets.puppetlabs.com

Feature #6644

Need test to guarantee that facts are written to fact YAML and inventory service database BEFORE node is classified

Added by Nick Fagerlund over 3 years ago. Updated almost 2 years ago.

Status:AcceptedStart date:03/08/2011
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:-
Target version:-
Affected Puppet version: Branch:
Keywords:

We've Moved!

Ticket tracking is now hosted in JIRA: https://tickets.puppetlabs.com

This ticket may be automatically exported to the PUP project on JIRA using the button below:


Description

My understanding is that we currently always write the node fact YAML and the inventory service database entry before the ENC gets called, but it’s considered a fortuitous coincidence and we don’t make any guarantees about the order.

Since some people use facts in their ENCs, and it’s an elegant solution to things that are impossible to do by hostname a lot of the time, we should guarantee that facts get written in both places before the ENC gets called. This should be backed with a spec test.

Considerations:

  • When we can’t write to YAML on disk, Nigel thinks we blow up weirdly, which is reasonable in cases where our disk is full, right?
  • What happens when we can’t write to the inventory service? Fail with a big warning in the logs?

History

#1 Updated by James Turnbull over 3 years ago

  • Status changed from Unreviewed to Needs Decision
  • Assignee set to Nigel Kersten

#2 Updated by Nigel Kersten over 3 years ago

  • Status changed from Needs Decision to Accepted
  • Assignee deleted (Nigel Kersten)

#3 Updated by Nigel Kersten over 3 years ago

  • Target version set to 2.7.x

#4 Updated by Andrew Parker almost 2 years ago

  • Target version deleted (2.7.x)

Also available in: Atom PDF