Feature #1424

Zone facts for Solaris

Added by Martin Englund almost 4 years ago. Updated 2 months ago.

Status:In Topic Branch Pending Review Start date:07/14/2008
Priority:Low Due date:
Assignee:shubhra sinha varma % Done:

0%
Category:library
Target version:-
Keywords:solaris zone Affected Facter version:
Branch:
Votes: 0

Description

Two new facts should be added for Solaris (10): zonename and zones

zonename is the output of /usr/bin/zonename which MAY differ from hostname.

zones is the number of zones on the system, and then one zone_ is added for each zone, which should contain the state of the zone, e.g. running, installed, incomplete, etc.

History

#1
Updated by James Turnbull almost 4 years ago

  • Status changed from Unreviewed to Accepted
  • Target version set to 14
  • Keywords set to solaris zone

#2
Updated by Paul Nasrat almost 4 years ago

Patch sent to list.

pull from

git pull git://github.com/pnasrat/facter.git tickets/1424

#3
Updated by Paul Nasrat almost 4 years ago

  • Status changed from Accepted to In Topic Branch Pending Review
  • Assignee set to Paul Nasrat

#4
Updated by Luke Kanies almost 4 years ago

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

As mentioned on the list, I’d prefer the zone list to be calculated each time the data is asked for, rather than one time at startup.

#5
Updated by Paul Nasrat over 1 year ago

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

#6
Updated by Paul Nasrat over 1 year ago

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

Sorry triage error.

#7
Updated by James Turnbull 9 months ago

  • Category set to library

#8
Updated by Nan Liu 4 months ago

zone, and zone state can be managed as puppet resources so not sure about the additional value of having facts. Couldn’t track down original code zonename added into triage-a-thon project: https://github.com/nanliu/puppet-solaris/tree/facter/feature/1424

#9
Updated by Martin Englund 4 months ago

the reason for having zones as a fact is to be able to gather information about the zones you have running on a system, and then zonename helps you act differently if you are dealing with a zone that hasn’t been created by puppet (i.e. being managed as a “normal” solaris instance)

#10
Updated by shubhra sinha varma 3 months ago

  • Status changed from Code Insufficient to In Topic Branch Pending Review
  • Assignee changed from Paul Nasrat to shubhra sinha varma

#11
Updated by shubhra sinha varma 3 months ago

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

Added the 3 facts – zones(#), zone_(status) and zonename(name). Sent a pull request. https://github.com/puppetlabs/facter/pull/167

#12
Updated by Ken Barber 3 months ago

Hi – please see my comments in the diff on github.

#13
Updated by shubhra sinha varma 3 months ago

updated the pull request. Test all good. Used the my_fixture() to read test file.

#14
Updated by shubhra sinha varma 3 months ago

the updates are on my pull request: https://github.com/puppetlabs/facter/pull/167/files

#15
Updated by Ken Barber 3 months ago

  • Status changed from Merged - Pending Release to In Topic Branch Pending Review
  • Target version changed from 14 to 186

#16
Updated by Daniel Pittman 2 months ago

  • Target version deleted (186)

Also available in: Atom PDF