/srv/irclogs.ubuntu.com/2009/12/10/#launchpad-dev.txt

lifelessmars: http://code.google.com/webtoolkit/speedtracer/get-started.html00:33
marslifeless, yep, took it for a spin.  Didn't help me diagnose anything.  It doesn't tell you what is going on, just "Your page is slow right about... here!"00:33
lifelessmars: oh ;(00:34
lifelessthe web page showed pretty graphs00:34
lifelessso I thought it might be usefuk00:34
marslifeless, it is nice to see exactly that "a script block at line 224 is slow", but I would like if it told me things such as "when was DOMContentReady fired?"00:34
marslifeless, thanks for sending the link though.  someone just beat you to it :)00:35
lifelessno worries, I'll console myself with a croissant & a sapporo00:37
marslifeless, we'll have to wait and see if this prods the Firefox team into adding proper profiling hooks to the Gecko engine.00:37
marsIE got it right the first time, Google spent money to make Webkit right, now Firefox has to catch up.00:38
thumperclucking bell00:39
* thumper wishes for a date_last_modified on merge proposals00:40
marsthumper, isn't there an activity log for them?00:40
marslike on bugs?00:40
thumpermars: no00:40
thumpermars: I have a db patch, but I'm not happy with it00:40
thumpermars: for the activity log that is00:41
wgrantCan someone please copy (with binaries) launchpad-dependencies 0.60 from https://launchpad.net/~wgrant/+archive/launchpad to hardy, karmic and lucid in the ~launchpad PPA?00:58
jmlwgrant, I don't know how to do that01:35
jmlwgrant, but if you tell me how, I'll try.01:35
jmlwgrant, also, do you know whether or not the "Rebuild testing in Launchpad" on https://dev.launchpad.net/Ubuntu/InfrastructureNeeds is done?01:40
* jml -> lunch01:44
* thumper does a little dance01:44
wgrantjml: Go to https://edge.launchpad.net/~wgrant/+archive/launchpad/+copy-packages?field.name_filter=launchpad-dependencies&field.status_filter=published&field.series_filter=karmic, select the package, select 'Copy binaries', select 'PPA for Launchpad Engineering', select Hardy. Repeat the process but select Karmic and then Lucid01:53
wgrantjml: It's sort of done.01:55
wgrantBut it's not exactly a complete implementation.01:55
wgrantjml: The status given on that page (with the URL involving my nick) remains current.01:56
fungaliciousjml: ping02:08
=== thumper is now known as eric-the-viking
=== fungalicious is now known as eric-the-half-a-
=== eric-the-half-a- is now known as fungalicious
=== eric-the-viking is now known as thumper
jmlhmm.04:26
jmlI wonder who fungalicious was04:26
wgrantI've seen him around here before.04:26
wgrantI think.04:27
thumperjml: it was kfogel at a friends place04:28
jmlthumper, ahh ok.04:28
wgrantjml: Can you please copy those packages?04:37
jmlwgrant, I'll load the page & see what I can do.04:46
jmlwgrant, "PPA for Launchpad engineering"?04:46
jmlwgrant, "copy", not "rebuild", right?04:47
wgrantjml: Yes, 'Copy existing binaries'.04:47
jmlwgrant, I'm on it like a leper messiah04:48
wgrantAha, it worked.04:49
jmlfor hardy...04:50
jmlThe following source cannot be copied:04:50
jmllaunchpad-dependencies 0.60 in karmic (same version has unpublished binaries in the destination archive for Karmic, please wait for them to be published before copying)04:50
wgrantWhat if you copy from the ~launchpad PPA to itself/04:51
wgrantOtherwise try again now, and it will work.04:51
wgrant(the binaries would have been publishing right as you pasted that error)04:52
jmldone04:54
jmlgreat success04:54
jmlwgrant, now, I would like you to answer a million questions about https://dev.launchpad.net/Ubuntu/InfrastructureNeeds :)04:55
wgrantjml: Thanks!04:55
wgrantAsk away.04:55
jmlThere's one called "Rebuild testing in Launchpad"04:55
jmlwgrant, Is it still current?04:56
wgrantjml: The comment there about the lack of publishing is current.04:56
wgrantSo, yes.04:56
jmlwgrant, ok. that's good. I'll chase up that RT then.04:56
jmlwgrant, do you know the best person, in general, to talk to about this page?04:57
wgrantjml: What *is* that RT? The code isn't there yet.04:57
jmlwgrant, the text of the wiki page makes it sound like the code is there :(04:58
wgrantjml: It does, but it is wrong.04:58
wgrantOdd.04:58
wgrantIt certainly requires significant sysadmin intervention too, though.04:58
wgrantspm: So, feel like building a buildbot image?04:59
wgrantI think all the pieces are together now.04:59
jmlwgrant, it's mostly just this conversation: http://paste.ubuntu.com/338470/04:59
wgrantAh, yes, that one.05:00
spmwgrant: sure; but not today; probably not tomorrow. monday is more likely/possible.05:01
wgrantArgh, OK.05:01
wgrantIs the release the usual time?05:02
spm2200 next week? ~9am here? I assume so. haven't hear otherwise so far.05:04
* wgrant must somehow arrange testing of the v3 branch on dogfood tonight, since it's not going to be merged long before release :/05:06
jmlwgrant, anyway, I want to start linking up that page: make sure there next action on each item is clear, and probably have a contact person for each thing (or just for the page as a whole)05:25
wgrantjml: Sounds good.05:26
wgrant"The current activity log in the Launchpad bug tracking system does not capture milestone changes." <- that is false05:26
wgrantUnless source packages are special.05:26
wgrantIt certainly works for products.05:27
jmlwgrant, so, for the rebuild thing05:31
jmlwgrant, do you know of anything that needs to happen other than the RT?05:31
wgrantjml: Besides all the sysadminy stuff, it probably just needs a rebuild archive mode to be added to process-accepted.py, publish-distro.py, etc.05:37
wgrantThere's little additional code required, now I think about it.05:38
jmlwgrant, could you please file a bug for it -- I wouldn't be able to describe what's needed as well as you could.05:42
wgrantjml: Sure.05:42
jmlthanks05:42
wgrantjml: Are you going to BFB/LCA?05:43
jmlindeed.05:44
jmlwgrant, are you confirmed to go?06:44
wgrantjml: Waiting on flight confirmation from the travel agent.06:46
cody-somervilleWhat is BFB/LCA?06:48
wgrantcody-somerville: BFB == build-from-branch sprint in January06:49
wgrantLCA == linux.conf.au06:49
cody-somervilleIs it intentional that you can target a bug task to a milestone that doesn't belong to the series the bug task is targeted to?06:57
wgrantI'm not sure that anybody knows how series targets are meant to work.06:59
daniloscody-somerville, wgrant: the problem with series is that you can't untarget it, at least not that I know off07:08
jmlinternet sucks.07:11
jmldanilos, hello07:11
wgrantdanilos: Correct, you cannot.07:11
danilosjml, good morning07:13
danilosjml, perhaps now is a good time to talk about https://dev.launchpad.net/VersionFourDotO/Roadmap07:14
jmldanilos, that's exactly what I was thinking.07:14
lifelessdanilos: yuou can decline, but not delete the record.07:17
daniloslifeless, but if you accepted it (for mistakenly thinking you can deal with it in the series), there's no way to decline it07:50
daniloslifeless, in those cases, I just mark them as 'won't fix' for the series, though it still is ugly07:50
* jml gone07:51
adeuringgood morning08:23
wgrantbigjools: Morning.09:12
mrevellGuten morgen!09:13
bigjoolsmoin all09:13
wgrantbigjools: How close is dogfood to v3-ready? It looks like I'll finally be able to get the branch merged on Monday (once a LOSA updates the buildbot AMI), but it would be nice to test this week.09:15
bigjoolswgrant: it's ready now09:15
bigjoolsI threw a load of old style stuff at it yesterday and it seemed ok09:16
wgrantbigjools: Even with the branch merged?09:16
wgrantExcellent.09:16
bigjoolsah there's one more to merge isn;'t there?09:16
wgrantRight. distroseries-source-format-selection, which cannot be merged until buildbot has dpkg >= 1.15.409:16
bigjoolsI'll merge that now09:17
wgrantThanks.09:17
bigjoolson DF I mean :)09:17
wgrantRight.09:17
wgrantWhile you're there, can you enable 3.0 (native) and 3.0 (quilt) for Lucid?09:17
bigjoolsok09:17
wgrantThen I'll try to upload stuff to my PPA, if that's possible from out here.09:17
bigjoolswgrant: I'm not sure if lamont updated the builders yet though09:21
wgrantbigjools: Ah, that could be a little inconvenient, although it would still allow all the Soyuz code to be tested.09:22
bigjoolsyep09:23
bigjoolswgrant: ok it's ready09:28
bigjoolsbuildd-manager is not running but poppy is09:28
* wgrant tries to remember the host from the PPA beta days.09:28
wgrantHm, I guess it's all mawson anyway.09:28
bigjoolsI'll run process-upload on demand09:29
bigjoolsyes ppa.df.l.n09:29
wgrantbigjools: There should be an upload sitting there. I believe it should be accepted.09:33
bigjoolswgrant: ok gimme 10m09:33
bigjoolsOTP09:33
wgrantbigjools: Sure.09:35
bigjoolswgrant: you might want to use an upload path that works :)09:55
wgrantbigjools: Oh, right, I copied my lpdev stanza.09:56
wgrantbigjools: That should work a bit better.09:56
bigjoolswgrant: it worked09:57
wgrantbigjools: There's now a Karmic upload, which should be rejected for being an unsupported format.09:58
* bigjools runs p-u09:58
bigjoolsrejected, yup09:59
bigjools"ftplib_3.1-1-8~wgrant1.dsc: format '3.0 (quilt)' is not permitted in karmic.09:59
wgrantbigjools: Perfect.09:59
bigjoolsso we just need to beat up lamont now09:59
wgrantI'll find something 3.0 (native)  and another with some component origs in a sec. But yes, then we need a lamont.10:00
wgrantbigjools: Two more uploads for you.10:09
bigjoolsok10:09
bigjoolsboth ok10:09
wgrantIndeed. Thanks.10:10
wgrantAnd one more to test component orig sharing. I think that's it.10:12
bigjoolsit went in ok10:12
wgrantAlso, do you have the appropriate stuff for testing syncs there?10:13
wgrantWould be good to be sure about that.10:13
bigjoolshmmm not entirely sure10:13
wgrantOr archive admins will break down my door and lynch me.10:13
wgrantAlthough I guess most of the sync-related scripts are owned by ~ubuntu-archive, so they can always be hacked post-release.10:14
wgrantI know that sync-source.py itself works.10:14
bigjoolsok10:15
bigjoolsthat's the only thing I know about10:15
wgrantThey have a whole set of secret scripts that will probably break in obscure ways, but we'll find out on Thursday...10:15
wgrantThis is new. Reporting Mandriva bugs against launchpad.10:28
deryckMorning, all.10:59
=== matsubara-afk is now known as matsubara
wgrantbigjools: You suggested last week that I move getBinaryPackageFormat from c.l.helpers to lp.soyuz.scripts.gina.handlers. But the only tests for that function are doctests in the docstring, and l.s.s.gina.handlers doesn't have anything looking for doctests in it. I suppose I could add a test module that just adds a DocTestSuite over that module -- does that sound OK?12:24
bigjoolswgrant: if you feel sufficiently motivated, yep!12:29
wgrantbigjools: There's also an unused getBinaryPackageExtension there which uses the same constant, so I'll remove that...12:30
bigjoolsok12:30
=== henninge is now known as henninge-lunch
bigjoolswgrant: lamont has installed the dpkg on ferraz so we can punt some 3.0 builds through it12:33
wgrantbigjools: Awesome.12:34
wgrantExcept for the big queue which will have to be eliminated.12:34
bigjoolsI'll make you a buildd admin12:35
bigjoolsthen you can rescore as you wish12:35
wgrantAh, that would be handy. Thanks.12:35
bigjoolswgrant: done12:38
wgrantbigjools: Thanks.12:38
wgrantbigjools: Since ferraz seems non-virtual, can you non-virtualise my PPA?12:42
bigjoolswgrant: or I can make ferraz virtual12:43
bigjoolsin fact you can do that :)12:43
wgrantbigjools: As long as it's a no-op reset trigger, I guess.12:43
* wgrant tries.12:43
wgrantbigjools: Is buildd-manager running?12:45
bigjoolswgrant: no... I'll start it12:50
wgrantbigjools: Ah, that would do it. Thanks.12:50
bigjoolswgrant: ok it's off and running12:51
=== mrevell is now known as mrevell-lunch
wgrantbigjools: buildd-manager is quite happy to deactivate ferraz due to an architecture mismatch, but won't start a build when the architecture is correct.12:57
bigjoolsmissing chroot probably12:58
wgrantAh, possibly :(12:58
bigjoolsI'm checking12:58
wgrantThanks.12:59
bigjoolswgrant: you're building lucid right?12:59
wgrantbigjools: Yes. lucid i386, since ferraz thinks it's i386 ATM.13:00
bigjoolsok will sort it shortly13:00
bigjoolsit's actually amd64, it's getting fixed13:02
wgrantAh.13:03
bigjoolswgrant: ok I'm running queue-builder, we'll get your build dispatched soon ish13:36
wgrantbigjools: Why do you need queue-builder?13:37
bigjoolswgrant: your source upload won't have had a build created for lucid without the chroot13:41
wgrantbigjools: The chroot exists, but the librarian file does not (I suspect).13:43
wgrantThe builds are very much there.13:43
bigjoolsoh right, fair enough, I thought the lot was missing13:43
bigjoolsso why isn't it dispatching :/13:44
bigjoolsah I see13:44
bigjoolsok off itgoes13:45
wgrantWhat was wrong with it?13:45
bigjoolswhen you flip to virtual you need to set the vm_host13:45
wgrantOh.13:45
wgrantOops.13:46
bigjoolswhich is the same as the real host since it's not a virtual machine13:46
wgrantRight.13:46
=== mrevell-lunch is now known as mrevell
* bigjools goes to lunch whiile it builds13:47
wgrantShould I flip it to manual so it doesn't start trying to build stuff that doesn't exist in the dogfood librarian?13:47
wgrantArgh fuck.13:49
wgrantpost-Karmic dpkg + unpatched lp-buildd == fail13:49
wgrant(https://bugs.edge.launchpad.net/launchpad-buildd/+bug/476036/comments/1)13:51
mupBug #476036: sbuild needs to run dpkg-source inside the chroot <Launchpad Auto Build System:Triaged> <https://launchpad.net/bugs/476036>13:51
sinzuiChex: ping14:12
=== danilos is now known as danilo[home]
Chexsinzui: hello14:16
sinzuiChex: Can you merge lp:~sinzui/launchpad/false-packages on staging so that I can verify it fixes a performance bug?14:17
Chexsinzui: yes sure, let me take a look14:17
Chexsinzui: All changes applied successfully.14:22
* sinzui eagerly awaits the restart14:23
bigjoolswgrant: gah, pqm will be closed on monday for the release, so we could do with these AMIs getting updated quicker14:23
wgrantbigjools: That's what I thought. But the new launchpad-soyuz-dependencies requirement delayed things a bit.14:24
wgrantBut that's all done now. It just needs the AMI to be upgraded.14:25
bigjoolswgrant: what exactly does it need, just the dpkg change or just a rebuild with the new meta-package update?14:25
wgrantbigjools: All it actually needs is the new dpkg(-dev). But I was told it should really have the metapackage too.14:26
bigjoolsyeah that's what I think14:26
bigjoolsso if the l-d-d meta package is updated it should all be ready?14:27
wgrantbigjools: It's all there and in the ~launchpad PPA.14:27
* bigjools files an RT14:28
Chexsinzui: staging restarted for you now.14:28
wgrant(launchpad-soyuz-dependencies 0.60 is sufficient)14:28
sinzuiChex: thanks14:28
bigjoolswgrant: l-d-d depends on that though right? so updating l-d-d should DTRT14:28
wgrantbigjools: Right.14:29
bigjoolsparfait14:29
wgrantbigjools: Although I'm not sure what apt sources the AMIs use.14:29
bigjoolsme neither14:29
wgrantBut the relevant dpkg is in both the PPA and somewhere in CAT, and l-d-d won't upgrade without it, so it should be fairly obvious to anybody who knows how to upgrade them.14:30
flacostemorning launchpadders14:30
bigjoolsflacoste!14:30
sinzuibac: Bug #495051 [UnboundLocalError editing proposed team membership]14:39
mupBug #495051: UnboundLocalError editing proposed team membership <oops> <Launchpad Registry:Triaged> <https://launchpad.net/bugs/495051>14:39
=== salgado is now known as salgado-lunch
=== matsubara is now known as matsubara-lunch
sinzuiEdwinGrubbs: ping15:20
EdwinGrubbssinzui: pong15:21
sinzuiEdwinGrubbs: As you can see on https://staging.launchpad.net/ubuntu/lucid/+source/grub2, grub2 has been published 2 times in lucid, and the DistroSeriespackageCache has three entries. That is why +packaging lists the entry more than once. I think bac's suggestion of using DISTINCT is the correct fix.15:22
EdwinGrubbssinzui: that shouldn't cause any problems15:25
wgrantsinzui: I'm confused about bug #495084. You say that hwtest is shown as not being in Lucid, and then cite a release of checkbox as evidence to the contrary. But those are different packages.15:35
mupBug #495084: source package +index says there is no current release <package-link> <Launchpad Registry:Triaged> <https://launchpad.net/bugs/495084>15:35
sinzuiwgrant: https://staging.launchpad.net/ubuntu/lucid/+search?text=hwtest lists the packages that are in lucid15:37
wgrantsinzui: Those are binaries.15:37
sinzuiYes, and https://staging.launchpad.net/ubuntu/lucid/+source/hwtest15:39
sinzuisays there is not source (wrong I think since the source came from karmic) and the Releases in Ubuntu look wrong because of this15:39
wgrantsinzui: There is no source.15:39
wgrantThere are just binaries from the new (checkbox) source with the same name as the old source.15:40
wgranthwtest hasn't existed as a source package since early Jaunty.15:40
sinzuiwgrant I think the warning message is misleading. It is often pointed to in bugs as an indication that the package is not in the series.15:42
wgrantsinzui: But the package is not in the series, so pointing to it as an indication of that is perfectly valid.15:43
wgrantbigjools: Do you think an AMI rebuild will be possible in time?15:48
henningesinzui: Hi!15:48
bigjoolswgrant: it's on the losas radar, I'm trying15:48
sinzuiwgrant: I agree with the points of your argument, but I am still dissatisfied. I think the message on the +source The source *must* be in lucid because I cannot rebuild lucid without it15:48
sinzuihi henninge15:49
wgrantsinzui: What do you mean "I cannot rebuild lucid without it"?15:49
wgrantbigjools: Thanks.15:49
henningesinzui: I'd call bug 165146 fixed, isn't it?15:50
sinzuiwgrant: I think this works in lucid:15:50
sinzui    apt-get source hwtest15:50
mupBug #165146: lp admins cannot deactivate accounts <Launchpad Registry:Triaged> <https://launchpad.net/bugs/165146>15:50
wgrantsinzui: Because 'apt-get source' looks up binaries if a matching source cannot be found.15:50
sinzuioh, I may have fixed that when I exposed accounts to LOSA15:51
wgrantsinzui: Note that 'apt-get source hwtest' will download a source package named 'checkbox'15:51
sinzuihenninge: maybe not. I think you need to ask a  LOSA. They can deactivate the account, but I am not certain they can deactivate the profile (which will be renamed ~name-deactivated)15:52
henningesinzui: I was about to ask who the account - profile split comes into play.15:52
henningesinzui: so this is not bug 49676?15:52
henningemup: bug 4967615:53
wgrantIt's private.15:53
henningeah, right15:53
henninge"Administratively disable account"15:53
sinzuihenninge: accounts are official SSO now. The foundations teams still works with them. The registry team manages the Launchpad profile . The admin in this case only want to deactivate the profile.15:54
sinzuiYes, that bug is private. It has user info that should not be public15:55
henningesinzui: so when a LOSA goes to IPerson:+review and deactivates the user, what happens?15:55
=== matsubara-lunch is now known as matsubara
=== beuno is now known as beuno-lunch
sinzuiThe status of the account is changed15:55
henningesinzui: what is the rationale for only letting LOSAs do that?15:56
sinzuiDeactivating a profile renames the profile and unlinks the Person from many object15:56
sinzuihenninge: SUSPEND15:56
matsubaraChex, gary_poster, rockstar, bigjools, danilos, sinzui, allenap: LP production meeting in 4 @ #launchpad-meeting15:56
gary_posterthx15:56
henningesinzui: ?15:57
bigjoolsok15:57
sinzuihenninge: a suspended account can be reactivated if we are in error or the user proves he will not do bad things again15:57
henningesinzui: and that is what's happening on +review, right?15:57
henningeand we don't have a UI to completely deactivate a profile, right?15:58
* sinzui starts dev and looks15:59
sinzuihenninge: +review allows a LOSA to change the user name and standing16:00
wgrantsinzui: anyway, the hwtest source has never existed in Lucid. https://edge.launchpad.net/ubuntu/+source/hwtest/+publishinghistory shows that.16:00
gmbAnyone: What's the quickest way to the the URL of the current rootsite (i.e. bugs.launchpad.net blueprint.launchpad.net, etc.?)16:00
henningesinzui: my core question is: why is that a LOSA task?16:00
wgrantAnd it is 3am, so I should sleep.16:00
sinzuihenninge: +reviewaccount allows the LOSA to set the account status and passowrd16:00
henningewgrant: good night, thanks for all your help16:01
sinzuihenninge: it is NOT a losa task.16:01
sinzuihenninge: The bug points out that the user does not know how to find the link16:01
henningesinzui: sorry, LP admin16:01
sinzuihenninge: do you know where the link is?16:01
henningesinzui: it is at the bottom of "Edit details"16:02
henningethat is were the user can deactivate his own account16:02
sinzuicorrect. many users do not want to change their details, they want them removed.16:02
sinzuihenninge: And the core bug is still open. Foo Bar cannot deactivate a profile....403 is raised16:03
henningesinzui: huh? I can select "Deactivated account" on +reviewaccount and the user is deactivated.16:04
henningesinzui: that is what spm does when you ask him to deactivate a user.16:05
sinzuiaccount is NOT profile16:05
sinzuiprofile = Person, account =Account16:05
henningesinzui: ok, so my question is "Why does it take a LOSA to deactivate an Account?"16:05
sinzuito stop spammer16:06
sinzuihenninge: I think you have a fundmental misunderstanding.16:06
henningesinzui: Exactly!16:06
henningesinzui: no, we are right were I was on Monday ;-)16:06
sinzuihenninge: We adding this feature so that admins could see the SSO data and address spammers.16:07
henningesinzui: I had 10 spam accounts and had to wait for spm's shift to get them deactivated.16:07
sinzuihenninge: The feature needs to move to SSO16:07
henningesinzui: again, what is the ratinale for having *only* admins to be able to do that?16:08
sinzuihenninge: escalate it to another LOSA. That is what I do when I see spam16:08
sinzuihenninge: THIS IS FUCKING SSO. THIS IS UBUNTU16:09
henningesinzui: Quoting tom "spm is handling LP requests this week."16:09
sinzuihenninge: we do not own this data16:09
henningesinzui: ok, so for spam accounts, I'd be happy if I could just remove homepage_content.16:10
sinzuihenninge: I know *you* want to fix this, but *you* are not thinking of the consequences of other systems and their owners16:10
henningesinzui: I get it, the account is not really Launchpad data.16:10
henningebut the homepage_conent is.16:11
sinzuitrue16:11
sinzuithe issue here is that answers is really for user requests. This is OUR request, and it needs prompt attention. check, or mthaddon can addresses16:12
henningesinzui: maybe it would be easiest to just rotate the current CHR into Launchpad Admins so that he can remove spam more easily.16:12
henningesinzui: so you know about the close-account.py script?16:13
sinzuiI know it, not its contents16:14
henningeIt does some sql magic to completely remove a profile (and an account).16:14
henningestub agreed, though, that it might fail on more complex data but luckily that is not the case for spam accounts.16:15
sinzuihenninge: I think we just want a safe and easy what to mark an account suspended. The profile and account can be restored if we were is error16:16
henningesinzui: and isn't +reviewaccount such a way?16:16
sinzuihenninge: and to be clear, we have unsuspended user accounts after the user verified their machines were disaffected16:16
sinzuihenninge: +reviewaccount is the only *correct* way16:17
henningesinzui: so could we not open it up to the CHR in some way? That would make processing quicker and save LOSA time.16:17
sinzuihenninge: you and I do not have the rights to see the openid16:17
sinzuior the password16:17
henningeits not in clear text, is it?16:18
henningeanother great feature would be "Resend registration mail."16:18
sinzuihenninge: The form needs to select the available field and info based on ~admin or ~somethingelse that is not all ~launchpad16:19
henninge~registry16:19
sinzuihenninge: the password is not ever shown, but the password can be changed16:19
sinzui~registry is traditionally project, not person.16:20
henningeok, so we create a new team?16:20
sinzuihenninge:  we are forbidden to create new teams/permission types without going before flacoste and his tribunal of standard bearers16:22
henningelol16:22
henningesinzui: so, the whole IPerson code is in registry, why not just use ~registry?16:23
sinzuiThat may be right, I ponder what will happen with teams which are also IPerson16:24
henningesinzui: they don't have an account, so they are not affected by changes to +reviewaccount, are they?16:25
sinzuiThey do not have an account, but the permission change may give me other powers, like merge!16:28
* sinzui may want merge powers16:28
henningesinzui: great, one thing less to delegate to the LOSAs16:29
sinzuihenninge: The reason we have not exposes team merging id because it often fails with timeouts16:29
sinzuihenninge: it often fails with profiles that have been used...16:29
henningesinzui: you see, I always thought we were delegating tasks to LOSAs because they required SQL magic.16:30
sinzuihenninge: barry's profile will take 22+ efforts to complete a merge16:30
henningesinzui: but for stuff that has a UI, why do the LOSAs have to do the clicking?16:30
sinzuidude!16:30
henningesinzui: document it on the CHR wiki ... ;)16:30
sinzuiYou've seen my list of stuff my team has to do, and surely you know that 1/3 of my work is personal contributions, and that I have landed more CHR features than anyone!16:31
sinzuimerging is really hard16:31
henningesinzui: wonderful, I am not critizing you!16:31
henningeor am I?16:31
sinzuiread the blueprint https://blueprints.edge.launchpad.net/launchpad-registry/+spec/cleansing-deactivated-accounts16:32
gary_posterbigjools: replied to your question about buildbot. summary: ignore, proceed with what you and muharem were already doing16:34
sinzuihenninge: merging truly sucks. The feature was designed for unclaimed profiles. But users are using it for active profiles. You and all CHR users are welcome to work on this.16:34
sinzuihenninge: It really is hard.16:34
henningesinzui: I understand.16:35
sinzuihenninge: My rule is to let users do anything needed to correct their mistakes. I do not put CHR/LOSA barriers in the way. But features created 5 years ago still have those barriers.16:35
bigjoolsgary_poster: ah I missed the sourcecode pull fail, thanks16:36
henningeI see.16:36
gary_postercool, np16:36
henningesinzui: so, originally we did not talk about merging, we talked about deactivation. As a quick and easy way to get rid of spam accounts.16:37
=== salgado-lunch is now known as salgado
henningesinzui: You are saying, that exposing this might also expose the merge functionality of teams, right?16:37
sinzuihenninge: We do not deactivate bad accounts because they can be reactivated by the user. we SUSPEND bad accounts16:37
henningesorry, suspend ;)16:38
=== danilos is now known as danilo-bblot
henningesinzui: could we not do as you suggested and make +reviewaccount select data to display and make the reduced version available for ~registry?16:39
sinzuihenninge: Since we are not adding new levels of permission to security.py, the implementation implies that ~registry is being added to IPerson launchpad.Admin. That gives you a lot of power over people and teams16:39
henningeright16:40
sinzuihenninge: as I said, adding celebrity teams or levels requires approval from others16:40
henningeyeah, got that ...16:41
flacostesinzui: could we use launchpad.Moderate instead of launchpad.Admin for suspending an account?16:42
flacostesinzui: and give ~registry launchpad.Moderate permission16:42
henningeflacoste: I was just about to suggest that.16:43
sinzuiflacoste: henninge The checker is ReviewByRegistryExpertsOrAdmins16:43
henningeor rename launchpad.ProjectReview to launchpad.Review and use that.16:43
flacostesinzui: checkers for launchpad.Moderate or launchpad.Admin?16:43
sinzuiWe use that for launchpad.ProjectReview because launchpad.moderate is taken by something else16:44
=== beuno-lunch is now known as beuno
henningeI think that is for ProjectReview16:44
flacosteiirc, there was a technical difficulty in merging ProjectReview and Review16:44
henningeflacoste: I don't see an existing launchpad.Review16:45
henningeMy itention was to broaden the scope of lp.ProjectReview16:45
sinzuihenninge: in security.py add16:49
sinzuiclass ModeratePerson(ReviewByRegistryExpertsOrAdmins):16:49
sinzui    permission = 'launchpad.moderate'16:49
sinzui    usedfor = IPerson16:49
sinzuithen change EditAccount to16:49
sinzuiclass EditAccount(ModeratePerson):16:49
sinzuihenninge: I am mistaken I think. The user needs access to his account (to deactivate it) so the permission may need to remain launchpad.Edit16:50
sinzuihenninge: lp.ProjectReview is correct. There is a conflict with FAQ moderate that blocks us from using the permission16:51
henningesinzui: yes, so the XXX says.16:55
henningesinzui: remain lp.Edit for EditAccount?16:55
henningeok, I'll put that in a bug.16:56
sinzuihenninge: I think so16:56
sinzuihenninge: I think the code needs to look like this: http://pastebin.ubuntu.com/338783/16:58
sinzuiwgrant: beuno: bigjools:I am have a very difficult time fixing +packaging because of the special case where we know exactly what is published in Ubuntu.  The problem is that users are creating packaging links for packages in PPAs, not the primary archive.17:16
adiroibancan I use YUI3 event-keys for Launchpad ?17:16
bigjoolssinzui: do you need to know the publishing state of PPA packages as well?17:17
beunosinzui, sinzui if you give me a little bit more context, I can try to help out17:17
sinzuiwgrant: beuno: bigjools: I am very tempted to add Ubuntu validation rules that prevent users from creating such links. I could then purge the PPA links from the DB, and remove the filter used in +packaging that ensure we only make links to published packages17:17
* bigjools has to go in 5 minutes17:17
sinzuibigjools: I really do not care about the publishing state. I want to avoid an ubuntu contributor seeing a link from lucid/+packaging to a lucid/+source that is not really in lucid17:19
bigjoolssinzui: ok, so simply using Ubuntu state will suffice, no?17:19
sinzuibigjools: I pushed the filtering into model, but it breaks tests that are very hard to reconstruct knowing that we hide packaging links17:20
beunosinzui, you will not get an opposition from me to get ubuntu-specific rules in Launchpad17:20
beunoin reality, there are no other distros really using Launchpad17:21
sinzuibeuno: I am also tempted to remove debian and all the other distros from the packaging form17:21
bigjoolssinzui: you could make it generic actually, if you look for something published in an archive with a purpose in (main_archive_purposes)17:21
sinzuimaking the feature Ubuntu specific is not project friendly, but to be clear, the only use for it is Ubunut17:22
bigjoolsif you use my suggestion you're guaranteeing that we only link to packages in distros that Soyuz publishes17:23
sinzuibigjools: I use that in the filter. I would reuse it for validation if we decide packaging links are not for ppas or other archive in cyberspace17:23
bigjoolssinzui: I have to run now, but perhaps I can take a longer look at this tomorrow17:24
bigjoolsand see if I can offer more help17:24
sinzuibeuno: look at the Packages by distribution section of https://staging.launchpad.net/bzr-gtk/+packages17:26
beunosinzui, yes...17:26
sinzuiBy making packaging links Ubuntu specific, we will remove about 100 links to Debian and other distros17:27
beunosinzui, how did those links get created?17:27
sinzuiProject create the links from this same page to communicate to users that their software has packages for distributions17:28
beunosinzui, one thing to do there, I think, would be to not have empty sections for the unlinked17:29
sinzuibeuno: click (+) Link package under a series17:29
sinzuibeuno: but we need the section to allow someone to create a link17:30
beunosinzui, yes, we can try and solve that in a different way17:30
beunosinzui, you could show Ubuntu by default17:30
sinzuibeuno: we talks about that.17:30
beunoand offer "Other distributions" separately17:30
sinzuibeuno: We agreed it should should the current series by default17:30
beunoin the +addpackage I mean17:31
beunosinzui, ok, then we need to fix the UI17:31
beunomake it a one liner17:31
beunoprobably drop the "No packages..."17:31
sinzuibeuno: Yes, that is what we talked about in August. The challenge is to remove/hide the complexity of the package, and to show the Ubuntu packaging history....17:31
sinzuibeuno: ... so that we merge the common form with the special ubuntu form: https://staging.launchpad.net/bzr-gtk/0.93/+ubuntupkg17:32
beunosinzui, in the  +addpackage?17:34
sinzuibeuno: To be clear, there are two form, one to link ubuntu and one for everyone else. User do not link the ubuntu one because they want to create links to show historical information17:34
sinzuibeuno: so we discussed merging the forms last August to do the right thing by default, but allow users to record historical/other-distro info if they want to17:35
beunosinzui, right. And the problem you are trying to solve now is...?17:36
sinzuibeuno: but we did not realise at the time that users are using this to indicate that packages are available in PPAs, not in the default distro release17:36
beunoaha17:36
beunosinzui, and how do they say in which PPA?17:36
sinzuiand we can only verify Ubuntu, so we must have two rules, or desupport other distros17:37
sinzuiabout 1/3 of lucid packaging links are for PPAs, not primary+partner17:38
sinzuibeuno: the user does not say it is a PPA. We do not let projects indicate when ppas they are packaged in17:38
beunosinzui, ah, ok, I'm starting to understand then17:39
beunoso, links to PPAs is something people really really super want17:39
sinzuibeuno: and the project pages talk about packages, not packages that are default in distros17:39
sinzuia link would be super17:39
beunoso we either make the linking to PPAs something we support (ie, let them say where)17:40
beunoor not support that use case17:40
beunoand bump the linkage feature  :)17:40
sinzuibeuno: This is a great example of a feature we designed for Ubuntu, that is co-opted by projects to do something else.17:40
sinzuiI favour making this feature only about Ubuntu. Then allow projects to indicate their PPAs, which already list the supported distros17:42
sinzuibeuno: also, Ubuntu only cares about primary packaging. If we only support Ubuntu, we can remove the nasty part of the form17:45
beunosinzui, I'm a bit uneasy about dropping support for debian specifically17:47
sinzuibeuno: this is what will be dropped: https://staging.launchpad.net/debian/sid/+packaging17:47
beunosinzui, ok, kill it17:48
beunoif we find a sensible use case for it, we can always bring it back in better shape17:48
sinzuiYes, if we want other distros to use it, it needs a different feature set17:51
mrevellnight all18:06
bacsinzui: ping18:12
sinzuihi bac18:13
=== deryck is now known as deryck[lunch]
bacsinzui: call soonish?18:14
sinzuiI am ready now18:14
bacme218:14
sinzuibac: I am looking at https://bugs.edge.launchpad.net/launchpad-registry18:33
sinzuibac: https://bugs.edge.launchpad.net/launchpad-registry/+bug/29692718:34
mupBug #296927: checking menu links <feature> <tech-debt> <Launchpad Registry:Triaged> <https://launchpad.net/bugs/296927>18:34
=== deryck[lunch] is now known as deryck
sinzuihttps://edge.launchpad.net/launchpad-registry/+milestone/series-3.119:25
thumpermorning20:04
=== salgado is now known as salgado-afk
adiroibanwhen creating a MP, is there a way to link it to a bug so that the bug will be updated to fix-commited on merge, and fix release later.20:18
adiroibanthe branch is already linked to the bug20:18
sinzuiadiroiban: no, the important link is between the bug and the branch20:18
adiroibansinzui: ok. so I have to manually update the bug status?20:19
sinzuiadiroiban: The bug subscribers will be notified with the bug is merged. There is no automatic status change because the branch may only partially fix a bug20:20
adiroibansinzui: thanks for the info. yep I was notified when the branch was landed20:21
sinzuiadiroiban: I am contemplating a rule to automatically mark bugs that are fix-committed to fix-released when when a milestone is released...but I need to hack bug karma to make sure the bug assignee get the credit20:21
adiroibansinzui: :) I don't care about karma, but it would be nice to have some magic help at updating bug status20:22
adiroibanlike bzr --fixes20:23
sinzuiThe karma rule is what is stopping me from doing this. it is important to some people. I stopped marking bugs fix-released when we do released because it steals karma20:23
adiroiban:(20:24
adiroibanwell... life suck :)20:24
adiroibansinzui: no problem. many thanks for the support. I can live with it20:25
=== matsubara is now known as matsubara-afk
wgrantsinzui: Do you now understand what I was explaining last night?20:31
sinzuiNo not really. I am certain the SP must be in the distro when we copy from the parent20:32
wgrantlamont: Hi. Do you have an lp-buildd with https://bugs.edge.launchpad.net/launchpad-buildd/+bug/476036/comments/1 fixed?20:32
mupBug #476036: sbuild needs to run dpkg-source inside the chroot <Launchpad Auto Build System:Triaged> <https://launchpad.net/bugs/476036>20:32
wgrantsinzui: What tells you that?20:33
sinzuiapt-get source20:33
sinzuidisregard the example of hwtest20:34
wgrantHave you another example?20:34
sinzuibzr apparently was not published in edgy or fiesty. yet apt-get source will get it.20:34
* sinzui looks for karmic example20:35
wgrantsinzui: It *was*, but is no longer.20:35
sinzuiwhy rewrite History? that sounds like a bug20:35
wgrantsinzui: Edgy and Feisty are no longer supported, so they were copied to old-releases.ubuntu.com, then all their publishings were removed from LP to save disk space on mirrors and the archive machine.20:35
wgrantHistory hasn't been rewritten; the publishing records are just now set to Obsolete.20:36
sinzuiThe page says it wasn't ever published though20:36
wgrantRight, that's not ideal.20:36
wgrantAlthough the message is true.20:37
sinzuihttps://edge.launchpad.net/ubuntu/edgy/+source/bzr20:38
sinzui^ That does not look right. There is a bug and am not certain what it is20:38
wgrantIt's true.20:39
wgrantEdgy has no packages left.20:39
wgrantEdgy is obsolete, so its packages have been removed.20:39
wgrantHowever that message could make that more obvious.20:39
lamont                        $dir = $1 if /^dpkg-source: (?:info: )?extracting \S+ in (\S+)/;20:39
sinzuimaybe we should not show that message at all for obsolete series20:39
lamontwgrant: ^^ that change in 2 places20:39
wgrantObsolete isn't like Superseded and Deleted (the two more normal removal states).20:39
lamontbut no, not quite packaged yet, will have it on dogfood for testing sometime tonight or tomorrow20:40
wgrantlamont: Right, I know the diff, but ferraz doesn't have it.20:40
wgrantlamont: Ah, great, thanks.20:40
lamontright20:40
lamontwgrant: is it blocking you?20:40
wgrantlamont: Not really. As long as it happens in the next day or two it's fine.20:41
lamontif so, I can smash it onto ferraz in a couple of hours, otherwise it'll just come as part of things on prolly monday (I want to test over the weekend)20:41
wgrantThe Soyuz code works, so that's no longer blocked on testing.20:41
lamontcool20:41
wgrantOK, sure.20:41
lamontoh hey - if you're done with dogfood, could you toss a no-change binutils upload at it?20:42
lamontwith ddeb exporting turned off20:42
wgrantI can't upload to main.20:42
lamonteven on dogfood?  how rude20:42
bigjoolsI can fix that20:42
lamontta - I need to go afk for about an hour or so, then I get to put in another long night20:43
lamontbigjools: the intention is to make sure that the pkgmanglebinary in the tarball I gave you earlier won't break production lp when binutils (e.g.) builds20:43
bigjoolswhich would be nice20:43
lamontyeah - there have been complaints about time served and all that20:44
lamontmust run20:44
bigjoolswgrant: you're now in core-dev on DF20:44
wgrantbigjools: Thanks.20:44
sinzuiwgrant: I think this page is bong I think the reason is that zope3 existed for a short time in karmic: https://edge.launchpad.net/ubuntu/karmic/+source/zope320:48
sinzuiwgrant: in which case we should state the package is was discontinued and we do not need an alert because this is not extraordinary20:49
wgrantsinzui: Right.20:50
=== EdwinGrubbs_ is now known as EdwinGrubbs
bachi sinzui22:44
sinzuiHi bac. Do you need me to break something?22:44
bacsinzui: no, but can you look at a query and run it on staging?22:45
bachttp://paste.ubuntu.com/338974/22:46
* sinzui does22:46
bacsinzui: ah, wait.  i need to look for .tar.bz2 also22:47
bachmm, i wonder if my branch handles those...22:47
sinzui bac: http://paste.ubuntu.com/338987/22:49
bacsinzui: run this one instead http://paste.ubuntu.com/338986/22:49
sinzuiyes master22:49
bacjust the select, not the update22:49
bacthanks22:50
sinzuiI worked that out after getting nasty errors when blind pasted it into the terminal22:50
sinzuipgsql hates that query22:51
bacreally?22:51
bachmm22:51
sinzuiwow that is a  very large response22:51
bacmy sql fu is limited22:51
sinzui4713 rows22:52
sinzuibac: i'm getting a full report. pastbin hates me22:56
sinzuibac: I sent the report by mail because it killed FF when I tried to pastebin it22:58
bacsinzui: ok, thanks22:58
sinzuiabout 1/3 are plain/text22:59
bacsinzui: could you do it again with a 'mimetypes = 'text/plain' ?  those are really the ones we're interested in.23:02
sinzuiokay23:03
baci hate stumbling around via you as a proxy23:03
sinzuibac: http://paste.ubuntu.com/338993/23:07
sinzuidoesn't look like anyone is using lp to distribute porn23:07
bacwhee23:09
bacsinzui: i think the update at http://paste.ubuntu.com/338986/ is ready to run on staging.23:09
baci'll see if i can get chex to do it23:09
sinzuispm: certainly can, he's got mad skilz23:10
bacoh, i forgot spm was about23:10
bacexcept he isn't23:11
sinzuihe's mysterious like that23:11
sinzuihmm23:12
thumperwhere is spm today?23:12
sinzuilooks like packaging-views.txt is predicated of the current behaviour that projects link to source packages that are not published. This may require a total rewrite.23:13
wgrantYay for unrealistic tests.23:14
sinzuiit is very realistic. I want to change reality. Link to Ubuntu or get stuffed23:15
sinzuiI doubt this will make this release. Maybe a pot of coffee will help me fix this23:17
sinzuiwgrant: do you care about packaging links between projects and debian?23:18
wgrantsinzui: Not particularly, no. Packaging links aren't useful for much.23:18
wgrantThe only thing we use them for is defaulting to the correct upstream project when we click 'Also affects project'23:18
sinzuithey should be23:18
wgrantAnd we don't do that for Debian.23:18
wgrantsinzui: Why do you ask?23:19
wgrantI think, ideally, we would say that our upstream is the Debian package, and Debian would say that their upstream is the project, and the relationship would be transitive.23:20
wgrantBut that sounds hard.23:20
sinzuiwgrant: +ubuntupkg only permits users to link to the current Ubuntu development series. Project seem to prefer +addpackage because it lets them record that their project is packaged in an older Ubuntu series, and in a rare case Debian23:20
wgrantsinzui: Ah, you're dealing with the multiple view madness?23:21
sinzuiI do not think debian is ideaa because they will not fix the bug or sync the translations. We want the links so that users and tools can send bugs and translations upstream, and for Ubuntu to have upstreams tip.23:21
sinzuiwgrant: It is madness, because I think users co-opted the feature to report that they made a PPA.23:22
wgrantsinzui: Debian does fix bugs. Lots of them.23:23
wgrantsinzui: But yes, the PPA situation is a bit stupid.23:23
wgrantThat needs a fix, but I'm not sure how.23:23
sinzuiLetting projects link to ppas would help.23:23
wgrantIt would, but it's not perfect.23:24
wgrantI'm not sure that forbidding creation of packaging links to unpublished sources is a good idea. Sure, do not display them, but it might be useful for eventually providing an automatic list of PPAs for a project, much like https://launchpad.net/ubuntu/+source/dpkg does now.23:26
sinzuiexactly what I have been think!23:26
sinzuiyet the work of preventing links to unpublished packages in a series is process intensive23:27
wgrantYes :(23:27
wgrantI think somebody needs to go through all the use cases before much more can be done.23:28
wgrantAt the moment packaging links are pretty pointless, but they have the potentially to be very very handy.23:28
sinzuiwgrant: If we had some process to guide a user from making a PPA, to getting it into universe, it would make more sense to allow the links to be made first23:28
sinzuiAll the canonical launchpad developers are focused on making those links useful. My first problem is that users are not using them as we intended them.23:30
wgrantsinzui: But has work been done to work out what they mean? What they should mean?23:30
wgrantIt seems nobody is sure how they are going to be changed to be more useful.23:31
wgrantAnd I think it would be handy to know that before an attempt is made.23:31
sinzuiwgrant: That is something I need solved by the end of January23:31
sinzuiwgrant: A link is only useful of the launchpad knows the project's bug-tracker, development branch, and is setup for translation syncing. We need a message that explain why this import. Projects and packages need to state what is missing. We can suggest links too because in most cases, the project and source package have similar names23:34
sinzuis/of launchpad/if launchpad/23:34
wgrantsinzui: But packaging links should be useful the other way too.23:35
sinzuiI have not found may compelling reasons for the other way.23:35
wgrantsinzui: If I go to a project page, it should tell me where I can packages.23:35
wgrant+get23:35
sinzuiwell23:36
wgrantIf I click on a release, it should tell me where I can get packages of that particular release.23:36
sinzuiI suggested that we should emphasise that downloads is overrated. My argument is not favoured23:36
sinzuiI suggested that we should emphasise that.   Downloads is overrated. My argument is not favoured23:37
wgrantHas the community been asked for ideas on how to improve things?23:38
sinzuiI do not think so23:39
wgrantThat seems like a bad thing.23:39
sinzuiThe registry is working the the other teams this series. code, translations, and bugs need the links. They do not care how they get there, they just need to links in place to enable their systems23:40
wgrantBut you need to consider all the possible uses before you make a potentially short-sighted model change.23:41
sinzuiindeed. I have not made model changes because I have not talks to anyone23:41
sinzuiAll these features I am discussing already exist. the issues are user experience, and the fundamental question should they exist?23:42
* thumper lunching23:48

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