821624 - Support for specifying promptWaitTime for updates

Status: RESOLVED FIXED

(Release Engineering :: Release Automation: Other, defect, P2)

Description

[Nick Thomas [:nthomas] (UTC+12)]

Bug 813322 added a new parameter to control how long the updater waits before showing a dialog to user asking them to restart. Bug 816408 is about adding support to AUSv2. This bug is about making sure we support it in the releases automation.

From a quick discussion with bhearsum this might turn out to be a no-op. We're proposing we handle this like 'actions  silent' for suppressing What's New pages after updates, in that the patcher config is modified by a release engineer if desired. If we use the same parameter name in patcher config, snippets and AUS we might not need any changes in the snippet generator due to
 https://github.com/mozilla/build-tools/blob/master/lib/python/release/updates/snippets.py#L66

Comment 1

[Nick Thomas [:nthomas] (UTC+12)]

Attachment: [tools] Generalise support for schema 2 attributes

This adjusts parseCurrentUpdate to support all the optional attributes, and adds promptWaitTime to the list of them.

Comment 2

[Nick Thomas [:nthomas] (UTC+12)]

I reran the update generation for Firefox 17.0.1 with this and it didn't regress anything. Adding a 
  promptWaitTime   999
to the <current-update> block appended 
  promptWaitTime=999
to every snippet generated.

Comment 3

[[e-mail] (:bhearsum)]

Comment on attachment 693810 [details] [diff] [review]
[tools] Generalise support for schema 2 attributes

Review of attachment 693810 [details] [diff] [review]:
-----------------------------------------------------------------

Looks like it still passes all tests too, looks good!

Comment 4

[Nick Thomas [:nthomas] (UTC+12)]

Comment on attachment 693810 [details] [diff] [review]
[tools] Generalise support for schema 2 attributes

Squashed a couple of trailing spaces on commit:
http://hg.mozilla.org/build/tools/rev/565761bac81f

Comment 5

[Nick Thomas [:nthomas] (UTC+12)]

Basic support is in place with the patch here. For build requests where RelMan ask for a non-default prompt time add a line like
   promptWaitTime  <time in seconds>
to the <current-update> block of the the appropriate patcher config.

Bug 823841 for the convenience of being able to specify the value in the release kickoff app.