summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Tidy up awkward reading commentlcg/application-base-classLamont Granquist2019-07-151-2/+1
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* support named run list on commandline in soloLamont Granquist2019-07-152-5/+5
| | | | | | this has to fix a bug for someone. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* consolidate some options in the base classLamont Granquist2019-07-153-53/+53
| | | | | | | | | | | | | | | most of these are options that apply to either chef-client -z or to chef-solo without legacy mode, so they are justifiably common options (that are handled by the trampolining into the Client code from Solo right now. the --legacy-mode switch is solo-specific (although we should wire it up for chef-client eventually once all the code is merged). old-reporting and skip-cookbook-sync only make sense for actual chef-client runs so are marked as such in the help text Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* --fips, --fatal-windows-admin-check and --disable-configLamont Granquist2019-07-153-19/+21
| | | | | | | | should fix fips mode for chef-solo, adds the windows admin check feature, but the distable-config option needs some future work to the base class to support it. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* add some comments on deprecation of the solo classLamont Granquist2019-07-153-0/+22
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* missing SELF_PIPE for the daemon loopLamont Granquist2019-07-152-3/+3
|
* migrate --delete-entire-chef-repo to base classLamont Granquist2019-07-153-10/+5
| | | | | | this is a trivial duplication Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Copy the daemon loop from client to baseLamont Granquist2019-07-153-157/+106
| | | | | | | These improvements never made it into solo and should be backwards compatible. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* add --target-mode to base classLamont Granquist2019-07-152-9/+9
| | | | | | this adds it to chef-solo Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* copy --ez to base classLamont Granquist2019-07-152-15/+16
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* change some uses of CLIENT to PRODUCT in the base classLamont Granquist2019-07-151-4/+4
|
* add --once to base classLamont Granquist2019-07-152-5/+5
| | | | | this was not present in chef-solo for whatever reason even though the code is there, it has always been broken
* normalize fetch_recipe_tarballLamont Granquist2019-07-153-25/+15
| | | | | | | | | | | this makes solo behave like client does. client was modified in these pulls: https://github.com/chef/chef/pull/7223 https://github.com/chef/chef/pull/7523 but those changes were never "ported" to solo.
* create application base class for de-duplicationLamont Granquist2019-07-153-346/+198
| | | | | | all of the changes in this commit should be mechanical and identical Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Bump version to 15.1.53 by Chef Expeditorv15.1.53Chef Expeditor2019-07-156-14/+15
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #8741 from chef/modern_spec_runnerTim Smith2019-07-155-40/+0
|\ | | | | Remove the unused rspec kitchen tests
| * Remove the rspec kitchen testsmodern_spec_runnerTim Smith2019-07-125-40/+0
| | | | | | | | | | | | We're not longer using these Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 15.1.52 by Chef Expeditorv15.1.52Chef Expeditor2019-07-156-14/+15
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #8740 from RubyTuesdayDONO/patch-1Lamont Granquist2019-07-151-1/+1
|\ \ | |/ |/| Fix RDoc copy-pasta in Chef::Mixin::ParamsValidate#validate
| * Fix RDoc copy-pasta in Chef::Mixin::ParamsValidate#validateReuben Garrett2019-07-111-1/+1
|/ | | | | | | | | | Signed-off-by: Reuben Garrett <ruby@rubydono.net> The RDoc for `Chef::Mixin::ParamsValidate#validate` appears to have inadvertently copied the (helpful) inline pseudocode of the `:is` option to the `:equal_to` option in commit 28f17b36a9d041be8ed50e20ae06fa5f5ea1fb38. Obvious fix.
* Bump version to 15.1.51 by Chef Expeditorv15.1.51Chef Expeditor2019-07-116-14/+15
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #8739 from chef/btm/omnibus-windowsTim Smith2019-07-111-1/+1
|\ | | | | Pull in latest omnibus-software to fix Windows builds
| * Pull in latest omnibus-software to fix Windows buildsbtm/omnibus-windowsBryan McLellan2019-07-111-1/+1
|/ | | | Signed-off-by: Bryan McLellan <btm@loftninjas.org>
* Bump version to 15.1.50 by Chef Expeditorv15.1.50Chef Expeditor2019-07-106-14/+15
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #8735 from chef/opensslTim Smith2019-07-101-1/+1
|\ | | | | Bump openSSL to 1.0.2s
| * Bump openSSL to 1.0.2sTim Smith2019-07-101-1/+1
| | | | | | | | | | | | This is a bugfix only release. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 15.1.49 by Chef Expeditorv15.1.49Chef Expeditor2019-07-106-14/+15
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #8736 from chef/prevent_double_bundlerLamont Granquist2019-07-102-18/+18
|\ \ | | | | | | Roll back Rubygems to 3.0.3 to prevent double bundler install
| * | Roll back Rubygems to 3.0.3 to prevent double bundler installTim Smith2019-07-102-18/+18
| |/ | | | | | | | | | | | | | | There's nothing important in this release other than a few minor bugfixes. The other alternative was to roll bundler forward to 1.17.3 so that the embedded rubygems bundler matched, but this meant we had bundler 1.17.2 built into ruby 2.6.3 and then 1.17.3 installed on top of that. There's little value in that and it bloats our package size. Let's not do that unless there's a critical bug or CVE we need in rubygems / bundler. This also bumps omnibus-software to include the new ruby cleanup def that fails if we have double bundler and the new rubygems def that removes the rubygems-update gem once rubygems is installed. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Fix the codeowners to work correctlyTim Smith2019-07-101-1/+1
| | | | | | | | | | | | | | Github skips empty groups even if they have child groups that have users in them. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 15.1.48 by Chef Expeditorv15.1.48Chef Expeditor2019-07-106-14/+15
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #8728 from chef/travisLamont Granquist2019-07-103-5/+4
|\ \ | | | | | | Remove rspec config for Travis
| * | Remove rspec config for TravistravisTim Smith2019-07-093-5/+4
| |/ | | | | | | | | | | We're running this all in buildkite w/o issues. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 15.1.47 by Chef Expeditorv15.1.47Chef Expeditor2019-07-106-14/+15
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #8726 from chef/expeditor/inspec-core_4.7.3Tim Smith2019-07-101-20/+18
|\ \ | |/ |/| Bump inspec-core to 4.7.3
| * Bump inspec-core to 4.7.3expeditor/inspec-core_4.7.3Chef Expeditor2019-07-091-20/+18
|/ | | | | | This pull request was triggered automatically via Expeditor when inspec-core 4.7.3 was promoted to Rubygems. This change falls under the obvious fix policy so no Developer Certificate of Origin (DCO) sign-off is required.
* Bump version to 15.1.46 by Chef Expeditorv15.1.46Chef Expeditor2019-07-096-14/+15
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #8719 from chef/examplesTim Smith2019-07-083-1/+65
|\ | | | | Add examples to the apt_repository resource
| * Add examples to the resourcesTim Smith2019-07-083-1/+65
|/ | | | | | | | | Add examples in markdown to the resources. We'll be using this on the docs site in the future and if we need to display it we can use tty-markdown which does a really nice job of displaying markdown in the terminal Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 15.1.45 by Chef Expeditorv15.1.45Chef Expeditor2019-07-086-14/+15
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #8711 from chef/lcg/new-chefstyle2Lamont Granquist2019-07-08676-3094/+3459
|\ | | | | More Chefstyle updates
| * fix multiline stringLamont Granquist2019-07-081-9/+9
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * Style/NegatedUnlessLamont Granquist2019-07-081-1/+1
| | | | | | | | | | | | found one. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * lazy procs are instance_exec in the resource alreadyLamont Granquist2019-07-057-14/+14
| | | | | | | | | | | | | | | | exposed by Style/SymbolProc mangling some bad code. no idea how this ever worked. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * fix 4 copsLamont Granquist2019-07-0560-189/+296
| | | | | | | | | | | | | | | | | | | | | | | | | | | | - Layout/MultilineMethodCallBraceLayout - Layout/ClosingParenthesisIndentation - Layout/IndentFirstArgument EnforcedStyle: consistent - Layout/BlockEndNewline the first of these autocorrected to horrible looking code which exposed that we really needed the other three as well, which also cleaned up a bunch of other terrible looking code. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * Style/SymbolProcLamont Granquist2019-07-0576-161/+121
| | | | | | | | | | | | enforce pretzels. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * Style/StringLiteralsInInterpolationLamont Granquist2019-07-05148-433/+433
| | | | | | | | | | | | since we use double quotes, be consistent everywhere. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * Style/ClassCheckLamont Granquist2019-07-0557-119/+119
| | | | | | | | | | | | convert kind_of? to is_a? Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * Style/RegexpLiteralLamont Granquist2019-07-0558-149/+149
| | | | | | | | | | | | | | given how many regexps we have with /'s in the match this seems like a very good one. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * Style/NegatedIfLamont Granquist2019-07-0587-149/+149
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>