Bug #10508

EC2 facts not available if endpoint timesout or is unavailable

Added by Matt Moran 7 months ago. Updated 10 days ago.

Status:Code Insufficient Start date:11/03/2011
Priority:Normal Due date:
Assignee:- % Done:

0%
Category:library
Target version:-
Keywords: Affected Facter version:1.5.9
Branch:https://github.com/puppetlabs/facter/pull/95
Votes: 0

Description

Using EC2 facts in templates or putting conditionals around them can be surprising, due to if the EC2 facts are unavailable your puppet runs may produce unexpected behaviour.

It would be good if variables from the EC2 metadata endpoint were cached in a file. This can be used to fulfil the facts if the endpoint is unavailable or timesout.

Related issues

blocked by Facter - Bug #11583: ec2 fact has no unit tests Closed 12/28/2011
blocked by Facter - Feature #4519: Facter should provide a convenient way to cache values Accepted 08/11/2010

History

#1
Updated by Adrien Thebo 7 months ago

Does it look like #1257 will add this functionality?

#2
Updated by Adrien Thebo 7 months ago

  • Status changed from Unreviewed to In Topic Branch Pending Review
  • Branch set to https://github.com/puppetlabs/facter/pull/94

#3
Updated by Adrien Thebo 7 months ago

  • Branch changed from https://github.com/puppetlabs/facter/pull/94 to https://github.com/puppetlabs/facter/pull/95

#4
Updated by Ken Barber 7 months ago

  • Status changed from In Topic Branch Pending Review to Code Insufficient

See my comments in the pull request.

#5
Updated by Ken Barber 6 months ago

  • Target version changed from 2.0.0 to 186

#6
Updated by Patrick Otto 4 months ago

  • Assignee changed from Matt Moran to Ken Barber

Ken, looking at the pull request I’d say this can be closed as it will be handled in #2157

#7
Updated by Ken Barber 4 months ago

Patrick,

Not necessarily ‘closed’ as we’ll need to think about how this fact could utilise the cache once its implemented I believe, as the suggestion here is to use cache as a fallback if it times out its slightly different behaviour then what is defined in #4519.

So I think its valid as a separate ticket, I’ll keep it open for now to track it.

ken.

#8
Updated by Daniel Pittman 2 months ago

  • Target version deleted (186)

#9
Updated by Ken Barber 10 days ago

  • Assignee deleted (Ken Barber)

Also available in: Atom PDF