562629 - Tweaks to 10.5 and 10.6 slaves

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Nick Thomas [:nthomas] (UTC+12)]

* disable screensaver (currently set to random, some of them use a lot of CPU)
 /Users/cltbld/Library/Preferences/ByHost/com.apple.screensaver.0016cbac694e.plist
* set VNC password to the current password rather than previous one
 /Library/Preferences/com.apple.VNCSettings.txt
* set desktop to flat color (less to transfer when using VNC)
 /Users/cltbld/Library/Preferences/com.apple.desktop.plist
* maybe change the resolution to 1280 x 1024 (current 800 x 600). Not running unit tests here, there's still the alive tests for debug builds
 /Users/cltbld/Library/Preferences/ByHost/com.apple.windowserver.0016cbac694e.plist

Pretty sure we can create the required files on one machine and roll out to the 10.6 pool using puppet. I've made all of those changes on moz2-darwin10-slave18.

This is worth doing soon because it's possible that the screensaver is stealing CPU time and causing bug 562260 and bug 558496.

Comment 1

[Mike Taylor [:bear]]

The issue with fixing anything that lives in the ByHost/ folder is that the filename contains the UUID of the machine

ioreg -rd1 -c IOPlatformExpertDevice | grep -E '(UUID)'

I'll have to research if there is a way in puppet to use this value to create the filename.

Would it be ok if this were some sort of on-startup fix?

Comment 2

[Mike Taylor [:bear]]

ahh - I found that puppetd on osx sends that info as part of the data facter provides:

sp_platform_uuid => 00000000-0000-1000-8000-0016CBAC675D

so all I will need to do is chop that up into usable substrings

Comment 3

[Nick Thomas [:nthomas] (UTC+12)]

At least two of the machines had the same uuid, perhaps because of the way they were cloned ?

Comment 4

[Mike Taylor [:bear]]

(In reply to comment #3)
> At least two of the machines had the same uuid, perhaps because of the way they
> were cloned ?

most likely - which is why this puppet style update i'm working on will be better - the UUID will be accurate for the target host

Comment 5

[Nick Thomas [:nthomas] (UTC+12)]

(In reply to comment #0)
> * disable screensaver (currently set to random, some of them use a lot of CPU)
> 
> /Users/cltbld/Library/Preferences/ByHost/com.apple.screensaver.0016cbac694e.plist

Had any time to work on this bug ? The screen saver issue & VNC password are the most pressing issues to fix. I dropped in on moz2-darwin10-slave04 just now and the screensaver was using ~90% CPU while trying to do a build.

Comment 6

[Mike Taylor [:bear]]

Attachment: change os/osx.pp manifest to force preferences values

Comment 7

[[e-mail] (:bhearsum)]

Comment on attachment 447774 [details] [diff] [review]
change os/osx.pp manifest to force preferences values


>+    # This section contains items that need tweaking post-refimage install

This comment is unnecessary and may end up wrong, we do sync ref images against Puppet sometimes. I'm going to remove it upon landing. The rest looks totally fine.

Comment 8

[[e-mail] (:bhearsum)]

Comment on attachment 447774 [details] [diff] [review]
change os/osx.pp manifest to force preferences values

changeset:   163:b2975b4d88c2

Comment 9

[Mike Taylor [:bear]]

Attachment: tweak to the darwin10 platform_uuid variable set to lowercase the uuid

brings the filename inline with the naming/case style used by Apples own Preferences tool

Comment 10

[[e-mail] (:bhearsum)]

Comment on attachment 447795 [details] [diff] [review]
tweak to the darwin10 platform_uuid variable set to lowercase the uuid

changeset:   166:58130d3bb6dc

Comment 11

[Mike Taylor [:bear]]

checked 10 darwin9 and 10 darwin10 build slaves and the files are being sent when an update is needed.

I forced the update by rm -f com.apple.screensaver.* to allow the screensaver fix to land faster

Comment 12

[Mike Taylor [:bear]]

re-opening to double check that this was also applied to darwin9 slaves

Comment 13

[Mike Taylor [:bear]]

ok, problem with moz2-darwin9-slave43 solved, it was a puppetd not running issue (due to a rogue lockfile) - once cleared updates flowed and screen saver was disabled.

re-closing