Closed Bug 1560577 Opened 6 years ago Closed 4 years ago

Ability to trigger a background update check via normandy

Categories

(Firefox :: Normandy Client, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED INACTIVE

People

(Reporter: robert.strong.bugs, Unassigned, Mentored)

Details

bhearsum suggested that Normandy could trigger an update check for chemspills instead of waiting the 12 hours between the client's automatic checks and this could be a good thing for chemspills.

The app update code to trigger this on the client is trivial (see below) though I don't know how difficult it would be to add this capability to normady.

Cc["@mozilla.org/updates/update-service;1"].getService(Ci.nsIApplicationUpdateService).checkForBackgroundUpdates();

Normandy's architecture is extensible enough to make this fairly straight forward. The point of extensibility is an action. You can see a simple example of an action here: https://searchfox.org/mozilla-central/source/toolkit/components/normandy/actions/ConsoleLogAction.jsm

Most of the work will be in figuring out how we want to handle the life time of these kind of triggers, and how we store any needed state. I think this makes sense to do. It may be useful to have someone from the updates team do this work, mostly as a way to spread knowledge about Normandy. I'd be happy to help guide this implementation.

Mentor: mcooper
Type: defect → enhancement
Component: Actions → Normandy Client
Product: Shield → Firefox

I spoke briefly about this with :bhearsum in Whistler. Last I heard we were talking about hooking into Megaphone. Does this replace that? I guess Normandy is nice because it gives us more precise targeting and a two-person-review workflow?

(In reply to Ethan Glasser-Camp (:glasserc) from comment #2)

I spoke briefly about this with :bhearsum in Whistler. Last I heard we were talking about hooking into Megaphone. Does this replace that? I guess Normandy is nice because it gives us more precise targeting and a two-person-review workflow?

Having a two persion review is unnecessary here IMO since it's only triggering code that already runs on a regular schedule in Firefox. If Normandy makes sense for other reasons I'm more than happy to use it though.

I think Robert and I may have got our wires crossed a bit though - it was always my intent to use Megaphone. (I think I talked about Normandy and Kinto when we talked, and may not have expressed my intent well.)

Having a two persion review is unnecessary here IMO since it's only triggering code that already runs on a regular schedule in Firefox. If Normandy makes sense for other reasons I'm more than happy to use it though.

Normandy requires two-person review. We don't have anything (and don't plan to add) to avoid this for specific action types.

Regarding Megaphone, Normandy is/will be hooked up to Megaphone via Remote Settings. However, it is a little round-about. Making something more purpose build might make more sense. Using Normandy might be less work and maintenance though.

I leave it to the people that understand the implemented functionality to decide what makes the most sense.

I don't think this is something we need to pursue anymore, though I'd be happy to collaborate with the releng or updater team if this is still needed.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INACTIVE
You need to log in before you can comment on or make changes to this bug.