{"id":211,"date":"2009-03-20T05:52:08","date_gmt":"2009-03-20T10:52:08","guid":{"rendered":"http:\/\/www.visophyte.org\/blog\/?p=211"},"modified":"2009-03-20T06:01:49","modified_gmt":"2009-03-20T11:01:49","slug":"using-review-board-for-bugzilla-request-queues-reviews","status":"publish","type":"post","link":"https:\/\/www.visophyte.org\/blog\/2009\/03\/20\/using-review-board-for-bugzilla-request-queues-reviews\/","title":{"rendered":"using review board for bugzilla request queues \/ reviews"},"content":{"rendered":"<p><a href=\"http:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-dashboard.png\"><img loading=\"lazy\" decoding=\"async\" class=\"alignnone size-thumbnail wp-image-212\" title=\"review-board-dashboard\" src=\"http:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-dashboard-600x160.png\" alt=\"review-board-dashboard\" width=\"600\" height=\"160\" srcset=\"https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-dashboard-600x160.png 600w, https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-dashboard-300x80.png 300w, https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-dashboard-1024x274.png 1024w, https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-dashboard.png 1236w\" sizes=\"auto, (max-width: 600px) 100vw, 600px\" \/><\/a><\/p>\n<p><a href=\"http:\/\/www.review-board.org\/\">Review board<\/a> is a delightful django-based review tool.\u00a0 Whereas bugzilla&#8217;s diff display is limited to taking the provided diff and coloring hunks without additional context, review board actually retrieves the underlying files from mercurial, applies the patch, colorizes the source files (using pygments), and thus can give you additional context.\u00a0 Also, it allows you to click on line(s) and attach a note at that location.<\/p>\n<p>Anywho, I made a new <a href=\"http:\/\/hg.mozilla.org\/users\/bugmail_asutherland.org\/bugzilla-meddler\/\">tool<\/a> that &#8216;synchronizes&#8217; bugzilla request queues with review board.\u00a0 It finds the things in your request queue, and then creates a review in review board, transferring the patch.\u00a0 There is some limited hard-coded biasing towards comm-central, but it&#8217;s arguably easily adaptable.\u00a0 It uses a slightly hacked up version of the post-review script found in review board&#8217;s contrib directory to serve as the API to review board.<\/p>\n<p><a href=\"http:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-diffy-comment.png\"><img loading=\"lazy\" decoding=\"async\" class=\"alignnone size-thumbnail wp-image-213\" title=\"review-board-diffy-comment\" src=\"http:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-diffy-comment-600x337.png\" alt=\"review-board-diffy-comment\" width=\"600\" height=\"337\" srcset=\"https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-diffy-comment-600x337.png 600w, https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-diffy-comment-300x168.png 300w, https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-diffy-comment-1024x575.png 1024w, https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-diffy-comment.png 1220w\" sizes=\"auto, (max-width: 600px) 100vw, 600px\" \/><\/a><\/p>\n<p>The intent is to make it easier for people, or at least me, to do reviews.\u00a0 I am definitely NOT proposing that the review &#8220;of record&#8221; live outside bugzilla.\u00a0 But I think it might make it easier to prepare reviews in the first place.\u00a0 For example, see the picture below?\u00a0 It&#8217;s the result of my clicking on lines of code and making notes.\u00a0 And those hyperlink-looking things that look like they might take you to the actual context, rather than manually having to try and determine context from people&#8217;s comments in the bugzilla notes?\u00a0 They are actual hyperlinks that do what I just said!<\/p>\n<p><a href=\"http:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-review.png\"><img loading=\"lazy\" decoding=\"async\" class=\"alignnone size-thumbnail wp-image-214\" title=\"review-board-review\" src=\"http:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-review-600x467.png\" alt=\"review-board-review\" width=\"600\" height=\"467\" srcset=\"https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-review-600x467.png 600w, https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-review-300x233.png 300w, https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-review-1024x797.png 1024w, https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2009\/03\/review-board-review.png 1182w\" sizes=\"auto, (max-width: 600px) 100vw, 600px\" \/><\/a><\/p>\n<p>If you are a mailnews hacker and would like to see your review queue in my reviewboard instance, ping me on IRC.\u00a0 I may have already processed your queue and made you an account, or maybe not.\u00a0 The review board API does not yet automate user creation and I got tired of doing things manually and gave up.<\/p>\n<p>Warnings:<\/p>\n<ul>\n<li>Things go awry on old-school patches.\u00a0 For example, a patch that thinks it is against (the CVS layout of) &#8220;mozilla\/mailnews&#8221; appears to not work.\u00a0 The right answer is a patch against &#8220;a\/mailnews&#8221; (I&#8217;m not sure if the git-style is essential).\u00a0 Things worked fine for my queue, but some people with old stuff in their queues experienced glitches.\u00a0 The script now inserts &#8220;problem!&#8221; into the summary when this happens, but in my reviewboard instance there may be some reviews that pre-date this logic.\u00a0 (There should be a note in the description though.\u00a0 Also, there will obviously be no diff attached.)<\/li>\n<li>I initially forgot about the whole review=one person, super-review=another person, and maybe review=yet another person.\u00a0 Although this is now fixed and reviews target all requested revieweres, there may be some reviews missing from your queue because they were created for one of the dudes who was not you and my manual nuking didn&#8217;t get to those.<\/li>\n<li>Uh, it creates the reviews, but it doesn&#8217;t destroy them.<\/li>\n<li>Also uh, there will be scaling issues if the reviews don&#8217;t eventually get closed out.<\/li>\n<\/ul>\n","protected":false},"excerpt":{"rendered":"<p>Review board is a delightful django-based review tool.\u00a0 Whereas bugzilla&#8217;s diff display is limited to taking the provided diff and coloring hunks without additional context, review board actually retrieves the underlying files from mercurial, applies the patch, colorizes the source &hellip; <a href=\"https:\/\/www.visophyte.org\/blog\/2009\/03\/20\/using-review-board-for-bugzilla-request-queues-reviews\/\">Continue reading <span class=\"meta-nav\">&rarr;<\/span><\/a><\/p>\n","protected":false},"author":1,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"inline_featured_image":false,"footnotes":""},"categories":[3],"tags":[34,35,36],"class_list":["post-211","post","type-post","status-publish","format-standard","hentry","category-mozilla","tag-bugzilla","tag-review-board","tag-reviews"],"_links":{"self":[{"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/posts\/211","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/comments?post=211"}],"version-history":[{"count":4,"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/posts\/211\/revisions"}],"predecessor-version":[{"id":217,"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/posts\/211\/revisions\/217"}],"wp:attachment":[{"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/media?parent=211"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/categories?post=211"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/tags?post=211"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}