If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Review-commit description is missing lines in corresponding bugzilla commit (because of "Note:" prefix?)

RESOLVED FIXED

Status

MozReview
Integration: Bugzilla
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: gerald, Assigned: glob)

Tracking

Production

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

a year ago
See this review page:
https://reviewboard.mozilla.org/r/50135/
Compare the Description (that came with the pushed commit), to what appears in bug 1268379 comment 4.
The Bugzilla comment is missing the first line of the 3rd paragraph, the one that starts with "Note:".

Either something went wrong in the comment-posting process, or "Note:" is handled in a special way -- which seems a bit more likely than a random line loss, and makes sense since another line starting with "MozReview-Commit-ID:" was also cut.
But in any case, I would not expect that line to be removed.

If some keywords are used to filter lines, I don't think they should be as simple as 'Note', especially when they appear in the midst of a longer description.
(Assignee)

Updated

a year ago
Component: Extensions: MozReview Integration → Integration: Bugzilla
Product: bugzilla.mozilla.org → MozReview
Yeah, mozautomation.commitparser appears to eat any line that starts with a single word (letters and hyphens) followed by a colon: http://hg.mozilla.org/hgcustom/version-control-tools/file/tip/pylib/mozautomation/mozautomation/commitparser.py#l201

There's already a TODO there that the parsing is overly simplified.  I'm not sure how many metadata keywords we actually expect; the only one I know of (and the only one mentioned in the function's docstring) is MozReview-Commit-ID.  gps, can we have a registry of them to avoid this function being overly greedy?
Flags: needinfo?(gps)

Updated

a year ago
Duplicate of this bug: 1265851
I think a whitelist of known metadata keywords is fine.
Flags: needinfo?(gps)
(Assignee)

Updated

a year ago
Assignee: nobody → glob
(Assignee)

Comment 4

a year ago
looking over the commit descriptions to mozilla-central and version-control-tools, the only meta tag i found was MozReview-Commit-ID.  Review-URL exists in a vct, but that appears to be an experiment at the end of may 2015.
(Assignee)

Comment 5

a year ago
Created attachment 8749054 [details]
MozReview Request: mozautomation: only strip MozReview-Commit-ID from commit descriptions (bug 1269649) r?gps

Instead of removing anything that may be a meta tag, just remove known tags.

Review commit: https://reviewboard.mozilla.org/r/50723/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/50723/
Attachment #8749054 - Flags: review?(gps)
Comment on attachment 8749054 [details]
MozReview Request: mozautomation: only strip MozReview-Commit-ID from commit descriptions (bug 1269649) r?gps

https://reviewboard.mozilla.org/r/50723/#review47581
Attachment #8749054 - Flags: review?(gps) → review+
https://hg.mozilla.org/hgcustom/version-control-tools/rev/1d81c634d7b4
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.