Closed Bug 590197 Opened 14 years ago Closed 14 years ago

Manage screen resolution and dock through puppet for Mac testing slaves

Categories

(Release Engineering :: General, defect, P2)

x86
macOS
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: armenzg, Assigned: armenzg)

References

Details

(Whiteboard: [buildslaves][puppet][10.5][10.6])

Attachments

(4 files)

This can be done by doing this:
> defaults write com.apple.dock autohide -int 1

Currently 10.5 machines have it disabled but not the 10.6 machines.
We should manage both through puppet.
Blocks: 580410
I have setup the 10.5 and 10.6 ref machines to place the dock on the right and I have placed the plist files that managed screen resolution and dock for a mac on puppet's staging server. I will be writing and testing the puppet patch soon.

10.5
http://staging-puppet.build.mozilla.org/staging/darwin9-i386/test/Users/cltbld/Library/Preferences/com.apple.dock.plist
http://staging-puppet.build.mozilla.org/staging/darwin9-i386/test/Library/Preferences/com.apple.windowserver.plist

10.6
http://staging-puppet.build.mozilla.org/staging/darwin10-i386/test/Users/cltbld/Library/Preferences/com.apple.dock.plist
http://staging-puppet.build.mozilla.org/staging/darwin10-i386/test/Library/Preferences/com.apple.windowserver.plist
Status: NEW → ASSIGNED
Summary: Disable dock size for Rev 3 testing machines through puppet → Manage screen resolution and dock through puppet for Mac testing slaves
Whiteboard: [buildslaves][puppet] → [buildslaves][puppet][10.5][10.6]
The only available screen resolutions on the ref machines above 1280x1024 is 1680x1050. I wonder why the slaves cloned off the ref machines offer more than 20 different screen resolution instead of the 4 that the ref machines offer.

I've also noticed that even if I change the resolution cscreen [1] sets it back to 1280x1024. cscreen is not anymore maintained and hard to find on the internet (http://hintsforums.macworld.com/showthread.php?t=59575).
I vote for getting rid of this dependency and use com.apple.windowserver.plist if that approach works.

Let's assume for now that we are not changing the screen resolution and let's not complicate our life by discussing if we should or not remove cscreen.

talos-r3-leopard-002:~ cltbld$ cat /Library/LaunchAgents/cscreen.resize.plist 
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN"
"http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
        <key>EnvironmentVariables</key>
        <dict>
                <key>PATH</key>
               
<string>/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin:/usr/X11/bin</string>
        </dict>
        <key>Label</key>
        <string>cscreen.resize</string>
        <key>ProgramArguments</key>
        <array>
                <string>/Users/cltbld/cscreen</string>
                <string>-x</string>
                <string>1280</string>
                <string>-y</string>
                <string>1024</string>
                <string>-r</string>
                <string>60</string>
                <string>-d</string>
                <string>32</string>
                <string>-f</string>
        </array>
        <key>RunAtLoad</key>
        <true/>
        <key>UserName</key>
        <string>cltbld</string>
        <key>WorkingDirectory</key>
        <string>/Users/cltbld/</string>
</dict>
</plist>
What's the motivation behind removing cscreen? We added it because some machines weren't maintaining screen resolution at reboot....
The plist for windoserver is not respected by OSX [1] so let's manage cscreen and its launching agent through puppet.

[1] I validated what this user was facing:
http://www.insanelymac.com/forum/lofiversion/index.php/t151236.html

For further reading wrt to csreen:
http://www.insanelymac.com/forum/lofiversion/index.php/t26845.html

I think we might be able to do it through boot.plist but I am not going to go that way if cscreen does that we need:
http://www.insanelymac.com/forum/lofiversion/index.php/t5513.html
Attachment #470502 - Flags: review?(bhearsum)
Comment on attachment 470502 [details] [diff] [review]
manage dock settings through plist and screen resolution through cscreen

Looks good to me
Attachment #470502 - Flags: review?(bhearsum) → review+
Blocks: 591055
I have deployed the necessary files unto /N/production for both platforms.
I would like to deploy it this week.
Comment on attachment 470502 [details] [diff] [review]
manage dock settings through plist and screen resolution through cscreen

http://hg.mozilla.org/build/puppet-manifests/rev/04612be9bfeb
and deployed.
Tested 2 slaves to make sure that they sync properly.
Attachment #470502 - Flags: checked-in+
We now manage screen resolution and dock settings for Mac *testing* slaves through puppet.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Attached file com.apple.dock.plist
Attached file cscreen.resize.plist
Attached file cscreen
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: