| thumper | mwhudson: bzr builder is offline, so not running at all | 01:10 |
|---|---|---|
| mwhudson | thumper: not surprised | 01:11 |
| mwhudson | thumper: it's a bit borked at the moment anyway because of api versioning issues :/ | 01:11 |
| thumper | yeah | 01:11 |
| wgrant | What does PQM's closure mean? Only RC submissions are accepted? | 01:56 |
| mwhudson | wgrant: yes | 02:04 |
| mwhudson | we should start a sweepstake on how many of those there will be for this release | 02:04 |
| spm | mwhudson: entries < 15, will be ignored | 02:05 |
| mwhudson | spm: after a fit of laughter? | 02:05 |
| spm | mwhudson: I didn't feel it politic or diplomatic to go so far as to state *that*. But. Well. Yes. ;-) | 02:06 |
| * wgrant thinks that bug #244145 is lying. | 02:07 | |
| mup | Bug #244145: process-death-row script chokes on rebuild archives <derivation> <Soyuz:Triaged by al-maisan> <https://launchpad.net/bugs/244145> | 02:07 |
| wgrant | process-death-row doesn't actually fail because of that in production, does it? | 02:07 |
| ajmitch | mwhudson: are you suggesting that there are still a few bugs to be fixed? | 02:07 |
| mwhudson | ajmitch: there are always bugs to fix, i hope we're not going to try to run out before thursday :) | 02:08 |
| wgrant | ajmitch: Never. 3.0 is perfect already. Or something. | 02:09 |
| ajmitch | wgrant: great to hear | 02:09 |
| * mwhudson afk for lunch & errands | 02:19 | |
| === Ursinha-afk is now known as Ursinha | ||
| poolie | https://help.launchpad.net/API/launchpadlib claims you must be in the beta team to use apis | 05:03 |
| poolie | is that really still true? | 05:03 |
| wgrant | poolie: No. | 05:04 |
| wgrant | poolie: There was a discussion on the ML about it this morning. | 05:04 |
| poolie | so i'll just take it out..? | 05:08 |
| wgrant | poolie: If you are so privileged. | 05:09 |
| poolie | not really, i just have a sense of entitlement ;-) | 05:09 |
| wgrant | It's a question of whether Moin thinks you are appropriately privileged. | 05:10 |
| wgrant | Most are not. | 05:10 |
| poolie | wgrant: really, which ml? i can't see it in my mail | 05:13 |
| poolie | and i thought i at least received all the lp mail | 05:13 |
| poolie | though not the reviews | 05:14 |
| wgrant | poolie: A launchpad-dev email from 3:07 AEST today. | 05:14 |
| wgrant | "[Launchpad-dev] https://dev.launchpad.net/API "Authenticated Access Only" needs rationale" | 05:14 |
| poolie | oh, sorry | 05:15 |
| poolie | mismatched replies, i thought you were talking about the restriction to beta team members | 05:15 |
| wgrant | I was. My memory was just fault and conflated the two issues. | 05:16 |
| wgrant | ... +y | 05:16 |
| wgrant | So there was no discussion. Sorry. | 05:16 |
| wgrant | However, the production configs from July confirm that there was no restriction in place. | 05:17 |
| wgrant | And as it's now available on lpnet too, I doubt it's restricted there either. | 05:17 |
| poolie | surely you should have access to that wiki page? | 05:18 |
| wgrant | Ah, /API/launchpadlib isn't restricted, just /API. | 05:19 |
| poolie | ok, thanks | 05:22 |
| poolie | wgrant: so are you (wgrant) really not allowed to update this page? | 05:38 |
| wgrant | poolie: I cannot edit /API. | 05:39 |
| wgrant | #acl AdminGroup:read,write All:read | 05:40 |
| poolie | that seems wrong to me, probably unintentionally | 05:52 |
| poolie | but i guess we should ask karl or mrevell | 05:52 |
| * thumper wants cake | 06:01 | |
| thumper | spm: when is the edge rollout? | 06:03 |
| thumper | spm: and when is the staging rollout? | 06:03 |
| thumper | :( | 06:24 |
| thumper | ... | 06:24 |
| * thumper thinks | 06:25 | |
| * thumper takes it to the mailing list | 06:26 | |
| spm | thumper: edge, in about 1.5 hours, or on request :-) | 06:35 |
| spm | thumper: staging restores were borked over the w/e, but one is currently underway as we speak | 06:35 |
| thumper | spm: thanks | 06:42 |
| * mwhudson eods | 07:04 | |
| spm | night mwhudson | 07:06 |
| stub | spm: This staging restore will fail too since it will be using an older backup - its needs a backup made after my fix to the production data (made about an hour ago). | 07:09 |
| spm | bugger | 07:09 |
| spm | poolie: you've mentioned better LP is down posting type stuff before. is this kinda - as a generic framework/idea - what you had in mind? http://advisories.internode.on.net/item/6562/ | 07:11 |
| poolie | spm, well, that would be nice, but i was thinking of something cheaper to start with | 07:11 |
| poolie | spm, what I suggest we do today, if lp is down at the moment, is just tweet it | 07:13 |
| spm | as well as tag the IRC topics in the obvious places - which is what we currently do, perhaps not as perfectly as we'd like... | 07:14 |
| wgrant | The current IRC notifications are pretty good. I think the biggest problem ATM is getting *SAs to notice issues. | 07:16 |
| lifeless | and the first 3 blog posts are AWOL | 07:18 |
| wgrant | lifeless: They're not normally published until release. | 07:19 |
| wgrant | Although the fourth blog post's text is borked. | 07:19 |
| spm | wgrant: what do you mean: getting *sas to notice issues? that could be takena few ways, and it's unclear which you mean. | 07:20 |
| wgrant | spm: Often issues (particularly one or two hung appservers) will persist for a couple of hours before somebody sufficiently empowered notices. | 07:21 |
| spm | stub: seeing as the staging restore is going to fail; are there any issues with simply pulling a kill -9 ( >:) ) on pg_restore and friends? | 07:21 |
| spm | wgrant: it's been my experience that we'll know about a hung app server within 5-10 minutes (which is still too slow granted...); it's only a longer period when it happens outside core hours. ? | 07:23 |
| wgrant | spm: Yes. Weekend, mainly. | 07:24 |
| wgrant | +s | 07:24 |
| wgrant | SA TZ coverage is otherwise pretty good nowadays. | 07:24 |
| spm | heh. it'd want to be; barring illness/hols etc. we now have LOSA 24x5.mumble-somthing coverage | 07:25 |
| wgrant | Really? | 07:25 |
| spm | monday morning to sat morning - AEST time; yah. | 07:25 |
| wgrant | Hmm. | 07:26 |
| wgrant | I thought there was an AU->US hole. | 07:26 |
| spm | not as of last week | 07:26 |
| spm | which is why you may have notcied I start later, and finish later :-) | 07:26 |
| wgrant | Ah. | 07:27 |
| wgrant | But weekends can still go disastrously. | 07:27 |
| spm | Murphys law. they always will. | 07:27 |
| spm | the hung app server woe is being tackled on two fronts | 07:27 |
| wgrant | Of course, things will always go wrong. But they shouldn't stay wrong for soooo looong. | 07:28 |
| spm | 1. the relevant bug(s) are being fixed | 07:28 |
| spm | 2. better LB in front of same to take busted systems out of rotation. atm the "dumb" LB sees a responsive port 80; it doesn't know that the server behind same has gone to gaga land | 07:29 |
| wgrant | Yes. Hate hate hate Pound. | 07:29 |
| lifeless | 'squid' :P | 07:30 |
| lifeless | yahoo use squid and some other one for this | 07:30 |
| lifeless | depending on what group | 07:30 |
| * spm ignores the accurate interjection from the gallery... ;-) | 07:30 | |
| wgrant | It would be really nice if placeholder projects were editable by everybody or at least some team larger than ~admins. | 07:30 |
| spm | my 2c, vaguely disinterested observer opinion on that issue? It's changing to allow that, slowly. | 07:31 |
| spm | actually - not that vaguely disinterested, cuase it'd be one less taks thrown at us to do minor changes/edits for.... | 07:32 |
| stub | Anyone recall where requests for launchpad-developer-dependencies modifications get filed? | 07:35 |
| stub | spm: You can kill 'em. | 07:36 |
| spm | sweet, ta. | 07:36 |
| henninge | Morning! | 07:50 |
| noodles775 | mwhudson: I've got an approved branch here ready to land that updates two blueprint pages to 3.0 - is pqm still open? | 07:50 |
| noodles775 | mwhudson: I'd just submit it, but I remember someone mentioning once that we shouldn't even if it's open after the deadline? | 07:51 |
| noodles775 | Morning henninge | 07:51 |
| henninge | Hey noodles775! | 07:51 |
| mwhudson | noodles775: no | 07:51 |
| mwhudson | (it's not open) | 07:51 |
| mwhudson | afaik | 07:51 |
| noodles775 | ok, thanks mwhudson | 07:51 |
| henninge | noodles775: I could imagine that you'd get r-c quite easily for changes like that ... | 07:51 |
| henninge | or not? | 07:52 |
| spm | hey henninge, noodles775 | 07:52 |
| noodles775 | Hi spm :) | 07:53 |
| henninge | Hi spm! | 07:53 |
| henninge | Oh, still a lot of unconverted blueprint pages ... | 07:54 |
| noodles775 | henninge: yes, I'd assume so. BjornT ? Do you know whether rc's are being granted for blueprint conversions? | 07:54 |
| noodles775 | https://code.edge.launchpad.net/~michael.nelson/launchpad/sprint-index-and-attend-3.0/+merge/12044 | 07:54 |
| henninge | noodles775: what is that wiki page for blueprint work? | 07:55 |
| noodles775 | henninge: https://dev.launchpad.net/VersionThreeDotO/BlueprintsConversion | 07:55 |
| henninge | cheers | 07:55 |
| BjornT | noodles775: no, i don't know, but i'd also assume so. | 07:59 |
| henninge | Do I need to add "release-critical" to testfixes, too? | 08:56 |
| stub | henninge: Instead of having UTF-8 doctests, you might want to use u'somestring'.encode('doctest') which gives an ascii readable representation | 09:00 |
| stub | >>> print u'hello\N{TRADE MARK SIGN}'.encode('doctest') | 09:00 |
| stub | hello\N{TRADE MARK SIGN} | 09:00 |
| henninge | stub: thanks! | 09:00 |
| beuno | henninge, hi | 09:01 |
| beuno | re: bug 433824 | 09:01 |
| mup | Bug #433824: Top featured project selection on Launchpad home page <Launchpad Foundations:New> <https://launchpad.net/bugs/433824> | 09:01 |
| beuno | project of the day I think is great | 09:01 |
| henninge | beuno: Hi! | 09:02 |
| henninge | beuno: Cool, I will implement that today, then. | 09:02 |
| thumper | beuno: hi, you in London? | 09:03 |
| henninge | stub, jml: the test is passing for me locally ... | 09:05 |
| henninge | ? | 09:05 |
| henninge | without a fix, I mean | 09:05 |
| beuno | thumper, hi | 09:08 |
| beuno | yes | 09:08 |
| beuno | henninge, awesome | 09:08 |
| beuno | thanks for picking that up | 09:08 |
| henninge | stub, jml: mwhudson already fixed that, forgot to check first ... | 09:08 |
| henninge | beuno: I was actually wondering about it when doing the page but forgot ... | 09:09 |
| henninge | beuno: you are aware that the home page has landed? | 09:09 |
| beuno | henninge, I am! | 09:09 |
| beuno | I spy on progress daily | 09:09 |
| henninge | ;-) | 09:09 |
| beuno | I'm back to work today | 09:09 |
| henninge | cool | 09:09 |
| mrevell | Morning | 09:33 |
| mrevell | jml: blimey, didn't think you'd be online | 09:33 |
| wgrant | Why does the new project bugs page show the bugtask ID? | 10:57 |
| intellectronica | wgrant: stupid mistake on my part. fix already reviewed, i hope i'll get to merge it as release-critical, since it's pretty awful | 10:58 |
| intellectronica | https://bugs.edge.launchpad.net/malone/+bug/433857 | 10:58 |
| mup | Bug #433857: Summary in bugs homepage hot bug lists uses the wrong field <ui> <Launchpad Bugs:In Progress by intellectronica> <https://launchpad.net/bugs/433857> | 10:58 |
| deryck | Morning, all. | 10:59 |
| intellectronica | hi deryck | 10:59 |
| wgrant | intellectronica: Ah, great. | 10:59 |
| wgrant | Also, why are the columns in the bugtask index's table all the same width? Status/Importance are unnecessarily wide, and cause target names to wrap for me on eg. https://bugs.edge.launchpad.net/launchpad-foundations/+bug/401043 | 11:00 |
| mup | Bug #401043: speed up deployment of changelogs <Launchpad Foundations:Invalid> <Soyuz:Invalid> <Ubuntu:Confirmed for mvo> <https://launchpad.net/bugs/401043> | 11:00 |
| allenap | BjornT: You've been doing some windmill stuff recently. Do you know if this is related to your changes? http://paste.ubuntu.com/275155/ | 11:02 |
| wgrant | allenap: Bug 432665 | 11:03 |
| mup | Bug #432665: 'make iharness' is broken <Launchpad Foundations:New> <https://launchpad.net/bugs/432665> | 11:03 |
| allenap | wgrant: Top, thanks :) | 11:03 |
| intellectronica | wgrant: because if you let them be variable width the table changes when you edit a field inline and that's a bit weird | 11:07 |
| BjornT | allenap: well, i added the import. but it looks like 'make iharness' doesn't set up the right python path | 11:07 |
| allenap | BjornT: Ah, okay, probably a buildout thing then. | 11:09 |
| wgrant | intellectronica: Well, yes, but they're currently like twice the width they need to be for me. | 11:09 |
| intellectronica | wgrant: yeah, i guess you're right, we can keep the width fixed per column but give them different proportions | 11:10 |
| intellectronica | wgrant: care to file a bug? | 11:10 |
| wgrant | intellectronica: Doing so. | 11:11 |
| intellectronica | lovely, thanks | 11:11 |
| BjornT | allenap: yeah. for now, 'make harness' seems to work at least. | 11:15 |
| allenap | BjornT: Okay, thanks. | 11:16 |
| wgrant | Is there a good reason for the 'download' class on the attachment <li>s? It causes sprite issues, and everything seems OK if I drop it. | 11:19 |
| wgrant | cprov: Thanks for fixing the copy archive OOPS. I would have, but I was hoping there would be a more elegant way to do it (ie. some way of expressing conjunction in TALES). | 11:31 |
| cprov | wgrant: there isn't, we could create a view property but it isn't any clearer than using TALES | 11:32 |
| wgrant | cprov: There's another related OOPS that I can't reproduce locally, and didn't get a number for. Is it easy to look it up given a URL and time? | 11:36 |
| cprov | wgrant: I'm not sure, but I can try | 11:37 |
| wgrant | cprov: It was a POST by cjwatson to https://api.edge.launchpad.net/beta/ubuntu/+archive/test-rebuild-20090909 at around 0740UTC on the 19th. | 11:38 |
| wgrant | (it was a newComponentUploader API request to give us privileges to retry builds in the test rebuild) | 11:39 |
| bigjools | morning cprov | 11:39 |
| cprov | bigjools: morning | 11:40 |
| bigjools | just the man, I was going to ask you about that CP to fix the "email already in use" errors, since wgrant was seeing some related errors still | 11:40 |
| wgrant | bigjools: It wasn't I that brought it up, but yes, they are still around. | 11:41 |
| cprov | wgrant: was the fix for newComponentUploader() API in edge by that time ? | 11:41 |
| wgrant | I think it should have been rolled out a night or two before. But that should have been an Unauthorized, shouldn't it? Plus cjwatson is in ~techboard, so should have had permission anyway. | 11:42 |
| cprov | wgrant: yes, that's right | 11:44 |
| cprov | wgrant: I could not find any oops for the rebuild archive on 19th and 20th reports | 11:50 |
| wgrant | cprov: Hmm. http://pastebin.ubuntu.com/275170/ | 11:52 |
| cprov | wgrant: just a case of trying again, I guess ... make sure you pass 'main' component and catch launchpadlib.errors.HTTPError (print exc.content) | 11:52 |
| wgrant | cprov: I guess so. | 11:52 |
| cprov | wgrant: the other components ought to be the problem | 11:53 |
| wgrant | cprov: Really? I was able to add permissions for all components on a rebuild archive locally without trouble. | 11:53 |
| === bac changed the topic of #launchpad-dev to: This is Launchpad Development Channel | Week 4 of 3.0 | PQM is closed - Release manager is bac - contact for release-critical | https://dev.launchpad.net/ | Get the code: https://dev.launchpad.net/Getting | On-call review in #launchpad-reviews | Use http://paste.ubuntu.com/ for pastes | 3.0 goal is to convert all templates: http://people.canonical.com/~beuno/conversions.html | ||
| cprov | wgrant: nah, it's a COPY | 11:53 |
| wgrant | intellectronica: lp:~intellectronica/launchpad/hot-bugs-summary appears to contain a fix for the summary, but it still uses bugtask/id rather than bugtask/bug/id. Is that the branch you were talking about? | 12:11 |
| intellectronica | wgrant: oh, thanks for pointing that out. yes, that was the branch i was talking about | 12:14 |
| === henninge_ is now known as henninge | ||
| === henninge is now known as henninge-lunch | ||
| wgrant | cprov: Well, newComponentUploader works fine now. How odd. | 12:26 |
| wgrant | cprov: What's the purpose of the restriction of copy archive retries to the owner? Why shouldn't people mentioned in ArchivePermissions be able to? | 12:30 |
| wgrant | In fact, it seems like the PPA/COPY special case could just be eliminated without any change in behaviour except respecting ArchivePermissions. | 12:38 |
| === mrevell is now known as mrevell-lunch | ||
| cprov | wgrant: PPA and COPY archive permissions don't necessarily match the 'ubuntu' way of doing things | 12:52 |
| wgrant | cprov: But if there are explicit ArchivePermissions, it seems like they should be obeyed... | 12:52 |
| cprov | wgrant: yes, we could tie build-retry to upload rights | 12:53 |
| cprov | wgrant: but before exploring more APs in PPA and COPY we need an UI, otherwise it will be very confusing to use it. | 12:54 |
| wgrant | cprov: Perhaps, but by removing four lines from c.l.security, we immediately get much more functional rebuild archives. | 12:55 |
| cprov | wgrant: which 4 lines ? | 12:56 |
| wgrant | cprov: The is_ppa/is_copy special case in EditBuildRecord.checkAuthenticated. The PPA case is already properly handled by Archive.canUpload. | 12:57 |
| wgrant | I presume there's a reason it's special-cased, but I cannot see one. | 12:58 |
| cprov | wgrant: there is no *reason*, it was done like that just for safety, I guess | 12:59 |
| cprov | wgrant: restrictive is safer than permissive. | 12:59 |
| wgrant | cprov: Of course. | 12:59 |
| cprov | wgrant: and of course you are right, we can audit the code a bit and open up actions on rebuild archives to uploaders | 13:00 |
| === henninge-lunch is now known as henninge | ||
| danilo-afk | jtv: hey | 13:15 |
| === danilo-afk is now known as danilos | ||
| === mpt_ is now known as mpt | ||
| danilos | jtv: https://translations.edge.launchpad.net/ubuntu/jaunty/+source/findutils/ | 13:32 |
| bac | hi salgado | 13:37 |
| salgado | hi bac! | 13:38 |
| henninge | danilos, jtv: https://translations.edge.launchpad.net/ubuntu/jaunty/+source/findutils/+pots/findutils | 13:39 |
| bac | salgado: good morning. i wanted to confirm our new QA rules. just one tester and not the owner, right? | 13:39 |
| bac | salgado: we've got a lot of QA to do... | 13:40 |
| salgado | bac, I thought it was two testers and not the owner, but I really don't know where I got that from | 13:40 |
| bac | salgado: i thought we were trying to streamline. we should decide at our standup | 13:41 |
| salgado | bac, do we have custom rules for every team? or is this LP-wide? | 13:41 |
| danilos | henninge, jtv: https://bugs.edge.launchpad.net/rosetta/+bugs?field.tag=import-queue | 13:41 |
| salgado | matsubara, do we have LP-wide rules for QA? | 13:41 |
| bac | salgado: i thought curtis said we were outliers and the new rule was to bring us in line with everyone else | 13:41 |
| matsubara | salgado, not yet | 13:42 |
| matsubara | salgado, francis will draft up QA plan for LP | 13:42 |
| salgado | cool | 13:42 |
| salgado | bac, that rule sounds good to me, then. let's go with it | 13:45 |
| bac | salgado: yeah, let's talk about it in 45 minutes | 13:45 |
| wgrant | What do contributors need to do about QA? | 13:46 |
| bac | wgrant: can you edit the wiki? | 13:51 |
| bac | Ursinha: do you have access to the email from staging? i'm looking for a registration email for QA | 13:51 |
| wgrant | bac: It seems I can edit the test plans, yes. | 13:52 |
| bac | wgrant: how 'bout i ask during our meeting and get back to you. personally, if you want to do some QA i'd think it would be very welcome. | 13:52 |
| bac | matsubara: ping | 13:56 |
| danilos | salgado: hi | 13:59 |
| salgado | hi danilos | 14:00 |
| danilos | salgado: I was wondering about something for the breadcrumbs | 14:00 |
| danilos | salgado: I know it's not possible today, but would it be worth thinking about the use case of https://translations.edge.launchpad.net/ubuntu/jaunty/+source/findutils/ | 14:01 |
| danilos | salgado: i.e. the final breadcrumb (for ISourcePackage) can't be on rootsite="translations", but I'd like the breadcrumb to be there | 14:01 |
| salgado | danilos, shouldn't the breadcrumbs there be 'Ubuntu > 9.04 > findutils > findutils translations"? | 14:03 |
| matsubara | hi bac | 14:04 |
| bac | hi matsubara. 1) do you have access to the staging outbox? i'm looking for a registration email for QA. 2) have we talked about community contributors doing QA? wgrant was curious. | 14:05 |
| danilos | salgado: I don't want them to be that | 14:06 |
| danilos | salgado: basically, because it doesn't make sense for people to traverse into a sourcepackage to get translations | 14:06 |
| intellectronica | wgrant: what we usually do is we put our nick next to changes we want to test in the test plan (to make sure nobody is working on them in parallel) | 14:06 |
| intellectronica | wgrant: one thing we try not to do, is test ones own code | 14:07 |
| bac | matsubara: brb | 14:07 |
| matsubara | bac, 1) yes, I'll look that up for you 2) not yet. | 14:07 |
| danilos | salgado: in general, this might be a moot point since we'll be totally restructuring the traversal (to go through language first, and not even mention the source package), but I am wondering if this might be an actual good use case | 14:07 |
| danilos | salgado: so, once we change the basic translations traversal, we can make it be 'Ubuntu > 9.04 > findutils > Translations', but before than, we want to enable people to go back to Ubuntu 9.04 translations page because that's a more likely starting point | 14:08 |
| danilos | s/than/that/ | 14:08 |
| intellectronica | wgrant: other than that there isn't anything too sophisticated about the qa process. if at any time you feel like helping with some testing it will be great. i think it always helps when you got a fresh set of eyes when trying to discover defects | 14:08 |
| salgado | danilos, that should be doable if we turn breadcrumbs into multi adapters (like views) so that you can specify a custom breadcrumb adapter for source packages on the translations layer. it's something that would make sense and would be relatively easy to implement, I think | 14:09 |
| allenap | intellectronica: On that note, I just marked a product picker branch of yours as ??. The functionality works, but it causes the status, importance and assignee pickers to stop working because the bugtask is now at a different URL (at least, that's my guess). | 14:11 |
| intellectronica | allenap: aha! is there a bug? | 14:12 |
| allenap | intellectronica: I haven't filed one, but I shall look for or file one now. | 14:13 |
| intellectronica | allenap: and i think that's a BAD, not a ?? | 14:13 |
| allenap | intellectronica: I was on the fence. It works after all ;) | 14:13 |
| intellectronica | allenap: well, i'm split too, in the sense that it's definitely a bug, but maybe not one worth fixing for r-c | 14:14 |
| intellectronica | so if you file a bug i can at least have a look what it would mean to fix it and we can decide then | 14:14 |
| allenap | intellectronica: Agreed. | 14:14 |
| jml | hi | 14:20 |
| noodles775 | Welcome jml :) | 14:21 |
| jml | noodles775, thanks :) | 14:23 |
| === mrevell-lunch is now known as mrevell | ||
| mrevell | jml: welcome to the top | 14:27 |
| danilos | salgado: in general, I'd like us to hold out on that until we have something of a real use case; what I have right now is a case which I want to get rid of anyway (i.e. I'd need it to have a nicer work around for the bad traversal, but it'd still be just a workaround), but perhaps I should file a bug about it for the future? | 14:28 |
| jml | mrevell, heh heh | 14:29 |
| salgado | danilos, please do | 14:29 |
| jml | mrevell, thanks. | 14:29 |
| danilos | salgado: bug 433997 | 14:36 |
| mup | Bug #433997: Breadcrumbs should allow links to be on different vhosts <Launchpad Foundations:New> <https://launchpad.net/bugs/433997> | 14:36 |
| salgado | thanks danilos | 14:38 |
| jml | so, what's up in Launchpad land? | 14:41 |
| beuno | salgado, any news on killing icons from breadcrumbs? | 14:41 |
| salgado | beuno, done already. do you see them anywhere? | 14:42 |
| beuno | salgado, ah! not anymore | 14:43 |
| beuno | salgado, thanks | 14:43 |
| salgado | beuno, np | 14:48 |
| salgado | barry, so, the bug is in fmt:pagetitle | 14:48 |
| salgado | return SEPARATOR.join( | 14:49 |
| salgado | breadcrumb.text for breadcrumb | 14:49 |
| salgado | in reversed(hierarchy_view.items)) | 14:49 |
| * barry remembers that code | 14:49 | |
| salgado | the text of breadcrumbs may come from a view's .page_title, which in turn may be instances of zope.i18nmessageid.message.Message | 14:49 |
| barry | salgado: shouldn't .text be a property that guarantees a unicode? | 14:50 |
| salgado | barry, so, when we do the join() there, we end up joining the Message objects without expanding | 14:50 |
| salgado | barry, I guess so | 14:51 |
| salgado | that hasn't been a problem in the past because zpt knows how to deal with Message objects properly | 14:51 |
| salgado | another option would be to have fmt:pagetitle pass the Message objects to the template, as it's handled properly there | 14:52 |
| barry | salgado: the only other choice i think is to unicode(breadcrumb.text) in that .join() but it seems like it's better to let the IBreadcrumb do it (since there may be other things it needs to do to turn it into a usable unicode) | 14:53 |
| salgado | barry, I think you misunderstood me. the problem is this: https://bugs.edge.launchpad.net/launchpad-foundations/+bug/433991 | 14:53 |
| mup | Bug #433991: Reverse breadcrumb titles don't expand zope.i18nmessageid.message.Message strings <Launchpad Foundations:Triaged> <https://launchpad.net/bugs/433991> | 14:54 |
| salgado | most likely I wasn't clear. ;) | 14:54 |
| barry | salgado: right, by unicode(breadcrumb.text) i meant "do whatever you have to turn what breadcrumb.text gives you into a unicode" ;) sorry for the short hand. i'm just not sure that the .join() code can always do the right thing. so that's why i think the contract for IBreadcrumb.text should be "returns a unicode". or is that insane? | 14:55 |
| salgado | barry, it is unicode already; just not expanded: u'Ask a question about ${context}' | 14:56 |
| salgado | or, actually, .text may return a Message object | 14:56 |
| salgado | ok, scratch that. I see what you mean now | 14:57 |
| barry | salgado: yeah. does that make sense? | 14:57 |
| salgado | barry, I think it does, but that means we'd have to deal with Message objects at two different levels. whereas if we change fmt:pagetitle to not join the breadcrumbs (leaving that for the template), we won't have to bother as the template deals with it properly | 14:59 |
| jml | this is so exciting, see Americans in their native timezone | 14:59 |
| barry | salgado: i see the problem. | 15:00 |
| barry | salgado: i don't think i want to do the join in base-layout.pt. what if fmt:pagetitle called a simple template to do the join? | 15:01 |
| salgado | barry, wfm, but I'm kinda concerned that having IBreadcrumb.text sometimes return Message objects (that need special casing) might cause all sorts of headaches | 15:04 |
| salgado | it might be better to do what you suggested, having IBreadcrumb deal with Message objects and return unicode | 15:05 |
| barry | salgado: i agree with that. am i right that we only see the problem in the titles though because the in-page breadcrumbs get converted properly by their template? | 15:06 |
| salgado | but I'm not sure that'd work from a translations POV | 15:06 |
| salgado | barry, yes, that's correct | 15:06 |
| salgado | barry, do you know how these Message objects would get translated in IBreadcrumb? | 15:06 |
| * salgado knows absolutely nothing about translations | 15:07 | |
| barry | salgado: i used to know, but it's been a looongggg time :( | 15:07 |
| salgado | barry, gary_poster knows, I'm sure | 15:08 |
| barry | salgado: so i definitely think that IBreadcrumb.text returning Message objects is crazy | 15:08 |
| salgado | gary_poster, around? | 15:08 |
| gary_poster | barry, salgado, yes reading backlog | 15:08 |
| gary_poster | salgado: not sure what the question is. I think it would make sense for IBreadcrumb to do the translations. It has the request and the Message, which should be all it needs. It has its own substitution mechanics (not %), so you'll have to make sure you get that right if that's pertinent. | 15:10 |
| allenap | BjornT: I'm QAing the FD_CLOEXEC fix you did. Do you know why libuuid forks a process? I'm interested, but I'm also trying to figure out how to confirm it (or justify not bothering). | 15:10 |
| salgado | gary_poster, we'd have to use get_current_browser_request(), as the IBreadcrumb adapter is not a multi one. but I guess that's still what makes the most sense | 15:13 |
| gary_poster | salgado: oh ok. cool. | 15:13 |
| BjornT | allenap: it's the way it's compiled in ubuntu. i don't know the exact reasons. you can confirm it by running tests in the BugsWindmillLayer, and make sure that libuuid doesn't own port 9025 | 15:13 |
| barry | salgado: that doesn't bother me too much | 15:13 |
| allenap | BjornT: Okay, thanks, I'll do that. | 15:14 |
| dobey | so what's the best way to get the list of bugs that are "assigned to me" with launchpadlib? | 15:19 |
| jml | dobey, good question! | 15:26 |
| jml | dobey, I don't know, but I'd love to know the answer. | 15:26 |
| dobey | yeah, i don't see an obvious path to it, looking at api docs | 15:27 |
| jml | given that basically everyone on the bugs team is awake right now... | 15:27 |
| jml | *hint hint* | 15:27 |
| dobey | (even a non-trivial path) | 15:27 |
| jml | ... I would hope that you'd get an answer fairly quickly :) | 15:28 |
| barry | noodles775: i'm sorry i couldn't get to your follow up over the weekend. i'm going to look at it now though | 15:32 |
| noodles775 | barry: great thanks - intellectronica already approved it, but I was keen for you to take a look at the bug + fix that I did (as our earlier work-around had some problems). | 15:32 |
| barry | noodles775: cool. thanks intellectronica | 15:32 |
| beuno | bigjools, hi, have you any news on the lazr-js sprint | 15:34 |
| bigjools | beuno: nope :/ still stuck on location | 15:34 |
| bigjools | I will chase once more ... | 15:34 |
| dobey | jml: the answer i got the other day was that it is non-trivial :) | 15:36 |
| dobey | jml: i don't know that anyone has actually done it in practice with launchpadlib yet | 15:37 |
| barry | noodles775: did you fix for bug 433852 land, or is it in ec2/pqm? | 15:40 |
| mup | Bug #433852: Breadcrumbs display for IHasMajorHeading views <Launchpad Foundations:In Progress by michael.nelson> <https://launchpad.net/bugs/433852> | 15:40 |
| noodles775 | barry: no, I've included it in this branch for the sprint index and attend pages, so feel free to request changes :) | 15:41 |
| barry | noodles775: the posted diff looks fine. i was only going to quibble about a few minor style things :) | 15:42 |
| barry | noodles775: actually better than fine; thanks for fixing this! | 15:42 |
| noodles775 | np! | 15:42 |
| salgado | gary_poster, I can't seem to find much info about translating Message objects other than a translate method in talinterpreter.py. can you point me somewhere to learn how to do it? | 15:43 |
| gary_poster | salgado: yeah, as soon as I find t. ;-) one sec | 15:43 |
| barry | noodles775: two small suggestions: | 15:43 |
| barry | noodles775: move the import of removeSecurityProxy to the module globals | 15:43 |
| noodles775 | barry: ah ok, I left it there because that's where salgado had it - wasn't sure if there was a reason. | 15:44 |
| barry | noodles775: probably just convenience. it can't cause circular imports (which would be the only reason to do an import in a method) | 15:44 |
| barry | noodles775: in display_breadcrumbs, it might be nicer to split the return line up; something like: | 15:45 |
| salgado | barry, noodles775, we used to keep removeSecurityProxy imports local because we don't want its usage to spread | 15:45 |
| barry | has_major_heading = IMajorHeading.providedBy(self._context_view) | 15:45 |
| barry | return len(self.items) > 1 and not has_major_heading | 15:45 |
| noodles775 | barry: ok, done. | 15:46 |
| barry | noodles775: also, though less important, maybe s/_context_view/_naked_context_view/ | 15:46 |
| barry | "_naked" being a common convention for naming unwrapped thingies | 15:46 |
| barry | salgado: cool. | 15:47 |
| bigjools | barry: I prefer "undressed" :) | 15:49 |
| barry | bigjools: isn't there some cute and quaint british saying that would be appropriate (or maybe *in*appropriate) here? :) | 15:50 |
| bigjools | barry: would it end in -monger? :) | 15:51 |
| barry | bigjools: starkers? :) | 15:53 |
| bigjools | barry: something like that :) | 15:54 |
| gary_poster | salgado: I have not found great docs either. The pertinent package is zope.i18n. You want to use zope.i18n.translate. simple usage is translate(msg, context=request). You probably don't need to worry with the mapping, but it is used to replace $name stuff. | 15:54 |
| salgado | gary_poster, cool, will check that out. the mapping is exactly what I need, though: https://bugs.edge.launchpad.net/launchpad-foundations/+bug/433991 | 15:55 |
| mup | Bug #433991: Reverse breadcrumb titles don't expand zope.i18nmessageid.message.Message strings <Launchpad Foundations:Triaged> <https://launchpad.net/bugs/433991> | 15:55 |
| barry | noodles775: do you need any further approval for your branch? bac needs to know | 15:55 |
| noodles775 | barry, bac: nope - I'm just about to do the recommended style changes you had, but nothing else is changing. | 15:56 |
| gary_poster | salgado: oh ok. is it a known set of possible $names that you have to interpolate? | 15:56 |
| barry | noodles775: do you have a branch with all your changes pushed? i'd like to take a look. it probably makes sense too to land it thorugh ec2 just to make sure there aren't any unintended consequences | 15:57 |
| noodles775 | barry: yes, there's no way I'd land it without ec2test :), I'll just make your changes and then push them (it's all in the branch on the MP). | 15:57 |
| barry | noodles775: :) | 15:57 |
| dobey | what's the resolution in pixels that the new lp web ui stuff is being targetted for? | 15:57 |
| barry | noodles775: cool, thanks | 15:57 |
| salgado | gary_poster, not really, but my code will get a Message object that should have a mapping already, so I was hoping I wouldn't have to worry about it. maybe that's what you meant? | 15:58 |
| gary_poster | salgado: ah, no, it's not, but that is much better, good. (you can also pass an explicit mapping; I forgot that Messages can and should typically/ideally have their own mapping. | 15:59 |
| salgado | gary_poster, cool, thanks a lot for the help | 15:59 |
| gary_poster | np | 15:59 |
| deryck | dobey, jml -- use searchTasks and pass the assignee as launchpad.me. | 16:02 |
| dobey | deryck: searchtasks on what? | 16:03 |
| deryck | dobey, there is one on project, distribution, milestone, and a few other places, so I guess that is context dependent a bit. To find all of your bugs on lp, maybe that's not so easy with the API. | 16:04 |
| dobey | deryck: exactly. i can't do it on person, which is what it would be most useful for, to me | 16:06 |
| jml | isn't there a top-level bugs collection? | 16:07 |
| noodles775 | barry: All changes are in lp:~michael.nelson/launchpad/sprint-index-and-attend-3.0 (except I didn't move the removeSecurityProxy as suggested above). | 16:08 |
| dobey | jml: yes. if you want to loop through ever bug on lp, i guess it's useful | 16:10 |
| dobey | jml: but it doesn't seem to have a searchTasks() | 16:10 |
| barry | noodles775: +1; i'll grab the branch after the ui call | 16:10 |
| jml | dobey, it would seem a natural place for one. | 16:12 |
| === matsubara is now known as matsubara-lunch | ||
| deryck | jml, yes, there is a bug set, but it doesn't have searchTasks. it should, or at least a way to search for assigned bugs. dobey, would you open a bug against malone for this? | 16:14 |
| dobey | sure | 16:15 |
| deryck | dobey, thanks! | 16:16 |
| barry | bac: i've got bug 434072 | 16:21 |
| mup | Bug #434072: Convert specifications-index to UI 3.0 <story-ui-3> <Launchpad Registry:Triaged by barry> <https://launchpad.net/bugs/434072> | 16:21 |
| bac | barry: cool | 16:24 |
| jml | kfogel, hi. | 16:28 |
| beuno | bigjools, any reason why the creation info, etc, is on a portlet: https://edge.launchpad.net/ubuntu/+source/gnome-panel | 16:29 |
| beuno | instead of on the top right, like in everywhere else? | 16:30 |
| bigjools | beuno: it was considered to be package activity | 16:30 |
| beuno | bigjools, I'm not super convinced, it feels off | 16:31 |
| bigjools | the information is top left | 16:31 |
| bigjools | I'm pretty sure you sanctioned it :) | 16:31 |
| beuno | yeah, I'm wrong a few times a week | 16:31 |
| beuno | maybe that was one of them... | 16:31 |
| bigjools | I think it's fine | 16:32 |
| === ursula_ is now known as Ursinha | ||
| kfogel | jml: hey! Made it over ok? | 16:36 |
| jml | kfogel, I did :) | 16:37 |
| kfogel | jml: whew :-) | 16:38 |
| bac | henninge: hold off on landing until we get the devel/db-devel sorted out | 16:46 |
| henninge | bac: uh oh ... | 16:47 |
| bac | henninge: too fast! :) | 16:47 |
| henninge | bac: ah, got another failure ;) | 16:47 |
| bac | henninge: you have run this branch against ec2? | 16:47 |
| henninge | bac: all good | 16:47 |
| henninge | bac: no, already did | 16:47 |
| bac | henninge: ok. | 16:47 |
| bac | hi EdwinGrubbs | 16:48 |
| barry | bac: shouldn't "Foo Bar" in https://launchpad.dev/~name16 be an H1? | 16:49 |
| bac | intellectronica, EdwinGrubbs: i'm seeing an ajax red error box when i try to change a bug task 'affects'. is this a known issue? | 16:49 |
| barry | noodles775: afaict, your branch looks fine from ui perspective | 16:49 |
| === Ursinha is now known as Ursinha-lunch | ||
| EdwinGrubbs | bac: I don't know about that. I could research that later today if necessary. | 16:50 |
| noodles775 | barry: great. Any other issues from a non-ui-perspective or shall I request the RC? :) | 16:51 |
| intellectronica | bac: not really, more info please: what's in the red error box? | 16:51 |
| noodles775 | Thanks! | 16:51 |
| bac | intellectronica, EdwinGrubbs: http://people.canonical.com/~bac/ajaxerror.png | 16:51 |
| bac | intellectronica, EdwinGrubbs: the change actually happened but that error box was displayed | 16:52 |
| bac | the entry on the page did not refresh to show the change | 16:52 |
| intellectronica | bac: is that safari? | 16:53 |
| bac | intellectronica: indeed it is. i haven't tried it in FF yet. could you do that? | 16:53 |
| intellectronica | i have a feeling that this may be a compatability issue | 16:53 |
| intellectronica | it works fine in ff | 16:53 |
| barry | noodles775: none that i could see. rc-away! | 16:54 |
| noodles775 | great thanks! | 16:55 |
| intellectronica | bac: i can reproduce with chromium, so i guess it's yet another webkit compatibility problem | 16:56 |
| bac | intellectronica: ok. could you open a bug? | 16:56 |
| intellectronica | bac: yes | 16:56 |
| bac | thanks | 16:56 |
| intellectronica | bac: https://bugs.edge.launchpad.net/malone/+bug/434093 | 17:02 |
| mup | Bug #434093: Inline bugtask product editing doesn't work in WebKit -based browsers <Launchpad Bugs:New> <https://launchpad.net/bugs/434093> | 17:02 |
| intellectronica | bac: also, can i please get r-c for https://code.edge.launchpad.net/~intellectronica/launchpad/hot-bugs-summary/+merge/12150 ? | 17:03 |
| bac | bigjools, deryck, barry: we've got three unassigned template conversions for blueprints that are rollout blockers. can your team take another? https://dev.launchpad.net/CurrentRolloutBlockers | 17:03 |
| bigjools | bac: yes, I'll take one | 17:03 |
| bac | bigjools: sweet. please claim it and move to in progress | 17:03 |
| bigjools | which is easiest? :) | 17:04 |
| bac | intellectronica: done | 17:04 |
| intellectronica | bac: thanks | 17:05 |
| bac | gah, i wondered why i couldn't change status for MPs. didn't see the action had moved over there ---> | 17:05 |
| deryck | bac, allenap can take one of those. ^^ | 17:06 |
| intellectronica | yeah, i spent good 10m this morning hunting for the button :) | 17:06 |
| bac | intellectronica: please add that bug to CRB if you haven't | 17:06 |
| bac | deryck: great! | 17:06 |
| bac | barry: can you see if salgado or EdwinGrubbs can take one? | 17:06 |
| salgado | I sure can | 17:07 |
| barry | EdwinGrubbs: can you take one and then we can fight over the third? | 17:09 |
| EdwinGrubbs | barry: I'm CHR today, so I don't think I can get to it until later today. Which one is it? | 17:10 |
| barry | EdwinGrubbs: okay. i might be able to get to it | 17:12 |
| barry | first, lunch.. | 17:12 |
| bac | salgado: thanks! barry, i think they are all claimed now | 17:16 |
| salgado | bac, barry, do I need to update the wiki page or something? | 17:17 |
| bac | salgado, deryck, bigjools: we may have over-assigned the items on https://dev.launchpad.net/CurrentRolloutBlockers. can y'all coordinate among yourselves, claim the bugs, and update the wiki page? | 17:18 |
| bigjools | bac: I've not claimed anything yet | 17:18 |
| bigjools | I have now | 17:20 |
| bac | henninge, intellectronica, noodles775: you are free to land with RC on devel now. | 17:22 |
| henninge | bac: really? cool. | 17:22 |
| === matsubara-lunch is now known as matsubara | ||
| flacoste | EdwinGrubbs: bug 434058 has the last unowned blueprints templates | 17:30 |
| mup | Bug #434058: Convert specification-linkbug, -edit, and specificationtarget-assignments to UI 3.0 <story-ui-3> <Launchpad Blueprints:New> <https://launchpad.net/bugs/434058> | 17:30 |
| EdwinGrubbs | flacoste: ok, I'll take it. | 17:30 |
| === flacoste changed the topic of #launchpad-dev to: This is Launchpad Development Channel | Week 4 of 3.0 | PQM is closed - Release manager is bac - contact for release-critical | https://dev.launchpad.net/CurrentRolloutBlockers | https://dev.launchpad.net/ | Get the code: https://dev.launchpad.net/Getting | On-call review in #launchpad-reviews | Use http://paste.ubuntu.com/ for pastes | | ||
| === deryck is now known as deryck[lunch] | ||
| === Ursinha-lunch is now known as Ursinha | ||
| === salgado is now known as salgado-lunch | ||
| === mrevell is now known as mrevell-dinner | ||
| dobey | hrmm. i am not liking some of these layout changes on edge :-/ | 18:25 |
| jml | dobey, I'd love to hear about it | 18:32 |
| jml | but not on IRC right now, since my brain is a little fungus-y atm | 18:32 |
| dobey | heh | 18:33 |
| dobey | jml: 'mark as duplicate' on bugs, and 'change status' on merge proposals, are the 2 that are sticking me in the side right now :) | 18:34 |
| dobey | hmm, and i'm slightly unsure if my API request should be filed against malone or not | 18:36 |
| jml | dobey, I'd file it against malone | 18:40 |
| * jml is going | 18:40 | |
| dobey | well i guess i need to file 2 bugs... one for searchTasks on bugs, and one for searchTasks on person/team | 18:40 |
| barry | bac: specifications-index.pt conversion is mp'd for review. are there any other pages that need conversion? | 18:41 |
| bac | EdwinGrubbs: i see you've grabbed bug 434058. are you going to be able to work on it today? | 18:43 |
| mup | Bug #434058: Convert specification-linkbug, -edit, and specificationtarget-assignments to UI 3.0 <story-ui-3> <Launchpad Blueprints:New for edwin-grubbs> <https://launchpad.net/bugs/434058> | 18:43 |
| bac | EdwinGrubbs: if not, perhaps barry would like it | 18:43 |
| barry | bac: only after your rc for https://code.edge.launchpad.net/~barry/launchpad/434072-spec-index/+merge/12175 :) | 18:44 |
| EdwinGrubbs | bac, barry: it will be at least a couple of hours before I can start on it, so you're welcome to it. | 18:44 |
| bac | barry: blackmail? | 18:44 |
| barry | bac: bacmail | 18:45 |
| barry | bac: i'll take it | 18:45 |
| bac | barry: great. will you update the bug and CRB? | 18:45 |
| barry | bac: yep | 18:45 |
| bac | barry: you're so much more efficient now with your new 4 core machine | 18:45 |
| barry | bac: oh man, tell me about it! | 18:46 |
| === deryck[lunch] is now known as deryck | ||
| ScriptRipper | kiko: ping | 18:50 |
| === salgado-lunch is now known as salgado | ||
| bac | hi rockstar | 19:08 |
| rockstar | bac, hi. | 19:10 |
| bac | rockstar: hey i see you worked on two answers templates. do you know if anyone is doing question-index.pt | 19:10 |
| rockstar | bac, question-index should be done. | 19:11 |
| bac | rockstar: ok, i was going by http://people.canonical.com/~beuno/conversions.html | 19:11 |
| rockstar | bac, oh, wait, is that the applicationhome one? | 19:11 |
| bac | yeah | 19:11 |
| rockstar | bac, I wasn't sure if that needed to be ported, but I can give it a shot. | 19:12 |
| rockstar | bac, I also have one more code template that needs to move and be ported. | 19:13 |
| bac | rockstar: that'd be nice if you could. would you mind opening a bug and listing it on https://dev.launchpad.net/CurrentRolloutBlockers | 19:14 |
| bac | rockstar: ok. your code work comes first so let someone know if the other needs attention. | 19:14 |
| rockstar | bac, well, technically, code is responsible for answers, so they both need to be done. | 19:15 |
| bac | rockstar: really? cool. | 19:16 |
| gary_poster | wgrant: ping? | 19:17 |
| barry | bac ping | 19:22 |
| bac | rockstar: remember to land your next changes on devel, not db-devel | 19:23 |
| bac | hi barry | 19:23 |
| rockstar | bac, cool. | 19:23 |
| gary_poster | deryck: hi. I reviewed and landed a branch from wgrant this cycle that moved subscription security into the model. Returning to the branch, it looks like this is about bug subscriptions. May I add this to the bugs QA list for 3.0.0? (commit info: r9371 "[r=gary][ui=none] (wgrant) Move structural subscription security to the model, rather than the view.") | 19:24 |
| deryck | gary_poster, yup, that will be fine. | 19:24 |
| gary_poster | deryck: thank you! | 19:24 |
| deryck | gary_poster, np! | 19:25 |
| flacoste | gary_poster, barry: do you know if i can use pkg_resources.resource_string('my.package', 'templates/file.txt') ? | 19:29 |
| flacoste | where templates is not a python package | 19:29 |
| gary_poster | I do not now | 19:29 |
| gary_poster | know | 19:29 |
| bac | deryck, bigjools, barry, rockstar: how is QA going for your team? | 19:29 |
| bac | gary_poster: ^^ | 19:29 |
| bac | matsubara, Ursinha: any thoughts from you about QA? | 19:30 |
| gary_poster | bac: There's a lot for Stuart to do. I'm about to write him about it. That's about it. | 19:30 |
| barry | bac: i need to talk about https://blueprints.launchpad.dev/firefox/+assignments i don't think this is a mechanical change | 19:30 |
| barry | flacoste: i've never tried that | 19:31 |
| * bac looks | 19:31 | |
| deryck | bac, slower than I'd like. We made good progress earlier, but now we | 19:31 |
| deryck | bac, we're all working on fixes | 19:31 |
| bac | deryck: and QA begets more fixes... | 19:31 |
| rockstar | bac, we're getting through it, but with staging down, we're kinda blocked on many things. | 19:31 |
| barry | bac: i think we're probably not doing much qa right now. EdwinGrubbs is ocr, salgado and i are working on bugs, your the rm, and curtis is on a sprint | 19:32 |
| * barry is trying to remember the magic incantation for landing an rc | 19:32 | |
| matsubara | bac, I'll take care of https://dev.launchpad.net/LaunchpadTestPlan/3.0 and talk the contributors about the items there | 19:32 |
| bac | barry: look at henning's landing | 19:32 |
| bac | for the RC magic | 19:32 |
| Ursinha | thanks matsubara | 19:33 |
| bac | matsubara: is there a page with links to all of the team pages | 19:33 |
| bac | matsubara: and thanks for taking that on | 19:33 |
| matsubara | bac, https://dev.launchpad.net/QATeam/TestPlans | 19:33 |
| barry | bac: my thinking on bug 434058 is to split specificationstarget-assignment into a separate bug, get the mechanicals reviewed and landed, and then come back to +assignments | 19:35 |
| mup | Bug #434058: Convert specification-linkbug, -edit, and specificationtarget-assignments to UI 3.0 <story-ui-3> <Launchpad Blueprints:In Progress by barry> <https://launchpad.net/bugs/434058> | 19:35 |
| barry | bac: i have the other two templates already converted. they were mechanical and easy | 19:39 |
| bac | barry: cool. sorry i've been distracted on another chat | 19:39 |
| barry | bac: no worries. okay, i'll file a separate bug for -assignment | 19:40 |
| bac | barry: looking at +assignments now | 19:43 |
| bac | barry: your concern is the portlets? | 19:43 |
| bac | barry: splitting the bug is a fine idea | 19:44 |
| bac | barry: i just lumped them together crudely | 19:44 |
| barry | bac: yep, the portlets | 19:45 |
| bac | barry: let's talk about the design later. | 19:46 |
| barry | bac: okay. let me get this one into review and then you can ping me when you're ready to chat | 19:46 |
| * rockstar finds fud | 19:47 | |
| dobey | rockstar: now all you need is ge | 19:51 |
| bigjools | bac: it's fine | 19:56 |
| bac | bigjools: thanks | 19:59 |
| bac | bigjools: i see you've only got 14 in NEEDSTESTING. that's great. | 19:59 |
| bac | bigjools: would you mind if i assign https://bugs.edge.launchpad.net/blueprint/+bug/434195 to noodles775 | 20:00 |
| mup | Bug #434195: Convert specificationstarget-assignements to UI 3.0 <story-ui-3> <Launchpad Blueprints:Triaged> <https://launchpad.net/bugs/434195> | 20:00 |
| bac | danilos: how is your team's QA going? | 20:01 |
| bigjools | bac: can you ask him tomorrow when he's working, but it's probably ok | 20:01 |
| bigjools | bac: yeah we have some items in 2.8 as well :) | 20:02 |
| danilos | bac: it's fine though TestPlans page do not reflect it very well; also, fyi, I've just added two items to current rollout blockers (we are not yet sure they are blockers, but I'll have to spend some time investigating tomorrow to decide if they are but would rather have them on the page) | 20:02 |
| bac | bigjools: i can. i thought you might have an understanding of his workload. i'll assign it to him and he can dump it if too busy | 20:02 |
| bac | danilos: thanks | 20:02 |
| bigjools | bac: well his workload is full-ish, that's why I said to ask to see if he wants to take it on | 20:02 |
| bac | bigjools: i don't want it to idle between euro day start and my day start. | 20:03 |
| bac | bigjools: i'll email him and ask him to try to find another owner if he cannot do it. otherwise i'll deal with it tomorrow a.m. | 20:04 |
| bigjools | ok | 20:04 |
| bigjools | I am 50% through the ones I took earlier, BTW | 20:05 |
| bigjools | talking of which, barry, halp | 20:05 |
| flacoste | bac, bigjools: danilos was willing to do some blueprints conversion | 20:06 |
| danilos | flacoste: I'll have to see about that tomorrow morning though (with these two bugs now, they might take some time off); simple templates are fine though | 20:06 |
| danilos | anyway, /me wanders off | 20:06 |
| === danilos is now known as danilo-afk | ||
| barry | bigjools: wassup?! | 20:06 |
| barry | bac: we remove stuff from BlueprintsConversion once we land the code? | 20:07 |
| bigjools | barry: you da headings man - I gotz me some heading probz. I have a label on my view and it appears above the breadcrumbs and also above a form on the page, how do you fix that? | 20:07 |
| barry | bac: and CurrentRolloutBlockers? | 20:07 |
| barry | bigjools: you've check that there's no heading slot filled on your template? | 20:08 |
| barry | bigjools: and no extraneous h1/h2 before the <form>? | 20:08 |
| bigjools | barry: there is not | 20:08 |
| bigjools | there is not | 20:08 |
| barry | dang | 20:08 |
| barry | bigjools: you should not be getting two headers <wink> | 20:09 |
| barry | bigjools: can you push the branch and give me a lp.dev url to look at? | 20:09 |
| bigjools | ok one sec | 20:09 |
| bigjools | lol, bzr diff > utilities/paste doesn't work does it :) | 20:11 |
| bigjools | barry: http://pastebin.ubuntu.com/275460/ | 20:11 |
| bigjools | apply that | 20:11 |
| bigjools | and visit https://blueprints.launchpad.dev/ubuntu/+spec/media-integrity-check/+requestfeedback | 20:11 |
| barry | bigjools: no, it's bzr diff | utilities/paste :) | 20:13 |
| barry | bigjools: k, sec. i haven't finish something up | 20:13 |
| bigjools | barry: the view inherts from zope's AddView, I wonder if that would bugger it | 20:13 |
| barry | bigjools: yikes! switch that to LFV or LV | 20:14 |
| bigjools | :) | 20:14 |
| bigjools | blueprints suck | 20:14 |
| barry | bigjools: can you just bzr rm them all? | 20:14 |
| bigjools | barry: LFV doesn't make any difference and cocks up my form labels | 20:17 |
| barry | crap | 20:17 |
| barry | bigjools: okay, i'll look once this mp is in | 20:17 |
| bac | barry: i didn't even know about the BlueprintsConversion page | 20:20 |
| bac | barry: keep them on CRB but move to the appropropriate section | 20:20 |
| barry | bac: neither did i until flacoste pinged me about it | 20:20 |
| barry | bac: done | 20:20 |
| bigjools | he did email about it :) | 20:20 |
| barry | ;) | 20:21 |
| barry | bigjools: okay, mp sent. let me check out your branch | 20:21 |
| bigjools | coolio thanks | 20:21 |
| flacoste | bigjools: AddForm, aaargh, i thought salgado had got rid of all of those | 20:24 |
| barry | flacoste: could that be messing up bigjools pages? | 20:25 |
| salgado | flacoste, only SQLObject*Form | 20:25 |
| barry | bigjools: okay, i see the double "Request feedback on specification" | 20:26 |
| flacoste | barry: probably, since i'm pretty sure your infrastructure doesn't cover addform | 20:26 |
| barry | flacoste: i'm nearly certain it doesn | 20:26 |
| barry | er, doesn't | 20:27 |
| bigjools | so is there an equivalent that I can use on this page? | 20:29 |
| bigjools | that will work with your heading infra? | 20:29 |
| barry | bigjools: it's gotta be LaunchpadFormView | 20:30 |
| bigjools | barry: ok | 20:31 |
| barry | bigjools: but yeah, i see that doesn't help a whole lot :/ | 20:32 |
| bigjools | barry: s/addform/form/ in the template? | 20:32 |
| barry | bigjools: take a look at the configure.zcml entry for that page. i'm think simplify and convert it to a browser:page will help you there | 20:34 |
| bigjools | barry: arghhhhhh | 20:35 |
| bigjools | barry: I see the label in the zcml now! | 20:37 |
| barry | bigjools: yeah. kill all this old shit! | 20:39 |
| barry | bac: can i get an rc for https://code.edge.launchpad.net/~barry/launchpad/434058-specs/+merge/12183 | 20:40 |
| bigjools | barry: I wish I knew what I were doing to be able to do that | 20:44 |
| === oink is now known as kiko | ||
| barry | bac: okay, i am going to start on qa now | 21:07 |
| bac | barry: \o/ | 21:21 |
| salgado | barry, will you start with the ones at the top? | 21:22 |
| barry | bac, salgado: at the top of this: https://dev.launchpad.net/RegistryTeam/RegistryTestPlans/3.0 ? | 21:27 |
| salgado | barry, yep, just wondering if I should start from the bottom so that we don't duplicate work/conflict | 21:27 |
| barry | salgado: good point. we also have 2.2.8 stuff to qa don't we? | 21:28 |
| barry | salgado: maybe one of us should take 2.2.8 and one should take 3.0? | 21:28 |
| salgado | good point | 21:28 |
| barry | salgado: you choose. i'll take the other | 21:28 |
| salgado | barry, I can take 3.0 as I've already done a few there | 21:29 |
| barry | salgado: +1. i'll start on 2.2.8 | 21:29 |
| rockstar | sinzui, what was your mechanical process for moving three column layouts to 3.0? | 21:34 |
| sinzui | rockstar: I do not understand. I do not recall what a 3 column layout looks like | 21:35 |
| rockstar | sinzui, answers.launchpad.net | 21:36 |
| bac | salgado, barry: i've just been picking them randomly from the middle of the wiki page | 21:36 |
| sinzui | rockstar: ahh, that is freeform | 21:36 |
| sinzui | rockstar: I think you are at liberty to invent. I would copy code or bugs. | 21:36 |
| bac | salgado, barry: i meant on the 3.0 page | 21:36 |
| rockstar | sinzui, roger. | 21:36 |
| sinzui | code is way to exciting for answers | 21:37 |
| bac | sinzui: i like your mugshot on LP. is that new? | 21:41 |
| sinzui | bac: The new ui exposed my old logo from two years ago | 21:42 |
| bac | ah | 21:42 |
| sinzui | I was thinking of updating it using my daughter's tablet to draw | 21:42 |
| bac | that'll surprise some people | 21:42 |
| bac | look, it's me back when i had hair | 21:42 |
| barry | salgado: do you have a qa plan for bug 290681? i didn't see a branch or mp cover letter for it | 21:44 |
| mup | Bug #290681: Superteams are not properly carried over when merging teams <Launchpad Registry:Fix Committed by salgado> <https://launchpad.net/bugs/290681> | 21:44 |
| salgado | barry, that's pretty tricky to reproduce, but if the fix wasn't good, we'd have gotten some warnings from the monitoring scripts about inconsistent memberships/participations. I'd say just mark that as ?? | 21:47 |
| barry | salgado: good point, thanks | 21:47 |
| bac | salgado: ping me when you aren't editing the 3.0 test plan | 21:48 |
| salgado | bac, just finished | 21:49 |
| bac | sinzui: a lot of the bugs you fixed don't have related branches, which makes finding the MP with QA plan hard. | 21:50 |
| sinzui | bac: yeah, I realised I fixed them after the branch landed. | 21:50 |
| bac | sinzui: ah, i see. | 21:50 |
| sinzui | bac: That is not just me. I have been adding everyone's name to a trvial or UI bug that was closed by a branch they landed | 21:51 |
| bac | ah... | 21:51 |
| bac | i wonder if there is a good way to find the MP based on the rev number... i doubt it | 21:51 |
| sinzui | hmm | 21:52 |
| sinzui | The bugs are almost always related to the landing of the branch that updated the page. | 21:53 |
| sinzui | portlets are not, and salgado and I landed a few branches that fixed portlets | 21:53 |
| sinzui | bac: I worry more about unwanted timeouts caused by trivial UI changes. | 21:54 |
| bac | sinzui: maybe it is best to work from the MP backwards... | 21:54 |
| sinzui | bac: I had to remove bug tags from the milestone page a few days after landing that. QA was fine, the oops report was not | 21:54 |
| bac | sinzui: right. those will be hard to casually find | 21:55 |
| sinzui | bac: salgado: I worry about the number of queries on the project page. We show all milestones, which is not just expensive, they are not important. I think we should only show the last 5-7, but am not sure a simple limit in the template will improve the number of queries. | 21:56 |
| * sinzui shrugs | 21:57 | |
| flacoste | barry: how do i use with in 2.5? | 22:01 |
| barry | flacoste: you need an object that supports the "context manager" protocol, but let's say you want to use an open file object in a with statement (files support the protocol) | 22:02 |
| barry | with open(foo, 'w') as fp: | 22:02 |
| flacoste | ah, ok, open, not file( | 22:02 |
| barry | print >> foo, 'wussup' | 22:03 |
| barry | flacoste: right! file ctor is deprecated :) | 22:03 |
| barry | er, that should have been: print >> fp, 'wussup' | 22:03 |
| flacoste | testplan.py:253: Warning: 'with' will become a reserved keyword in Python 2.6 | 22:03 |
| flacoste | barry: why do i get the above warning^^^ | 22:03 |
| flacoste | barry: do I need a from __future__ ? | 22:04 |
| barry | flacoste: in py2.5, yes | 22:04 |
| barry | from __future__ import with_statement | 22:04 |
| flacoste | ah, ok | 22:04 |
| flacoste | thanks | 22:04 |
| barry | in 2.6, no :) | 22:04 |
| barry | np! | 22:04 |
| mwhudson | good morning | 22:08 |
| bac | hi mwhudson | 22:08 |
| bac | salgado: do you have a team that will show the issue for https://bugs.edge.launchpad.net/launchpad-registry/+bug/423080 | 22:08 |
| mup | Bug #423080: Team "Maintained Packages" link has incorrect capitalisation and location <trivial> <Launchpad Registry:Fix Committed by salgado> <https://launchpad.net/bugs/423080> | 22:08 |
| salgado | bac, not really, just looked at a dozen teams that I'd expect to have that but none of them do | 22:11 |
| bac | yeah, and wgrant's screenshot doesn't give context | 22:12 |
| * bac wonders if we can conjure wgrant by mentioning wgrant several times. | 22:12 | |
| bac | salgado: so does that mean we picked bad teams or the link has been killed? | 22:13 |
| salgado | bac, I think we're just picking bad teams -- the link is conditional | 22:14 |
| salgado | bac, it is the same you see on a person's page, though | 22:14 |
| salgado | https://edge.launchpad.net/~salgado | 22:15 |
| rockstar | thumper, we should do that standup thing that we do sometimes. | 22:16 |
| bac | salgado: yeah, that's perfect then | 22:17 |
| EdwinGrubbs | salgado: ping | 22:20 |
| salgado | hi EdwinGrubbs | 22:20 |
| * bac relocates. bbiab. | 22:20 | |
| EdwinGrubbs | salgado: is it possible to remove a sourcepackage from a distro? https://answers.edge.launchpad.net/launchpad/+question/83436 | 22:20 |
| salgado | EdwinGrubbs, I don't think so. cprov should be able to tell for sure | 22:21 |
| EdwinGrubbs | cprov: can I assign you that question and another one on ppas? | 22:22 |
| === salgado is now known as salgado-afk | ||
| === mrevell-dinner is now known as mrevell | ||
| bigjools | EdwinGrubbs: I will answer it | 22:32 |
| EdwinGrubbs | bigjools: thanks | 22:33 |
| bigjools | EdwinGrubbs: actually - he's asking about upstream links, not the package itself | 22:33 |
| EdwinGrubbs | bigjools: ok, who can answer that? | 22:34 |
| bigjools | EdwinGrubbs: it's a registry question - and I don't know how the link can be removed | 22:34 |
| bigjools | there's an upstream link but the package doesn't exist in Debian | 22:35 |
| EdwinGrubbs | bigjools: can you look at this ppa question? https://answers.edge.launchpad.net/launchpad/+question/83441 | 22:35 |
| bigjools | Debian experimental that is | 22:35 |
| bigjools | EdwinGrubbs: yes | 22:36 |
| awilkins | Hello, is it intentional that LP only runs on 64-bit machines (I'm referring to the 64-bit libraries that have been checked into the source tree) | 22:44 |
| rockstar | awilkins, example? | 22:44 |
| awilkins | _lsprof.so gives you an ImportError on 32-bit machines with an ELFCLASS64 | 22:44 |
| rockstar | awilkins, make clean and then make build again. | 22:45 |
| awilkins | rockstar: I did that. The problem with that library was resolved by removing the checked-out one and linking the python 2.5 (32-bit) version of it bu there are more libraries like it | 22:45 |
| rockstar | awilkins, file a bug. It used to run fine on 32-bit systems. | 22:46 |
| awilkins | rockstar: I think it's because they are not in the python 2.4 distribution | 22:46 |
| awilkins | rockstar: Just in the process :-) | 22:46 |
| maxb | Hrm, before you file a bug, let's make sure it's not just your system that's broken. | 22:46 |
| maxb | I certainly didn't have these problems running LP on i386 | 22:46 |
| awilkins | The file has been there for some time, according to it's changelog | 22:48 |
| awilkins | Introduced into devel at 6535 | 22:48 |
| maxb | Which file exactly are you talking about? | 22:48 |
| awilkins | lib/_lsprof.so | 22:48 |
| === matsubara is now known as matsubara-afk | ||
| maxb | awilkins: I think you'll find that's a symlink :-) | 22:48 |
| awilkins | Aha. | 22:49 |
| * awilkins does a make clean on lsprof | 22:50 | |
| bigjools | barry: still around? | 22:51 |
| awilkins | maxb: This is probably because I did a recursive copy from another drive... | 22:51 |
| maxb | mystery solved. | 22:53 |
| barry | bigjools: yep | 22:53 |
| maxb | bigjools: Are you handling Q 83441 or shall I answer it? | 22:54 |
| bigjools | barry: I am in the middle of converting that page to use LFV, but one of the fields that I specify in field_names doesn't show the input box, just the label. Any ideas? | 22:54 |
| awilkins | maxb: Needs a "make uberclean" :-) | 22:55 |
| bigjools | maxb: go for it, I am in form hell... | 22:55 |
| barry | bigjools: weirdness. can't think of anything... :( | 22:55 |
| barry | bigjools: but if you want another pair of eyes, let me know | 22:56 |
| bigjools | barry: yes please! | 22:56 |
| bigjools | http://pastebin.ubuntu.com/275523/ | 22:56 |
| * barry looks | 22:57 | |
| bigjools | barry: the widget was not specified in the original form zcml, I wonder if LPV needs one? | 22:57 |
| barry | bigjools: which field? | 22:57 |
| bigjools | barry: "reviewer" | 22:58 |
| bigjools | barry: I see the label, the description and its title but no input box... it's bizarre | 22:59 |
| barry | bigjools: reviewer is in ISpecificationFeedback? | 22:59 |
| bigjools | yes | 22:59 |
| barry | how weird! | 22:59 |
| bigjools | flacoste: any ideas? | 22:59 |
| barry | bigjools: will that diff apply to devel? | 22:59 |
| bigjools | barry: I bloody hope so | 22:59 |
| barry | bigjools: :) i mean, it's the diff of your entire branch? | 23:00 |
| bigjools | barry: no, but the only other commit is for a different page | 23:00 |
| flacoste | bigjools: i don't think you need the HiddenUserWidget anymore | 23:01 |
| bigjools | flacoste: yeah you're probably right :) | 23:01 |
| flacoste | bigjools: you can replace that with self.user | 23:01 |
| bigjools | I am blindly copying the original form for the moment | 23:01 |
| barry | bigjools: still rf-getting | 23:02 |
| flacoste | bigjools: i think it's because the field is readonly in the interface | 23:02 |
| bigjools | flacoste: ah | 23:02 |
| flacoste | bigjools: i think it's fine for the model, but the UI it's sure not | 23:03 |
| bigjools | flacoste: is it safe to make readonly=False? | 23:03 |
| flacoste | bigjools: here, yes | 23:03 |
| barry | flacoste: you're talking about the widget bigjools doesn't see? | 23:03 |
| flacoste | bigjools: when we export it over API, it won't | 23:04 |
| flacoste | bigjools: but this whole feature should be removed anyway | 23:04 |
| bigjools | flacoste: yeah, we can worry about that later | 23:04 |
| flacoste | bigjools: or re-implemented properly | 23:04 |
| * bigjools is just doing mechanical changes ... :) | 23:04 | |
| barry | bigjools: demo url? | 23:05 |
| bigjools | barry: no matter, flacoste has figured it out! | 23:06 |
| bigjools | barry: thanks anyway :) | 23:07 |
| barry | bigjools: coolio | 23:07 |
| bigjools | I love these 10 minute jobs that turn into an all-night hacking session! | 23:08 |
| maxb | "love", as in "hate" :-) | 23:09 |
| thumper | beuno: ping | 23:09 |
| * bigjools looks for matchsticks to prop eyelids open | 23:10 | |
| thumper | flacoste: do you have a few minutes to talk about your email reply to my authorisation in the model code? | 23:12 |
| flacoste | thumper: a little later, yes, i was going afk for dinner, be back later to speak to the other side | 23:13 |
| EdwinGrubbs | barry, bac: did you need any help converting templates? | 23:13 |
| thumper | flacoste: eta? | 23:13 |
| bac | EdwinGrubbs: all are assigned right now | 23:13 |
| bac | EdwinGrubbs: we need major QA-ness | 23:13 |
| EdwinGrubbs | ok | 23:13 |
| flacoste | thumper: 2-3 hours? | 23:14 |
| thumper | flacoste: hmm, I'll be out then | 23:14 |
| bac | EdwinGrubbs: try to pick items to QA that affect "major" pages | 23:14 |
| bac | EdwinGrubbs: and don't forget we have stuff on the 2.2.8 page too! :( | 23:14 |
| flacoste | thumper: ok, let's do this now then | 23:14 |
| barry | bac, EdwinGrubbs i'm working my way through the 2.2.8 page | 23:15 |
| bac | barry: cool. as above, try to hit the big pages first | 23:15 |
| barry | bac: k | 23:16 |
| mwhudson | ffs | 23:26 |
| mwhudson | why can i never remember how to drive R | 23:26 |
| thumper | bac: ping | 23:27 |
| thumper | mwhudson: drive R ? | 23:29 |
| mwhudson | thumper: the data analysis package | 23:29 |
| thumper | ah | 23:29 |
| * barry -> fude | 23:30 | |
| thumper | rockstar: ping | 23:32 |
| rockstar | thumper, pong | 23:32 |
| thumper | rockstar: call? | 23:32 |
| rockstar | thumper, sure, but I can't guarantee the life of my voice. | 23:33 |
| thumper | ok | 23:33 |
| wgrant | bac, salgado-afk: That fix is good. The link is now at the top - see https://edge.launchpad.net/~ubuntu-dev | 23:33 |
| thumper | rockstar: https://code.edge.launchpad.net/~mysql/mysql-server/mysql-6.0-bugteam | 23:37 |
| bac | thanks wgrant | 23:38 |
| thumper | bac: ping again | 23:47 |
| bac | hi thumper. sorry i missed your earlier one | 23:48 |
| thumper | bac: skype? | 23:48 |
| wgrant | Argh. | 23:54 |
| wgrant | Who reintroduced the mega-portlet on Ubuntu bug pages? | 23:55 |
| wgrant | The tags portlet now shows *every single tag* again. | 23:55 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
