krkhanin testing layer, after creating a bugattachment with factory.makeBugAttachment, i get 404 whenever i try to access the attachment's url00:02
krkhanthat is, doing just a = self.factory.makeBugAttachment and then accessing a.data.read() results in a lookuperror00:07
kb9vqfAnyone have recommendations for an OpenID server that will work with a local installation of Launchpad?00:15
=== matsubara-afk is now known as matsubara
thumperkrkhan: that is because the attachment is stored in the librarian00:38
thumperkrkhan: which is a different process00:38
thumperkrkhan: so the local transaction needs to be committed for the librarian to see it00:38
thumperkb9vqf: I just use the local test openid server00:38
* thumper afk for lunch00:40
spmconfigs question: we have a script: scripts/process-accepted.py which is oops'ing. and then failing to write said oops as it's going to the wrong place; which is then aborting the shell script it's running from. Q, how do I find which config stanza I need to fix for that specific script *only*? I've found the master it's inheriting from ftpmaster/launchpad-lazr.conf but that controls a bunch of semi-related things I don't want to break. ???00:54
wgrantrockstar: Hi.00:54
krkhanthumper: how do i commit the local transaction?00:55
krkhanthumper: nvm. transaction.commit. thanks a bunch for the quick help :)00:56
rockstarwgrant, can you take a look at this: https://code.edge.launchpad.net/~rockstar/launchpad/fix-buildds/+merge/2699000:57
wgrantrockstar: Good description, good patch.01:00
rockstarwgrant, yeah, I awoke to find shit all over me this morning.01:00
wgrantSorry -- I would have submitted a branch myself, but exams are eating lots of time at the moment.01:01
wgrantAh, so it was recipe builds that killed everything over the weekend?01:01
rockstarwgrant, yea, but it's not your job..  :)01:01
rockstarwgrant, yup, we're assuming this is why.01:01
pooliehi all01:01
rockstarwgrant, so do you feel like this fixes the bug you reported (I'll test as well, but you know this better than I do)01:01
wgrantrockstar: I believe it does. I'll fire buildd-manager up and get a build through.01:03
wgrantAlthough fixing this right now may not be a good idea.01:03
wgrantSince it means the master will start crashing instead of the slave.01:03
rockstarwgrant, yeah, I tried to do it locally, and my launchpad chroot shit the bed twice, so I tried to test on dogfood, but I don't really have access to it.01:03
wgrantBug 58711301:03
mupBug #587113: BuildBase result handling broken <Soyuz:Triaged by michael.nelson> <https://launchpad.net/bugs/587113>01:03
wgrantOK, I'll test locally.01:04
pooliethat's vivid :)01:04
rockstarwgrant, why will the master start crashing?01:04
wgrantrockstar: The big model refactor last master broke the master's handling of completed recipe builds.01:04
wgrant(the bug I referenced above)01:04
wgrantBut we can at least test that the slave works.01:04
rockstarwgrant, wait, so this isn't going to be able to be cherry picked to production?01:14
wgrantrockstar: Unless we get noodles' master fix in quickly too, no.01:14
wgrantBut that fix should't be too big.01:15
rockstarwgrant, hm, I'll chat with noodles about that tomorrow then.01:15
wgrantFor now, can't you just disable the recipe UI?01:16
rockstarwgrant, that's what we did.01:16
wgrantAh, good.01:16
rockstarwgrant, but we would like to be testing recipes through edge this whole cycle.01:16
rockstarWe've found that dogfood is a terrible place to be testing recipes01:16
wgrantHmmm, production appservers need a CP before that can happen.01:17
wgrantBug #58711001:17
mupBug #587110: Need fmt:link for SourcePackageRecipeBuild <recipe> <ui> <Launchpad Bazaar Integration:Triaged> <https://launchpad.net/bugs/587110>01:17
wgrantWhat's wrong with dogfood?01:18
wgrantBesides being dogfood?01:18
rockstarwgrant, can you compile this into a mail to the list.01:18
rockstarwgrant, we ran recipes through dogfood and it went well.  As soon as they got into edge, they just became a pain.01:18
wgrantrockstar: Well, all of this broke during week 3.01:19
wgrantEvery one of these bugs.01:19
wgrantrockstar: As expected, the recipe build works fine with your branch merged, and then the master has a fit.01:24
wgrantSo your branch is good.01:24
rockstarwgrant, okay, so there are three needed cherry picks before we can re-enable recipes, correct?01:24
wgrant1) Your buildd branch to all of the buildds -> lamont01:25
wgrant2) noodles master fix, which is yet to be written -> cesium01:25
wgrant3) The SPRB fmt:link -> production appservers01:25
rockstarwgrant, okay, I'm compiling a this in an email to the list then.01:26
rockstarwgrant, no, thank you for the help.01:26
wgrantI was able to get them going 1.5 weeks ago with those three changes.01:27
wgrantAnd it probably hasn't broken much since then.01:27
wgrant(although my master fix was a hack)01:27
wgrantrockstar: Is it known that the distroseries are in an arbitrary order on +request-builds?01:41
rockstarwgrant, for recipes?01:41
wgrantrockstar: Yes.01:41
rockstarwgrant, I think abentley had some issue with ordering by distroseries, but I don't remember what it was.01:42
rockstarBut there was method to the madness.01:42
wgrantI really wish dogfood was less omgslow.01:43
rockstarwgrant, and that the buildmanager was actually running?01:49
wgrantrockstar: Well, I don't care much about that.01:50
wgrantI was hoping to log in and check if the distroseries were in any obvious order.01:50
wgrantBut it turns out you can't log in either.01:50
wgrantAnd editing a recipe would probably time out, too...01:50
rockstarwgrant, yeah, dogfood is Soyuz's baby...01:51
jmljames_w, you are maybe asleep now, but http://mumak.net:8080/job/txrestfulclient/02:17
spmhey jml, how's the big M doing?02:27
thumperspm: what? McDonalds?02:36
rockstarwgrant, technically bug# 587110 doesn't need to be cherry picked, because the UI is not enabled on production.02:36
mupBug #587110: Need fmt:link for SourcePackageRecipeBuild <recipe> <ui> <Launchpad Bazaar Integration:Triaged> <https://launchpad.net/bugs/587110>02:36
spmthumper: heh, close. Tho I believe Montréal is more accurate. ??02:36
=== matsubara is now known as matsubara-afk
* rockstar shudders at McDonald's...02:46
wgrantrockstar: Erm, well.03:00
wgrant /builders will start crashing if an SPRB gets dispatched.03:00
rockstarwgrant, ?03:00
wgrantSo the index views themselves are disabled too?03:00
rockstarwgrant, well, the views are available, but in the words of abentley, "if you url hack, you get to keep both pieces"03:01
rockstarThere are no links to those views though.03:01
wgrantOK. Maybe we should cherrypick a special fmt:link that doesn't actually generate a link.03:02
rockstarwgrant, sigh, I hadn't thought about that.03:02
rockstarwgrant, please reply to the my email about that.03:02
wgrantrockstar: Your third point has a hanging "This does"03:03
rockstarwgrant, gimme a break, it's 2000 here.  :)03:03
wgrantWas there anything important omitted?03:04
wgrantReplied, anyway.03:07
=== ajmitch_ is now known as ajmitch
jmlspm, Montreal is great04:14
lifelessjml: are you on leave now ?05:54
noodles775Hi ppl07:28
adeuringgood morning08:15
=== lifeless_ is now known as lifeless
=== almaisan-away is now known as al-maisan
=== gary_poster_ is now known as gary_poster
bigjoolshello world09:09
bigjoolshi guys09:11
=== mwhudson_ is now known as mwhudson
bigjoolshow's the service station mwhudson? :)09:11
mwhudsonbigjools: terrible09:11
mwhudsonthe fire alarm went off at about 01:30 last night09:11
bigjoolsI looked on google maps and it's in. the. middle. of. nowhere!09:12
mwhudsonyou thought dolce la hulpe was isolated?09:12
wgrantIt was in a forest!09:12
wgrantIs this even worse?09:12
mwhudsonwgrant: it's a travel hotel, with a mcdonalds next door and about 5km from anywhere09:14
cody-somervilleThere is a bloody handprint above my bed.09:18
wgrant... that is concerning.09:18
bigjoolscody-somerville: lol09:51
=== cody-somerville_ is now known as cody-somerville
deryckMorning, folks.10:58
=== cody-somerville_ is now known as cody-somerville
noodles775When using codehosting locally, I'm trying to push a branch for a user I've created, but it's trying to authenticate as sabdfl... can someone point me in the right direction? http://pastebin.ubuntu.com/446558/11:03
wgrantnoodles775: You have an old ~/.ssh/config.11:20
wgrantrf-setup used to put an extra stanza in there.11:20
noodles775Thanks wgrant.11:21
bilalakhtarHi there, people. Can anyone tell me what does this line mean in https://dev.launchpad.net/Running/Schroot "Don't rm -rf /chroot/karmic-lp, because this will descend into /chroot/karmic-lp/home and delete your actual home directory!"?11:36
bilalakhtarSorry I am a newbie when i comes to chroots11:36
bilalakhtarsorry have to go11:36
noodles775wgrant: still around? I've got failed SPRecipeBuilds going through properly (ie. log gets appended, etc.). The log shows they're failing with "bzr: ERROR: no such option: --append-version". I assume I need to make sure the buildd has a newer version of bzr-builder?12:15
wgrantnoodles775: Right, you need to steal a new bzr-builder from production.12:16
wgrantOr add the right sources.list entries to the config option.12:16
wgrantEither steal from the prod configs, or download and upload from https://edge.launchpad.net/~launchpad/+archive/bzr-builder-dev12:17
noodles775Yep, doing so now. Thanks.12:17
bilalakhtarHi there, noodles775 is it possible to run lp on a computer already running LAMP? (apache2-mpm-prefork)12:33
bilalakhtarRocketfuel-setup runs apt-get install apache2-mpm-worker. This removes php12:34
noodles775bilalakhtar: I've not tried.12:35
bilalakhtarnoodles775: What apache module does launchpad use? mod-python or cgi?12:36
wgrantIt's mostly proxied.12:38
bilalakhtarwgrant: proxied?12:38
wgrantIt mostly uses mod_proxy.12:38
wgrantApache does not run the actual application.12:38
wgrantprefork should be fine, if you are PHP-inclined.12:38
bilalakhtarwgrant: oh. Thanks. I will modify the line in rocketfuel-setup so that it tries to install prefork, which is actually installled. thanks12:39
=== Ursinha-afk is now known as Ursinha
* bigjools finds the problem why PPAs are not getting deleted and groans13:22
bilalakhtarHi people, I am confused aboout which branch should I download to begin hacking. db-devel or devel? I just want to make a minor change (no db change). Acoording to the wiki, I should work on devel, but it is said "db-devel is default stacking branch". What should I do?13:46
beunobilalakhtar, go for devel13:50
bilalakhtarbeuno: And stack on db-devel?13:50
beunobilalakhtar, yes13:51
bilalakhtarbeuno: then merge with devel?13:51
beunodon't worry about the stacking13:51
beunoit's not important13:51
beunouse devel, propose merging into devel13:51
beunoignore db-devel13:51
bilalakhtarbeuno: Last question: I am not using rocketfuel. I am manually downloading db-devel right now. I installed apache and lp-developer-deps. Now, what should I do about locations.conf?13:51
beunoI think locations.conf is a convenience to push branches13:52
bilalakhtarbeuno: What should I set it to, if I want to take advantage of the "convenience"?13:53
lifelessyou're worrying about a bunch of automatic details13:53
lifelessjust follow the wiki :)13:53
bilalakhtarthanks, beuno and lifeless13:53
bilalakhtarbeuno and lifeless: What is bzr pqm-submit? Is it necessary to run that as well when proposing merge? Or proposing via lp web UI is enough?13:55
lifelessbilalakhtar: just run bzr lp-propose13:55
lifelesspqm-submit is used by folk with commit-to-devel privileges13:56
bilalakhtarlifeless: ahha, a new plugin. Is it the same as proposing via web?13:56
lifelessnot a new plugin, its built in13:56
bilalakhtarlifeless: thanks13:56
* bilalakhtar is bumming around with the code :)13:56
bilalakhtarbeuno: running bzr branch lp:launchpad/devel is stacking on the devel branch. is it fine?14:08
bilalakhtarhehe, most of the people here are lp-ers14:12
bilalakhtarI mean members of ~launchpad team14:12
maxb<bilalakhtar> running bzr branch lp:launchpad/devel is stacking on the devel branch14:21
maxbThat is meaningless/impossible. The act of branching locally can't stack on a remote launchpad branch14:21
bilalakhtarmaxb: I cannot run link-external-sourcecode. The problem is that the sourcecode folder is empty. What to do?14:48
maxbThe sourcecode folder should not be empty if you've successfully completed rocketfuel-setup14:49
bilalakhtarmaxb: I didn't use rocketfuel only.14:50
maxbIf you do not, then you are responsible for reading it through carefully and doing what it would have done14:51
bilalakhtarmaxb: ok, now I will use it. Many probs are coming thanks.14:51
maxbI do not run rocketfuel either14:51
maxbBut I do treat it as a reference document14:51
deryckSo what's the rule for using memcached in tal and testing?14:53
deryckCache and move on, trusting the memached testing itself?14:53
deryckgary_poster, have any thoughts there ^^ ?14:54
gary_posterderyck: yes, a few thoughts.  So, first of all, there's a bug on making testing easier.  I could find it; the gist is that we should be able to verify that memcache is working, and there's an idea on what that spelling might be.14:56
gary_posterThat probably should be pretty easy to do but we haven't gotten around to it yet14:56
deryckok, np.  I do understand.14:57
gary_posterSo, looking into that is an option.  However, meanwhile, depending on what you are doing, there may be some other approaches for testing this second.14:57
gary_posterIn particular, curtis wanted to show that one snippet (with memcache sharing characteristic X) was used for one situation, and another was used in another (with characteristic Y).14:58
gary_posterHe did that with simple comments in the code that actually didn't have too much to do with memcache except by convention.14:59
deryckgary_poster, yeah, I saw Curtis' test but it seemed to be just testing html comment.  but I guess that does ensure that the right caching is in effect.14:59
deryckI guess that would work for me.14:59
deryckgary_poster, and normal page tests are run outside cache, right?  So adding caching has no effect on those tests?15:00
gary_posterOK.  Let me find the bug for improving memcache testing, and then maybe you can add your example there so we can change it when we add the better testing story.  Normal page tests are run outside cache: actually, I think memcached is available and running15:01
adiroibannoodles775: do you have time to discuss an Firefox/HTML-accesskey issue affecting bug 359180?15:01
gary_posterderyck: https://bugs.edge.launchpad.net/launchpad-foundations/+bug/586466 FWIW15:02
mupBug #586466: Testing tal cache states could be easier <Launchpad Foundations:Triaged by stub> <https://launchpad.net/bugs/586466>15:02
noodles775adiroiban: in 0.5hr or so?15:02
noodles775Will you still be around?15:02
adiroibannoodles775: yep. we can also do it tomorrow ...no hurry15:02
adiroibanI just wanted to schedule it15:03
stubpagetests use memcache15:03
deryckgary_poster, you say memcached is available and running.  Does it affect tests then?  i.e. if I fetch a page that is cached, do some work, and then want to verify a cached section, should the test fail?15:03
deryckstub, ^^15:03
noodles775adiroiban: today is good, I'm just finishing something off.15:04
gary_posterderyck: AIUI, yes, it affects tests.  That is certainly the intent.15:04
deryckah, ok.15:04
stubmemcached affects tests. You might need to clear the cache if you need changes to be visible - there is a helper on MemcacheLayer I think to do that easily15:04
deryckthat works perfectly for me and my concerns then.  Thanks, gary_poster and stub.  And many thanks for this work from you guys, too! :-)15:05
gary_postercool :-)15:05
* deryck plans to cache liberally over the next couple months15:06
stubderyck: And use the slave db liberally too.15:07
stubderyck: I'd like to see all bug searches hitting the slave instead of the master - they are pretty heavy weight.15:07
deryckstub, right.  That's a good point.15:07
* deryck makes reminder note to discuss with team15:07
stublib/canonical/launchpad/doc/db-policy.txt goes over how to install a db policy to make the slave a default for a section of code, or just ISlaveStore(Bug).find(...) to be explicit.15:08
bigjoolsdb-policy.txt ROCKS15:08
derycknote made15:09
bigjoolslifeless: is lp-propose a new bzr command?  I don't have it.15:14
jmllifeless, I am.15:17
=== leonardr is now known as leonardr-afk
noodles775adiroiban: ok, ready when you are.15:26
adiroibannoodles775: ok. I start working on keyboard shortcuts for Launchpad translations page15:26
adiroibanand I went for Shift+Alt+KEY15:26
adiroibanas they are also used for HTML Access key15:27
adiroibanand thinking that Web browser will not overlap with them15:27
adiroibanI have implemented them using YUI3 key events and Chromium / Webkit family is fine15:28
adiroibanbut in Firefox, if there is no HTML Access key on the page, Shift+Alt+Key will behave as Alt+Key15:28
noodles775(except for the last bit :) ).15:28
adiroibanand instead of performing the YUI3 event15:28
adiroibanit will pop up a menu15:29
adiroibanas I workaround, I found out that if I add an empty <a accesskey=F> in the page15:29
adiroibanFirefox will not popup the menu15:29
adiroibanand will trigger the YUI3 event15:29
adiroibanand I would like to ask you if this is an acceptable workaround15:30
adiroibanor there are other side effect for this solution15:30
adiroibanor do you have other ideas for implementing it15:30
noodles775adiroiban: could it not be a feature if it was for a skip to content-type link?15:30
adiroibannoodles775: I lost the conversation... what is a feature?15:31
noodles775Have I understood correctly: you're asking if it would be acceptable to add a default access key to all pages (ie. the base template) so that FF won't pop up the window?15:31
adiroibanyes. that what I was asking.15:32
adiroibanbut it will requre an „blind” access key for each keyboard shortcut15:33
noodles775So, couldn't that be combined with providing a "skip-to-content" link with an access key (eg. http://diveintoaccessibility.org/day_11_skipping_over_navigation_links.html )15:33
noodles775OK, I'm not sure what you mean by a blind access key.15:33
adiroibana blind accesskey is just a tag like <a accesskey=KEY/>15:34
adiroibanit will not be a proper anchor15:34
adiroibanas it does not have a name15:34
noodles775But why couldn't it be a proper anchor if it was providing useful functionality (but not displayed) as in the above link?15:35
adiroibanbecause the YUI3 event is doing more than just focusing a link15:36
adiroibanadding an Skip to content accesskey will not solve this problem15:37
adiroibanas we need an accesskey for each YUI3 key event15:38
adiroibanand the accesskey should not have an attached ID or name15:38
adiroibannoodles775: to demonstrate the behaviour, go to https://translations.edge.launchpad.net/ubuntu/lucid/+source/ubiquity/+pots/ubiquity-debconf/de/+translate using Firefox15:40
* noodles775 opens FF15:40
adiroibanand while the focus is on the first translation field (it should be auto focused)15:40
adiroibanpress Shift+Alt+F (if you are using FF in English)15:41
adiroibanif you are repeating these steps in Chromium, the Search field will be focused15:41
adiroibanbut in Firefox it opens the File menu15:41
noodles775(as does simply ALT-F)15:41
noodles775Which is the first translation field? (It didn't seem to auto focus...)15:42
noodles775nm... I wasn't logged in with FF, so no fields :)15:42
adiroibanyou need to log in15:43
noodles775OK, and you say that can be avoided specifically by adding *one* dummy <a accesskey=KEY/> to the template?15:45
adiroibanone dummy for *each* YUI3 event15:45
noodles775ah... OK, now it's clearer.15:45
adiroibando you know other ways for solving this problem15:48
noodles775Nope, but I'm just reading atm.15:48
adiroibanCtrl+Key, Alt+Key and Shift+Ctrl events are already well established in browsers menus15:49
adiroibanand I think that Ctrl+Alt is used by the window managers15:50
=== Pilky_ is now known as Pilky
noodles775adiroiban: ok, another point where I'm confused: why not add a valid anchor with a valid access key for each?15:57
adiroibanHTML accees key actions can only focus or activate (depending on the browser) a link15:58
adiroibanin the translations page we have Shift+Alt+C15:58
bilalakhtarPeople, How bug is the launchpad branch? I have been downloading it from an hour. Its more than 100MB by now15:58
adiroibanwhich triggers a javascript function15:58
noodles775adiroiban: yes, but I was hoping that the YUI code would override that if present?15:59
adiroibanalso, for some access key there is no associated anchor16:00
noodles775But ok, it makes more sense now, and no I don't know of a better option (nor can I find one). If you add the empty ones to your template only I wouldn't see it as an issue. When you've got the MP, please ping me with it, as I'd be keen to play with it a bit.16:00
adiroibannoodles775: another option is overwriting Ctrl+KEY16:01
adiroibanI have tested with Chromium and Firefox16:01
adiroibanand it looks like Ctrl+Key events are first passed to javascript and only if not handled there16:02
adiroibanthey are triggering menu actions16:02
noodles775aha, seems quite standard (at least, gmail keyboard shortcuts use it)16:02
adiroibanyes... but user will no longer have access to their shortcuts16:02
noodles775Is there any drawback to Ctrl+Key? Why weren't they the original option?16:02
adiroibanlike Ctrl+C16:02
* noodles775 reads bug again.16:02
adiroibannoodles775: list of keyboard shortcuts in other web apps http://mashable.com/2007/06/29/keyboard-shortcuts/16:04
adiroibannoodles775: the main reason for using Shift+Alt was to avoid overlapping with other key shortcuts16:05
adiroibanand Ctrl+Key are widely usind in browsers16:06
adiroibanalso afaik IE 7.0 will not allow the overiding of Ctrl+P, Ctrl+O and Ctrl+F .... maybe there are more restrictions16:09
noodles775adiroiban: OK, I'm tending towards keeping the keys you've got and updating just your template so it also works in FF (adding real accesskeys where possible, but blank otherwise). As this is the first time LP has had accesskeys (I can't see any other egs?), it's probably worth a discussion on the dev list... there may be people on there in a better position to make a more permanent recommendation.16:13
gary_posterderyck, sinzui: in a fit of pique, I tried for a fix to bug 586466, which gives a test convenience for memcached bits.  I'm running it through ec2 test now to see if my change has any nasty side effects, but otherwise it seemed to work.16:15
gary_posterOnce I get that to pass, I'm going to ask Curtis to review, for several reasons (making sure I'm doing something reasonable with my lpconfig check, for instance, or if he thinks I should be more defensive).16:15
gary_posterHowever, as a sort of pre-impl or mid-impl or something, would be happy to get any feedback from you two on the change.  Essentially, if you go to http://bazaar.launchpad.net/~gary/launchpad/bug586466/revision/10964 and look at milestone-views.txt, you will see that, if you are in test or dev modes, the memcached integration will now add comments describing the start (and end) of the cache hit.16:15
gary_posterDoes that look good to you two? 16:15
mupBug #586466: Testing tal cache states could be easier <Launchpad Foundations:Triaged by stub> <https://launchpad.net/bugs/586466>16:15
adiroibannoodles775: I think this is the first place to use accesskeys. I will also send an email to dev list. Thanks! I will let you know when the MP is ready16:15
deryckgary_poster, oh, wow.  JFDI FTW! :-)  looking at your rev from above now...16:16
deryckgary_poster, I like that a lot.  That works for my concerns, too.  It's more explict about whether you've hit cache or not, which is what I wanted.16:18
gary_posterGreat, deryck.  Thank you for looking at it.16:19
sinzuigary_poster, deryck: remove the end comment or put them before the call. I personally do not think they are needed...it know what purge does. r=me, as deryck said jfdi16:23
gary_postersinzui: ack, cool, thank you16:23
=== beuno is now known as beuno-lunch
sinzuigary_poster, deryck: My initial landing of cached content was disappointing. I thought my second would also be disappointing, but most of the problems are now gone from the oopses. I will add this tactic to person and teams next16:26
deryckyes, I appreciate your initial work, sinzui.  Makes my use easier.16:27
deryckand I'm on a memcached onslaught now16:28
=== leonardr-afk is now known as leonardr
sinzuitake a look at the milestone bug task listings. I use two different cache times (which required a macro for the content) to handle stable verses unstable information. eg, cache old comments 6 hours, but cache new ones 10 minutes (so losas can delete them)16:31
=== Ursinha is now known as Ursinha-lunch
noodles775abentley: I'll include that change in the branch that I'm landing.16:54
abentleynoodles775, tx16:54
noodles775Night all.17:01
=== beuno-lunch is now known as beuno
=== Ursinha-lunch is now known as Ursinha
adiroibanhi. it looks like LP API (https://launchpad.net/+apidoc/devel.html) is not updated together with edge. Is there a way where I can find the revision of the deployed LP API code19:02
=== krkhan_ is now known as krkhan
=== al-maisan is now known as almaisan-away
jmladiroiban, https://edge.launchpad.net/+apidoc/devel.html ?19:19
adiroibanjml: true. stupid me :)19:20
jmlis there a stable ppa for launchpadlib?19:23
dobeyhey guys19:24
dobeywhat are the metrics for determining what reviews show up on my launchpad.net/~me/+activereviews page?19:25
=== deryck is now known as deryck[lunch]
beunodobey, I think it's "reviews that you can do, or reviews that you have been asked to do"19:27
beunothe "you can do" part inherits from your teams19:27
jelmerunfortunately it doesn't include reviews for teams that you are in19:27
beunobut this is totally out of memory19:27
beunoso that was the plan19:27
dobeyjelmer: didn't it used to?19:27
jelmerdobey: not that I remember, but I have only been using the +activereviews page since bzr switched to using launchpad for reviews.19:28
dobeyooh. no more bundle buggy?19:28
jelmerno, we moved away to launchpad quite a while ago (two years?)19:29
beunojelmer, a bit under 2 years, when we actually made code reviews usable  :)19:30
* beuno gets nostalgic19:30
dobeyit's definitely less than 2 years, becuase i haven't been at canonical that long yet, and the branches i submitted to bzr had to go to bundle buggy :)19:31
jelmertime flies, just realized I've been hacking on bzr for almost 5 years now19:33
marsgary_poster, want to switch the Windmill test suite back on?  https://code.edge.launchpad.net/~mars/launchpad/re-enable-windmill-ec2-suite/+merge/2707719:43
gary_postermars, sure, once LP loads :-P19:43
jmlleonardr, is it deliberate that NamedOperation.__call__ has a variable named "request" that is probably supposed to be a ResponseDefinition instance? (I'm saying that because get_representation_definition is called on it)19:53
=== deryck[lunch] is now known as deryck
leonardrjml: i think it's probably a RequestDefinition that _contains_ a ResponseDefinition19:59
jmlleonardr, RequestDefinition doesn't have a get_representation_definition method though.19:59
jmlleonardr, it has a representation_definition method :)19:59
kb9vqfDoes Launchpad have something like an admin console where projects and users can be deleted by an administrator?20:06
kb9vqfThis is for a local installation of course ;-)20:06
jmlleonardr, oh, never mind. I'm pulling from an old trunk.20:08
jmlleonardr, sorry for the noise.20:08
leonardrjml, np20:09
krkhanfor local launchpad, what service_root should i give to the login_with method of launchpadlib?20:12
jmlleonardr, ok, now I have another question :) how come https://bugs.edge.launchpad.net/wadllib/+bug/274074 is marked as fix released, but https://code.edge.launchpad.net/~leonardr/wadllib/fix-bind-to-anon-collection/+merge/24613 isn't merged into wadllib trunk?20:13
mupBug #274074: Missing total_size on collections returned by named operations <api> <wadllib:Fix Released by leonardr> <https://launchpad.net/bugs/274074>20:13
leonardrjml, most likely the code is there but the branch wasn't marked merged for unknown reasons20:14
leonardri'm checking20:14
jmlleonardr, thank you20:15
jmlleonardr, fwiw, when I merge the branch into trunk, it changes stuff and makes the failing test that triggered james_w's initial report pass20:15
marsjml, btw, I have that patch now for zope.testing.  I can push the branch, see if you think it's sane.20:16
jmlmars, thanks.20:16
marsjml, lp:~mars/zope.testing/fix-subunit-utf8-traceback-reporting20:18
jmlmars, ta. is that related to ec2 land being broken?20:19
marsjml, that is the cause.  It isn't just ec2 land, it causes the test suite to outright forget previous test failures, so it ends up lying.20:20
* jml nods20:20
Ursinhasinzui, hi20:20
sinzuiHi Ursinha20:20
Ursinhasinzui, I see you landed something that was a testfix and a fix for three bugs at the same time, right?20:20
Ursinhar10951 on devel20:20
Ursinhasinzui, problem is tagging script ignores testfixes20:21
sinzuiUrsinha, understood20:21
sinzuiI also had no idea that my testfix would fix a bug I did not know was reported20:21
marsjml, I don't think it is a stellar fix, but it works, and should be forward-compatible.20:21
jmlmars, that branch wasn't made from lp:zope.testing :)20:22
Ursinhasinzui, that was unexpected20:22
marsjml, ?  Says "Created new stacked branch referring to /~ztk-steering-group/zope.testing/trunk"20:22
jmlmars, it'll say that regardless of what you push up.20:23
Ursinhasinzui, I wonder if that could happen too often that would be worth changing the script's behaviour20:24
marsjml, looking at the recent revisions on https://code.edge.launchpad.net/~mars/zope.testing/fix-subunit-utf8-traceback-reporting, it still looks right?  Unless lp:zope.testing uses something different?20:24
jmlmars, the fix itself looks good to me (and damn, I wish I felt confident enough to use unit tests when originally contributing the zope support)20:25
leonardrjml: looks like the branch i thought was lp:wadllib was just a random branch20:25
jmlleonardr, heh20:25
marsjml, sidnei said it was OK ;)20:25
jmlleonardr, the ~launchpad-pqm owned one?20:25
sinzuiUrsinha, My actions were extraordinary. I do not think it will happen often20:25
Ursinhasinzui, right20:26
sinzuiJust knowing is enough for me to ensure they get updated myhand20:26
jmlmars, yeah, there shouldn't be a problem.20:26
leonardrjml: no, it was something in ~leonardr/wadllib20:26
sinzuiby hand20:26
marsjml, and lp:zope.testing points to https://code.edge.launchpad.net/~ztk-steering-group/zope.testing/trunk - still OK I would think20:26
jmlmars, yeah, it's ok, it's just that you can't merge one branch into the other with bzr20:27
jmlmars, which isn't a big deal, because it'll be applied as a text patch in svn anyway :)20:27
marsjml, oh, you mean I didn't use svn!  Right.  Nope, I figured lifeless did it, so it must get through the contributor process somehow20:27
jmlmars, no, I just mean that when I try to merge your branch into my local copy of lp:zope.testing, bzr tells me there's no common ancestors.20:28
leonardrjml, should be fixed now20:28
marsjml, ok, that's weird20:28
jmlmars, lp:zope.testing has changed since I last pulled.20:28
jmlmars, my bad20:28
marsno problem20:29
jmlleonardr, thanks! was the latest wadllib release based on the actual trunk?20:29
leonardrthe latest wadllib release was based on my oddball branch. but since i think i'm the only one who's ever modified wadllib, it should contain all the latest code20:29
jmlleonardr, good to know. Also, do you know if there's a stable PPA?20:30
leonardrjml: not to my knowledge20:31
jmlleonardr, thanks.20:32
kb9vqfDoes Launchpad have something like an admin console where projects and users can be deleted by an administrator?20:35
kb9vqfThis is for a local installation of course ;-)20:35
marskb9vqf, log in as an administrator, visit the project or user page and hit the "Administer Foo" link in the upper-right.  There is no central page for doing so.20:36
kb9vqfOK, thanks20:36
* kb9vqf is still learning...20:36
kb9vqfsecond question: How do I promote a standard user to administrator status?20:37
krkhani have exported a method for Bug in lp.bugs.model.bug, it is accessible from lp.bugs.model.bug.Bug but it doesn't show up in lp_operations of the lazr Entry. any ideas what i'm doing wrong?21:26
krkhanleonardr: ping21:27
leonardrkrkhan, does it show up in /+apidoc/devel.html in your local launchpad?21:27
leonardrhow did you export it? typically exporting happens on the interface level21:29
leonardrif you give me a link to your branch i can take al ook21:29
krkhanleonardr: i exported it this way: http://bazaar.launchpad.net/~inspirated/launchpad/bug-findAttachment/revision/1093721:29
krkhanlet me check apidoc21:29
krkhanleonardr: no. it doesn't show up in apidoc21:30
leonardrok, so it's not published in the wadl, so lazr.restful doesn't really think it's published21:30
krkhanhow do i publish it in wadl?21:31
leonardronce lazr.restful recognizes what you're trying to do, it will show up in the wadl, and launchpadlib and apidoc will pick it up from there21:32
krkhanthat's what i'm stuck at :-( why isn't lazr.restful picking up what i'm trying to do. i've tried reading implementation of other read operations which do show up and i don't see anything different21:34
leonardrit also looks ok to me21:38
marssidnei, ping21:48
sidneimars, aye21:48
marsHi sidnei, just wondering if there is anything I need to do in the way of copyright assignment before or after submitting code to zope.testing?21:49
sidneimars, oh, good one. there's a contributor agreement that you have to sign to get commit access. i could checkin the patch for you though, since i signed the contributor agreement.21:51
marssidnei, well, the patch is in bzr, so I guess you would have to land it for me anyway :)21:52
marssidnei, ok, I'll prep a merge proposal then21:52
leonardrkrkhan, afaict an adapter is being generated for your new method21:54
leonardri put a breakpoint in the lazr.restful egg, in declarations.py#generate_operation_adapter21:55
leonardr   if "Attachments" in method.__name__:21:55
leonardr    import pdb; pdb.set_trace()21:55
leonardryou can try it yourself21:55
leonardrif you don't make any progress i'll come back to it tomorrow21:55
jkakarleonardr: Thanks for looking at my fake-launchpad branch!22:10
jkakarleonardr: Your comments look very sensible.  I need to digest them a bit, but will respond tomorrow.22:10
marssidnei, https://code.edge.launchpad.net/~mars/zope.testing/fix-subunit-utf8-traceback-reporting/+merge/2708622:10
jkakarleonardr: Also, totally agree that this should ideally live in restfulclient and also that FakeResource is doing too much...22:10
jkakarleonardr: I was basically discovering the way the WADL and launchpadlib itself works as I went, so I'm not surprised that someone with clue about them see that the design I've come up with has problems.  Thanks for explaining them, I'm happy to work on fixes based on your suggestions.22:11
marsjml, you may be interested in this: https://code.edge.launchpad.net/~mars/zope.testing/fix-subunit-utf8-traceback-reporting/+merge/2708622:14
krkhanleonardr: indeed, both canBeNominatedFor (example read operation that works) and findAttachments pass through generate_operation_adapter. so lazr.restful is seeing the method too but it still isn't getting exported. anyways, will ask you tomorrow then22:14
marsmorning thumper22:14
sinzuibac, are you really online?22:15
bacsinzui: i am now22:23
sinzuibac: I think there are 4 or more bugs about bug field descriptions and documentation. I think we should fix these this week in a single branch22:24
bacsinzui: sounds good22:25
sinzuiI can find the bugs, but I think it is best we commit to reviewing all the fields so that they are consistent. I think doing this will help us define what we intend to do with the new forms22:26
krkhanleonardr: fwiw, i'm also noticing that changing the method names in IBug interface doesn't affect anything in the apidoc23:28
jmlwhat's bugtask.date_left_new mean?23:51

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