| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Original error: https://github.com/chef/chef-apply/pull/9
I thought I fixed this in https://github.com/chef/chef/pull/7223 but it
turns out relative or absolute paths continue to match the URI regular
expression. This means that Windows continues to fail to unpack
policyfile tarballs.
Reversing the order of these checks ensures that real paths do not fall
prey to the URI error.
Signed-off-by: tyler-ball <tball@chef.io>
|
|\
| |
| | |
knife config and a bunch of UX improvements
|
| |\
| | |
| | |
| | | |
Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
|
| |\ \
| | | |
| | | |
| | | | |
Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
the existing $KNIFE_HOME.
Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
|
| |_|/
|/| |
| | |
| | | |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| |/
|/|
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| | |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| |
| |
| |
| | |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|/
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
|
|
| |
Provide the user with feedback if it's invalid or missing.
Signed-off-by: Thom May <thom@chef.io>
|
|
|
|
|
|
|
| |
I'm guessing open-uri doesn't handle local file paths well with `open`?
Either way this fixes the issue.
Signed-off-by: tyler-ball <tyleraball@gmail.com>
|
|
|
|
| |
Signed-off-by: Stuart Preston <stuart@chef.io>
|
|
|
|
| |
Signed-off-by: Thom May <thom@chef.io>
|
|
|
|
|
|
|
| |
also more broadly applies the integration tests to legacy mode which was
the original intent of the chef-solo tests in client_spec.rb here
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
|
|
|
|
| |
without --interval we default to --no-fork, with --interval we default
to running --fork. --once and --daemonize are handled before this code
so they will also DTRT.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
|
|
|
| |
If we find a config file in the exported tarball, load it up. This way
we can use --recipe-url with the direct results of `chef export -a`
Signed-off-by: Thom May <thom@chef.io>
|
| |
|
|
|
|
|
|
| |
closes #4824
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| |
|
| |
|
|
|
|
|
|
|
| |
We claimed we were going to remove this in Chef 13. Let's stop rewriting
ARGs now.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|
|
|
| |
Signed-off-by: Thom May <thom@chef.io>
|
|
|
|
|
|
|
| |
Signed-off-by: Giedrius Rekasius <giedrius.rekasius@gmail.com>
This resolves a bug that caused chef-client service to go into
infinite sleep after handling SIGHUP.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We can't reconfigure in a trap context because we might open files or another
activity that could cause a deadlock. Ruby 2.0+ now prevents this by raising a
ThreadError.
This now runs reconfigure after every daemonized run, but still takes a HUP in
case you need to reconfigure while sleeping.
Fixes #4578
Signed-off-by: Bryan McLellan <btm@loftninjas.org>
|
|
|
|
| |
Signed-off-by: nimisha <nimisha.sharad@msystechnologies.com>
|
|
|
|
| |
Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
|
|
|
|
| |
Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
|
|
|
|
|
|
|
|
|
|
| |
This was probably a bit too aggressive. There are people who pass
logger objects in client.rb directly into the log_location, and
we'd break them, and there's no general solution. We have a couple
of the popular configs that can be passed in as symbols, but not
the general case.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Steven Murawski <steven.murawski@gmail.com>
|
|
|
|
| |
Signed-off-by: Steven Murawski <steven.murawski@gmail.com>
|
|
|
|
| |
Signed-off-by: Thom May <thom@chef.io>
|
|
|
|
|
|
| |
department of redundancy department
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
|
|
|
| |
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>
|