summaryrefslogtreecommitdiff
path: root/lib/chef/knife
Commit message (Collapse)AuthorAgeFilesLines
* changes for rubocop engine upgrades.lcg/rubocop-upgradesLamont Granquist2019-01-1510-20/+20
| | | | | | | this is the result of changes to rules we already previously had enabled. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* change HTTPServerException to HTTPClientException for ruby 2.6Lamont Granquist2019-01-038-8/+8
| | | | | | includes backcompat for 2.5 Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Merge pull request #7852 from chef/cookbook_createTim Smith2018-11-051-29/+0
|\ | | | | Fully remove knife cookbook create command
| * Fully remove knife cookbook create commandcookbook_createTim Smith2018-11-021-29/+0
| | | | | | | | | | | | We already removed all functionality and replaced it with an error message. This prevents the command from showing up in help. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #7869 from chef/knife_descriptionsTim Smith2018-11-0514-14/+14
|\ \ | | | | | | Make knife command banners consistent
| * | Make knife command banners consistentknife_descriptionsTim Smith2018-11-0414-14/+14
| | | | | | | | | | | | | | | | | | Add (options) everywhere you can pass in options. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Remove the remaining OSC 11 knife user commandsknife_osc_11Tim Smith2018-11-045-267/+0
|/ / | | | | | | | | | | | | | | The previous PR removed the argv rewrite, but didn't remove the actual commands it rewrote to. Now we remove the commands so they don't show up in help. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Remove knife user support for open source Chef Server < 12Tim Smith2018-11-048-344/+50
|/ | | | | | We shipped Chef 12 server on 2014-11-25. We've maintained backwards compatibility here by shipping double plugins, constantly checking for Chef 11 and then rewriting passed args. We deprecated it with Chef 14. This fully removes support. chef-client will continue to work on chef 11, but users cannot be managed with knife from a workstation running the latest version of Chef/ChefDK. Signed-off-by: Tim Smith <tsmith@chef.io>
* Remove knife cookbook test featureTim Smith2018-11-011-95/+0
| | | | | | We deprecated this during the Chef 13 cycle. It's time to fully remove it. Signed-off-by: Tim Smith <tsmith@chef.io>
* Remove deprecated knife status --hide-healthy flaghide_healthyTim Smith2018-10-291-13/+0
| | | | | | We've been telling people this was going away in Chef 15 Signed-off-by: Tim Smith <tsmith@chef.io>
* Remove cookbook merging/shadowing from the cookbooker loaderLamont Granquist2018-10-261-23/+2
| | | | | | This has been deprecated since forever. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Merge pull request #7489 from chef/bootstrap_identity_fileTim Smith2018-10-261-5/+1
|\ | | | | Remove the deprecated knife bootstrap --identity-file flag
| * Remove the deprecated knife bootstrap --identity-file flagbootstrap_identity_fileTim Smith2018-08-131-5/+1
| | | | | | | | | | | | Remove this deprecated flag Signed-off-by: Tim Smith <tsmith@chef.io>
* | better docs for Chef::Knife::Bootstrap#validate_options!Alexandre IGNJATOVIC2018-10-041-1/+7
| | | | | | | | Signed-off-by: Alexandre Ignjatovic <alexandre.ignjatovic@gmail.com>
* | replace some instances of to_hash with to_hlcg/to-h-cleanupLamont Granquist2018-09-264-8/+8
| | | | | | | | | | | | | | | | | | | | | | to_hash on a lot of these objects should go away, but even eliminating all our calls to to_hash on these objects internally is difficult. (e.g. converting the knife ui code to call #to_h means we wind up calling nil#to_h which "helpfully" becomes '{}' which is hilarious and i don't know why someone thought that was a good idea). Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | Check for an actual value, rather than the key being set.Noah Kantrowitz2018-08-211-1/+1
|/ | | | Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
* Merge pull request #7473 from chef/knife_sentences_2Tim Smith2018-07-2341-96/+98
|\ | | | | Make sure knife descriptions all have periods
| * Make sure knife descriptions all have periodsknife_sentences_2Tim Smith2018-07-1641-96/+98
| | | | | | | | | | | | Some did. Some didn't. Looks weird that way. Signed-off-by: Tim Smith <tsmith@chef.io>
* | @returns should be @return in yardyard_fixesTim Smith2018-07-161-1/+1
| | | | | | | | | | | | Avoid yard warnings about an invalid tag Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #7472 from chef/json_cleanupTim Smith2018-07-1522-23/+1
|\ \ | |/ |/| Remove require json_compat where not used
| * This plugin needs role not node.json_cleanupTim Smith2018-07-151-1/+1
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Remove require json_compat where note usedTim Smith2018-07-1522-22/+0
| | | | | | | | | | | | We're not directly manipulating json in these knife plugins. We're using mostly using the API class or environment/role classes directly. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Improve the error message when knife bootstrap windows isn't installedknife_cleanupTim Smith2018-07-131-1/+1
| | | | | | | | | | | | If the user really wanted to run knife boostrap windows then the "hostname" bit here doesn't make sense. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Add some yard / commentsTim Smith2018-07-131-0/+10
|/ | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Move all plugin logic into knife supermarketknife_supermarketTim Smith2018-07-1314-575/+585
| | | | | | | | We want to deprecate knife cookbook site. This moves all the logic into the knife supermarket plugins and has the legacy knife cookbook site plugins inherit from that. Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #7455 from coderanger/config-commandsTim Smith2018-07-134-1/+210
|\ | | | | Add knife config get/use-profile commands
| * Feed the rubocop.Noah Kantrowitz2018-07-111-5/+5
| | | | | | | | Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
| * New `knife config list-profiles` command to show available profiles.Noah Kantrowitz2018-07-111-0/+121
| | | | | | | | Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
| * Add two new knife commands for getting and setting the credentials profile name.Noah Kantrowitz2018-07-102-0/+87
| | | | | | | | Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
| * Add support for knife commands with hyphens.Noah Kantrowitz2018-07-101-1/+1
| | | | | | | | | | | | This works exactly like with an underscore, but is a bit more user-friendly sometimes (e.g. `get_config` vs `get-config`). Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
| * Don't mutate the argument because that makes debugging weird.Noah Kantrowitz2018-07-101-0/+1
| | | | | | | | Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
* | Provide a better error when knife plugins are missing from the cachebetter_rehash_errorTim Smith2018-07-111-1/+1
|/ | | | | | No one outside chef developers knows what the manifest is and we don't actually tell them what command to run. Leave the details out. Just tell them how to fix it. Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #7390 from coderanger/yet-more-credsLamont Granquist2018-07-031-0/+126
|\ | | | | knife config and a bunch of UX improvements
| * Merge branch 'master' into yet-more-credsNoah Kantrowitz2018-07-025-12/+12
| |\ | | | | | | | | | Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
| * \ Merge branch 'master' into yet-more-credsNoah Kantrowitz2018-07-0280-903/+1087
| |\ \ | | | | | | | | | | | | Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
| * | | Rename the command to `knife config get`.Noah Kantrowitz2018-06-281-2/+2
| | | | | | | | | | | | | | | | Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
| * | | Add fallback when asking for a specific key that isn't in the non-all data.Noah Kantrowitz2018-06-231-6/+7
| | | | | | | | | | | | | | | | Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
| * | | Show nicer labels for STDOUT/ERR.Noah Kantrowitz2018-06-231-0/+9
| | | | | | | | | | | | | | | | Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
| * | | Add a new `knife config` command.Noah Kantrowitz2018-06-231-0/+116
| | | | | | | | | | | | | | | | | | | | | | | | This helps to inspect the configuration, aid in debugging, and work with scripts. Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
* | | | fix Style/MutableConstantLamont Granquist2018-07-022-2/+2
| |_|/ |/| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | fix Style/PreferredHashMethodsLamont Granquist2018-07-025-12/+12
| |/ |/| | | | | | | | | | | absolutely hard requirement on the fixes that went into chef-config 2.2.11, so the floor of that gem is bumped up. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | fix some Style/StringLiterals introduced by the UnneededInterpolationLamont Granquist2018-07-022-2/+2
| | | | | | | | | | | | cop Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | fix Style/UnneededInterpolationLamont Granquist2018-07-023-3/+3
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | fix Layout/LeadingCommentSpaceLamont Granquist2018-07-024-6/+6
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | clean up whitespace againLamont Granquist2018-07-023-19/+19
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | fix Layout/IndentHeredocLamont Granquist2018-07-0214-154/+154
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | fix Style/HashSyntaxLamont Granquist2018-07-0267-756/+756
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | Merge pull request #7400 from chef/missing_knife_requireTim Smith2018-06-271-0/+2
|\ \ | | | | | | Add missing require knife
| * | Add missing require knifeTim Smith2018-06-261-0/+2
| |/ | | | | | | | | | | This should be in all the plugins and it is except for this one. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Add missing knife license headersTim Smith2018-06-2622-3/+185
|/ | | | | | Fix some spacing as well Signed-off-by: Tim Smith <tsmith@chef.io>