/srv/irclogs.ubuntu.com/2010/02/25/#launchpad-dev.txt

jtvOtherwise you're counting bugtasks instead of bugs.00:00
jtvsinzui: still ok?00:05
sinzuijtv: I can tune this. I think 150 should be the minimum heat threshold because that is the minimum number for a private bug00:07
sinzuiwell, 250 might be best because private bugs are not community driven, but security are, which is 25000:08
jtvsinzui: how does bug privacy factor into it all?00:08
sinzuijtv: I am reading the heat calculator. It add 150 points00:09
jtvoh ah00:09
sinzuiI am sure the numbers are arbitrary. so are mine. I just want to get the packages that need the most love to be listed first00:10
jtvsinzui: do you see a big performance difference between those threshold?00:10
jtvs00:10
sinzuijtv: 67.82..27941.89 is better than 85994.40..85994.4000:11
jtvsinzui: uh yes, looks like00:11
jtvsinzui: you could multiply the bug count by some factor to compensate.  The ideal factor might be something like the average bug heat for bugs that stay below your chosen threshold.00:14
sinzuijtv: maybe: The heat filter keeps the top packages 100 very distinct00:15
jtvsinzui: you mean it sets them very far apart from the packages further down the list?00:16
sinzuiyes, there is less variation in the mid range of packages because their bug heat scores are 000:17
jtvIn the context of this list, do you care a lot about those packages?00:18
jtvBecause maybe the list should just stop at some pseudo-arbitrary point.00:19
thumperIs anyone tackling the merge conflict from stable -> db_devel?00:26
mwhudsonthumper: i'd got as far as thinking about it00:26
mwhudsonthumper: seems trivial to fix, i'll do it00:28
thumpermwhudson: ta00:28
thumpermwhudson: rs=me if you don't want to rs=you :)00:29
mwhudsonheh thanks :)00:29
jtvsinzui: I'm off... good night & good luck!00:32
sinzuime too00:32
* mwhudson lunches00:35
lifelesshttps://wiki.ubuntu.com/PackagingGuide/PatchSystems00:46
lifelessbah sorry00:46
thumpermwhudson: Add a new code import result for success of partial import01:20
thumpermwhudson: was that the commit that you fixed after01:20
thumper?01:20
thumpermwhudson: just wondering about the qa status01:20
mwhudsonthumper: yeah, that's the commit that was fixed in two followup branches01:20
thumpermwhudson: we should change it's status to FIXED01:21
thumpermwhudson: I'm not sure I like the RCFIXED, as it wasn't an RC01:21
thumpermwhudson: perhaps just move it to OK now01:21
mwhudsonthumper: ok01:21
mwhudsonthumper: well, the fix hasn't been QAed01:21
thumperah01:21
thumperpoo01:21
mwhudsonyep01:25
=== bjf is now known as bjf-afk
kfogeljml: you on?02:19
* kfogel guessing not...02:19
kfogelthumper: ah, but you might know: do we have any bugs filed for work on patch/branch equivalence?02:19
kfogelor story pages, etc?02:20
kfogelmwhudson: ^^   (if you know)02:21
mwhudsonkfogel: i don't know of anything formal02:22
kfogelmwhudson: thanks02:22
kfogelmwhudson: I'm looking at bugs and branches under ~jelmer, just to see02:22
thumperkfogel: what exactly are you interested in?02:26
thumperkfogel: worth skyping about?02:26
kfogelthumper: hmrmrm.  It is worth skyping about, but I can't do a skype right now (need to wrap up this draft blog post about the new patch report feature).  I'm just listing, at the end of the blog post, some of the stuff we haven't gotten to yet, and that's on the list.  If there were a bug or a spec to link to, I'd link to it.02:27
thumperI don't think there is a bug about it02:27
thumperkfogel: one thing we haven't gotten around to02:27
thumperkfogel: is automatically turning a patch into a branch if it applies to trunk cleanly02:28
kfogelthumper: :-)  yes, that's one of the things I'm mentioning in the post02:28
kfogelthumper: you can't see this, right?  http://blog.launchpad.net/?p=135602:42
* thumper looks02:46
thumpernope02:46
thumpermwhudson: ping03:27
mwhudsonthumper: pong03:27
thumpermwhudson: what version of pygments are we using ATM?03:28
mwhudsonthumper: good question03:28
thumpermwhudson: easy to find out?03:28
thumpermwhudson: is it an egg?03:28
mwhudsonno, i think it's from the package03:28
mwhudsoncould/should switch to an egg though03:29
mwhudsonthumper: seems like 0.9-203:29
thumperhmm...03:30
thumperbug 39240603:30
lifelessmwhudson: why should?03:30
mupBug #392406: Update Pygments library to version 1.1 or later <codebrowse> <Launchpad Bazaar Integration:Triaged> <https://launchpad.net/bugs/392406>03:30
mwhudsonlifeless: easier for us to upgrade, more in line with what we do for everything else03:30
lifelessmwhudson: I don't really like what we do for everything else ;)03:31
lifelessits odd for us, a large chunk of an OS vendor, to ignore the OS for our products.03:31
mwhudsonlifeless: i am having a little difficultly composing a civil reply here03:33
lifeless:>03:33
lifelesstroll:1 burdened-programmer:0 ?03:33
mwhudsoni guess03:33
lifelessI wasn't intending to troll, but I realise picking one package isn't a good way to change the systemic approach03:33
mwhudsonright03:34
lifelessso - sorry; and please get back to what you're doing03:34
mwhudsonthanks!03:34
thumpermwhudson: bug 153779 still an issue?03:38
* thumper wanders off, back UK morning time03:39
mwhudsonthumper: not really i guess03:45
* mwhudson closes it03:45
* mwhudson EODS03:46
mwhudsonnot the most productive day, oh well03:46
lifelessheh03:46
lifelessI have had similar03:46
lifelessslow progress03:46
mwhudsonmaybe staging will be magically up to date when i start tomorrow03:46
wgrantIt does seem particularly strange to me that a product from an OS vendor does not use a custom archive of dependencies, when a large part of the product is designed to produce custom archives.03:57
adeuringgood morning08:27
wgrantWhen are the buildds expected back?09:37
wgrantThis is the longest queue I have ever seen.09:37
noodles775I haven't heard. bigjools ^^09:37
bigjoolsI don't know09:38
* noodles775 tries to find out.09:38
wgrant12 hours is way beyond "absolutely ridiculous" territory.09:38
bigjoolsFWIW the PPA service only has 4 builders, the others are a bonus09:38
bigjools(per arch)09:38
wgrantI thought it had only three.09:39
bigjoolsyou could be right09:39
bigjoolswe're getting some more but they will be reserved for daily builds09:39
wgrantHmmm.09:39
adiroibanhenninge: Hi. do you have time to land my last branch review or should I ask the OCR ? https://code.edge.launchpad.net/~adiroiban/launchpad/bug-509252-take-2/+merge/1948410:45
henningeadiroiban: oh sorry, forgot about that. Sure. Does it have a commit message?10:46
henningeadiroiban: ah, yes it does ;)10:46
adiroibanhenninge: yes. and I have updated it10:47
henningeadiroiban: ok, landing it now.10:48
adiroibanhenninge: if you have time, can you also QA the fix for this bug https://bugs.edge.launchpad.net/rosetta/+bug/127171 ?10:49
mupBug #127171: Rosetta experts not allowed to "Change translators" <qa-needstesting> <Launchpad Translations:Fix Committed by adiroiban> <https://launchpad.net/bugs/127171>10:49
adiroibanI don't have rosetta admin rights10:49
adiroibanso I don't know how to test it on edge10:49
henningeadiroiban: we can probably make you an admin on staging10:49
adiroibanhenninge: that is also OK10:50
henningedanilo__, danilos: ^^?10:50
henningeadiroiban: as a reminder: Please don't push to the branch anymore now that it is landing.10:50
adiroibanhenninge: ok10:51
daniloshenninge, /me is thinking... are there any privacy concerns? is rosetta-admins subteam of any generic team?10:51
adiroibandanilos: I just need that to test that fix10:53
adiroibanif some of you can qa it10:53
adiroibanit's ok for me10:53
daniloshenninge, adiroiban: I've looked and it seems it's ok, I'll just do it and let Adi QA this :)10:55
danilosadiroiban, haha, there's also "Poor Adi" account, nice name for what I assume is a testing privilege-less account :)10:55
danilosadiroiban, anyway, you should be in rosetta-admins on staging now10:55
adiroibandanilos: yep :)10:55
danilosadiroiban, fwiw, staging will be refreshed sometime on weekend, so that's until when you've got the chance :)10:56
adiroibandanilos: ok. well I will try to qa it now as I will not have time on friday weekend10:57
danilosadiroiban, cool, thanks10:57
adiroibanand I don't want to fix it when pqm is closed10:58
deryckMorning, all.11:02
adiroibandanilos: can you please target this bug for the next release https://bugs.edge.launchpad.net/rosetta/+bug/46201311:03
mupBug #462013: Don't ask me to import obsolete templates <qa-ok> <trivial> <ui> <Launchpad Translations:Fix Committed by adiroiban> <https://launchpad.net/bugs/462013>11:03
adiroibanI think this is requred to me marked as „fix-released”11:03
gmbderyck: Do you have access to run a SELECT on staging?11:10
deryckgmb, I do, yeah.11:10
gmbderyck: Could you run http://pastebin.ubuntu.com/383617/ and paste me the output?11:10
* gmb neatly sidesteps a) the LOSAs being sprinty and b) the fact that staging is 10 revs behind where he needs it to be.11:11
deryckgmb, sure.  doing now....11:11
gmbThanks.11:11
adiroibangmb: when you have time, can you please try a new landing for https://code.edge.launchpad.net/~adiroiban/launchpad/bug-522188/+merge/19395 ? Thanks!11:12
gmbadiroiban: Sure, it's already on my list for today :).11:13
adiroiban:)11:13
deryckintellectronica, https://code.edge.launchpad.net/~deryck/launchpad/max-heat-by-target-511382/+merge/1999811:28
intellectronicaderyck: as it happens i already read through it and ran it working on the patch for updating the column, so r=me :)11:29
deryckintellectronica, excellent!  easy-peasy. Thanks! :-)11:29
intellectronicaderyck: actually, scratch that. did you want me to review just the db patch, or the rest of the code?11:29
deryckintellectronica, the code11:30
intellectronicaif the latter then i need to go through it more thoroughly11:30
intellectronicaok11:30
deryckintellectronica, yeah, the code.  stub and BjornT are on the patch.11:30
intellectronicaderyck: i'm wondering, isn't there a more general getOrCreate sort of thing for DSPs? this can't be the first time we do this in the code11:34
intellectronicaand even in your test you repeat the code twice11:34
intellectronicaehrm, not in the test, in the distro model object11:35
intellectronicamaybe bigjools will know if there's something like that11:36
deryckintellectronica, hmmmm, let me look at the diff....  and I greped for something and didn't find anything, and I was also following what is already there for another bit of code.11:36
deryckintellectronica, ah, I see what you mean, to get the DB version of the class.11:36
bigjoolswhu what?11:36
intellectronicaderyck: yes11:36
intellectronicabigjools: is there a standard getOrCreate thingster for DistributionSourcePackageInDatabase ?11:37
bigjoolsI have no idea, Gavin wrote it!11:37
intellectronicaa getOrCreateDistributionSourcePackageInDatabase :)11:37
bigjoolsI'd be looking for it11:37
deryckintellectronica, I copied the bits from what I assume Gavin had before, but I can refactor into a nicer getInDB method.11:38
deryckI did think about that and never went back.11:38
intellectronicaderyck: if there isn't anything already in existence i'm fine with just filing a bug for doing it later (if you don't want to spend time on it now)11:39
intellectronicaderyck: why do we need a set on distroseries if all it does it delegate it to the distro?11:40
intellectronicaisn't it better to just not implement it?11:40
deryckintellectronica, but in the distro series bug view you'll have to get at the max_bug_heat for that, right?  I thought it made using max_bug_heat less tricky if everything implemented it.11:41
intellectronicaderyck: yes, it makes sense to have a getter, but does it make sense to have a setter11:42
intellectronicasince we don't currently have any code that relies on it, i think it might be better to throw a NotImplementedError or something11:43
deryckintellectronica, ah, right.  Yeah, I went back and forth on that.  I don't know that it makes sense.  You would never set on the series, but there's no harm in doing so.11:43
deryckintellectronica, and given past bugs I fixed I have a nervousness about NotImplementedError :-)11:43
intellectronicaderyck: in fact, i'm not even sure we need a setter anywhere. as i just discussed with stub, it looks like we'll update this column from a trigger.11:43
intellectronicathough i guess at the very least it's useful for testing11:44
deryckintellectronica, yeah, that was my reaction.  without a setter, from the test's pov it's just a useless attribute.11:44
intellectronicaderyck: so, i rather not set if it's done through delegation, because it can be confusing and lead to subtle errors. if you throw an error then whoever is trying to write to the attribute will become aware of the way it's implemented and do the right thing11:45
deryckintellectronica, good point.  I'll do that then.11:46
intellectronicacool11:46
intellectronicaderyck: r=me with that change11:47
deryckintellectronica, excellent.  thanks!11:49
wgrantadiroiban: What is the 'archive name' of a source package?12:37
adiroibanwgrant: repository12:37
adiroibanmaybe the info is already available in API... but it is not obvious how to retreive it12:38
wgrantA SourcePackage doesn't really have an archive.12:39
wgrantExcept potentially primary vs. partner.12:39
adiroibanah...12:40
adiroibanlooks like in the UI is called component12:40
=== matsubara-afk is now known as matsubara
adiroibanhttps://edge.launchpad.net/ubuntu/karmic/+source/kubuntu-docs12:40
wgrantOh. that's something completely different to an archive or a repository.12:40
adiroibanit may be...12:41
adiroibanbut on the Internet12:42
adiroibanit is said that an Ubuntu package is part of main or universe repository12:42
wgrantThey are wrong :)12:43
wgrantThe term has always been component, both inside and outside Launchpad.12:43
adiroibanyep. someone is always wrong on the Internet :)12:43
adiroibanI will update the bug report12:43
* wgrant looks around for somebody who can be bribed into bumping a build score.12:45
=== thekorn_ is now known as thekorn
deryckthree cheers for adeuring killing 4 bugs with one branch.  errr, 4 cheers, then.13:41
adeuringderyck: thanks but the cheers should go to sinzui13:49
deryckhe does know our bugs well now.13:49
sinzuiadeuring: It would have been cruel for me to ask you to fix it without first finding out why the page has been broken for 18 months13:50
leonardrwgrant, i'm looking at https://code.edge.launchpad.net/~wgrant/launchpad/export-das-chroot/+merge/1975913:53
leonardrcan you explain to me what exactly a chroot is? some sort of massive file stored in the librarian?13:53
james_wleonardr: yep, it's a tarball of a filesystem stored on the librarian and used by the buildds when building packages13:54
wgrantleonardr: "massive" being about 60MiB a the moment, but potentially some hundreds of megabytes.13:55
leonardrwgrant: i haven't swapped in all the details yet, but to publish a librarian file on the web service you should export a Bytes field14:10
leonardrthe example i'm working from is IBugAttachment.data14:10
wgrantleonardr: But that's not going to do anything useful like giving me hashes, is it?14:11
leonardrwgrant: it will not give you hashes, but it will let users read the librarian file from within launchpadlib rather than having to fire up httplib2 to grab the librarian url14:12
BjornTleonardr: what's stopping us from exporting ILibraryFileAlias?14:14
leonardrBjornT: well, there's no code in lazr.restful for that because the library file alias is launchpad-specific14:15
leonardrbut there is code in lazr.restful for publishing 'bytes' fields as generic hosted files, and code in launchpad for hooking up LibraryFileAlias to that implementation14:15
wgrantCould one not easily export ILFA with a Bytes field inside it?14:16
leonardrwgrant, what information does your end-user want?14:17
wgrantleonardr: I would like to be able to verify the content of the files, given that I downloaded them over HTTP.14:17
leonardrso like the md5 sum?14:18
wgrantYes.14:19
leonardrok, i filed a bug at one point to publish a json representation of a hosted file resource which would contain that information14:20
wgrantThanks.14:20
leonardrthis would be a change to lazr.restful (and to the launchpad implementation of the hosted file), and publishing ILibraryFileAlias vs publishing IBytes would not change anything14:20
leonardri'm trying to find that bug14:20
leonardri really hate how that bug could have been filed in one of four projects14:21
wgrantHmm.14:21
leonardrwgrant: see if this is what you want14:21
leonardrhttps://bugs.edge.launchpad.net/lazr.restful/+bug/40691214:21
mupBug #406912: Publish a JSON representation of a binary file containing metadata <lazr.restful:New> <Ubuntu:Invalid> <https://launchpad.net/bugs/406912>14:21
wgrantleonardr: Looks good to me.14:23
leonardrok, i'll respond to the review14:23
wgrantThanks.14:23
leonardrwgrant: shall i add on to this bug that you requested the md5 sum of the file?14:23
BjornTleonardr: i just find it odd that in python code we say that an attribute is an ILFA, while we have to export it as Bytes. seems easier and more intuitive if we could export ILFA themselves, with a Bytes field inside them.14:23
wgrantleonardr: And the other hashes, too.14:23
leonardrwgrant: i don't know what other hashes you're refering to14:24
leonardroh, sorry14:24
leonardri thoguht you meant hash as in the data structure14:24
wgrantAh, no, the cryptographic variety, sorry.14:24
leonardrbjornt: i'm doing research to understand what you mean, but lazr.restful wouldn't care what is inside an exported field. if we got some way to export ILFA, then some representation of the ILFA would go into the entry's representation, and that's is14:25
leonardrs/is/it/14:25
jmlgood morning Launchpadders14:27
jmlwgrant, are you visiting New Zealand or perhaps some Pacific island?14:27
jmlwait.14:27
jmlwgrant, I mean, are you visiting Perth or perhaps some island in the Indian Ocean?14:27
wgrantjml: No..14:29
wgrantIt's not yet 1:30.14:29
jmlwgrant, that's still kind of late :)14:31
wgrantjml: I blame Enablement for making PPA builds take too long.14:34
wgrantBut it looks like my build still isn't going to happen for at least another hour, so i might as well sleep.14:35
=== henninge_ is now known as henninge
leonardrBjornT: the basic reason is that lazr.restful is not part of launchpad, so it can't know anything about ILibraryFileAlias14:41
leonardrif this was a huge problem i could put some indirection in place such that ILibraryFileAlias could substitute for Bytes, but i just don't consider it that big a problem14:42
BjornTleonardr: right. i was talking about how we export things in LP.14:43
leonardrme too14:43
leonardrthere's a slight possibility you could get it to work by making ILibraryFileAlias subclass Bytes, but if that doesn't work i don't think it's worth putting in additional work14:45
leonardrs/Bytes/IBytes/14:45
BjornTleonardr: i don't get why we need to hide ILFA. Bytes and ILFA are different things. why does the exported API need to be different from our internal API?14:46
leonardrBjornT: when the web service starts up, the launchpad classes are run through lazr.restful14:48
leonardrwhen lazr.restful encounters a published Bytes field, it thinks "this is a hosted binary file"14:49
leonardrand when the time comes to create a representation of it, lazr.restful performs a lookup, adapting the field to some interface (provided by launchpad)14:49
leonardrwhen lazr.restful encounters a published ILibraryFileAlias field, it thinks "i've never heard of this", and crashes14:50
adiroibanhi. do you know where I cound find some info about how to write API tests? I was looking at this API https://api.launchpad.net/+apidoc/#translation_import_queue_entries , and grepping the whole source tree after „translation_import_queue_entries” did not reveal any test14:51
BjornTleonardr: but ILibraryFileAlias have (or should have) a Bytes field for its content, and ILibraryFileAlias could be exported just like any other interface, like IBug, IPerson, etc. no?14:52
leonardrbjornt: ok, you want to publish ILibraryFieldAlias as an entry, not a field14:53
leonardri didn't understand that14:53
leonardrbjornt: what you describe is technically possible, and it's the launchpad team's decision how to do that, but i think publishing the hosted file as a Bytes field would give a better end-user experience14:57
leonardrbjornt: because right now you have, eg. a bugattachment object that has a 'data' field15:00
leonardrright now you can do something analogous to open(bugtask.attachment.data).read()15:00
leonardrif data was an ILibraryFileAlias then that would be open(bugtask.attachment.data.file).read()15:01
leonardrbut it should still be possible15:01
BjornTleonardr: well, i see now that IBugAttachment is lying. it says that IBugAttachment.data is a Bytes field, yet BugAttachment.data is an ILibraryFileAlias15:02
BjornTwe should make sure that our content objects provide what the claim to provide15:04
=== matsubara is now known as matsubara-lunch
leonardrbjornt: unless ILibraryFileAlias can subclass IBytes i don't see any good way to resolve that problem15:08
leonardrflacoste might be able to help15:08
BjornTleonardr: i think there are a few options, although i'm not sure how feasible they are. one is of course to have our exported API to behave the same way as our internal API15:09
leonardractually, we could define a new interface in lazr.restful and have ILibraryFileAlias subclass that15:09
BjornTleonardr: if we want to have it work like it works today, we could mark ILibraryFileAlias, and say that this is our hosted file, and provide an adapter, or something like that15:10
leonardrbjornt: how about this. define lazr.restful.interfaces.IHostedFile15:10
leonardrhave ILibraryFileAlias subclass IHostedFile15:10
leonardrchange lazr.restful to treat IHostedFile the same way it treats Bytes15:10
leonardruse IHostedFile instead of Bytes in launchpad15:10
BjornTleonardr: what would IHostedFile look like?15:11
leonardrBjornT: i don't think it has to have any fields15:12
leonardrthis is work i don't want to do, but it should work15:12
flacosteleonardr: i think your idea sounds good, the end result is that we want to be able to export a Bytes or LibraryFileAlias field as a hosted file15:13
BjornTleonardr: why do we call export_as_webservice_entry() when exporting interfaces, rather then subclassing marker intefaces?15:13
BjornTleonardr: or a better question: does lazr.restful deal with ILibraryFileAlias, or LibraryFileAlias to make the decision that it's a hosted file?15:19
leonardrBjornT: lazr.restful deals with Bytes to make that decision15:27
leonardrthe only Launchpad-specific code it uses in this process is lib.canonical.launchpad.rest.bytestorage.LibraryBackedByteStorage15:27
leonardrbasically it adapts the Bytes object to IByteStorage15:28
leonardrand launchpad says the IByteStorage adapter to use is LibraryBackedByteStorage15:28
leonardrLBBS hides all the details of LibraryFileAliases, how to create and delete them, etc15:28
leonardrso the change i'm proposing is to replace Bytes with IHostedFile, and adapt IHostedFile to IByteStorage15:29
BjornTleonardr: so it doesn't make sense to say that LibraryFileAlias is an IHostedFile then? if so, it seems like you should mark ILibraryFileAlias, rather then subclassing. but i'm not sure i follow completely15:33
=== matsubara-lunch is now known as matsubara
leonardrbjornt: it makes sense to say that LibraryFileAlias is an IHostedFile, but it doesn't make sense to tell lazr.restful anything about LibraryFileAlias15:53
=== salgado is now known as salgado-lunch
adeuringintellectronica: http://paste.ubuntu.com/383753/ For larger maxet values, you'll get a sort of "gap" after the relative heat 2/316:00
adeuringintellectronica: but i think ii have a proposal for a better function16:00
adeuringjust a second16:00
adeuringintellectronica: http://paste.ubuntu.com/383761/ is at least "smoother"16:07
adeuringintellectronica: I'm getting quite bad error in lib/lp/bugs/tests/../stories/bugtask-searches/xx-listing-basics.txt with your branch16:35
intellectronicaadeuring: damn16:40
intellectronicalet me check16:40
james_wnoodles775: still around?16:41
intellectronicaadeuring: yeah, i'm getting the errors too. looking at them now16:42
rockstarabentley, your superseded comments stuff is made of awesome.16:44
abentleyThanks, but you designed about half of it.16:45
abentleyrockstar, chat?16:45
rockstarabentley, gimme a bit to clear the backlog of reviews here.16:46
abentleyrockstar, sure.16:46
deryckintellectronica, I just merged you branch and will push up now.  If you're fixing errors could you merge our combined branch, or else branch new from me?16:54
deryckintellectronica, since I fixed some of the changed elements.16:54
intellectronicaderyck: sure, i'm remerging your branch16:55
intellectronicawill ping you when i've got a fix for you guys to pull16:55
deryckintellectronica, still running a test.... a minute more and will push up the version from me.  sorry.16:55
deryckI started the test before I saw the scrollback here. ;)16:56
deryckintellectronica, it's up now.  thanks!16:57
intellectronicacheers16:57
jtveod! eod! eod!17:00
intellectronicaderyck: strange, i get ForbiddenAttribute: ('max_bug_heat', <lp.registry.model.distributionsourcepackage.DistributionSourcePackage object at 0xd224aec>)17:07
intellectronicacould it be that wer're missing the zcml to allow the new attribute or something?17:07
deryckintellectronica, hmmm, maybe. and I just realized I haven't done the NotImplementedError you suggested yet.  Sorry.17:08
deryckintellectronica, yeah, I bet DSP needs some zcml.  Sorry.  because I used zopeless layer in test it by passes that check, right?17:08
intellectronicaderyck: yeah, i now see that dsp uses the verbose attribute-by-attribute format for permissions17:09
intellectronicai'll fix it in my branch, since you have to re-merge it anyways17:09
deryckexcellent.  thanks!17:10
intellectronicawhoa, it gets weirder. after fixing the permission problem i get 'InternalError: transaction is read-only'17:12
intellectronicaderyck: too early to tell, but my first guess is that something is not ok with your getOrCreate trick17:15
intellectronicaderyck: i pushed the fixes in my branch, if you want to look17:16
deryckintellectronica, ok, thanks.17:16
intellectronicaderyck: my guess is we're using the read-only store, and then you try to create the dsp but you can't, so somehow we need to force it to use the read-write store17:16
deryckah, right17:17
deryckintellectronica, ok, I'll look closely here in just a sec.  Trying to go over the latest bug about log() with adeuring.17:17
deryckintellectronica, I think we may have to adjust your branch a bit, too.  Sorry to send more work you way again.17:18
adeuringderyck: intellectronicagive me a second for a proposal of a better function17:18
intellectronicaderyck: in what way?17:18
deryckintellectronica, in that the current branch will make anything 2/3 and up 4 flames.  We want something more like:  if max_heat is 100, use intervals 0..40, 41..70, 71..80, 91..10017:19
intellectronicaderyck: i don't think it will. see the documentation17:19
=== beuno is now known as beuno-lunch
* deryck looks again17:20
intellectronicabut as i said this is hardly my expertise. if you have a better transformation go for it17:20
* deryck is playing with numbers in the doc test17:24
deryckintellectronica, I'm actually getting the numbers of how many bugs will get what flames right now.  That might help us decide.17:31
intellectronicacool. nothing like real data :)17:32
adeuringderyck, intellectronica: http://paste.ubuntu.com/ (no lint check ;)17:33
intellectronicaadeuring: are you asking for input? ;)17:33
adeuringintellectronica: well, that's a proposaal for other thresholds in calculate_heat_display()17:34
intellectronicaadeuring: right, but you only gave the url to the pastebin, not to the pasted item17:34
adeuringintellectronica: argh, I am idiot... http://paste.ubuntu.com/383823/17:34
adeuringanother option are median statstics, BTW17:36
intellectronicaadeuring: yes, that looks like a good way to do it17:36
adeuringintellectronica: how/when do we calculate heat values (or max_heat for targets)?17:37
adeuringif this is done by a script, we could set the threshold based on actual data to build thresholds that match the data.17:37
intellectronicaadeuring: i'm creating a trigger that calculates it whenever a heat value is entered for a bug17:38
adeuringso that we have the same number of bugs with 0, 1, 2,... flames17:38
intellectronicayes, that was the original idea, but i think for now it's too complicated17:38
adeuringintellectronica: ah, no, running median statistics tehre is not a good idea, I think.17:38
adeuringintellectronica: I suspect that the function can be simplified....17:41
adeuringlet me see17:41
deryckadeuring, intellectronica -- I'm preparing real numbers with both approaches for what ubuntu and malone (extreme and average) would do in terms of bug distribution in these scales.17:44
adeuringderyck: cool, I'm curious17:44
intellectronicaderyck, adeuring: so i fixed the problem with getting max_heat from a dsp. you can now re-merge my branch17:52
adeuringintellectronica: too late: EC2test is already running...17:52
adeuring...for my branch...17:52
intellectronicaadeuring: well, it will fail17:52
intellectronicaunless you fixed it independently17:53
adeuringyours is already merged?17:53
intellectronicaderyck: ftr, i did that by removing the bit that creates the record in the getter. thinking about it it really shouldn't have been there, since it potentially allows anonymous users to create new records17:53
intellectronicaadeuring: no, that's the branch you presumably branched from. you notified me of errors and i fixed them.17:53
deryckintellectronica, ok.  I'll look closer here in a minute.17:54
adeuringintellectronica: yeah, but that's not what I got reviewed, and so deryck suggested I just go ahead with my branch.17:55
intellectronicaderyck, adeuring: i have to go out for a few hours soon. if you need me to update the scaling, or whatever, drop me a line and i'll do that later in the evening when i'm back. sorry to have to leave you like that.17:55
deryckintellectronica, no worries.  So far, your scaling is looking better.  Though both have issues.  Will email.17:55
adeuringintellectronica: seems that I'm also getting tired. I think I can proposed a more efficient varaint than what is in the pastebin, but i need a bit more time ;)17:56
intellectronicaadeuring: i'm not sure i understand. in my scale-heat branch there was one error that originated in my branch, and another error the originated in deryck's branch. i fixed both and pushed, and thought that if you branched from it you might want to re-merge.17:56
adeuringintellectronica: no, I simply stared from a devel branch...17:57
deryckintellectronica, abel's branch is based on devel.  He's just changing icons, no?17:57
adeuring...started...17:57
adeuringderyck: well, no...17:57
adeuringi changed also the HTML rendering17:57
adeuringand the place where that happens changed17:58
adeuringin intellectronica branch17:58
deryckah, right.17:58
intellectronicaadeuring: right, and i told you that you will have to look at my branch since the place where the rendering is done has changed in my branch17:58
intellectronicaheh17:58
=== salgado-lunch is now known as salgado
deryckheh.17:58
adeuringargh... I missed that....17:58
deryckadeuring, can you just kill your run. and we can merge your branch into intellectronica's.  Or vice versa.  And one of you can fix conflicts.  yes/no?  Likey? :)17:59
intellectronicaadeuring: well, if your branch lands in devel then i guess i'll just have to merge and resolve the conflict. no big deal17:59
adeuringintellectronica: ok, thanks17:59
deryckyeah, that works.17:59
deryckintellectronica is waiting on me anyway :-)17:59
intellectronicai'm not. i'm going to a concert :)18:00
deryckheh18:01
deryckintellectronica, adeuring -- so enjoy the night.  I'll send my numbers in email.  We'll stick with Tom's scale now and re-evaluate later if need be.18:01
deryckwe still end up with large numbers in 0 flames with both.18:01
intellectronicaderyck: but that's desirable, no?18:03
intellectronicamost bugs are not hot18:03
deryckintellectronica, maybe so.18:03
deryckintellectronica, I worry in Ubuntu that the drop off is too quickly.  For average projects it looks nice.  But we can revisit if we need.18:04
deryckbut the package view should be nice, I imagine.18:05
* intellectronica --> pere ubu18:10
=== beuno-lunch is now known as beuno
=== gary_poster is now known as gary-lunch
=== deryck is now known as deryck[lunch]
EdwinGrubbssinzui, bac: Can you give me your feedback on these two mockups on the Upstream connections portlet? It looks like Brad recently moved name/value pairs onto a single line, which has some advantages, although it is inconsistent with other portlets. https://chinstrap.canonical.com/~egrubbs/upstream/19:09
sinzuiEdwinGrubbs: yes, he did, it is inconsistent, but it solved a space and scanning problem. We can do the same if we have the same problem again19:10
sinzuiEdwinGrubbs: I started to read the blueprint a few minutes ago19:10
bacEdwinGrubbs: i didn't know i was introducing an inconsistency but i think it looks better19:11
sinzuibac: I think I mentioned it was odd in my comments in the blueprint19:12
bacsinzui: perhaps.19:12
EdwinGrubbssinzui, bac: I'm not against it, but it is a little confusing on how to move forward with it. It also might be nice to structure projectgroup/project/series so that it is left to right and possibly a little terser similar to the breadcrumbs.19:14
=== gary-lunch is now known as gary_poster
pakihi all19:27
pakii have a problem19:27
pakihow to show changelog in update manager?19:28
pakianyone can help me?19:39
sinzuipaki: update-manager? the desktop application?19:42
pakiyes19:42
sinzuipaki: this channel is for building the launchpad.net website19:43
* sinzui checks his update-manager19:43
pakiuff19:44
pakiwhere could i ask?19:44
=== deryck[lunch] is now known as deryck
pakiis a very stupid thing19:44
sinzuipaki: You are asking why some packages do not include a meaning changelog? The answe most of the time is that packager did not prepare one19:45
* sinzui has looked at the package in launchpad often and not found a log19:45
pakino, i'am a mantainer..can i include my changelog for update-manager?19:46
pakihow to do?19:46
sinzuipaki: okay, now I understand. This is a motu question. let me look for a channel19:47
pakiexcuse my crappy english :S19:47
pakiok, where is this channel?19:48
pakiok, thanks19:48
sinzuipaki: try #ubuntu-motu19:50
pakithanks sinzui :)19:50
EdwinGrubbssinzui, bac: I added two more screenshots. https://chinstrap.canonical.com/~egrubbs/upstream/19:51
sinzuiEdwinGrubbs: I think we need to be clear what is set by the series and what is set by the project. The reason fixing this is so effing hard is that the locations are not obvious, and changing a project changes the information for all series, and all packages that come from that series19:57
sinzuiEdwinGrubbs: have you noticed the new behaviour of the bug app? https://bugs.staging.launchpad.net/gedit20:03
mwhudsongood morning20:04
sinzuiEdwinGrubbs: bac: your blueprint extends scope to allow jumping to alternate services. I am not sure we want to do this, but it is worthy of discussion. Conversely, we (the registry team) may want to put a small barrier like the bugs app on Answers and Blueprints to make communities explicitly set the app locations20:06
EdwinGrubbssinzui: I had not seen the new behavior of the bug app before. That seems relevant to the involvement portlet but unrelated to the upstream connections portlet that I'm working on now.20:23
sinzuiWell, no, it is not20:23
pakione question20:26
pakihow can update my pot file to launchpad?20:26
pakiexcuse me..20:26
pakiupload my pot file..20:26
sinzuiEdwinGrubbs: Our goals are to capture this information for the Ubuntu community, and they need to see it. I agree that Answers and blueprints are separate issues. But by locking the first page (/gedit does have bugs, but they cannot be viewed until someone clearly states how bugs should be used)20:26
sinzuiEdwinGrubbs: Translations is already locked, but code is not20:27
pakiin documentation i don't find a button to upload pot file20:27
pakiexplained in documentation but i don't find a button to upload pot file20:28
sinzuiEdwinGrubbs: and per your portlet, we do not have to allow links to alternate services for users, that does not help ubuntu. What we need to do is callout information that Ubuntu needs to know20:28
sinzuipaki: I think you want to use the #launchpad channel, which has many translations users tending it20:29
pakiok thanks20:31
pakibye20:31
sinzuiEdwinGrubbs: I think the project homepage and freshmeat url can stay where they are. Your design hints at a problem in the UI right now. I think we should move the external downloads link to the downlaods portlet.20:32
EdwinGrubbssinzui: as far as the information that Ubuntu needs to know, are you saying that just the Bugs and Source Code links in the involvement portlet should complain when they are not set?20:34
sinzuiEdwinGrubbs: And translations20:34
EdwinGrubbssinzui: I can updated the wiki and mockups and then send it to the mailing list.20:34
sinzuiokay20:35
sinzuiEdwinGrubbs: I really want to move the wiki link and change blueprints to use it when prompting for the blueprint URL. That will have to be on my own time though20:35
EdwinGrubbssinzui: do you have any opinions on these screenshots: https://chinstrap.canonical.com/~egrubbs/upstream/vertical_hierarchy.png    https://chinstrap.canonical.com/~egrubbs/upstream/horizontal_hierarchy.png20:37
sinzuiEdwinGrubbs: I saw those. As I said they do not make it clear what information is on the series, and what is on the project. That information will require different permissions to change...20:38
leonardrgary, flacoste: there's a multiversion test case it's proving difficult to handle. i don't think it's worth it to make it work properly right now (though i have a test case that i can use to let people know that the behavior is the way it is)20:39
sinzuiEdwinGrubbs: ...Setting the project affects all series, and thus affects all packages from that series. Some series produce 10 different packages20:39
leonardrlet me know if you agree20:39
gary_posterleonardr: back soon20:40
leonardrthe test case is this:20:40
leonardri'll use launchpad versions to make it concrete20:40
leonardrwe stop publishing mutator methods as named operations in version 1.020:40
sinzuiEdwinGrubbs: Maybe this will not be an issue if project-reconfiguration via AJAX rocks.20:40
leonardrnamed operations like 'transitionToStatus'20:41
leonardrright now, it is impossible to publish *any* named operation called 'transitionToStatus' in version 1.020:41
sinzuiEdwinGrubbs: I need to get another child from school. I'll be back in a bit20:41
leonardrwe can publish a transitionToStatus in version 2.0, but not in the version with the mutator named operation cutoff20:43
leonardragain, i don't think it's worth it, and if you agree, i can get this branch into review today20:43
gary_posterleonardr, back and reading20:56
gary_posterleonardr: by "2.0" in your description you meant "any version that is not 1.0" right?20:58
bacsinzui: have time for a quick call?  i'm a bit wedged on a testing issue21:10
sinzuibac: sure, I'll get something to drink first21:11
bacok, call when ready21:11
leonardrgary: yes, exactly21:13
gary_posterleonardr: sounds fine21:13
leonardrok, cool21:13
leonardralso ready for call when you are21:13
gary_postercool, will ping (on another call)21:13
=== salgado is now known as salgado-afk
sinzuibac: I am ready21:17
bacsinzui: ok, can you dial?21:17
sinzuicalling, skype say you do no exist21:17
mwhudsonjames_w: around?21:19
mwhudsonwgrant: or you21:20
* mwhudson just wants to ask some questions about building packages21:20
bacsinzui: can you hear me?21:23
sinzuino21:23
baci could21:23
bacstupid skype on phone21:23
james_whi mwhudson21:33
mwhudsonjames_w: hi21:33
james_whow do?21:34
mwhudsonjames_w: i think i just wanted to ask about the mangling that needs to be done to a source package to build a different version21:34
mwhudsonjames_w: pretty good!21:34
mwhudsonalmost entirely unpacked in the new house...21:34
james_wexcellent21:34
james_wthe easiest way to do it is just to add a new changelog entry21:35
james_wlet me see how they do it in Debian21:35
mwhudsonthis is where i get a teeny bit confused by questions like "what is a source package, again"21:35
wgrantmwhudson: Hi.21:35
mwhudsonisn't the version specified in the .changes file?21:36
wgrantThe binary build process cannot see the changes file.21:36
mwhudsonah ok21:36
mwhudsonand 'make' seems to be broken in devel21:36
wgrantThe changes file isn't really part of the package.21:36
mwhudsonError: There is a version conflict.21:39
mwhudsonWe already have: zope.interface 3.4.021:39
mwhudsonanyone seeing this?21:39
james_wmwhudson: the .dsc has the version, taken from the debian/changelog inside the unpacked source package (unless you are nasty)21:39
james_wthe .changes is a description of the upload, so is more about the delta than the state at the end21:39
wgrantmwhudson: I saw that around a month ago. A clean fixed it.21:40
mwhudsonjames_w: ah ok21:41
mwhudsonso mangling the dsc is *an* option, although possibly not a very good one21:42
wgrantIt won't work, will it?21:42
mwhudsonfor https://dev.launchpad.net/BuildBranchToArchive/MultipleSeriesBuilds21:42
wgrantIt won't affect the unpackaged source, so binary builds will ignore it.21:42
wgrants/unpackaged/unpacked21:43
mwhudsonwgrant: make clean not helping :(21:44
wgrantjames_w: Soyuz has no work towards binary rebuilds yet, and supporting them would require quite a few changes on both slave and master.21:47
james_whmm21:47
james_wCelso told me there was some work towards it21:47
wgrantThis is the same infrastructure that is needed for native backports.21:47
james_wit might have been very early steps though21:47
james_wwgrant: happen to know of any package in Debian with +b1 currently? ;-)21:48
wgrantjames_w: No. Why?21:48
james_wjust want to examine one21:48
wgrantI'm pretty sure they just magically somehow gain the extra changelog entry at the start of the build.21:49
wgrantIt's added on the buildd side.21:49
james_wyeah, that's what I think too21:49
james_wjust want to confirm before asserting it to be true21:50
wgrantExamining sbuild 0.60.0 might work.21:50
wgrantI would, but my connection really sucks at the moment.21:50
james_whttp://packages.debian.org/sid/i386/libfacile-ocaml-dev/download21:51
james_wmwhudson: so yeah, we just tell the buildd to write a new changelog stanza as it starts the build21:51
james_was I said, not exactly elegant, but does buy us several other useful features21:52
wgrantPresenting this in the UI is going to be interesting.21:52
mwhudsonjames_w: oh ok21:52
wgrantBut it is a very useful feature.21:52
wgrantAnd probably a bit less evil than mutating the source.21:52
james_wyes21:53
james_weasier to implement21:53
wgrantjames_w: Do you know how distro people feel about binNMUs?21:53
james_wbut it still leaves us without no-change rebuilds and native backports21:53
wgrantI've heard some negativity around that area before.21:53
james_wit's mixed I think21:53
wgrantDoes it?21:53
james_wdone right I think it would be accepted21:54
james_wwgrant: you are saying we could use the same infrastructure to do them?21:55
wgrantjames_w: Native backports and multi-series recipe builds without source changes seem to be the same concept.21:55
mwhudsonjames_w: for the "What happens when we open a new development release, there will be zero recipes associated with that series for people to choose from." question21:56
mwhudsoni think that "build for all supported series" should be a distinct option from (currently) build for "lucid, karmic, jaunty and hardy"21:56
=== matsubara is now known as matsubara-afk
mwhudsonand uh intreprid i guess21:57
wgrantI'm not sure we do.21:57
wgrantThere's probably no point doing daily builds for the first little while in each series.21:58
james_wmwhudson: yeah, the question may be moot, but we need to consider what happens when we open a new seris21:58
james_wwgrant: quite possibly21:58
mwhudsonwgrant: well, maybe, but that's not actually in too much opposition to what i said21:59
james_wwgrant: I was going to say that it doesn't fit no-change rebuilds as it rebuilds the source package, so it may not be no-change, but that's the status quo anyway21:59
mwhudsonmaybe not "all supported series" but "some default set of series"21:59
wgrantjames_w: It doesn't really rebuild the source package; it unpacks it, adds a changelog entry, and builds a binary from it.22:00
james_wwgrant: sorry, status quo in Ubuntu22:01
james_wapt-get source; dch -i; debuild -s; dput22:01
wgrantjames_w: Oh, you meant the status quote may not be no-change.22:01
wgrantTrue.22:01
james_wyes22:02
james_wwe hope it is no substantive change22:03
wgrantBut autofoo or blah blah.22:03
james_wwgrant: but regardless of the implementation we will be able to do no-change rebuilds server side with build from branch :-)22:06
wgrantTrue.22:07
mwhudsonoops, i probably didn't want to delete download-cache in my trunk directory...22:11
EdwinGrubbsmaxb: ping23:16
mwhudsonhm23:54
mwhudsonthumper: does 'make' work in devel for you?23:54

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