Bug #10509

Display local_volume mount points as a fact

Added by Matt Moran 7 months ago. Updated 6 months ago.

Status:Duplicate Start date:11/03/2011
Priority:Normal Due date:
Assignee:Matt Moran % Done:

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

Description

Comma delimited fact that displays all the local volumes of a host.

Very handy to use with monitoring applications etc.

Related issues

duplicates Facter - Feature #2847: mountpoint fact Code Insufficient 11/21/2009

History

#1
Updated by Adrien Thebo 7 months ago

Would this be covered by the blockdevice fact that would be added as part of #10238?

#2
Updated by Adrien Thebo 7 months ago

  • Status changed from Unreviewed to Investigating

#3
Updated by Krzysztof Wilczynski 6 months ago

Adrien Thebo wrote:

Would this be covered by the blockdevice fact that would be added as part of #10238?

Adrien, I believe that Matt will be using this ticket as a place-holder to add slightly more polished version of his fact with added tests. Not quite the same as #10238 (unless I am mistaken completely :-D).

He was working on it as per this thread on puppet-dev.

I took his initial version, and came up with my take on it (Matt takes all the credit for coming up with the idea): local_volumes.rb

KW

#4
Updated by Matt Moran 6 months ago

Not really, I want to expose the mount points rather than the block devices.

#5
Updated by Adrien Thebo 6 months ago

I clearly wasn’t reading closely enough when I commented initially, pardon that error. Could you clarify why you’re handling thread safety in this fact? Have you seen invalid facter runs as a result of improper locking?

#6
Updated by Krzysztof Wilczynski 6 months ago

Adrien Thebo wrote:

[…]

Could you clarify why you’re handling thread safety in this fact? […]

Because it was written from before I have had conversation with Ken about that sort of things :)

Feel free to remove mutexes, etc; and possibly move everything in under setcode block, so it complies with the new caching framework :–)

KW

#7
Updated by Ken Barber 6 months ago

  • Target version changed from 2.0.0 to 186

#8
Updated by Ken Barber 6 months ago

  • Status changed from Investigating to Duplicate
  • Target version deleted (186)

This is one of four tickets that are duplicated in many ways. I’m closing this – reference the older #2847 from now on.

Also available in: Atom PDF