mozilla :: #bmo

16 May 2017
00:03bugbotNew Administration bug 1365107 filed by bkerensa@gmail.com.
00:03bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1365107 Administration, normal, --, nobody, NEW , Emails
00:16bugbotdylan@mozilla.com changed the Status on bug 1365107 from NEW --- to RESOLVED FIXED.
00:16bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1365107 Administration, normal, --, nobody, RESOLVED FIXED, Emails
03:56bugbotglob@mozilla.com granted review for attachment 8867901 on bug 1361621.
03:56bugbotAttachment https://bugzilla.mozilla.org/attachment.cgi?id=8867901&action=edit patch, 1361621_1.patch, 1361621_1.patch
03:56bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1361621 General, normal, P3, dylan, NEW , chunk resolve_bug.pl bug updates into multiple transactions
04:05bugbotglob@mozilla.com denied feedback for attachment 8867028 on bug 1364285.
04:05bugbotAttachment https://bugzilla.mozilla.org/attachment.cgi?id=8867028&action=edit patch, user_settings_cache_1.patch, user_settings_cache_1.patch
04:05bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1364285 General, normal, --, dylan, NEW , Cache user settings in memcached (extracted from bug 1021795)
13:04bugbutlerdylanwh pushed 1 to mozilla-bteam/bmo. Compare: https://github.com/mozilla-bteam/bmo/compare/0ae72757feb8...527d3cb69e1f
13:04bugbotdylan@mozilla.com changed the Status on bug 1361621 from NEW --- to RESOLVED FIXED.
13:04bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1361621 General, normal, P3, dylan, RESOLVED FIXED, chunk resolve_bug.pl bug updates into multiple transactions
13:11bugbutlerdylanwh pushed 12 to mozilla-bteam/bmo. Compare: https://github.com/mozilla-bteam/bmo/compare/1be1a16b9261...78ac59a688d1
13:12bugbotNew Infrastructure bug 1365247 filed by dylan@mozilla.com.
13:12bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1365247 Infrastructure, normal, --, dylan, NEW , push updated bugzilla.mozilla.org live
13:16bugbutlerdylanwh pushed 1 to mozilla-bteam/bmo. Compare: https://github.com/mozilla-bteam/bmo/compare/527d3cb69e1f...619fefa7ebb1
13:18bugbutlerdylanwh pushed 2 to mozilla-bteam/bmo. Compare: https://github.com/mozilla-bteam/bmo/compare/78ac59a688d1...a3eaeceb7c08
13:31bugbutlerdylanwh pushed 1 to mozilla-bteam/bmo. Compare: https://github.com/mozilla-bteam/bmo/compare/619fefa7ebb1...e1db12e2aec5
13:33bugbutlerdylanwh pushed 2 to mozilla-bteam/bmo. Compare: https://github.com/mozilla-bteam/bmo/compare/a3eaeceb7c08...bd799a0a2b30
13:58bugbotdylan@mozilla.com changed the Status on bug 1365247 from NEW --- to RESOLVED FIXED.
13:59bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1365247 Infrastructure, normal, --, dylan, RESOLVED FIXED, push updated bugzilla.mozilla.org live
14:31smaugmcote: so there is no way to get Differential review comments to bugzilla?
14:33smaugI'm trying to understand what the history tracking model would be in the new world
14:33smaugor even development model
14:33smaugwhere some discussion would happen in the bug and some in Differential
14:38mcotesmaug: it would be possible, yes
14:38mcoteit's not a technical limitation per se
14:39mcoteI initially proposed not doing it for two reasons: it's added complexity that I wasn't sure if we needed, and there has been some discussion amongst some engineers about whether code review should be kept separate from issue tracking
14:39mcoteclearly there's some disagreement here :) I need to talk to more people, probably some engineering managers or directors
14:40smaugok, I thought it was technical limitation
14:41mcoteoh no, we can always get the data, either via the API or, at worst, via a Phabricator extension
14:41smaugmy experience doing some standardization related work using github has shown that the model it has is causing problems
14:41smaugsome important discussions happening in issues, some in pr reviews
14:42mcoteI personally wonder if it's just a matter of being more intentional and getting used to the idea of talking in the right places
14:42smaugand if one doesn't realize that both should be followed, one may miss some critical information
14:42mcotebut I also realize that's not easy :)
14:42smaugit is so easy to just comment some higher level things in a review comment
14:43mcoteI also wonder if this is a case of older, experienced contributors vs new contributors
14:43mcotenew contributors might find it easier to follow a bug or code review if the topics are kept to the appropriate places
14:43mcotebut I don't know that for sure, just a theory
14:45mcotepart of the goal of moving to a better-known, user-friendlier tool is for the ease of new contributors after all
14:45smaugnew contributors rarely end up looking history data so much
14:45smaugbased on my experience
14:46mcotewell so far one new person espoused support for splitting them
14:46mcotebut that's just one data point I realize
14:46mcotethe problem is that without bidirectional mirroring you still risk missing part of the conversation
14:46mcoteand bidirectional anything is far from trivial
14:46smaugThis is similar to shutting down bonsai. Couple of pre-historical devs really needed it, but for most of the devs it wasn't an issue at all
14:47mcoteI get what you are saying though I question "really needed it", as I assume they are still doing their jobs :)
14:47mcote"really wanted it" seems more accurate
14:48mcotetools engineering is all about trade offs in system complexity vs individualized workflows
14:48smaugI couldn't do my reviews without reasonable easy way to track changes
14:48mcoteright, and presuming you are CCed on both the bug and its Phabricator revision(s), you still get both feeds, though I realize not in the way you'd prefer
14:48smaug(http://52.25.115.98/viewvc/main/ was a temporary replacement for bonsai and luckily billm wrote searchfox and based on my comments added bonsai like blame :) )
14:49mcotesure, and that's a good example of moving forward with tooling :)
14:49smaugwell, we didn't replace bonsai with anything reasonable until billm wrote searchfox
14:50smaugfor years we had horrible tools for hg blame
14:50smaugcomparing to what bonsai offered for cvs
14:50smaugoh well :)
14:50smaugI'm all supporting making reviewing easier for most people
14:57mcoteanyway this isn't set in stone; we're still working on BMO integration
14:57mcotethere are possibly alternate solutions like just mirroring over links to comments, to encourage discussion of the code review to happen in the one place
14:58mcotethe main thing I'd like to avoid is having comments mirrored over to the bug, and then replies to those comments only in the bug, and then you have a forked discussion of the code review
14:58mcotewhich would undoubtedly be confusing for new contributors (and possibly experienced ones :)
14:58mcoteand I'd like to use Phabricator's emails instead of doing everything through BMO as we did with MozReview
15:00smaugI really wish comments are mirrored to the bug
15:00smaugso that there one single place of truth
15:00smaugis
16:15bugbotNew Infrastructure bug 1365301 filed by dylan@mozilla.com.
16:15bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1365301 Infrastructure, normal, --, nobody, NEW , Update nagios health checks to use __heartbeat__
16:18bugbotdylan@mozilla.com granted review for attachment 8865084 on bug 1362198.
16:18bugbotAttachment https://bugzilla.mozilla.org/attachment.cgi?id=8865084&action=edit text/plain, Updated copy for rate limited page, file_1362198.txt
16:18bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1362198 General, normal, P2, ehumphries, NEW , Write copy for rate-limited page
16:24bugbotehumphries@mozilla.com changed the Status on bug 1364636 from NEW --- to RESOLVED INCOMPLETE.
16:24bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1364636 Infrastructure, normal, --, nobody, RESOLVED INCOMPLETE, esfrontline.bugzilla.mozilla.org returning 400s
16:36bugbutlerdylanwh opened new pull request: https://github.com/mozilla-bteam/bmo/pull/79
16:36bugbutlerdylanwh edited pull request: https://github.com/mozilla-bteam/bmo/pull/79
17:07bugbotNew Bug Creation/Editing bug 1365317 filed by sebastinssanty@gmail.com.
17:07bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1365317 Bug Creation/Editing, normal, --, nobody, NEW , [Meta] Modal Bug Entry
17:32bugbotNew General bug 1365322 filed by dylan@mozilla.com.
17:32bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1365322 General, normal, --, dylan, NEW , Make root page static
17:34bugbutlerdylanwh closed pull request: https://github.com/mozilla-bteam/bmo/pull/79
17:34bugbutlerdylanwh pushed 1 to mozilla-bteam/bmo. Compare: https://github.com/mozilla-bteam/bmo/compare/e1db12e2aec5...2216a50a64cf
17:56bugbutlerdavehouse pushed 1 to mozilla-bteam/bmo-nubis. Compare: https://github.com/mozilla-bteam/bmo-nubis/compare/4247e3147d38...baf313a20249
18:19bugbotNew Bug Creation/Editing bug 1365340 filed by sebastinssanty@gmail.com.
18:19bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1365340 Bug Creation/Editing, normal, --, sebastinssanty, ASSIGNED , Add /new-bug (new_bug.cgi)
18:23bugbotNew Bug Creation/Editing bug 1365342 filed by sebastinssanty@gmail.com.
18:23bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1365342 Bug Creation/Editing, normal, --, sebastinssanty, ASSIGNED , Extract the preview comment functionality and make it a reusable template
18:26bugbotNew Bug Creation/Editing bug 1365344 filed by sebastinssanty@gmail.com.
18:26bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1365344 Bug Creation/Editing, normal, --, sebastinssanty, ASSIGNED , Extract Status and null modules
18:28bugbotNew Bug Creation/Editing bug 1365345 filed by sebastinssanty@gmail.com.
18:28bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1365345 Bug Creation/Editing, normal, --, sebastinssanty, ASSIGNED , Modal Bug Entry Prototype
18:35bugbotklibby@mozilla.com changed the Status on bug 1365301 from NEW --- to RESOLVED FIXED.
18:35bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1365301 Infrastructure, normal, --, klibby, RESOLVED FIXED, Update nagios and zeus health checks to use __heartbeat__
19:31bugbotNew Administration bug 1365374 filed by mats@mozilla.com.
19:31bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1365374 Administration, normal, --, nobody, NEW , monkeydrugs01@gmail.com is a spam account
19:44bugbotdylan@mozilla.com changed the Status on bug 1365374 from NEW --- to RESOLVED FIXED.
19:44bugbotBug https://bugzilla.mozilla.org/show_bug.cgi?id=1365374 Administration, normal, --, nobody, RESOLVED FIXED, monkeydrugs01@gmail.com is a spam account
20:38bugbutlerdylanwh opened new pull request: https://github.com/mozilla-bteam/bmo/pull/80
20:50bugbutlerdylanwh reviewed pull request 75: https://github.com/mozilla-bteam/bmo/pull/75#pullrequestreview-38508591
20:51bugbutlerdylanwh reviewed pull request 75: https://github.com/mozilla-bteam/bmo/pull/75#pullrequestreview-38508838
20:51bugbutlerdylanwh reviewed pull request 75: https://github.com/mozilla-bteam/bmo/pull/75#pullrequestreview-38508984
20:52bugbutlerdylanwh reviewed pull request 75: https://github.com/mozilla-bteam/bmo/pull/75#pullrequestreview-38509146
20:54bugbutlerdylanwh reviewed pull request 75: https://github.com/mozilla-bteam/bmo/pull/75#pullrequestreview-38509590
20:54bugbutlerdylanwh reviewed pull request 75: https://github.com/mozilla-bteam/bmo/pull/75#pullrequestreview-38509756
20:57bugbutlerfloatingatoll reviewed pull request 75: https://github.com/mozilla-bteam/bmo/pull/75#pullrequestreview-38510465
20:57bugbutlerdylanwh reviewed pull request 75: https://github.com/mozilla-bteam/bmo/pull/75#pullrequestreview-38510510
20:57bugbutlerfloatingatoll reviewed pull request 75: https://github.com/mozilla-bteam/bmo/pull/75#pullrequestreview-38510555
20:57bugbutlerfloatingatoll reviewed pull request 75: https://github.com/mozilla-bteam/bmo/pull/75#pullrequestreview-38510562
20:58bugbutlerdylanwh reviewed pull request 75: https://github.com/mozilla-bteam/bmo/pull/75#pullrequestreview-38510832
20:59bugbutlerdylanwh reviewed pull request 75: https://github.com/mozilla-bteam/bmo/pull/75#pullrequestreview-38510936
21:00bugbutlerdylanwh reviewed pull request 75: https://github.com/mozilla-bteam/bmo/pull/75#pullrequestreview-38511114
21:00bugbutlerdylanwh reviewed pull request 75: https://github.com/mozilla-bteam/bmo/pull/75#pullrequestreview-38511331
22:50bugbutlerdylanwh pushed 1 to mozilla-bteam/bmo. Compare: https://github.com/mozilla-bteam/bmo/compare/2216a50a64cf...38b13ae3f188
17 May 2017
No messages
   
Last message: 93 days and 6 hours ago