| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Signed-off-by: Marc A. Paradise <marc.paradise@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This was discussed in the last Chef Triage meeting.
In the great move to trace logging, which is great, idempotency check
logs moved to trace, which defeats the purpose. The goal is to keep
"internal chef" logging in trace so that when users are trying to figure
out why their cookbook isn't acting as expected they can look at debug
without having to see all the other cruft. But "here's why I didn't
install the package you said to install" is table-steaks "I'm trying to
debug my cookbook."
This moves those logs back to debug. There were a variety of cases where
logging was pretty nonstandard in other ways, but to keep this PR
scoped, I didn't change those.
I did fix a few related logging issues:
* A few warns that should have been debug in windows_task
* A few places where there was basically no logging whatsoever, I put a
few of the "I did X" logs to 'debug' (from trace). It coudl be argued
some of those should be 'info', but that's a bigger change that I
didn't want to put into this PR
Signed-off-by: Phil Dibowitz <phil@ipom.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
`apt_package` never even looks at `new_resource.version` when deciding
what version to install. That's silly.
This fixes that. It requires the version be exact (as the `yum_package`
did for most of Chef's history), and if you specify a non-existent
version, it'll raise an exception.
The unit tests for this provider are a bit of a mess, so I kinda stuck
in the tests where I could.
```
[2021-02-07T15:59:35-08:00] TRACE: apt_package[hub] installed version for hub is 2.14.2-1vcrs.1
[2021-02-07T15:59:35-08:00] TRACE: apt_package[hub] candidate version for hub is 2.14.2-1vcrs.1
[2021-02-07T15:59:35-08:00] TRACE: apt_package[hub] hub 2.14.2-1vcrs.1 needs updating to 2.7.0~ds1-1vcrs.1
[2021-02-07T15:59:38-08:00] INFO: apt_package[hub] installed hub at 2.7.0~ds1-1vcrs.1
```
Signed-off-by: Phil Dibowitz <phil@ipom.com>
|
|
|
|
|
|
|
| |
Also suppresses a version comparison that normally fails when it is
called on the dnf provider which was ignorable.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|\
| |
| |
| | |
Signed-off-by: Daniel Schauenberg <d@unwiredcouch.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
This commit includes a trimmed version of rubygems.org API output.
To recreate this file:
* Download the file from rubygems.org:
curl "https://rubygems.org/latest_specs.4.8.gz"
* Read the file, uncompress it, filter for the data you want, and
reverse the process:
File.write("latest_specs.4.8.gz", Gem::Util.gzip(Marshal.dump(Marshal.load(Gem::Util.gunzip(File.read("latest_specs.4.8.gz"))).select {|m| m.first == "sexp_processor" })))
|
| |\
| | |
| | | |
Chef 17: Assume Rubygems 1.8 in the rubygems provider / specs
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Rubygems 1.8 came out May 4, 2011 and we're still checking for rubygems
older than that.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| | |
| | |
| | |
| | | |
Signed-off-by: mwrock <matt@mattwrock.com>
|
| | |
| | |
| | |
| | | |
Signed-off-by: mwrock <matt@mattwrock.com>
|
| |/
| |
| |
| |
| |
| | |
Resolve the issues that pete pointed out plus a few others
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
| |
| |
| |
| |
| | |
It is slow but it does catch real bugs
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| |\
| | |
| | | |
Avoid requiring spec_helper more than once
|
| | |
| | |
| | |
| | | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| |/
| |
| |
| | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| |
| |
| |
| | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| |
| |
| |
| | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| |
| |
| |
| |
| |
| | |
changed.
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| |
| |
| |
| | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| |
| |
| |
| | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| |
| |
| |
| | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| |
| |
| |
| | |
Signed-off-by: Phil Dibowitz <phil@ipom.com>
|
| |
| |
| |
| |
| |
| | |
powershell gallery
Signed-off-by: sanga17 <sausekar@msystechnologies.com>
|
| |
| |
| |
| | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| |
| |
| |
| |
| |
| | |
Autocorrected with chefstyle after enabling that cop. It was very hard to read some complex autocorrected blocks w/o this.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
| |
| |
| | |
Signed-off-by: Kapil Chouhan <kapil.chouhan@msystechnologies.com>
|
| |
| |
| |
| | |
Signed-off-by: Ingo Becker <ingo@orgizm.net>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Tiny tweak to grab the first item so we actually get the hash back in the error handling.
Given:
```ruby
package %w{curl totally_junk_pkg} do
action :upgrade
end
```
```
* No candidate version available for totally_junk_pkg
================================================================================
Error executing action `upgrade` on resource 'homebrew_package[curl, totally_junk_pkg]'
================================================================================
Chef::Exceptions::Package
-------------------------
No candidate version available for totally_junk_pkg
```
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
| |
| |
| | |
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
| |
| |
| | |
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
| |
| |
| | |
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
| |
| |
| | |
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
| |
| |
| | |
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
| |
| |
| | |
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
| |
| |
| | |
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
| |
| |
| | |
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
| |
| |
| | |
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
| |
| |
| |
| |
| |
| |
| | |
Legally incredibly dubious, particularly since we don't follow it
strictly as policy, and we have git history instead, which does it right.
This is just a waste of time and a cargo cult.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| |
| |
| |
| | |
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
| |
| |
| |
| | |
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
| |
| |
| |
| | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| |
| |
| |
| | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| |
| |
| |
| | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| |
| |
| |
| | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|
| |
| |
| |
| | |
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
|