| StevenK | wgrant: My only concern with that branch was: | 02:54 |
|---|---|---|
| StevenK | 291- u'There is no project in Launchpad named "N\xf6 Such Product&<...] | 02:54 |
| StevenK | 294+ There is no project in Launchpad named "N... Such Product&<... | 02:54 |
| StevenK | I couldn't get print_feedback_messages() to show me \x..., it always printed the codepoint | 02:55 |
| wgrant | That's correct. | 02:55 |
| wgrant | Because it prints the string, not its repr. | 02:55 |
| StevenK | Right | 02:55 |
| StevenK | wgrant: Hm, I seem to have tripped over https://code.launchpad.net/~stevenk/launchpad/drop-kick-getBugTasks/+merge/74941 ; getBugTasks isn't in the monthly PPR | 06:10 |
| wgrant | StevenK: What do you mean? | 06:12 |
| StevenK | wgrant: I had forgotten all about that MP -- looking at the latest monthly PPR, I can't see getBugTasks at all, but it's still exported | 06:14 |
| wgrant | Ah, didn't notice it was old. | 06:14 |
| wgrant | If it's not in the PPR that shows the other methods, then it wasn't called at all. | 06:14 |
| StevenK | wgrant: As long as pageids is the right page to be searching | 06:18 |
| wgrant | StevenK: I believe so. Do the other methods appear there? | 06:18 |
| StevenK | IDistribution.getBranchTips shows up as Distribution:EntryResource:getBranchTips, so looks like. (DSP doesn't have any other exported methods, but has other exported properties) | 06:21 |
| wgrant | Right | 06:21 |
| StevenK | It was called 38 times in March, that is the only time I've seen it | 06:23 |
| StevenK | But other monthly PPRs have been eaten | 06:23 |
| SpamapS | Not sure anymore where to ask, but is there any way we can get a user unsubscribed from bugs on the heat project? Their out of office mail is spamming every change we make. | 06:51 |
| lifeless | wgrant: StevenK: ^ halp. | 07:04 |
| wgrant | Have you adequately slapped the user in the face for having a moronic autoresponder? | 07:05 |
| wgrant | Which user? | 07:05 |
| lifeless | SpamapS: | 07:06 |
| SpamapS | wgrant: https://launchpad.net/~faramir | 07:08 |
| * SpamapS is quite certain this, as most problems, is Lotus Notes' fault. | 07:08 | |
| wgrant | SpamapS, lifeless: lp-shell production devel; lp.load('/heat/+subscription/faramir').lp_delete() | 07:09 |
| wgrant | The project owner should be able to do that. | 07:09 |
| SpamapS | wgrant: maintainer? as in, ~heat-drivers? | 07:10 |
| wgrant | Yes | 07:11 |
| * SpamapS tries it | 07:12 | |
| SpamapS | wgrant: lovely, thank you. subscription removed. | 07:13 |
| wgrant | Excellent. | 07:13 |
| wgrant | Do stab the user and/or their email admins as well, though. :) | 07:13 |
| StevenK | wgrant: So, do you think that MP is landable? | 07:14 |
| wgrant | StevenK: I don't see why not. | 07:14 |
| StevenK | The only thing I can come up with is it drops the method from 1.0 | 07:14 |
| StevenK | But we probably don't care | 07:14 |
| wgrant | If it's not called frequently then we do not care. | 07:15 |
| SpamapS | wgrant: right, slapping subscribed. :) | 07:15 |
| === vila is now known as vila-afk-biab | ||
| === vila-afk-biab is now known as vila | ||
| === Ursinha is now known as Ursinha-afk | ||
| cjwatson | Ursinha-afk: Just having a go at your QA now in order to unblock myself | 15:38 |
| === Ursinha-afk is now known as Ursinha | ||
| cjwatson | Copied https://dogfood.launchpad.net/%7Eubuntu-unity/+archive/daily-build/+sourcepub/3476923/+listing-archive-extra to the primary archive on DF | 15:41 |
| cjwatson | wgrant: I restored the dogfood-publish config and its associated paraphernalia on labbu | 15:51 |
| cjwatson | Ursinha: This looks like a failure, I'm afraid - notes in the bug | 15:53 |
| Ursinha | cjwatson, will look | 15:53 |
| cjwatson | I wonder if this is DF-specific, since I thought you mostly just moved that code around ... | 15:53 |
| Ursinha | (sorry the delay in the answers, my ISP is not helping today) | 15:54 |
| cjwatson | Ah, but I guess that code never had to deal with copies from a PPA before | 15:54 |
| cjwatson | Odd, though, since _publishCustom uses self.packageupload too and it's clearly never None in those cases | 15:55 |
| cjwatson | So the PackageUploadCustom mustn't be being constructed in the right way, but why ... | 15:56 |
| Ursinha | cjwatson, yeah, I recall talking to wgrant about such cases and that never should be none | 15:56 |
| Ursinha | when I was transforming that code in a job | 15:56 |
| cjwatson | And for that matter how | 15:57 |
| cjwatson | I don't see anything other than PackageUpload.addCustom that constructs new PackageUploadCustom objects, and that sets packageupload=self | 15:59 |
| cjwatson | But the whole publisher looks kind of unhappy | 16:00 |
| Ursinha | cjwatson, have you tried to copy any other kind of package other than from a PPA? | 16:01 |
| cjwatson | No | 16:01 |
| cjwatson | That's the only case that matters though :) | 16:02 |
| Ursinha | cjwatson, yeah :/ | 16:03 |
| cjwatson | In fact this makes zero sense because the PUC is surely looked up from the PU in the first place | 16:05 |
| cjwatson | Oh, they have a PU but no SPR, I misread | 16:06 |
| cjwatson | Ursinha: OK, so PackageUploads that refer to copies (PackageCopyJobs) don't have a sourcepackagerelease - only direct uploads do | 16:10 |
| cjwatson | Ursinha: I think you need to do the same kind of things that PackageUpload.package_name and friends do, and look at either bits of packageupload.package_copy_job or packageupload.sourcepackagerelease | 16:10 |
| cjwatson | Ursinha: And of course have a test that exercises this in the context of copies | 16:11 |
| Ursinha | cjwatson, yeah.. I should have tested the ppa copies, I felt that was missing after submitted to pqm | 16:12 |
| Ursinha | I'm into it | 16:12 |
| cjwatson | Cool - I think you can use package_copy_job.target_distroseries (I think) or package_copy_job.component_name | 16:13 |
| cjwatson | For the various cases | 16:13 |
| cjwatson | But then you have to do something with the PackageTranslationsUploadJob creation ... | 16:14 |
| Ursinha | cjwatson, right, thanks for the huge help | 16:14 |
| Ursinha | yeah, I'll have to adapt it all | 16:14 |
| cjwatson | But I guess you know what to do there :) | 16:14 |
| Ursinha | :) | 16:14 |
| cjwatson | Yikes, attachTranslationFiles lives on SPR | 16:15 |
| cjwatson | I guess maybe there needs to be a similar thing for PCJ? Not sure | 16:16 |
| Ursinha | cjwatson, hm. that sounds ugly but might be the way to go | 16:17 |
| Ursinha | as pu doesn't have a spr it doesn't make sense to use the one it's there for such cases... but I'll check if there any other (less intrusive) way | 16:19 |
| cjwatson | I think you have to go through PCJ - I remember dealing with this for the queue API exposure | 16:20 |
| cjwatson | Ursinha: (see also my comments in https://bugs.launchpad.net/launchpad/+bug/851562, tangentially) | 16:23 |
| _mup_ | Bug #851562: Diffs not available for syncs on +queue page like for regular uploads <derivation> <package-copies> <package-diff> <Launchpad itself:Triaged> <https://launchpad.net/bugs/851562> | 16:23 |
| Ursinha | cjwatson, yep, I was reading that right now | 16:24 |
| Ursinha | cjwatson, I couldn't understand well in the call, internet is crappy, are you working on fixing that? | 16:25 |
| cjwatson | Ursinha: Not actively | 16:25 |
| cjwatson | Ursinha: I don't think it's a blocker for this, just context | 16:26 |
| cjwatson | You don't really *need* the SPR here, it's just that that's where the code currently lives and it uses some attributes from it | 16:26 |
| Ursinha | cjwatson, that's okay, I ask because I could look at that next, as the context is similar | 16:28 |
| cjwatson | Awesome | 16:29 |
| === BradCrittenden is now known as bac | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!