| === maxb_ is now known as maxb | ||
| thumper | jml: could you enlighten me to how it's hocked up? | 00:13 |
|---|---|---|
| jml | thumper, IBranchSet is exported as a collection, getByUniqueName is a read operation on that collection | 00:14 |
| thumper | gotcha | 00:14 |
| thumper | jml: thanks | 00:14 |
| jml | thumper, np | 00:14 |
| thumper | mwhudson: when is good for you? | 00:18 |
| mwhudson | thumper: argh | 00:19 |
| mwhudson | thumper: about an hour? | 00:19 |
| thumper | mwhudson: ok | 00:19 |
| mwhudson | i haven't had lunch yet | 00:19 |
| thumper | mwhudson: at least it'll timebox our discussion | 00:19 |
| thumper | mwhudson: I have to go get the girls at 3 | 00:19 |
| mwhudson | thumper: ok | 00:19 |
| === jamalta is now known as jamalta-afk | ||
| * mwhudson lunches | 00:27 | |
| thumper | sinzui: ping | 00:50 |
| mwhudson | thumper: hello | 01:06 |
| thumper | mwhudson: hi | 01:06 |
| mwhudson | thumper: ready for a call now, but if you can wait a few minutes more you won't have to listen to me eating my lunch :-) | 01:07 |
| thumper | mwhudson: I can wait a few minutes :) | 01:07 |
| thumper | mwhudson: now? | 01:20 |
| mwhudson | thumper: yep | 01:20 |
| mwhudson | thumper: https://code.edge.launchpad.net/~vcs-imports/pydoctor/trunk | 01:31 |
| thumper | https://edge.launchpad.net/ubuntu/+source/gwibber/2.29.90-0ubuntu1 | 01:33 |
| thumper | https://edge.launchpad.net/~gwibber-daily/+archive/ppa | 01:35 |
| thumper | https://edge.launchpad.net/~gwibber-daily/+archive/ppa/+build/1514631 | 01:38 |
| mwhudson | thumper: https://edge.launchpad.net/builders/ | 01:40 |
| mwhudson | thumper: https://edge.launchpad.net/~launchpad/+archive/ppa/+build/1056751 | 01:43 |
| poolie | did abentley or anyone write a script that will automatically feed approved reviews to pqm? | 02:18 |
| poolie | or semiautomatically? | 02:18 |
| abentley | poolie, not that I know of. | 02:18 |
| poolie | maybe i will then | 02:19 |
| abentley | poolie, however, Tarmac behaves that way. | 02:19 |
| poolie | it automatically pulls? | 02:19 |
| poolie | do you think we should switch? | 02:19 |
| abentley | It automatically merges approved merge proposals and commits. | 02:19 |
| poolie | right | 02:19 |
| abentley | I think we should switch. | 02:20 |
| poolie | so presumably one should only use status:Approved if you really mean 'approved with no further changes' | 02:20 |
| poolie | not conditionally approved | 02:20 |
| abentley | poolie, true. | 02:20 |
| abentley | poolie, when we get a chance to work on merge queues again, that will be less of a concern. | 02:21 |
| poolie | i think it's ok, as long as people use it that way | 02:22 |
| poolie | it's probably a good idea anyhow | 02:22 |
| poolie | in piloting i find myself wanting a state that's not "in progress" but not "needs review" | 02:22 |
| poolie | like, it needs a pp to fix it | 02:22 |
| poolie | somewhere in between | 02:22 |
| abentley | poolie, if we can generalize it sufficiently, cool. | 02:24 |
| abentley | poolie, another option was to assign merge proposals to people. | 02:24 |
| poolie | i don't think adding another state is precisely right | 02:24 |
| poolie | maybe that's it | 02:24 |
| poolie | because actually i want to distinguish 'the author is going to go away and fix this' from 'i will' | 02:24 |
| poolie | so maybe assignment is right | 02:40 |
| lifeless | sounds like a bug | 02:47 |
| abentley | lifeless, what sounds like a bug? | 02:57 |
| lifeless | something that gets assigned to someone to do, and wanting to track that | 02:57 |
| poolie | yeah, but it's one of those vague difficult bugs | 02:58 |
| abentley | lifeless, ah. I thought you meant the discussion suggested a bug was present. | 02:58 |
| poolie | it's questionable to file it until you can make it concrete imo | 02:58 |
| lifeless | poolie: I guess I'm saying that maybe merge proposals lie on the same spectrum as specs and bugs | 02:59 |
| poolie | oh i see | 02:59 |
| poolie | IAssignable | 02:59 |
| lifeless | while I'd like to see be less crisp about what they are, and more malleable about what we want them to do | 02:59 |
| lifeless | s/while/which/ | 03:00 |
| === bjf is now known as _bjf | ||
| mwhudson | grr what's locking bugbranch | 03:49 |
| mwhudson | oh probably the scanner | 03:49 |
| poolie | lifeless: i was interested by the idea of using rpc signing rather than https | 04:01 |
| poolie | http://developer.amazonwebservices.com/connect/entry.jspa?externalID=1928 | 04:01 |
| poolie | "use ssl if at all possible though it is slower" | 04:01 |
| lifeless | poolie: yes | 04:01 |
| poolie | so i think we should fix lplib (or lazr or whatever) to hold the connection | 04:02 |
| lifeless | poolie: you still get to avoid round trips though - and note that more and more corporates are deploying SSLBump and equivalents | 04:02 |
| lifeless | poolie: ofh for sure. | 04:02 |
| poolie | 'still get to' how? | 04:03 |
| lifeless | poolie: using message signing | 04:04 |
| poolie | using message signing and http? | 04:04 |
| poolie | yes | 04:04 |
| poolie | it's kind of a shame it's not enough | 04:04 |
| mwhudson | can you upload files to the librarian without a database connection? | 04:12 |
| mwhudson | hm | 04:13 |
| * mwhudson eods | 04:30 | |
| thumper | gah | 07:36 |
| thumper | can't get balsamiq to start | 07:36 |
| thumper | :( | 07:36 |
| adeuring | good morning | 08:08 |
| mrevell | Morning | 09:07 |
| * wgrant is surprised to see the product/project rename actually happening! | 10:29 | |
| deryck | Morning, all. | 11:01 |
| bigjools | morgen | 11:04 |
| adeuring | morning deryck | 11:06 |
| didrocks | hey guys, I've at last a LP checkout working in a karmic VM. I'm tried to export and import GPG key from the launchpad API (cf bug #389872). Following jml's explanation, I have added the export_read… export_write… decorator to function but I don't have any idea of other stuff to add. For the attribute, appart from adding exported() to them, same thing, quite lost | 12:13 |
| mup | Bug #389872: GPG keys not exposed in API <api> <Launchpad Registry:In Progress by didrocks> <https://launchpad.net/bugs/389872> | 12:13 |
| didrocks | is there any doc or help I can get? :) | 12:13 |
| thekorn | didrocks, there is https://dev.launchpad.net/API/ImplementingAPIs which explains how doing such things, | 12:21 |
| didrocks | thekorn: sweet, it seems to be what I need, let get it a try :) | 12:22 |
| didrocks | thanks | 12:22 |
| thekorn | didrocks, did you push your code somewhere? | 12:22 |
| didrocks | thekorn: not yet, I'll try to get it working first and check locally before pushing | 12:23 |
| thekorn | aha, ok | 12:23 |
| wgrant | It's not the easiest export. | 12:23 |
| wgrant | Because there's no URL yet. | 12:24 |
| thekorn | right, this part is missing on the wiki page I mentioned above | 12:25 |
| wgrant | So you'll need to add a new method to PersonNavigation, and you'll need a browser:url directive in lib/lp/registry/browser/configure.zcml | 12:25 |
| didrocks | wgrant: hem, I'm already fighting to do the right thing, seems more difficult then :) | 12:53 |
| didrocks | (and I need to do the same for ssh) | 12:54 |
| === bigjools is now known as bigjools-lunch | ||
| === lionel_ is now known as lionel | ||
| didrocks | well, I pushed my branch into ~didrocks/launchpad/devel but it still needs work and I'm unsure to know understand enough the documentation for the decorators handling | 13:32 |
| === mrevell is now known as mrevell-lunch | ||
| deryck | adeuring, gmb, kfogel -- just a reminder to update the kanban board before the standup, to make walking the board quicker. | 13:36 |
| adeuring | OK, I'll try it... | 13:36 |
| deryck | matsubara -- the qa burndown seems not to be updating any longer for 10.02. | 13:37 |
| deryck | Assuming I'm still looking at the right page. | 13:37 |
| matsubara | deryck, I'll take a look | 13:37 |
| deryck | matsubara, thanks! | 13:38 |
| deryck | adeuring or gmb -- could I get one of you to review a branch for me? | 13:56 |
| adeuring | deryck: sure | 13:56 |
| deryck | I need a bugs dev to make sure I haven't missed something that exists somewhere else. | 13:56 |
| deryck | adeuring, thanks! https://code.edge.launchpad.net/~deryck/launchpad/enable-tracking-link-perms-512378/+merge/19600 | 13:56 |
| deryck | adeuring, feel free to ping me in #launchpad-review to follow up. | 13:57 |
| adeuring | deryck: r=me | 14:10 |
| deryck | adeuring, excellent, thanks! | 14:10 |
| BjornT | deryck: why don't you check whether the user has edit permission for the project? | 14:11 |
| deryck | BjornT, because I'm dumb. :-) | 14:12 |
| deryck | BjornT, this is why I wanted someone from bugs. I felt I was missing something obvious. | 14:12 |
| deryck | but I guess that's obvious beyond bugs even. | 14:12 |
| BjornT | deryck: you can do a tal:condition on something like context/required:launchpad.Edit | 14:13 |
| deryck | BjornT, ah, cool. I never knew that actually. | 14:14 |
| deryck | much simpler, too. | 14:15 |
| === mrevell-lunch is now known as mrevell | ||
| directhex | i just came across bug #139855. is there any drive from canonical to resolve the bug, or in the short term is the easiest way to track usage by mirroring a PPA off-site where you have your own access.log to parse? | 14:23 |
| mup_ | Bug #139855: Display stats about PPA usage <feature> <Soyuz:Triaged> <https://launchpad.net/bugs/139855> | 14:23 |
| === mup_ is now known as mup | ||
| deryck | adeuring, BjornT -- updated: https://code.edge.launchpad.net/~deryck/launchpad/enable-tracking-link-perms-512378/+merge/19600 | 14:26 |
| deryck | adeuring, BjornT -- look good? | 14:26 |
| === bigjools-lunch is now known as bigjools | ||
| adeuring | deryck: yes | 14:27 |
| bigjools | directhex: it's not a priority at the moment, although we'd love to do it at some point | 14:27 |
| deryck | adeuring, gmb, kfogel -- call, in two? | 14:28 |
| gmb | yep | 14:28 |
| kfogel | deryck: yup | 14:29 |
| directhex | bigjools, the post by jml presents an action list. how much of that list can be worked on by normal plebs without secret canonical web server access? | 14:30 |
| BjornT | deryck: looks good | 14:30 |
| deryck | BjornT, thanks! | 14:30 |
| bigjools | directhex: most if not all of it | 14:30 |
| bigjools | directhex: some work was done already to scan logs | 14:31 |
| directhex | bigjools, i'll take a look then. my cpu/ram seem to be on fire, courtesy of "bzr branch lp:launchpad". | 14:31 |
| bigjools | yeah it's a big branch | 14:32 |
| directhex | 243Mlaunchpad/ | 14:33 |
| directhex | right, there we go | 14:33 |
| === Chex_ is now known as ChEx | ||
| === ChEx is now known as Chex | ||
| === salgado is now known as salgado-lunch | ||
| directhex | bigjools, as i read this source, a thought occurs. how exactly do i *test* my changes? | 14:47 |
| bigjools | directhex: depends on what you want to implement, there's a lot of work to do | 14:47 |
| bigjools | UI, backend, database schema etc | 14:47 |
| directhex | bigjools, the UI is the kind of easy bit i could probably do, but unfortunately the backend stuff is a prerequisite | 14:48 |
| bigjools | most of the existing testing is done in the test suite, unit tests or doc tests | 14:48 |
| bigjools | well you could start by proposing some schema changes, we'll get those done, and then you can poke values in locally when doing the UI | 14:49 |
| bigjools | the scanning part is a separate task | 14:49 |
| directhex | bigjools, is the bug report the best place to discuss my thoughts? | 14:54 |
| bigjools | directhex: yes, absolutely | 14:59 |
| fran6co | hi | 15:01 |
| fran6co | i need some help setting up launchpad in my machine, anybody that want to give a hand with it? | 15:02 |
| fran6co | well, I managed to install it and make it work, but the thing that i don't get is the cronscripts | 15:03 |
| fran6co | i couldn't find info about how to i should run them | 15:04 |
| fran6co | thanks in advance | 15:04 |
| === matsubara is now known as matsubara-lunch | ||
| matsubara-lunch | deryck, burndown is up to date | 15:15 |
| deryck | matsubara-lunch, thanks! | 15:16 |
| deryck | mrevell, call time? | 15:16 |
| sinzui | jelmer: ping | 15:16 |
| jelmer | sinzui, hi | 15:16 |
| sinzui | bug 523844 does not look like a bug, it looks like a question, and I think it is one should should be able to answer yourself. | 15:17 |
| mup | Bug #523844: experimental Debian distroseries <Launchpad Registry:New> <https://launchpad.net/bugs/523844> | 15:17 |
| sinzui | jelmer: do you see the Add series link: https://edge.launchpad.net/debian | 15:17 |
| jelmer | sinzui: Ah, yep | 15:18 |
| sinzui | jelmer: I think you have a better understanding about creating a debian series than myself. | 15:18 |
| jelmer | sinzui: I hadn't realized I could do that sort of thing myself now. :-) | 15:19 |
| jelmer | sinzui: I'd like to double-check that experimental isn't missing by intention though, who should I talk to about that? | 15:19 |
| sinzui | jelmer: No idea. I can fix the code, but distroseries were designed for Ubuntu. Launchpad really ignores all other distros | 15:20 |
| * sinzui tries to edit lenny to see if experimental exists | 15:21 | |
| sinzui | Experimental does not appear to be supported | 15:22 |
| sinzui | but I think that is because series cannot go backwards | 15:22 |
| sinzui | jelmer: try creating the series on staging, the editing it to fix the status | 15:23 |
| sinzui | jelmer: you may need to hack the URL because only losas are really offered the link | 15:23 |
| mrevell | msg deryck I'll get Skype up and running | 15:23 |
| mrevell | heh | 15:24 |
| sinzui | jelmer: I just confirmed that you can hack the url for +edit ti set a active series to experimental | 15:24 |
| jelmer | sinzui: ah, thanks | 15:25 |
| jelmer | sinzui: I'll see if I can confirm that it's ok to create an experimental distroseries and then I'll give it a try. I'll close the bug. | 15:26 |
| sinzui | jelmer: the missing series status sounds familiar, I think this is a bug that also affects projects | 15:26 |
| jelmer | sinzui: In particular I'd like a distroseries named experimental, I'm not too worried about having its status set to Experimental | 15:27 |
| james_w | erm, https://edge.launchpad.net/debian/experimental | 15:39 |
| james_w | is that not what you are looking for | 15:39 |
| james_w | ? | 15:39 |
| === Ursinha-1fk is now known as Ursinha | ||
| === Ursinha is now known as Guest50427 | ||
| deryck | gmb, ping | 15:49 |
| gmb | deryck: Hi | 15:49 |
| === Ursinha_ is now known as Ursinha | ||
| === beuno is now known as beuno-lunch | ||
| === matsubara-lunch is now known as matsubara | ||
| === salgado-lunch is now known as salgado | ||
| === sinzui1 is now known as sinzui | ||
| deryck | rockstar, you around? | 16:03 |
| kfogel | deryck: terminology question: do we have a name for a not-yet-filled-in-by-separate-javascript-request portlet? Do we call the raw, unfilled portlet a "base" or a "template" or a "raw" portlet or anything like that? | 16:35 |
| deryck | kfogel, you mean the name of the template file itself? | 16:36 |
| kfogel | deryck: well, no, I have that. I mean a word for the unfilled portlet (except I don't know if "unfilled" is what we'd use). | 16:36 |
| kfogel | like if you were writing a test, and one phase of your test examined the unfilled portlet, and another phase pulled the real contents and examined those, what would be the words to describe each phase? | 16:37 |
| adiroiban | gmb: did you had time to put this branch for ec2 test https://code.edge.launchpad.net/~adiroiban/launchpad/bug-522188/+merge/19395 ? | 16:38 |
| deryck | kfogel, in just the test comment or doc portion? In plain English not code, is that what you mean? | 16:38 |
| kfogel | deryck: well, in a doc comment, but also in the names of two methods I'm abstracting out of xx-bugs-statistics-portlet.txt (I'm moving them to pages.py, because they're needed in multiple tests now). | 16:39 |
| kfogel | deryck: right now, they're called, respectively, print_portlet() and print_portlet_contents(). | 16:39 |
| === beuno-lunch is now known as beuno | ||
| deryck | kfogel, ah, ok. yeah, I'm not aware of any convention for that. Those names seem fine to me. Or print_empty_portlet for the first? | 16:39 |
| kfogel | deryck: the first of those names is unclear on its own; then if you see the second name it makes more sense, but I'd like to avoid that window of confusion. I'll probably go with "unfilled" and "filled" (as "empty" implies no content at all, which isn't the case). | 16:40 |
| kfogel | deryck: I just wanted to check that there wasn't already a term in use within Launchpad for these two states. | 16:40 |
| deryck | kfogel, not that I'm aware of, no. | 16:41 |
| kfogel | deryck: ok, thx | 16:41 |
| deryck | np | 16:42 |
| gmb | adiroiban: Strange, I thought that branch had landed; let me check | 16:46 |
| adiroiban | gmb: I have not received any email from buildbot. Same for this branch https://code.edge.launchpad.net/~adiroiban/launchpad/bug-340664/+merge/18452 | 16:50 |
| gmb | adiroiban: Hmm. I'll look into that. | 16:51 |
| adiroiban | I am not sure how buildbot is sending notification... as some of my branch did land without a notification from buildbot | 16:51 |
| gmb | adiroiban: How did only some of it land? That doesn't make sense. | 16:52 |
| adiroiban | I mean that for some of my branch that landed | 16:52 |
| adiroiban | i received an email from buildbot | 16:52 |
| adiroiban | others were landed without a notification from buildbot | 16:52 |
| kfogel | jtv: saw the characters displayed correctly in my email from your review response. yes, they're right :-) | 16:53 |
| jtv | kfogel: different chars this time. Got the full text from google translate. Is that last character the same one I hear in the polite version of "hello"? | 16:54 |
| kfogel | jtv: yup -- it means "you' | 16:55 |
| kfogel | "you" | 16:55 |
| jtv | In the polite form, right? I always thought it sounded more like níŋ | 16:55 |
| gmb | adiroiban: Ah, looks like your branches were submitted close to the same time but we were in testfix mode so they didn't land and I subsequently forgot to resubmit. Apologies; I'll do so now. | 16:56 |
| adiroiban | gmb: no problem. thanks | 16:57 |
| === deryck is now known as deryck[lunch] | ||
| === henninge is now known as henninge-afk | ||
| === gary_poster is now known as gary-lunch | ||
| === deryck[lunch] is now known as deryck | ||
| sinzui | adiroiban: See my answer to your question. I want to know if you see what I see | 18:37 |
| adiroiban | sinzui: http://bayimg.com/eaKPPAAcJ | 18:39 |
| adiroiban | this is whay I see | 18:39 |
| adiroiban | only „Reactivate ML” | 18:39 |
| sinzui | :( | 18:40 |
| sinzui | adiroiban: I will purge and report a bug | 18:40 |
| adiroiban | thanks! | 18:40 |
| kfogel | deryck: just fyi, jcastro and I are having this conversation over in the community channel (and are about to turn it into a phone call): http://paste.ubuntu.com/379256/ | 18:41 |
| sinzui | adiroiban: It is purged | 18:41 |
| kfogel | deryck: it's a dicey UI question w.r.t. the bugs-with-patches stats. your opinion welcome. | 18:41 |
| * deryck looks | 18:41 | |
| adiroiban | sinzui: yep. Many thanks! | 18:41 |
| kfogel | sinzui: (you might be interested in above two comments addressed to deryck too) | 18:43 |
| sinzui | adiroiban: this may not be time to celebrate: bug 471274 | 18:43 |
| mup | Bug #471274: No option to purge list archives when purging a mailing list <feature> <mailing-lists> <oem-services> <Launchpad Registry:Triaged> <https://launchpad.net/bugs/471274> | 18:43 |
| deryck | kfogel, simple fix -- just have the X bugs with patches be inclusive of closed bugs OR have +patches exclude closed by default and add filter by status. | 18:43 |
| kfogel | deryck: I think I like the former, actually. Since people can sort by state once they get to the +patches page anyway. | 18:44 |
| adiroiban | sinzui: that's ok. we didn't have top secret stuff there :) | 18:45 |
| kfogel | deryck: thanks. | 18:45 |
| deryck | kfogel, sure | 18:45 |
| mwhudson | jelmer: i'd like to upgrade dulwich & bzr-git so i can get my changes to do partial imports landed, will lp:dulwich and lp:bzr-git be ok? | 18:49 |
| mwhudson | i guess lp:bzr-git is required :-) | 18:49 |
| mwhudson | jelmer: thanks for merging my branch btw | 18:49 |
| sinzui | bac: ping | 18:54 |
| sinzui | EdwinGrubbs: ping | 18:55 |
| bac | hi | 19:00 |
| bac | sinzui | 19:01 |
| sinzui | bac: look at https://edge.launchpad.net/ubuntu/lucid/+source/alacarte The "Unspecified" is a db hit, and when you list a lot of projects on a page, it is a costly repeat | 19:03 |
| sinzui | bac: I have never liked this this ambiguous, possibly pathetic attempt to say the license is unknown. Do you want to argue for this feature's continued existence? | 19:04 |
| bac | sinzui: not i | 19:04 |
| jelmer | mwhudson, yeah, both trunks should be fine | 19:04 |
| bac | sinzui: but that lookup is no more costly than anything else, is it? getting rid of it b/c it is ugly is one thing but i can't see how it is hammering us more than any other attributte | 19:05 |
| kfogel | sinzui: advice -- I've abstracted out two functions that were formerly in a .txt test file. Now they live in pages.py -- see http://paste.ubuntu.com/379268/ . But, the code refers directly to "http://bugs.launchpad.dev/", because the functions are specific to the bugfilters portlet. I think that's okay, but I also notice this is the only .py code in that directory that hardcodes a service URL. Is this ok? | 19:06 |
| sinzui | bac: true, but if +packaging shows 20 of these packages, we are doing repeat queries for the license, plus the bug supervisor, tracker, branch, and translations. I care about the last four items, but the license is a formatter quirk, and the only person who can change the license if the project owner | 19:07 |
| sinzui | kfogel: I see. I think both helpers are kind of bogus because they are making assumptions about the markup or host. They break every year | 19:09 |
| sinzui | kfogel: as long as the bug team is willing to do the maintenance, there is no problem | 19:10 |
| kfogel | sinzui: well, I think we are (will check with the team). my question is really about the host aspect -- whether that's likely to break ever. But if you think it's no worse than any of the other assumptions in that test, then I'll leave it. | 19:12 |
| kfogel | s/test/test function/ | 19:12 |
| sinzui | The host is less brittle then the markup. I would question why the test helper is going into 1) a deprecated module, and 2) a module used by by non-bugs apps. I think the tests should be in lp.bugs.tests.bug | 19:15 |
| sinzui | kfogel: ^ | 19:16 |
| sinzui | kfogel: actually, lp.bugs.tests.bug is exactly where the helpers belong. doctests. stories, and unittests can import them as needed | 19:17 |
| kfogel | sinzui: thx, will move | 19:17 |
| * sinzui wonders why lp.bugs.pagetests exists | 19:17 | |
| * kfogel wonders how one is supposed to know it's deprecated | 19:17 | |
| sinzui | kfogel: it is canonical.launchpad. we build in lp.<app> | 19:18 |
| === gary-lunch is now known as gary_poster | ||
| sinzui | kfogel: everything in canonical.launchpad was left behind during the open source apocalypse | 19:19 |
| kfogel | sinzui: ahhh. Yes, I remember that, but as I wasn't developing at the time, I didn't have a reflexive understanding of it. got it | 19:20 |
| sinzui | oh dear, as I feared lp.pagetests do not work because that is not a legitimate directory | 19:21 |
| sinzui | deryck: ping | 19:27 |
| deryck | hi sinzui | 19:29 |
| sinzui | lp.bugs.pagetests should not exist. The tests in those directories are not played by the test runner. They should be moved to lp.bugs.stories | 19:30 |
| sinzui | I was about to report a bug | 19:30 |
| sinzui | deryck: just doing a move to the stories directory should make the command work | 19:32 |
| sinzui | ./bin/test -vvc -t cve-pages -t xx-bug-text-pages | 19:32 |
| deryck | sinzui, oh, wow. Didn't realize we had anything in pagetests still. | 19:33 |
| kfogel | deryck: since this came up as I was adding some test infrastructure, if there's a simple rearrangement/move to do, I can just do it on my branch here... | 19:34 |
| kfogel | sinzui: ^^ | 19:34 |
| deryck | sinzui, I'll move them this evening or tomorrow am. | 19:34 |
| deryck | ah | 19:34 |
| kfogel | deryck: lib/canonical/launchpad/testing/pages.py is the file I was adding to, when sinzui noticed | 19:34 |
| deryck | kfogel, ok, you could move the tests under lp.bugs.pagetests to lp.bugs.stories if you don't mind. | 19:35 |
| deryck | kfogel, see the command above sinzui gave. it should run after the move. | 19:35 |
| deryck | kfogel, but I don't mind doing this tomorrow AM first thing if you don't want to mess with it. | 19:35 |
| sinzui | deryck: I think this is an issue where someone found some tests in the old location, and tried to move them to the now location, but created a bad directory in the process. | 19:36 |
| kfogel | deryck: I don't -- nearly complete non-understanding of the totality of this issue, now that I look carefully at the commands and paths above, coupled with a desire to close out my WIP, makes me defer to you :-). | 19:36 |
| deryck | sinzui, ah, right. that would make sense. | 19:36 |
| deryck | kfogel, sure. it's trivial for me. So happy to do it. | 19:37 |
| sinzui | deryck: kfogel: The tests are pre-3.0. they might not pass | 19:37 |
| deryck | gah! | 19:37 |
| deryck | right | 19:37 |
| kfogel | ? | 19:37 |
| deryck | they haven't been running this long rm -rf anyone? :-) | 19:37 |
| * deryck ducks | 19:37 | |
| deryck | sinzui, kfogel -- I'll sort it out proper tomorrow. I have another test to fix, so no worries. | 19:38 |
| kfogel | deryck: thx | 19:38 |
| sinzui | deryck: you have my rs to delete any test that duplicates the authority of another test. | 19:39 |
| deryck | sinzui, will do. thanks. | 19:39 |
| mwhudson | hmm | 19:53 |
| mwhudson | jelmer: do you have any clever ideas about managing ~/.bazaar/svn-cache on the import slaves | 19:53 |
| mwhudson | jelmer: these directories are about 7 gigs on the slaves already | 19:54 |
| jelmer | mwhudson: do we need clever ideas? | 19:54 |
| jelmer | mwhudson: ah | 19:54 |
| jelmer | mwhudson: are those sizes really a problem? | 19:54 |
| mwhudson | and as we add more slaves, each slave is going to have to do the work to rebuild the cache | 19:54 |
| mwhudson | which seems pretty wasteful | 19:55 |
| mwhudson | jelmer: well, it depends how much more it grows, i guess | 19:55 |
| mwhudson | 10 gigs isn't really a problem, i guess | 19:55 |
| mwhudson | 100 gigs we'd need to know about | 19:55 |
| mwhudson | 1000 would be a definite problem | 19:55 |
| jelmer | mwhudson: it'll be a while before you'd hit 100 gigs I think | 19:55 |
| jelmer | we should have a better mechanism in place in bzr to store this sort of cache data before that I hope | 19:56 |
| jelmer | mwhudson, alternatively, we could see if we could turn off the cache for bzr-svn and see how much extra bandwidth that requires | 19:56 |
| jelmer | mwhudson: since lp only does pulls we might be able to get away with that | 19:56 |
| mwhudson | jelmer: hmm | 19:58 |
| mwhudson | jelmer: the "determining revisions to fetch" step takes a considerable amount of time for large repos | 19:59 |
| mwhudson | jelmer: isn't that the step that the cache helps with? | 19:59 |
| jelmer | mwhudson, I wonder why that is, it is much quicker locally | 19:59 |
| mwhudson | jelmer: f'ex https://code.edge.launchpad.net/~timmie/grass6/grass_trunk | 20:00 |
| mwhudson | looks like "determining revisions to fetch" too 7 hours here (!) | 20:00 |
| mwhudson | *took | 20:00 |
| jelmer | whoa | 20:00 |
| * jelmer tries locally | 20:01 | |
| jelmer | mwhudson, does a lot of terminal out cause the slave process to hang perhaps? | 20:01 |
| jelmer | terminal outPUT | 20:01 |
| mwhudson | and copying revisions took 5 | 20:01 |
| mwhudson | jelmer: i don't think so, it's only outputting one line a minute | 20:01 |
| mwhudson | it's possible the machine was slammed while this was happening | 20:02 |
| jelmer | mwhudson: I've seen it with more importants so I don't think this was a glitch | 20:02 |
| jelmer | *imports | 20:02 |
| jelmer | man, my brain is sleeping | 20:02 |
| jelmer | mwhudson, I'll see if I can reproduce it here locally doing a pull. If not, I wonder how we can profile the problems on the lp slaves.. | 20:03 |
| mwhudson | other things we can do are preserve the svn-cache between runs like we do the git.db | 20:06 |
| mwhudson | or maybe just gzip the caches, they seem to compress pretty well | 20:06 |
| === salgado is now known as salgado-afk | ||
| mwhudson | jelmer: the reason i'm asking is because i've been asked about how much disk space to give new code import slaves | 20:06 |
| jelmer | mwhudson: the svn-cache is not preserved between runs, the home directory of the slave is cleared? | 20:07 |
| mwhudson | jelmer: uh no | 20:07 |
| mwhudson | that's not what i meant | 20:08 |
| mwhudson | we currently preserve git.db files by copying them from the imported branch onto escudero, and fetching them before updating the import | 20:08 |
| mwhudson | we *could* copy the ~/.bazaar/svn-cache/$foo directory to and from escudero after and before an import in the same way | 20:09 |
| mwhudson | with some trickery to know which file to get, i guess | 20:09 |
| jelmer | I don't think that's worth the effort to be honest | 20:14 |
| jelmer | unless the "fetching revision info" step is taking too much time | 20:14 |
| mwhudson | oh right | 20:15 |
| mwhudson | that step seem to hardly be taking any time | 20:15 |
| mwhudson | hm, but maybe the cache already existed for that branch | 20:16 |
| mwhudson | seems like it took 10 minutes for that repo in fact | 20:17 |
| mwhudson | that's more than you'd want to spend on every import, for sure | 20:17 |
| mwhudson | but i guess once per slave isn't that bad | 20:17 |
| jelmer | mwhudson: it'll only happen once per slave per repo | 20:17 |
| jelmer | after that it should be seconds | 20:17 |
| mwhudson | jelmer: but it's not really about time, it's about disk space | 20:17 |
| mwhudson | right now anyway | 20:18 |
| mwhudson | if we preserve them somewhere else, we can only have the ones on the slave that are running at a given time | 20:18 |
| jelmer | if it's about disk space we can look at whether we actually need the cache | 20:18 |
| jelmer | mwhudson, there's definitely something wrong on launchpad | 20:19 |
| jelmer | "determing revisions to fetch" for the branch you just linked to took less than a minute here | 20:19 |
| mwhudson | !! | 20:19 |
| maxb | ooi, why isn't "determining revisions to fetch" the equivalent of a single "svn log" operation? | 20:20 |
| mwhudson | jelmer: is this a clue? | 20:20 |
| mwhudson | 2010-02-16 21:43:44 WARNING Upgrade to svn 1.5 or higher for faster retrieving of revision properties. | 20:20 |
| jelmer | maxb: if you're not using the cache, it is | 20:20 |
| maxb | Shouldn't 'svn log' be fast enough to make a cache pointless? | 20:20 |
| jelmer | maxb: there's more you can do on a svn repo than svn log | 20:20 |
| jelmer | maxb: more than bzr pull I mean | 20:20 |
| mwhudson | jelmer: the slaves seem to be on subversion 1.4.6dfsg1-2ubuntu1.1 | 20:21 |
| jelmer | mwhudson: Hmm, I don't get that here | 20:21 |
| jelmer | mwhudson: Ah, it might be useful to upgrade the version of subversion on the slaves | 20:22 |
| jelmer | mwhudson: but it also looks like we're hitting a very slow code path when we're using svn 1.4, so we should probably fix that anyway | 20:22 |
| mwhudson | jelmer: last time i talked about upgrading svn apart from as part of a distro upgrade i was told it was impossible | 20:24 |
| jelmer | mwhudson: I'll see if I can reproduce the issue in a dapper vm | 20:24 |
| mwhudson | jelmer: hardy should be fine... | 20:25 |
| maxb | mwhudson: interesting, was a reason given? | 20:25 |
| mwhudson | maxb: cscvs uses both python-subversion and pysvn | 20:25 |
| maxb | That sounds merely slightly complex, not impossible | 20:26 |
| mwhudson | and at least one of those is really hard to build | 20:26 |
| mwhudson | well | 20:26 |
| mwhudson | maybe we didn't ask you :-) | 20:26 |
| mwhudson | anyway, it might be better now | 20:26 |
| maxb | I've built pysvn on Slackware 10.0, I'm sure I can build it on hardy | 20:26 |
| maxb | :-) | 20:27 |
| mwhudson | backporting from say feisty to dapper might have been more awkward than karmic -> hardy | 20:27 |
| kfogel | sinzui: several days in a row now, my machine has gradually slowed down to a crawl, becoming more and more unusable; running lp tests seems to be the cause, though I'm not 100% sure yet. 'top' indicates that python2.5 consumes most of the CPU. Have you experienced anything similar? | 20:28 |
| sinzui | kfogel: the testrunner does take most of one cpu, but all is better when the test completes | 20:30 |
| kfogel | sinzui: the "all is better" part doesn't always seem to happen for me | 20:30 |
| sinzui | kfogel: I never run into swap, so I think memory may be important. I have 2 gigs @ 32 bit usually allocated to development | 20:31 |
| kfogel | sinzui: well, up to several seconds just to switch virtual desktops, maybe 1/2 second keystroke delay now. I'm rebooting. back in a bit | 20:34 |
| === EdwinGrubbs is now known as Edwin-lunch | ||
| === matsubara is now known as matsubara-afk | ||
| kfogel | deryck: in a searchTasks() query, is there a recommended way to get the search to include all statuses? If you do status=None, it reverts to default statuses, which don't include (e.g.) BugTaskStatus.FIXRELEASED. | 21:11 |
| kfogel | deryck: should I just do status=any(*UNRESOLVED_BUGTASK_STATUSES, *RESOLVED_BUGTASK_STATUS) or something? | 21:11 |
| sinzui | Is anyone with postgress guruness awake? | 21:12 |
| kfogel | deryck: (well, that syntax may be wrong but you get the idea) | 21:12 |
| kfogel | sinzui: sorry, no guruness, but if you ask I'll look and see if I know the answer off the top of my head. | 21:12 |
| deryck | kfogel, yes, the idea is right, but not the syntax. Pass in a single combined list. | 21:13 |
| kfogel | deryck: will do | 21:13 |
| deryck | I'm out. Later all. | 21:14 |
| sinzui | kfogel: we have bad data in production that I though we fixed in November. patch-2207-09-0.sql drops packaging_uniqueness and adds packaging__distroseries__sourcepackagename__key which is a more restrictive UNIQUE rule. But I do not see this patch applied when I look at the db | 21:16 |
| * sinzui thinks he needs to email stub and jtv to understand the screwup | 21:16 | |
| kfogel | sinzui: I think so -- this is probably more about lp db patching process than about postgresql per se | 21:16 |
| sinzui | I also need to find the query that identified the insane data too | 21:17 |
| mwhudson | sinzui: do you need to "drop index" not "drop constraint" ? | 21:36 |
| mwhudson | just a guess | 21:36 |
| sinzui | maybe | 21:36 |
| mwhudson | strange for it not to error though | 21:37 |
| sinzui | the new constraint was not added, so there is either a big failure or two small failures | 21:37 |
| === Edwin-lunch is now known as EdwinGrubbs | ||
| kfogel | sinzui: I tried to request a code review (non-ui this time) from you for https://code.edge.launchpad.net/~kfogel/launchpad/255868-patches-view-from-bugs-page/+merge/19438 -- it wouldn't let me, I guess because you had already approved it for ui. Can you do a code review? | 22:18 |
| sinzui | kfogel: I saw it and am reading right now | 22:19 |
| kfogel | sinzui: thanks (hunh -- you got the email, and yet the page doesn't show you as a pending reviewer) | 22:19 |
| sinzui | kfogel: r=me | 22:19 |
| kfogel | sinzui: thanks | 22:19 |
| bac | sinzui: Q: in the new portlet under "Needs linking" is that really "needs linking or more info", i.e. using getPrioritizedUnlinkedSourcePackages? | 22:24 |
| bac | sinzui: b/c some of those can be *linked* but still lacking signficant information | 22:25 |
| bac | if that's the case i need to change the heading | 22:25 |
| kfogel | sinzui: is it normal for EC2 to try to merge devel into a db-devel-based branch before testing? http://paste.ubuntu.com/379375/ | 22:31 |
| sinzui | bac: yes getPrioritizedUnlinkedSourcePackages | 22:31 |
| bac | sinzui: so "Needs linking or more information"? very wordy | 22:32 |
| sinzui | bac: the ones that are missing information need to be fixes by upstream contributors most of the time | 22:32 |
| kfogel | sinzui: 'utilities/ec2 test --headless -b launchpad=db-devel' is probably the answer | 22:33 |
| sinzui | bac: Maybe we needed to be clear that the packages need to be associated with an upstream project...I do not like the use of "link" | 22:33 |
| bac | kfogel: yes, if you're not using devel you need to specify db-devel. there is a wiki page devoted to how to work with db-devel | 22:34 |
| sinzui | kfogel: if you branched from db-devel, then yes you need to merge to that, but I have read your changes and they could be extracted and applied to a devel branch so that edge users can see it | 22:34 |
| kfogel | sinzui: my changes depend on stuff that's only in db-devel, though (the entire +patches view is not present in devel yet, IIRC) | 22:35 |
| kfogel | sinzui: so these particular changes would work, but the link they add would be a broken link | 22:35 |
| sinzui | fair enough, send it to staging | 22:36 |
| bac | sinzui: http://people.canonical.com/~bac/link-portlet.png | 22:41 |
| bac | that's a lot of use of "link" | 22:41 |
| sinzui | bac: yes, which is why I wonder is linked is right. We can let the users decide | 22:42 |
| bac | sinzui: so you want to go with linked now? | 22:42 |
| bac | or do you want to try to come up with a new term now? | 22:43 |
| jelmer | mwhudson, hmm, looks like subvertpy doesn't actually compile with svn 1.4 at the moment.. | 22:44 |
| sinzui | bac: if you can, that would be grand. I wont make it a condition for landing since I think we need users to help find the words that explains the connections between packages an project releases, and communities and kinds of contributions | 22:44 |
| mwhudson | jelmer: i guess we'd better be careful when upgrading that then! | 22:45 |
| jelmer | mwhudson: fixing it atm | 22:45 |
| bac | sinzui: i'm having a hard time coming up with an alternative right now. "associate"/"association" is the only thing i can think of and it would look silly in that many places i think | 22:46 |
| mwhudson | jelmer: ok | 22:46 |
| sinzui | bac: yes, it is hard | 22:46 |
| sinzui | bac: I now see that the bug I am working on is in fact a duplicate of the one I asked for help with on the list: look at this: https://edge.launchpad.net/gtk/+packages | 22:47 |
| bac | i think i'll do a first pass as is knowing that inertia is a terrible thing | 22:47 |
| bac | sinzui: so any clue why that db patch is not applied? | 22:50 |
| sinzui | no, and I wish I could find my dup finder query. I could clean up the dup data in an evening if I could get a list | 22:50 |
| mwhudson | oh | 23:04 |
| mwhudson | i know why the patch isn't applying | 23:04 |
| mwhudson | the filename is wrong | 23:05 |
| mwhudson | sinzui: ^ | 23:05 |
| sinzui | rock. I am incompetent | 23:05 |
| sinzui | mwhudson: I do not see the file name wrong, but maybe the revision is wrong? | 23:07 |
| sinzui | INSERT INTO LaunchpadDatabaseRevision VALUES (2207, 9, 0); | 23:07 |
| sinzui | 9 should be 09? | 23:07 |
| mwhudson | sinzui: no | 23:07 |
| mwhudson | ot | 23:07 |
| mwhudson | it's called .0.sql | 23:08 |
| mwhudson | all the others are -0.sql | 23:08 |
| * sinzui bangs head | 23:08 | |
| sinzui | thanks mwhudson I will update the bug. We cannot land a name fix until the data is cleaned up again | 23:09 |
| mwhudson | sinzui: np | 23:09 |
| mwhudson | i guess it's "unusual rollout requirement" time | 23:09 |
| sinzui | mwhudson: thanks a lot. there are only two bad condition in the data, I got lucky when I saw one in the list of 10,000 packagings | 23:20 |
| jelmer | mwhudson, 1.4 compatibility has been restored | 23:21 |
| mwhudson | jelmer: cool | 23:25 |
| sinzui | spm: ping | 23:32 |
| sinzui | hmm, losas are sprinting aren't they | 23:33 |
| mwhudson | not yet, but spm isn't around | 23:34 |
| mwhudson | mbarnett might still be? | 23:34 |
| * mwhudson lunches | 23:34 | |
| * thumper is back but grabbing lunch | 23:36 | |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
