Closed Bug 1122875 Opened 7 years ago Closed 7 years ago

When getting a stack for an assertion, OS X 10.10 puts up a dialog asking for permission for atos to take control of another process


(Infrastructure & Operations Graveyard :: CIDuty, task)

Not set


(Not tracked)



(Reporter: philor, Assigned: kmoir)




(1 file) is how every debug mochitest-other on 10.10 goes.

We have five assertions ("bad inline size: 'metrics.ISize(lineWM) >= 0", dunno why a caps test hits layout asserts, but that's not the current problem) in caps/tests/mochitest/test_bug995943.xul, then several more failures in mochitest-chrome, then we start running mochitest-a11y, hit a timeout in jsat/test_content_text.html, and take a screenshot.

And that is the current problem, since it shows an OS dialog up over the top of Firefox, saying "atos needs to take control of another process for debugging to continue. Type the name and password of a user in the "Developer Tools" group to allow this."

Unknowable how many of our other 10.10 debug failures are the result of that, but it's almost certainly the cause of mochitest-a11y's destruction, since a11y tests hate trying to run underneath an OS dialog (and thus hate their life, having to run after mochitest-chrome, which is prone to leaving things up like that).
Boy, this is terrible. I'll see if I can dig up anything.
I'm not actually sure where the call to atos happens, I was suspecting but we're passing --symbols-path here so we shouldn't be using that file. I don't want to over-rotate on that, but I am kinda interested. There is this code path if we hit an unhandled ObjC exception:

so it might be that. Regardless, I think we need to add whatever user buildbot runs tests as to the developers group to make this work. This seems like the right answer:
I'll run this command on a 10.10 slave and then retrigger the job on it to see if it resolves the problem. If so, I can add it to our puppet config
Looks like this is fixing the issue, I have a puppet patch to address it, just have to tweak it a bit. Thanks Ted for finding the article on how to set this preference.
Assignee: nobody → kmoir
Attached patch bug1122875.patchSplinter Review
Patch to enable DevToolsSecurity on 10.10 platforms and add cltbld to the _developer group so that debug tests can run 10.10.  Tested in staging.
Attachment #8553780 - Flags: review?(coop)
Attachment #8553780 - Flags: review?(coop) → review+
Comment on attachment 8553780 [details] [diff] [review]

and merged to production
Attachment #8553780 - Flags: checked-in+
Verified that the puppet patch sets the appropriate permissions on a recently puppetized slave
Verified the tests pass on try.
Closed: 7 years ago
Resolution: --- → FIXED
Component: Platform Support → Buildduty
Product: Release Engineering → Infrastructure & Operations
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in before you can comment on or make changes to this bug.