/srv/irclogs.ubuntu.com/2010/10/20/#launchpad-dev.txt

LPCIBotProject parallel-test build (7): STILL FAILING in 2 hr 9 min: https://hudson.wedontsleep.org/job/parallel-test/7/00:06
marswgrant, did I hear you say you could reproduce the windmill threading error locally?00:13
marsor something similar00:13
lifelessmars: bug 46176000:15
_mup_Bug #461760: Shouldn't check for left-over threads between Windmill tests <story-windmill-layer> <Launchpad Foundations:Fix Released by bjornt> <https://launchpad.net/bugs/461760>00:15
lifelessq00:18
marslifeless, thaks00:20
marsthanks00:20
abentleylifeless: when I have claimed a review, feel free to review it yourself, but please don't set the status "approved" without letting me get my 2 cents in.00:29
lifelessabentley: ok, can do00:30
lifelessI thought you had EOD'd00:30
lifelessand curtis had addressed your point00:30
abentleylifeless: thanks.00:30
abentleylifeless: I have EOD'd.  Curtis should have also EOD'd by now.  I would have reviewed it tomorrow, before he clocks in.00:31
lifelessdid you have other unaddressed concerns?00:32
abentleylifeless: I'm a little worried by what he wrote about the diff being a lot bigger due to alphabetical sorting.00:34
abentleylifeless: It's a principle thing.  If I claim a review, it means I intend to do it.00:34
wgrantmars: I have had several EC2 runs fail with a LayerIsolationError after test_slow_threads.00:34
wgrantmars: It manifests itself as a "make check returned an error code, but no tests failed" sort of thing, which you have to dig through the subunit log for.00:35
lifelesswgrant: mars: I can trigger that by just running it; OTOH I'm futzing with the infrastructure in my branch, so its not necessarily representative00:35
lifelessabentley: I've acked your request; I'm sure you were doing the review.00:35
abentleylifeless: I don't follow.00:36
lifelessabentley: on the thing you're worried about, I've noticed many imports in lib/canonical were not normalised by edwins script00:36
lifelessabentley: I've said 'ok, I won't set the status on reviews you've claimed'00:36
wgrantmars: Now I think about it, my log in bug #661931 might well be that issue.00:38
_mup_Bug #661931: make check sometimes fails on EC2 without test failures <Launchpad Foundations:Triaged> <https://launchpad.net/bugs/661931>00:38
* wgrant checks.00:38
wgrantIndeed it is.00:38
abentleylifeless: you went on to discuss the proposal as if I was concerned about the specific state of the proposal, so I wanted to make it clear that the specific circumstances of the proposal weren't my motivation.00:38
lifelessabentley: ok; I understood you to be making a generic request.00:38
lifelessI was also interested if in the specific case I had missed something important to the review.00:39
rockstarwallyworld__, awesome.00:39
lifelesse.g. 'could I, if I had been the original reviewer, have missed something important'00:39
wallyworld__rockstar: i don't know why it fails but the change brings the tests in line with how others have been written. it seems the flakiness on *my* system has been solved at least00:40
rockstarlifeless, I hate windmill's random failures.00:40
rockstarwallyworld__, yeah, the flakiness may or may not be fixed for other systems.00:40
abentleylifeless: Aside from the "this is much bigger than before" thing, I'm not aware of anything more to check.00:41
wallyworld__rockstar: i'm running it through ec2 to see what happens. consider it an experiment :-) i'll let you know how it goes00:41
lifelessgrrr zcml01:11
LPCIBotProject devel build (134): STILL FAILING in 3 hr 32 min: https://hudson.wedontsleep.org/job/devel/134/01:54
LPCIBot* Launchpad Patch Queue Manager: [r=allenap,01:54
LPCIBotbac][ui=none][no-qa] extract some refactorings from the abandoned01:54
LPCIBotcheck-in-wadl branch01:54
LPCIBot* Launchpad Patch Queue Manager: [r=adeuring][ui=none][bug=126516] Refactor doctest for bugtask status01:54
LPCIBotchanges into separate unit test and documentation.01:54
LPCIBot* Launchpad Patch Queue Manager: [r=rockstar][ui=none][incr][bug=663436] Automatically determine (and01:54
LPCIBotactually enforce) the expiration date of an OAuth request token,01:54
LPCIBotfreeing up the expiration_date field to be used for other purposes.01:54
LPCIBot* Launchpad Patch Queue Manager: [r=gmb][ui=none][bug=636060] Adds latest upstream release version to01:54
LPCIBotthe sourcepackage upstream portlet.01:54
LPCIBot* Launchpad Patch Queue Manager: [r=rockstar][ui=rockstar][bug=652126] Moves statistic information on01:54
LPCIBotthe product branches view into the main page and out of the01:54
LPCIBotsidebar, where it didn't really fit.01:54
lifelessgary_poster: / benji: if either of you are still around01:57
lifelessI'd love some comment/tip on bug https://bugs.edge.launchpad.net/launchpad-foundations/+bug/66362001:57
_mup_Bug #663620: generate overrides writes to global state <paralleltest> <Launchpad Foundations:New> <https://launchpad.net/bugs/663620>01:57
lifelesswgrant: aroud ?02:00
lifelesshmm02:46
lifelessfile:///var/launchpad/sourcecode/pygettextpo/ needs an upgrade to 2a02:46
mwhudsonon ec2?02:54
lifelessyeah02:55
lifelessalso02:56
lifelessUpdating bzr-loom to revision 4802:56
lifelessDoing on-the-fly conversion from RemoteRepositoryFormat(_network_name='Bazaar repository format 2a (needs bzr 1.16 or later)\n') to RepositoryFormatKnitPack1().02:56
lifelessThis may take some time. Upgrade the repositories to the same format for better performance.02:56
lifelessstarting upgrade of file:///var/launchpad/sourcecode/bzr-loom/02:56
lifelessUpdating pygettextpo to revision 2402:56
lifelessDoing on-the-fly conversion from RemoteRepositoryFormat(_network_name='Bazaar repository format 2a (needs bzr 1.16 or later)\n') to RepositoryFormatKnitPack1().02:56
lifelessThis may take some time. Upgrade the repositories to the same format for better performance.02:56
lifelessstarting upgrade of file:///var/launchpad/sourcecode/pygettextpo/02:56
mwhudsonmaking a new image with bin/ec2 update-image should work for that02:58
LPCIBotProject db-devel build (86): STILL FAILING in 3 hr 34 min: https://hudson.wedontsleep.org/job/db-devel/86/02:59
LPCIBotLaunchpad Patch Queue Manager: [rs=buildbot-poller] automatic merge from stable. Revisions: 1174702:59
LPCIBotincluded.02:59
wgrantlifeless: Hi.03:20
lifelesswgrant: hi, I was going to ask if you'd looked into a particular subsystem03:21
lifelessbut I found the damage03:22
lifeless(tight coupling)03:22
wgrantWhat's broken?03:22
lifelessif you run a unit test that uses TestMailer in a config with an instance name other than testrunner03:23
lifelessthen it uses a real mailer03:23
wgrantHah.03:24
wgrantlifeless: How are you going to deal with tests that touch the FS?03:36
wgrantIdentify all the relevant config values, push a new config overlay, and hope that everything uses the config?03:37
lifelesswgrant: BaseLayer.config.add_section(...)03:38
lifelesswgrant: + grep for the existing literal to find things that don't use the config03:38
lifelesswgrant: its working so far :)03:38
lifelesswgrant: want to help?03:38
=== Ursinha-afk is now known as Ursinha
wgrantlifeless: The publisher currently stores its path config in the DB, but that will be fixed soon.03:41
lifelessthe publisher layer/fixture can just update that config03:41
=== Ursinha is now known as Ursinha-afk
lifelessdepend on the DatabaseLayer (it should anyhow003:42
wgrantTrue.03:42
lifelessmy databasefixture branch is sufficient for you to experiment with that03:42
lifelesswgrant: when do you start?03:50
wgrantlifeless: With Soyuz?03:50
lifelessyeah, whatever that means :)03:50
wgrantHeh.03:50
wgrantDec 1303:50
lifelesscool03:52
marslifeless, have a moment for a question about testing?03:54
lifelessof course03:54
lifelessirc or voice03:54
marsirc is fine03:54
lifeless... :)03:55
marslifeless, so - layers are bad, test suite setup and teardown are bad.  How then do I implement a fixture that must run for the duration of a test suite?  Like, for example, rewriting the BaseWindmillLayer.03:55
lifelessok03:55
lifelessso the layers core implementation has some problematic aspects03:55
lifelesswe can get to a tolerable place without replacing it.03:56
mwhudsonfixtures + testresources!03:56
lifelessI would do this.03:56
lifelessWrite a Fixture03:56
lifelesshave a layer which does03:56
lifelesscls._fixture = MyFixture()03:56
lifelesscls._fixture.setUp()03:56
lifeless...03:56
lifelesscls._fixture.cleanUp()03:56
lifelessat the relevant places03:56
lifelessyou can see this sort of thing emerging in my parallel test branches03:57
lifelesse.g. lp:~lifeless/launchpad/databasefixture03:57
lifelessthe reset method on a fixture is what you'd call in testSetup03:57
marsah03:57
marssuites can't do that, can they?03:58
marstestSetup03:58
marsyou would have to use layers03:58
lifelesstestresources can do it03:58
lifelessbut I wouldn't try and mix n match03:58
lifelesstestresources is a complete replacement for layers03:58
lifelessfor bonus points, you could write an adapter from Fixture to Layer as a currying helper03:59
marsok, so that makes sense to me - that covers what we have, what is available, and why we do what we do now04:00
marscool04:00
lifelesswould you like me to describe this in a more permanent fashion somewhere?04:00
lifelessI wasn't really going to bother until tackling removal of layers was a feasible do-it-soon kindof project04:01
lifelessbut if you're liking what you're seeing of fixtures, perhaps its worth writing it up earlier04:01
marslifeless, a message to the dev list would be permanent enough IMHO. And anyone else interested in test engineering will see it there.04:01
marslifeless, you explained it quite concisely04:02
lifelessheh :)04:03
marsheck, you could copy-n-paste from 22:55 through 23:00 and be done with it :)04:03
lifelessthe [dis]advantange of a more permanent record is that folk may have less context and desire more exposition04:03
marscan't help that - people are going to start asking "what are fixtures?" and "why do layers suck?" at some point anyway.04:05
marswhat other question might people ask if they read such a message?04:06
lifelessdoes it make coffee?04:07
lifelesswhat does it look like in anger ?04:07
marsthat's a good one04:07
lifelessone thing that fixtures doesn't do well yet is arbitrary object graphs - the delegation to a child fixture can't handle anything other than linear lists (setUp doesn't cleanly skip if already setUp)04:08
lifelesstotally fixable, I just need to port that logic from testresources04:10
mwhudsonlifeless: aaaaaargh, since you created your testtools branch it looks like other branches have landed that import them from lp.testing.matchers :/04:12
mwhudsons/them/Startswith/04:12
mwhudsonoh maybe only one04:13
lifelessmwhudson: perhaps I should have left a forwarder in place04:14
lifelessmwhudson: I hope that folk will get in the habit of contributing straight to testtools and rolling a temp build of it for LP04:15
mwhudsonhard to say04:15
lifelessditto storm.04:15
lifelessand other deps04:15
mwhudsonlifeless: btw, you could consider assigning this bug to yourself: https://bugs.edge.launchpad.net/launchpad-foundations/+bug/10737104:19
_mup_Bug #107371: Make the test suite able to be run in parallel on a single machine <build-infrastructure> <feature> <test-system> <Launchpad Foundations:Triaged> <https://launchpad.net/bugs/107371>04:19
lifelessmwhudson: heh, I wouldn't want to stop other folk working on it :)04:20
mwhudsonheh heh04:20
jammwhudson: I see that the test failed again, but those tests look unrelated to my work. And they are *all* windmill tests.04:20
mwhudsonjam: :(04:20
jamdo you want to just try again?04:21
mwhudsoni'04:21
mwhudsoni'04:21
mwhudsonbah04:21
mwhudsoni'm not sure i WANT to04:21
mwhudsonbut i will try :-)04:21
jammwhudson: well, if you think they will fail again, you don't have to04:24
jamI just wonder about windmill, etc04:24
lifelessmwhudson: trivial changes to the databasefixture branch (I forgot to tell its tests that the librarianlayer now implied db access working)04:24
lifelessmwhudson: do you care?04:24
mwhudsonlifeless: i'll take a very quick peek i guess04:25
mwhudsonoh, it was jml who reviewed this04:27
lifelessmwhudson: oh yeah >< :)04:28
mwhudsonlifeless: i guess the comment in what is now testUploadsSucceed is a bit wrong?04:31
lifelessgarh, I thought I updated it04:32
mwhudsonmaybe you did04:32
mwhudsoni was looking at the individual diffs04:32
LPCIBotProject parallel-test build (8): STILL FAILING in 1 hr 32 min: https://hudson.wedontsleep.org/job/parallel-test/8/04:33
LPCIBot* Robert Collins: Merge unique config improvements.04:33
LPCIBot* Robert Collins: Fix config key.04:33
lifelessno, I didn't04:35
lifelessfixing04:35
mwhudsonlifeless: i think the branch is fine now04:39
lifelessthanks for looking04:40
lifelessok,good news04:53
lifelessstubs landing was broken, so we can fix windmill easily.04:53
lifelessis there a bug open for the issue with windmill ?04:59
lifelessanyone that wants ec2 to behave better, please review https://code.edge.launchpad.net/~lifeless/launchpad/threads/+merge/38908 :)05:04
StevenK\o/05:04
StevenKlifeless: Then reply to 'Failures on Hudson' on the -dev list?05:04
lifelessStevenK: well, after pqm takes it05:05
StevenKAh, it's a work-around05:05
StevenK.. somewhat05:05
lifelessno05:06
lifelessits the fix05:06
lifelessthe disable-check code path is currently still checking05:06
StevenKIt is that one-line?05:07
lifelessyes05:07
StevenKOh bloody hell, rs=everyone ?05:07
lifelessyou can try it yourself05:07
lifelessbin/test -vvt g.tests.test_layer05:07
lifelesswith and without05:07
* StevenK prods thumper towards that MP05:08
lifelessjtv: ^05:09
StevenKlifeless: Oh, just to troll for one second: When branching using bzr, when it says Estimate: x/y, it's not fair if both x and y increase at the same rate05:09
jtvlifeless: ?05:10
lifelessblame tree structures05:10
lifelessjtv: one line patch to fix all the windmill thread remaining failures05:10
jtvoh great thanks05:10
jtvNeed a review?05:10
StevenKI've added my conditional stamp to it05:10
lifelesshttps://code.edge.launchpad.net/~lifeless/launchpad/threads/+merge/3890805:10
* jtv looks05:10
wgrantDoes this mean that Hudson will magically stop spamming the channel?05:10
lifelesswgrant: hopefully05:10
StevenKNo, it will now say it built sucessfully05:11
StevenKHopefully05:11
wgrantHeh.05:11
StevenKwgrant: You didn't reply to the thread on -dev about it? :-P05:13
* lifeless polls the page looking for jtv's stamp05:13
wgrantStevenK: No. I'm glad it spams when it fails.05:14
* StevenK idly wonders how efficent it is for lifeless to loop in select()05:14
wgrantStevenK: I will also invoke EBUSY in my defence.05:15
jtvMy stamp is there now, but I couldn't set a review type.05:15
lifelesstossed straight at ec205:16
StevenKAwww, not even pqm?05:17
lifelesssorry, pqm :)05:19
StevenKWhich still says nothing doing05:20
lifelessah we're in testfix again05:21
lifelessfortunately this *is* a testfix.05:22
StevenKBut has lp.translations.windmill.tests.test_pofile_translate.POFileTranslatorAndReviewerWorkingMode.test_pofile_reviewer_mode actually been fixed?05:23
lifelessthat fix might now be, but the regular thread leak one will be :)05:25
LPCIBotProject devel build (135): STILL FAILING in 3 hr 32 min: https://hudson.wedontsleep.org/job/devel/135/05:27
LPCIBot* Launchpad Patch Queue Manager: [r=lifeless][ui=none][no-qa] Add support for logging SQL bind values05:27
LPCIBot* Launchpad Patch Queue Manager: [r=mwhudson][ui=none][bug=624009][incr] Add a05:27
LPCIBotBranchMergeProposalNeedsReviewEvent but keep the functionality05:27
LPCIBotthe same.05:27
LPCIBot* Launchpad Patch Queue Manager: [r=mwhudson][ui=none][bug=624009][incr] Rename05:27
LPCIBotMergeProposalCreatedJob to MergeProposalNeedsReviewEmailJob.05:27
LPCIBot* Launchpad Patch Queue Manager: [r=rockstar][ui=none][bug=638637] Stop loading the entire ancestry of05:27
LPCIBota branch during the scan05:27
LPCIBot* Launchpad Patch Queue Manager: [r=adeuring][ui=none][no-qa] Change permission name for05:27
LPCIBotEditDistributionSourcePackage to launchpad.BugSupervisor.05:27
lifelesswallyworld__:     def __call__(self):05:40
lifeless+05:40
lifeless+        result = []05:40
lifelessis a bit unusual, it separates the function definition and body visually, which makes it harder to read.05:40
wallyworld__lifeless: ECONTEXT05:40
lifelessI realise you're experimenting05:40
lifelessimproved-broken-link-handling05:40
wallyworld__i'm just writing up the description. looks like the email went out as sonn as a changed it from "onhold"05:41
lifelessyes, it does that :)05:41
lifelesstim is improving it05:41
wallyworld__which class was that code from again?05:41
lifelessfairly early in the diff05:42
lifelessI've closed it already though - sorry05:42
wallyworld__ lifeless: oh, you just want some white space added?05:43
lifelesswallyworld__: removed05:45
lifelessthe blank line05:46
lifelessthis is very surface, I'm just mentioning cause it stood out in a casual glance05:46
lifelessI'm happy to discuss the deeper stuff you're doing if you like (but not right now, am naffed)05:46
wallyworld__lifeless: it's not in the diff i'm seeing, nor in the code i have locally unless i've missed something. i'll triple check again. whoever picks up the review can check as well05:48
lifelesswallyworld__: the diff mailed out may have been stale05:49
wallyworld__lifeless: could be. go have a beer or something. i'm sure it's past beer o'clock over there :-)05:50
LPCIBotProject parallel-test build (9): STILL FAILING in 1 hr 27 min: https://hudson.wedontsleep.org/job/parallel-test/9/06:00
LPCIBot* Robert Collins: With the Librarian depending on the database, having the layer is enough to be useful.06:00
LPCIBot* Robert Collins: Update Librarian test expectations.06:00
LPCIBot* Robert Collins: Import BaseLayer at the right place.06:00
wgrantI like that branch.06:02
wgrantIt doesn't hide the author behind PQM.06:02
StevenKwgrant: parallel-test looks at branch of lifeless', so they're all going to be him06:03
wgrantYeah, I know.06:03
wgrantBut it's better than06:03
wgrantPQM06:03
wgrantPQM06:03
wgrantPQM06:03
wgrantPQM06:03
wgrantPQM06:03
rockstarwgrant, Tarmac sets the author properly, and then sets committer as "Tarmac"06:23
StevenKrockstar: Does Tarmac work faster than PQM? ;_)06:25
rockstarStevenK, what's slow about PQM?06:25
* rockstar answers a question with a question06:25
StevenKrockstar: 30 minutes for an answer back from PQM with "Okay, your branch is merged"06:26
wgrantrockstar: That sounds like it might not be completely insane.06:26
rockstarStevenK, yeah, but that's mostly a function of the pre-commit hook it runs.06:26
wgrantWasn't buildbot introduced as part of the 5-minute-PQM effort?06:27
wgrantDid you just give up after reaching half-hour-PQM?06:27
rockstarwgrant, yeah, PQM still wants to build a tree and such.  It spends most of its time cleaning up after itself, actually.06:28
spmhrm. it used to be, when I started, that PQM+the entire LP test suite would happen in 90 minutes. it was when that creeped to ~ 2-2.5 hours, that 5 min pqm was introduced.06:31
spmso I'm not sure the fault lies entirely with pqm here; rather what it's being asked to do, is what's taking so long.06:32
StevenKspm: I just like digging at pqm since lifeless wrote it06:33
spmroger roger06:33
spmI just have to (roundabout) defend him every now and then. afterall, it's easier to stab someone in the back, if you're standing behind them. :-P06:33
rockstarStevenK, AIUI, lifeless didn't write it, he just performs CPR on it every once in a while.  I think PQM pre-dates baz.06:35
lifelessStevenK: I didn't write it :)06:36
rockstarFrankly, I'm surprised that bzr is the only DVCS with something like robotic landings.06:37
=== jtv is now known as jtv-eat
LPCIBotProject parallel-test build (10): STILL FAILING in 1 hr 27 min: https://hudson.wedontsleep.org/job/parallel-test/10/07:28
LPCIBotRobert Collins: Update missed comment.07:28
=== vila_ is now known as vila
=== almaisan-away is now known as al-maisan
=== jtv-eat is now known as jtv
adeuringgood morning08:50
gmbAaaaaargh.08:53
wgrantgmb's back!08:54
gmb:)08:54
gmblifeless: Is there some special dance that I have to do to use feature flags in pagetests or has no-one been stupid enough to try doing that yet?08:58
lifelessgmb: should work fine08:58
gmbHmm.08:58
lifelessbe sure to cargo cult something appropriate08:58
gmbYeah. I'm working on it :)08:59
lifelessmars has a branch adding a context manager08:59
gmblifeless: Yes, I've got that branch merged. I'll keep prodding things until it works.09:00
lifelesshave you added a rule09:00
gmbAt least I know I'm not (entirely) mad.09:00
lifelesslib/canonical/launchpad/doc/librarian.txt is an example that I know works.09:00
lifelessor09:00
gmbAha, lemme look...09:00
lifelesslib/canonical/launchpad/doc/timeout.txt09:00
gmblifeless: Yep. Got it working now, thanks.09:12
lifelessgmb: awesome09:12
lifelessgmb: if you want feedback on your use, just shout09:13
bigjoolsmorning09:14
wgrantEvening.09:15
* thumper waves09:17
* bigjools shivers09:17
* thumper gives bigjools the chills09:18
bigjoolsyes, baby09:19
thumperdb-devel failing with lp.translations.windmill.tests.test_serieslanguages.LanguagesSeriesTest.test_serieslanguages_table09:21
wgrantStevenK: Why would anything except the publisher care about how they're published?09:23
StevenKwgrant: In relation to my comment?09:23
wgrantStevenK: Yes.09:23
StevenKwgrant: Since I can think of at least two other places that would benefit from it09:24
mrevellMorning09:24
wgrantOh?09:24
StevenKwgrant: IDS for one09:24
wgrantHm, I don't see why that would care.09:24
wgrantWhy would it care?09:24
StevenKBecause it copies from PRIMARY and DEBUG09:25
wgrantBut that's just because they're PRIMARY and DEBUG.09:25
wgrantNot because they're a-f (which is also copy archives).09:25
StevenKwgrant: If you add one there, the publisher can use and IDS can as well09:25
wgrantStevenK: But IDS doesn't want this one.09:26
wgrantIt wants (PRIMARY, DEBUG), not (PRIMARY, COPY[, soon DEBUG])09:26
StevenKOh, duh09:26
* StevenK should learn to read09:27
wgrantHeh.09:27
StevenKwgrant: Oh, and I tried Evolution now that I'm on Maverick. It still hates me.09:30
wgrantStevenK: Thanks.09:30
wgrantStevenK: :(09:30
StevenKHowever, Thunderbird has gotten scary09:30
wgrantI haven't used it since 2.0.0.0 was fresh.09:31
StevenKYou type in your e-mail address and it guesses and probes for the IMAP/POP3/SMTP servers it should use.09:31
wgrantAh.09:31
StevenKHow do I get the test suite to print out all SQL that gets run?10:02
lifelessLP_DEBUG_SQL10:03
* StevenK drowns in it10:06
LPCIBotProject devel build (136): STILL FAILING in 3 hr 58 min: https://hudson.wedontsleep.org/job/devel/136/10:09
StevenKlifeless: When did your windmill thread fix land?10:10
StevenKlifeless: Ah ha. #136 contained your fix, but six windmill tests still failed10:13
lifelessStevenK: with thread errors?10:14
StevenKlifeless: Yes10:14
lifelessfrell10:14
StevenKhttps://hudson.wedontsleep.org/job/devel/lastFailedBuild/testReport/junit/lp.bugs.windmill.tests.test_official_bug_tags_management/TestOfficialBugTags/test_official_bug_tags_management_2/ for example10:15
lifelesswell10:15
lifelesstheres a few possibilities10:15
lifelessthe layer may not be setting the flag correctly10:15
lifelesssomething else may be wrong10:15
lifelesshttps://hudson.wedontsleep.org/job/devel/lastFailedBuild/consoleFull10:17
lifelesssearch for test_official_bug_tags_management10:17
lifelessraise WindmillTestClientException(result['result'])10:17
lifelessthe _2 is just reporting that there are new threads10:17
lifelessits not erroring *because* of them10:17
lifelesshowever this is an issue10:18
lifelesstest: canonical.testing.tests.test_layers.TestThreadWaiting.test_slow_thread10:18
StevenKSo subunit is masking the failures?10:18
lifelesssuccessful: canonical.testing.tests.test_layers.TestThreadWaiting.test_slow_thread10:18
lifelesserror: canonical.testing.tests.test_layers.TestThreadWaiting.test_slow_thread [ multipart10:18
lifelessStevenK: huh, no.10:18
lifelesshttps://hudson.wedontsleep.org/job/devel/136/testReport/junit/lp.bugs.windmill.tests.test_official_bug_tags_management/TestOfficialBugTags/test_official_bug_tags_management/10:19
lifelessthats the failure10:19
bigjoolsthis broken person-picker is really getting on my tits now10:19
StevenKI see, it appears twice10:19
lifelessthe subunit fragment I just pasted is a broken api usage in the test environment10:19
lifelesswhich is also an issue10:19
StevenKlifeless: However, it is the only one that does appear twice10:19
lifelessI'm not sure where to file that one; jml may know10:19
lifelesssuccessful: lp.bugs.windmill.tests.test_bug_also_affects_new_upstream.TestBugAlsoAffects.test_bug_also_affects_picker10:20
lifelesstest: lp.bugs.windmill.tests.test_bug_also_affects_new_upstream.TestBugAlsoAffects.test_bug_also_affects_picker10:20
lifelesstags: zope:threads10:20
lifelesserror: lp.bugs.windmill.tests.test_bug_also_affects_new_upstream.TestBugAlsoAffects.test_bug_also_affects_picker [ multipart10:20
lifelesssame issue10:20
lifelessits a bug in the zope testrunner output glue10:20
lifelesss/bug/questionable behaviour/10:20
StevenKAh10:20
lifelesssee how it reports the same test twice10:20
lifelessonce successful10:20
StevenKThe problem is that subunit/hudson is tripping over it10:21
lifelessonce error with the tag zope:threads10:21
lifelessyes, needs a fix for sure10:21
lifelessalso10:23
lifeless\o/ reading-back the ports from the librarian10:23
lifelessnight all10:35
jtvgary_poster: do we have any concrete plans for getting that storm is_empty fix into zc.buildout?11:17
* jtv wonders vaguely if he's asking the right person11:17
jtv…and concludes yes, probably11:19
henningeDoes anybody know what's wrong with the icon alignment on (some) Launchpad pages? It seems to be related to the "link" formatter.11:36
henningehttp://people.canonical.com/~henninge/screenshots/icon_misalignment_chromium.png11:37
henningeIt's slightly less in Firefox but still visible.11:37
henningehttp://people.canonical.com/~henninge/screenshots/icon_misalignment_firefox.png11:37
wgrantIt's been a problem since CSS sprites were introduced.11:37
henningeBut this is worse now. Could it be related to the Ubuntu font I am using now?11:38
henningeLooking at bac's screenshot, I suspect that: http://people.canonical.com/~bac/branchvis/forbidden-with-teams.png11:39
henningewgrant: also, the branch icons for "bug" and "merge proposal" now come out on top of each other.11:39
henningebut that may be a result of the higher line height introduced by the misalignment.11:39
wgranthenninge: Branch emblems have always been a problem in WebKit. Or does it show up in Gecko now too?11:40
jcsacketthenninge: i just checked a few pages in safari, they appear slightly unaligned, as in bac's screenshot, not as bad as in the screenshots you posted.11:40
wgrantI think the sprite alignment problem is just because the text height is larger.11:40
jcsackettsafari is definitely *not* using the ubuntu font. :-P11:40
wgrantSo the sprite appears at the top.11:40
henningewgrant: so it is related to the new font.11:40
wgrantRelated, but not specific to.11:40
henningejcsackett: do you have the new Ubuntu font installed?11:41
jcsacketthenninge: no, and i'm away from my development machine right now. i can install it and let you know if i see an increased problem later today, when i'm home.11:42
jcsackettprobably a bit after 9AM EST?11:42
henningejcsackett: yes, that's fine.11:43
jcsacketthenninge: okay, i'll let you know then.11:43
henningejcsackett: are you on EST? You are up early!11:43
henningeAlthough it's EDT, actually.11:43
henninge;-)11:43
jcsackett:-P11:44
jcsacketthenninge: i usually get up at five to walk my dogs and hit the gym.11:44
henningegood man! I am glad when I make 6 o'clock ...11:45
jcsackettit took a fair bit of retraining. i used to be a sleep 10 guy. :-)11:45
jcsackettsleep till ten, rather.11:45
henningeluckily I have a family that won't allow me to do that ...11:45
* gmb lunches. 11:46
deryckMorning, all.11:58
=== matsubara-afk is now known as matsubara
=== Ursinha-afk is now known as Ursinha
jcsacketthenninge: was able to look at the site with the font; it does look a bit worse.13:41
jcsackettbut the alignment does seem off across browsers, even without the font. just not as bad as with.13:42
salgadomrevell, when you have a moment, can you have a look at https://code.edge.launchpad.net/~leonardr/launchpad/temporary-integration/+merge/38836 ?13:44
mrevellcertainly salgado13:44
allenapderyck: Robert Collins gets two entries for Launchpad Suite on https://edge.launchpad.net/malone/+subscribe. Have you seen that before?14:20
* deryck looks14:20
deryckallenap, no, never seen that.  Odd.14:21
allenapderyck: That could be a problem with new code. I'll investigate.14:21
deryckallenap, ok, thanks14:22
abentleybigjools, chat?14:23
marsrockstar, ping, the mail you sent yesterday still has not made it through to Gary and I.  Strange.14:47
rockstarmars, bugger. That means my SMTP server has done its weird "fall over but don't tell anyone" thing that I thought I fixed months ago.14:56
=== adeuring1 is now known as adeuring
=== abentley_ is now known as not-abentley
jkakarAm experiencing more (I think) timeout related issues... can't download a 276kb image from a bug.15:15
jkakarI appreciate the rationale for ratcheting down timeouts, but it's been *really* annoying.15:15
jkakarI think it might be too extreme.  Maybe I just have bad luck and most people don't notice.15:15
deryckjkakar, hi.  what bug and attachment?15:16
jkakarderyck: https://bugs.edge.launchpad.net/landscape/+bug/66378615:17
jkakarderyck: The first attachment, called 'Design'.15:17
deryckjkakar, I can't see it because it's private.  I think this is the private attachments can't be downloaded bug, not timeout related.15:18
* deryck looks for a bug #15:18
jkakarderyck: Ah, okay.  I was able to download the other attachment on the same bug, though.15:18
deryckyeah, it's an inconsistent condition.15:18
jkakarderyck: Ah, fun times. :)15:19
deryckjkakar, yeah, you could call it that :-)15:19
jkakarAnyway, sorry for jumping to conclusions about timeouts, but been experiencing them a lot for the last few weeks. :/15:19
deryckjkakar, I subscribed you to Bug #638054, since it's private.  But I think that's what you might be experiencing.15:21
deryckjkakar, and yeah, timeouts are also an issue.  In some cases, it's the transition to postgresql 8.4.15:22
jkakarderyck: Cool, thanks a lot and thanks for listening. :)15:23
deryckjkakar, np! :-)15:23
marshenninge, looks like the lucid-db-devel build failure is in translations, so in your court?  https://lpbuildbot.canonical.com/builders/lucid_db_lp/builds/33415:30
marshenninge, can't remember if that is a known issue - someone was working on it last night IIRC?15:30
marsso it may be fixed in devel15:30
not-abentleybigjools: chat?15:31
danilosmars, I can't remember anything related to that being changed in literally ages, so it's really weird15:31
bigjoolsabentley: 'sup?15:31
marsdanilos, ok, not fun15:31
abentleybigjools: several things.  buildd install issues and a big build queue.15:32
=== matsubara is now known as matsubara-lunch
abentleybigjools: can we chat on mumble or skype?15:32
bigjoolsabentley: I'm on mumble15:33
marsdanilos, maybe it was intermittent?15:33
abentleybigjools: I'm in the Soyuz channel.  Where are you?15:33
danilosmars, yeah :( I'd even say we haven't changed anything related since the last Epic, and it's the first time I see it15:35
marsdanilos, ok, I'll force a rebuild then, and we'll see15:35
marsdanilos, ah, same failure in lucid_lp, eh?15:36
marsdanilos, https://lpbuildbot.canonical.com/builders/lucid_lp/builds/26815:36
marsdanilos, and the following test on that same builder passed just fine15:37
bigjoolsabentley: https://lpstats.canonical.com/graphs/BuildersActiveVirtual/20101019/20101021/15:38
danilosmars, :( I'll take a look15:46
sinzuiallenap, a I have said in many emails. registry just models user. authentication is launchpad foundation15:47
allenapsinzui: Sorry. I assume you've re-targeted that bug, but I'll try to remember for future bugs.15:48
sinzuiI have15:48
allenapsinzui: I saw the bit about merged accounts and jumped to a decision.15:48
sinzuiI am powerless to help with openid issues15:48
danilosmars, the failing tests are completely different15:49
sinzuiallenap, openid login to a wiki is the issue.15:49
allenapsinzui: I was hasty :(15:49
marsdanilos, I forced the build anyway, we'll see what happens.15:50
=== salgado is now known as salgado-lunch
jcsackettcan someone help me out with some weird issues with a method over launchpadlib?15:59
jcsacketttest that brings up the issue is here: https://pastebin.canonical.com/38848/16:00
jcsackettpdb session showing odd thing happening inside test is here: https://pastebin.canonical.com/38849/16:00
sinzuijcsackett, I am confused about the issue here.16:05
jcsackettsinzui: person.join is never actually called in the codebase--i'm getting a return of the raw json dump of the test person as a return.16:05
sinzuijcsackett, methods() do update the instance (they are transactional)16:05
sinzuileonardr, ^ ca you help jcsackett with his lplib test?16:06
leonardri'll look16:06
jcsackettleonardr: feel free to interrogate me a lot; this is easier to describe than pastebin an illustrative fashion.16:07
leonardrok16:07
sinzuijcsackett, you use lp_save() when you change a property (eg, commit). You have not changed a property. I do not think it is needed16:08
jcsackettsinzui: yeah, i wasn't sure. i was experimenting a bit there.16:08
jcsackettthe same scenario occurs regardless of lp_save, so i agree it's unnecessary.16:08
sinzuijcsackett, regardless of the join behaviour, does the test pass?16:09
jcsackettsinzui: no, the test is failing, because person.join doesn't seem to actually get called.16:10
jcsackettso no membership is created. or in the case where i'm trying to catch an exception, the exception isn't raised.16:10
jcsackettthis being lp.registry.model.person.join, not the NamedOperation.16:11
=== Ursinha is now known as Ursinha-lunch
=== matsubara-lunch is now known as matsubara
henningesinzui: found another one of these: https://edge.launchpad.net/~timstoaster-deactivatedaccount16:16
leonardrjcsackett: my advice based on zero information is to re-run the test with 'httplib2.debuglevel = 1' and show my the http requests and responses16:16
sinzuihenninge, that is because login/reset is broke16:16
henningesinzui: he is appaerently active because he just joined a team.16:17
henningeyeah, I know.16:17
henningeCan we not fix that, though?16:17
sinzuihenninge, he is active because her has a preferred email address16:17
henningeI mean, the account.16:17
sinzuihenninge, the rename rule did not run. that is the only thing wrong. the user does not seem to care16:17
sinzuihttps://edge.launchpad.net/~timstoaster-deactivatedaccount/+review16:18
henningesinzui: Yes, I just found that.16:18
leonardrjcsackett: also, tell me lazr.restfulclient.__version__16:19
henningesinzui: is it harmless to just rename it?16:19
leonardrand see if this starts working if you access test_team.name before invoking me.join()16:19
sinzuihenninge, yes, and it should have been renamed16:19
henningesinzui: ok, I'll  do that and inform the user.16:19
sinzuihenninge, if there was something like a ppa, the rename field would not appear16:19
henningeah, good to know16:19
henningesinzui: hm, appearently I cannot do it.16:20
sinzuihenninge, this is the E'FING hidden location bug that bac reported16:25
sinzuiThe user can fix it, but we cannot16:25
jcsackettleonardr: requests/responses are here https://pastebin.canonical.com/38851/16:28
jcsackettversion is 0.10.016:29
jcsackettcalling test_team.name didn't have any effect.16:29
leonardrok, i guess that's good16:29
jcsackettleonardr: i see a number of 303s around calls involving me; should i try getting the test person out of people['test-person'] instead of using .me?16:30
leonardrjcsackett: yes, that's the problem. clients aren't allowed to re-submit post information if they get a 30316:30
jcsackettleonardr: trying that now.16:30
* jcsackett wishes he had thought of httplib2.debuglevel before.16:30
leonardrit is the general purpose tool16:31
jcsackettleonardr: that worked, thanks for your help. i'll remember debuglevel next time. :-)16:31
leonardrgreat16:31
leonardrjcsackett: you ran into https://bugs.edge.launchpad.net/launchpadlib/+bug/50429717:04
_mup_Bug #504297: calling named operation on launchpad.me <launchpadlib :Triaged> <https://launchpad.net/bugs/504297>17:04
jcsackettleonardr: yup, that's exactly it.17:05
=== benji is now known as benji-lunch
jammwhudson: when you wake up. we were down from 4 failing windmill tests to only 1 failing windmill test \o/ for reproducibility. I guess we have an open bug about the windmill tests being unstable? Can we submit anyway?17:24
=== Ursinha-lunch is now known as Ursinha
=== deryck is now known as deryck[lunch]
jamabentley: I forgot to mention yesterday, but the db is at chinstrap:~jameinel/test.db.bz2 if you want it.17:29
abentleyjam, cool.17:29
=== salgado-lunch is now known as salgado
=== beuno is now known as beuno-lunch
lifelesslamont: hi17:58
=== deryck[lunch] is now known as deryck
lifelessmatsubara: commented18:11
matsubarathanks lifeless18:11
marsjam, do you have a copy of the test failure?18:13
jammars: when there was 1 failure, or when there were 4 ? :)18:14
jamlp.code.windmill.tests.test_branchmergeproposal_review.TestReviewCommenting.test_merge_proposal_replying18:14
jamwas the single failure18:14
jam_StringException: Text attachment: garbage18:14
marsjam, I mean, what was the failure you were seeing?  Ah, 'garbage threads', Thread-1234, etc.18:14
lamontmorning lifeless18:15
jammars: yeah, something like that18:15
jamI can forward you the email if you want18:15
marsjam, please do, and I'll tell you if it is intermittent or not18:15
jammars: well, I'm pretty sure it is intermittent. As we submitted, and 4 windmill tests fails, we resubmitted without changes, and only 1 failed18:16
jam(I guess a fix could have landed in the meantime)18:16
jammars: I sent you the single failure, and the 4-failure emails18:16
marsjam, lifeless had a fix, may or may not have worked for that.  In the meantime we are saying "run ec2 test, if it looks good, then bzr lp-land"18:16
marsthanks18:17
jammars: so ignore the failures and land anyway?18:17
=== al-maisan is now known as almaisan-away
marsjam, from what I see, yes18:17
marsthat is the intermittent failure that everyone has been getting18:17
=== benji-lunch is now known as benji
lifelessmatsubara: the windmill threads thing is fixed18:34
lifelessbah18:34
lifelessmars: ^18:34
matsubara:-)18:34
lifelessmars: we shouldn't be saying that any more18:34
marslifeless, proven fixed?18:34
lifelessyes, hudson shows errors still because of a defect in zope.testrunner18:35
lifelessit reports *any* new threads as an 'error:' at the subunit level.18:35
marslifeless, and when it comes to fixes for stuff like this, you have to deal with fallout for a week or two after, until everyone is only working on fresh branches of devel.18:35
lifelessmars: buildbot won't report it as an error18:35
lifelessmars: no, ec2 merges devel, so everyone gets it immediately18:35
marsah, right18:36
=== beuno-lunch is now known as beuno
marsjam, so as lifeless says, you can ec2 land it again if you want, it should pass18:37
marsbut since you already ran it by, you can probably just lp-land it18:37
marsno need to run the suite again18:37
=== gary_poster_ is now known as gary_poster
lifelesslamont: ping19:08
mtaylorsinzui: I just added information to an incomplete bug report - should _I_ pop the status back to new, or will just adding info be good enough?19:14
sinzuiI get emails. changing the status mean I need to change it back if I still do not see a problem19:15
mtaylorok. cool19:15
sinzuimtaylor, I can see the sorting is working as designed. The common problem here is that this page seems to be designed for no one. Each person sees, tries to change it, then makes other people irrate.19:20
mtaylorsinzui: really? what is causing cherry to show up first?19:20
sinzuimtaylor, *who* needs to use this drizzle page19:20
marsrockstar, the mail went through.  Thanks!19:20
sinzuimtaylor, The only legitimate use case I have found for this page is for packagers who need the latest package to create an installer19:21
rockstarmars, awesome.19:21
mtaylorsinzui: well... I _was_ using it in the debian watch file as per the answer in the question about using launchpad via uscan19:21
mtaylorsinzui: right. I was using it as part of packaging19:21
sinzuimtaylor, why do you not release from the focus of development?19:21
* sinzui is search for more evidence that +downloads is so bad is needs to be shot, not fixed19:22
mtaylorsinzui: oh - well, usually we do - I think we moved dev focus to freemont prematurely19:22
mtaylorsinzui: well.... actually, I take that back19:22
mtaylorsinzui: "focus of developement" seems to be slightly more vague than I'd like to it be ... focus of development sets lp:drizzle to something ... which is what our "trunk" branch is - which is where crazier things go, whereas elliot is in beta and thus is the thing we're working on solidifying and releasing19:23
sinzuiunderstood19:24
mtaylorsinzui: I have no personal need for +download myself, as long as there is a sensible thing to put in debian/watch :)19:25
sinzuithis is a similar pattern to backporting and releasing to supported series, but +download and the latest release rules do not know how to negotiate that arrangement19:25
mtaylorsure. nor would I expect them to19:26
mtaylorbut the latest download link on the main project page seems to have gotten it right19:26
mtaylorwhy is it that the cherry release comes up as the most recent on +download ?19:26
sinzui50% of people think it is wrong :(19:26
mtaylorand/or why is that 'as designed'19:27
mtaylorhaha19:27
jcsackettso has anyone gotten an ec2 email back that says every test failed but it's all text attachment: garbage? or am i just especially unlucky today?19:27
sinzuiI honestly this this is unfixable. There are many user and usecases at play here and one very mediocre solution19:27
mtaylorI think I'm tending to agree with you19:28
sinzuidebian/watch though is a very important use case and I will look into how we can ensure this is supported19:28
mtaylorsinzui: well, for now it's working for me to use http://launchpad.net/drizzle rather than http://launchpad.net/drizzle/+download19:28
mtaylorbut - if 50% of the people think that http://launchpad.net/drizzle is showing the right thing...19:29
mtaylors/right/wrong/19:29
sinzuiI think we need to ensure know one has to guess at which url to use with debian watch. The portlet on the project page could change in a few weeks and there is no test says that debian watch needs the highest version number19:30
mtaylorgotcha19:30
mtaylorspeaking of - I'd still like to understand what criteria would cause something other than drizzle7-2010.10.01.tar.gz to show up as the latest download... so that I can perhaps manage my meta data differently19:31
sinzuiwell, maybe that is what we want and we can do that on the download page. Add a portlet that shows the highest version number release and ensure there is a url for it too19:31
mtaylor(sorry if I'm being a pest about it, I just can't understand why a release from april would match "latest" under any set of criteria)19:31
sinzuiSince there are no release from trunk, the series are sorted by number high to low, but aphabetically low to high.19:32
sinzuielliot comes after cherry19:33
mtaylorah19:33
mtaylorok.  so the series are not sorted by release date19:34
sinzuiWe could change the rules to always sort alphabetically. I am not sure how many people would notice. I think most have adopted numbers when they know they are developing for a long time because Lp does that better19:35
mtaylorbut why not use the explicit release date?19:35
sinzuiWe do not sort series be release date because people add releases to old series often. The sort we use addressed the last group of users who noted that the listing was unsorted19:35
mtaylorheh19:36
mtaylorwell, I would personally prefer this page not exist rather than be sorted alphabetically reversed19:36
sinzuithe product-release-finder is also a factor. it add releases to series that may be ancient19:36
leonardrsalgado (or someone), i'm having problems getting testopenid.dev to work on my dev machine. i think this is the first time i've ever actually had to use testopenid.dev. can you help me?19:37
=== almaisan-away is now known as al-maisan
sinzuimtaylor, your issue is a dup of bug 231797, but I am updating that bug to explain the issues with the project index page and download page19:38
_mup_Bug #231797: no sensible way to use debian/watch files with launchpad hosted tarballs <Launchpad Registry:Triaged> <devscripts (Ubuntu):Invalid> <https://launchpad.net/bugs/231797>19:38
mtaylorcool19:38
sinzuioh, I see you have already comment on the bug19:38
leonardrsalgado (or whoever), my specific error seems to be the lack of a session cookie19:39
leonardrsalgado: got it working, nm19:42
Ursinhalifeless, hello hello19:48
lifelesshiya19:50
Ursinhalifeless, so, bad-commit-xxxxx tag is added by hand, right?19:50
lifelessyes19:51
lifelessfor now; if you want to add a scrpit to do it, awesome19:51
Ursinhalifeless, I'm asking because there's a wiki page saying it's done by the script: https://dev.launchpad.net/QAForContinuousRollouts19:52
Ursinhaso I got confused19:52
lifelessmeep19:52
lifelesswell its *read* by the script19:52
lifelessbut the script can't assess good/bad19:52
lifelessit can only read our decisions19:53
Ursinharight, I thought so19:53
Ursinhalifeless, bdmurray landed a fix for bug 347218 but didn't mention the rollback tag19:54
_mup_Bug #347218: Allow bug supervisors to make tags official <bad-commit-11701> <qa-bad> <story-define-official-tags> <Launchpad Bugs:Fix Committed by brian-murray> <https://launchpad.net/bugs/347218>19:54
UrsinhaI've added the bad-commit tag19:54
lifelessUrsinha: cool19:55
lifelessUrsinha: so we need some way to say 'commit X fixes bad commit Y' other than [rollback]19:55
lifelessand19:55
lifelessUrsinha: so we need some way to say 'commit X fixes bad commit Y' other than by committing a new revision19:55
lifelessthese are separate but could be fixed in the same way19:56
UrsinhaI guess the tag19:56
Ursinhaif you add the tag, and mentions the bug in both commits19:56
Ursinhahmm19:57
lifelessmm19:57
lifelessso I'm saying 'lets assume people fix by mistake'19:57
Ursinhalifeless, how would the second work?19:57
lifelessor thereabouts19:57
lifelesswell, one way would be:19:57
lifelessbad-commit=xxxx,yyyyy19:57
lifelesswhere xxxx is the bad commit19:57
lifelessand yyyy is the fix for it19:57
lifelessso the script cannot deploy xxx without yyyy19:57
Ursinhawhere are we placing this?20:05
Ursinhabug tag?20:05
Ursinhalifeless, ^20:11
lifelessotp20:12
Ursinhasure20:15
gary_posterrockstar, do you have a few min for a tarmac phone call?20:25
rockstargary_poster, indeed I do.20:25
gary_posterawesome, thanks rockstar.  mumble or skype?20:25
gary_posterI'm on both atm20:25
rockstargary_poster, skype please.20:25
abentleyjam: skype?20:26
Ursinhalifeless, also, could you comment on bug 623239 later, please?20:30
_mup_Bug #623239: please don't squash qa-untestable tags <qa-tagger:Incomplete> <https://launchpad.net/bugs/623239>20:30
jamabentley: sure, let me go grab a drink, brb20:31
jamabentley: back20:36
jamcalling20:36
jamwell, would if you were online :)20:37
rockstargary_poster, also, if yous guys have questions, there's also #tarmac20:46
gary_posterrockstar: ah!  good to know, thanks20:47
lifelessUrsinha: hi20:47
lifelessI is back20:47
lifelessUrsinha: I'm saying we could use the same bug tag20:47
lifelessbad-commit=12345,1235020:47
lifelessmeaning broken in 12345 fixed in 1235020:48
Ursinharight20:49
lifelesscommented on the bug, but perhaps we need some realtime discussion20:50
mwhudsonmorning20:54
Ursinhalifeless, want to mumble?20:54
lifelessI could skype if you like20:54
lifelessmumble works very badly here20:54
Ursinhasure, a minute20:55
lifelessI'm rbtcollins20:56
Ursinhacalling20:57
mwhudsoni wonder why mumble is so terrible from nz20:59
mwhudsonlag?20:59
=== matsubara is now known as matsubara-afk
thumpermwhudson: I think it is telecom voip traffic shaping21:01
thumpermwhudson: yet to confirm though21:01
mwhudsonthumper: i'21:01
mwhudsonthumper: i'm not on telecom though21:01
thumpermwhudson: who are you through?21:01
mwhudsonthumper: voda at home, snap in the office i think21:02
mwhudsonthumper: the international bandwidth is all the same though21:02
mwhudsoni guess?21:02
thumpermwhudson: how's the office thing working out?21:02
thumperrockstar: skype?21:02
rockstarmwhudson, you have an office, huh?21:02
mwhudsonthumper: well21:02
rockstarthumper, sure.21:02
mwhudsonrockstar: yeah, i'm renting some space about 50m from where lca was21:03
mwhudsondecided some more work/life separation was in order21:03
mwhudsonalso being in town regularly is nice21:03
jmllifeless: when are we going to just split canonical.buildd into its own tree21:03
rockstarmwhudson, yeah, I tried that, only to realize that I am socially "handicapped."21:05
lifelessjml: when someone gets a tuit21:12
lifelessoh yay21:14
lifelessjml: bug 66410721:14
mwhudsonrockstar: i'm not sure quite what you mean, but there are some pretty socially handicapped people in this office too21:16
jmllifeless: well, I guess we escalate that to Canonical Legal21:17
lifeless'punt'21:17
jmllifeless: and hope that the amateur discussion is at least entertaining21:17
lifeless:)21:17
mwhudsonrockstar: hey, when am i going to be able to click "merge this" on a mp?21:20
cr3can someone recommend a good example of project packaging eggs? I'm not sure where to ask but perhaps launchpad folks might know21:20
lifelessgary_poster: I'd like a few cycles to talk zcml if you can spare them today21:20
cr3gary_poster: ^^^ you might know?21:20
lifelesscr3: what do you mean?21:20
gary_posterlifeless, sure thing.  about to go another call, then will ping21:21
lifelessthansk21:21
lifelessjml: also21:21
lifelessjml: I have a zope testing question I *know* you know hte answer too21:21
gary_postercr3: I know a fair amount about packaging Python things as eggs, not sure what your question is.  Feel fee to ping me later if you like, but swamped atm, and others can probably answer21:21
cr3lifeless: if a project creates eggs as its build process, they probably need to be packaged as part of the project package itself. I was hoping to find a good exmaple of a project that does something like that21:22
lifelessjml: on hudson, thread leaks show up as errors with the same test id as the test that passes21:22
lifelessjml: how do we change that to a skip, or something similar.21:22
cr3gary_poster: ah, I meant "debian packaging" as opposed to "packaging eggs" :)21:22
jmllifeless: hmm.21:22
lifelessjml: or should StevenK filter them with subunit-filter21:22
lifelessjml: this isn't in our logic - our logic is 'fixed' now.21:22
jmllifeless: these are errors raised during testTearDown, right?21:23
lifelessno21:23
LPCIBotProject devel build (137): STILL FAILING in 3 hr 58 min: https://hudson.wedontsleep.org/job/devel/137/21:23
lifelessthere are two, overlapping, thread checks.21:23
jmloh, zope's and our own custom one21:23
lifeless1) in BaseLayer. Fixed (the disable_thead_check flag was miscoded)21:23
lifeless2) in zope.21:23
gary_postercr3 ah gotcha.  Played with it.  assume you saw https://wiki.ubuntu.com/PackagingGuide/Python .  now really on call21:23
lifelessit took 3 months for my last change to zope.testing to get in; I'm not stoked about the idea of trying again :(21:24
cr3gary_poster: thanks, I didn't know about that guide, I'll certainly have a look21:24
thumperjml: hi21:25
jmlthumper: hello21:25
jmllifeless: hm.21:25
thumperjml: I want you to look at the recipe stuff and let me know what needs to be done to say "we're done"21:25
jmlthumper: I already took a good look today21:25
lifelesscr3: eggs and python packages are a little odd :)21:25
jmlthumper: writing it up tomorrow. who should I send it to?21:25
lifelesscr3: the python packaging standard unpacks them on disk.21:25
cr3how is ez_setup.py generated or where is it copied from? I copied mine from the distribute package but I'm not sure that's right21:26
thumperjml: at least flacoste, me, abentley, rockstar, and wallyworld21:26
jmlthumper: will do.21:26
lifelessif that what I think it is, its 'download unsigned code from the net and run it'21:26
thumperjml: maybe the internal list?21:26
lifelessso I would avoid it like the plague.21:26
jmlthumper: sure21:27
cr3lifeless: it seems to be quite prevalent in lazr packages for example21:27
jmlthumper: btw, heard about open build service's "Source services"21:27
thumperjml: yeah, saw the email21:27
thumpernot looked at it though21:27
lifelesscr3: no comment ;)21:27
cr3lifeless: at least the md5sum is checked, for what it's worth21:28
wgrantcr3: Against what?21:28
wgrantAn HTTP page?21:28
lifelesswgrant: yes, if they MITM you, they have to change *two* http requests.21:29
jmlthumper: I had a play today21:29
wgrantlifeless: Awesome!21:29
jmlthumper: there's a lot I admire about OBS – I think they've got a great product.21:29
jmlthumper: but in this area we are still well ahead, I think.21:29
jmlthumper: particularly if we smarten up the UI21:29
thumperjml: so lets get some good designers to focus on it...21:30
jmllifeless: http://paste.ubuntu.com/517034/ that's the event. the error is tagged, so subunit-filter ought to be able to filter it, pending a new feature :)21:30
=== al-maisan is now known as almaisan-away
wgrantRecipe builds would be really useful if they supported non-native packages, and customisable changelogs.21:30
cr3wgrant: the downloaded setuptools egg21:30
lifelessjml: Yeah, I'm wondering what we /want/ though21:30
jmllifeless: oh. I reckon we want to filter out based on tags, probably.21:30
lifelessok, I'll file a bug on subunit to permit that21:31
jmllifeless: or disable zope's checking21:31
jmlbut it's pretty... embedded21:31
lifelessjml: shrug, We're within a few months of dropping the zope testrunner entirely.21:31
jmllifeless: frabjous day etc21:31
lifelesslet me just ask the bandersnatch for that lovely vorpal sword21:32
* thumper restarts server21:38
jmlI am going to bed. Happy Thursday, future folk.21:40
jamabentley: I rememebered what I was going to say. The new DVCS "Veracity" has gdfo in its data structures (calling it 'generation')21:41
abentleyjam, ah.  Cool.21:41
jamabentley: they have some fairly interesting C code for finding LCA, etc, in graphs fairly well documented21:41
jamI think it comes from whatever their earlier VCS was21:42
jamthey also seem to do something where "merge" will merge multiple tips in one go21:42
LPCIBotProject db-devel build (87): STILL FAILING in 3 hr 51 min: https://hudson.wedontsleep.org/job/db-devel/87/21:47
=== Ursinha is now known as Ursinha-afk
flacostethumper: udd meeting is starting in #ubuntu-meeting22:02
thumper:(22:02
thumperwas just about to go and make another coffee22:02
wallyworldabentley: rockstar: looks like thumper has a meeting now. so our standup will be delayed22:03
thumperhopefully not for too long22:04
rockstarwallyworld, want to have a chat about merge queues?22:04
wallyworldrockstar: ok22:04
mwhudsonhm, is the disable thread check fix in db-devel yet?  if so, that hudson failure is a bit :(22:12
lifelessmwhudson: hudson lies22:13
mwhudsonheh22:13
lifelessmwhudson: because the zope->subunit glue reports leaked threads as errors22:13
mwhudsonhow so?22:13
mwhudsonoh right22:13
lifelessmwhudson: we need to filter them out, its separate to the disable_thread_check flag22:13
mwhudsonok22:13
jcsackettback22:14
mwhudsongrar, the same issue bounced testtools22:14
lifelessmwhudson: from ec2?22:14
mwhudsonlifeless: yeah22:14
lifeless:<22:14
mwhudsonoh no22:14
mwhudsonwell22:14
mwhudsonthat didn't have your fix in the branch tested22:14
mwhudsonsame with lp-service22:15
lifelessmwhudson: StevenK: https://bugs.edge.launchpad.net/subunit/+bug/66417122:15
_mup_Bug #664171: support filtering on tags <subunit:New> <https://launchpad.net/bugs/664171>22:15
sinzuiEdwinGrubbs, I think your change broke some menus. This kind of check now fails: self.user.inTeam(self.context)22:18
lifelesssinzui: if it did, can you mark the bug bad-commit=xxx to stop it deploying? thanks.22:19
sinzuiEdwinGrubbs, the cases look like where we know a user is a member of himself22:19
EdwinGrubbssinzui: yes, I fixed that error already and landed a branch for it. Unfortunately, I couldn't get anyone to restart lucid_lp without restarting everything.22:20
lifelessEdwinGrubbs: did you tag it bad-commit ?22:20
EdwinGrubbslifeless: no, I've never heard of bad-commit before.22:21
lifelessEdwinGrubbs: errata: its been announced on the list repeatedly as part of the RFWTAD workflow.22:22
lifelessI'm just grabbing the wiki page about it for you22:22
lifelesshttps://dev.launchpad.net/QAProcessContinuousRollouts22:23
wgrantthumper: So WIP MPs will be usable from the next rollout?22:26
thumperwgrant: hopefully even before that (from edge)22:27
thumperwgrant: or whatever we are using then...22:27
wgrantthumper: Hm, aren't they sent by a script?22:27
thumperwgrant: yes...22:27
thumperso?22:27
thumperthe job is created by the appserver22:27
wgrantOr is it the same job, but now repurposed?22:27
thumpersame job22:28
wgrantAhh.22:28
thumperjust not triggered on wip22:28
thumperand triggered on wip -> needs review22:28
rockstarwallyworld, you keep cutting out.  I can't understand you.22:29
wallyworldrockstar: i can hear you22:29
lifelessthumper: wgrant: remember edge deployments are stopping22:29
lifelesslike asap22:29
thumperlifeless: where does qastaging email go?22:30
lifelessthumper: same story as staging I hope22:30
thumperdon't hope...22:30
thumpermake sure :-)22:30
lifelessthumper: well, i can't tell, can I ?22:30
lifelessthumper: you can try, trigger something on qastaging22:31
lifelessif you get mail, we need to change stuff22:31
lifelesshttp://qastaging.launchpad.net/22:31
thumperoops 1754QS12 on that22:32
lifelesshttps://qastaging.launchpad.net/ worked for me22:32
lifelessI think it may have been paged out22:32
lifeless:>22:33
wallyworldrockstar: you ended the call?22:34
rockstarwallyworld, headset is gone.22:34
wallyworldbollocks22:34
thumperwhatever we have behind qastaging seems to be timing out a lot22:35
lifelessits the same as staging22:36
lifelesssame boxes for everything22:36
rockstarabentley, stand up?22:37
abentleyrockstar: sure.22:38
abentleyrockstar: who's hosting?22:38
jammwhudson: any chance you can lp-land my patch? mars at least approved of doing it. Or you can ec2land it, as they claimed the windmill stuff is fixed in devel23:12
rockstarwallyworld, https://code.edge.launchpad.net/~rockstar/launchpad/merge-queues-model/+merge/3876223:12
mwhudsonjam: i've just sent it off to ec2 again23:13
jammwhudson: thanks23:15
mwhudsonjam: this has been a pretty long drawn out process, i hope your next launchpad branch goes more easily :)23:16
thumperLoading results failed. for a person search on edge23:23
thumperhas this been fixed or looked at?23:23
lifelessits the person search23:24
lifelessedwin has a fix in progress23:24
=== Ursinha-afk is now known as Ursinha
marsthumper, I was writing a bug for that same problem - it breaks merge proposals too, you can't request new reviewers23:40
mwhudsonusually middle click will short circuit the js23:42
marsthumper, bug 66421423:43
_mup_Bug #664214: "Request another review" search returns "Loading results failed" <Launchpad Bazaar Integration:New> <https://launchpad.net/bugs/664214>23:43
marsmwhudson, oops, doesn't work for MPs23:44
marsthumper, you are right, the "Subscribe someone else" link on bug reports has stopped working too23:46
marsI'll file a new bug about it23:46
* thumper nods23:47
* thumper afk for lunch meetup23:48

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