Skip to content

Fix icacls permission grant error - #550

Draft
Dany9966 wants to merge 1 commit into
cloudbase:mainfrom
Dany9966:windows-user-permissions-fix
Draft

Dany9966 wants to merge 1 commit into
cloudbase:mainfrom
Dany9966:windows-user-permissions-fix

Conversation

@Dany9966

Copy link
Copy Markdown
Contributor

Fix an issue where the permission grant would fail when migrating hardened Windows Server images. The permission grant was initially added when using Nano Server as the minion image, which needed these extra permissions in order for DISM to write driver directories under FileRepository. This can fail either way if the migrated image is hardened, so best chance we have at success is to not fail on the icacls command and attempt installation anyway.
NOTE: DISM only needs Modify permissions when installing drivers, while the permission grant command needs Full Control in order to change permissions, so there's a great chance that DISM installation would work anyway in a standard Server minion.

Fix an issue where the permission grant would fail when migrating
hardened Windows Server images. The permission grant was initially added
when using Nano Server as the minion image, which needed these extra
permissions in order for DISM to write driver directories under
`FileRepository`. This can fail either way if the migrated image is
hardened, so best chance we have at success is to not fail on the
`icacls` command and attempt installation anyway.
NOTE: DISM only needs Modify permissions when installing drivers, while
the permission grant command needs Full Control in order to change
permissions, so there's a great chance that DISM installation would work
anyway in a standard Server minion.
@Dany9966
Dany9966 marked this pull request as draft October 23, 2026 13:12
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

Successfully merging this pull request may close these issues.

1 participant