Mark Cornmesser [:markco] OOO 2024/04/15 Assignee
	Description • 9 years ago

Complete error: 

Mon Jun 01 09:52:38 -0700 2015 Puppet (err): Failed to set owner to 'S-1-5-21-3861705028-2700629737-4079167078-500': Failed to add access control entry: No more memory is available for security information updates. 
Mon Jun 01 09:52:38 -0700 2015 /Stage[main]/Puppet::Atboot/File[c:/etc/puppetmasters.txt]/owner (err): change from NT AUTHORITY\SYSTEM to B-2008-IX-0178\root failed: Failed to set owner to 'S-1-5-21-3861705028-2700629737-4079167078-500': Failed to add access control entry: No more memory is available for security information updates.

	Dustin J. Mitchell [:dustin] (he/him)
	Comment 2 • 9 years ago

I think you want to exec that on *every* run, right?

	Mark Cornmesser [:markco] OOO 2024/04/15 Assignee
	Comment 3 • 9 years ago

My bad. I thought we decided on one invocation of it.

	Dustin J. Mitchell [:dustin] (he/him)
	Comment 4 • 9 years ago

Oh, I may have missed that decision.  Whichever one you and Q agreed on (attachment 8614875 [details] [diff] [review] and attachment 8614920 [details] [diff] [review] are identical, btw).

	Mark Cornmesser [:markco] OOO 2024/04/15 Assignee
	Comment 6 • 9 years ago

Comment on attachment 8615366 [details] [diff] [review]
BUG1170587_exec.patch

https://hg.mozilla.org/build/puppet/rev/e8c388acdf5e

	Mark Cornmesser [:markco] OOO 2024/04/15 Assignee
	Comment 7 • 9 years ago

Needed to call icacls directly by path to avoid error.

	Dustin J. Mitchell [:dustin] (he/him)
	Comment 8 • 9 years ago

Comment on attachment 8615413 [details] [diff] [review]
BUG1170587_correct_path.patch

Heh, I should have spotted that, sorry.

	Mark Cornmesser [:markco] OOO 2024/04/15 Assignee
	Comment 9 • 9 years ago

Comment on attachment 8615413 [details] [diff] [review]
BUG1170587_correct_path.patch

 https://hg.mozilla.org/build/puppet/rev/9dc9fb513b7d

	Dustin J. Mitchell [:dustin] (he/him)
	Comment 11 • 9 years ago

Comment on attachment 8627878 [details] [diff] [review]
BUG1170587_require.patch

Review of attachment 8627878 [details] [diff] [review]:
-----------------------------------------------------------------

::: modules/puppet/manifests/atboot.pp
@@ +16,4 @@
>          windows: {
>              include dirs::etc
>              $puppetmasters_txt = "${dirs::etc::dir}/puppetmasters.txt"
> +            # Temp work around on to DACLs being appneded. 

Can you fix this typo while landing?

	Mark Cornmesser [:markco] OOO 2024/04/15 Assignee
	Comment 12 • 9 years ago

Comment on attachment 8627878 [details] [diff] [review]
BUG1170587_require.patch

https://hg.mozilla.org/build/puppet/rev/900b448db2a9

	Mark Cornmesser [:markco] OOO 2024/04/15 Assignee
	Comment 13 • 9 years ago

https://tickets.puppetlabs.com/browse/PUP-4802