/srv/irclogs.ubuntu.com/2011/08/29/#launchpad-dev.txt

* jelmer will have a look at the merge conflict00:05
wgrantHah.00:06
wgrantI was just going to ask if you were still around.00:06
wgrantThanks.00:06
pooliehi wgrant00:53
wgrantMorning poolie.00:59
wallyworldwgrant: with the webservice api if something is exported for version 'beta', can i change it?02:10
StevenKEdit the decorator in the interface02:11
wallyworldStevenK: you are meant to be away? anyway, you mean edit the decorator to change the export_for_version() ?02:12
wallyworldi didn't think that was allowed?02:12
StevenKwallyworld: (Where are your multiple underscores?) Just because I'm holidays and not working doesn't mean I can't answer questions.02:13
wgrantwallyworld: It depends. What are you changing?02:14
wallyworldi need to change the semantics of the private attribute on IBranch02:14
wgrant(note that beta is officially EOLed, but 1.0 is still supported, but we will change things still if required)02:14
StevenKChanges to API functions need to be considered very carefully.02:14
wallyworldit is read/write (via setPrivate) but i need to to be readonly02:14
wallyworldand there's a new explicitly_private attribute instead02:15
wallyworldi could keep it read/write and add a new mutator method for explicitly_private02:16
wgrantI would just break compatibility.02:16
wgrantThat02:16
wgrant's the sort of thing that nobody is likely to care about, and those that do can fix their code.02:16
wgrant1.0 exists mostly to not break code in Ubuntu releases.02:16
wallyworldthat's what i was hoping for :-)02:16
wgrantNone of which uses Branch.private.02:16
wallyworldcool. it will still appear the same for r/o access02:17
wallyworldbut now you write using 'explicitly_private' instead of 'private'02:17
luke-jrIs there a git-supporting branch yet?02:17
wgrantwallyworld: Right.02:18
wgrantluke-jr: No.02:18
nigelbMorning!05:04
nigelbwgrant: I'm going to take a screenshot of you telling lifeless to go away :P05:04
wgrantHeh.05:04
nigelbI'm trying to do a html = create_initialized_view(person, '+index')()05:06
nigelbthe html is turning out empty.05:06
nigelb(this is in registry)05:06
nigelbwhat am I doing wrong?05:06
wgrantIs person actually a Person?05:21
nigelbwell, yes. I used factory05:23
nigelbThis is right, isn't it? person = self.factory.makePerson(time_zone='Asia/Kolkata')05:23
wgrantThat is indeed correct.05:24
wgrantHmm.05:24
wgrantWhat is returned from create_initialized_view(person, '+index')?05:24
nigelbempty html as far as I can tell from the test failure05:25
nigelberr, empty string05:26
nigelbThe user story from the guy earlier is depressing.05:29
nigelbAlso, slightly troll-ish.05:30
wgrantMmm, it's pretty reasonable.05:39
wgrantLP hasn't changed significantly since 2005.05:39
wgrantUI-concept-wise, that is.05:39
nigelbSure, but LP has a slightly different concept for branches05:40
nigelbIts tied to a project and not a person.05:40
wgrantAlthough all those fields he talks about are hidden when unset, except when you're looking at your own page...05:40
nigelbYeah.05:40
wgrantRight, but registering a project and stuff is difficult and awkward.05:40
nigelbThough, I'd love to see UI improvements.05:41
nigelbDoes making private branches need an admin? I've never created one or used one.05:41
wgrantlet's not go there.05:41
spmnigelb: not only is that sequence of lines - wgrant telling lifeless to go away - worthy of being screenshotted. I'm tempted to also print out said shot and place prominently on the wall. for whenever I need giggles.05:42
nigelbspm: That's the point of the screenshotting! Lets frame it ;)05:42
wgrantnigelb: My theory is that Launchpad's privacy features were designed by GitHub.05:42
spmframing might be considered excessive. but not by me. good idea™05:42
nigelbwgrant: Oh. That bad?05:42
wgrantGitHub's team infiltrated Launchpad before GitHub was created.05:43
wgrantAnd designed Launchpad's privacy features.05:43
nigelbheh05:43
wgrantTo make GitHub win,.05:43
wgrantIt is probably the most hopelessly incoherent and useless aspect of Launchpad.05:43
nigelbAre UI improvements on the way? Because even bugzilla is getting a facelift soonish.05:43
wgranthuwshimi has plans.05:44
wgrantBut we're without him for a while.05:44
wgrantBut I don't really think he can do it on his own.05:44
wgrantand UI designers never stay with us for long.05:44
wgrantSo the outlook is not stunning.05:44
nigelbOh.05:45
LPCIBotProject db-devel build #827: STILL FAILING in 4 hr 41 min: https://lpci.wedontsleep.org/job/db-devel/827/05:45
wgrantMost of the points in that blog entry are reasonable, apart from eg. condemning LP for not offering free private branches.05:46
nigelbI wish launchpad were more social.05:46
wgrantAhh, and then he goes into the usual "why can't I check out the branch I created thing".05:47
wgrantGitHub and BitBucket have poisoned everyone's minds :(05:47
nigelbWait, you can't checkout a branch you just created on github.05:47
wgrantCan't you?05:47
wgrantThe difference with GitHub is that you have to do a server-side fork.05:47
wgrantRather than just pushing up a new branch.05:48
wgrantSo people expect that from LP.05:48
wgrantNobody had a problem with just pushing before GitHub came along.05:48
nigelbHeh.05:48
nigelbThe problem is, well, they're good. With good UI.05:48
nigelbLaunchpad's UI has never been its strenghts.05:48
nigelbCode review, however, is.05:48
wgrantEh, not much had good UI in 2005.05:49
wgrantIt's just every other website in the world has moved on.05:49
nigelbHeh, that too!05:49
nigelbAnyone tried mergebox yet?05:49
nigelbwgrant: Also, any clues with the test I can't get to work?05:50
wgrantnigelb: Ah, sorry, got distracted. I was interested in the output of create_initialized_view(person, '+index'), *without* the trailing ().05:51
wgrantIt should give you a view object.05:51
wgrantThe trailing () executes the view to turn it into HTML, which I want to avoid.05:51
wgrantSo we can see what the view is.05:51
nigelbOkay, what's the easiest way to output a view object?05:52
wgrantJust get the repr for now.05:52
wgrant"I want to make a branch and then be able to push it back to the original, through the web interface. This should and has to be easy to do and not require juggling between the LP server and my local PC in order to either branch or merge."05:53
wgrantI'm confused.05:53
wgrantDoesn't one normally develop on one's local PC, necessitating that juggling?05:54
nigelbhm, does github/whatever support merging from the web UI?05:55
nigelbI thought not yet.05:55
wgrantI'm not sure.05:56
wgrantBut his thing didn't seem to mention actually merging.05:56
nigelbBUt that's what it is right? Create a branch, push back into original.05:56
nigelbbut why would you create a branch and push to LP if you don't want an MP.05:57
nigelbHm.05:59
nigelbAssertionError: 'Asia/Calcutta' not in '<zope.browserpage.metaconfigure.SimpleViewClass from /home/nigelbabu/launchpad/lp-branches/188187-time-zone-offset/lib/lp/registry/browser/../templates/person-index.pt object at 0xe97830c>'05:59
wgrantHmm. So that should render to more than ''...05:59
nigelbYeah, I did try to figure out and then give up.06:00
nigelbI seem to be having a complicated relationship with tests :D06:01
wgrantLaunchpad has that effect on people :)06:02
nigelbHeh.06:02
nigelbI thought I'd get this done without asking too much help, etc. Bam! Tests!06:03
nigelbwgrant: What should be the next move?06:07
nigelbI wonder if I can skip tests for this :D06:11
wgrantNo, this definitely needs tests :)06:21
wgrantLet's see...06:21
wgrantAlmost got my new lucid-lxc-on-oneiric development environment set up adequately.06:22
nigelbI still don't see the charm of developing on bleeding edge.06:22
wgrantHuh.06:22
wgrantIt's '' here too.06:23
nigelb\o/06:23
nigelbI'm not going mad!06:23
wgrantUsing oneiric is nice.06:23
wgrantGet to iron out bugs, whine to thumper about the new unity being crap, fglrx 11.8 is slightly less awful than 11.5...06:24
nigelbthumper works on Unity?06:24
wgrantHe defected to the Unity team earlier this year.06:24
* nigelb had a vague feeling he used to be LP or bzr.06:24
wgrantHe was lp-bzr.06:24
wgrantOr lp-code, as it was known in its later years.06:24
nigelbAh06:24
nigelbRight now, there isn't a subteam working on this component is there? All squads working on bits of everything.06:25
pooliehi nigelb06:25
wgrantRight.06:25
nigelbHey poolie, Morning!06:25
nigelb(well, its probably evening for you by now :P)06:25
poolieafternoon06:26
wgrantnigelb: Is your test in the same class as other view tests?06:26
nigelbYeah, let me grab that class name06:27
nigelbIts inside TestPersonIndexView06:27
wgrantI'm not quite sure what's going on here. I guess it's not using the root template that includes the slots.06:35
wgrantnigelb: Where are you adding this?06:37
wgrantThe "Time zone" section under "User information"?06:37
wgrantnigelb: If so, try asking for +portlet-contact-details instead of +index.06:39
wgrantMore specific, and it will probably even work.06:39
nigelbwgrant: doing that now06:40
wgrantAlthough nothing seems to render that directly either.06:41
wgrantYou may just have to give up and test an attribute on the view and not the actual display.06:41
wgrantOr resort to a browser test.06:41
nigelbbrowser test?06:41
wgrantgrep for user_browser06:42
nigelb\o/06:42
wgrantzope.testbrowser is a minimal web browser that exercises most of the Zope stack.06:43
nigelb+portlet-contact-details worked06:43
wgrantSo you'll be able to see the output of the view that way.06:43
wgrantGreat.06:43
wgrantAha.06:44
nigelbGah, so I get to use doctests again. Win.06:46
nigelbok, looks like everything works.06:53
* nigelb forces a failure06:53
nigelbaaargh. lint failure.06:54
nigelbI should be fixing lint failure that I didn't cause, right?06:54
nigelbOk, fixing this lint failure might lead to breakage.06:56
nigelbwgrant: I get a bunch of "assigned to but never used" lint failures.06:59
wgrantnigelb: In doctests?06:59
nigelbIts unrelatd to code I touched.07:00
nigelbBut in the files I touched07:00
wgrantIf it's not a doctest, you should just be able to remove the assignment.07:00
wgrantie, s/a = b/b/07:00
nigelbA bunch of these http://pastebin.ubuntu.com/677037/07:01
wgrantThose should all be trivially fixable.07:01
nigelbliene 308 is interesting.07:02
nigelbI think the view is created so that a login token exists.07:02
wgrantYeah, you probably need to continue creating the view, but you don't have to assign it to anywhere.07:03
wgrantThe return value is not important.07:03
nigelbaah07:03
nigelbwgrant: what does the last one in that lint failure mean?07:08
wgrantnigelb: Normally that there's a = or + without a space on both sides.07:09
nigelbwgrant: Could you review https://code.launchpad.net/~nigelbabu/launchpad/188187-time-zone-offset/+merge/7316807:14
nigelbstub doesn't seem to be here yet.07:14
rvbaMorning all.07:20
thumperwgrant: not entirely sure that defected is the right word07:33
wgrantthumper: Escaped?07:34
nigelbheh07:34
nigelbthumper: "ran away"07:35
thumperhow about, "needed a change"07:35
wgrantBah.07:35
nigelbA lot Canonical seem to be Launchpad defectors. :P07:36
wgrantHeh.07:36
nigelbwgrant: You picky picky person :D07:36
wgrantI do that :)07:38
nigelbwhen I do tal:foo07:38
nigelbfoo is a variable?07:38
wgrantIt's irrelevant.07:39
wgrantmeans nothing at all.07:39
nigelbwgrant: Updated!07:48
nigelbpoolie: ooh, that's a nice next bug :D07:48
pooliethe team timezones thing?07:48
poolieyeah07:48
poolieit has limits07:49
poolieespecially as you know nothing about people's habitual online hours07:49
nigelbTrue07:49
nigelbI set my timezone on Launchpad as UTC.07:49
adeuringgood morning07:49
pooliei think thisi s one thing people specifically said they missed when we took out the google maps display07:49
pooliehi abel07:49
nigelbWhy did we take out maps? I remember there was some javascript error and then it wasn't around the next day.07:50
pooliei think part of it was about google maps over https being a for-pay service07:50
wgrantnigelb: Google wanted to charge a heinous sum for HTTPS maps.07:50
poolieand maybe some licencing problem07:50
wgrantBut I believe that is no longer an issue.07:50
nigelbOh.07:50
pooliepeople complained about it being nonfree07:51
wgrantAs they have decided that HTTPS should be everywhere now.07:51
wgrantBut it has been suggested it should be replaced with OSM, yes.07:51
nigelbI'm always about what works vs non-free :)07:51
pooliethey have decided SPDY should be everywhere07:51
nigelbWe had something like that in Loco directory using google maps.07:51
nigelbAn OSM developer requested we not use OSM because that's not what its meant for.07:52
poolieoh?07:52
poolieit's meant for precise positioning?07:52
wgrantIt's interesting that LP has been HTTPS-only forever, and many people used to complain, and now the rest of the world is finally starting to go HTTPS-only. We were years ahead of the curve on one thing, at least.07:52
poolieyes, which is quite cool07:53
nigelbwgrant: We're ahead only in one curve :)07:53
poolieof course many of these maps are not all that accurate07:53
nigelbpoolie: Because OSM is not ready to get hit with lots of people loading pages with it or something.07:53
wgrantnigelb: Indentation on prejoins has the same indentation issue.07:53
nigelbGRAR07:54
pooliejust showing "... which is on the south-east coast of india" would be enough07:54
nigelbwgrant: okay, fixed.07:56
nigelbpoolie: But still, that doesn't help when you want to figure out when is a good time to set a meeting with someone :)07:56
stubpoolie: If we wanted to be invasive, we probably could work out an awful lot about peoples habitual online hours :-)07:56
wgrants/invasive/Google/07:57
nigelbNo, please don't. My hours are horrible.07:57
nigelbI slept at 4 am, woke up at 11, and I'm now going to work at 1:30 pm. Fun.07:57
poolieperhaps it can be a separate service07:58
poolieoh i guess if we had a timeline view07:58
wgrantHeh.07:59
nigelbtimeline of commits? that should be fun.07:59
nigelbshould also overlap with marking who's on vacation when.07:59
wgrantAward karma points if you don't obtain any karma between midnight and 8am in your designated timezone.07:59
pooliebadges07:59
pooliefix one bug every hour for 24h08:00
nigelbheh.08:00
nigelbDouble karma between 10 pm and 8 am your timezone.08:00
stubI prefer the first variant to try and stop burnout rather than encourage it :)08:00
nigelbHeh08:01
nigelbI'll then reset the timezone to the timezone my brain lives in.08:01
nigelb"Oh, I'm working in Hawaii time today.  Iceland tomorrow!"08:01
wgrantI've filed 50 LP UI bugs in 24 hours, but not fixed 24...08:01
stubYer, need roving to cope with my 25 hour cycle :-/08:02
nigelbLaters! Off to lunch and then work.08:03
nigelbwgrant: Gah. formatimports. I knew I missed something.09:20
=== henninge changed the topic of #launchpad-dev to: #launchpad-dev ist: https://dev.launchpad.net/ | On call reviewer: henninge | Critical bugs: 237 - 0:[#######=]:256
wgrantnigelb: When you fix, I will send it off to ec2.09:31
wgrantAlthough it won't land until jelmer hopefully sorts out the testfix on db-devel.09:31
nigelbI'll fix when I get home. In about 5 hours.09:32
wgrantnigelb: Thanks!09:32
nigelbOne more inch closer to 128.09:32
jelmerwgrant: whoops, I hadn't seen that yet09:47
wgrantjelmer: Ah, wasn't expecting you around yet.09:49
jelmertestfix is on the way10:41
wgrantjelmer: Thanks!10:45
=== henninge is now known as henninge-lunch
jelmerin other news, bzr code imports seem to work on qastaging :)11:04
jelmerhttps://code.qastaging.launchpad.net/~jelmer/bzr-svn/revprops11:04
jelmerwgrant: and it looks like staging was updated?11:20
=== luke-jr_ is now known as luke-jr
wgrantjelmer: Indeed, spm unbroke it this afternoon :)11:43
wgrantjelmer: What UI is there for them? "Import details" doesn't seem to show anything...11:45
jelmerwgrant: The UI bit hasn't landed yet, the only way to create them on qastaging is using the API11:45
wgrantjelmer: Ah, that would do it.11:45
jelmerwgrant: I have another branch up for review that changes the web UI to register "bzr" code imports rather than mirrors11:45
wgrantGreat.11:46
jelmerthen someday, perhaps, we can get rid of the branch puller11:46
wgrantWell, we'd need to fix the way imports work first.11:47
wgrantThey still use the puller :(11:47
jelmeryeah, that bit I'm still pondering11:48
wgrantBut that is a significantly easier problem than general branch mirroring.11:48
jelmera related issue is that it would be really useful to do stacking of code import branches, but the current architecture makes that hard (especially if the development focus is not a code import)11:48
wgrantIndeed.11:48
wgrantI think they should probably just be bzr+ssh clients or so.11:49
wgrantOr maybe use the internal branchfs.11:49
matsubaraadeuring, hey, good morning. could you take another look at that oops-tools branch you reviewed on Friday? https://code.launchpad.net/~matsubara/oops-tools/829460-typeerror-qsmmx/+merge/7306211:49
adeuringmatsubara: sure, but I'd like to have lunch first :)11:49
wgrantAssuming I don't accidentally delete all the hosted areas again.11:49
matsubaraadeuring, sure.11:50
jelmerwgrant: :)11:50
wgrantjelmer: I guess we probably don't have to consider stacking on private branches.11:51
wgrantWhich might make things easier.11:51
jelmerwgrant: What about private users who are importing multiple branches?11:52
wgrantjelmer: We don't support credentials in URLs, do we?11:53
wgrantSo private imports are not hugely likely at the moment.11:53
jelmerwgrant: We do, at least for bzr-svn/bzr-git/bzr-hg imports (some people have silly requirements like logging in with guest/guest. Not sure about mirror URLs.11:53
wgrantBah.11:53
wgrantOK.11:53
wgrantSo we just have to grant the service user access to dev focus branches sometimes too, I guess.11:54
jelmerwgrant: hmm, yeah11:56
jelmerwgrant: I guess just supporting stacking on public dev focus branches would be easiest for now, and already be a huge improvement over the current situation.11:57
wgrantIt's all going to be special-cased anyway, because branches can be owned by anyone.11:57
wgrantBut only owners can write.11:57
jelmerdoes zope support impersonation ? :)11:57
wgrantIt can, but LP doesn't use it.11:58
wgranteg. SCA is special-cased in security adapters.11:58
wgrantI imagine this would, for now, be something similar.11:58
jelmerah, interesting11:59
StevenKWe just killed bazaar-experts, now you want to resurrect it?12:00
jelmer:) I'm glad that's gone..12:01
wgrantA celebrity for the importds is probably best for now, I fear. Unless we want to give them access to the branchfs, which sounds bad.12:01
StevenKjelmer: The team still exists, since lifeless is a wimp. But it is no longer a celebrity.12:02
StevenKwgrant: That makes me very sad.12:02
=== benji changed the topic of #launchpad-dev to: ist: https://dev.launchpad.net/ | On call reviewer: henninge, benji | Critical bugs: 237 - 0:[#######=]:256
wgrantStevenK: For now.12:02
wgrantStevenK: Until we have a less bad permission system.12:02
wgrant"now" in Launchpad time -- that is, at least a decade.12:03
jelmerwgrant: I agree, about having a celebrity12:04
=== henninge-lunch is now known as henninge
jelmerwgrant: Is it just my change of perception, or is Launchpad development quicker these days?12:05
StevenKbenji: O hai -- did Irene spare you her wrath?12:05
wgrantjelmer: Hmmm.12:05
wgrantjelmer: I think it depends where you're working.12:05
wgrantAnd how you're working.12:05
wgrantand what you're doing.12:05
wgrantBut it may be.12:05
benjiStevenK: yep, it wasn't too bad here; we just got a good dose of rain12:06
StevenKbenji: Ah, rain is easily coped with.12:06
wgrantjelmer: Code's always been quicker than the rest, I think.12:06
wgrantjelmer: Because it's been designed.12:06
wgrantAt least a bit.12:07
wgrantAt least I've always found it nice.12:07
rvbahenninge: benji Hi guys, could any of you have a look at this (Javascript) MP https://code.launchpad.net/~rvb/launchpad/confirmation-overlay-bug-830982/+merge/73078 ?12:07
benjirvba: I'm available.12:08
jelmerwgrant: hmm12:08
rvbabenji: Thanks a lot!12:08
henningebenji: I'd like to take ;)12:08
benjihenninge: be my guest12:08
lifelesswgrant: can you do me a favour tomorrow? put the performance tuesday tag on in your morning12:08
wgrantlifeless: Only if you go away, but sure :)12:08
nigelbheh12:08
nigelbmoar screenshots.12:09
lifelesswgrant: given I just drove back from the hospital and am about to crash for 6 hours; I can guarantee I am going away.12:09
wgrant:)12:09
wgrantNight!12:09
lifelessnight, thanks!12:09
=== wgrant changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: henninge, benji | Critical bugs: 244 - 0:[#######=]:256
wgrantdanilos: Hi.12:10
wgrantHmmm.12:28
wgrantThis is interesting.12:28
wgrantTrying to upload via FTP to a poppy running in LXC, and I have the hanging-on-the-last-kilobyte bug.12:28
wgrantbigjools: ^^12:29
bigjoolsnice12:29
wgrantReproducible too.12:29
bigjoolsrepeatable for any upload?12:29
* wgrant debugs.12:29
wgrantDoes suggest it's odd networking, though.12:29
bigjoolsyes12:29
bigjoolsdoes the LXC NAT?12:29
wgrantI'm using libvirt's bridge.12:30
bigjoolsI'll lay real money on it being a nat type issue12:30
wgrantIt does NAT, but this shouldn't be using it.12:30
wgrantBecause it's host-to-guest.12:30
StevenKHaha. How much real money?12:30
bigjools2 pence12:30
wgrantSo should just go straight over the internal bridge without NAT.12:30
bigjoolsit'll be doing something internally still12:30
StevenKSo given the UK currency that's about 0.5 Australian cents12:30
wgrantLet's hope.12:30
bigjoolsStevenK: miaow12:31
wgrantWow.12:31
wgrantWireshark doesn't hang Unity!12:31
wgrantA complex application that doesn't hang oneiric's unity is a rarity these days.12:32
nigelbwell, its moved from simple to complex, that's worth rejoicing.12:32
wgrantAh, there we go, lagging up a bit now.12:32
=== jtv is now known as jtv-eat
wgrantHmm.12:39
wgrantSo it does the gpg verification, then doesn't send anything back.12:39
StevenKwgrant: I wonder if an SFTP upload also hangs with LXC involved12:40
wgrantI doubt it.12:40
wgrantIt works.12:41
bigjoolsremember that this happened with the zope server too12:41
* bigjools -> food12:41
wgrantYeah, that is pretty odd.12:42
wgrantSo, it turns out that in this case it's the GPG verifier crashing.12:53
StevenKNice!12:54
StevenKDo you have a traceback?12:54
wgrantBut I can't see the exception by default...12:54
wgrantNot until I add any extra log entry right before the crash.12:54
wgrantonce I do that, I get an error traceback.12:54
wgrantWTF twisted?12:54
wgrantHmm.12:55
wgrantI wonder if logging doesn't get completely initialised at startup.12:55
wgrantWriting to the Python log somehow finishes setup, allowing the Twisted logger to log.12:56
StevenKCan you paste the traceback?12:56
wgrantThe problem with it is clear: the "key not registered" error is unicode, while twisted wants a str.12:57
wgrantBut this doesn't explain all the hangs, sadly.12:58
wgrantIt may explain why we can't work them out, though.12:59
StevenKI wonder if that can tested sanely12:59
deryckMorning, all.13:02
adeuringmatsubara: r=me13:03
adeuringmorning deryck13:03
matsubarathanks adeuring13:03
nigelbmm, LP commits visualized. http://www.youtube.com/watch?v=9Pe70fw83Zs13:15
=== jtv-eat is now known as jtv
deryckadeuring, hey, could you join me in mumble for 2 seconds?  Just need to test I'm working.13:23
deryckadeuring, ping for standup.13:31
adeuringderyck: ok13:31
jelmerhenninge, benji: can I add a MP to your queue?13:33
benjijelmer: sure13:34
=== henninge changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: benji | Critical bugs: 244 - 0:[#######=]:256
jelmerbenji: thanks13:34
jelmerhttps://code.launchpad.net/~jelmer/launchpad/no-code-import-approval/+merge/7318713:34
henningebenji: I won't take another after this one.13:34
henningeI mean, the one I am working on.#13:34
benjihenninge: k13:34
flacosteadeuring: hi, thanks for following up on bug 83510313:35
_mup_Bug #835103: HWDB submissions since Lucid are marked as Invalid <hwdb> <regression> <Launchpad itself:In Progress by adeuring> < https://launchpad.net/bugs/835103 >13:35
flacosteadeuring: do we need to open a task against checkbox because of the missing sysfs-attributes?13:35
=== brianchidester_ is now known as brianchidester
matsubaraanyone have an idea how to solve this: https://code.launchpad.net/~matsubara/oops-tools/829460-typeerror-qsmmx/+merge/73062? it's not finding pytz in the download-cache even though I added it there...13:39
stubmatsubara: you running with Python2.5 or something like that?13:44
matsubarastub, it's the tarmac chroot for oops-tools. I'm not sure what version of python is running there13:45
matsubaralosas: ping13:45
matsubarastub, adding pytz-2011h-py2.5.egg to the download cache would solve the problem, right?13:45
stubit would work around the problem13:46
stub(if it does, you still have the problem that our code is only known to work with Python 2.6)13:46
adeuringflacoste: yes, I think we need a checkbox task too -- at least for the next version of checkbox13:49
matsubarathat's what's in the LTS release, isn't it? maybe I should request that the chroot to be update to 2.613:49
deryckabentley, just need to reboot after upgrade, and I'll ping for pre-imp call.13:49
abentleyderyck: cool.13:50
flacosteadeuring: do you have an ETA on the fix and the time it will take to re-parse all the invalid submissions?13:50
adeuringflacoste: I hope to have a fix tormorrow. Processing one submission need 1 or 2 seconds; multiply that by the number of broken submissions for the total processing time13:51
adeuring...and allow for some pausing13:51
flacosteadeuring: how can we re-process the invalid files? we flip the status back to processed and allow the script to do its normal thing?13:52
adeuringflacoste: exactly13:52
flacosteadeuring: is it DBLoopTunabled yet?13:52
adeuringflacoste: yes13:53
flacosteawesome!13:53
flacosteadeuring: do we record the schema error as an OOPS? maybe user-generated one?13:53
flacosteor it was only the logs?13:53
adeuringflacoste: I think this is only in the logs13:54
flacosteok, we should probably record oopses13:54
abentleydanilos: when you commit changes to sourcedeps.conf, please also commit the changes to sourcedeps.cache.13:54
flacostethat would have made this error show in our report13:54
flacostei'll file a bug for that13:54
deryckflacoste, I suggested an integration test of some sort between hwdb/checkbox as well as lpstats for invalid submissions in the bug.  FWIW.13:55
adeuringflacoste: well... people may submit totally bogus data, so this would mean also some potential noise in the OOPS reports13:55
flacosteadeuring: you mean checkbox can submit invalid data?13:55
flacosteor since it's a user-accessible service, someone could upoad bogus data?13:55
flacostein any cases, we could treat those as "user-generated" error13:56
adeuringflacoste: well, we want to record this :) But what if people send deliberatley nonsensical data, by messing with checkbox? Or by simply uploading arbitrary files?13:56
flacostethat's not really an issue either way13:57
flacostederyck: the integration tests should probably live in checkbox13:57
deryckflacoste, yeah, agreed.13:58
flacostegary_poster, matsubara: where does the QA-SLA document for API scripts live? (for things like apport)13:58
* deryck has to reboot now13:58
flacostethe submission HWDB isn't our regular API, but I think the thoughts there should work13:58
gary_posterabentley, jam approved https://code.launchpad.net/~gary/bzr/bug835035/+merge/73124 .  I have three questions now for when you have a moment.13:59
gary_poster(1) I have an XXX comment in there that jam did not comment on, even though I highlghted it in the cover letter.  He approved the branch as-is, so should I just not worry about it, or should I...do something else (remove the XXX, remove the comment, ...)13:59
gary_poster(2) What should I do to actually get this MP landed on the 2.3 line?13:59
gary_poster(3) How do I make an egg for LP to use?  egg_info does not seem to be honored.  Should I change bzrlib.version_info from (2, 3, 5, 'dev', 0) to (2, 3, 5, 'dev', 1) or something, and then run ./setup.py build?13:59
jamgary_poster: ask us :)13:59
gary_posterjam, hi :-)13:59
gary_posterdidn't realize you were here13:59
jamyou proposed it to 2.3, so once I get off my tukus and submit it, it will land in 2.313:59
jamfor ~1 more hour14:00
gary_posterjam, ok cool :-)14:00
jamgary_poster: so a few things now that I think of it14:00
jam1) It should be a per_repository_reference test14:00
gary_posterflacoste, not sure; I think it is on wiki.canonical.  looking14:00
jamsince it only needs to be run on repositories that support fallbacks14:00
matsubaraflacoste, gary_poster: https://wiki.canonical.com/Launchpad/PolicyandProcess/ApiSupport14:01
gary_posteryay matsubara, thanks :-)14:01
matsubaranp14:01
jam2) Could we not hard-code the target format, but use a generic 'incompatible' fallback format?14:01
jamthat way if we introduce a new Repository, it also will have to conform to this behavior14:01
flacostematsubara: thanks14:01
jam(you could either do it by picking a format that is too old for stacking, or by just saying "if 2a, then 1.9, else 1.9"14:01
flacostematsubara, gary_poster: is the comment at the top of the wiki page still true? waiting on the releasing of the OEM scripts?14:02
jamelse 2a)14:02
abentleygary_poster: I'll leave you two to discuss it, but feel free to ping me.14:02
jelmergary_poster: fwiw bzr 2.4 has landed on db-devel and should land on devel at the end of this week, barring any issues that come up during QA14:03
=== salgado_ is now known as salgado
jamjelmer: nice. Though it still doesn't hurt us to target 2.3 to start14:05
gary_posterjam, re (1): I'll look for that string and see if I can figure out, cool.  re (2): yeah, I thought of something like that but I wasn't quite sure it would be correct.  I'll try that approach then.  Does your (2) negate your (1) or not?14:05
jamgary_poster: you should do both14:05
gary_posterjelmer, great, thanks.  I intend to make a 2.4 and trunk MP too.14:05
jamif you do (2) without (1) then the target format will be something like WeaveFormat which will fail for different reasons.14:06
jamgary_poster: we automatically up-merge14:06
jamso if it lands in 2.3 it will auto-merge into 2.4 and trunk14:06
jamwell "automatically"14:06
jamit is still requested by a human, but we always do it eventually (everything in 2.3 is in 2.4, etc.)14:06
gary_posterjam, (2) and (1) both: ack, thanks.  I'll try it and get back if I have questions14:06
gary_posterjam, automatically: ok, cool.  I was thinking it would need to be manual because this function changed files between 2.3 and 2.414:07
matsubaraflacoste, not sure how that conversation ended. I requested approval and I think gary took over back then14:07
gary_poster(the function that we are changing/testing)14:07
jamgary_poster: well, if it needs updating, then we'll need something when we go to merge 2.3 up to 2.414:07
gary_posterjam, right, so just wait to let you all ask for it, or do it yourself?14:08
jamchecking14:09
gary_posterflacoste, matsubara, I think Jane had an issue with the draft and then I never followed up.  I don't remember details.  matsubara, do you have an email from Jane about it?14:09
matsubaragary_poster, yes. I forwarded it to you back then. I'll forward it again to you and flacoste14:10
gary_postermatsubara, ack thanks14:10
flacostematsubara, gary_poster: thanks, i'll follow-up from there14:10
jamgary_poster: it looks like the function in question just got moved (probably to bzrlib/vf_repository.py), I think we can sort that out once your patch has landed.14:10
gary_posterok cool thanks jam14:11
jamThe general rule is that we would wait, then base a change on the updated bzr/2.3 to merge it into bzr/2.414:11
gary_posterok14:11
bigjoolsI wonder if people ever look at the dupe finder list when filing bugs14:12
jelmerbigjools: I always assume the bug file thingy will suggest dupes to me, if there are any14:14
bigjoolsjelmer: yes.  But lately I've seen stuff that is so obviously a dupe I wonder if people pay any attention to the list14:14
gary_posterbigjools, that would involve reading, you know.14:15
bigjools:)14:15
jelmerbigjools: the list does often contain a lot of irrelevant stuff. Perhaps it shouldn't show anything if there isn't anything significantly similar.14:15
bigjoolsyeah, could be people fed up with unrelated bugs14:16
nigelbbenji: Hi!14:21
nigelbCould you review https://code.launchpad.net/~nigelbabu/launchpad/188187-time-zone-offset/+merge/73168 :)14:21
benjinigelb: sure, I'm a little behind so it might be a minute14:22
nigelbbenji: Sure, no problem :)14:22
deryckabentley, I can meet in mumble now, if you'd still like a pre-imp chat.14:30
abentleyderyck: sure, let's do it.14:30
deryckok, cool14:30
abentleyderyck: https://bugs.launchpad.net/launchpad/+bug/82840914:31
_mup_Bug #828409: cross-format stacked branches can't be accessed <branch-stacking> <codehosting> <Launchpad itself:Triaged> < https://launchpad.net/bugs/828409 >14:31
=== matsubara is now known as matsubara-lunch
=== salgado_ is now known as salgado
=== benji changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: - | Critical bugs: 244 - 0:[#######=]:256
nigelbbenji: hi15:34
nigelbbenji: still around?15:35
* nigelb needs someone to land his branch15:35
LPCIBotYippie, build fixed!15:47
LPCIBotProject db-devel build #828: FIXED in 4 hr 42 min: https://lpci.wedontsleep.org/job/db-devel/828/15:47
gary_posterjam, it turns out there was a test very similar to what I had done in per_repository_reference already.  I adjusted it.  https://code.launchpad.net/~gary/bzr/bug835035/+merge/73124 is updated with the change.  You good with that?15:55
nigelbgary_poster: would you mind sending a branch of mine to e2 land?15:56
gary_posternigelb, was just about to ask you :-)15:56
gary_posterwhich is it?15:56
nigelbheh15:56
nigelbhttps://code.launchpad.net/~nigelbabu/launchpad/188187-time-zone-offset/+merge/7316815:56
gary_posterok, on it nigelb.  I assume we need ec2 tests run?15:56
nigelbyes!15:57
gary_postercool15:57
nigelbgary_poster: Thank you! :)16:01
gary_posternigelb, welcome :-) instance is firing up now.16:01
* gary_poster should have added you to the email recipients16:02
* gary_poster hopes ec2 land does that anyway16:02
nigelbUsually, I get emails16:02
gary_postercool16:02
nigelbI assme there's nothing special to do.16:02
nigelb*assume16:02
* nigelb marks the time and counts down to 4 hours16:02
henningervba: review sent, talk to you tomorrow ;-)16:04
rvbahenninge: I'll have a look right now ... I bet I have quite a reading to do ;)16:04
henningervba: yes16:04
henninge... and some fixing ;)16:04
rvbahenninge: ;)16:05
=== deryck is now known as deryck[lunch]
henninge;-)16:05
henningervba: I am off now and will be late tomorrow morning.16:06
rvbahenninge: I'll get back to you when I have fixed it so late tomorrow sounds good.16:06
gary_posternigelb, ec2 land has disconnected from my terminal and doing its thing, so hopefully we'll hear something in four or five hours16:08
nigelbgary_poster: Cool, thanks!16:09
gary_posterjam, I'm going to lunch.  biab, and https://code.launchpad.net/~gary/bzr/bug835035/+merge/73124 is ready for you16:10
LPCIBotProject devel build #1,008: STILL FAILING in 4 hr 35 min: https://lpci.wedontsleep.org/job/devel/1008/16:26
=== matsubara-lunch is now known as matsubara
=== deryck[lunch] is now known as deryck
=== salgado is now known as salgado-lunch
jamgary_poster: nice that you could use an existing test. I tweaked it slightly because the actual test was being a bit redundant.17:19
jamI'll submit it17:19
gary_postergreat, thanks jam.17:20
jtvAny reviewers in the house?  Got this one: https://code.launchpad.net/~jtv/launchpad/bug-836743/+merge/7326717:21
=== salgado-lunch is now known as salgado
jamgary_poster: my tweak to the test is at: lp:~jameinel/bzr/2.3-gary-bug835035 if you want to look at it. It is in PQM now.17:26
gary_posterjam, cool, thanks.  I thought about that but was lazy, I guess, since the code & checks of test_add_fallback_doesnt_leave_fallback_locked subsumes test_add_fallback_repository_rejects_incompatible (maybe unless we asserted in test_add_fallback_repository_rejects_incompatible referring was not locked?); but I can see the point of dividing them.17:31
jamgary_poster: well a few things17:33
jam1) We don't need to make the 'test' repository17:33
jamjust test the actual repository we are going to use :)17:33
jam2) Your new test is strictly a superset, yes17:33
jamHowever, I thought it would be good to have an explicit test of the different behavior17:34
gary_posterjam, ah, yes, I hadn't noticed # 1!17:34
jamgary_poster: yeah, you didn't write that code17:34
gary_posteryeah, like I said, I see where you are coming from with #217:34
jamwe *do* have to 'make_repository' because RemoteRepository infers its format over the wire17:34
jamit isn't known until we query it17:34
gary_posterbut once is enough17:35
jamright17:35
=== dpm is now known as dpm-afk
=== salgado_ is now known as salgado
=== luke-jr_ is now known as luke-jr
abentleylifeless: So I assumed TwistedJobRunner.runJobInSubprocess always returns a Deferred, but sometimes it returns None.  Is it better style to fix the callsite by calling maybeDeferred, or the method by making it always return a Deferred?18:53
nigelblifeless has gone away.18:54
nigelbOr at least wgrant tried to get him to go away.18:54
=== beuno is now known as beuno-lunch
bachi abentley19:26
abentleybac: hi.19:26
abentleynigelb: :-)19:26
bacabentley: would you have time for a call on a weird OOPS re: recipes?19:26
abentleybac: Okay.19:26
bacabentley: skype ok?19:27
abentleybac: sure.19:27
bacabentley: it is about bug 82891419:27
_mup_Bug #828914: +request-daily-build oops with an AttributeError: 'NoneType' object has no attribute 'published_archives' <oops> <recipe> <Launchpad itself:In Progress by bac> < https://launchpad.net/bugs/828914 >19:27
=== beuno-lunch is now known as beuno
=== matsubara is now known as matsubara-afk
adeuringflacoste: fancy a review? https://code.launchpad.net/~adeuring/launchpad/test-hwsubm/+merge/7329420:28
flacosteadeuring: sure, i'm on it20:28
flacosteadeuring: just FYI, Lucid is also affected, we have very few valid Lucid submissions, and I guess they date from before the problematic checkbox changes20:36
adeuringflacoste: right; I'll check a few lucid reports too20:36
flacosteadeuring: do we have hwdb integration tests? If we do, is it worth adding one for a problematic report?20:37
adeuringflacoste: right, I am aware that such a test is missing -- the main point is: I am too tired to write one today...20:37
flacosteadeuring: but do we have such tests already?20:38
adeuringflacoste: only one or two: for correct data and for "trivially wrong data", IIRC20:38
flacosteadeuring: ok20:38
flacosteadeuring: r=me with the note that Lucid and later are affected, and with a reference to the bug report20:39
adeuringflacoste: ok, thanks20:40
flacostehow can we have valid person without a preferred email address ?!?21:23
thumperflacoste: we can't?21:28
thumperor perhaps better put, we shouldn't21:28
flacostethumper: i thought we couldn't actually21:29
thumperI thought we couldn't either21:29
flacosteseems that this assumptions still hold21:39
flacostei was mistaken by a bad bug comment21:39
LPCIBotProject devel build #1,009: STILL FAILING in 4 hr 50 min: https://lpci.wedontsleep.org/job/devel/1009/22:56
* wallyworld has to go out and run some errands23:03

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