/srv/irclogs.ubuntu.com/2017/07/11/#launchpad.txt

=== maclin1 is now known as maclin
=== chihchun_afk is now known as chihchun
=== gurmble is now known as grumble
=== chihchun is now known as chihchun_afk
=== JanC is now known as Guest50617
=== JanC_ is now known as JanC
rbasakWho can push to a git repo? The team owner, or the "repository reviewer", or both?15:48
rbasakOr something else?15:48
cjwatsonrbasak: Just the owner of the repository, or anyone in that team if the repository owner is a team.  (Also currently Launchpad admins, but that's a bug.)15:59
rbasakThat's useful. Works well for us. Thanks!15:59
cjwatsonrbasak: Planned future change: the default repository for a source package in a distribution (i.e. lp:ubuntu/+source/PACKAGE) should be pushable-to by anyone who can upload to the corresponding source package; but that's not done yet.16:00
rbasakWe don't want that - unless we can further filter that by ref wildcard.16:01
rbasakIOW, we want an uploader of a package to be able to push refs/tags/upload/*, but nothing else.16:01
rbasakAnd only the importer should be able to push refs/*16:01
rbasakOtherwise an uploader could cause the git repository to mismatch the archive's single source of truth16:02
cjwatsonRef-wildcard ACLs have long been planned though are also not yet implemented.16:02
rbasakDo you have a plan for us to be able to combine those two things though?16:03
cjwatsonIn general it sounds like we need a bit of an uplift in policy capabilities to match those requirements.16:03
cjwatsonThey aren't planned in a lot of detail.16:03
rbasakUnderstood, thanks16:03
cjwatson(If you need it any time soon you probably need to go through my management chain so I get some time ...)16:04
nacccjwatson: did you have any further thoughts on where I could debug the JSONDecodeError I was hitting yesterday? I'm getting it pretty consistently now (and if i ignore it, the review does get requested correctly)16:36
cjwatsonnacc: if it's returning an empty byte-string then that sounds like an LP bug; it might be worth seeing if you can construct a matching situation in the LP test suite16:42
cjwatsonI don't see anything obviously wrong with that method16:42
nacccjwatson: will do, thanks!16:43
cjwatson(sorry, I realise that's quite a big next step, but ...)16:44
nacccjwatson: not a problem! I know you've got a lot on your plate -- I think for now I can handle that exception in our script and i'll note it down as a bug to follow-up on my end16:44
=== santa is now known as Guest89941
=== Guest89941 is now known as santa_

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!