| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
ensure that trace level is selectable if necessary via -VVV
Pass the correct object to net/ssh
Signed-off-by: Thom May <thom@chef.io>
|
|
|
|
| |
Signed-off-by: Thom May <thom@chef.io>
|
|
|
|
| |
Signed-off-by: Chibuikem Amaechi <cramaechi@me.com>
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
| |
Signed-off-by: Chibuikem Amaechi <cramaechi@me.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
| |
Fixes #6680
Signed-off-by: Allan Espinosa <aespinosa33@bloomberg.net>
|
|
|
|
|
|
| |
We're already doing this same thing in the other location where we throw the editor error.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|
|
|
|
|
|
|
|
|
|
| |
Fixes https://github.com/chef/chef/issues/6277
Added `node['platform']` and `node['platform_version']` attributes in
the unit tests to trigger a "can't modify frozen String" error.
Fixed by doing shallow copy of `node['platform']` on assignment.
Signed-off-by: Jeremy J. Miller <jm@chef.io>
|
|
|
|
|
|
|
|
| |
This should help bootstrapping with policies
Fixes: #5353
Signed-off-by: Thom May <thom@chef.io>
|
|
|
|
|
|
| |
forgot to clean this up
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Thom May <thom@chef.io>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When a node is bootstrapped in FIPS mode and then tries to run a
chef-client run it gets the following error:
```
STDERR: /opt/chefdk/embedded/lib/ruby/gems/2.3.0/gems/chef-config-12.18.31/lib/chef-config/workstation_config_loader.rb:163:in `rescue in apply_config': You have an error in your config file /var/opt/delivery/workspace/.chef/knife.rb (ChefConfig::ConfigurationError)
NameError: uninitialized constant Chef
/var/opt/delivery/workspace/.chef/knife.rb:2:in `eval'
/var/opt/delivery/workspace/.chef/knife.rb:2:in `eval'
/var/opt/delivery/workspace/.chef/knife.rb:2:in `from_string'
```
Signed-off-by: tyler-ball <tyleraball@gmail.com>
|
|
|
|
|
|
| |
department of redundancy department
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|\
| |
| |
| |
| | |
MsysTechnologiesllc/dheeraj/configurable_log_details
Knife Bootstrap: Passing config_log_level and config_log_location from config.rb
|
| |
| |
| |
| | |
Signed-off-by: dheerajd-msys <dheeraj.dubey@msystechnologies.com>
|
| |
| |
| |
| | |
Signed-off-by: dheerajd-msys <dheeraj.dubey@msystechnologies.com>
|
| |
| |
| |
| |
| |
| | |
specs failure
Signed-off-by: dheerajd-msys <dheeraj.dubey@msystechnologies.com>
|
|/
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For the commands that allow you to filter which attributes it shows you
using the `--attribute` flag, add the `--field-separator` flag to allow
the user to customize which character(s) they want to use to deliniate the
attribute hierarchy.
```shell
knife node show NODE -S: -a "packages:Chef Client v12.12.15"
```
Signed-off-by: Tom Duffield <tom@chef.io>
|
|
|
|
|
|
|
|
| |
Make all the Knife subcommands that include
`Chef::Knife::Core::AttributeReturnOption`
consistent in how they handle attributes.
Signed-off-by: Tom Duffield <tom@chef.io>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| |
|
|\
| |
| | |
Add bootstrap proxy authentication support.
|
| | |
|
| |
| |
| | |
We want client_d, not cliend_d
|
|\ \
| | |
| | | |
Improve wording in Knife
|
| | |
| | |
| | |
| | |
| | | |
Double spacing is for typewriters. Knife doesn't run on typewriters
http://www.writersdigest.com/online-editor/how-many-spaces-after-a-period
|
|\ \ \
| | | |
| | | | |
fix some UselessAssignment cases
|
| | | | |
|
|/ / /
| | |
| | |
| | |
| | | |
this is part of our informal style guide, lets make it formal since
clearly its not getting followed very well.
|
|/ /
| |
| |
| |
| | |
Removes all remaining references to the deprecated escape_glob
helper routine.
|
|\ \
| | |
| | | |
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
|
| | | |
|
| | |
| | |
| | |
| | | |
error message when plugin files are missing from the cache
|
| | |
| | |
| | |
| | | |
instead of using the hash file
|
|\ \ \
| | | |
| | | | |
Have client.rb verify that FIPS mode can be enforced
|
| |/ / |
|
|/ /
| |
| |
| |
| | |
When knife finds a client.d/ directory, it will upload
all files nested under that directory.
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
This just codifies the behaviour we're actually using - that we're
passing a json string and expecting a hash back.
Also adds a deprecation warning to the use of Chef::JSONCompat.from_json
|
| |
| |
| |
| | |
this looks nicer.
|
| |
| |
| |
| |
| | |
this is from the same ruleset that we had, but the new code catches more
conditions.
|