lifelesscan has review ? https://code.launchpad.net/~lifeless/python-oops-wsgi/extraction/+merge/7196900:50
lifelessmwhudson: btw - ^ django is wsgi based yes? you might like that :)00:51
wallyworldlifeless: ^^ as a drive by, did you want to fix the ordering of the imports and __all__ declaration?01:11
lifelesswallyworld: I can, I didn't notice they were unordered?02:55
lifelesswallyworld: indeed, they look ordered to me02:55
wallyworldlifeless: the __all__ is below the imports when it should be above?02:55
lifelessthats correct02:55
lifelesspep8 '      You should put a blank line between each group of imports.02:56
lifeless      Put any relevant __all__ specification after the imports.'02:56
wallyworldal of our lp py code is the other way around then02:56
wallyworldie it has __all__ first then imports02:56
lifelesswe should fix that :)02:56
wallyworldthat's a *lot* of fixes02:57
lifelessbut easier than fixing all of pypi !02:57
wallyworldi like lp's way of doing it02:57
wallyworldeasier to see what's exported02:58
jtvIs the error reporting utility meant to be broken?  It seems to have killed dogfood.02:58
lifelessjtv: broken? no. Changed? yes.02:58
lifelessjtv: symptoms?02:58
jtvHang on, still pasting02:58
jtvlifeless: http://paste.ubuntu.com/668808/02:58
jtvNot too helpful, I'm afraid.02:59
lifelessprobably need to drop a pdb in there and have a peek02:59
lifelessI have just overhauled oops handling entirely02:59
lifelessmake sure your download-cache is up to date03:00
lifelessand that you have bin/buildout03:00
jtvDoes it require a config change by any chance?03:00
jtvI ran rocketfuel-get, which I think takes rare of the other stuff.03:01
lifelessno, no config changes03:01
lifelessjtv: can you try bin/buildout ?03:01
jtvHmm the tree ended up in an unbuilt state.  Rebuilding first.03:02
lifelesswallyworld: did you have any other comments on that mp ?03:03
wallyworldlifeless: no, looks ok to me. there's not much to it03:04
=== lifeless changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: - | Critical bugs: 238 - 0:[#######=]:256
jtvlifeless: dogfood's running now.  Does rocketfuel-get not run buildout?03:29
lifelessnot directly03:30
lifelessit runs make03:30
lifelessrocketfuel-get is unmaintained AFAIK - you're the first person I've heard using it some time03:31
wgrantIt still works fine.03:33
mwhudsoni imagine it gets run very roughly once per someone joining the lp/buying a new machine?03:37
mwhudsonoh no, that would be -setup03:37
* mwhudson hides again03:37
jtvWell, -setup runs -get.03:38
jtvNow, why didn't "make" run buildout?03:39
lifelessmwhudson: hey03:40
lifelessmwhudson: so, did you see my oops stuff ?03:40
mwhudsonlifeless: i didn't click the link, i'm afraid03:41
lifelessmwhudson: tsk!03:41
lifelessmwhudson: what do you do for error reporting / aggregation / analysis with your dango sites?03:41
mwhudsonlifeless: currently embarrassingly little03:42
lifelessmwhudson: so the goal of oops-* is to be easy to use.03:42
lifelessmwhudson: *hint* *hint*03:42
mwhudsonlifeless: i believe there's a plan to deploy http://pypi.python.org/pypi/django-sentry03:42
lifelesssentry is interesting03:51
lifelessisd are evaluating it03:51
lifelessit really operates on single log entries FWICT03:52
jtvlifeless: are you reviewing today?  I've got a small one here: https://code.launchpad.net/~jtv/launchpad/bug-824553/+merge/7182304:00
lifelessyes, and its done :)04:05
lifelessjtv: I will happily review most any day04:05
jtvThanks.  Yes, I know—but I also know how much other stuff you do and I wonder if you have enough clones in that closet of yours...04:05
nigelblifeless: wha, rocketful-get is unmaintained? I keep using it!04:59
StevenKnigelb: I do as well -- but the last thing rf-setup does is copy rf-get to /usr/local/bin. If there any other changes to rf-get later, you don't get them ...05:00
nigelbisn't sort of easier to just modify $PATH?05:02
nigelbthat way the changes do get propogated.05:02
nigelbor alternatively, symblink instead of copy.05:03
StevenKnigelb: And then when devel breaks, you can't use rf-get to fix it ...05:03
lifelessok, web.py is off the microservice list05:03
nigelbStevenK: err, why?05:03
nigelbor rather, why not05:03
StevenKlifeless: What has it done? Or not done?05:03
lifelessStevenK: it -very much- does not want oops_wsgi in the call tree05:04
lifelessStevenK: it helpfully doesn't use wsgi internally!05:04
lifelessStevenK: so there is nowhere in its stack to inject the middleware05:04
lifelessStevenK: and if you warap the stack, it does its own error handling.05:04
nigelbtry Flask? I don't know about Flask /that/ deep to know if it will work or not.05:05
nigelbIf there was some code without tests and I add some more code to it, do I write tests for the entire thing?05:12
nigelb(I'm fairly sure the answer is yes, but I could try ;) )05:14
StevenKYou aren't required to, but it would be awesome.05:15
nigelbGreat! Can I bug you for help with it? ;)05:19
StevenKIf you like05:19
lifelessnigelb: nope, flask is also overly clever.05:46
lifelessnigelb: rather than having separate concerns, its monolithic,05:47
nigelbyay ;)05:47
lifelessI will port gpgverify to paste tomorrow05:47
nigelbwrite your own wgi app then? :)05:47
lifelessnigelb: no, we use paste for loggerhead and its great05:47
lifelessa little *too* bare bones in some ways, but it doesn't get in the way.05:47
wgrantAlso, it's not CherryPy.05:51
wgrantThat's the main thing.05:51
wgrant(Loggerhead used to use CherryPy. Those days were bad.)05:52
nigelbSo, what happens to IPerson(self.request.principal) when I'm not logged in?05:56
lifelessNone IIRC05:57
nigelbhm, why is my code breaking then05:57
lifelessok, I'm kindof EODing, allergy injections suck :)05:57
nigelbuser=None for search should just work right?05:57
nigelbTypeError: ('Could not adapt', <zope.principalregistry.principalregistry.UnauthenticatedPrincipal object at 0xb3f028c>, <InterfaceClass lp.registry.interfaces.person.IPerson>)06:08
nigelbis it kosher to do a try catch?06:09
nigelbwallyworld__: around? :)06:11
lifelessshould just be self.user if you're using a form, or call_with in the API06:11
wallyworld__nigelb: hi06:11
lifelessanything else and you're doing it wrong :)06:11
nigelblifeless: no no.06:12
wallyworld__lifeless: it's not a form and it's not the ws api. it's a view06:12
nigelbYou warned me about the anonymous use case, only I don't remember how to catch it :)06:12
wallyworld__similar to HugeVocabularyJSONView for example06:12
lifelessblah, I meant view06:12
lifelessLaunchpadView sets an attribute to Just Work06:12
lifelessanyhow, I'm really not here, ciao.06:13
wallyworld__nigelb: i think in the case of anon, request.prinicpal is None06:14
wallyworld__nigelb: because LinkChecker doesn't extend LachpadView, there's no self.user06:14
wallyworld__nigelb: perhaps you should make LinkChecker a subclass of LaunchpadView06:14
wallyworld__it may then allow you to just use self.user06:15
nigelblet me do that then06:15
wallyworld__just a guess, it may work06:15
nigelbwallyworld__: \o/ works :)06:21
wallyworld__nigelb: excellent!06:21
* nigelb now needs to wwite tests06:21
nigelbwallyworld__: how do I test behavior with a public bug number and a privaet bug number?07:20
nigelbmake the bug private and then try with anonymous user?07:20
nigelbor something.07:21
wallyworld__nigelb: you need to create a private bug and log in as the user who owns the project the bug is reported against07:21
wallyworld__nigelb: with_person_logged_in is one way07:22
wallyworld__nigelb: i assume you are talking about a python unit test here07:22
nigelbso, create a person, create project with owner that person, create private bug, then logging as that user and try and try as anonymous07:22
nigelbwallyworld__: yeah, unit tests, I should have been further clear :)07:22
wallyworld__yes what you say above sounds right. do it as 2 tests07:23
nigelbalways the fun bit in LP07:25
nigelbunit testing07:25
adeuringgood mornning07:35
danilospoolie, hi, if you are still around, are you familiar with our loggerhead QA/rollout setup?08:06
pooliehi there08:09
pooliedanilos: slightly08:09
pooliewhat do you want to know?08:10
danilospoolie, well, I've landed a fix yesterday, and I would like how to best QA it, and how to ensure it gets rolled out08:10
daniloswould like *to know* :)08:10
poolieoh, ok, i think i saw your mp08:11
pooliedo you know if that revision is live on qas?08:11
poolieor staging?08:11
danilospoolie, how can I tell that? :) (I assume that means it's not)08:13
poolieone trick here is that there's only actually back-end storage for lp:bzr on qas and staging08:14
pooliei think08:14
poolieat any rate lots of branches are in the db but will break if you try to touch them08:14
pooliefor disk space reasons08:14
lifelesspushing tp +junk is ideal08:15
danilospoolie, right, I know that, but I should be able to push stuff to lp://staging/, right?08:15
pooliedanilos: so, does landing something into lp:loggerhead cause it to be rolled out?08:16
pooliei don't think so but i'm not sure08:16
danilospoolie, I have no idea, those are the things I am trying to find out, but I doubt it08:16
poolielet's see if there's version info08:17
pooliethe losas are sprinting but they should be awake now08:17
spmpoolie: hi08:17
lifelesspoolie: no, you need to update the version ref in lp08:18
pooliespm, hi, can you tell me if normal deployments to qas pull in loggerhead from trunk?08:18
poolieoh ok, i thought so08:18
lifelesspoolie: the qa deployment includes loggerhead, but not trunk08:18
pooliedanilos: so, i think you need to make an additional landing into lp:launchpad for it to be live, like that08:18
pooliein fact it says 1.18 on the web page, and isn't trunk later than that?08:18
pooliedanilos: does that help at all?08:20
danilospoolie, yeah, trunk does have a few revisions compared to what I have in sourcecode/loggerhead08:21
poolieok trunk still says it's v 1.1808:21
danilospoolie, yeah, I figure I might need to land to loggerhead through PQM to ~launchpad-pqm/loggerhead/devel08:21
spmpoolie: sorry, quite distracted atm. have your questions been answered satisfactorily?08:22
poolieyes, thanks08:22
poolieenjoy your brown sauce08:22
daniloslifeless, do you know if I can just bump the LP loggerhead to trunk (I see issues like bug 826082 in a bit of indeterminate state), or if I should just include my own revision that I want rolled out?08:27
_mup_Bug #826082: BadUrl from safe_open opening a branch <oops> <regression> <Launchpad itself:Triaged> < https://launchpad.net/bugs/826082 >08:27
pooliedanilos: i would like to get lp running trunk08:28
poolieif necessary by reverting stuff that's unstable08:28
pooliei don't have an detailed opinion about that specific bug08:28
danilospoolie, so would I, but I ain't sure what are those08:28
danilospoolie, so, I followed the trail of bug 820065 and bug 812583, which got me to the above bug08:29
_mup_Bug #820065: TypeError: unsupported operand type(s) for -: 'int' and 'NoneType' <oops> <qa-untestable> <Launchpad itself:In Progress> <loggerhead:Fix Committed by pr0gg3d> < https://launchpad.net/bugs/820065 >08:29
pooliewell it looks like that might be fixed08:29
_mup_Bug #812583: IndexError in add_template_values in loggerhead annotate <loggerhead> <oops> <qa-untestable> <Launchpad itself:In Progress> <loggerhead:Fix Committed by pr0gg3d> < https://launchpad.net/bugs/812583 >08:29
lifelessso, we rolled back loggerhead after deploying coincided with a 1000 oops a day increase.08:29
lifelessthat increase appears to be jelmers safe-open refactoring08:29
lifelesswhich he was fixing08:29
lifelessIn the spirit of one-change-at-a-time, I think we should get that sorted, then upgrade again.08:30
lifelessif jelmers patch is landed, then upgraded loggerhead should be pretty straight forward08:30
daniloslifeless, jelmer's patch is not landed in either loggerhead trunk or Launchpad08:30
pooliehe should be up soon and we can ask him08:31
danilos(that would be clearer if it was "is landed in neither...nor" :)08:31
jelmerit was in ec2 overnight but appears to have failed08:31
lifelessdanilos: it would be in launchpad08:31
danilospoolie, lifeless, any suggestions on where should I put the docs for developers wanting to QA/deploy loggerhead? (something that'd be easy to find, I found no pages about it on dev.launchpad.net and wiki.canonical.com)08:33
lifelessdanilos: README.txt in loggerhead ?08:33
daniloslifeless, I was more thinking in terms of Launchpad QA/deployment08:34
lifelessdanilos: I know its a bit LP specific, but you could note that this is the primary place its deployed :)08:34
lifelessdanilos: otherwise, yeah somewhere on d.l.n.08:34
wgrantEw, that does not belong in the loggerhead tree.08:34
daniloslifeless, right, fair enough, though I'll add a few pages titled /Loggerhead to the dev wiki pointing at the readme then08:34
pooliei think dev.l.n08:35
lifelessdanilos: its only a though, wiki is fine too08:35
danilosso, I guess d.l.n is the consensus choice here, thanks all for chipping in :)08:35
pooliehow to run or deploy it in general belongs in the tree; not the quirks of lp deployment though08:35
lifelessthe reason I think its reasonable to put it in the loggerhead tree is we want to run trunk08:35
poolieif you ask me08:35
pooliepartly becaues there's much less chance losas will touch that08:35
lifelessif we want to keep trunk deployable, we should have the instructions for that clear and visible to all contributors08:36
pooliedanilos: so, ping me or someone else if you want more08:44
danilospoolie, lifeless: any fixes/improvements welcome on https://dev.launchpad.net/Code/Loggerhead, I am sure I got some bits of the process wrong08:49
danilosI've also removed a reference to inactive project launchpad-loggerhead from dev.l.n/Code and pointed at this page instead08:53
danilosjelmer, so, I assume I'd better wait for you to land your changes and get them deployed before I try that with my changes to loggerhead08:57
danilosjelmer, I am sure you might have a few suggestions for https://dev.launchpad.net/Code/Loggerhead08:57
danilosa buildbot failure in an unrelated test :(09:00
jelmerdanilos: Looking at the test failures right now..09:05
danilosjelmer, cool, thanks09:05
danilosjelmer, though, I was referring to a spurious buildbot failure for my branch :)09:05
daniloslifeless, since I am lazy, what was the policy for triaging bugs about spurious test failures? critical? (bug 828584)09:06
_mup_Bug #828584: Spurious test failure: test_sigint_exits_nicely <Launchpad itself:New> < https://launchpad.net/bugs/828584 >09:06
poolievila, jam, so i wonder how we could unstack it safely09:09
jampoolie: reconfigure --unstacked isn't safe?09:10
jamIf you want the quick bzrlib invocation09:11
jamjust open the repository directly09:11
jamwhich fills in everything09:11
vilajam: without unstacking right ?09:11
jamvila: if you do Repository.open() instead of Branch.open().repository09:11
pooliejam, oh, i mean it _should_ be safe09:11
jamthe repo won't be stacked09:11
pooliemy question was really, is it safe, and how could we mitigate any risks09:12
jamand .fetch() without a search recipe or target revision09:12
jamfetches everything09:12
jampoolie: I think it is safe, but if you want to do it easily Repo.fetch(repo) works just fine09:12
pooliedanilos: that looks pretty good; though i'd like to know whetehr things automatically go onto buildbot, *staging, etc09:13
pooliei guess you would too :)09:13
jamSpeaking of which, I just did: jameinel@chinstrap:~$ time bzr branch lp:gcc-linaro/4.6 lp:~jameinel/gcc-linaro/test-4.609:13
jamUsing default stacking branch /+branch-id/355386 at bzr+ssh://bazaar.launchpad.net/~jameinel/gcc-linaro/09:13
danilospoolie, exactly :)09:13
jamAnd it was still transferring at 500MB or so when I killed it.09:13
jambut you know, that could be the "created a branch and didn't populate it" or something like that.09:14
jamI'll try again with debug flags set09:14
=== almaisan-away is now known as al-maisan
jampoolie: very strange "bzr branch lp:gcc-linaro/4.6 lp:~jameinel/gcc-linaro/test-4.6" is at about 1GB transferred so far.09:32
jamthe discovery phase was fast, but the bytes transferred seems really wonky09:33
jamthere is only like 100MB in /4.6, so it doesn't line up at all, unless something is broken with the target stacking.09:34
jambut the request says that it is trying to stack.09:34
jamthough this is bzr-2.1.4... because it was the binary on Chinstrap09:35
jamnot sure how much that matters or not.09:35
jamIt had "transferred" 1.2GB (probably down and then up again), and there was 465MB in the target. Still too much.09:37
=== allenap changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: allenap | Critical bugs: 238 - 0:[#######=]:256
rvbaallenap: Can you please review this MP https://code.launchpad.net/~rvb/launchpad/overlay-bug-827220/+merge/71915 ?10:16
allenaprvba: Sure.10:24
rvbaallenap: : Thanks.10:24
lifelessdanilos: thats fine I think10:25
lifelessI will look at the docs tomorrow :)10:25
allenaprvba: Ah, interesting, I would have suspected that you'd need to use getComputedStyle(). r=me/10:25
daniloslifeless, please extend them with as much as you know how stuff gets to *staging, what gets run in the buildbot and similar :)10:26
rvbaallenap: looks like getStyle is enough ...10:27
allenaprvba: Cool.10:27
henningelifeless: Hi!10:40
henningelifeless: about python fixtures10:41
henningelifeless: I don't see where we include it10:45
lifelesshenninge: setup.py10:45
lifelesshenninge: or do you mean inside LP? grep 'Fixture' lib -r.10:46
henningelifeless: yeah, in LP10:46
henningebut I just saw that it is called "fixtures", let me look10:46
henningelifeless: ah ok, I was looking for python-fixtures10:47
lifelesshenninge: grep Fixture)10:47
henningeit's in download-cache10:47
lifelesswill show where its used.10:47
henningeno, I was not looking for that, I have used it in tests myself ...10:47
lifelessat least directly; it won't show indirectly like in python-gbouncer, the rabbit notifier etc10:47
henningeI just wanted to see which version we used10:47
lifelesshenninge: oh, :) I get you.10:47
henningelifeless: so you are saying fixtures will need an update to work with the latest testtools?10:48
lifelessyes, incompatible change to gather_details10:48
henningeprobably the same that I ran into10:49
henningebecause the output changed10:49
lifelessrev 207 in testtools10:49
lifelessbreaks fixtures10:50
lifelessfixed in rev 29 of fixtures10:50
henningelifeless: that's what I was trying to get at.10:50
henningeso I don't need to fix fixtures, I just need to package the latest version of it.10:51
lifelesswgrant: yes10:52
lifelesshenninge: yes10:52
henningelifeless: thanks, that's much easier of course ;-)10:53
=== al-maisan is now known as almaisan-away
=== henninge is now known as henninge-lunch
=== almaisan-away is now known as al-maisan
=== henninge-lunch is now known as henninge
Ursinhaguys, anyone up to help me to understand what exactly launchpad understands when I say ubuntu.searchTasks(bug_supervisor=oneteam) ?12:31
jmlUrsinha: sure.12:34
Ursinhaafaik, packages don't have bug supervisors, only ubuntu does12:35
Ursinhabut using that returns me different tasks for different packages12:35
UrsinhaI can't ubuntu.searchTasks(structural_subscriber=oneteam) to compare, because that times out12:36
jmlUrsinha: just found http://paste.ubuntu.com/669193/ in bugtask.py12:37
Ursinhabdmurray: ^12:38
Ursinhajml: so it should work pretty much as structural_subscriber does?12:38
jmlUrsinha: I'm still grokking it. one sec.12:38
Ursinhajml: sure12:39
jmlUrsinha: if I understand correctly, it'll match any package that oneteam is subscribed to12:40
jmls/any package that/any bugtask on a package that/12:40
jmlas well as any bug task on ubuntu itself12:40
Ursinhathat's exactly what we need, and also what we tried to accomplish with structural_subscriber but it times out now...12:41
Ursinhabdmurray: so we can use bug_supervisor12:41
jmland it doesn't do team unrolling12:41
jmlexact match of user ids12:41
_mup_Bug #191809: A DistributionSourcePackage needs a bug supervisor role to control permissions <lp-bugs> <Launchpad itself:Triaged> < https://launchpad.net/bugs/191809 >12:41
nigelbmrevell: hi12:42
mrevellHey nigelb12:43
Ursinhathanks jml :)12:43
nigelbmrevell: hey, did you get a chance to look at that bug? :)12:43
jmlUrsinha: np12:43
mrevellnigelb, Not yet. I'm at an event today. I'll be back tomorrow.12:45
nigelbmrevell: cool, ok :)12:45
mrevellgary_poster, nigelb is interested in bug 816105. As your squad worked on the feature, I'd love to hear your thoughts on that bug report.12:46
_mup_Bug #816105: UI for subscribing to a bug is confusing <bugs> <ui> <Launchpad itself:Triaged> < https://launchpad.net/bugs/816105 >12:46
gary_postermrevell, any solution needs to support the new notification levels.  IMO, a solution should also communicate if "[y]ou have subscriptions that may cause you to receive notifications".12:55
gary_posterA more perfect solution IMO (also fixing another related UI bug filed by mpt, I think) would also remove the necessity for the "Edit bug mail" link by making the notification of "[y]ou have subscriptions that may cause you to receive notifications" have a link to an in-page overlay that explains what is meant and what can be done about it other than muting.12:55
gary_posterSpecifically, this would be the "Other subscriptions" information on the "Edit bug mail" page included as an overlay on the bug page.  That information should only be loaded dynamically--it is somewhat expensive to calculate and +index is already slow enough.12:55
gary_posterI suspect that the "Other subscriptions" code is factored well enough that this would not be too tricky to do.12:55
* nigelb reads the wall of text12:55
StevenKnigelb: It's even more awesome when you talk to gary_poster IRL. Since then it's a wall of speech. :-P12:56
nigelbStevenK: haha12:56
gary_posterha.  ha.  :-)12:56
nigelbgary_poster: my main irk was this. it was way more easier to find how to subscribe others than subscribe myself.12:57
gary_posternigelb, I can see what you mean.12:57
wallyworld__sinzui: both new picker feature flags are now on lp.net12:59
gary_posternigelb, I believe it is mrevell's responsibility to actually decide how it should work, or at least delegate and approve it.  The above is my take.12:59
nigelbgary_poster: 'm still parsing that ;)12:59
wallyworld__seems to work well13:00
gary_posterthe "wall" or the mention of mrevell as the responsible party as acting product mgr?13:00
mrevellgary_poster, nigelb: Thanks both. I'll return to this tomorrow, when I'm back from the QA CoP sprint.13:00
nigelbgary_poster: heh, the wall ;)13:01
gary_posternigelb, mrevell, the mistake we made IMO was not not take this part through user testing.  It was at the end of the project and we were in a rush.  Whatever is done, I suggest you not make our mistake, and actually produce mockups and run user tests.13:02
nigelbgary_poster: I agree there. I'm only helping implement. Like I said the other day, I think the code bit is probably triivial. the UX bit is not.13:04
=== jcsackett changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: allenap, jcsackett | Critical bugs: 238 - 0:[#######=]:256
abentleysinzui: I have a question about a distro in Launchpad: https://launchpad.net/cinux13:52
sinzuiask away13:52
flacostehello everyone!13:58
deryckHi, flacoste.  Welcome back!14:00
flacostehi deryck!14:00
deryckadeuring, hi, shall we mumble now?14:02
adeuringderyck: ah, sure!14:02
bachi flacoste, welcome back to the land of the internets14:10
flacostehi bac!14:10
=== bigjools-afk is now known as bigjools_
stubBah. We can't logout from the appserver layer as it redirects through secure codehosting to clear tokens before ending up on the openid side.14:14
stubNo tests for logout :-(14:14
gary_posterwelcome back flacoste14:35
flacostehi gary!14:35
henningederyck, abentley: I am really unsure with this javascript test. I am not sure how and what to test, maybe taht's why I keep finding excuses to do something else ... ;-)15:04
henningeI have this now: http://paste.ubuntu.com/669304/15:04
abentleyhenninge: looking.15:05
henningeIt's just checking that the handler works properly.15:05
abentleyhenninge: Okay, do you want to mumble about it?15:06
henningeabentley: sure15:07
=== al-maisan is now known as almaisan-away
=== deryck is now known as deryck[lunch]
cjwatsonCould somebody please try landing https://code.launchpad.net/~cjwatson/launchpad/dpkg-xz-support-619152/+merge/32868 ?  The necessary python-apt and apt-utils packages are now installed on production16:32
cjwatson*try ... again16:32
=== deryck[lunch] is now known as deryck
=== beuno is now known as beuno-lunch
sinzuijcsackett, do you have time to mumble?17:40
abentleyallenap or jcsackett: could you please review https://code.launchpad.net/~abentley/launchpad/twisted-logging/+merge/72082 ?17:46
jcsackettabentley: i'm otp now, but can look after that.17:46
abentleyjcsackett: thanks.17:46
jcsackettabentley: looking at your branch now.18:02
abentleyjcsackett: great.18:03
abentleysinzui: is bug 828493 related to your current work?18:05
_mup_Bug #828493: Bogus results in package search <Launchpad itself:New> < https://launchpad.net/bugs/828493 >18:05
cr3anyone know if there's a ui bug open against launchpad about the watermark and maincontent header being repetitive, like bugs.launchpad.net/checkbox showing "Checkbos System Testing" and "Bugs for Checkbox System Testing" respectively?18:06
sinzuiNo, It is a duplicate of an old bug my team will fix18:06
sinzuiabentley, I will find the master and mark it as a dupe18:06
abentleysinzui: rawk.18:07
jcsackettabentley: looks good. r=me.18:09
abentleyjcsackett: thanks.18:09
=== beuno-lunch is now known as beuno
=== allenap changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: jcsackett | Critical bugs: 238 - 0:[#######=]:256
sinzuijcsackett, are you about to do a code review?21:05
mtaylorspm: around?21:23
mtaylorspm: if I did a staging import of bug a few weeks ago, and now I'm ready to go live, but re-ran the script to get more recent bugs - do we need to do another staging import? and do you want me to open a new ticket? or re-use the old one?21:24
sinzuimtaylor, Another staging import is not needed, but someone, maybe spm, needs to run out preprocessor over your data to generate the bug nicknames. You need to post the URL to the updated data or send someone an email with the attachment.21:29
mtaylorsinzui: k. awesome. https://answers.launchpad.net/launchpad/+question/16846321:31
mtaylorsinzui: btw- I now have a script that can export bugs from a github project and generate the xml if you guys want a copy of it21:32
mtaylorsinzui: https://github.com/openstack/openstack-ci/blob/master/gh_issues_to_lp_bugs.py21:33
mtaylorsinzui: the lack of great way to map users and the lack of readily accessible email addresses sort of blows, but other than that it's workable at least21:33
sinzuimtaylor, thanks21:34
sinzuiThe user/email issue also happens when we import a mailing list. We can do it, but the feature was never exposed in the UI because of the troubles21:35
sinzuiI am creating the preprocessed xml file and updating the bug so that a system admin can do the real import21:37
=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away
mtaylorsinzui: thanks!21:52
=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away
=== jcsackett changed the topic of #launchpad-dev to: https:dev.launchpad.net/ | On call reviewer: - | Critical bugs: 238 - 0:[#######=]:256
lifelessmtaylor: cool22:16
lifelessjelmer_: hi22:16
lifelessjelmer_: did the loggerhead fix land overhnight ?22:17
jelmer_g'morning lifeless22:17
jelmer_lifeless, r1372622:18
lifelesssinzui: around ?22:31
sinzuiI am22:31
lifelessI'd like to understand this 'spying' thing more22:31
lifelesswhats the best way for me to do that ?22:31
sinzuiSince a private team cannot subscribe to a bug or branch you cannot22:32
sinzuiput if someone could. they could be subscribed to all of canonical's projects and the owners could not see that.22:32
lifelessok, so its not a feature. Its a potential antifeature ?22:32
lifelessI will follow up to the bug to note that22:33
sinzuiwe talked about fixing the subscription rules for bug supervisor and security contact a few days ago. Doing that will address most of the issue.22:33
lifelesshow does that connect?22:34
sinzuiWe would also need a rule when retargeting to a private project to change the subscribers.22:34
lifelessso the structural subs won't need any handling : they are per project anyhoew22:35
lifelessexplicit subs to the bug might need to be carried over22:35
sinzuiBug.setPrivate() converts the structural subscriber to a real subscriber. the private subscription would not be visible22:35
lifelessor ditched22:35
lifelesssetPrivate shouldn't do that22:35
lifelessits a bug22:35
sinzuiyes, that was the gist of our conversation22:36
lifelessok, I'm assuming that that is fixed in this discussion :)22:36
lifelessbeyond that, we have explicit subscriptions to consider22:36
sinzuiI think the set-based rules to determine who should be subscribed on transitions to private/security will work.22:37
sinzuiWe need to ensure those rules are in play when a bug it retargeted to a private project22:37
lifelessuhm, let me look for the other bug22:38
lifelessit was something like 'need to ask who should be subscribed to a bug when it is made private'22:38
lifelesswhich I think was duped on the strucsub issue, but actually shouldn't have been22:38
sinzuiThe person who can make a bug private does not necessarily know who should be subscribed. I could subscribe someone who the owner do not want subscribed22:41
lifelessthe person who makes it private gets a subscription themselves.22:41
lifelessthey can then subscribe anyone.22:41
lifelessso they can violate the owners desire anyway [given the current visibility rules22:41
lifelessok, but it wasn't a dupe22:43
lifelessit was assuming implicit subscriptions being carried across was broadly desirable.22:43
lifelessI think there is a latent bug here but we can ignore it :)22:43
lifelesssinzui: bug https://bugs.launchpad.net/launchpad/+bug/82790222:43
_mup_Bug #827902: Private teams not able to subscribe to bugs or blueprints for email updates <disclosure> <privacy> <teams> <Launchpad itself:Triaged> < https://launchpad.net/bugs/827902 >22:43
lifelesssinzui: my comments 2 and 3 - could you confirm I understand correctly?22:43
sinzuilifeless, that is a good call on the rule to reveal the team's name22:47
lifelessthanks !22:48
lifelessits nice when a plan comes together22:48
lifelesssinzui: and my neurons just rubbed together :) - cgregan linked the team in the report ;)22:53
lifelesssinzui: so the rules we had previously put together are sufficient.22:53
lifelesspoolie: when do you want to catch up ?22:53
lifelesspoolie: ping ;)23:07
pooliehi lifeless, i was on the phone23:26
lifelessno worries23:26
pooliehow about if i make some coffee and we talk in 15m?23:26
cjwatsonCould somebody please try landing https://code.launchpad.net/~cjwatson/launchpad/dpkg-xz-support-619152/+merge/32868 ?  The necessary python-apt and apt-utils packages are now installed on production23:31
StevenKcjwatson: Certainly.23:34
cjwatson(I have another one after this, but this one is probably simpler)23:35
sinzuiStevenK, wallyworld__ http://www.staff.uni-mainz.de/neuffer/scsi/fun.html23:38
sinzuiwallyworld__, https://code.launchpad.net/~sinzui/launchpad/code-review-affiliation/+merge/7210623:40
=== wallyworld__ changed the topic of #launchpad-dev to: https:dev.launchpad.net/ | On call reviewer: wallyworld* (jtv) | Critical bugs: 238 - 0:[#######=]:256
StevenKcjwatson: Now that I'm off the phone -- your branch is in ec2, you should get a mail in ~ 4 hours.23:48
StevenKcjwatson: Out of interest, has that branch been QA'd on dogfood before landing?23:49
cjwatsonit has not AFAIK23:50
pooliehi lifeless23:50
cjwatsonStevenK: ^-23:50
cjwatsonStevenK: mawson doesn't yet seem to have the relevant apt-utils version, oddly23:51
cjwatsonStevenK: I'll whine at RT about that23:51
StevenKcjwatson: mawson is a law unto itself, sadly.23:51
StevenKcjwatson: I would be nice to QA it on qastaging, but asuka doesn't run any of the required services.23:52
cjwatson(mawson == dogfood, right?)23:52
StevenKcjwatson: Right23:52
cjwatsonI've sent a whinge to RT#4713223:53
_mup_Bug #47132: Documentation index not available -- yelp confused <bzr (Ubuntu):Fix Released> <bzr (Debian):Fix Released> < https://launchpad.net/bugs/47132 >23:53
cjwatsonTBH, for the .tar.xz branch, I can't see how it would break anything until/unless somebody uploads something .data.tar.xz-ish23:54
StevenKcjwatson: Right, so my QA plan for the branch on mawson was upload a 'normal' package and then a different xz-using package and see that they both get processed correctly.23:56

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