886360 - Make sure that the userShutdown mode correctly handles all possible scenarios

Status: RESOLVED WONTFIX

(Testing Graveyard :: Mozmill, defect)

Description

[Henrik Skupin [:whimboo][⌚️UTC+1]]

With bug 865690 we are fixing mostly all possible cases for quit/restart scenarios. But this mentioned bug will only care about the usual restarts, which are triggered by Mozmill itself.

As we have decided in our automation development meeting today, we will not add full support for the userShutdown feature in Mozmill 2.0, but will move this out to 2.1. It's way too complicated to cover all the possible paths how a quit/restart can be triggered. And there is also still a risk of race conditions given that not all shutdown logic will trigger the quit-application-requested observer notifications.

Comment 1

[Andrei Eftimie]

Attachment: 0001-Bug-886360-Disabled-userShutdown-mutt-tests.-r-hskup.patch

Lets disable affected mutt tests for now as they fail.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Landed skip patch:
https://github.com/mozilla/mozmill/commit/c6ecca8d20d04945b2a7b0a405749460266a5742 (master)
https://github.com/mozilla/mozmill/commit/ad8e7622909d76c3aba5a1c9e54c748942fbe35f (hotfix-2.0)

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+1]]

As seen on bug 959551 comment 17 the reason why user shutdown tests are not working could come from another kind of zombie process. There might be another case which needs to be fixed.

For now we are blocked on bug 959551.

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Attachment: disable python user shutdown tests v1

This disables the remaining user shutdown tests.

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Landed patch to disable Python user shutdown tests:
https://github.com/mozilla/mozmill/commit/d4558dffae8fb97f9ef8888e5d556463817ddcb2 (master)
https://github.com/mozilla/mozmill/commit/a95511744826f2ee88f1ea6a1f27f3e7405b484b (hotfix-2.0)

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+1]]

This will most likely not make version 2.1 given that we have to release it ASAP. But someone might want to check how the latest changes helped us here.

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Too complex and not appropriate for the last hotfix branch of Mozmill before it reaches its end of life. Marking as WONTFIX.