Oh, didn't read far enough down --- looks like bug 600899 was indirectly breaking 10.6. Now that that's fixed, anything else in the way?
Priority: P2 → --
Whiteboard: [10.6][screen resolution]
8 years ago
We should set the desktop background to something plain at the same time, eg a solid color. Loading the 10.5 machines at high res with the default background takes simply ages from the bottom of the world.
Armen, this has become a priority again since we've disabled OpenGL on 10.5. As it is, we don't have any OS X OpenGL coverage. REFTEST INFO | WARNING: USE_WIDGET_LAYERS disabled REFTEST INFO | drawWindow flags = DRAWWINDOW_DRAW_CARET | DRAWWINDOW_DRAW_VIEW; window size = 801,998; test browser size = 800,1000 (At minimum we need the dock moved/autohidden, I think.)
Rob proposed specially-enabling GL for 10.5 on tinderbox as a workaround. Not sure what that would entail.
joe we need approval fro drivers. Would Tuesday/Wednesday work? What has higher priority this or DirectX/Drivers deployment? The dock is hidden at the bottom. Changing the screen resolution and moving the dock to the right will take the following: - deploy a new cscreen.resize.plist/com.apple.windowserver.plist on staging (create it on the ref machine) - check results are good - deploy cscreen.resize.plist/com.apple.windowserver.plist on production (no puppet changes are needed ) - make sure that file shows in all 3 different puppet locations This will affect talos numbers for sure.  http://production-puppet.build.mozilla.org/production/darwin10-i386/test/Library/Preferences/com.apple.windowserver.plist  http://production-puppet.build.mozilla.org/production/darwin10-i386/test/Library/LaunchAgents/cscreen.resize.plist  http://hg.mozilla.org/build/puppet-manifests/file/6ff0a47aa738/os/talos_osx.pp#l54
Assignee: nobody → armenzg
Priority: P5 → P3
Created attachment 505944 [details] [diff] [review] cscreen.plist - change resolution to 1600x1200 on 10.6 machines How does the change affect the reftest runs? * Before the change: REFTEST INFO | WARNING: USE_WIDGET_LAYERS disabled REFTEST INFO | drawWindow flags = DRAWWINDOW_DRAW_CARET | DRAWWINDOW_DRAW_VIEW; window size = 801,998; test browser size = 800,1000 * After the change: REFTEST INFO | drawWindow flags = DRAWWINDOW_DRAW_CARET | DRAWWINDOW_DRAW_VIEW | DRAWWINDOW_USE_WIDGET_LAYERS; window size = 801,1022; test browser size = 800,1000 I incorrectly mentioned com.apple.windowserver.plist but it is ~/Library/Preferences/com.apple.dock.plist that takes care of dock changes. This currently not managed by puppet. I tried to get it to work but I got puppet errors. 3.6.14's GO to build is imminent and I can't debug the desktop colour change. Let's focus only on the screen resolution which is what we need. Here is what I am testing: [root@staging-puppet Preferences]# md5sum /N/staging/darwin9-i386/test/Library/LaunchAgents/cscreen.resize.plist f7f260e7c8e7d24a889e7198c1709bca /N/staging/darwin9-i386/test/Library/LaunchAgents/cscreen.resize.plist
Attachment #505944 - Flags: review?(coop)
Comment on attachment 505944 [details] [diff] [review] cscreen.plist - change resolution to 1600x1200 on 10.6 machines Fix looks fine, but isn't this file in VCS somewhere?
Attachment #505944 - Flags: review?(coop) → review+
Not that I know of. We only have it in here AFAIK: http://production-puppet.build.mozilla.org/production/darwin10-i386/test/Library/LaunchAgents/cscreen.resize.plist TODO figure out where to store it.
It should be in a module in puppet-manifests.
Created attachment 506772 [details] [diff] [review] screen resolution module dustin is this correct? Right now it happens that both Leopard and Snow Leopard machines share the same cscreen and cscreen.resize.plist. How could I differentiate both if I needed to?
I checked results on staging and we are ready to go tomorrow. If there is enough time I could test the patch I just attached and move to the modules style.
Status: NEW → ASSIGNED
Priority: P3 → P2
Comment on attachment 506772 [details] [diff] [review] screen resolution module I'd like to have modules named after their task, rather than the slaves that should be assigned to them. So this should be 'screensize' or, perhaps more generally, 'gui' (in which case we can add things like autologin, background, etc. to it) A UserName is redundant (and will cause a warning) in a LaunchAgent plist, since it always runs as the console user (cltbld in this case). You can distinguish the plists in whatever way makes sense. Personally, I would probably insert the relevant info in the middle, so that the files appear together, and have the same filename extension. Note that plists are usually named with a reversed DNS name. So IMHO the best would be puppet://gui/org.mozilla.build.cscreen-leopard.plist puppet://gui/org.mozilla.build.cscreen-snow.plist This would come with a corresponding change to the Label key in the plist to 'org.mozilla.build.cscreen'. also, note that you can use templates if the plists are only slightly different: http://hg.mozilla.org/build/puppet-manifests/file/6ff0a47aa738/modules/buildslave/templates/org.mozilla.build.buildslave.plist.erb
FTP the new screen resolution was turned live at 8AM PDT on Wednesday January 26th. The reference machine has been updated with the new /Library/LaunchAgents/cscreen.resize.plist, it has been rebooted, have verified it through VNC and have been requested to be refreshed in bug 617105. The documentation has been updated to track the new screen resolution: https://wiki.mozilla.org/ReferencePlatforms/Test/SnowLeopard#Manage_dock_and_screen_resolution I have to debug talos-r3-snow-014 on bug 629598. We will keep moving cscreen.resize.plist to "modules" on another bug. I will file another bug to change the background colour to something plain. AFAIK there is nothing left on this bug to be done beside fixing the dependent bugs.
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Depends on: 617105
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.