summaryrefslogtreecommitdiff
path: root/lib/chef/knife
Commit message (Collapse)AuthorAgeFilesLines
* Remove deprecated bootstrap optionsTim Smith2018-03-121-23/+3
| | | | | | We said we would remove these in Chef 13 in the code. Lets do it for 14 instead. Signed-off-by: Tim Smith <tsmith@chef.io>
* also not a hashLamont Granquist2018-02-281-1/+1
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* not actually a hashLamont Granquist2018-02-281-1/+1
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* fix for master of chefstylelcg/fix-chefstyleLamont Granquist2018-02-282-2/+2
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Merge pull request #6781 from linyows/notice-duplicated-ipThom May2018-02-271-0/+16
|\ | | | | Raise fatal error If FQDN duplicated
| * :police_car:linyows2018-02-271-1/+1
| | | | | | | | Signed-off-by: linyows <linyows@gmail.com>
| * support behavior for duplicated_fqdns optionlinyows2018-02-131-8/+10
| | | | | | | | Signed-off-by: linyows <linyows@gmail.com>
| * use skip_on_duplicated_fqdns as command optionlinyows2018-01-251-5/+13
| | | | | | | | Signed-off-by: linyows <linyows@gmail.com>
| * to warning instead of fatal errorlinyows2018-01-251-3/+2
| | | | | | | | Signed-off-by: linyows <linyows@gmail.com>
| * rename variable to fqdnslinyows2018-01-241-5/+5
| | | | | | | | Signed-off-by: linyows <linyows@gmail.com>
| * ui.fatal if ssh nodes are duplicatedlinyows2018-01-231-0/+7
| | | | | | | | Signed-off-by: linyows <linyows@gmail.com>
* | Merge pull request #6861 from smcavallo/bootstrap_preinstall_commandThom May2018-02-272-0/+11
|\ \ | | | | | | Add support for knife bootstrap-preinstall-command
| * | Add support for knife bootstrap-preinstall-commandS.Cavallo2018-02-152-0/+11
| | | | | | | | | | | | Signed-off-by: S. Cavallo <smcavallo@hotmail.com>
* | | [knife] Don't crash when a deprecated cookbook has no replacement (#6853)rlyders2018-02-271-2/+7
| | | | | | | | | * Don't crash when a deprecated cookbook has no replacement
* | | Added Flag to distinguish between gateway and host key to fix issue #6210 ↵Erik Parra2018-02-271-7/+8
| | | | | | | | | | | | | | | | | | | | | (#6514) * Added Flag to distinguish between gateway and host key to fix issue #6210 Signed-off-by: Erik <Erik.Parra4@gmail.com>
* | | updating paranoid to verify_host_key for net-ssh for 4.2 greater and ↵Vern Burton2018-02-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | updating gemspec. This breaks net-ssh 4.1 and lowers backwards compatibility Signed-off-by: Vern Burton <me@vernburton.com>
* | | fix ohai tests after require_plugin removaltm/fix_ohai_testsThom May2018-02-191-3/+1
|/ / | | | | | | Signed-off-by: Thom May <thom@chef.io>
* | Ignore user SSH key authentication if password is givenChibuikem Amaechi2018-02-011-7/+7
| | | | | | | | Signed-off-by: Chibuikem Amaechi <cramaechi@me.com>
* | Fix "knife ssh" authentication scheme #4342Chibuikem Amaechi2018-02-011-0/+5
| | | | | | | | | | | | | | | | Affects at least knife 12.5.1 When a user uses knife ssh in "password, not key" mode, it fails. Signed-off-by: Chibuikem Amaechi <cramaechi@me.com>
* | Merge pull request #6582 from sarkis/6581-fix-knife-ssh-on-error-behaviorThom May2018-01-292-8/+6
|\ \ | | | | | | use a stricter comparison so knife ssh only fails if --exit-on-error
| * | change to boolean in bootstrap as wellSarkis Varozian2017-11-131-1/+1
| | | | | | | | | | | | Signed-off-by: Sarkis Varozian <svarozian@gmail.com>
| * | refactor on_error optionSarkis Varozian2017-11-111-8/+6
| | | | | | | | | | | | Signed-off-by: Sarkis Varozian <svarozian@gmail.com>
| * | use a stricter comparison here so knife ssh only fails ifSarkis Varozian2017-11-101-1/+1
| | | | | | | | | | | | | | | | | | --exit-on-error Signed-off-by: Sarkis Varozian <svarozian@gmail.com>
* | | Merge pull request #6756 from cramaechi/CHEF-3287Thom May2018-01-291-17/+22
|\ \ \ | | | | | | | | Update error handling for "knife status" #3287
| * | | Update status lineChibuikem Amaechi2018-01-231-22/+22
| | | | | | | | | | | | | | | | Signed-off-by: Chibuikem Amaechi <cramaechi@me.com>
| * | | Update error handling for "knife status" #3287Chibuikem Amaechi2018-01-231-0/+5
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | If a node object's ohai_time value is "nil" then the calculated time-since-last-run value is incorrect because the "nil" gets converted to integer "0". "knife status" should signal an error when a node's ohai_time value is nil. Signed-off-by: Chibuikem Amaechi <cramaechi@me.com>
* | | Added support for setting node policy name and group from knifepiyushawasthi2018-01-231-0/+79
|/ / | | | | | | Signed-off-by: piyushawasthi <piyush.awasthi@msystechnologies.com>
* | Merge pull request #6742 from zanecodes/fix_knife_id_only_formattingThom May2018-01-221-1/+3
|\ \ | | | | | | Prevent knife search --id-only from outputting IDs in the same format…
| * | Change single quotes to double quotes to appease the Style GodsZane Geiger2018-01-151-1/+1
| | | | | | | | | | | | Signed-off-by: Zane Geiger <zane.geiger@relativity.com>
| * | Prevent knife search --id-only from outputting IDs in the same format as an ↵Zane Geiger2018-01-151-1/+3
| | | | | | | | | | | | | | | | | | empty hash Signed-off-by: Zane Geiger <zane.geiger@relativity.com>
* | | Merge pull request #6445 from chef/knife_ssh_identity_fileTim Smith2018-01-221-6/+1
|\ \ \ | | | | | | | | Remove deprecated knife ssh --identity-file option
| * | | Remove deprecated knife ssh --identity-file optionknife_ssh_identity_fileTim Smith2017-09-261-6/+1
| | | | | | | | | | | | | | | | | | | | | | | | This was replaced with --ssh_identity_file Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | Merge pull request #6444 from chef/knife_ssh_csshxThom May2018-01-221-4/+0
|\ \ \ \ | | | | | | | | | | Chef 14: Remove deprecated knife ssh csshx command
| * | | | Remove deprecated knife ssh csshx commandknife_ssh_csshxTim Smith2017-09-261-4/+0
| |/ / / | | | | | | | | | | | | | | | | | | | | This was replaced with knife ssh cssh Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | Merge pull request #6728 from chef/reindexThom May2018-01-221-133/+0
|\ \ \ \ | | | | | | | | | | Remove knife index rebuild command that requires Chef < 11
| * | | | Remove knife index rebuild command that requires Chef < 11reindexTim Smith2018-01-041-133/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | knife index rebuild only works on Chef Server < 11. We no longer support Chef Server 10 and anyone still using 10 should use an older DK or gem install of chef for management. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | Merge pull request #6749 from chef/lcg/rubocop-0.52.1Thom May2018-01-217-14/+14
|\ \ \ \ \ | | | | | | | | | | | | prepping for rubocop 0.52.1
| * | | | | prepping for rubocop 0.52.1lcg/rubocop-0.52.1Lamont Granquist2018-01-177-14/+14
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | Merge pull request #6725 from chef/osc_11Thom May2018-01-185-0/+10
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Add a warning that Chef 11 server support in knife user is deprecated
| * | | | | Add a warning that Chef 11 server support in knife user is deprecatedosc_11Tim Smith2018-01-035-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're currently shipping 2 of each of these plugins and rewriting the user input when we think they actually meant to support Chef 11 server. It's code that really needs to go away at this point. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | | Fix IndentationChibuikem Amaechi2018-01-111-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Chibuikem Amaechi <cramaechi@me.com>
* | | | | | Fix knife status to show seconds when needed #5055Chibuikem Amaechi2018-01-101-2/+6
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a node.save operation is executed and a "knife status" is immediately executed I get "xx minutes" instead of "xx seconds". The code behind 'knife status' isn't designed to have the ability to format the time difference in seconds. To remediate issue, make sure there is a condition in place to format the time difference in seconds when appropriate. Signed-off-by: Chibuikem Amaechi <cramaechi@me.com>
* | | | | Update the knife editor error message to point to the correct documentnew_knife_docTim Smith2018-01-022-2/+2
|/ / / / | | | | | | | | | | | | | | | | | | | | I updated the structure of the knife documentation a while back and we have a dedicated doc for setting up knife now. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | Escape single-quoted strings from the contextAllan Espinosa2017-12-201-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Fixes #6680 Signed-off-by: Allan Espinosa <aespinosa33@bloomberg.net>
* | | | Merge pull request #6552 from anoadragon453/masterThom May2017-12-201-4/+4
|\ \ \ \ | | | | | | | | | | Add an 's' for quantity of 0 cookbooks.
| * | | | Add an 's' for quantity of 0 cookbooks.Andrew (anoa)2017-11-021-4/+4
| | |_|/ | |/| | | | | | | | | | | | | | | | | | Uploaded 0 cookbook. -> Uploaded 0 cookbooks. Obvious fix.
* | | | Merge pull request #6660 from chef/tm/credentialsThom May2017-12-181-36/+12
|\ \ \ \ | | | | | | | | | | implement credential management
| * | | | Generate a credentials fileThom May2017-12-141-36/+12
| | |_|/ | |/| | | | | | | | | | Signed-off-by: Thom May <thom@chef.io>
* | | | Cleanup some YARD warningsyard2Tim Smith2017-12-131-1/+1
|/ / / | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Add custom prefix attribute support to knife sshMal Graty2017-11-281-5/+30
| | | | | | | | | | | | Signed-off-by: Mal Graty <mal.graty@googlemail.com>