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)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: armenzg, Assigned: armenzg)
References
Details
(Whiteboard: [buildslaves][puppet][10.5][10.6])
Attachments
(4 files)
(deleted),
patch
|
bhearsum
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
text/plain
|
Details | |
(deleted),
text/plain
|
Details | |
(deleted),
application/octet-stream
|
Details |
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.
Assignee | ||
Comment 1•14 years ago
|
||
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]
Assignee | ||
Comment 2•14 years ago
|
||
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>
Comment 3•14 years ago
|
||
What's the motivation behind removing cscreen? We added it because some machines weren't maintaining screen resolution at reboot....
Assignee | ||
Comment 4•14 years ago
|
||
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 5•14 years ago
|
||
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+
Assignee | ||
Comment 6•14 years ago
|
||
I have deployed the necessary files unto /N/production for both platforms.
I would like to deploy it this week.
Assignee | ||
Comment 7•14 years ago
|
||
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+
Assignee | ||
Comment 8•14 years ago
|
||
We now manage screen resolution and dock settings for Mac *testing* slaves through puppet.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 9•14 years ago
|
||
Assignee | ||
Comment 10•14 years ago
|
||
Assignee | ||
Comment 11•14 years ago
|
||
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•