/srv/irclogs.ubuntu.com/2011/02/18/#launchpad-yellow.txt

gary_postergmb, to your knowledge is there a way to get to https://qastaging.launchpad.net/~gary/+structural-subscriptions right now?13:09
gary_posterI mean, via a link exposed in Launchpad?13:09
gary_posterand hi btw :-)13:10
gmbgary_poster: Hi. Nope, you can only get there by URL hacking.13:10
gary_posterok cool thanks gmb13:10
gmbgary_poster: btw, I took on adding a mute button as it's a relatively simple change for me to tackle before the end of the week and won't block anyone if I don't finish it.13:11
gary_postergmb, sweet, thanks13:11
gary_posterheh, the kanban representation is a bit unorthodox, but whatever :-)13:11
gary_posterbac, benji, danilos, gmb, hi.  We'll do our delivery meeting after the standup (https://dev.launchpad.net/yellow#Due%202011-02-18).  If you need us to get a particular branch to show off your work (or some other preparation) please let us know beforehand.13:14
gary_posterdanilos, Benji and I talked yesterday about "Every structural subscription should have a filter."  He'll also need it for his work.  I said you'd be working on it, and lo and behold, you are. :-) He suggested that we could have a script that we could do this in three steps: 1) make a script that gradually adds a "include everything" filter to all structural subscriptions that do not have a filter, and get that13:21
gary_posterrunning on the production database. 2) get a devel branch that makes a filter whenever you create a structural subscription (i.e., enforced in Python).  3) make a db patch that enforces this for next db rollout.  That sounded like a nice plan, because it would mean that we could rely on a filters being around after step 2 was rolled out and step one reported that it didn't see any more structural subscriptions13:21
gary_posterwithout filters (and before a db rollout, which would help him).  So I pass that along for your consideration, if you haven't thought of it already.13:21
danilosgary_poster, we have only around 23k StructuralSubscriptions atm, so we can create matching BugSubscriptionFilters during a DB rollout as well13:22
danilosgary_poster, also, just having a BugSubscriptionFilter is enough for it to be all-inclusive (i.e. no need to set it to all statuses or something like that)13:23
danilosgary_poster, so, I already have a branch where I am doing 1 and 2, and I thought about enforcing it in the DB directly, but that kinda gets ugly (multi-table constraints are ugly)13:24
gary_posterdanilos, ok.  The only advantage of the other plan is that we don't have to be blocked by a db rollout for work on bug 674422.  Ack and agree on just having a BugSubscriptionFilter13:24
_mup_Bug #674422: Add a workflow to update StructuralSubscriptions <lp-bugs> <story-better-bug-notification> <Launchpad itself:Triaged by yellow> < https://launchpad.net/bugs/674422 >13:24
bacgary_poster: my branch is at lp:~bac/launchpad/accordionoverlay -- i added it to the wiki page along with a screenshot13:24
danilosgary_poster, I can still land it on devel, sure :)13:24
gary_posterdanilos: ack and awesome, sounds perfect13:24
* gary_poster has not written multi-table constraints so was speaking naively13:25
gary_posterbac, awesome thank you13:25
danilosgary_poster, as for the migration for existing data, we can ask for a manual DB query run as well13:25
gary_posterdanilos, you don't think it would produce a particularly long transaction?13:25
danilosgary_poster, it'd be best to check it, I'll prepare a query and have it tested on staging13:26
gary_postercool thanks danilos13:26
danilosnp13:26
gary_posterbac, benji, danilos, gmb, apologies for spamming you with IRC messages today, but here's the usual reminder: mumble/kanban in 213:28
benjiI like reminders.13:28
gary_poster:-)13:28
bac+113:28
daniloslib/lp/bugs/javascript/bugtask_index.js13:45
daniloslib/lp/bugs/javascript/bugtask_index_portlets.js13:46
bacgmb, benji: I'd like to have a mid-imp sanity-check call today at your conveniences14:01
benjibac: cool, I'll get with you after this call14:02
gmbbac: I'll be available at the same time as benji14:02
daniloshum, any suggestions for a catch-all BugSubscriptionFilter.description that we'll be creating? I am thinking 'Default', 'Catch-all' or empty14:26
danilos(Catch-all might be inappropriate if for the first actual filter we reuse this one)14:26
danilosgary_poster, fwiw, a transaction on staging to insert 23k BugSubscriptionFilters took 3.5s (iow, a full "migration"), so I believe we can do it live on the production DB as well14:28
gary_posterdanilos OK awesome.  I suspect we need to clear it with lifeless and/or stub?14:31
danilosgary_poster, yeah, we'll need to clear it with a TL, and I'd definitely like to have stub look it over and confirm replication won't have trouble catching up either14:32
danilosgary_poster, at least it was previously a policy for a TL to be able to approve queries :)14:33
danilosgary_poster, so it can be you as well :)14:33
gary_posterdanilos, ok, I'm happy to give it a TL OK if stub approves. :-)14:33
danilosgary_poster, excellent, thanks14:33
danilosgary_poster, the only thing I need to come up with is the BugSubscriptionFilter.description value we want these entries to have14:34
danilosbenji, considering you had a need for this as well, any opinion? :)14:34
gary_poster...I'm inclined to not put in a description (or empty string if necessary)14:35
gary_posterbecause that's how a subscription will look like be default when you add one manually14:35
gary_posterIIUC14:35
gary_posterdanilos, so you see any issues with that?14:36
danilosgary_poster, no, it's one of the options I considered as well, the only worry I have is that we'll have to make sure our implementation reuses this filter when a first "real" filter is created (or people will get all emails without realizing it is due to this catch-all filter)14:37
danilosgary_poster, though, we'd want to make sure our implementation does that for more than this reason, so I don't see it as a real issue14:38
gary_posterprecisely, danilos14:40
bachi benji, gmb: i'm having some issues on my desktop.  if you proposed a meeting time i missed it.14:56
benjibac: any time14:56
bacbenji: ok, let's wait for gmb14:57
benjik14:57
gmbI'm free any time too.14:57
gmbLet me re-headphone myself and I'll join you on mumble14:58
gmbbac, benji : Ready when you are14:58
bacgmb: can you grab that branch i posted earlier on the wiki?14:59
gmbbac: The accordionoverlay branch?14:59
bacyes14:59
gary_posterhey gmb, I thought malone-alpha team people would have the new JS widget on production now, but I don't see it.  Is that expected?15:16
gmbHrm.15:16
* gmb looks15:16
gary_posteryou are probably on call--no worries, reply when you can15:16
gmbI'm not, actually15:16
gmbSurprisingly :)15:16
gary_posteroh ok cool15:16
gary_poster:-)15:16
gmbgary_poster: Hmm. WFM.15:17
gary_posteroh!15:17
gmbLet me check your membership of malone-alpha...15:17
gary_posterI'm a member of https://launchpad.net/~malone-alpha15:17
gmbYeah.15:18
gmbgary_poster: What page are you looking at that you're not seeing the overlay on?15:18
gmbMaybe it's a jS problem.15:18
gary_posterMaybe it's user problem.  https://launchpad.net/lazr.restful , clicking "Subscribe to bug mail...oh!!! duh. :-P15:19
gary_posterThere we go15:19
gary_posterdirect subscriptions work finr15:19
gary_posterfine15:19
gary_postersorry :-P15:19
gmbgary_poster: Yeah, it's bac and benji who are on the hook for the other one :)15:19
gary_poster:-)15:19
gary_postercool, thanks gmb15:20
gmbnp15:20
gary_postergmb, have we announced to malone-alpha people?  Should I do that next week or today or something?15:20
gmbgary_poster: I meant to do it earlier this week but mockups ate most of my time. If you could announce it that would be awesome.15:20
gary_postercool, will do.15:21
gmbThanks15:21
gary_posternp15:21
gary_posterdanilos, with your work for bug 720826, it will be trivial to then have the email link back to "unsubscribe" include information on what filters caused the email to be sent, right?  The information will already have to be available.15:39
_mup_Bug #720826: Add subscription description header for bug notifications <story-better-bug-notification> <Launchpad itself:In Progress by danilo> < https://launchpad.net/bugs/720826 >15:39
danilosgary_poster, that's what I expect, yes15:39
gary_postercool thanks danilos15:40
danilosgary_poster, i.e. I will be introducing a link to the actual BugSubscriptionFilter on BugNotification table15:40
gary_posterdanilos, I was wondering about that, only one?15:40
gary_postermay need to be many to one15:40
gary_posterone way r another15:41
gary_posteror15:41
danilosgary_poster, well, each notification is for one action, so when we construct the email, we should worry about that side15:41
danilosgary_poster, one email includes multiple BugNotifications, so that's how you get one-to-many15:41
gary_posterdanilos, but an action may have been sent because of multiple filters15:41
gary_posterI'm not sure we have to worry about this, but it should be a conscious decision not to15:41
danilosgary_poster, hum, right, good point15:41
danilosgary_poster, I am not going to even start on that today (I hope only to finish auto-creation of BugSubscriptionFilter by the EOD)15:42
gary_posterdanilos, cool, I won't stop you, but it can be in the back of your head for Monday15:42
danilosgary_poster, right, I'll have to think about it, though I am pretty sure that we'd have to restructure our code for that as well (i.e. maybe a notification gets sent atm when a first filter is matched), but I don't expect that to be a big deal15:44
gary_posterdanilos, yeah, was thinking similarly, though I was merely *hoping* it was not going to be a big deal ;-)15:45
danilosgary_poster, heh :)15:46
gary_posterbenji, I have this in my notes from this morning: " * We Should Do Something about mentioning that request/features always works in templates, and features only sometimes works."  Can I change that to "Benji will announce or document that request/features always works in templates, and features only sometimes works, and propose that request/features be the One True Way."?15:56
gary_posterMay I change that. :-)15:57
benjigary_poster: sure; I'll do so by replying to Marin Pool's recent documentation effort email.15:58
gary_postercool thanks benji15:59
* benji lunches.16:02
gary_postergmb, unless you mind, I'm gonna move the kanban card for bug 204980 into quick jobs, so feature work 1 is more focussed. I'm then going to add a card for an API to get a person's structural subscriptions for a bug's tasks, as we described on the call.  Can I move the two mockup related cards in feature work 1/review over to Done/Done?16:25
_mup_Bug #204980: bug contacts should be able to unsubscribe from implicit subscriptions <canonical-losa-lp> <lp-bugs> <motu> <story-better-bug-notification> <ubuntu-upstream-relations> <Launchpad itself:In Progress by gmb> < https://launchpad.net/bugs/204980 >16:25
gary_posteroh and another hand-off: can you make sure that we know how to qa the two cards in QA/landing?16:26
gary_poster(the two cards of yours, I mean)16:26
gmbgary_poster: In order: 1) Well, you can move it, but I don't think it's a quick job after all (will explain in an email, but basically it breaks things in an intereting fashion). 2) Okay, 3) Sure, 4) Certainly.16:27
gary_poster1,huh, ok16:27
gary_posterand the rest, ok thanks :-)16:27
gmbgary_poster: Quick summary of problem #1: All the JS works fine, but Zope subsequently says "LOLWUT?" when you've muted a bug, due to the validation we do for the +subscribe page.16:28
gary_posterOh! So, you have a nice way to turn everything on and off, but the muting itself is a problem16:28
gmbRight.16:28
gary_posterI see. :-/16:29
gmbBUT.16:29
gmbI think I can see ways through it.16:29
gmbSo I'll file bugs (oh for bug relationships) about that and summarise the order I think things need to be addressed in an email to the team.16:29
gary_postercool thanks gmb.  um...I wonder what we should do about the janban board.16:30
gary_posteror kanban16:30
gmbgary_poster: Well, can we move that card back to the backlog without breaking stats or something?16:30
gary_posterno not really :-P :-)16:30
gmbAh.16:30
gmbBother.16:30
gmbgary_poster: Well, let's do this:16:30
gmb1. Update the card to reflect that what I'm *actually* going to do today is land incremental but non-user-impacty stuff16:31
gary_posterI was going to suggest we delete it and remake it, or change it to "investigate" and move it to archive/done16:31
gmbOr that.16:31
gmbThat seems simpler16:31
gmbSimple wins.16:31
gmbgary_poster: I'll do the latter.16:31
gary_posterok cool16:32
gary_posterthat sounds like a reasonable compromise between easy and reflecting reality16:32
gmbRight.16:32
gary_poster(change it into a task card too please, and remove the bug number or the kanban board will whine and complain)16:32
gary_poster(or at last it does in my experience, when you try to make a new one with the same number)16:33
gmbDone16:33
gary_postercool thanks16:33
gary_posterthen if you could claim a backlog lane for the tasks/bugs for this, with a new feature card for the bug number, that would be fab16:34
gmbSure, will do.16:34
* gmb -> afk for half an hour or so to braindump my thoughts about hte mute button16:42
gary_postercool16:42
* gary_poster lunching17:08
gary_posterhttps://wiki.canonical.com/Launchpad/Strategy/2011RelCalDraft <- maybe useful17:09
gary_posterMight become https://dev.launchpad.net/Releases/2011Calendar17:09
gmbgary_poster: Brain dump: https://dev.launchpad.net/yellow/MuteButton. Filing bugs now.17:30
* gary_poster no longer lunching, now reading. Thanks18:18
gary_postergmb, reading that page, one of the first thoughts I had was "I wonder if the enumeration is the wrong way to express muting"18:21
gmbgary_poster: Possibly. I can't remember what we said we were going to do with the enumeration on direct subscriptions.18:21
gary_posterA BugMuteTable that contains records of person, bug for thinsg that should be muted might be better18:21
gmbI suspect so. My thinking is couched in terms of years of "oh, we used to have Ignore Subscriptions" cruft.18:22
gmbThe BugMute table solves more problems. OTOH, it needs yet another DB patch, but maybe that's not so terrible.18:22
danilosmy branch for always creating a BugSubscriptionFilter when a SS is created is up for review, though I'm about to leave for the week18:23
gary_posterAt this point, I think that's a sunk cost18:23
danilosgary_poster, fyi ^18:23
gmbTrue.18:24
gary_posterdanilos, ok cool.  Do you want me to try and land it if the review flies by?18:24
danilosgary_poster, query already approved by Stuart, and I'll have it added as a special rollout requirement18:24
gary_postergreat danilos18:24
danilosgary_poster, yeah, that'd be nice, I am putting it through ec2 test anyway (already under way)18:24
gary_postercool danilos.18:25
gmbgary_poster: I'm sanguine about going either way with muted subscriptions. Simpler is better, and BugMute seems simpler.18:25
daniloscheers, enjoy the weekend all18:25
gary_postercheers danilos18:25
gary_postergmb, are you still here for another 34, or are you 26 over?18:26
gmbgary_poster: I'm 26 over and about to head down for dinner.18:26
gary_posterOK cool, then I won't bother you18:26
gary_posterHave great vacation!18:26
gmbWill do, thanks.18:26
* gmb -> exeunt, in pursuit of not thinking about JavaScript18:27
gary_poster:-)18:27
gary_posterI've decided to take Monday off, so I'll see y'all Tuesday.  Have a great weekend.22:18

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