/srv/irclogs.ubuntu.com/2012/11/30/#launchpad-dev.txt

SpamapSSo, before I run off experimenting with things.. say I want to use the django-openid-auth thing just for openid, but then I want to check for membership in a private team by doing LP calls on login. Has somebody already done this?00:03
mwhudsonSpamapS: pretty sure that's doable00:10
mwhudsonSpamapS: you'll need to get your RP specifically whitelisted to be able to ask about private teams00:10
=== slank is now known as slank_away
rick_h_StevenK: you know what's up with build-not?02:35
StevenKrick_h_: Huh?02:43
=== Ursinha is now known as Ursinha-afk
=== Ursinha-afk is now known as Ursinha
rick_h_StevenK: buildbot is in a strange place with the exception stuff?02:50
rick_h_ah looks like someone did another force02:50
rick_h_I hadn't seen buildbot blow up in that way so was curious if you had any idea what's up.02:51
rick_h_both my ec2 lands bounced due to pqm buildbot fail02:51
StevenKrick_h_: subunit stream corruption02:51
StevenKrick_h_: And yes, I forced02:55
rick_h_ok cool, thanks02:55
=== Ursinha is now known as Ursinha-afk
=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away
=== Ursinha-afk is now known as Ursinha
=== almaisan-away is now known as al-maisan
adeuringgood morning09:05
czajkowskiadeuring: ello09:08
adeuringhi czajkowski!09:08
czajkowskiadeuring: is it cold in your neck of the woods today ?09:09
adeuringczajkowski: depends on how you define cold ;) 5C and fotunatley no rain09:10
czajkowskiwell no rain is good!09:14
czajkowskiit's 3C here and cold!09:15
stubcome on over, the weather is lovely09:18
czajkowskistub: are you in AU or Nz?09:19
stubThailand09:20
czajkowskioh even further09:20
stubno, closer09:20
czajkowskistub: ah yes but ye have the weird rain weather over there09:20
czajkowskiI want to avoid places with rain and snow. clearly moving to UK from ireland I didn't think this one through :)09:20
stubmonsoon storms are awesome09:21
czajkowskihttp://www.irishpressreleases.ie/content/flooded%20house.6885.jpg  is what happens to irelan when it floods!09:22
czajkowskior http://news.bbcimg.co.uk/media/images/61218000/jpg/_61218905_jex_1449008_de27-1.jpg09:22
stubWhen Thailand floods, we cause worldwide harddrive shortages :)09:22
czajkowskilol09:23
stubAnyone here on maintenance?09:23
czajkowskiit's ok we still ge tour cider and guinness exported, we have our priorities! :)09:23
czajkowskistub: StevenK if you can catch him09:24
czajkowskistub: and we're down to one person for next week09:24
czajkowskiin USA timezone09:24
stubwho is that?09:24
czajkowskistub: sinzui09:25
=== al-maisan is now known as almaisan-away
=== _mup__ is now known as _mup_
deryckMorning14:00
rick_h_morn14:01
sinzuistub, I saw your remarks on bug 1050191. I agree with your suggestions. I will make this my next issue to work on14:17
_mup_Bug #1050191: allocate-revision-karma.py is too slow to complete <branches> <karma> <lp-code> <oops> <Launchpad itself:Triaged> < https://launchpad.net/bugs/1050191 >14:17
stubsinzui: ok. The code changes are trivial if I'm right. Test fallout likely worse.14:18
sinzuiindeed14:18
stubmoving to garbo would be nice, but that is more than maintenance14:19
sinzuistub, I was thinking the same. So you agree that this process looks like a garbo candidate. I will consider it.14:20
stubit is perfectly suited to be a garbo job. migrating tests will again be the issue :)14:21
=== slank_away is now known as slank
=== bac changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On-call reviewer: bac | Firefighting: - | Critical bugs: ~170
deryckadeuring, here's the MP I need reviewed, https://code.launchpad.net/~deryck/launchpad/remove-product-info-typo-garbo/+merge/13721015:04
adeuringderyck: already looking at it :)15:04
deryckadeuring, ah thanks :)15:04
adeuringderyck: r=me, one nitpick15:08
deryckadeuring, thanks15:08
deryckadeuring, good catch, thanks15:08
=== almaisan-away is now known as al-maisan
sinzuirick_h_, https://bugs.launchpad.net/launchpad/+bug/1079678 is still a private-project issue because Lp has to prevent the object from being created...other wise Lp's db has a integrity issue. Question cannot be deleted.15:26
_mup_Bug #1079678: You're able to create new questions on Launchpad even when questions is not avilable to a project <distributions> <lp-answers> <projects> <ui> <Launchpad itself:Triaged> < https://launchpad.net/bugs/1079678 >15:26
sinzuiI think someone also needs to verify that questions cannot be retargeted to a proprietary project15:27
=== matsubara is now known as matsubara-lunch
=== yofel_ is now known as yofel
=== al-maisan is now known as almaisan-away
rick_h_sinzui: ah, that makes sense.16:26
rick_h_sinzui: hmm, I thought that was checked though. I remember some work around the sprint stuff before UDS around that16:27
rick_h_hmm, never mind, mixing up blueprints and questinos16:31
rick_h_sinzui: so ok. I'll add a card/ticket around validating the question so that it can't be targetted around a non-public project16:33
deryckrick_h_: so I agree with sinzui on that issue.  but retargeting is separate from people URL hacking, which can be done on any kind of project....16:34
rick_h_deryck: right, so I'm filing a second bug that we'll address16:34
rick_h_and then we can decide if the right fix for the url hack is to catch it at the view when the form is loaded, or as a validation error like this retargetting issue later16:34
=== matsubara-lunch is now known as matsubara
deryckrick_h_, sinzui -- and I still stand by my thinking that the first issue is nothing special because of privacy.  it's just a bug, and not dangerous for privacy.16:34
sinzuirick_h_, maybe the solution is to ensure the vocab for question targets only contains Public information type projects. That would allow the existing validation processes to co unchanged16:35
derycki.e. yes, it creates bad data, but it doesn't risk a leak, so we don't care.  it also creates bad data for public projects.16:35
rick_h_sinzui: yea, I've not delt with the questions to know the best implementation for it, but would assume it would be a simple fix in the model perhaps.16:36
sinzuideryck, It was agreed that questions don't make sense and they DO contain confidential information that we do not have infrastructure to clean16:36
sinzuideryck, That is why we ask commercial enquiries to use an email address instead of Lp answers. We could not delete the confidential information in the question16:37
derycksinzui: I agree they don't make sense.  but I don't see how this bug relates to private projects.  what confidential info are you thinking?16:37
derycksinzui: I'm sorry man, I'm lost.  How does using an email address and being able to url hack to file a question relate?16:38
sinzuideryck, more than 100 hardening/transition bugs blocked Lp from implementing safe sharing. This is the same case, Once users discover that Lp is not trustable, you lost and cost someone and NDA16:38
rick_h_I suppose there's the use case where someone submits a question on a non-public project via the url hack and it contains sensitive data16:38
rick_h_if things ever do go public, and questions are turned on16:39
rick_h_it'd be exposed16:39
sinzuiyes, that was what the issue was about16:39
rick_h_sinzui: ok, from the comments it was more czajkowski wanted to make sure users didn't get ignored16:39
rick_h_sinzui: so it wasn't 100% clear at first16:40
sinzuiCanonical has partners with NDA information spewing from their keyboards into projects that are working in secret16:40
rick_h_however, if we fix the model so that you can't create/target to a non-public project the potential leak issue goes away16:40
deryckbut do those people have permission on private projects AND know how to url hack to file a question?16:40
rick_h_so I think the *right* bug for us to fix and work on is #1079678 and #1079678 is a more general bad UX bug16:41
_mup_Bug #1079678: You're able to create new questions on Launchpad even when questions is not avilable to a project <distributions> <lp-answers> <projects> <ui> <Launchpad itself:Triaged> < https://launchpad.net/bugs/1079678 >16:41
_mup_Bug #1079678: You're able to create new questions on Launchpad even when questions is not avilable to a project <distributions> <lp-answers> <projects> <ui> <Launchpad itself:Triaged> < https://launchpad.net/bugs/1079678 >16:41
sinzuiLaura's issue one example of a root cause of bad data getting into a project16:41
deryckI absolutely do not agree this warrants our precious time left until Monday, but I'm not going to fight about it.  I just don't see any real risk.16:43
czajkowskire that bug I logged it not for a privacy issue, but more for the point people create answers on projects and they are not looked at. perhaps the people creating private projects and blueprints arent possibly going to be creating answers on lp16:46
deryckThat's my point, czajkowski.  It is absolutely a bug.  And it should be fixed.  I don't think it's risky or even related to privacy, and don't think we should agree to fix it before we release private projects from beta.16:48
czajkowskideryck: that I don't know :) would there be a case if people didn't know it was turned off and if they were automatcally used to asking questions they could then.  I guess flacoste would have final say on it.  I just see it happening from time to time on projects on here16:51
czajkowskiwhich was why I logged it16:51
deryckczajkowski: proprietary and embargoed projects are not the kind to file questions against anyway.  they're secret.  and I can see why anyone would accidentally do it because there's no use case for asking a question.16:54
deryckbut talking to rick_h_ in another channel, his fix for the retargeting issue will fix this as a by product, so no point arguing it anymore.16:54
deryckrick_h_: what is the bug number of the new bug you filed?16:55
rick_h_#107967816:55
_mup_Bug #1079678: You're able to create new questions on Launchpad even when questions is not avilable to a project <distributions> <lp-answers> <projects> <ui> <Launchpad itself:Triaged> < https://launchpad.net/bugs/1079678 >16:55
rick_h_sorry, wrong one16:56
rick_h_#107967816:56
_mup_Bug #1079678: You're able to create new questions on Launchpad even when questions is not avilable to a project <distributions> <lp-answers> <projects> <ui> <Launchpad itself:Triaged> < https://launchpad.net/bugs/1079678 >16:56
rick_h_ugh, /me fail16:56
deryckheh16:56
rick_h_#108510216:57
_mup_Bug #1085102: A non-public project can currently be the target of a question. <lp-answers> <private-projects> <Launchpad itself:Triaged> < https://launchpad.net/bugs/1085102 >16:57
rick_h_there we go16:57
rick_h_card updated on the board as well.16:57
deryckrick_h_: thanks.  I reply in the original bug.16:58
=== cyclicflux is now known as Guest57501
=== cyclicflux is now known as Guest80149
=== deryck is now known as deryck[lunch]
=== deryck[lunch] is now known as deryck
sinzuibac, do you have time to review https://code.launchpad.net/~sinzui/launchpad/system-bug-comments-api/+merge/13730820:02
bacsinzui: i do20:02
bacsinzui: done20:06
sinzuithank you bac20:07
bacde nada20:07
abentleybac: Could you please review https://code.launchpad.net/~abentley/launchpad/no-private-translations/+merge/137316 ?20:21
bacabentley: on it20:21
bacabentley: in the error messages you alternately refer to "private projects" and "proprietary projects".  should you pick the one most common description -- or are you really trying to convey something different?20:41
abentleybac: Okay.  (We don't really have a good umbrella term for Proprietary and Embargoed products, because "private" means something else in the context of branches and bugs.)20:44
abentleybac: So that's my own dissatisfaction with either term showing up in the code.20:45
bacabentley: gotcha20:45
bachi statik!20:45
statiko/ bac20:46
lifelessstatik: o/20:50
lifelessand o/ everyone here ;)20:50
=== bac changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On-call reviewer: - | Firefighting: - | Critical bugs: ~170
=== almaisan-away is now known as al-maisan
jcsackettderyck: you around? can i request a review of https://code.launchpad.net/~jcsackett/launchpad/blueprint-private-traversal/+merge/13733021:40
jcsackett(or abentley or rick_h_) ^21:40
deryckjcsackett: I can get it.21:40
jcsackettawesome. thanks deryck.21:40
deryckjcsackett: np21:41
=== al-maisan is now known as almaisan-away
deryckjcsackett: am I reading the diff right, was there really a pdb in this code that your code removes?21:48
* jcsackett looks21:49
jcsackettderyck: it's pipe madness. let me resubmit with the right target.21:49
deryckjcsackett: ok, that's why I asked.  I didn't mind a missing pdb mind you. :)  But just wanted to make sure the diff was correct.21:50
jcsackettderyck: if you want the fully proper diff: https://code.launchpad.net/~jcsackett/launchpad/blueprint-private-traversal/+merge/13733321:50
jcsackettthought i had fully destroyed that pipe. clearly not. :-P21:50
deryckjcsackett: heh, np.  thanks.21:51
deryckjcsackett: looks great to me. r=me22:03
jcsackettderyck: thanks!22:03
derycknp22:03
=== Ursinha is now known as Ursinha-afk
=== matsubara is now known as matsubara-afk
=== Ursinha-afk is now known as Ursinha

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