I'd suggest defining:

     Virtual_systems = %w{ xenu openvzve vmware kvm vserver jail }

and then writing

     setcode do
         Virtual_systems.include?(Facter.value(:virtual)).to_s
     end

or

     setcode do
         "#{Virtual_systems.include? Facter.value(:virtual)}"
     end

instead of retaining the clunky case statement, but either way does the job.

-- Markus
-----------------------------------------------------------
The power of accurate observation is
commonly called cynicism by those
who have not got it.  ~George Bernard Shaw
------------------------------------------------------------

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Developers" 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-dev?hl=en.

Reply via email to