Closed Bug 926636 Opened 7 years ago Closed 6 years ago

No longer preventing apps from being restored on 10.7/10.8 startup

Categories

(Infrastructure & Operations :: RelOps: Puppet, task)

x86_64
macOS
task
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: philor, Assigned: Callek)

Details

Attachments

(1 file)

<philor>: did we do something to overwrite what was supposed to keep iCal from being perpetually open on 10.7 test slaves?
<philor>: "what" being https://bug843545.bugzilla.mozilla.org/attachment.cgi?id=717277
<Callek|buildduty>: philor: hrm yes yes we did
<Callek|buildduty>: philor: can you file a bug on that in Infra&Ops::Relops: Puppet ?
<Callek|buildduty>: LoginwindowLaunchesRelaunchApps is the flag we want
Should I just assume this isn't going to be fixed, and disable the test that opens iCal, and then, um, get every 10.7 test slave reimaged I guess?
We should do this.  Callek, can you either write the patch or give me the details I'd need to do so?
Assignee: relops → bugspam.Callek
Solution is based on what we did in old-puppet and http://apple.stackexchange.com/questions/24783/uncheck-reopen-windows-when-logging-back-in-by-default
Attachment #8356190 - Flags: review?(dustin)
Comment on attachment 8356190 [details] [diff] [review]
[puppet] disable it

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

::: modules/disableservices/manifests/common.pp
@@ +110,5 @@
> +	        'disable-relaunch-apps':
> +		    domain => "/Library/Preferences/com.apple.loginwindow",
> +                    key => "LoginwindowLaunchesRelaunchApps",
> +                    value => "0",
> +                    require => Class['users::builder'];

This require isn't, um, required.  I suspect the one above is not either.
Attachment #8356190 - Flags: review?(dustin) → review+
I left the requires in (for now) -- I felt they didn't hurt at least.

http://hg.mozilla.org/build/puppet/rev/18afa2bef4a2
+ merge (http://hg.mozilla.org/build/puppet/rev/83ca0e396389)
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.