Last modified: 2014-01-31 09:55:32 UTC
Gerrit change #107037 the first patch contain a not valid puppet manifest: modules/generic/manifests/upstart_job.pp line 13: wner => 'root', instead of owner => 'root', it could be useful if we try to understand why the build passed
The Jenkins job operations-puppet-validate is running the command 'puppet parser validate' on each .pp file. That command does not detect such issue: $ echo "file { '/tmp/foo': wner => 'root' }"|puppet parser validate; echo $? 0 $ So that is valid as far as puppet parser is concerned. Such issue might be caught by compiling the catalog, I am not sure how we can achieve that though. Rephrased summary.
That wouldn't get picked up because it's valid syntax. The only solution is compile it on all nodes, get the output and diff it with the previous one (and check if all the diff are expected). Not so easy to implement
Closing bug, we can not really catch such issues :/