* mwhudson takes a late lunch break01:25
wgranthttps://edge.launchpad.net/ubuntu/karmic/i386/cpp-3.4 reproducibly OOPSing -- what's the traceback on that?03:29
lifeless10509/lib/lp/soyuz/browser/../templates/distroarchseriesbinarypackage-index.pt object at 0xe60c850>, 'isRedirectInhibited')03:32
lifeless  Module zope.traversing.adapters, line 52, in traverse03:32
lifeless   - __traceback_info__: (<zope.browserpage.metaconfigure.SimpleViewClass from /srv/edge.launchpad.net/edge/launchpad-rev-10509/lib/lp/soyuz/browser/../templates/distroarchseriesbinarypackage-index.pt object at 0xe60c850>, 'isRedirectInhibited', [])03:32
lifelessLocationError: (<zope.browserpage.metaconfigure.SimpleViewClass from /srv/edge.launchpad.net/edge/launchpad-rev-10509/lib/lp/soyuz/browser/../templates/distroarchseriesbinarypackage-index.pt object at 0xe60c850>, 'isRedirectInhibited')03:32
wgrantOh, that one.03:33
lifelesswgrant: de nada03:38
lifelesspopping out for a bit03:38
thumperwgrant: know about that?03:50
thumperwgrant: I had that on a team page to add a member03:50
wgrantthumper: General issue with anything that isn't a LaunchpadView.03:52
wgrantI saw discussion about it somewhere.03:52
wgrantIt's known by foundations/registry, at least.03:52
mwhudsonthumper: i think i've unexpectedly fixed https://bugs.edge.launchpad.net/launchpad-code/+bug/34029506:42
mupBug #340295: exceptions.AttributeError: 'NoneType' object has no attribute 'write' <codehosting-ssh> <oops> <Launchpad Bazaar Integration:Triaged> <https://launchpad.net/bugs/340295>06:43
thumpermwhudson: cool. How?07:41
adeuringgood morning08:15
mwhudsonthumper: by fixing https://bugs.launchpad.net/bugs/33515609:46
mupBug #335156: "not allowed to execute..." message sent over ssh stdout channel <Launchpad Bazaar Integration:In Progress by mwhudson> <https://launchpad.net/bugs/335156>09:46
deryckMorning, all.10:02
maxb  File "bootstrap.py", line 151, in <module> PYTHONPATH=ws.find(pkg_resources.Requirement.parse('setuptools')).location)10:04
maxbAttributeError: 'NoneType' object has no attribute 'location'10:04
maxbwgrant: ^^^ Is that the python2.6 distribute intermittent error you have mentioned in the past?10:04
wgrantmaxb: That's one aspect of it, yes.10:05
wgrantIt manifests itself in around three different ways.10:05
wgrantOr none at all.10:05
wgrantDepending on $UNKNOWN.10:05
maxbmm. lovely10:07
wgrantmaxb: Oh, 2.6? It happen(s|ed) with 2.5 sometimes too.10:08
maxbah. never had that happen to me so ar10:16
wgrantI haven't had it happen for a while.10:17
wgrantSo maybe it's fixed now.10:17
maxbyet I've just had it happen for the first time :-/10:17
wgrantBut 2.6?10:18
henningejtv1: while I have you here alive and kicking, can you look at this if it looks sane? Looks almost too simple to me  ...10:37
=== jtv1 is now known as jtv
henningejtv: I moved findProductSeries into Branch10:38
henningejtv: also, I assume that these imports are *not* published elsewhere and set the branch.owner to be the importer.10:39
jtvhenninge: templates don't have the distinction... effectively they're always published.10:40
henningejtv: yes, I was just wondering about that ....10:40
jtvI'd pass True, but purely as a matter of form.10:40
jtvBut yes, that looks pretty much alright...  assuming "tarball" is the string of contents of the tarball.10:42
henningejtv: it is10:42
jtvhenninge: the findProductSeries method isn't currently in IBranch, is it?10:43
henningejtv: yes, I moved it there. I found it odd importing the BranchJobSource here and it really is a method if the branch, isn't it?10:48
henningejtv: I did the same with ...10:48
* henninge looks it up10:48
jtvhenninge: to me it looks more as if it should be a method of ProductSeriesSet, if there is such a beast.10:49
henningejtv: hm, that is the other alternative, I admit10:49
jtvThere is such a beast.10:49
henningejtv:  findHasBranch ?10:50
jtvIt's going to be ugly pigeonholing this under one part of LP when it's clearly a bridge between Code and Translations, with one foot in Registry.10:50
henningefindHasTranslationsBranch and leave out the parameter10:50
henningebetter, Importers has been used too many times10:51
henningefindRelatedBranch ?10:51
jtvhenninge: one part I probably just don't know enough about...  why do you have a seemingly generic function for finding productseries related to a branch, with a parameter saying what behaviour you want?  Is there more than this one use-case that needs to call the same function?10:54
henningejtv: not yet, but this is just following a simple database fk relation, so I found it a generic problem plus a filter.10:55
henningejtv: oh no, the parameter was there for a different reason!10:56
henningejtv: to enable "force translation imports" behaviour.10:56
henningejtv: we need to look at all series for that, not just those that have translation imports enabled.10:57
jtvI still say that's the wrong thing to do though.10:57
henningeand I inverted the meaning of the parameter to make it more general10:57
henningejtv: what is the wrong thing?10:58
jtvI still say it's the wrong thing to let people request this on the productseries and then force it on any productseries that uses the same branch.10:58
henningehm, not sure it really works that way10:59
jtvIt should be either an action in the branch UI (not my favourite way to do it) or it should force imports only on the productseries you request it on.10:59
jtvIt doesn't currently work that way, and I'm saying that's wrong.10:59
henningethat does sound wrong and I was not aware of that.10:59
henningeI thought it did the filtering in code10:59
jtvWe discussed a few weeks back.10:59
jtvThe "force" flag really needs a productseries argument, not a boolean11:00
henningejtv: yes, you are right. i don't remember talking about it, though.11:04
* jtv resists temptation to also claim that henning borrowed money from him on that occasion11:04
jtvIt was weeks ago.11:05
henningejtv: ok, I don't want to fix both things in the same branch. I'll just try to avoid making it harder than necessary.11:06
jtvIIRC a solution could be to make the browser code pass the productseries as an argument when creating the IRosettaUploadJob, have that productseries' id stored in the BranchJob.json_data instead of just the boolean flag, and then if it's present, only upload to the given productseries (regardless of setting)11:06
henningejtv: do we have a bug for that?11:06
jtvI don't think so.11:06
jtvNo wait, I think I would've filed one.11:07
jtvHang on.11:07
henningeyes, I would expect ... ;)11:07
jtvbug 52109511:07
mupBug #521095: One-time import acts on all series for the branch <code-integration> <Launchpad Translations:Triaged> <https://launchpad.net/bugs/521095>11:07
henningejtv: so all we need is a method to get all product series that have imports enabled on a specific branch.11:15
jtvhenninge: yup, that's what I thin11:15
henningejtv: and I agree, it would fit best on the the set.11:15
jtv\o/ then we _all_ agree!  :-)11:16
henningejtv: I don't like to introduce a new term by using "Consumers"11:16
jtvthere's that.11:16
henningejtv: yup, we'll take that.11:17
henningejtv: I'll see if i can fix both bugs after all. I am just wondering about affected tests.11:18
jtvhenninge: if an existing test should break, hopefully all that's needed is to change the import setting on a productseries.11:19
deryckadeuring, hi.  Since you did the patch sniffing work... is Bug #538219 easy to fix?11:37
* adeuring is looking11:38
adeuringderyck: yes, that should be easy to fix.11:39
deryckadeuring, ok.  I'll mark it high and stick on our easy bugs backlog then.11:39
adeuringderyck: sure, it sould be easy to fix it this cycle11:40
deryckadeuring, excellent.  thanks.11:41
wgrantThe quick fix is to change a hack in lp.services.mime which overrides .debdiff to text/plain.11:43
=== matsubara-afk is now known as matsubara
=== daniloff is now known as danilos
=== mrevell is now known as mrevell-lunch
=== salgado changed the topic of #launchpad-dev to: Launchpad Development Channel | Week 2 of 10.03 | PQM is in open | https://dev.launchpad.net/ | Get the code: https://dev.launchpad.net/Getting | On-call review in irc://irc.freenode.net/#launchpad-reviews | Use http://paste.ubuntu.com/ for pastes |
=== jamalta-afk is now known as jamalta
kfogeladeuring: for bug #534736 ("patch attachments portlet should appear above regular attachments portlet on a bug page"), all tests pass without any changes other than the code change itself.  Is this the sort of thing that needs a new test?13:14
mupBug #534736: patches portlet should appear above attachment portlet <trivial> <ui> <Launchpad Bugs:In Progress by kfogel> <https://launchpad.net/bugs/534736>13:14
kfogeladeuring: Also, should I get UI review on an ordering change like this?13:15
kfogelintellectronica: see above questions to adeuring; if you're there and can answer, please do :-).13:16
adeuringkfogel: I think a small addition to an existing page test would be good for this change13:20
adeuringkfogel: and I am not sure if this really requires a UI revoew ;)13:20
kfogeladeuring: thanks.  I'll look for a test.13:20
kfogeladeuring: hmmm.  I'm not sure how to test this.  Usually when testing a portlet, we use find_portlet().  But in this case, it's the order in which two portlets appear that we care about.13:28
adeuringkfogel: something like peint extract_text(browser.contents), then ellipises (?) to skip the unintersting text?13:29
kfogeladeuring: aaaah.  gotcha, that makes sense.13:29
deryckgmb, hi.  Have you seen Bug #538097?13:32
mupBug #538097: +storeblob fails with "500 Internal server error" on production (works on edge) <apport-collected> <lucid> <Launchpad Bugs:Confirmed> <apport (Ubuntu):Invalid by pitti> <https://launchpad.net/bugs/538097>13:32
kfogeladeuring: I'm still a little confused about the difference between doc/ tests and stories/ tests.13:33
deryckgmb, related to the filebug store blob work maybe?13:33
kfogeladeuring: they appear to be the same kind of thing.  What is the distinction?13:33
adeuringkfogel: storeis are intended to, well "tell stories from a user's perspective". They are not necessarily comprehensive tests. But we have quite often tests of details there tooo13:34
kfogeladeuring: oh, so the difference is purely conceptual -- not technical.13:34
adeuringkfogel: mostly. In doc, you can more easily do also things like run tests with another DB user13:35
gmbderyck: Ihadn't seen that. I'll look into it.13:35
deryckgmb, I put a card on the board for you.  Since this is blocking Lucid bug reports we should move it through quickly... if you need help from me, just ping.13:37
gmbderyck: Righto, thanks13:37
gmbderyck: Interestingly, all the OOPSes I'm seeing so far for +storeblob today are of this nature: https://lp-oops.canonical.com/oops/?oopsid=OOPS-1535E148713:52
deryckgmb, looking....13:52
=== henninge_ is now known as henninge
gmbderyck: Security machinery gone maaaaaad?13:53
deryckgmb, interesting.  so apport needs to set referrer header?13:53
gmbderyck: Well, I can't be sure yet, need to test.13:53
gmbBut that's my working hypothesis, yes.13:54
deryckgmb, ok.  and I'm curious too why this is an issue for lpnet but not edge?13:54
gmbderyck: Still investigating whether it actually isn't an issue for edge too.13:55
deryckgmb, ok, cool.  thanks for looking into it.  sorry to sideline your checkwatches work13:56
gmbderyck: No worries. I think I might need to get an up-to-date Lucid vm running to test this properly; I'll get that started and go back to CW until it's ready.13:57
deryckgmb, that works13:57
=== mrevell-lunch is now known as mrevell
intellectronicakfogel: i'm back. do you still need help?14:15
kfogelintellectronica: no, adeuring helped me.  Thanks.14:16
kfogeladeuring: having trouble getting this test text to match.  The new test is: http://paste.ubuntu.com/395636/   And the failure is: http://paste.ubuntu.com/395637/14:25
kfogeladeuring: I'm fiddling with lots of different combinations of whitespace and "..."  but no luck yet.14:25
adeuringkfogel: did you try to start the expected text with "Bug #11"?14:26
mupBug #11: Rosetta says there are untranslated strings, but it isn't <Launchpad Translations:Fix Released by carlos> <https://launchpad.net/bugs/11>14:26
adeuringi mean; instead of the ellipsis14:26
adeuringerm, before the ellipsis14:26
henningekfogel: what he means is: You cannot start with an ellipsis, there must be some test first.14:28
henningeadeuring: :-P14:28
kfogeladeuring, henninge: thank you!14:29
sinzuididrocks: ping14:30
didrockssinzui: hey14:30
sinzuididrocks: gpgkeys are not visible over anon api.14:31
sinzuididrocks: Is this intentional?14:31
didrockssinzui: no, it's not. as jml finished the change and I was able to see them in +apidoc, I don't know what happened14:32
daniloskfogel, that initial text can be as simple as "<..." (i.e. opening < of a <html> tag)14:32
kfogeldanilos: *nod*14:32
danilos(#11 bug mention pinged me :)14:32
kfogelhenninge, adeuring, danilos: test passes now, thanks.14:32
kfogeldanilos: heh14:32
sinzuididrocks: It needs a anon security function and a 5 line test to verify anon users can access it14:32
didrockssinzui: oh, to anynomous gyus you mean?14:33
didrockssinzui: but still, I'm logged into LP and https://edge.launchpad.net/+apidoc/ is only has we were anonymous?14:33
didrocks(I don't see it there)14:34
sinzuididrocks: I do not think I should have to log in in my script to get a gpgkey that I could get by scraping the web site while not logged in14:34
didrockssinzui: I agree, but my question wasn't that one:14:35
didrockssinzui: I'm logged into LP, and I don't see the api in +apidoc14:35
didrocks(the gpg in the api)14:35
didrocksis it because I'm considered as being anonymous?14:36
sinzuiinteresting, I read the api from the page.14:36
didrockserf, ctrl + R for the win14:36
sinzuididrocks: https://edge.launchpad.net/+apidoc/#gpg_key14:37
didrockssinzui: right, the cache wasn't refreshed apparently14:37
sinzuididrocks: you can see the collection listed on https://edge.launchpad.net/+apidoc/#team14:37
didrockssinzui: well, I can do that if you point me to the right doc, but I would prefer to have my ssh patch which prevents me release Quickly 0.4 to be reviewed first and not really have the time to fix that before Friday14:37
sinzuididrocks: I can do it, I just wanted to be certain that the anon was intentionally omited14:38
didrockssinzui: no no, it wasn't intentional14:39
didrockssinzui: same fix has to be done to the ssh key I guess14:39
sinzuiyes, I think every object that we have to write a browser:url directive for must get an anon security function, because this is the first time the object is published14:41
danilosjtv, what was the page that describes setting up a local buildfarm for translations?14:47
jtvdanilos: it's linked from dev.launchpad.net/Translations14:47
jtvunder Technical topics14:47
jtvlast entry there14:48
danilosjtv, https://dev.launchpad.net/Translations/BuildTemplatesOnLocalBuildFarm14:48
danilosjtv, does that integrate any discoveries henning has made? :)14:48
jtvdanilos: I may have copied some small things, but mainly I refer to the page he's written.14:49
jtvHe documented how to get the slave running in a chroot, which imho is infinitely more useful than the local install we used to have.14:50
jtvdanilos: I'm not aware of any inconsistencies between the two14:51
danilosjtv, cool, thanks14:51
kfogelderyck: I see bug 538226 and bug 538219 in the backlog; they look like good ones for me to grab, when I get back to coding, so I'll just pick one when ready.14:52
mupBug #538226: "What is a patch?" links to wrong help page <help> <trivial> <ui> <Launchpad Bugs:Triaged> <https://launchpad.net/bugs/538226>14:52
gmbderyck: I've just confirmed pitti's reproduction of the bug in https://bugs.edge.launchpad.net/ubuntu/+source/apport/+bug/538097/comments/2914:53
mupBug #538097: +storeblob fails with "500 Internal server error" on production (works on edge) <apport-collected> <lucid> <oops> <Launchpad Bugs:Triaged> <apport (Ubuntu):Invalid> <https://launchpad.net/bugs/538097>14:53
gmbderyck: Doesn't occur on edge.14:53
gmbderyck: At the moment I've no idea why that is, unless some security gubbins got CP'd to production.14:54
deryckgmb, hmmm, ok, interesting.14:54
deryckgmb, we should ask around about recent CPs.14:54
gmbderyck: Or perhaps there's some redirect happening with request to https://edge.launchpad.net/+storeblob (guessing)14:55
gmbderyck: Simple work around is to try posting to http://launchpad.net/+storeblob; the http->https redirect adds a referrer header.14:55
gmbderyck: So we can pretty much confirm that that's the problem.14:55
salgadogary_poster, ^14:56
gary_posterreading back...14:56
salgadolooks like the change to not accept POSTs without a referrer broke apport: https://launchpad.net/bugs/53809714:56
mupBug #538097: +storeblob fails with "500 Internal server error" on production (works on edge) <apport-collected> <lucid> <oops> <Launchpad Bugs:Triaged> <apport (Ubuntu):Invalid> <https://launchpad.net/bugs/538097>14:56
gary_postersalgado: ah :-(14:57
salgadogmb, ^14:57
deryckhey, at least we know what happened now.14:57
derycksalgado, thanks for pointing this out to us.14:58
sinzuimatsubara: I cannot find the bug that is marked as bad for the registry: http://people.canonical.com/~lpqateam/test-plan-report-10.03.html . I do not see anything related to my team on https://bugs.edge.launchpad.net/launchpad-project/+bugs?field.tag=qa-bad14:58
gmb(But subdued, becase, hey, bug)14:58
gmbgary_poster, salgado: I'll update the bug report; want me to re-target to foundations?14:58
gary_postergmb: yes, thanks.14:58
gary_postergmb: please don't go int ospecifics; this is a security bug after all14:59
matsubarasinzui, it's fix released, that's why it doesn't show up in https://bugs.edge.launchpad.net/launchpad-project/+bugs?field.tag=qa-bad14:59
matsubara(i.e. default search doesn't show fix released bugs)14:59
gmbgary_poster: Okay. Should I mark it as such? It' got a lot of subscribers already so there may not be much point.14:59
gary_posterAlthough perhaps that's closing the barn door after the animals have all escaped...14:59
gary_postergmb: no.  just assign to foundations with no explanation, perhaps.  I'll deal with it...somehow. :-)15:00
gmbgary_poster: Ok.15:00
deryckgmb, and ping pitti off list/channel about it, just so he knows what's up15:01
gmbderyck: I think gary_poster's already taking care of that.15:02
deryckgmb, ok, cool15:02
deryckgmb, and let's do our call 15 after.  To give us time to close out some of this.  (and me more coffee ;))15:02
deryckgmb, cool?15:02
gmbderyck: Sure. Besides, I need tea.15:03
gary_posterderyck, gmb: I don't have a reply from pitti on irc.  Is he known to be around-ish (i.e., he will be back soon) or should I just shoot him an email and look for another resource?15:05
gmbgary_poster: Have you tried on the internal server?15:06
gary_posteryeah, he just got back thanks15:06
sinzui‏‎mars: is this bug still valid bug 43583215:09
mupBug #435832: Launchpad email verification uses sso server template <post-3-ui-cleanup> <ui> <launchpad-web:Triaged> <https://launchpad.net/bugs/435832>15:09
adiroibanEdwinGrubbs, sinzui: do you know what is the revision number where the referrer/next_url mixin was introduced?15:09
sinzuiadiroiban: no, but I think It landed last week15:10
adiroibanok. thanks. I will search the log and the code15:11
=== matsubara is now known as matsubara-lunch
adiroibansinzui: do you have any hints how / where should I look for that mixin?15:17
salgadoadiroiban, tests are passing now; I've submitted your branch again15:18
adiroibansalgado: thanks! Can you please run the test using screen as some tests were not related to „active” attribute and they were still failing on ec2-test15:19
sinzuiadiroiban: ReturnToReferrerMixin15:20
salgadoadiroiban, too late, but you don't have to worry now -- if there are spurious failures I'll fix them and then land your branch15:20
sinzuifrom canonical.launchpad.webapp.launchpadform.ReturnToReferrerMixin15:20
sinzui^ adiroiban15:20
adiroibansinzui: thanks!15:21
adiroibansinzui: ReturnToReferrerMixin is not working for my bug15:22
adiroibanas the referrer url can be made invalid by the changes from the form15:23
adiroibanalso looking at ReturnToReferrerMixin, it looks like it is not checking if the referrer URL is identical with the current URL15:26
adiroibanand preventing a redirection to the same page15:26
kfogeladeuring: https://code.edge.launchpad.net/~kfogel/launchpad/534736-patches-portlet-first/+merge/21377  (when you get a chance)15:49
=== salgado is now known as salgado-lunch
adeuringkfogel: sure, let me first do some organsational stuff for the mini sprint. I have to use a travel agency that is absolutely unresponsvie to emails ;)15:51
kfogeladeuring: sure.  You know, I've really found that the travel agencies cost more time than they save.15:51
adeuringkfogel: you have point there...15:52
=== deryck is now known as deryck[lunch]
=== jamalta is now known as jamalta-afk
adeuringkfogel: r=me16:04
kfogeladeuring: thank you16:05
kfogeladeuring: I'm going to go with ui=none, unless you counsel otherwise.16:07
adeuringkfogel: I think that's fine.16:07
kfogeladeuring: I just got a testfix mode failure:16:10
kfogelCommand failed!16:10
kfogelAll lines of log output:'Commit message [[r=abel][bug=534736][ui=none] On a bug page,\n\tlist patches before non-patch attachments.] does not match commit_re [(?is)^\\s*\\[testfix\\]\\s*\\[(?:release-critical=[^\\]]+|rs?=[^\\]]+)\\]]'16:10
kfogeladeuring: ...from PQM.  Any idea what's up?16:11
adeuringkfogel: seems that buildbot failed recently16:11
adeuringthat's not related to your branch16:11
=== jamalta-afk is now known as jamalta
=== matsubara-lunch is now known as matsubara
=== bac` is now known as bac
=== beuno is now known as beuno-lunch
jtvbigjools: I'm solving the UI's inability to deal with non-Soyuz buildfarmjobs by adding some interfaces and templates...  wgrant turned out to be thinking in the same direction, as per our discussions this weekend.  How would you feel if I added browser & templates directories to lib/lp/buildfarm?16:42
bigjoolsjtv: I would be against that16:43
bigjoolsbut you could persuade me16:43
jtvbigjools: there's not a lot of the stuff now (after some simplification) but it'd be a more appropriate place I think.16:43
bigjoolsjtv: we already have lp.buildmaster, or are you talking about a new module?16:44
jtvbigjools: sorry, lib/lp/buildmaster is what I meant16:44
bigjoolsjtv: ok, the /builders and builder-related pages could all be moved there16:46
jtvphew :)16:46
jtvNot that I'm looking forward to the job, but the end result seems more desirable.16:46
jtvThis sort of validates my sanity.  :)16:46
bigjoolsjtv: I'd like to see this discussed on the mailing list though16:46
bigjoolsso I can see more details16:46
jtvbigjools: sure.  Still waiting for responses to this weekend's design proposal though.16:46
jtv(hint, hint ;-)16:47
bigjoolsah hrm yes16:47
bigjoolsjtv: the refactoring of the "name" stuff is a good plan but I need to re-read your email to get a better handle on things16:47
jtvI should add that the complexity of the existing design became a bit more explicit with a branch I just landed:16:48
bigjoolsjtv: I don't know why that cross check is so complicated either16:48
bigjoolsbut it's there to protect against builders appearing and disappearing with random builds16:48
jtvHeretofore it was assumed that one buildfarmjob's "self.build.id" might be completely unrelated to another's, since their self.build objects might be in different db tables.16:48
bigjoolsand to prevent the wrong build being assigned to a source16:49
jtvWell, the buildqueue check would do that—barring determined attack (which would have only a tiny chance of messing things up anyway aiui)16:49
bigjoolswell buildqueue will be common16:49
bigjoolsI can't remember the current format, it names directories buildid-buildqueue-id-something doesn't it?16:49
bigjoolsI suspect it was originally belt and braces as you say16:50
jtvthat, grumble, depends on the fritzin' type of the confounded thing16:50
jtvNo directories are involved that I know of, but Build uses <Build.id>-<BuildQueue.id>16:51
jtvSorry, a binary package build uses that.16:51
jtvThen there's other types that are encouraged to use <BuildBase.id>-<BuildQueue.id>, where BuildBase doesn't really have an id but any of its implementing leaf classes will.16:51
jtvAnd then of course there's us folks who don't have a self.build16:51
jtv...so we use <Branch.name>-<BuildQueue.id> instead, mainly to help debug.16:52
jtvI'm suggesting to use <BuildQueue.id>-<hard-to-guess-string>16:52
jtv...without going into any specifics of different build-farm jobs at all.16:53
=== salgado-lunch is now known as salgado
jtvbigjools: wgrant, bless his toes, said something like "this dates back to the beginning of time, remember."  I would like to state once again for the record that I am not that old.16:55
jtvI did mention that I could ask barry though.16:55
bigjoolsjtv: you really should have a build table16:55
bigjoolsfor history16:55
bigjoolsI talked about this with noodles16:55
bigjoolsthat doesn't preclude this change though16:56
jtvwell, there's all this talk of different ways we could store history...  what I'm doing with the UI atm will facilitate that, I think.16:57
jtvI'm using inheritance between interfaces to achieve a kind of polymorphism in displaying things, assuming I'll always get the view for the most-specific applicable interface.  Is that bad?16:58
=== gary_poster is now known as gary-lunch
bigjoolsjtv: once your job is finished, there will be no permanent record of it.  You need a build table so we can see the history of a builder.17:01
jtvbigjools: I know, yes.  But as you say that doesn't preclude the simpler slave build id, and I think the UI work is worthwhile regardless as well because it gives us a base for having specific UI (elements) for different types of build record.17:03
bigjoolsjtv: did you discuss that with noodles?17:05
jtvbigjools: no, not since the weekend17:06
=== beuno-lunch is now known as beuno
=== deryck[lunch] is now known as deryck
jtvbigjools: I'm not proposing to implement a full UI here, just taking away breakage from "buildfarm == soyuz" assumptions.17:09
jtvBut it does have useful little things like a link formatter for builds.17:09
jtvTaking some logic out of the TAL.17:10
jtvHmm...  for the record, I did not mean to imply that the TAL needs to be less logical.17:11
bigjoolsjtv: it could be more <something> though :)17:12
=== danilos is now known as daniloff
jtvbigjools: you mean, less can be more?  :)17:17
bigjoolsless is more - it's my favourite pager17:18
rockstarsalgado, hep me! I can't build. http://pastebin.ubuntu.com/395727/17:38
rockstarOr maybe anyone from foundations can help. ^^17:39
salgadorockstar, I know absolutely nothing about gpgme. are you on Lucid?17:40
rockstarsalgado, nope,  Karmic.17:40
salgadorockstar, looks like you don't have launchpad-dependencies installed17:46
rockstarsalgado, Oh, crap.  I wasn't in my development chroot.17:47
* rockstar facepalms.17:47
* rockstar gets up to go on a walk and get re-focused17:47
=== gary-lunch is now known as gary_poster
=== jtv is now known as jtv-zzz
james_whow do I run all the tests in lib/lp/code/stories/branches/?18:25
beunojames_w, ./test -tv code.stories.branches?18:25
beunomaybe, I don't remember it super well18:26
=== matsubara is now known as matsubara-afk
james_wbeuno: nope, doesn't run any tests18:43
james_wnor lp.code.stories.branches or lib.lp.code.stories.branches or lib/lp/code/stories/branches/18:44
* james_w tries a more brute-force way18:46
=== NCommander is now known as Guest5889
* mwhudson_ watches his other machine fsck18:49
=== mwhudson_ is now known as mwhudson
mwhudsonbit odd though, it just rebooted18:50
Ursinha-afksinzui: hi19:06
=== Ursinha-afk is now known as Ursinha
sinzuihi Ursinha19:06
Ursinhasinzui: matsubara-afk asked me about that bad item that was showing up in registry count, it seems that the script was considering salgado in registry team, but I've checked the config file and he was in the proper team; I've stopped and started the script again and the bad item is in its right place19:08
sinzuioh..yes I consider salgado being misclassified, but he wasn't last month so I assumed I was being confused19:09
Ursinhasinzui: that's weird because I've put him in the same team he was last month19:09
Ursinhathis was spurious19:10
Ursinhamwhudson: hi, is bug 536486 really qa-ok? by your comment it seems it's still not working as it should..19:12
mupBug #536486: Svn import of ruby taking 2.7G RSS <qa-ok> <Launchpad Bazaar Integration:Fix Committed by mwhudson> <https://launchpad.net/bugs/536486>19:12
mwhudsonUrsinha: i think i hijacked the bug to link a branch to it inappropriately19:13
mwhudson(it turns out)19:13
mwhudsonUrsinha: i'll fiddle things around19:14
EdwinGrubbssinzui: ping19:14
Ursinhathanks mwhudson19:14
sinzuiHi EdwinGrubbs19:14
jmlg'night floks19:15
=== salgado is now known as salgado-afk
EdwinGrubbssinzui: on the mailing list, interest was expressed in removing the "Register a blueprint" link from the involvement portlet. The official_blueprints boolean appears to only be useful for showing/hiding that one link on the index page. Can I remove it from the form so it's impossible to edit, since "Register a blueprint" is always visible on blueprints.lp.net?19:16
sinzuiEdwinGrubbs: I do not think so. I still think most people were confused by your proposal19:17
sinzuiEdwinGrubbs: That link to enable blueprints is only for the project owner. He cannot find it to enable or disable it19:18
sinzuiEdwinGrubbs: So if we remove the proposal for the owner see how to control blueprints on his project, how will he ever accomplish his task?19:19
sinzuiEdwinGrubbs: I think the confusion is that jml and poolie thought you were going to let any person use that link19:19
jmlsinzui, that's not what I was confused about, I think.19:20
mwhudsonUrsinha: bugs fiddled, i think19:20
EdwinGrubbssinzui, jml: it sounded like going to the blueprints tab is the desired place for the "Register a blueprint" link since registering a blueprint is not an important enough activity to be on the index page.19:21
jmlEdwinGrubbs, yeah, that's what I meant.19:22
jmlEdwinGrubbs, it's orthogonal to the other things you were discussing.19:22
sinzuijml:setting aside my dislike for blueprints, I see that some project like the app, and project owners should not have to hunt through the UI to enable it. conversely, project owners that are disappointed about the app should be able to disable it.19:22
jmlsinzui, I suggest that we make it a little hard to enable blueprints for now.19:23
sinzuiWhy is blueprints different from ask a question or register a bug?19:23
sinzuijml: it is pretty fucking hard right now19:23
jmlsinzui, because the app isn't very good.19:23
jmlsinzui, heh heh19:23
sinzuiIn fact only someone who has setup a project before, probably in 2.0 can enable any application19:24
sinzuijml: EdwinGrubbs: I am looking at https://blueprints.edge.launchpad.net/subunit19:25
EdwinGrubbsit is odd how "Register a blueprint" appears twice19:26
sinzui^ blueprints is not an official app. the Involvment link to register a blueprint be removed from the blueprint page?19:27
sinzuiEdwinGrubbs: yes it is weird. I think is a UI inconsitency. The bugs team replaced the action buttons with involvement links. I updated the other apps to do the same, but I think the action link is more appropriate19:28
sinzuiEdwinGrubbs: The bugs team went farther last month when they removed the entire bugs page if the app is not official19:29
jmlsinzui, did you see the registry bugs I filed recently?19:30
sinzuijml: my email is littered with your email filings. I do not know which one you are referring to19:30
jmlsinzui, me neither :)19:31
jmlsinzui, don't worry about it then.19:31
sinzuijml: you want to say your project does not have a source package in ubuntu?19:31
sinzuijml: or The ajax ok/cancel button is rendered poorly when the primary context's title is edited19:32
sinzuijml: Project description and summary field could explain formatting rules19:34
sinzuiwhich relates to a markup bug you moved to launchpad-web19:34
jmlsinzui, let me find it. it's none of those. I think it's to do with enabling bugs for a project19:34
sinzuiman, there are really old bugs related to that. I found them in malone19:35
jmloh wait, that's a malone thing19:35
* sinzui assigned two to himself last week19:35
sinzuijml: I have a prototype single bug configuration screen19:35
jmlanyway, we have a real inconsistency when it comes to enabling applications for a Launchpad project19:36
jmlit looks and feels really sloppy.19:36
sinzuisingle screen that allows you to configure bugtracking19:36
jmlsinzui, that'd be awesome19:36
sinzuiI was very tempted to propose we abandon the bool states last week19:36
jmlsinzui, I'd like more elaboration before you do that, but it sounds like the right line of thinking to me.19:37
sinzuijml: and it is hard to make apps behave consistently when some apps have lots of community involvement, and by the fact that two of our apps are not owned19:37
jmlsinzui, I'm not sure that's the reason.19:38
sinzuiHow do you deal with requests from project owners to disable the branch tab and remove all the branches...including the ones not owned by the owner?19:39
jmlsinzui, now _that's_ a more compelling reason :)19:40
jmlsinzui, the short answer is "say 'no'"19:40
sinzuiThat is what we say. The project owner thinks of launchpad has a project hosting service19:41
jmlsinzui, the longer answer is find out why they want it19:41
sinzuijml: In most cases it is because the owner prefers git or subversion, in fact since he thinks launchpad is a project hosting service, he want the code removed19:42
jmlsinzui, you are saying that they don't actually have a benefit in mind when they ask for it?19:44
sinzuijml: yes19:44
sinzuiI think some were concerned that launchpad always shows up first in google listing19:45
jmlsinzui, hmm19:45
sinzuiOthers were worried about old code confusing users19:45
jmlsinzui, there's a blog post in that.19:45
sinzuibut the latter case is always addressed by switching to an improt19:46
jmlsinzui, I'm less concerned about the old code reason -- it's easy enough to update trunk to point at the right branch if you are a project owner.19:46
jmlsinzui, right.19:46
sinzuiEdwinGrubbs: I think there are two action regarding blueprints. One, do not add a link to configure blueprints. two I will find the bug about the blueprints tab and update it to include disabling the feature when it is not configured...and provide a link for the owner to enable it19:50
EdwinGrubbssinzui: do I move the "enable blueprints" boolean back to the +edit form, or do I just make it inaccessible by removing the "configure blueprints" link?19:54
sinzuiEdwinGrubbs: back to +edit.19:54
sinzuiEdwinGrubbs: do you have a separate for for blueprints?19:55
sinzuiseparate form?19:55
jmlanyway guys, I'm gone for real19:56
jmlgood chatting.19:56
EdwinGrubbssinzui: yep, I have a separate form for all the services. Except for bugs, they are just one boolean. Though this is weird, I think it is better than searching through all the +edit fields.19:57
sinzuiEdwinGrubbs: that is fine, great in fact.19:57
EdwinGrubbssinzui: so, what do I do about the "configure blueprints" link and field then?19:58
sinzuiEdwinGrubbs: We can add a link to the blueprint app menu for product. Make sure the permissions are launchpad.Edit19:59
sinzuiEdwinGrubbs: It should be the first link in the list19:59
sinzuiEdwinGrubbs: I am looking at my own page: https://blueprints.edge.launchpad.net/gdp19:59
EdwinGrubbssinzui: although I am risking running the conversation in circles, it's odd to have the "configure blueprints" link have no effect on blueprints.launchpad.net, but it will show/hide the "Register a blueprint" link in the involvement portlet.20:01
sinzuiEdwinGrubbs: It does have an effect, it make the Register a blueprint link appear in the involvement portlet20:02
sinzuiEdwinGrubbs: And by landing your change, it make it easier to disable the blueprint root page like bugs does.20:02
EdwinGrubbssinzui: but only on the index page. The blueprints.lp.net/$project page always shows "Register a blueprint". I haven't done any work on disabling the blueprint root page. Has that already landed? Maybe I just need to merge in RF again.20:03
sinzuiEdwinGrubbs you are definitely confused is said "I" would find the blueprint bug and update it. You are NOT doing anything disabling the root20:05
sinzuiEdwinGrubbs: Do not render the configure link blueprints link in the involvement portlet as jml asks.20:05
sinzuiEdwinGrubbs: Do add the link as the first link the project blueprints menu20:06
sinzuiEdwinGrubbs: If the second point bothers you, you can put the official_blueprints back on +edit20:06
sinzuiEdwinGrubbs: want to keep the scope of this exception out of your work20:07
sinzuiSomeone has to volunteer to change blueprints and maybe answers. We had no volunteers last month20:08
james_wwow, branch-rewrite.py imports windmill20:11
mwhudsonjames_w: launchpad is a tangle :(20:12
EdwinGrubbssinzui: was that a request for volunteers?20:12
sinzuiEdwinGrubbs: no, just a statement about the likely hood the blueprints or answers getting changed20:13
wgrantAre we in testfix?20:24
wgrantI have two "submitted to PQM" ec2 emails from 5 hours ago.20:25
wgrantNeither of them have actually landed.20:25
wgrantjames_w: There's a bug for that branch-rewrite.py Windmill PATH issue.20:38
wgrantgary_poster: Since I guess Launchpad won't have bothered to email you, there is a new comment on https://bugs.edge.launchpad.net/launchpad-foundations/+bug/52934820:39
wgrantjames_w: There is a workaround that I've been using for a while on https://dev.launchpad.net/Translations/BuildTemplatesOnLocalBuildFarm20:40
james_wI didn't need it to test this change, but thanks20:40
gary_posterwgrant: I had not received an email, oddly.  You know, you could have requested a patch.  Thanks for the comments.  The sarcasm is perhaps less warmly received.  :-)  I'll contact you privately, because I'm afraid I don't see how the webservice aspect can be addressed.20:46
wgrantYeah, sorry about that. I was pretty displeased.20:47
wgrantgary_poster: (it's not odd that you haven't received an email; ~launchpad-security emails go through ~launchpad, which redirects all emails to launchpad-bugs@lists.c.c, which nobody reads)20:49
gary_posterah :-/20:49
wgrantWhy yes, this does mean that nobody gets automatically notified of new security issues.20:50
NCommanderBjornT: re: raw source changelogs, I don't mind rewritting the code to use librarian, but I fear hitting the librarian to pull it on the fly and stick it in the UI might really put a strain on loading an Soyuz page that needs the raw changelog20:52
lifelessNCommander: why are you worried about that?20:53
lifelessNCommander: as in, do you have a metric that says its a concern?20:53
NCommanderlifeless: no, but we keep the debian/copyright in the database. my knowledge of librarian is fairly limited (I'm still getting my around to LP source trees)20:54
lifelessso the librarian is pretty fast20:54
lifelessand there is a squid in front of it to boot20:54
NCommanderlifeless: ah. I didn't want to be the one that causes a change to land that causes LP to DOS its own librarian :-)20:54
lifelessthe librarian is just a big disk, with stuff named by hash, and a front end that takes urls in and serves the files from the disk.20:55
lifelessNCommander: bug attachments, all packages (ever), crashdumps - they all go in the librarian20:55
wgrantlifeless: But none of those are ever rendered in the UI.20:56
lifelesswgrant: no, but the load on the librarian is related to use, of which rendering is one20:56
lifelessthe key questions are:20:56
lifeless - how many loads per second are you expecting20:56
lifeless - how many loads per second the librarian already does.20:57
NCommanderlifeless: well, ideally, the SRP in Soyuz will pull raw source changelog directly.20:57
NCommanderlifeless: we also need it for generating changelog diffs as part of native source sync20:58
lifelessso source sync is low frequency20:58
lifelessits (at most) one every 5 seconds - if every package changed every day20:59
dobeyis there a good way to getting bugs for API calls escalated?20:59
lifelessdobey: the most effective way is to submit a patch20:59
lifelessfailing that try begging, and if that doesn't work, ask your manager :P21:00
dobeyis there good documentation on making changes to the REST API bits? :)21:00
lifelessdobey: generally you don't need to change the REST API bits at all, just the object definitions. what is your bug?21:01
dobeylifeless: i haaven't filed the bug yet, as i wanted to understand more process first. but I want date filtering on {person,team}.{getMergeProposals,getRequestedReviews}21:02
lifelessok, thats just an object change21:02
lifelessreflecction is used to export interface defintions to rest21:03
lifelessso the bug belongs on the relevant team21:05
dobeynot being a lp developer, i am not familiar with these things :)21:05
lifelessin this case, launchpad-code21:05
lifelessdobey: et moi21:05
lifelessdobey: anyhow, launchpad-code, and if you haven't filed it, I wouldn't even consider escalation yet :P21:05
dobeylifeless: i'm just wondering what the process is :)21:06
henningewgrant: your branches passed ec2 testing but did not land because of testfix. They are in pqm now.21:16
henningetestfix mode21:16
wgranthenninge: Ah. Thanks.21:17
elmoI'm getting unexpected form data trying to update bugs in LP, specifically using +editstatus slider21:17
elmois that a  known thing?  this is on prod21:17
wgrantelmo: You block referers?21:18
elmowgrant: yeah21:19
wgrantelmo: LP now hates you.21:20
=== matsubara-afk is now known as matsubara
=== jamalta is now known as jamalta-afk
* mwhudson lunches23:32

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