Issue #11041 has been updated by Ken Barber.

MAC addresses can be spoofed/changed. In a virt environment they let you 
specify your own in some cases so its not perfect. Again - this comes back to 
the discussion we had regarding gathering a lot of various pieces and weighing 
it up - but mac addresses being very mutable only carry a little weight.

Like I said the /sys/class/dmi/id/* area is there to use - I would use that in 
preference for kernels that support it and fall back to dmidecode in its 
absence. In the cases where OS don't have /sys/class/dmi/id then is it all bad 
to really on dmidecode from a package perspective in this case? We are probably 
talking What is wrong with dmidecode in a minimal system if the information is 
needed and the other alternatives are complex? As we discussed - it taps 
/dev/mem which is going to be hard to do on our own :-). 

BTW - this does affect Redhat 5 .... it doesn't have /sys/class/dmi ... so you 
are stuck using other methods anyway ... from a package perspective its going 
to be pciutils or dmidecode ...

I think the other side of the coin is a concept of 'certainty'. If lspci, 
dmidecode, vmware doesn't exist, nor the sysfs - how can we be totally sure the 
system is virtual or not? Falling back to 'physical' ... is this really correct 
in this case? Maybe we should default to 'unknown'? I would see this is a 1.7.x 
change though :-).
----------------------------------------
Bug #11041: facter-1.6.3 rpm is missing dmidecode as dependency
https://projects.puppetlabs.com/issues/11041

Author: Florian Koch
Status: Accepted
Priority: High
Assignee: 
Category: library
Target version: 1.6.x
Keywords: vmware rpm demidecode
Branch: 
Affected Facter version: 1.6.3


Hi,

i have a minimal Scientific Linux installation on a Vmware virtual Server, no 
dmidecode is installed. I install facter  and get 

    facter | grep virtual
    is_virtual => false
    virtual => physical

i install dmidecode and get 
    facter | grep virtual
    is_virtual => true
    virtual => vmware

so dmidecode is needed by facter to detect vmware and should a rpm dependency


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to