StevenKcjwatson: r=me00:03
cjwatsonGreat, thanks00:04
=== slank is now known as slank_away
StevenKcjwatson: Do you want to cowboy it onto mawson while it plays through buildbot, or what is your plan?00:04
cjwatsonIf you have bright ideas on disentangling the underlying cause (notes in the bug), that'd be lovely, or else I'll try to mock something up in tests00:05
cjwatsonStevenK: I was about to go to bed, TBH00:05
StevenKcjwatson: If you want to crash, I can land it for you00:06
cjwatsonWill it be a problem if I don't QA it until tomorrow afternoon?  I'm going to be out tomorrow morning (hence wanting to go to bed)00:06
cjwatsonWell, I can land it00:06
StevenKcjwatson: We have one revision waiting, and nothing urgent, so it should be fine00:06
cjwatsonJust QA will be with slightly less alacrity than usual00:06
wgrantGiven that we still have no builders and the change is isolated and trivial, QA on that seems less than essential.00:07
cjwatsonAs I suggested in the MP, we can QA this without builders by injecting a production upload00:07
cjwatsonProbably quicker too00:07
cjwatsonThough I guess we need to mock up the build id in the directory name or something to get it into the right archive, but that's easy enough00:08
cjwatsonAnyway, it's on its way into buildbot, so I'll deal with that tomorrow; feel free to revert if it causes a problem, obviously00:09
StevenKwgrant: How goes the BFJO murder?03:37
wgrantI think all the failures are fixed03:38
wgrantAnother ec2 run is 3/4 done03:38
StevenKAh, how bad was the ec2 carnage?03:38
wgrantOnly about 60 failures.03:38
StevenKNot bad.03:39
wgrantStevenK: Looking03:47
wgrantStevenK: Does the context of that specificationdependency.py change reveal another place that needs to respect privacy?03:50
wgrantAlso, have you confirmed that this actually fixes the blocking issue?03:53
StevenKI've confirmed it passes tests03:54
StevenKShall I cowboy it onto mawson?03:54
wgrantIf you can reproduce it on mawson today, that would indeed be a good idea03:55
StevenKReproduce the problem?03:55
wgrantBut the existing tests do not catch the blocking issue03:55
lifeless16:09 < ajmitch> heh04:04
lifeless*Bah* copy-paste fail.04:04
StevenKwgrant: https://blueprints.dogfood.launchpad.net/production-auditor/+spec/spec-c04:48
StevenKI think I need to revert the change to make use of all_deps, rename dependencies to _dependencies, and write a method called dependencies that filters by visibility04:50
wgrantStevenK: Why not just make dependencies filter by visibility?04:50
StevenK1) You need a user, and 2) dependencies is a SQLRelatedJoin04:51
wgrantDoes dependencies need to be an SQLRelatedJoin?04:51
wgrantHm, but if it's an SQLRelatedJoin then it shouldn't recurse04:52
wgrantSo how's it relevant?04:52
StevenKWe don't want it to recurse for the dep tree04:52
wgrantOh, does it only show immediate deps?04:52
StevenKThat is exactly what is causing bug 109523504:52
_mup_Bug #1095235: Bogus dependencies in Blueprint graph <lp-blueprints> <regression> <specifications> <Launchpad itself:Triaged> < https://launchpad.net/bugs/1095235 >04:52
StevenKYeah, dependencies and blocked are immeadiate children and parents04:53
wgrantSure, but I thought the graph showed more than that04:53
StevenKr16333 changed the dep tree to use all_deps and all_blocked, which is the entire tree04:53
StevenKwgrant: It uses dependencies and blocked for each node04:54
StevenKspec-c all_blocked shows 'spec-a' and 'spec-b' since it's recursive04:54
StevenKspec-b all_blocked shows 'spec-a'04:54
wgrantStevenK: So it just uses the recursive query to grab all the nodes, and then queries them all anyway?04:54
wgrantIf not, then what's the recursive query for in the first place?04:55
StevenKI'm not sure, but the deptree didn't used to use them04:55
* StevenK finally figures out this traceback06:05
StevenKlib/lp/_schema_circular_imports.py, I will DESTROY you06:06
=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away
=== yofel_ is now known as yofel
adeuringgood morning08:51
czajkowskiadeuring: morning08:52
adeuringhi czajkowski!08:52
czajkowskiadeuring: cold and snowing over there?08:54
adeuringczaqright, we have snow :) But "cold" is more a question of peception. It's not like in sebreia here ;)08:55
czajkowskiwe have snow too :)08:56
czajkowskipoxy snow :(08:56
StevenKczajkowski: It's currently 29 here ...08:59
lifelessczajkowski: it was 45 a week ago09:01
czajkowskitad warm :)09:02
=== matsubara-afk is now known as matsubara
=== almaisan-away is now known as al-maisan
=== Ursinha-afk is now known as Ursinha
=== Ursinha-afk is now known as Ursinha
=== slank_away is now known as slank
=== matsubara is now known as matsubara-lunch
cjwatsonwgrant,StevenK: Review of https://code.launchpad.net/~cjwatson/launchpad/avoid-copy-archive-spam-2/+merge/144727 would be good when you have a chance (and now I'm glad I bothered to QA that)15:32
=== matsubara-lunch is now known as matsubara
=== al-maisan is now known as almaisan-away
=== benji___ is now known as benji
=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away
=== matsubara is now known as matsubara-afk
plommeI was wondering if there was anything new regarding bug 1100164 . I know you guys are crazy busy, just curious how far back this has been pushed.19:21
_mup_Bug #1100164: Private projects are forbidden from having releases when only files are problematic <privacy> <private-projects> <releases> <Launchpad itself:Triaged> < https://launchpad.net/bugs/1100164 >19:21
=== almaisan-away is now known as al-maisan
=== slank is now known as slank_away
=== slank_away is now known as slank
=== al-maisan is now known as almaisan-away

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