carlasouza / puppet-virt

Puppet Module for Virtualization Management
http://puppetlabs.com
166 stars 72 forks source link

Fix initial configuration file validation #19

Closed codec closed 10 years ago

codec commented 11 years ago

Previous validation of the configfile parameter made it impossible to use a file managed by puppet. This is because the validation happens during compilation time when the file does not yet exist.

I've changed the validation to check for an absolute pathname and to autorequire the corresponding file resource in the catalog. Unfortunately autorequire won't fail if the resource does not exist.

carlasouza commented 10 years ago

Nice catch! Thanks for the PR