Closed
Bug 1907120
Opened 7 months ago
Closed 2 months ago
Remove the multi-instance service / locking
Categories
(Toolkit :: Application Update, task, P3)
Toolkit
Application Update
Tracking
()
RESOLVED
FIXED
135 Branch
Tracking | Status | |
---|---|---|
firefox135 | --- | fixed |
People
(Reporter: mpohle, Assigned: bytesized)
References
Details
(Whiteboard: [fidedi])
Attachments
(1 file)
Bug 1907120 - Remove other multi-instance checks now that we check at startup r=nrishel!,nalexander!
48 bytes,
text/x-phabricator-request
|
Details | Review |
We want to remove all of in-product (i.e, in the Update Service) multi-instance lock checking and want to replace it with a new update state to delay the installation of updates, a lot like it was described in the last paragraph of this comment
Assignee | ||
Comment 1•7 months ago
|
||
To be clear, we need to keep the multi instance lock itself around because we are going to use it at startup to know if other instances are running. But, yeah, Update Service shouldn't need it anymore.
Reporter | ||
Updated•7 months ago
|
Whiteboard: [fidedi]
Updated•7 months ago
|
Assignee | ||
Updated•7 months ago
|
Type: enhancement → task
Priority: -- → P3
Assignee | ||
Comment 2•3 months ago
|
||
This does keep the checks in the interactive update interface, since we don't want to to show the user the "Restart to Update" button at the end if restarting will not update.
Updated•3 months ago
|
Attachment #9434103 -
Attachment description: WIP: Bug 1907120 - Remove other multi-instance checks now that we check at startup r=nishel!,nalexander! → WIP: Bug 1907120 - Remove other multi-instance checks now that we check at startup r=nrishel!,nalexander!
Updated•3 months ago
|
Assignee: nobody → bytesized
Attachment #9434103 -
Attachment description: WIP: Bug 1907120 - Remove other multi-instance checks now that we check at startup r=nrishel!,nalexander! → Bug 1907120 - Remove other multi-instance checks now that we check at startup r=nrishel!,nalexander!
Status: NEW → ASSIGNED
Pushed by rsteuber@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/386145953b08
Remove other multi-instance checks now that we check at startup r=nrishel,nalexander,desktop-theme-reviewers,application-update-reviewers,dao
Comment 4•2 months ago
|
||
Pushed by rsteuber@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2a19a01424f1
Remove other multi-instance checks now that we check at startup r=nrishel,nalexander,desktop-theme-reviewers,application-update-reviewers,dao
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
status-firefox135:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 135 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•