Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Default owner of 'root' fails on Windows #168

Closed
sgtcoolguy opened this issue Nov 17, 2016 · 0 comments
Closed

Default owner of 'root' fails on Windows #168

sgtcoolguy opened this issue Nov 17, 2016 · 0 comments

Comments

@sgtcoolguy
Copy link
Contributor

Cookbook version

2.0.2

Chef-client version

12.16.42

Platform Details

Windows 10

Scenario:

Attempting to use ark to grab a 7z file (https://sourceforge.net/projects/boost/files/boost/1.62.0/boost_1_62_0.7z) and extract it to C:\Program Files(x86)\boost

Steps to Reproduce:

ark 'boost' do
  url "https://sourceforge.net/projects/boost/files/boost/1.62.0/boost_1_62_0.7z"
  extension '7z'
  version '1.62.0'
  checksum 'b91c2cda8bee73ea613130e19e72c9589e9ef0357c4c5cc5f7523de82cce11f7'
  win_install_dir 'C:\\Program Files (x86)\\boost'
end

Expected Result:

I expect the file to get downloaded, the directory to get created, the content extracted inside.

Actual Result:

Once I've applied a number of patches for various other issues: #166, #167, #161, #165 - it still fails because I didn't override the default setting for owner ('root'). On Windows systems, there is no 'root', and the cookbook should default to a proper equivalent like root_owner here: https://github.com/chef-cookbooks/chef-client/blob/master/libraries/helpers.rb#L37

@tas50 tas50 added the Type: Bug label Dec 6, 2016
@tas50 tas50 closed this as completed in 451cc72 Dec 6, 2016
tas50 added a commit that referenced this issue Dec 6, 2016
Fix #168 Default owner of 'root' fails on Windows
soylentseth pushed a commit to GannettDigital/chef-ark that referenced this issue Oct 4, 2018
Deleting the clusterIP field when the service should be headless will
cause it to be assigned a new IP on restore; instead it should retain
the headless state after restoration.

Fixes sous-chefs#168

Signed-off-by: Nolan Brubaker <[email protected]>
(cherry picked from commit d87e8ee)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants