Closed Bug 1649723 Opened 4 years ago Closed 4 years ago

Pushlog query with cloned fromchangeset errors

Categories

(Developer Services :: Mercurial: Pushlog, defect)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Pike, Assigned: Pike)

References

()

Details

Attachments

(1 file)

When querying the pushlog for pushes newer than a revision, there's a 500 error if that changeset was never pushed. I.e., if it was cloned.

Example URL would be https://hg.mozilla.org/releases/comm-esr78/json-pushes?tipsonly=1&tochange=default&fromchange=548b029d0a156dca9ef3954b8ec489b40484bb2f.

We're hitting that bug while adding ESR 78 to the l10n cross-channel setup, bug 1649420

Assignee: nobody → l10n
Status: NEW → ASSIGNED

Looking at the error, it's so generic that I'm concerned to add workarounds. It'd be "whenever you get a 500 error, pull the complete pushlog".

It feels easier to me to patch hgmo instead ;-)

Thus, marking this as blocking bug 1649420.

I got to the point where I can run tests locally, but hgext/pushlog/tests/test-user.t doesn't pass for me locally on the test-runner image.

Blocks: 1649420
See Also: 1649420

Pushed by cosheehan@mozilla.com:
https://hg.mozilla.org/hgcustom/version-control-tools/rev/19a77a5d7b2a
pushlog: Start pushlog at 0 if fromchange is a cloned revision r=sheehan

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED

Any chance to deploy this? My understanding is that deployment only happens on-demand, not on schedule.

We need this for bug 1649420, which needs to happen before 78 gets kicked out of mozilla-release.

Flags: needinfo?(sheehan)

(In reply to Francesco Lodolo [:flod] from comment #4)

Any chance to deploy this? My understanding is that deployment only happens on-demand, not on schedule.

We need this for bug 1649420, which needs to happen before 78 gets kicked out of mozilla-release.

Done!

Flags: needinfo?(sheehan)

Sweet, thanks, that worked.

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: