-
Notifications
You must be signed in to change notification settings - Fork 82
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
SmbShare: Configuration drift due to sort ordering of access properties #247
Comments
Great catch @dappleby300 - hopefully is an easy fix. Will get onto it as soon as possible. |
I'll be able to implement the fix in this one once #252 is merged. |
#252 has been merged so I can get onto this one. |
@PlagueHO Sorry to bug you again but wondering whether it'd be possible for you to fix this? Alternatively happy to have a go myself - I'm guessing the new -SortArrayValues parameter should be passed to Test-DscParameterState - anything else you're aware of? |
Hi @Borgquite , I'm happy if you want to have a go at fixing! I'm a bit tied up with Global Azure event today :/ |
Thanks to @Borgquite for fixing this. Please test and verify that it works by using the preview that will be released shortly. |
Details of the scenario you tried and the problem that is occurring
When using the SmbShare resource the access array properties are not sorted so if the order does not match the operating system (which doesn't appear to sort alphabetically), the configuration constantly drifts.
Verbose logs showing the problem
Suggested solution to the issue
Sort both the arrays both from the configuration and in the Get-TargetResource function before performing the compare.
The DSC configuration that is used to reproduce the issue (as detailed as possible)
The operating system the target node is running
Version and build of PowerShell the target node is running
Version of the DSC module that was used ('dev' if using current dev branch)
ComputerManagementDsc , Version: 6.5.0.0
The text was updated successfully, but these errors were encountered: