| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Signed-off-by: mwrock <matt@mattwrock.com>
|
|
|
|
|
|
| |
We're working too hard here. No need for these.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|
|
|
|
|
| |
So much easier to read and probably faster this way too.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|\
| |
| | |
Move the alias for attribute to property right into the property mixin
|
| |
| |
| |
| |
| |
| | |
Stick it right next to the property def so it's pretty obvious how this works.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|/
|
|
|
|
|
| |
It doesn't look like we're using this anymore and the comment states
that we wouldn't need the implementation after Ruby 2.0 anyways.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|\
| |
| | |
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
| |
| |
| | |
Signed-off-by: mwrock <matt@mattwrock.com>
|
| |
| |
| |
| | |
Signed-off-by: mwrock <matt@mattwrock.com>
|
|/
|
|
| |
Signed-off-by: mwrock <matt@mattwrock.com>
|
|\
| |
| | |
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
| |
| |
| | |
Signed-off-by: Luigi <007.lva@gmail.com>
|
|/
|
|
| |
Signed-off-by: mwrock <matt@mattwrock.com>
|
|
|
|
|
| |
Signed-off-by: Tim Smith <tsmith@chef.io>
Co-authored-by: pete higgins <pete@peterhiggins.org>
|
|
|
|
|
|
| |
Avoid a sort then last and just grab the max.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|
|
|
| |
Signed-off-by: mwrock <matt@mattwrock.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
While this behavior was fixed for attributes a long, long time ago it
turns out we don't use this deep_merge mixin for constructing the
deep merge of attribute precedence levels any more.
This bug however surfaces because we still use deep_merge to merge
json attributes into normal attributes. That means that it is not
possible to pass a nil / null value in through -j and have it override
a normal attribute which is pulled back in from the node object on
the chef-server.
So this fixes that behavior to be consistent with the behavior of
attributes that we've had for several years now.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Thomas Heinen <theinen@tecracer.de>
|
|
|
|
| |
Signed-off-by: Thomas Heinen <theinen@tecracer.de>
|
|
|
|
|
|
|
| |
Only require external libraries if we need to. Also use require_relative
everywhere we can.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|
|
|
|
|
|
| |
It ain't quite private, but if people find it useful but not quite what
they want they need to maintain their own implementation.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Mostly does what the title says, there wasn't much that depended on
compile/converge ordering and mostly this might fix a bug or two
in intention.
Renamed a method that had a misspelling.
Used tap in two places.
Removed the mutation of the new_resource in two places which is
the most disruptive part of this change.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
|
|
|
| |
Fixing a few things that may have broken selinux on EL6 through 'which'
support.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
|
|
| |
also fixes a FIXME in the YARD
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
|\
| |
| | |
Avoid assigning variables before returning if we don't have to
|
| |
| |
| |
| |
| |
| | |
These seemed pretty straight forward. There's a new Rubocop cop for these.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|/
|
|
|
|
| |
This makes it easier to read the classes.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|
|
|
|
|
| |
Autocorrected from RuboCop Performance which is now smart enough to detect when you use the match and when you don't. Using match? does not create any objects so it's slightly faster and uses less memory.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|
|
|
|
|
| |
There's a RuboCop cop to fix these before Ruby 3 breaks it.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|
|
|
| |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
|
|
|
| |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
|
|
|
| |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
|
|
|
|
|
| |
Autocorrected with chefstyle after enabling that cop. It was very hard to read some complex autocorrected blocks w/o this.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|
|
|
|
|
| |
This matches powershell_out and it's something we need to easily migrate resources to the new helper.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
|
|
| |
I think we may finally have them all now.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|
|
|
|
|
|
|
| |
Legally incredibly dubious, particularly since we don't follow it
strictly as policy, and we have git history instead, which does it right.
This is just a waste of time and a cargo cult.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
|
|
|
| |
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
|
|
|
|
|
|
|
| |
If no other property is an identity property then the name_property is
the default identity property. The name_property is also marked as not
being desired state by default (the 'name' should never change).
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
|
|
|
|
|
| |
When we migrated to hugo the URLs changed a bit. Nothing ends in .html and we moved all the resources into their own dir.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|
|
|
| |
Signed-off-by: Julien Huon <julien@huon.email>
|
|
|
|
| |
Signed-off-by: Julien Huon <julien@huon.email>
|
|
|
|
|
|
| |
x509_certificate resource
Signed-off-by: Julien Huon <julien@huon.email>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is the implementation of [RFC-087](https://github.com/chef-boneyard/chef-rfc/blob/master/rfc087-distro-sugar-helpers.md)
although some of the specifics have been iterated on and changed.
The documentation will be in the [README.md](https://github.com/chef/chef/tree/master/chef-utils/README.md) once this is merged.
While this PR mostly moves chef-sugar utilities into core-chef via this chef-utils gem, the scope of the chef-utils gem
should be considered larger than just that. As an example this PR moves the Mash class into this gem for reuse in ohai
as well.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|