Closed
Bug 1021929
Opened 10 years ago
Closed 10 years ago
Initial deployment of Review Board with review repo
Categories
(MozReview Graveyard :: General, defect, P1)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mcote, Unassigned)
References
Details
(Keywords: bmo-big)
This is a tracking bug for the remaining issues in deploying the updated Review Board fork, the new rbbz extension, and the hg review repo & hook.
Updated•10 years ago
|
Comment 1•10 years ago
|
||
Added Bug 889431.
Docs for prod at https://mana.mozilla.org/wiki/display/websites/reviewboard.mozilla.org
I believe the only remaining bits are updating the mysql schema (bug 1013156), nagios monitoring, backups, and actual user testing.
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Comment 2•10 years ago
|
||
I agree that bug 508541 (putting the associated review with the r+) would be nice, but I can't justify it as a blocker unless there's some crucial reason for it that I don't understand.
No longer depends on: 508541
Comment 3•10 years ago
|
||
(In reply to Mark Côté [:mcote] from comment #2)
> I agree that bug 508541 (putting the associated review with the r+) would be
> nice, but I can't justify it as a blocker unless there's some crucial reason
> for it that I don't understand.
That bug means that bugmails are missing the comments when the review is requested. For current workflows, people often ask questions/point out which parts the reviewer should be looking at in the comment.
Bug 508541 has resulted in bzexport being reverted back to the legacy bzapi, although it's more painful in that workflow - since (at least for me, but I imagine others) I typically skip the bug after reading the bugmail, and go straight to the splinter review page, which doesn't include the bug comment.
I'm not that familiar with reviewboard, but it seems like the entry point is normally to the overview, which does list the description (and so people are less likely to skip over it), in which case this is less painful than the bzexport case. However it's still slightly annoying, since I often use review request comments in bugmails to decide whether the review will be quick or not, and as such whether to take a look immediately, or in a few hours time - so this issue means having to open every reviewboard request just to see those comments.
Assignee | ||
Comment 4•10 years ago
|
||
Ah I see, I misunderstood this bug; I thought it was about the comments being sent separately, not them being eaten. So I imagine this bug would affect any comments sent to the API when a review flag was set to "+" as well.
In that case, I think I'll work around this bug by first posting the r? and then posting the comment right after. That will result in two emails (like you actually get from an r+ right now, before my "fix"), but at least all the info should be there. That work for you until we can get bug 508541 fixed?
Flags: needinfo?(emorley)
Comment 5•10 years ago
|
||
That should work well enough such that it's no longer an issue for rollout I think. We'll need to make sure the review request is sent first, so it auto-CCs the user to the bug (if they've left that set in prefs - if not, then they'll just have to read it in the bug or reviewboard overview) and then makes the comment second.
Flags: needinfo?(emorley)
Assignee | ||
Comment 6•10 years ago
|
||
Okay, I've addressed this in my patch for bug 1033489, leaving FIXME comments for whenever bug 508541 is fixed.
Assignee | ||
Comment 7•10 years ago
|
||
I'm retargetting deployment for end of September, since smacleod is still out on sick leave and gps is on PTO for a couple weeks. Doesn't make sense to launch with no engineers on hand.
Updated•10 years ago
|
Product: bugzilla.mozilla.org → Developer Services
Updated•10 years ago
|
Priority: -- → P1
Comment 9•10 years ago
|
||
monitoring is all set.
Assignee | ||
Comment 10•10 years ago
|
||
Last blocker fixed! We've already gotten some more people using this; I think we can advertise more widely now.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
Product: Developer Services → MozReview
You need to log in
before you can comment on or make changes to this bug.
Description
•