/srv/irclogs.ubuntu.com/2012/07/24/#launchpad-dev.txt

nigelb2600:05
nigelbgrr00:05
StevenKwgrant: I have private bug with nothing, with subscriber and with structural subscriber, asserting on what getBugNotificationRecipients() returns. I'm obviously missing a bunch of scenarios00:55
wgrantThere are three implicit subscriber cases: structural subscriptions, dupe subscriptions, and assignees.00:56
wgrantI'm not sure if dupe assignees and dupe structsubs also count.00:57
StevenKwgrant: So assignee sounds a good plan -- I've not thought about dupes yet00:58
StevenKwgrant: Blink.01:25
StevenKwgrant: IBug.markAsDuplicate() recalculates heat, but so does IBug._markAsDuplicate() which the former calls.01:25
wgrantStevenK: markAsDuplicate recalculates the heat of the new master, _markAsDuplicate recalculates the heat of the old.01:27
StevenKAh01:27
StevenKwgrant: http://pastebin.ubuntu.com/1107437/01:38
wgrantStevenK: A good start, indeed.01:54
StevenKwgrant: Do you think it's worthwhile expanding out to public bugs?01:56
wgrantStevenK: Certainly02:04
wgrantStevenK: It is to be a thorough set of tests for getBugNotificationRecipients02:04
wgrantIt's hardly comprehensive or thorough if it doesn't do public bugs too.02:04
=== _mup__ is now known as _mup_
=== Beret- is now known as Beret
lifelessgary_poster: nice mail :)02:15
StevenKwgrant: http://pastebin.ubuntu.com/1107562/  bugwatch is up next, and then subscribed to question linked to bug04:15
StevenKThen I might be done04:15
StevenKwgrant: So now that hilarity has passed, bugwatches?04:32
wgrantStevenK: How are watches involved?04:34
StevenKwgrant: sinzui implicates them04:34
wgrantHm04:35
wgrantThat's odd.04:35
StevenKwgrant: I don't know how they work. :-(04:35
StevenKWell, I know they link to an external bugtracker and checkwatches does something to some watches and then hangs.04:36
StevenKBut can a watch have subscribers or impact on getBugNotificationRecipients()?04:36
wgrantNo04:40
wgrantI don't believe they're relevant here at all.04:40
StevenKI shall ignore them, then04:41
* StevenK moves onto working on how to staple a question and bug together04:41
StevenKwallyworld, wgrant: https://code.launchpad.net/~stevenk/launchpad/bugnotification-testcases/+merge/11640205:17
wgrantStevenK: Looking05:23
wgrantStevenK: https://code.launchpad.net/~wgrant/launchpad/faster-checkCanBeDeleted/+merge/11640405:30
StevenKwgrant: r=me, looks great05:32
wgrantThanks.05:33
StevenKwgrant: What about my review? :-)05:33
wgrantStevenK: Looking05:34
wgrantWhat's + def test_public_bug_with_duplicate_same_pillar_subscriber(self):?05:34
StevenKwgrant: Both bugs are against the same pillar05:35
wgrantYes, but why?05:35
wgrantHow's it relevant as a separate case?05:35
StevenKThe private case will be different, and I did the same cases for public bugs05:36
StevenKI can condense the two public duplicate tests if you wish05:36
wgrantI don't see how the private case is different.05:36
wgrantIt's matters whether the subscriber has access05:36
wgrantNot whether it's a different pillar.05:37
StevenKwgrant: It will be -- give an APG against the pillar05:37
wgrantStevenK: Und?05:37
wgrantThat's the case where there's an APG against the pillar05:37
wgrantNot the case where they're the same pillar05:37
StevenKwgrant: Right, so condense the four down to test?05:38
StevenKDown to two, rather05:38
wgrantRight05:38
wgrantTest things that should be tested because they make a difference.05:38
StevenKI split it out due to sinzui's mail05:38
wgrantAlso, do the tests pass?05:39
StevenKThey do05:40
wgrantHuh05:41
wgrantHow does the question one work...05:41
wgrantOh god05:41
wgrantQuestion.linkBug subscribes the question owner05:41
wgrantWhat could possibly go wrong.05:41
StevenKWCPGW, right?05:42
StevenK:-)05:42
wgrantImplicit access grants are bad, m'kay.05:42
StevenKYes, I'm not sure the two question tests add value05:42
wgrantThey don't.05:42
StevenKBin them too?05:42
wgrantYes05:42
wgrantLet me read sinzui's mail05:42
wgrantAh05:42
wgrantSo he05:42
wgrantprobably speaks of the bug->question notification forwards05:43
wgrantI'm not quite sure how they work05:43
wgrantBut they're separate from this05:43
wgrantSo don't need to be considered here, at least not yet.05:43
wgrantStevenK: Are you sure that all of xx-initial-bug-contacts.txt is duplicated?05:52
StevenKwgrant: It was sinzui's suggestion that it be binned since much of the test will fail to be true in a few weeks with the bug supervisor and security team playing05:54
wgrantStevenK: Sure, but it covers a whole lot of stuff that's only barely related to what you're adding tests for05:55
wgrantLike setting the bug supervisor05:55
wgrantAnd the security contact being subscribed by default05:55
StevenKBug supervisor is also dying horribly too?05:56
StevenKSecurity contact is also changing after we can filter on information type05:56
wgrantStevenK: Bug supervisor is not dying.06:00
wgrantIts role is changing.06:00
wgrantSecurity contact will probably die entirely.06:00
wgrantWe will still need to be able to set the bug supervisor.06:00
StevenKI can shuffle that part into a browser test easily06:01
wgrantRogjt06:01
wgrantRight06:01
wgrantBut you can't just delete it06:01
wgrantWithout confirming that it's been replaced, or replacing it yourself06:01
StevenKlib/lp/bugs/browser/tests/test_bugsupervisor.py:class TestBugSupervisorEditView(TestCaseWithFactory):06:02
StevenKDoes that count?06:02
wgrantIf it tests the same stuff, sure.06:02
wgrantOops06:08
StevenKwgrant: Diff updated, I've added new tests for the Change bug supervisor link06:31
StevenKAnd I'm still 164 lines in credit06:31
wgrantStevenK: Thanks06:48
StevenKwgrant: Thanks. Did you see the branch I just landed?06:49
wgrantAha06:51
wgrantI think only two of those were mine06:51
wgrantYay06:53
wgrantOnly one actual bug in bugsummary maintenance06:53
wgrantAnd one in the rebuild script06:53
wgrantStevenK: https://code.launchpad.net/~wgrant/launchpad/bug-1028279/+merge/11641107:32
wgrantStevenK: (yes, this is very similar to your thing which broke everything, but I like to live dangerously)07:32
adeuringgood morning07:44
=== matsubara-afk is now known as matsubara
=== almaisan-away is now known as al-maisan
Ergo^morning10:34
Ergo^gary_poster, hello, im looking for some info on the position10:35
Ergo^http://blog.launchpad.net/were-hiring/were-hiring-software-engineers-to-work-on-cloud-projects-in-canonical10:35
Ergo^czajkowski, said i should be asking you about it10:35
lifelessor gmb10:35
czajkowskigmb: ping10:36
lifelessgary will be asleep right now (I hope!)10:36
Ergo^haha ok :]10:36
czajkowskiErgo^: mornign more of the developers are online now10:36
czajkowskiand in fact gmb is on hols10:36
czajkowskiwgrant: might you be able to answer some of Ergo^ questions on lp dev please, he's thinking of applying for one of the roles10:36
wgrantI'm trying to not be here :)10:38
wgrant'cause it's getting late10:38
czajkowskiI know but I cant find any EU folks up atm10:38
wgrantgmb can't be too far away10:39
Ergo^wgrant, carry on, i can ask later ;-)10:39
czajkowskiadeuring: allenap ping are either of you about ?10:39
jpdswgrant: From where you are?10:39
* adeuring is looking10:40
wgrantHeh10:40
czajkowskijpds: AU10:40
czajkowskialthough he never sleeps10:40
czajkowskiadeuring: thanks10:40
czajkowskiErgo^: adeuring might be able to answer some of your questions10:40
czajkowskiErgo^: thanks for coming and asking though10:40
adeuringErgo^: I think you should wait for gmb if you have more specific questions about the job.10:42
adeuringBut we can of course anwser more generic questions about Canonical ;)10:43
Ergo^well, i would be interested to what i would work on and what kind of skill level would be expected for the position10:45
Ergo^adeuring, i've never worked for a IT organization that would have many employees, so im not sure what i would ask about :-)10:47
adeuringErgo^: I don't know much what the the work is about aside from the general "cloudy" stuff... Regarding an org with "many" employees: I myself do not notice that much about the number of people employed by Canonical. You will work i a relatively small team, I assume. four or five people probably.10:50
Ergo^czajkowski, can i pm you ?11:05
czajkowskiErgo^: sure11:10
allenapErgo^: gmb will be able to tell you more about the team's first project, but, like adeuring, there isn't the feel of "big company IT". We try to produce the best code we can, in a professional way; I have learnt *so* much about how to produce large pieces of software from working here. We're rigorous about testing, and we try to make decisions based on evidence and measurement. We don't always get it right, but we tend to see mistakes a11:12
allenaps an opportunity to learn rather than to blame.11:12
lifelessErgo^: the new squad will be working on ceilometer initially.11:12
lifelessallenap: +111:12
czajkowskilifeless: go sleep!11:12
lifelessone word: Cynthia.11:13
allenapHa!11:13
czajkowskilifeless: awww11:14
Ergo^ok, well i work in environment where we dont write tests at all - they move more like early facebook - the boss doesnt want to budget for tests11:15
Ergo^but theres no problem to write tests11:15
Ergo^i would love to move to environment where i can learn new things11:16
=== matsubara is now known as matsubara-afk
Ergo^lifeless, so ceilometer - what kind of web stack will be used for the project? pylons/pyramid? something home brewed ?11:19
lifelesshave a poke around yourself - https://launchpad.net/ceilometer11:23
Ergo^oh it uses webob11:25
Ergo^nice :]11:25
Ergo^and sqlalchemy, even nicer11:25
Ergo^i was expecting it would be using sqlstorm11:25
rick_h_Ergo^: hey howdy11:25
Ergo^ohai rick_h_11:26
rick_h_Ergo^: you were hacking on some of the min-webhelpers stuff right?11:26
Ergo^rick_h_, yeah ive made html.grid stuff11:26
rick_h_Ergo^: heh, not going to see much of the usual stack around :/11:26
rick_h_but really good dev practices, some stuff to learn for sure11:26
Ergo^rick_h_, well i would love that because i wont learn new things the place im at atm11:27
Ergo^different focus11:27
rick_h_Ergo^: definitely apply if you might be interested and chat with the guys. They're a really good group of smart peeps.11:27
Ergo^i would be, question is if they would be :P11:27
Ergo^rick_h_, well even if its not stack im used to it has some parts i know and love, so thats a good start11:30
rick_h_right, the launchpad stack is kind of strange, but you're allowed to learn it. I didn't know anything more than some articles of zope stuff11:30
rick_h_but most other things are django based11:30
* Ergo^ knows nothing about django11:31
rick_h_heh, it's 'easy' :P11:31
Ergo^i would expect pylons/pyramid based wsgi apps, since i would expect its more flexible for this kind of environment11:32
rick_h_you'd think...but no. I've not won a battle on that front yet though there are some of us trying11:32
Ergo^rick_h_, i would expect mramm to help ;-)11:33
rick_h_hah, I'll have to ask him what his team's up to on that end I guess.11:33
lifelessgo11:34
lifelessthey are all about go11:34
Ergo^yeah he mentioned he writes in go right now last time ive asked11:34
rick_h_ah, he's over there11:35
Ergo^well, the ive browsed the code for ceilometer, it seems understandable and nice for me :]11:47
rick_h_czajkowski: looks like the report a bug link failure is fixed yay12:23
Ergo^rick_h_, when do you plan to add a human friendly interface to launchpad ? ;-)12:24
rick_h_Ergo^: heh, that's huw's job :P12:24
Bert_2Hi, I have setup an instance of launchpad with an actual domain (not launchpad.dev) and a clean db using wgrant's bootstrap-db-from-scratch and have added users using his user creationscript, now nor the admin nor regular users seem to be able to login into the subdomains of my instance (code, answers, blueprints), when I login with any account it redirects back to the page I came from but I'm still not logged in, does anyone know what'12:49
Bert_2s wrong ? Did I misconfigure something ?12:49
Ergo^gmb, ping13:07
deryckGood morning, all.13:11
abentleyderyck: morning.13:17
abentleyderyck: http://www.goodreads.com/quotes/14559-good-morning-said-bilbo-and-he-meant-it-the-sun13:18
=== Guest49605 is now known as slank
=== slank is now known as Guest96367
deryckabentley, heh.  Great passage.13:28
deryckI knew there was a reason I normally typed only "morning" :)13:28
abentleyderyck: Hehe.13:30
deryckadeuring, https://plus.google.com/hangouts/_/5c02c8b06ff0d06aaf9314c93315d217f5459977?authuser=0&hl=en13:33
Ergo^gmb, ping13:57
czajkowskiErgo^: he's not here. and won't be for two weeks14:00
Ergo^ah :]14:01
Ergo^rick_h_, so you say i should apply ? ;-)14:01
rick_h_Ergo^: give it a go and talk with them.14:01
deryckErgo^ is a friend of rick_h_?14:08
=== al-maisan is now known as almaisan-away
abentleyderyck: Currently, the mustache_model and the mustache_template from the server can be used by the javascript verbatim to render an identical result to the server-side rendering.15:49
abentleyderyck: This is because the mustache_model includes control variables that affect the rendering, even though they're not logically part of the model.15:49
deryckabentley, ok.  are you wanting to change this where only the template can be rendered?  Or something else?15:50
abentleyderyck: With the new pystache, we could remove all control variables from the mustache_model, which would be cleaner, but would break this property that the JS can use the model verbatim.15:51
deryckabentley, is there a downside to not using the model verbatim?15:52
abentleyderyck: The downside is it's a bit more work to get an identical rendering between client-side and server-side rendering.15:52
abentleyderyck: (but it's work we already do)15:53
deryckabentley, ah, I think I see.  We just do it via variables in the model, where as we need to specify those values at the call site now. right?15:54
abentleyderyck: Yes, we need to inject the control variables at the call site, if they're not in the data sent by the server.15:55
deryckabentley, sounds fine to me.  Seems cleaner on both ends to me.  i.e. model is cleaner and rendering is explicit.15:57
abentleyderyck: Cool.15:57
=== salgado is now known as salgado-lunch
Ergo^heh i think i should install ubuntu :/16:02
=== deryck is now known as deryck[lunch]
=== matsubara-afk is now known as matsubara
rick_h_doh, forgot to put myself as ocr17:20
=== rick_h_ changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: rick_h | Firefighting: - | Critical bugs: 4.0*10^2
rick_h_and since I am... jcsackett can I bug you please? https://code.launchpad.net/~rharding/launchpad/lpclient_fix/+merge/11633817:20
jcsackettrick_h_: sure.17:24
rick_h_jcsackett: ty much kind sir17:24
=== salgado-lunch is now known as salgado
=== Ursinha` is now known as Ursinha
jcsackettrick_h_: sorry, forgot to ping you when i finished reviewing your branch.19:05
=== Ursinha is now known as Guest69114
jcsackettit is r=me.19:05
rick_h_72Thanks it's off to ec2. thanks again!19:05
cody-somervilleWhy are some versions listed twice under 'Releases in Ubuntu'? https://launchpad.net/ubuntu/precise/+source/thunderbird19:32
Bert_2wgrant: are you here ?19:43
abentleyrick_h_: still around?20:00
rick_h_abentley: yep, what's up?20:01
abentleyrick_h_: Could you please review https://code.launchpad.net/~abentley/launchpad/restore-queue-test/+merge/116534 ?20:01
rick_h_abentley: loading up now20:02
abentleyrick_h_: Thanks.20:03
rick_h_abentley: what's task_init meant to do?20:05
rick_h_abentley: nvm, looking20:06
abentleyrick_h_: It sets up a launchpad scripting environment.20:06
rick_h_abentley: couple of questions (side notes) r=me ty much20:16
abentleyrick_h_: Thanks.'20:16
=== mbarnett` is now known as mbarnett
rick_h_jcsackett: ping20:35
rick_h_jcsackett: in your MP it says the 'helper method is removed' which I'm reading as userCanAccessUserData as being removed but it's not?20:35
Bert_2Does anyone know why I'm perfectly able to log in on the main section of my launchpad instance but not on the other sections like code and answers ? I don't get any errors but after supplying info to openid I just don't get logged in, any suggestions ?20:39
=== rick_h_ changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: - | Firefighting: - | Critical bugs: 4.0*10^2
=== salgado is now known as salgado-afk
sinzuiwgrant: StevenK: jcsackett: wallyworld: this is the non-published link of what I did today: http://youtu.be/Vq6VsKuFY_o21:47
wgrantBert_2: Did you set cookie_domains to include your new domain?22:04
wgrantBert_2: That controls how wildcardy the cookies are22:04
sinzuiwallyworld: you are proposing to fix 3 or more of thses bugs https://bugs.launchpad.net/launchpad/+bugs?field.tag=bug-links22:35
sinzuiwallyworld: http://blog.launchpad.net/general/bug-linking-part-2 "Linking existing bugs"22:38
Bert_2wgrant: where can I set the cookie_domains ?22:46
wgrantBert_2: the development config inherits the default value from lib/lp/services/config/schema-lazr.conf. You can override it in your own config.22:47
wgrant# Session cookies being sent to a subdomain of the parent22:47
wgrant# domains listed here will be sent to the parent domain,22:47
wgrant# allowing sessions to be shared between vhosts.22:47
wgrant# Domains should not start with a leading '.'.22:47
wgrantcookie_domains: demo.launchpad.net, staging.launchpad.net, launchpad.net, launchpad.dev22:47
Bert_2so I shouldn't edit it inside schema-lazr.conf ?22:47
wgrantProbably not. The configs form an inheritance hierarchy, so you can just override the key in yours22:48
Bert_2so I would override from launchpad-lazr.conf in my custom configsdir ?22:48
wgrantRight22:48
Bert_2awesome, thx a ton :D22:49
wgrantnp22:49
sinzuiwallyworld: StevenK, wgrant: this is the one I was thinking of https://bugs.launchpad.net/launchpad/+bug/100424822:52
_mup_Bug #1004248: hook to force page refresh on context url changing is not working <javascript> <regression> <ui> <Launchpad itself:In Progress by rharding> < https://launchpad.net/bugs/1004248 >22:52
rick_h_sinzui: something up with that?23:10
sinzuirick_h_: I was asking if a cache issue when a bug was marked a dupe related to that bug23:16
sinzuiThe conclusion was no23:16
rick_h_sinzui: ah ok cool. Wanted to make sure there wasn't anything up since I put that to ec2 today23:16
Bert_2wgrant: I just noticed that your bootstrap-db-from-scratch's bootstrap-lp-db script is missing the bug-watch-updater Celebrity, is that possible ?23:36
Bert_2(I have an oops page if you need more info)23:39
wgrantBert_2: Yeah, if you look in the script where it creates the celebrities, you'll see comments instead of makeTeam/makePerson calls for some of them. I mainly just created the essential/interesting ones.23:40
wgrantYou can alter the script or create the team manually to resolve it.23:40
wgrant    bazaar_experts = factory.makeTeam(23:40
wgrant        name='bazaar-experts', displayname='Bazaar Experts', owner=admins,23:40
wgrant        subscription_policy=TeamSubscriptionPolicy.RESTRICTED)23:40
wgrant    # bug_importer23:40
wgrant    # bug_watch_updater23:40
wgrant    buildd_admin = factory.makeTeam(23:40
wgrant        name='launchpad-buildd-admins',23:40
wgrant        displayname='Launchpad Builder Administrators', owner=admins,23:40
wgrant        subscription_policy=TeamSubscriptionPolicy.RESTRICTED)23:40
wgrantSome of the missing ones may be more essential now than they were when I wrote it.23:41
wgrantBut the intention was that it'd get the web UI to work enough that you could create the rest if you needed them.23:41
Bert_2I see, so I'm missing the bug-watch-updater person then ?23:42
Bert_2it's apparently required to file a bug ;)23:42
wgrantYeah. You can run the makePerson/makeTeam calls in bin/harness to fix that up23:43
wgrantIn bin/harness, say eg. "factory.makePerson(name='bug-watch-updater', displayname='Bug Watch Updater', email='bugwatch@example.com')" then "transaction.commit()"23:44
Bert_2if you're interested: http://pastebin.com/yFzyDNCU it was on a call to bugs.[maindomain]/[project]/+filebug23:44
wgrantAh right23:45
wgrantThat is indeed a relatively recent change.23:45
Bert_2wgrant: why should I add it to bin/harness and not eg. make a utilities file for it and just execute that ? (just wondering)23:45
wgrantWell, you can update the script to do it too, but if it's a one-off it's usually easier to do it in bin/harness23:46
wgrantrick_h_: awk: cannot open build/js/yui-3.3.0/yui/yui.js (No such file or directory)23:49
wgrantrick_h_: Any idea on that build error?23:49
wgrant$ ls -l build/js/yui-3.3.0/23:49
wgranttotal 023:49
wgrantlrwxrwxrwx 1 wgrant wgrant 30 2012-07-25 09:49 yui-3.3.0 -> ../../../../build/js/yui-3.3.023:49
wgrantIt's a recursive symlink..23:50
wgrantrick_h_: Also, your yui buildout changes still use a predictable /tmp name23:51
wgrantStevenK: Can you build a fresh tree?23:52
rick_h_wgrant: yes, sorry. While we talked about it, webops didn't have feedback on that part and so only changed that which was required.23:52
StevenKwgrant: My devel build a few minutes ago worked23:52
rick_h_wgrant: I can make a follow up that will adjust that dir.23:52
StevenKwgrant: I can clean and then do a fresh make?23:52
wgrantStevenK: Try make clean_buildout build23:52
rick_h_we were tossing stuff at the wall at the time23:52
wgrantHeh23:53
StevenKrick_h_: May I suggest you not do that with our Makefile?23:53
rick_h_wgrant: added a card to move that dir in the morning23:53
Bert_2wgrant: so bin/harness is meant for one-off scripts ?23:53
StevenKwgrant: make clean_buildout build worked for me23:53
wgrantStevenK: Hmm23:54
wgrantBert_2: It just gives you a Launchpaddy Python console23:54
rick_h_StevenK: heh, the sticking to the wall was guessing what issue my change had on qas not our makefile...and it's buildout.cfg now makefile :P23:54
Bert_2wgrant: I see23:54
StevenKrick_h_: Even worse, because buildout loves giving obtuse error messages.23:54
rick_h_StevenK: so think it's ok, will adjust the path to keep things inside of the build dir vs /tmp23:56
Bert_2wgrant: just so you know, same goes for bug-importer, apparently23:56
wgrantYup23:56
wgrantStevenK, rick_h_: Hm, even 'make clean build' breaks23:58
wgrantOh23:58
wgrantI don't have unzip installed23:58
wgrantIt's nice that buildout doesn't actually notice or display the error23:58
StevenK[09:54] < StevenK> rick_h_: Even worse, because buildout loves giving obtuse error messages.23:59
StevenKwgrant: ^23:59
wgrantStevenK: It also loves not giving any error message at all, though23:59
wgrantObtuse errors are fine :)23:59
Bert_2Does anyone know what the bugtask tasks and nominations table is ?23:59

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