Closed Bug 1221827 Opened 9 years ago Closed 7 years ago

Send upgrade notice to users not running bundle2

Categories

(Developer Services :: Mercurial: hg.mozilla.org, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: gps, Assigned: gps)

References

Details

Attachments

(1 file)

Users are encouraged to run the latest stable Mercurial release. The lack of bundle2 on push means they are running 3.4 or older. We should nudge pushers on old versions to upgrade to a modern Mercurial version.
hghooks: advertise mercurial upgrade to non-bundle2 clients (bug 1221827); r?dminor

Mercurial 3.5 and newer support a new wire protocol exchange format
called "bundle2." Not using bundle2 almost certainly means the client is
on an old version of Mercurial.

In an attempt to nudge people into using a modern version of Mercurial,
this commit adds a hook that will print a message to non-bundle2 users
during push encouraging them to upgrade. The message is purely advisory.
Attachment #8683406 - Flags: review?(dminor)
Attachment #8683406 - Flags: review?(dminor) → review+
Comment on attachment 8683406 [details]
MozReview Request: hghooks: advertise mercurial upgrade to non-bundle2 clients (bug 1221827); r?dminor

https://reviewboard.mozilla.org/r/24315/#review21877

::: hghooks/mozhghooks/advertise_upgrade.py:11
(Diff revision 1)
> +        ui.write('upgrade instructions are at the following URL\n')

I'd put a ':' at the end of this sentence

::: hghooks/mozhghooks/advertise_upgrade.py:13
(Diff revision 1)
> +        ui.write('you will see this message on every push until you upgrade\n')

This sounds a little punitive to me, but I'll leave it to your discretion if you want to keep this sentence or let them figure it out for themselves.
https://hg.mozilla.org/hgcustom/version-control-tools/rev/f2a7fa602ab4eef4b5ea34c0d13b58f50d01e159
hghooks: advertise mercurial upgrade to non-bundle2 clients (bug 1221827); r=dminor
Deploying now.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Backing out because MozillaBuild only ships 3.4 currently. Sad panda.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Depends on: MozillaBuild2.1
An updated MozillaBuild with Mercurial 3.6.1 has been released. All Mozillians should now have the ability to install Mercurial 3.5+ easily. So, I'm going to reland this shortly.
Advertisement deployed.
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
Backing out because git-cinnabar is not using bundle2 yet.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
glandium: is cinnabar in a good enough state that we can re-deploy this?
Flags: needinfo?(mh+mozilla)
There is no release of cinnabar with bundle2 support. It's still experimental and partial (no support for bundle2 pushes, yet). You should know, the status is in the issue you opened ;)
Flags: needinfo?(mh+mozilla)
QA Contact: hwine → klibby
Modern versions of git-cinnabar appear to use bundle2 by default. So I'm going to rebase and land this.
Status: REOPENED → ASSIGNED
Pushed by gszorc@mozilla.com:
https://hg.mozilla.org/hgcustom/version-control-tools/rev/fe3153ff2508
ansible/hg-ssh: enable upgrade advertisement ; r=dminor
Status: ASSIGNED → RESOLVED
Closed: 9 years ago7 years ago
Resolution: --- → FIXED
I also deployed this shortly after it landed. Nobody has said anything yet. That's usually a good sign.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: