/srv/irclogs.ubuntu.com/2010/05/27/#launchpad-dev.txt

thumperI have a feeling that this branch is going to be large02:15
lifelessthis==?02:15
thumperthis == the branch that adds subscribed_by to BranchSubscription02:15
* thumper wants to delete the branch-notifications test02:59
thumperhowever not just yet02:59
poolielifeless, feed-pqm would be nicer in gtk i think, where you can directly point to the thing you want03:16
poolieis this what you were heading for?03:16
lifelesspoolie: not so much03:16
lifelesslptools has a clickable list already03:16
lifelessif you're heading in that direction I *highly* encourage you to build on that03:17
=== _thumper_ is now known as thumper
henningeHello! ;)08:11
henningeCan packaging links be removed again?08:12
wgranthenninge: there are at least delete links on https://launchpad.net/ubuntu/+source/SOMEPACKAGE08:40
wgrantI'm not sure about the other end.08:40
wgrantnoodles775: Morning.08:48
noodles775Hi wgrant :)08:48
wgrantnoodles775: Could you please re-EC2 lp:~wgrant/launchpad/bug-558905-get-archive? It failed a week or so ago now in the MailmanLayer breakage, and I forgot to deal with it.08:49
noodles775wgrant: sure. Sending now.08:49
wgrantThanks.08:49
=== almaisan-away is now known as al-maisan
henningewgrant: thanks, the relevant method I was looking for is in PackagingUtil.08:56
wgranthenninge: Ah, code-wise, I see. Sorry.08:58
henningewgrant: no, no, don't be. I found the code through the view which I found through your link ... ;-)08:58
henningedidn't know about PackaginUtil before08:59
mrevellMorning09:07
=== mrevell is now known as mrevell-lunch
=== mrevell-lunch is now known as mrevell
wgrantHave we lost half the test suite again?12:57
wgrantThat was a sub-3-hour EC2 run.12:57
=== matsubara-afk is now known as matsubara
deryckwgrant, windmill tests are not run in ec2 now.13:18
deryckand something else.  mailman tests maybe?13:19
maxbI don't think the mailman ones ever were (recently)13:20
wgrantMailmanLayer hasn't ever been run, except for a few hours two weeks ago.13:21
wgrantBut the Windmill thing is a good point.13:21
krkhanhas anyone here communicated with a locally running launchpad over rest?13:23
wgrantkrkhan: yes. Drop the 'api.' from the domain you're using.13:23
wgrantedge.launchpad.net -> launchpad.dev13:23
wgrantapi.edge.launchpad.net -> api.launchpad.dev13:23
krkhanwgrant: thanks, let me try13:23
krkhanwgrant: worked like a charm :-) thanks again13:25
wgrantkrkhan: Excellent.13:26
=== Ursinha-afk is now known as Ursinha
=== cody-somerville_ is now known as cody-somerville
BjornTmatsubara: ping14:50
matsubaraBjornT, hi14:50
BjornTmatsubara: i'm trying to get the oops-tools running locally. i've followed the instructions in the readme.txt, but when i go to /oops, i get ProgrammingError: relation "oops_oops" does not exist14:51
matsubaraBjornT, did you run bin/django syncdb?14:52
BjornTmatsubara: yes. the output looked a bit odd, though. let me get you a paste...14:53
jelmer_Are there any plans to switch to python 2.6 in the near future ? I vaguely recall seeing something but don't remember where.14:54
matsubarajelmer_, gary_poster is working on that14:55
gary_posterjelmer_, matsubara, yeah, at least in a supervisory capacity.  here are the plans14:55
BjornTmatsubara: nm. i guess i should do what the output tells me to do (although, it said to use manage.py, which doesn't exist)14:56
gary_posterWell, don't know how much detail you want :-)14:56
BjornTmatsubara: fwiw, the output was this: http://paste.ubuntu.com/440408/14:56
gary_posterThe summary is that we plan to switch to Python 2.6 and Lucid for 10.06, the release after this one.14:56
matsubaraBjornT, when it says manage.py use bin/django instead.14:56
gary_posterjelmer/jelmer_: ^^^14:56
matsubaraBjornT, bin/django is a buildout wrapper for manager.py14:57
jelmer_gary_poster: ah - thanks!14:57
matsubaraBjornT, right, now you just bin/django migrate and then you should be good14:57
gary_posternp :-)14:57
BjornTmatsubara: now, how do i actually use it? :)14:59
matsubaraBjornT, do you think the README.txt could be improved? Would you add the missing steps to it and provide a diff?15:00
matsubaraBjornT, make run15:00
BjornTmatsubara: yes, i plan to add what's missing, if anything (e.g. the migrate part)15:00
BjornTmatsubara: right, it's running. and then?15:00
matsubaraBjornT, you probably want to add some sample data to it, so `make load_sample_data` first and then `make run`15:00
BjornTmatsubara: well, i want to load the oopses in /var/tmp/lperr15:01
matsubaraBjornT, then you can point your browser to localhost:8000/oops15:01
matsubaraBjornT, if you know the OOPS id, you can enter it in the search oops field, click the button and it should give you the OOPS page15:04
matsubaraBjornT, or do you want a summary of all oopses in /var/tmp/lperr?15:05
BjornTmatsubara: i do know the oops id. it doesn't find any oops with that id15:05
BjornTmatsubara: getting a summary would be useful as well15:06
matsubaraBjornT, can you make sure /var/tmp/lperr is in src/oopstools/settings.py ?15:06
matsubaraBjornT, oh nm15:07
matsubarait probably is and I know why you're not finding those oopses15:07
matsubaras/you're/oops-tools/15:07
matsubaraBjornT, can you try bin/update_db and see if loads the oopses you want?15:09
BjornTmatsubara: tried that already: Loaded 0 OOPS into the database.15:10
matsubaraBjornT, what are the dirs you have in /var/tmp/lperr?15:12
BjornTmatsubara: 2010-05-19 and 2010-05-2615:12
matsubaraBjornT, can you try this patch: https://pastebin.canonical.com/32723/ and then bin/update_db again (also check that you do /var/tmp/lperr in settings.py, it should be but it doesn't hurt to double check)?15:16
adiroibanany idea why I get this error „ We only know about GET, HEAD, and POST” when trying to save an entry using launchpadlib ? the +apidoc page for that entry specify that it support PATCH method and entry.lp_save() is using PATCH for updating that entry.15:19
BjornTmatsubara: ok, that work (after changing datetime.datetime to datetime.date)15:20
matsubaraBjornT, cool15:25
matsubaraBjornT, to generate the summary you can: bin/analyse_error_reports -pX --html=<path-to-html-summary> --text=<path-to-text-summary> --from=2010-05-18 --db-summary15:27
matsubaraBjornT, I'm not sure the summary/ view would work for X prefixed oopses15:28
=== matsubara is now known as matsubara-lunch
gmbsinzui, Can I get a preemptive RC for https://code.edge.launchpad.net/~gmb/launchpad/stored-proc-for-bug-heat-bug-582195/+merge/26064 please? It has been db-approved by BjornT and stub16:17
gmbsinzui, It's part one of a two-part fix for bug heat calculation taking too damn long.16:17
* sinzui knows your pain16:18
sinzuigmb, you have my rc16:19
gmbsinzui, Thank you.16:19
UrsinhaChex, gary_poster, rockstar, bigjools, danilos, sinzui: production meeting on #launchpad-meeting @ Freenode in 12 minutes16:48
gary_posterthank you16:48
danilosUrsinha, can you move it for 17 mins please?16:48
Ursinha17 mins?16:49
danilosUrsinha, oh, it doesn't work like that? ok then, in 11 minutes16:49
Ursinhadanilos, if everyone else agree with that, sure16:49
danilosUrsinha, no, no, j/k16:49
Ursinhadanilos, .... ok _boss_, sure16:50
Ursinha:P16:50
=== matsubara-lunch is now known as matsubara
adiroibanhenninge, danilos : Hi. Any idea why shortlist is used here http://paste.ubuntu.com/440458/?17:02
danilosadiroiban, because it's a smarter thing to use than to use list() which may produce gobbles of information17:03
adiroibandanilos: but if the list is bigger than 300 elements17:04
danilosadiroiban, I am not sure why are we not returning a resultset though, but I'd have to read more code for that17:04
adiroibanI get a warning17:04
=== salgado is now known as salgado-lunch
danilosadiroiban, yes, it will traceback17:04
adiroibanis that ok ?17:04
danilosadiroiban, that's the idea17:04
lifelessedge is on a go-slow17:04
lifelessanyone else noticed ?17:04
lifelessnot timing out, but it must be damn close to it17:04
adiroibandanilos: I have create 4000 templates and using the API, I only get a warning (no traceback) and the batch of 300 templates is fast in the API17:05
adiroibandanilos: +templates from the browser is taking 5 minutes to render them17:05
danilosadiroiban, +templates in the browser is broken, one of the reasons is your introduction of using non-cached data with getPOTMsgSets().count(), but it has other problems as well17:06
danilosadiroiban, anyway, I have a meeting now, sorry17:06
adiroibandanilos: np. thanks!17:06
=== beuno is now known as beuno-lunch
=== gary_poster is now known as gary-lunch
danilosadiroiban, anyway, have you checked that somebody without permissions can't really touch potemplates?17:41
adiroibandanilos: yes17:41
danilosadiroiban, cool, then I'll have another look at your branch tomorrow; 300 seems a reasonable limit for productseries, but I'll have to check why we are not returning a resultset there instead17:42
adiroibandanilos: the limit in shorlist is only „informative”17:43
adiroibanthe full result is returned as a list17:43
adiroibanand a warning is issued if the limit is exceded17:43
adiroibanalso the +templates page is not using getTranslationTemplates, but rather callling getUtilit(IPOTemplateSet).getSubset() directly17:45
adiroibanright now the API is the only user for the getTranslationTemplates method17:45
danilosadiroiban, ah, right17:47
danilosadiroiban, we should probably use a hardlimit there then17:47
danilosadiroiban, that's very weird17:48
danilosadiroiban, all of that should be fixed17:48
adiroibandanilos: getTranslationTemplates is defined in many places, but only used in lp/registry/templates/distroseries-packaging.pt17:51
adiroibanand in that template, it is used in a very strange way17:51
adiroibanassigned to a variable in a tal:define17:51
adiroibanbut that variable is not used later17:51
danilosadiroiban, it's used in a distroseries copy, and we should probably move the +templates page to it as well17:52
sinzuiadiroiban, I discovered that two months ago...17:53
adiroibandanilos: ok. maybe we can add pagination to +templates and by doing so avoid the timeout17:53
danilosadiroiban, I'd like to try to make it work without pagination first: with pagination, we'll have to hack sorting in and I'd rather avoid that17:53
sinzuiadiroiban, danilos. Edwin is working on a model fix that will remove my need for the method17:53
adiroibansinzui. ok. thanks!17:53
adiroibandanilos: regarding the soft/hard limit on shortlist. Do we realy need it?17:56
danilosadiroiban, yes, because we know we'll time out if we try to process that many, so we can either: 1) ensure we won't time out with unlimited number of entries 2) introduce batching 3) introduce a hard-limit so we see in the OOPSes that our users have started needing it17:57
=== salgado-lunch is now known as salgado
danilos1) is very hard to do, but we can probably raise the limit to a few thousand (thus satisfying ubuntu as well)17:57
adiroibandanilos: in distroseries, the softlimit is 200017:58
adiroibanon for productseries and sourcepackage it is set to 30017:58
danilosadiroiban, but that page has never worked, and it has gotten only worse17:58
danilosadiroiban, all of those sound like sane limits to me (and they should be hardlimits at that)17:58
adiroibandanilos: ok. but from what I think, with or without the hard limit, the page will crash anyways if it tries to get a big number of templates17:59
danilosadiroiban, yes, and with one approach we'll be using a lot of memory and cpu cycles on our servers (thus exposing a possible, albeit unlikely, DoS attack vector), and in another we'll be giving an error to the user instantly18:02
danilosadiroiban, when it comes to "choose one", I know which one I am choosing :)18:03
adiroiban:)18:03
adiroibandanilos: but, then we need to change the implementation of shortlist18:03
adiroibandanilos: sorry. nope. just use softlimit instead of hardlimit18:04
sinzuidanilos, adiroiban: We are adding a sum of templates to distrosourcepackage that is cached so that we will not need to do a lot of work for getTranslationTemplates()18:05
=== deryck is now known as deryck[lunch]
mrevellRighto, friends, I'm offski18:07
=== al-maisan is now known as almaisan-away
jelmer_are there any known issues with oopses not being cleaned up properly when tearing down tests?18:25
=== almaisan-away is now known as al-maisan
matthew_how do you remove launchpad from your computer (rocketfuel) ?18:27
jelmer_matthew_: I don't think we have anything that does that at the moment.18:28
=== gary-lunch is now known as gary_poster
matthew_do you know what dependencies that it installes then i can remove them manually?18:29
=== beuno-lunch is now known as beuno
jelmer_matthew_: for the packages at least you can deinstall launchpad-dependencies, launchpad-developer-dependencies, launchpad-database-dependencies and launchpad-soyuz-dependencies18:37
matthew_and is all the downloaded items just in /home/matthew/launchpad?18:38
matthew_ok removed thoese and then did a sudo apt-get autoremove18:39
matthew_*those18:39
jelmer_matthew_: in ~/launchpad and it installs some scripts (all starting with rocketfuel- I believe) in /usr/local/bin/, adds some entries to /etc/hosts and adds some files to /etc/apache2 IIRC18:40
matthew_thanks18:45
Ursinharockstar, bug 58646718:47
mupBug #586467: upgrade_branches script failing  <Launchpad Bazaar Integration:New> <https://launchpad.net/bugs/586467>18:47
rockstarUrsinha, ta18:48
=== deryck[lunch] is now known as deryck
=== al-maisan is now known as almaisan-away
marsabentley, ping, did using xvfb and 'make check' work for you as a substitute for VNC?  Your mail does say whether switching got you back up and running again or not.20:03
abentleymars, yes it works for me.20:06
marsabentley, great.  May have been a known issue in Karmic.  IIRC they fixed xvfb in August or September.20:07
lifelessmars: what was xvfb doing ?20:08
abentleymars, I am running Lucid.20:08
marsabentley, didn't you say you switched to VNC to get around a bug in Xvfb/Karmic that was stopping you?20:08
abentleymars, when I was using Karmic, I switched to VNC.  When I switched to Lucid, I continued using VNC.20:09
marslifeless, don't remember, just that a fix landed in the repos literally minutes before one of our developers needed it :)20:10
marsabentley, ah, ok.20:10
gmbsinzui, Can I get a pre-emptive RC for this https://code.edge.launchpad.net/~gmb/launchpad/hook-up-stored-proc-boom-bug-585803/+merge/26191 please? It's the second part of project-make-bug-heat-not-suck.20:11
sinzuigmb, you have my rc. since buildbot threw a wobbly you may still need to use it with PQM submit if ec2 is faster than buildbot20:17
gmbsinzui, Yeah, that's why I asked now :)20:17
gmbThanks20:17
brycehI got this error message, am I interpreting it correctly as trying to say, "Your branch could not be landed because PQM is turned off now"?21:11
bryceh[]21:11
brycehCommand failed!21:11
brycehAll lines of log output:'Commit message [[r=intellectronica][ui=none][bug=373508,\n\t455203] Flesh out docs21:11
brycehfor interfaces/bugs.py and fix various\n\tspelling errors.] does not match commit_re21:11
bryceh[(?is)^\\s*\\[testfix\\]\\s*\\[(?:release-critical=[^\\]]+|rs?=[^\\]]+)\\]]'21:11
thumperbryceh: no, it needs a testfix21:18
thumperwhich means there is a broken test on trunk21:18
thumper(or buildbot had some other error)21:18
bdmurrayWhere can I find the value of config.launchpad.default_batch_size ?21:19
brycehthumper, hum odd, I got a "Test results: SUCCESS" at the same time as the above21:20
brycehand ec2 test has passed ok on this branch previously to that21:21
thumperbryceh: ec2 isn't running all the tests right now21:21
brycehoh21:21
brycehI assumed it was running all the tests...  Is there a way to run all the tests locally or something so I can see what the problem is?  The above error doesn't really give me enough clue as to what's wrong.21:22
brycehmaybe I should just wait and try again after PQM is back on21:22
=== matsubara is now known as matsubara-afk
=== salgado is now known as salgado-afk
jelmerbryceh: hi22:42
brycehhi jelmer22:43
jelmerbryceh: I saw you did some work on Launchpad support for gtg22:43
brycehyes22:43
jelmerbryceh: What were you working on exactly? I'm looking for a way to integrate my assigned bugs into gtg22:43
brycehjelmer, two things22:43
=== ubuntujenkins_ is now known as ubuntujenkins
jelmermwhudson, thumper: hi23:27
thumperjelmer: hi23:28
jelmerthumper: What's the policy for landing things on ~launchpad-pqm/{dulwich,bzr-git,bzr-svn,subvertpy}/devel exactly?23:28
thumperjelmer: relatively lax given that we need to adjust the revnos in the launchpad branch23:29
thumperjelmer: if you have something that will work, land it23:29
thumperjelmer: the justification comes in the LP branch change23:29
jelmerthumper: ok - thanks!23:29
jelmerthumper: I'll try and land as part of the releases processes, that way we get the benefit of the QA that happens before a bzr-* release.23:30

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