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

Bug #12147

iphostnumber fact, on Darwin, is redundant to other cross-platform facts.

Added by Daniel Pittman about 2 years ago. Updated almost 2 years ago.

Status:ClosedStart date:01/25/2012
Priority:NormalDue date:
Assignee:Jeff Weiss% Done:

0%

Category:-
Target version:2.0.0
Keywords: Affected Facter version:
Branch:https://github.com/puppetlabs/facter/pull/205

We've Moved!

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

This issue is currently not available for export. If you are experiencing the issue described below, please file a new ticket in JIRA. Once a new ticket has been created, please add a link to it that points back to this Redmine ticket.


Description

lib/facter/iphostnumber.rb defines the “primary IP address” and “primary mac” of a Darwin system, and is redundant to other facts that extract the same thing.

It should go away in favour of the other implementation, and be just a thin alias until we can kill the fact entirely.

History

#1 Updated by James Turnbull about 2 years ago

Thin alias. People use the fact.

#2 Updated by Daniel Pittman about 2 years ago

James Turnbull wrote:

Thin alias. People use the fact.

Didn’t I say that, or are you suggesting we need to carry the fact forever because it is presently used?

#3 Updated by James Turnbull about 2 years ago

I read “and” as “or”.

#4 Updated by Jeff Weiss almost 2 years ago

  • Assignee set to Anonymous
  • Target version set to 2.0.0

#5 Updated by Jeff Weiss almost 2 years ago

  • Assignee changed from Anonymous to Jeff Weiss

These facts are only resolved on the 6.X version of the Darwin kernel (aka Jaguar), circa 2003. Considering we have already said the minimum kernel we support is 8.X (aka Tiger, with Panther in between), I’m removing it with extreme prejudice. There’s no reason to keep this cruft around.

#6 Updated by Jeff Weiss almost 2 years ago

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

#7 Updated by Daniel Pittman almost 2 years ago

  • Status changed from In Topic Branch Pending Review to Merged - Pending Release

#8 Updated by Matthaus Owens almost 2 years ago

  • Status changed from Merged - Pending Release to Closed

Released in facter 2.0.0rc1

Also available in: Atom PDF