Review: Approve
Lgtm, +1
--
https://code.launchpad.net/~cjwatson/launchpad/remove-unnecessary-exception-handling/+merge/370616
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/remove-unnecessary-exception-handling into lp:launchpad.
Review: Approve
+1
--
https://code.launchpad.net/~cjwatson/launchpad/readthedocs-copyright-2019/+merge/370480
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/readthedocs-copyright-2019 into lp:launchpad.
Review: Approve
+1
--
https://code.launchpad.net/~cjwatson/launchpad/s-s-s-eoan/+merge/369439
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/s-s-s-eoan into lp:launchpad.
___
Mailing list:
Review: Approve
+1
--
https://code.launchpad.net/~cjwatson/launchpad/upgrade-difftacular/+merge/368837
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/upgrade-difftacular into lp:launchpad.
___
Review: Approve
+1
--
https://code.launchpad.net/~cjwatson/launchpad/remove-lpreview/+merge/368835
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/remove-lpreview into lp:launchpad.
___
Mailing
Review: Approve
Looks good to me.
--
https://code.launchpad.net/~cjwatson/launchpad/bs4-initial/+merge/366479
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/bs4-initial into lp:launchpad.
___
Review: Approve
Looks good to me, celerywise - but I'm not particularly familiar with
amqp/amqplib.
--
https://code.launchpad.net/~cjwatson/launchpad/celery-4/+merge/364202
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/celery-4 into
Review: Approve
LGTM.
--
https://code.launchpad.net/~cjwatson/launchpad/celery-worker-discard-stderr/+merge/364158
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/celery-worker-discard-stderr into lp:launchpad.
Review: Approve
LGTM
--
https://code.launchpad.net/~cjwatson/launchpad/celery-disable-pickle/+merge/364045
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/celery-disable-pickle into lp:launchpad.
Review: Approve
LGTM
--
https://code.launchpad.net/~twom/launchpad/no-rescan-with-no-jobs/+merge/362479
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~twom/launchpad/no-rescan-with-no-jobs into lp:launchpad.
___
Nice feature, thanks! Just a quick review, with a couple of questions.
Diff comments:
>
> === modified file 'lib/lp/code/model/branch.py'
> --- lib/lp/code/model/branch.py 2018-12-10 13:54:34 +
> +++ lib/lp/code/model/branch.py 2019-01-23 16:23:25 +
> @@ -1295,6 +1295,15 @@
Review: Approve
Looks good to me.
--
https://code.launchpad.net/~cjwatson/launchpad/git-ssh-url-username/+merge/353748
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/git-ssh-url-username into lp:launchpad.
Review: Approve
LGTM, thanks.
--
https://code.launchpad.net/~cjwatson/launchpad/delete-ssh-key-text-type/+merge/352210
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/delete-ssh-key-text-type into lp:launchpad.
Diff comments:
> === modified file 'lib/lp/registry/interfaces/ssh.py'
> --- lib/lp/registry/interfaces/ssh.py 2018-02-24 09:11:39 +
> +++ lib/lp/registry/interfaces/ssh.py 2018-06-19 21:32:30 +
> @@ -124,4 +124,21 @@
>
> @error_status(httplib.BAD_REQUEST)
> class
The proposal to merge lp:~maxiberta/launchpad/sshkeyadditionerror-msg-format
into lp:launchpad has been updated.
Commit message changed to:
Fix crash while adding an ssh key with unknown type.
For more details, see:
The proposal to merge lp:~maxiberta/launchpad/sshkeyadditionerror-msg-format
into lp:launchpad has been updated.
Description changed to:
Fix encoding crash while adding an ssh key with unknown type. E.g. on an
attempt to add this key in SSO /ssh-keys:
ssh-rsa asdfasdf comment
LP raises
The proposal to merge lp:~maxiberta/launchpad/sshkeyadditionerror-msg-format
into lp:launchpad has been updated.
Description changed to:
Add SSHKeyAdditionError constructor with a warning to try to prevent a mismatch
with SSO regex matching. It's already tested in
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/sshkeyadditionerror-msg-format into lp:launchpad.
Commit message:
Add SSHKeyAdditionError constructor to try to prevent a mismatch with SSO regex
matching.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers
The proposal to merge lp:~maxiberta/launchpad/sshkeyadditionerror-msg-format
into lp:launchpad has been updated.
Description changed to:
Add SSHKeyAdditionError constructor to try to prevent a mismatch with SSO regex
matching. It's already tested in lib/lp/registry/tests/test_ssh.py.
For more
Review: Approve
--
https://code.launchpad.net/~cjwatson/launchpad/new-privacy-policy/+merge/346743
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/new-privacy-policy into lp:launchpad.
___
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/sitesearch-drop-google-backend into lp:launchpad.
Commit message:
Remove the Google search service backend.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https://code.launchpad.net
Review: Approve
LGTM.
--
https://code.launchpad.net/~cjwatson/launchpad/snap-store-release-refresh/+merge/344997
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/snap-store-release-refresh into lp:launchpad.
The proposal to merge lp:~maxiberta/launchpad/sitesearch-cleanup-2 into
lp:launchpad has been updated.
Description changed to:
Add sitesearch tests based on doctests.
- sitesearch/doc/google-searchservice.txt => sitesearch/tests/test_google.py
- sitesearch/doc/bing-searchservice.txt =>
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/app-browser-dedup-sitesearch-doctests into lp:launchpad.
Commit message:
Deduplicate lp.app.browser sitesearch doctests.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/app-stories-dedup-sitesearch-doctests into lp:launchpad.
Commit message:
Run lp.app.stories.launchpad_search.site-search.txt on all search backends.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/sitesearch-dedup-tests into lp:launchpad.
Commit message:
Deduplicate sitesearch testservices testing code.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https://code.launchpad.net
The proposal to merge lp:~maxiberta/launchpad/sitesearch-cleanup-2 into
lp:launchpad has been updated.
Description changed to:
Add sitesearch tests based on doctests.
- sitesearch/doc/google-searchservice.txt => sitesearch/tests/test_google.py
- sitesearch/doc/bing-searchservice.txt =>
The proposal to merge lp:~maxiberta/launchpad/sitesearch-cleanup-2 into
lp:launchpad has been updated.
Commit message changed to:
Add sitesearch tests based on doctests.
For more details, see:
https://code.launchpad.net/~maxiberta/launchpad/sitesearch-cleanup-2/+merge/343236
--
Your team
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/sitesearch-cleanup-2 into lp:launchpad with
lp:~maxiberta/launchpad/sitesearch-cleanup-1 as a prerequisite.
Commit message:
Assorted sitesearch fixes and improvements (part 2).
Requested reviews:
Launchpad code reviewers
The proposal to merge lp:~maxiberta/launchpad/sitesearch-cleanup-2 into
lp:launchpad has been updated.
Status: Work in progress => Superseded
For more details, see:
https://code.launchpad.net/~maxiberta/launchpad/sitesearch-cleanup-2/+merge/343235
--
Your team Launchpad code reviewers is
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/sitesearch-cleanup-1 into lp:launchpad.
Commit message:
Assorted sitesearch fixes and improvements.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https://code.launchpad.net
Review: Resubmit
I'm splitting this into more manageable MP's. The first one:
https://code.launchpad.net/~maxiberta/launchpad/sitesearch-cleanup-1/+merge/343131.
--
https://code.launchpad.net/~maxiberta/launchpad/sitesearch-cleanup/+merge/342472
Your team Launchpad code reviewers is requested
The proposal to merge lp:~maxiberta/launchpad/sitesearch-cleanup into
lp:launchpad has been updated.
Status: Needs review => Work in progress
For more details, see:
https://code.launchpad.net/~maxiberta/launchpad/sitesearch-cleanup/+merge/342472
--
Your team Launchpad code reviewers is
Thanks!
--
https://code.launchpad.net/~cjwatson/launchpad/bing-xss/+merge/342950
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/bing-xss into lp:launchpad.
___
Mailing list:
Review: Approve
LGTM. An extra test in sitesearch/tests/test_bing.py would be great though.
--
https://code.launchpad.net/~cjwatson/launchpad/bing-xss/+merge/342950
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/bing-xss into lp:launchpad.
The proposal to merge lp:~maxiberta/launchpad/sitesearch-cleanup into
lp:launchpad has been updated.
Description changed to:
Sitesearch code cleanup:
- Add unittests based on sitesearch doctests.
- These doctests are still tested; unsure of removing as they're still useful
documentation.
-
The proposal to merge lp:~maxiberta/launchpad/sitesearch-cleanup into
lp:launchpad has been updated.
Description changed to:
Sitesearch code cleanup:
- Add unittests based on sitesearch doctests.
- These doctests are still tested; unsure of removing as they're still useful
documentation.
-
The proposal to merge lp:~maxiberta/launchpad/sitesearch-cleanup into
lp:launchpad has been updated.
Description changed to:
Sitesearch code cleanup:
- Add unittests based on sitesearch doctests.
- These doctests are still tested; unsure of removing as they're still useful
documentation.
-
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/sitesearch-cleanup into lp:launchpad.
Commit message:
Sitesearch code cleanup.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https://code.launchpad.net/~maxiberta/launchpad
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/sitesearch-default-bing into lp:launchpad.
Commit message:
Make Bing the default site search engine.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https://code.launchpad.net
The proposal to merge lp:~maxiberta/launchpad/bing-search into lp:launchpad has
been updated.
Description changed to:
Add basic Bing Custom Search support.
Should be pretty unobtrusive, while adding a basic site search implementation
around Bing Custom Search (shamelessly copied from the
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/bing-search into lp:launchpad.
Commit message:
Add basic Bing Custom Search site search support.
Requested reviews:
Colin Watson (cjwatson)
For more details, see:
https://code.launchpad.net/~maxiberta/launchpad/bing-search
I believe most issues from feedback are fixed now. Still pending: turn related
doctests into unittests (and deduplicate lots of doctests, btw), which I'll
work on in a following branch to unlock and hopefully land/deploy this branch
ASAP.
Note this branch depends on
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/generalized-sitesearch-testservice into lp:launchpad.
Commit message:
Generalize googletestservice.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https://code.launchpad.net
Updated branch:
- Register ISearchService implementations with name="google" and name="bing"
(defaults to "google" for now; will add a feature flag next).
- Add extra doctests (sorry, copied from google implementation as we are in a
rush; will migrate to proper unit tests later).
--
Also, see a number of preparatory branches:
https://code.launchpad.net/~maxiberta/launchpad/googlesearchservice-improvements/+merge/342146
https://code.launchpad.net/~maxiberta/launchpad/rename-google-as-sitesearch-extra/+merge/342148
Replied each comment inline. Most issues are now fixed. Still missing:
- Add feature flag to switch search engine.
- Code deduplication.
- Turn doctests into proper unit tests.
- Run tests on search page with both Google and Bing.
Diff comments:
>
> === modified file
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/deduplicate-process_exists into lp:launchpad.
Commit message:
Deduplicate code into lp.services.osutils.process_exists().
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/rename-google-as-sitesearch-extra into lp:launchpad.
Commit message:
A few extra renamed classes from Google* to SiteSearch*.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/googlesearchservice-improvements into lp:launchpad.
Commit message:
A few improvements on GoogleSearchService.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https
The proposal to merge lp:~maxiberta/launchpad/bing-search into lp:launchpad has
been updated.
Status: Needs review => Work in progress
For more details, see:
https://code.launchpad.net/~maxiberta/launchpad/bing-search/+merge/341549
--
Your team Launchpad code reviewers is subscribed to
The proposal to merge lp:~maxiberta/launchpad/bing-search into lp:launchpad has
been updated.
Commit Message changed to:
Add basic Bing Custom Search site search support.
For more details, see:
https://code.launchpad.net/~maxiberta/launchpad/bing-search/+merge/341549
--
Your team Launchpad
The proposal to merge lp:~maxiberta/launchpad/bing-search into lp:launchpad has
been updated.
Description changed to:
Add basic Bing Custom Search support.
Should be pretty unobtrusive, while adding a basic site search implementation
around Bing Custom Search (shamelessly copied from the
The proposal to merge lp:~maxiberta/launchpad/bing-search into lp:launchpad has
been updated.
Description changed to:
Add basic Bing Custom Search support.
Should be pretty unobtrusive, while adding a basic implementation around Bing
Custom Search. Might need some more testing; and could use
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/bing-search into lp:launchpad.
Commit message:
Add basic Bing Custom Search support.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https://code.launchpad.net/~maxiberta/launchpad
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/rename-googlesearch-as-sitesearch into lp:launchpad.
Commit message:
Rename lp.services.googlesearch as lp.services.sitesearch.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https
Review: Approve
Wow, a 10 year old issue! Thanks!
--
https://code.launchpad.net/~cjwatson/launchpad/reject-bad-ssh-keys/+merge/339445
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/reject-bad-ssh-keys into lp:launchpad.
Makes sense, thanks.
--
https://code.launchpad.net/~cjwatson/launchpad/ptuj-repr/+merge/338559
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/ptuj-repr into lp:launchpad.
___
Mailing list:
Review: Approve
Looks good to me. One question though: why move those @properties from
`PackageTranslationsUploadJob` to its parent class
`PackageTranslationsUploadJobDerived`?
--
https://code.launchpad.net/~cjwatson/launchpad/ptuj-repr/+merge/338559
Your team Launchpad code reviewers is
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/lp-1729580-2 into lp:launchpad.
Commit message:
Use target="_blank" in extended snap build error links.
Requested reviews:
Colin Watson (cjwatson)
For more details, see:
https://code.launchpad.net/~maxiberta/la
Diff comments:
> === modified file 'lib/lp/snappy/browser/tests/test_snapbuild.py'
> --- lib/lp/snappy/browser/tests/test_snapbuild.py 2017-10-20 13:35:42
> +
> +++ lib/lp/snappy/browser/tests/test_snapbuild.py 2018-02-16 21:53:15
> +
> @@ -130,6 +130,38 @@
>
Maximiliano Bertacchini has proposed merging lp:~maxiberta/launchpad/lp-1729580
into lp:launchpad.
Commit message:
Expose extended error messages (with external link) for snap build jobs (LP:
#1729580).
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see
Review: Approve
Looks good to me.
--
https://code.launchpad.net/~cjwatson/launchpad/snap-delete-with-builds/+merge/308767
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/snap-delete-with-builds into lp:launchpad.
Review: Approve
Looks good to me. Thanks
--
https://code.launchpad.net/~cjwatson/launchpad/limit-faq-editing/+merge/303658
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/limit-faq-editing into lp:launchpad.
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/bug-feed-fix-private-team into lp:launchpad.
Commit message:
Prevent rendering of private team names in bugs feed.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
Related bugs:
Bug #1592186 in Launchpad
Review: Approve
Looks good!
--
https://code.launchpad.net/~cjwatson/launchpad/github-bugtracker-uppercase-in-names/+merge/301053
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/github-bugtracker-uppercase-in-names into lp:launchpad.
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/snap-pocket-help-icon-placement into lp:launchpad.
Commit message:
Improve placement of Snap build pocket help icon.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https
Removed ugly string concatenation in snap pocket unit test.
--
https://code.launchpad.net/~maxiberta/launchpad/snap-pocket-fix-tests/+merge/300895
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~maxiberta/launchpad/snap-pocket-fix-tests into lp:launchpad.
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/snap-pocket-fix-tests into lp:launchpad.
Commit message:
Fix broken Snap test.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https://code.launchpad.net/~maxiberta/launchpad/snap
Updated with improved wording.
Diff comments:
> === modified file 'lib/lp/snappy/browser/snap.py'
> --- lib/lp/snappy/browser/snap.py 2016-07-16 07:46:23 +
> +++ lib/lp/snappy/browser/snap.py 2016-07-20 15:26:47 +
> @@ -242,11 +242,17 @@
>
Granted, the default help link icon placement is somewhat awkward. It could be
improved I guess? (in a different branch).
--
https://code.launchpad.net/~maxiberta/launchpad/snap-pocket-help/+merge/300626
Your team Launchpad code reviewers is requested to review the proposed merge of
Review: Approve
+1! The usage of testscenarios makes tests much cleaner. Thanks.
--
https://code.launchpad.net/~cjwatson/launchpad/restore-new-snap-links/+merge/300326
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/restore-new-snap-links
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/named-auth-tokens-fix-apidoc into lp:launchpad.
Commit message:
Fix wrong apidoc format.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https://code.launchpad.net/~maxiberta
Review: Approve
Looks good to me.
--
https://code.launchpad.net/~cjwatson/launchpad/fix-snap-job-mailer-perms/+merge/300103
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/fix-snap-job-mailer-perms into lp:launchpad.
Also, added an optional `names` parameter to `Archive.getNamedAuthTokens()` to
keep uniformity with `Archive.newNamedAuthTokens()` and
`Archive.revokeNamedAuthTokens()`.
--
https://code.launchpad.net/~maxiberta/launchpad/named-auth-tokens-bulk-api/+merge/300016
Your team Launchpad code
Updated branch with fixes and improvements based on code review.
--
https://code.launchpad.net/~maxiberta/launchpad/named-auth-tokens-htaccess/+merge/32
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.
___
Mailing list:
Updated branch with fixes and improvements based on code review.
--
https://code.launchpad.net/~maxiberta/launchpad/named-auth-tokens-bulk-api/+merge/300016
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.
___
Mailing list:
Review: Approve
+1!
--
https://code.launchpad.net/~cjwatson/launchpad/fix-snap-store-upload-status/+merge/300091
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/fix-snap-store-upload-status into lp:launchpad.
Review: Approve
Looks good, thanks.
--
https://code.launchpad.net/~cjwatson/launchpad/fix-snap-job-refresh-perms/+merge/300077
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/fix-snap-job-refresh-perms into lp:launchpad.
Diff comments:
>
> === modified file 'lib/lp/archivepublisher/scripts/generate_ppa_htaccess.py'
> --- lib/lp/archivepublisher/scripts/generate_ppa_htaccess.py 2014-10-29
> 06:04:09 +
> +++ lib/lp/archivepublisher/scripts/generate_ppa_htaccess.py 2016-07-13
> 20:42:37 +
> @@ -274,6
Diff comments:
>
> === modified file 'lib/lp/soyuz/model/archive.py'
> --- lib/lp/soyuz/model/archive.py 2016-07-14 00:25:14 +
> +++ lib/lp/soyuz/model/archive.py 2016-07-14 00:25:14 +
> @@ -1985,20 +1986,57 @@
> else:
> return archive_auth_token
>
> -
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/named-auth-tokens-bulk-api into lp:launchpad with
lp:~maxiberta/launchpad/named-auth-tokens-htaccess as a prerequisite.
Commit message:
Add API for bulk creation and revocation of named auth tokens.
Requested reviews
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/named-auth-tokens-htaccess into lp:launchpad with
lp:~maxiberta/launchpad/named-auth-tokens as a prerequisite.
Commit message:
Add support for named auth tokens in ppa htpasswd creation.
Requested reviews:
Maximiliano
Review: Approve
Looks good to me! Thanks
--
https://code.launchpad.net/~cjwatson/launchpad/reduce-mp-timeouts/+merge/298967
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~cjwatson/launchpad/reduce-mp-timeouts into lp:launchpad.
Added unique constraint on ArchiveAuthToken(archive,name).
--
https://code.launchpad.net/~maxiberta/launchpad/db-named-auth-tokens/+merge/299433
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~maxiberta/launchpad/db-named-auth-tokens into
Updated with feature flag + fixes and improvements.
--
https://code.launchpad.net/~maxiberta/launchpad/named-auth-tokens/+merge/299432
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.
___
Mailing list:
Fixed the new ArchiveAuthToken index based on analysis of query execution plan.
Thanks!
--
https://code.launchpad.net/~maxiberta/launchpad/db-named-auth-tokens/+merge/299433
Your team Launchpad code reviewers is requested to review the proposed merge of
Diff comments:
> === modified file 'lib/lp/soyuz/interfaces/archive.py'
> --- lib/lp/soyuz/interfaces/archive.py2016-01-26 15:47:37 +
> +++ lib/lp/soyuz/interfaces/archive.py2016-07-07 14:30:49 +
> @@ -2074,6 +2087,57 @@
> :param dependency: is an `IArchive`
Will fix asap. Thanks!
Diff comments:
>
> === modified file 'lib/lp/soyuz/model/archive.py'
> --- lib/lp/soyuz/model/archive.py 2016-03-14 23:42:45 +
> +++ lib/lp/soyuz/model/archive.py 2016-07-07 14:30:49 +
> @@ -1948,6 +1950,53 @@
>
The proposal to merge lp:~maxiberta/launchpad/named-auth-tokens into
lp:launchpad has been updated.
Description changed to:
Add new named ArchiveAuthToken API. This is required for UA customer delivery
of kernel livepatches. See required DB changes in
The proposal to merge lp:~maxiberta/launchpad/named-auth-tokens into
lp:launchpad has been updated.
Description changed to:
Add new named ArchiveAuthToken API. This is required for UA customer delivery
of kernel livepatches. See required DB changes in
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/db-named-auth-tokens into lp:launchpad/db-devel.
Commit message:
Add ArchiveAuthToken.name; make ArchiveAuthToken.person nullable.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers): db
For more details, see
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/named-auth-tokens into lp:launchpad.
Commit message:
Add new named ArchiveAuthToken API.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https://code.launchpad.net/~maxiberta
Updated CSS to be a little less specific. Thanks!
--
https://code.launchpad.net/~maxiberta/launchpad/new-snap-select-processors/+merge/298242
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.
___
Mailing list:
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/fix-git-merge-editstatus into lp:launchpad.
Commit message:
Fix +edit-status for Git merge proposals.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
Related bugs:
Bug #1538355 in Launchpad itself: "
* Also, moved the processors selector up, between series and upload to store.
--
https://code.launchpad.net/~maxiberta/launchpad/new-snap-select-processors/+merge/298242
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.
___
* Check all available processors by default.
* Dropped `SnapSet.availableProcessors()` for populating the processors widget,
in favor of `getUtility(IProcessorSet).getAll()` to keep the same logic as
`SnapSet.new()`.
* Indent processor checkboxes to keep them aligned with the series selector.
*
Maximiliano Bertacchini has proposed merging
lp:~maxiberta/launchpad/new-snap-select-processors into lp:launchpad.
Commit message:
Add processor selection in new Snap form.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https://code.launchpad.net
* Dropped the snap.allow_new feature flag and fixed lp.snappy tests accordingly.
* Synced snappyseries names with production.
* Updated snappydistroseries rows to not be full cross product.
* Applied changes to current.sql.
Thanks!
--
The proposal to merge lp:~maxiberta/launchpad/snappy-sampledata into
lp:launchpad has been updated.
Commit Message changed to:
Enable snappy unconditionally; add snappy sample data for development.
For more details, see:
1 - 100 of 124 matches
Mail list logo
