| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
We've noticed a problem at Braintree along the lines of:
file { '/foo':
ensure => directory,
}
file { '/bar':
ensure => directory,
require => File['/foo/'],
}
This catalog compiles and is valid, but fails validation in octocatalog-diff.
It seems that puppet itself attempts to normalize file resource titles (and indeed
can normalize any type of resource title, given that a methodology exists in the
resource type). See:
https://github.com/puppetlabs/puppet/blob/4.10.x/lib/puppet/type/file.rb#L42
This commit attmepts to mimic that (and lifts the regex directly from the 4.10.x
branch of puppet, because it doesn't matter in puppet 5 since reference validation
is built-in).
…ulisse-ruby-version-test-update
| Back | FazBrowse Home | New Git URL |
Overview
This pull request updates the build matrix to cover Puppet's official puppet-ruby combinations, and updates our requirements documents too. In addition, this bumps the Puppet versions used in our CI to the latest available (today) releases of 4.10.10 and 5.4.0.
This necessitated a change to one of the tests since the error message for a reference validation failure in 5.4.0 is slightly different than 5.0.0. Since #176 deals with reference validation too, I just pulled that one into here too (cc @ahayworth).