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

thumpermwhudson: I'm back now.  When do you want to talk about recipies?01:10
mwhudsonthumper: i haven't really started anything after my lunch break so now ish is good01:10
mwhudsonthumper: gimme a minute to fetch a drink01:11
thumpermwhudson: ok, let me grab a glass of water and I'll call you01:11
mwhudsonthumper: ready now01:13
dhillon-v10mwhudson: ping :)01:18
=== jamesh_ is now known as jamesh
mwhudsonhttp://people.canonical.com/~michaeln/bfb/build_now_overlay_v2-expanded.png01:52
* thumper off to take advantage of sunshine, back later03:29
=== mwhudson_ is now known as mwhudson
noodles775G'day all07:32
noodles775lifeless: Hi, will you have a chance to look at bug 509370 solution before I get it reviewed and landed?07:37
mupBug #509370: PPA access granted for XXXX mail does not identify the PPA. <email> <ppa> <trivial> <Soyuz:In Progress by michael.nelson> <https://launchpad.net/bugs/509370>07:37
noodles775s/at/at the07:37
NCommander_hey noodles77507:39
wgrantMorning.07:39
lifelessnoodles775: I'm sure you understood the issue, so whatever you've done should be fine07:41
lifelessnoodles775: if you have some reason to think it won't be, I can make time to look at it.07:41
NCommander_noodles775: do you know if there are any Soyuz specs to implement binary rebuilds?07:42
noodles775lifeless: ok, I was slightly confused about it, as it is a private PPA that the subscriber doesn't actually have access to, but I've just added the actual PPA name (as opposed to the displayname) and included the description in the email. Thanks.07:44
noodles775NCommander_: hi, it seems to be currently without priority - mentioned at https://dev.launchpad.net/Soyuz/WantedFeatureList with the relevant bug.07:46
NCommander_noodles775: I was hoping there were some design docs or spec on the topic07:47
noodles775Evening wgrant.07:47
lifelessnoodles775: the key thing I'd want to see is the ppa: url07:49
lifelessnoodles775: which is perhaps what you mean by 'actual PPA name'07:49
lifelessnoodles775: the problem being that many many people have a PPA called 'releases' :)07:49
noodles775lifeless: by 'actual PPA name' I mean the IArchive.name property which is what is used in the URL. Including the URL has a few issues, the main being that the user *doesn't* have access to view the URL.07:50
noodles775(and won't even when they are subscribed - they can download software, but not see the details of uploads etc.07:51
lifelessnoodles775: I don't understand the issue: the URL is what they add to add-apt-repository etc - its the unique key for the thing.07:52
lifelessnoodles775: I don't know what IArchive.name property /means/07:52
noodles775lifeless: ah, the apt url - right, I thought you meant the LP url.07:52
lifelessnoodles775: but here is my test: If I can be confused because someone else has a similarly named PPA, then the bug won't be fixed.07:52
wgrantThe URL, or ppa:wgrant/ppa?07:53
=== mup_ is now known as mup
noodles775wgrant: right - As long as owner ? ppa name is there it's sufficient to uniquely identify it. Currently the email includes the name, but not the owner (instead displaying info about the person who subscribed you, which won't always be the same)07:56
lifelesswgrant: so, ppa:wgrant/ppa, and https://launchpad.net/~wgrant/+archive/ppa - either would be fine for me07:56
noodles775Neither of those are actually accessible, but yes, the unique info is contained there.07:57
lifelessand I don't want to pin noodles775 down to doing something that doesn't make sense, which is why I'm trying to keep it high level - that both the owner and url element, not the visible description, need to be visible.07:57
lifelessnoodles775: great, I shall be happy then07:57
noodles775Yep, that's great. Thanks.07:58
adeuringgood morning08:46
maxbWhat is the ettiquette for submitting branches to devel, which you know will cause a conflict in the stable->db-devel automerge?08:49
maxb(It's just an unfortunately clashing import change)08:50
thumpermaxb: propose for devel, and land08:58
thumpermaxb: once in devel, merge into db-devel08:58
thumpermaxb: when the automatic merge from stable -> db-devel happens, it is a no-op08:58
mrevellHi.08:58
thumperhi mrevell08:58
maxbThanks09:00
maxbgmb: When you have time, could I trouble you for review of the additional unreviewed changeset in https://code.edge.launchpad.net/~maxb/launchpad/use-hashlib/+merge/18894 ? - thanks09:04
lifelesshi jml09:10
jmllifeless, hi09:14
jmljelmer, hi09:20
gmbmaxb: Sure, I'll look now. Sorry, I forgot all about that yesterday09:43
gmbmaxb: Looks good.09:45
gmbmaxb: Want me to ec2  it ?09:45
maxbExcellent ... yes please09:45
gmbIt's on its way.09:46
jelmerjml, 'morning10:41
nigelbany launchpad devs around? I have a feature request and want to know if it is technically feasible before requesting10:42
nigelbRecently, we've been having quite a large amount of spam, is it possible to revert all the changes made by a user in this case?10:42
nigelbnow, we are manually having to change it.  That comes to modifying around 40 or so bugs10:43
=== matsubara-afk is now known as matsubara
jelmernigelb: The required information is present to do something like that, but I have no hard it would be to implement.10:52
jelmernigelb: please file a wishlist bug against Launchpad10:52
nigelbjelmer, so shall I log a bug and hope for the best that it would be tried at some point?10:52
nigelboh, great. thanks10:53
wgrantjelmer: It's not necessarily all present, sadly.10:53
wgrantThe activity log isn't perfect.10:53
wgrantAlthough it's much better than it was a few months ago.10:53
nigel_nbjelmer, bug 520413 opened in LP :)11:43
mupBug #520413: All changes by user must be revertable <Launchpad Bugs:New> <https://launchpad.net/bugs/520413>11:43
jelmernigel_nb, thanks :-)11:44
nigel_nbcould you please set that to wishlist?11:45
sinzuinigel_nb: we do not use wishlist (we should remove that status). We use the feature tag and usually mark it as low. Low mean it will be fixed opportunistically, but the feature tag means we need a blueprint for it12:41
nigel_nbsinzui, and the blueprint would be made at your end ?12:42
nigel_nbi.e., anything nothing more I can do?12:42
sinzuinigel_nb: not necessarily us. anyone may, and any feature should have one.12:43
nigel_nbsinzui, so can I write (or get help from some one to write) one and add a feature tag?12:43
sinzuinigel_nb: We only commit to working on high priority work. The priority is determined by our goals, oops, and some bugs. contributors can and do work on low priority features to improve launchpad for their community.12:44
sinzuinigel_nb: I added that tag before you asked to have it set to wishlist :)12:45
nigel_nbsinzui, well, I'm not a dev at all.  But I can create a feature spec if someone wants to work on it :)12:46
nigel_nbI'm afraid this *could* potentially be high priority later on12:46
sinzuinigel_nb: I think we really want to understand what needs to be accomplished and why. A spec implies we know that. We may want to change ACLs and that will make the suggested solution moot.12:48
nigel_nbsinzui, um, what are ACLs?12:48
sinzuiaccess control list. If the the untrusted user could not make the change, there would have been no request to rollback the data.12:49
nigel_nbsinzui, ah, well that makes sense too!  Either ways, it has become insane the last week rolling back changes12:52
wgrantI think it's just been a particularly bad week.12:56
wgrantMost weeks have no idiots. The last week had two.12:56
=== mrevell is now known as mrevell-lunch
nigel_nbTrue.  But it would be nice to have a fail-safe12:57
sinzuijpds: ping13:33
=== mrevell-lunch is now known as mrevell
jpdssinzui: Pong.13:36
sinzuijpds: my test for your api changes never enters the loop: http://pastebin.ubuntu.com/373951/ can you hit me with a clue-by-four so that I can test  the api changes?13:38
wgrantIs that the broken unauthenticated launchpad.View adapter problem?13:40
thekornI think so13:40
sinzuiyes it is13:41
sinzuithank you everyone.13:41
thekornbug 51576113:41
mupBug #515761: Anonymous API access to some collections returns nothing <qa-ok> <Launchpad Foundations:Fix Committed by jamalta> <https://launchpad.net/bugs/515761>13:41
sinzui...I should not have cargo-culted my anonymous test for this test13:41
wgrantYou should have.13:41
wgrantIt exposed another bug.13:41
intellectronicathekorn: thanks for the work on the iperson_ihasbugs branch. the branch looks in good shape now. set the commit message and i'll land it for you13:46
thekornintellectronica, super, thanks. will set this message in a bit13:47
jpdssinzui: I just mirror = ubuntu.getMirrorByName(name="releases.ubuntu.com") personally.13:52
sinzuijpds: yes that works, but as wgrant and thekorn rightly noted, there is another bug. I tested it as anonymous and I could not get the mirrors. I am looking for the bad interface13:54
jpdsGroovy.13:54
wgrantThere is no launchpad.View for mirrors.13:54
wgrantThere needs to be, for other reasons, which will fix this.13:54
thekornintellectronica, I added a commit message13:57
intellectronicathekorn: lovely, thanks.13:58
=== salgado is now known as salgado-lunch
=== matsubara is now known as matsubara-lunch
maxbWhat's PQM up to, and is lp:~maxb/launchpad/use-hashlib coming up soon?15:32
bigjoolsmaxb: should have gone through now, the queue is empty15:49
maxbSomething's eaten my branch :-(15:51
maxbI have a test results email confirming ec2 success15:51
=== deryck is now known as deryck[lunch]
maxbWell, if it's not in devel and its not in PQM, could someone resubmit it directly to PQM for me?15:59
maxblp:~maxb/launchpad/use-hashlib  -> devel15:59
maxb"[r=gmb][ui=none] Switch from using sha and md5 to hashlib. Also use hashlib.sha256 instead of the python-apt implementation."15:59
=== matsubara-lunch is now known as matsubara
=== salgado-lunch is now known as salgado
bigjoolsit's in testfix mode :/16:19
thekornallenap, thanks for your comments ;)16:58
=== deryck[lunch] is now known as deryck
thekornallenap, ...and your patch16:59
allenapthekorn: Cool, no worries. Thank you for doing the branch :)17:00
thekornallenap, I promise, future branches will have smaller diffs :)17:01
allenapthekorn: It grew naturally, so no complaints here.17:02
thekornyeah that's exactly the problem here17:03
thekornallenap, when I move get_related_bugtasks_search_params() to bugs.model.bugtask should this IllegalRelatedBugTasksParams exception stay in registry.interfaces.person or can this exception be moved somewhere to bugs.*17:15
leonardrflacoste: do you have time for a quick pre-impl chat?17:15
leonardrthe bug is https://bugs.edge.launchpad.net/lazr.restful/+bug/52054217:16
mupBug #520542: confusing latest_version_uri_prefix should be merged into active_versions <lazr.restful:New> <https://launchpad.net/bugs/520542>17:16
leonardrbasically, people who encounter latest_version_uri_prefix are confused17:16
leonardrmy plan is to not spell that our separately, just have it be the last version mentioned in active_versions17:17
leonardrthe underlying implementation would be the same17:17
leonardrthe only downside i can think of is it might be confusing to always have 'devel' or 'trunk' at the end of the list, and to slip versions into the list near the end instead of adding them at the end17:17
leonardrbut i think on balance it's better without latest_version_uri_prefix17:18
leonardrlet mek now if you agree17:18
flacosteleonardr: agreed17:20
allenapthekorn: Good question. lp.bugs.interfaces.bugtask seems likely, but I'll check.17:23
allenapthekorn: Yeah, that's the place.17:24
thekornokidoki,moving it there, thanks17:25
kfogeljml: thanks for the quick re-review17:25
salgadoderyck, around? is bug 520584 known?17:49
mupBug #520584: Person (assignee) picker is broken on +filebug <Launchpad Bugs:New> <https://launchpad.net/bugs/520584>17:49
derycksalgado, yes.  I thought there was a bug for that already.17:50
salgadoI couldn't find one17:50
* deryck looks17:51
deryckvery aggravating.  I know I've seen a bug for this before.17:55
jtvI'm tired of writing one-liner stub methods in my tests.  Unless something *simple* already exists, I'm introducing a helper class FakeMethod that gives you a stub for any function or method and counts the invocations.17:56
derycksalgado, found it.  and marked yours accordingly.17:59
derycksalgado, thanks for the heads up, though.17:59
salgadothanks deryck17:59
mrevellnight18:03
jmlg'night all.18:07
thekornallenap, still around?18:20
kfogeljml: g'night18:25
kfogelderyck: any idea what's going on here?  http://paste.ubuntu.com/374133/   (i.e., why are there instances of "\n\t" in the submit message that PQM is quoting back to me, when they're not in the original?)  I don't see why we'd be in testfix mode...18:26
* deryck looks18:26
deryckkfogel, we must be in testfix.  Checking builders now...18:27
deryckkfogel, db_lp is in testfix18:27
kfogelderyck: "db_lp" means db-devel or something else?18:28
deryckkfogel, yeah, the db-devel builder.18:28
kfogelderyck: hm, but this submission is to devel18:28
deryckkfogel, yeah, but if either buildbot is broken we go into testfix mode.18:29
kfogelderyck: oh, I didn't know that.  Okay, thanks.18:30
maxbIs anyone working on un-testfixing?18:30
deryckI seem to recall something about db devel in email this morning.18:31
=== Pilky_ is now known as Pilky
=== bjf is now known as bjf-afk
mwhudsongood morning19:29
maxbAre we un-testfixed now? If so, please could someone do a PQM submit for me?19:35
mwhudsonbarry: ping?20:11
barrymwhudson: pong20:11
mwhudsonbarry: do you know anything about getting the magic gdbinit file to work with python2.5?20:11
mwhudsonmaxb: yes, untestfixed it seems20:12
barrymwhudson: i wasn't aware that it didn't work, but it's been a long time since i tried.20:12
maxb<maxb> lp:~maxb/launchpad/use-hashlib  -> devel20:13
maxb<maxb> "[r=gmb][ui=none] Switch from using sha and md5 to hashlib. Also use hashlib.sha256 instead of the python-apt implementation."20:13
maxbThat got ec2tested OK but apparently dropped on the floor presumably due to testfix20:13
barrymwhudson: which gdbinit file are you using?20:13
mwhudsoni think it'll be ok with a non-debug build, but too much seems to get inlined into PyEval_EvalFrameEx in the release build20:13
mwhudsonman gdb's command language sucks20:13
bdmurraywhere do the losa's hide out?20:14
mwhudsonbdmurray: here, among other places20:14
=== matsubara is now known as matsubara-afk
=== salgado is now known as salgado-afk
mwhudsonbarry: i'm using the one from the release2.5-maint branch currently20:15
bdmurrayI'd like a Launchpad account suspended.20:15
mwhudsonaaaah20:16
mwhudsongdb has an embedded python interpreter now20:17
barryBOGGLE20:17
mwhudsonOH MAN20:18
mwhudsonACTUAL PROGRAMMING WOO20:18
maxbSorry to nag, but as my message is vanishing up in the scrollback: Would someone be able to do a PQM resubmit for me?20:22
mwhudsonmaxb: i guess i'm not very trusting, but i'd like confirmation that it passed ec2 from somewhere20:22
mwhudsondid gmb try to land it?20:23
maxbquite understandable. Shall I forward the ec2test email to mwh@canonical?20:23
mwhudsonmaxb: that'd be good20:24
* mwhudson grrs at gdb's python thing20:24
mwhudsonclose but no cigar20:24
maxbsent20:27
mwhudsonok, got20:28
mwhudsonmaxb: in pqm20:39
maxb\o/20:39
=== jtv is now known as jtv-afk
wgrantjelmer: I notice that your sync-source.py fix tests for lzma tarballs, which are not supported by dak or LP.21:40
EdwinGrubbsmaxb: ping22:36
maxbEdwinGrubbs: pong22:37
EdwinGrubbsmaxb: I saw your email. I wasn't sure if that was something you were planning to work on, or if I should plan on doing it.22:38
maxbThis is about PIL 1.1.7, right?22:38
EdwinGrubbsmaxb: right22:39
maxbThe relevant change to debian/control is a mere 10 characters or so. I figured I'd leave it to you, as you can better explain why we need 1.1.722:40
EdwinGrubbsok, cool22:40
* mwhudson lunches23:41
maxbHrm23:42
maxblaunchpad-database-dependencies is no longer installable on lucid, because postgresql-8.3-slony1 has been dropped23:42
wgrantOops.23:42
wgrantRebuild rebuild rebuild?23:42
wgrantOr convince stub to fix 8.4..23:43
wgrantmaxb: Do you have any ideas on the lucid python-setuptools incompatibility?23:43
maxbI've not looked at the slony1 source package, but adapting it to build for multiple postgresql versions doesn't sound all that fun23:43
maxbwgrant: I've not examined it - I've literally just upgraded my desktop to lucid now, so I've been focussing on karmic+2.6 up til now23:44
wgrantmaxb: Ah, right.23:44
wgrantWell, you'll probably need to download python-setuptools and python-pkg-resources to their karmic versions, because of the migration to distribute.23:44
wgrantI must file a bug on that.23:45
maxbOh, the setuptools guys are starting to think about fixing the bug that stops us using pytz source#23:45
maxbI need to get back to them about that, actually23:45
wgrantAh, good.23:45
* maxb wonders what the lp buildbot is up to23:50
maxb6 more revs in devel over stable seems high23:51
wgrantIt should be just a few minutes off merging.23:52
wgrant(I'd guess < 10)23:53
maxbI wonder if slony1 is actually needed for launchpad development environments23:55
* maxb ponders the idea of a slony1pg83 renamed source package that only builds the postgresql-8.3-slony1 binary (slony1-bin and slony1-doc unbuilt and coming from the lucid archive)23:57

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