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

windows_share is not idempotent when using local groups #9242

Open
xorima opened this issue Jan 20, 2020 · 0 comments
Open

windows_share is not idempotent when using local groups #9242

xorima opened this issue Jan 20, 2020 · 0 comments

Comments

@xorima
Copy link
Contributor

xorima commented Jan 20, 2020

Description

The resource windows_share is not idempotent when using local groups

Chef Version

14.12.9

Platform Version

Windows 2016

Replication Case

directory 'C:\foo' do
  recursive true
  rights :modify, "#{node['hostname']}\\mygroup"
end

windows_share 'foo' do
  path            'C:\foo'
  read_users      "#{node['hostname']}\\mygroup"
  change_users    "#{node['hostname']}\\mygroup"
end

Client Output

* directory[C:\foo] action create (up to date)
         * windows_share[foo] action create
           - update foo
           -   set change_users to ["WIN-4EVKS4M49GI\\mygroup"] (was ["mygroup"])
           -   set read_users   to ["WIN-4EVKS4M49GI\\mygroup"] (was ["mygroup"])

It appears that the return is bringing back just the group name where the set has to have the server name for local groups to work.

@xorima xorima added the Status: Untriaged An issue that has yet to be triaged. label Jan 20, 2020
@tas50 tas50 added the Focus: Desktop Issues that impact the Chef Desktop offering label Aug 3, 2021
@sanga1794 sanga1794 removed their assignment Jan 14, 2022
@PrajaktaPurohit PrajaktaPurohit added Backport: 16 New features and bug fixes should be backported to older branches Backport: 17 and removed Backport: 16 New features and bug fixes should be backported to older branches Backport: 17 labels May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants