Closed Bug 1907120 Opened 7 months ago Closed 2 months ago

Remove the multi-instance service / locking

Categories

(Toolkit :: Application Update, task, P3)

task

Tracking

()

RESOLVED FIXED
135 Branch
Tracking Status
firefox135 --- fixed

People

(Reporter: mpohle, Assigned: bytesized)

References

Details

(Whiteboard: [fidedi])

Attachments

(1 file)

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

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.

Blocks: 1907124
Whiteboard: [fidedi]
Type: enhancement → task
Priority: -- → P3

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.

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!
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

Backed out for causing build bustages

Backout link

Push with failures

Failure log

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
Resolution: --- → FIXED
Target Milestone: --- → 135 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: