| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
Somewhat unintuitively, mixlib-cli needs both `boolean => true` as well as
`long => "--[no-]option"` for the boolean feature to work.
Signed-off-by: Bryan McLellan <btm@loftninjas.org>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|\
| |
| | |
Core: fix Lint/UnifiedInteger cop
|
| |
| |
| |
| |
| |
| | |
believe this becomes necessary for ruby 2.4/3.0
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|/
|
|
|
|
|
|
| |
This adds URLs to each class of deprecation, and correctly prints and
formats them for maximum user efficiency. We also provide the URL to the
data collector for Visibility to ingest.
Signed-off-by: Thom May <thom@chef.io>
|
|
|
|
|
|
| |
a few cops we have enabled got a little more accurate/stricter
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Jeremy J. Miller <jm@chef.io>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|\ |
|
| |
| |
| |
| |
| |
| |
| | |
this allows us to be truely cross platform, and also to ignore unsafe
paths and permissions.
Signed-off-by: Thom May <thom@chef.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Solo has always defaulted to `/var/chef` but local-mode uses the user's
home directory, so we need to ensure that we set the correct path before
entering local-mode.
Fixes: #5051
Signed-off-by: Thom May <thom@may.lt>
|
|/ |
|
|\
| |
| | |
Ensure recipe-url works right in solo
|
| |
| |
| |
| | |
Signed-off-by: Thom May <thom@chef.io>
|
|/
|
|
|
|
|
|
|
|
|
|
| |
* exit code functional specs
* audit exit codes
* reboot now/reboot failed/reboot pending exit codes
* Deal with forked and unforked process and get the right exit code
* Reboot Now should really be reboot scheduled
* pass exception rather than exit code
* updated with sigint and sigterm
* support legacy fatal!("", 2) behavior
* fixup all fatal! and exit! calls
|
|
|
|
|
|
| |
per review
Signed-off-by: Thom May <thom@may.lt>
|
|
|
|
| |
Signed-off-by: Thom May <thom@may.lt>
|
|
|
|
|
|
| |
ie, only when we're in OG mode and not in local mode.
Signed-off-by: Thom May <thom@chef.io>
|
|
|
|
| |
Signed-off-by: Thom May <thom@chef.io>
|
| |
|
| |
|
| |
|
|\
| |
| | |
fix some UselessAssignment cases
|
| | |
|
| | |
|
|/
|
|
|
| |
this is part of our informal style guide, lets make it formal since
clearly its not getting followed very well.
|
|\
| |
| | |
Autofixing new Perf cops in 0.37.2
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
6 Performance/Casecmp
18 Performance/Detect
1 Performance/RangeInclude
27 Performance/RedundantBlockCall
6 Performance/RedundantMatch
5 Performance/RedundantMerge
18 Performance/StringReplacement
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
Passing the value of `escape_glob` to Dir.glob does
not always work correctly on Windows. Always use
forward slashes when globbing, even on Windows. We
now do this via `escape_glob_dir`, which calls
Pathname.cleanpath, which does the foward slash thing.
|
| |
| |
| |
| | |
Also, do less mocking so the test will catch the error
|
|/
|
|
|
| |
client.d/*.rb will be read in sorted order. All directories will be
ignored.
|
|
|
|
| |
no enforced trailing comma on arguments...
|
| |
|
|
|
|
|
| |
Use with caution. Useful for patching a set of cookbooks on a machine
when iterating during development.
|
| |
|
|
|
|
|
| |
this is from the same ruleset that we had, but the new code catches more
conditions.
|
|\
| |
| | |
fix a batch of cops
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Style/NegatedWhile
Style/ParenthesesAroundCondition
Style/WhileUntilDo
Style/WordArray
Performance/ReverseEach
Style/ColonMethodCall
|
| | |
|
| | |
|
| |
| |
| |
| | |
this really closes #1515 and #1751
|
|/
|
|
|
|
|
|
|
|
|
| |
- deprecates '-r' used for the recipe_url in chef-solo
- adds --delete-entire-chef-repo option for users who want the
old behavior back.
- cleans up some old code
closes #3802
closes #1515
closes #1751
|
|
|
|
| |
second round of autofixing for some reason...
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
4174 Style/SpaceInsideHashLiteralBraces
1860 Style/SpaceAroundOperators
1336 Style/SpaceInsideBlockBraces
1292 Style/AlignHash
997 Style/SpaceAfterComma
860 Style/SpaceAroundEqualsInParameterDefault
310 Style/EmptyLines
294 Style/IndentationConsistency
267 Style/TrailingWhitespace
238 Style/ExtraSpacing
212 Style/SpaceBeforeBlockBraces
166 Style/MultilineOperationIndentation
144 Style/TrailingBlankLines
120 Style/EmptyLineBetweenDefs
101 Style/IndentationWidth
82 Style/SpaceAroundBlockParameters
40 Style/EmptyLinesAroundMethodBody
29 Style/EmptyLinesAroundAccessModifier
1 Style/RescueEnsureAlignment
|
|\
| |
| | |
Allow use of command line fips switch for knife
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This somewhat worked before. However, it was just for knife bootstrap.
It also didn't support --no-fips in the case where it was in your
knife.rb and you decided that you didn't want to use fips for
that one call.
The assumption here is fips mode you run knife with is the fips
mode the node will get. This has the nice property that validatorless
bootstraps will talk to the chef server in fips mode if the node
is requested to be in fips mode.
|
| | |
|
| |
| |
| | |
Generated via git ls-files | xargs perl -pi -e "s/(Author.*?<[^@]+@)(?:opscode\\.com|getchef\\.com)(>)/\\1chef.io\\2/gi"
|