chef / chef-apply

The ad-hoc execution tool for the Chef ecosystem.
https://www.chef.sh/
Apache License 2.0
14 stars 14 forks source link

[SHACK-290] Set minimum Chef 13 version to be 13.10.4 (fix policyfile tarball use on Windows) #35

Closed tyler-ball closed 6 years ago

tyler-ball commented 6 years ago

Description

Fixes https://github.com/chef/chef-apply/issues/22

Chef suffered from a variety of errors when trying to run with Tarball support. On Windows the URI file unpacker fails with a GZip error. If we specify relative paths on Chef 13 it thinks the files are at the root of the directory.

We fix these by specifying full paths, and setting the minimum chef versions to ones that include support for: https://github.com/chef/chef/pull/7522 https://github.com/chef/chef/pull/7523

NOTE Astute observers will notice that the minimum Chef 14 version on Windows (14.4.22) is not yet released. This is planned for next week. I did this because it kept the logic simpler and more straightforward. Otherwise I would have to make another PR next week to fix some complicated logic I added today. Windows is such a small target base, and it already does not work, so this is a temporary bug that will automatically be fixed when Chef 14.4 is released next week.

Issues Resolved

Fixes https://github.com/chef/chef-apply/issues/22 SHACK-290

Check List

Signed-off-by: tyler-ball tball@chef.io