64bc74f4b91a7730c7d03c4fcbe9f4a723813a75: mozreview: copy more read-only processing code (bug 1229468); r=smacleod, dminor
Gregory Szorc <gps@mozilla.com> - Tue, 12 Jan 2016 15:07:29 -0800 - rev 362099
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
mozreview: copy more read-only processing code (bug 1229468); r=smacleod, dminor This is basically copying a lot of code from pushhooks.py. We don't yet make any database modification as part of this code. This just proves that a rough 1:1 copy continues to work.
44901a8bbcd8e06252578b171b2c6c5271b6023a: mozreview: add web API to submit an entire series of commits (bug 1229468); r=smacleod, dminor
Gregory Szorc <gps@mozilla.com> - Wed, 13 Jan 2016 12:08:27 -0800 - rev 362098
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
mozreview: add web API to submit an entire series of commits (bug 1229468); r=smacleod, dminor Currently, the Mercurial server makes several HTTP requests to Review Board's Web API to turn the pushed commits into review requests. Series with many commits take longer to submit than series with a single commit because we make additional requests for each review request / commit. This commit introduces a new Review Board Web API that aims to eventually replace the several existing Web API requests with a unified / single API call. Right now, this new API simply creates/updates the parent/squashed review request and diff. Eventually, more of the logic from pushhooks.py will move into the new Web API. I'm not an expert in the Review Board API. So there should be a thorough code review, which likely involves reading the new code next to existing code in Review Board's Web API.
4521cdc7a1d79470c90f3a912d0fb9682d3810a1: reviewboard: add test for pushing to a discarded series; r=smacleod
Gregory Szorc <gps@mozilla.com> - Wed, 13 Jan 2016 12:02:05 -0800 - rev 362097
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
reviewboard: add test for pushing to a discarded series; r=smacleod We didn't have explicit test coverage of this before.
da72429dca189827af3b3135210314617bcff287: reviewboard: add test for pushing to submitted review request; r=smacleod
Gregory Szorc <gps@mozilla.com> - Tue, 12 Jan 2016 14:01:20 -0800 - rev 362096
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
reviewboard: add test for pushing to submitted review request; r=smacleod We swallow output because it currently spews an exception stack, which is a bug already on file.
05e92374b355c8d288c122856a67c7aef9f1b4f7: mozreview: log user events; r=smacleod
Gregory Szorc <gps@mozilla.com> - Thu, 07 Jan 2016 16:59:32 -0800 - rev 362095
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
mozreview: log user events; r=smacleod As part of developing an upcoming feature, I ran into bugs related to user state. To help me debug the issue, I added logging of key user events. This log should also be useful for forensics and is generally good to have.
db08a2e7c5e89e3bcdacb4c4ffb2dc66fb881f5c: mozreview: print more diff info; r=smacleod
Gregory Szorc <gps@mozilla.com> - Thu, 07 Jan 2016 16:59:17 -0800 - rev 362094
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
mozreview: print more diff info; r=smacleod As part of changing how review requests are submitted, I ran into trouble getting diffs and diffsets to transition properly. To help me debug, this commit adds dumping of diff state from the published review request along with a few other fields to the dumped review request output.
2b1fc9fc244165bebde9be74889beeb9686c101d: reviewboard: use privileged RBTools client when setting LDAP association; r=smacleod
Gregory Szorc <gps@mozilla.com> - Thu, 07 Jan 2016 13:38:43 -0800 - rev 362093
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
reviewboard: use privileged RBTools client when setting LDAP association; r=smacleod 7d30e50718a9 accidentally dropped the "as rbc" from this code, meaning that we use the existing rbc variable/connection to adjust LDAP association. This would almost certainly fail. And, since this entire block is inside an "except APIError," we'd swallow the obvious failure. Derp. There are surprisingly no test failures as a result of this change. Wat. test-ldap-association.t explicitly tests the "LDAP account becomes associate during ssh push" case. How this was working before I don't know.
ef8435864f8c0aa53342fa5c82db161e539a416d: mozreview: Autoland-in-progress message should include target branch (bug 1238166) r=mdoglio
Dan Minor <dminor@mozilla.com> - Tue, 12 Jan 2016 08:13:10 -0500 - rev 362092
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
mozreview: Autoland-in-progress message should include target branch (bug 1238166) r=mdoglio This adds the branch name to the title for autoland messages. This also adds a line indicating who triggered the autoland request.
db4401543437ffdad6c4d7fe35af008f49b1b6b2: testing: pin Ansible to 1.9
Gregory Szorc <gps@mozilla.com> - Tue, 12 Jan 2016 19:57:38 -0800 - rev 362091
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
testing: pin Ansible to 1.9 Ansible 2.0 was released today. Unfortunately, it is roughly 2x slower executing our local playbooks. This will drastically slow down how long it takes to build Docker images for running tests. So ping to version 1.9 until we identify the slowdown or it is fixed upstream.
7c551b82a49953f7c2f63e6e121758bf0c52c4ae: reviewboard: mark server extension as compatible with 3.6
Gregory Szorc <gps@mozilla.com> - Mon, 21 Dec 2015 12:57:53 -0800 - rev 362090
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
reviewboard: mark server extension as compatible with 3.6
cb01b0d61153146a1e42cf80fa9afa9c4f60be38: docs: link to SSH host key fingerprints
Gregory Szorc <gps@mozilla.com> - Tue, 12 Jan 2016 18:09:56 -0800 - rev 362089
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
docs: link to SSH host key fingerprints Reduce the duplication of these values. Define them in a single point.
3661a0782bc36939b710729023f67e4a38b3cafb: docs: document MD5 host key fingerprints
Gregory Szorc <gps@mozilla.com> - Tue, 12 Jan 2016 18:09:32 -0800 - rev 362088
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
docs: document MD5 host key fingerprints Some SSH clients still display these.
0dabe93e9ddfb41ba3c46fc0dd418a2a961bd6fe: docs: correct typo in Mercurial for Mozillians (bug 1239181)
Bryce Van Dyk <bvandyk@mozilla.com> - Wed, 13 Jan 2016 14:46:23 +1300 - rev 362087
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
docs: correct typo in Mercurial for Mozillians (bug 1239181)
89b0c07dc6753b25677e62cc4bd816f5889be2e0: docs: remove section about hg host fingerprints
Gregory Szorc <gps@mozilla.com> - Tue, 12 Jan 2016 18:03:30 -0800 - rev 362086
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
docs: remove section about hg host fingerprints It should not be necessary when running modern Python and Mercurial. This should be documented in auth.rst anyway.
87167c3b8e6cf67ef0feb3786819d23e2f466d48: docs: add SHA-256 host keys for reviewboard-hg.mozilla.org (bug 1239049)
Gregory Szorc <gps@mozilla.com> - Tue, 12 Jan 2016 14:14:23 -0800 - rev 362085
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
docs: add SHA-256 host keys for reviewboard-hg.mozilla.org (bug 1239049)
2e54125275cb93926026301f40091cc0ea442ba9: mozreview: don't render filediff buttons on drafts (bug 1234161); r=smacleod
Mauro Doglio <mdoglio@mozilla.com> - Mon, 21 Dec 2015 11:16:46 +0100 - rev 362084
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
mozreview: don't render filediff buttons on drafts (bug 1234161); r=smacleod
2248273cff1c9789e941b14cb0a303dbfca0ff98: mozreview: people besides review requesters should be able autoland (bug 1205018) r=mdoglio
Dan Minor <dminor@mozilla.com> - Fri, 18 Dec 2015 15:12:07 -0500 - rev 362083
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
mozreview: people besides review requesters should be able autoland (bug 1205018) r=mdoglio This removes the check for whether or not the review is mutable by the current user, which allows people other than the review requester to autoland provided they have sufficient privileges.
2ab3575259951b0de52c06924533aa2993252d0e: testing: make it possible to specify repo name in create_basic_repo r=mdoglio
Dan Minor <dminor@mozilla.com> - Fri, 08 Jan 2016 11:29:07 -0500 - rev 362082
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
testing: make it possible to specify repo name in create_basic_repo r=mdoglio
1279676979808a6a4bfd04d17263e55b0165246c: testing: stop patching bmo when deploying to test (bug 1225836) r=gps
byron jones <glob@mozilla.com> - Tue, 01 Dec 2015 15:42:34 +0800 - rev 362081
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
testing: stop patching bmo when deploying to test (bug 1225836) r=gps Remove the last bmo patch and the supporting patching code. Also fixes pyflakes and pep8 issues.
05e8212f9aad03d5e2400357807448b306c99584: mozhginfo: add pushlog_client.egg-info to .hgignore
byron jones <glob@mozilla.com> - Mon, 11 Jan 2016 14:11:44 +0800 - rev 362080
Push 16998 by rwood@mozilla.com at Mon, 02 May 2016 19:42:03 +0000
mozhginfo: add pushlog_client.egg-info to .hgignore
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 +300000 tip