| Age | Commit message (Collapse) | Author |
|---|
|
puppetlabs-augeas_core and puppet-augeasproviders_core both provide the
augeas feature. However the both have slightly different files (this
lacking the frozen strings magic comment. This causes puppet to perform
a change on every puppet run.
This PR updates the feature in this repo to add the magic comment so
both files match. More then happy to explore alternate fixes
|
|
Puppet Strings prefers a `@summary` tag with a max of 140 characters to
fill in the Table of Contents summary. Without this, the description
will appear truncated.
Add `@summary` and `@example` tags to the type documentation.
|
|
|
|
Also updated `.sync` file to include the `auto_release.yml` workflow
|
|
Fix a regression introduced by MODULES-7397[1] which incorrectly assumed
that the application name can be queried through
`Puppet::Application.name`, causing lenses that are pluginsynced not to
be loaded.
Since we only need to find out whether or not we're running as part of
`puppet agent`, it should be enough to use `Puppet.run_mode.name`, which
returns `:agent` in this case, and `:user` otherwise.
[1] https://github.com/puppetlabs/puppetlabs-augeas_core/pull/27
|
|
When creating the load path for Augeas, the module directories are now
also searched for lenses and their folder path is added accordingly.
This happens in every context except `agent` application. Through
plugin syncing (and later cached lenses) it does not need these extra
paths.
|
|
Prior to this commit, when the augeas_core module was installed via
`puppet module install`, the helper code loaded came from the vendored
module rather than the installed module. Moving the `require` to
`require_relative` fixes that.
|
|
evaluate: undefined method strip! for nil:NilClass
|
|
|
|
|
|
This ports PUP-8974, and the related follow-up maintenance commits from
the Puppet repo.
The augeas provider used Kernel#eval to convert stringified arrays to Ruby
arrays. For example, it extracted the array part of the "clause" below:
onlyif => 'values HostKey == ["/etc/ssh/ssh_host_rsa_key"]'
and called Kernel#eval with '["/etc/ssh/ssh_host_rsa_key"]'. Using eval is
bad because it executes arbitrary code.
This commit changes the provider to convert the comma delimited string to
a Ruby array. This mostly maintains the functionality of the original
Kernel#eval (minus running arbitrary code) except for no longer handling
the \M-x, \M-\C-x, \M-\cx, \c\M-x, \c?, and \C-? escape sequences in
double-quoted strings, and \u{nnnn ...} is more lenient about whitespace.
|
|
Previously we were using eval to convert stringified arrays from the
manifest into a ruby array. Use JSON instead, and ensure values are
double quoted as required by JSON.
|
|
See https://github.com/rubocop-hq/ruby-style-guide/commit/df933c0f9b3a805c00e5ef81889992b1aec787a7
|
|
|
|
Add exclusions for:
Layout/IndentHeredoc,
Metrics/BlockNesting:
Metrics/LineLength:
Style/AccessorMethodName:
Style/DoubleNegation:
Style/MultipleComparison:
Style/NumericPredicate:
Style/PredicateName:
Style/SignalException:
Style/VariableName:
SignalException is disabled because Puppet::Type#fail overrides Kernel#fail,
and we really do want fail and not raise.
|
|
|
