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

Feature #14570

Separate fact definition from fact resolution

Added by Jeff Weiss over 2 years ago. Updated over 1 year ago.

Status:AcceptedStart date:05/17/2012
Priority:NormalDue date:
Assignee:Jeff Weiss% Done:

0%

Category:interface
Target version:2.x
Keywords: Affected Facter version:
Branch:https://github.com/puppetlabs/facter/tree/integration/fact_definition

We've Moved!

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

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


Description

Saying what a fact is should be separate from the mechanics of how we resolve that fact.

The fact definition might include things like:

  • description
  • data type (string, boolean, numeric, array, hash, etc)
  • cache-able?
    • time-to-live in cache?

Related issues

Related to Facter - Feature #14623: Warn if fact has no definition Rejected 05/21/2012
Precedes Facter - Feature #4561: Structured data should be supported Closed 05/18/2012 05/18/2012
Precedes Facter - Feature #2157: External fact support Closed 05/18/2012 05/18/2012

History

#1 Updated by Jeff Weiss over 2 years ago

I view this as a prereq to structured facts: #4561 and tangentially related to external facts: #2157

#2 Updated by Jeff Weiss over 2 years ago

  • Description updated (diff)

#3 Updated by Jeff Weiss about 2 years ago

  • Branch set to https://github.com/puppetlabs/facter/tree/integration/fact_definition

#4 Updated by Jeff Weiss about 2 years ago

  • Category set to interface

After talking with Pieter and our external caching decision (#14667), we’ll keep the existing DSL, but add on capabilities to define the types (when structured facts #4561 come along) though defaulting to :string and make some minor modifications to the confine logic so the the fact doesn’t need to be re-defined for the a special case (e.g. do this for Solaris and that for HP-UX).

Additionally, we’ll likely alias setcode and resolve with the hope of deprecating the terribly named setcode later.

#5 Updated by eric sorenson over 1 year ago

  • Target version changed from 2.1.0 to 2.x

Also available in: Atom PDF