[13:22] <gary_poster> bac benji danilos gmb, aloha.  slight change to morning meeting call: please have kanban board up-to-date 5 min before call (3 min from now).  I will review before call, and just ask questions about things I wonder about.  Then I'll circle through everyone once and ask if they have anthing to add, then we're done.
[13:22] <gmb> ok
[13:22] <gary_poster> gmb, hey, how goes the db war?
[13:23] <gary_poster> oh, also, I'm going to use Skype
[13:23] <gary_poster> barring any objections
[13:23] <bac`> none from me
[13:23] <gmb> gary_poster: It turned out to be a one-line fix. I had to tell Person.merge() to ignore BugSubscriptionFilterMutes when merging. That wasn't obvious from the failures though. I went from having 30-odd test failures to 0
[13:23] <gary_poster> cool
[13:23] <gmb> Skype works for me.
[13:23] <gary_poster> heh
[13:23] <gary_poster> ok cool
[13:24] <gary_poster> gmb, did you happen to merge db-devel or something similar (given the upcoming deploy, maybe devel already has db-devel stuff)?
[13:25]  * gary_poster can't qa the bits I need to 'cause qastaging is down. :-/
[13:25] <gmb> gary_poster: No. I fixed the failures and pushed and ran. When's the DB deploy? Wednesday AM?
[13:26] <gary_poster> gmb, yeah, I think so.  I saw that devel is in RC until qa is finished
[13:26] <gary_poster> my interest is in making an MP though
[13:26] <gmb> Ah.
[13:26] <gmb> gary_poster: I can merge db-devel and push now for you.
[13:26] <gary_poster> I have necessary changes both from your branch and from trunk-ish
[13:26] <gary_poster> awesome thank you
[13:27]  * gary_poster starting to stare an kanban board...
[13:28] <gary_poster> yay, my stuff is qa'd by wgrant!
[13:30] <gary_poster> danilos, benji, skype plz?
[13:30] <danilos> gary_poster, ack
[13:30] <gary_poster> thanks
[13:30] <gmb> gary_poster: lp:~gmb/launchpad/team-subscription-opt-out and lp:~gmb/launchpad/team-subscription-opt-out-apis now contain db-devel.
[13:31] <gary_poster> yay, thank you gmb
[13:31] <benji> gary_poster: there now
[13:31] <gmb> AAAAH.
[13:32] <gmb> gary_poster: Please try again.
[13:32] <danilos> gary_poster, I am here
[13:40] <bac> benji: skype?
[13:40] <benji> bac: sure
[13:41] <bac> benji: what is your skype id?
[13:41] <benji> bac: um... let me see
[13:41] <benji> bac: benji_york
[13:41] <gary_poster> benji_york
[13:55] <danilos> bac, benji, gary_poster, gmb: fwiw, my refactoring branch does change the structural_subscriptions.js quite a bit, at least for the setup code
[13:55] <bac> danilos: ok.  shouldn't affect my work
[13:55] <gary_poster> danilos, yeah I was afraid of that. :-) I have a branch thst changes JS
[13:55] <gary_poster> I'll figure out what to do about it in a few minutes
[13:57] <danilos> gary_poster, I did a lot of sanity refactoring and got rid of that context-passing-around hack that we had as well in the process
[13:57] <gary_poster> danilos, cool
[13:57] <danilos> gary_poster, I am hoping nobody gets affected too much though
[13:57] <gary_poster> :-) I'll see.  it was good stuff to do anyway.
[13:59] <gmb> gary_poster: I just submitted AWS expenses for March. Can you sign them off when you get a chance?
[13:59] <gary_poster> will do gmb, thanks for heads up
[14:00] <gmb> np
[14:00] <gmb> Thanks
[14:04] <bac> feature flag enabled for ~yellow on staging
[14:06] <gary_poster> gmb, approved expenses.  https://code.launchpad.net/~gary/launchpad/muteteamsub-email/+merge/56154 is now ready when you are.
[14:06] <gmb> Ooo, a trade of web forms.
[14:06] <gmb> On it.
[14:06] <gary_poster> :-) thanks
[14:07] <danilos> gary_poster, oh, I've filed the amazon ec2 expense as well (just now so hoping it doesn't transfer into next month :)
[14:07] <gary_poster> :-) k looking
[14:09] <gary_poster> approved danilos.  I'm thinking about what I should do to handle your changes.  Maybe I should merge your refactor branch now?  Or would you suggest I wait until later for some reason?
[14:10] <danilos> gary_poster, I'd be very happy if you branch now, I don't expect big changes there (and if I do, I'd do it as a separate branch)
[14:10] <danilos> gary_poster, what particular bits are you changing?
[14:11] <danilos> gary_poster, it's for muting/unmuting links on the subscriptions list?
[14:11] <gary_poster> right
[14:11] <gary_poster> I doubt that will be affected much but
[14:11] <gary_poster> I wonder if the tests might be
[14:11] <gary_poster> (which I started, and then stopped because I wondered if I might be stomping on you)
[14:12] <danilos> gary_poster, I didn't change any tests so far (they all pass, though), and I fully expect to have to pretty soon now since I did split it all up into smaller functions
[14:13] <danilos> gary_poster, iow, I only expect to be introducing new tests, and my changes are only from make_edit_handler and "below"
[14:13] <gary_poster> danilos.  ack.  One exciting bit is that there are no tests for the edit form rendering, as you probably know
[14:13] <gary_poster> right, cool
[14:13] <danilos> gary_poster, yeah, and I think my refactor should help with some of that when we get to it
[14:14] <gary_poster> ok, but since your changes are make_edit_handler and below...
[14:14] <gary_poster> then you won't be writing tests for edit stuff now?
[14:14] <gary_poster> So I can?
[14:14] <gary_poster> (without fear of duplicating work)
[14:15] <danilos> gary_poster, well, "only below" is probably not completely correct, but I am not doing any tests yet, no
[14:15] <gary_poster> ok
[14:15] <danilos> gary_poster, if you are planning to do tests, however, I strongly suggest you merge my branch first
[14:15] <gary_poster> then I'll start working on thos in just a few.  I have other stuff I should get to first
[14:15] <danilos> ack
[14:16] <gary_poster> ok, danilos, will do.
[14:40] <gmb> gary_poster: Your branch is r=me.
[14:41] <gary_poster> cool, thanks gmb.  I have something else for your attention in just 1 sec...
[14:41] <gmb> ok
[14:47] <gary_poster> gmb, please take a glance at https://dev.launchpad.net/LEP/BetterBugSubscriptionsAndNotifications/FeatureReviewNotes
[14:47] <gary_poster> You might be able to fill in a couple fo XXXs (or remove them as appropriate)
[14:49] <gmb> Ok, will do.
[15:12] <gmb> gary_poster: I've removed a couple of XXXs and left the rest (that I didn't know much about) alone.
[15:13] <gmb> gary_poster: Should I take a look at the Bug 1 muting timeout problem?
[15:13] <_mup_> Bug #1: Microsoft has a majority market share <iso-testing> <ubuntu> <Clubdistro:Confirmed> <Computer Science Ubuntu:Invalid by compscibuntu-bugs> <EasyPeasy Overview:Invalid by ramvi> <GNOME Screensaver:Won't Fix> <Ichthux:Invalid by raphink> <JAK LINUX:Invalid> <The Linux OS Project:In Progress> <metacity:In Progress> <OpenOffice:In Progress by lh-maviya> <Tabuntu:Invalid by tinarussell> <Tivion:Invalid by shakaran> <Tv-Player:New> <Ubunt
[15:15] <gary_poster> gmb, ...sure.
[15:16] <gary_poster> and thank you
[15:16] <gmb> np
[15:19] <gary_poster> danilos, just merged.  18 conflicts. ;-) will work on resolving in a few.
[15:19] <gary_poster> (18 all within the one js file)
[15:19] <gary_poster> most are very small
[15:19] <gary_poster> but some not so much :-P
[15:20] <gary_poster> danilos, nice yellow sub btw :-)
[15:21] <gary_poster> ...not clear on what triggered mup to get excited there...
[15:23] <benji> gary_poster: Bug 1 was mentioned earlier.  I wonder if the general heft of Bug 1 makes mup take a long time to do its thing.
[15:23] <_mup_> Bug #1: Microsoft has a majority market share <iso-testing> <ubuntu> <Clubdistro:Confirmed> <Computer Science Ubuntu:Invalid by compscibuntu-bugs> <EasyPeasy Overview:Invalid by ramvi> <GNOME Screensaver:Won't Fix> <Ichthux:Invalid by raphink> <JAK LINUX:Invalid> <The Linux OS Project:In Progress> <metacity:In Progress> <OpenOffice:In Progress by lh-maviya> <Tabuntu:Invalid by tinarussell> <Tivion:Invalid by shakaran> <Tv-Player:New> <Ubunt
[15:23] <_mup_> Bug #1: Microsoft has a majority market share <iso-testing> <ubuntu> <Clubdistro:Confirmed> <Computer Science Ubuntu:Invalid by compscibuntu-bugs> <EasyPeasy Overview:Invalid by ramvi> <GNOME Screensaver:Won't Fix> <Ichthux:Invalid by raphink> <JAK LINUX:Invalid> <The Linux OS Project:In Progress> <metacity:In Progress> <OpenOffice:In Progress by lh-maviya> <Tabuntu:Invalid by tinarussell> <Tivion:Invalid by shakaran> <Tv-Player:New> <Ubunt
[15:23] <benji> I guess I can DOS mup by mentioning Bug 1 a lot.
[15:23] <_mup_> Bug #1: Microsoft has a majority market share <iso-testing> <ubuntu> <Clubdistro:Confirmed> <Computer Science Ubuntu:Invalid by compscibuntu-bugs> <EasyPeasy Overview:Invalid by ramvi> <GNOME Screensaver:Won't Fix> <Ichthux:Invalid by raphink> <JAK LINUX:Invalid> <The Linux OS Project:In Progress> <metacity:In Progress> <OpenOffice:In Progress by lh-maviya> <Tabuntu:Invalid by tinarussell> <Tivion:Invalid by shakaran> <Tv-Player:New> <Ubunt
[15:24] <gary_poster> ah true
[15:24] <gary_poster> heh
[15:26] <danilos> gary_poster, not sure what you mean with "yellow sub"?
[15:30] <gary_poster> danilos, I though you might be source of icon change in https://launchpad.net/~yellow ?
[15:31] <gary_poster> So, I should let jml and diogo get the new flag
[15:31] <gary_poster> I could add them to ~yellow
[15:32] <gary_poster> But I'm inclined to ask jml if he wants a particular team to be added to the flag
[15:32] <gary_poster> any opinions?
[15:37] <benji> gary_poster: I'm doing QA for the structual susbscription JS and I noticed that for the "are added or changed in any way" radio button we use "(more options...)" and for "Bugs must match this filter" we use just "(...)".  Is that intentional?
[15:39] <gary_poster> benji, yeah.  I thought at that point it would be clear, and didn't want the clutter.
[15:39] <benji> k
[15:39] <gary_poster> don't feel strongly about it, but OTOH would not be keen on a change unless we got people actually being confused or compaining about it
[15:47] <gary_poster> in case this affects anyone, here's a rough outline of my plans right now.  I'm going to address the JS conflicts, since danilos is still around, in case I need to confer with him.
[15:47] <gary_poster> Then, because I kind of need to wait for either danilos' branch or gmb's API branch to land on db-devel for me to get a sane MP diff, I'm going to switch to small things such as my long-outstanding expenses.
[15:47] <gary_poster> Then I'll get back to the UI branch, and hopefully have an MP waiting for gmb in the morning with tests.
[15:47] <gary_poster> Finally, maybe worth noting is that I will be out tomorrow for a few hours after our call, and will continue to do that every Tuesday until the baby pops out.
[16:11] <benji> gary_poster: here are my QA items for structual subscriptions: http://pastebin.ubuntu.com/589310/; how should I handle them?
[16:13] <gary_poster> benji, this is in regards to landing the client branch, right?
[16:13] <benji> right
[16:15] <gmb> gary_poster: So, this timeout... as far as I can tell it's a query that takes too long, but there isn't much info available in the OOPS reports: https://lp-oops.canonical.com/oops.py/?oopsid=OOPS-1914I723. Can you think of any way that I make useful headway here? I'm a bit stumped.
[16:15] <gary_poster> benji, ok.  first, if this is blocking any card from being qa-ok, mark it as qa-ok.  We are feature flagged.
[16:18] <benji> gary_poster: these branches were landed with no-qa; so we're good there, these problems aren't big so I assume the two cards I have in the QA lane should be moved to deployable.
[16:18] <benji> my main question is what should I do with this list of small things
[16:19] <gary_poster> benji, yes.  List of small things: 1 sec.
[16:19] <benji> no rush
[16:22] <gary_poster> gmb, I'd try to figure out where the sql is coming from.  I'm guessing you agree. :-) Since you are asking, I'm guessing you are not sure what the source is.  I'd try running mute locally with the "show my all the SQL environmental variable.  If that gives you enough to figure it out, great.
[16:22] <gary_poster> If it does not, we might need to do a pdb *with the SQL env variable also*.  I'd do it in zope.publisher.publish debug_call myself, since we have no freaking idea.  Then you can dig around until you see the SQL being generated, and figure out the source that way.
[16:23] <gmb> gary_poster: Okay, that sounds like as good a way to go as any. Thanks.
[16:23]  * gmb refuels the tea tank before starting
[16:23] <gary_poster> :-)
[16:24] <gary_poster> benji, no activity animation: please look at the card in the design section of the backlog starting with ""Unsubscribe" on IBug:..." and make sure it encompasses your concern.
[16:24] <gary_poster> dialog hangs around oddly: I'm not clear on that one.
[16:24] <gary_poster> I don't knw what it means
[16:24] <gary_poster> Maybe make a new card?  Maybe try to describe it for me first here?  your call <shrug>
[16:25] <gary_poster> tag text box: new card
[16:25] <gary_poster> status "Unknown": new card + I suggest an associated bug
[16:25] <gary_poster> overlay start out closer to top of page: new display card
[16:25] <benji> gary_poster: sorry, I forgot that I started making those notes just for me; the dialog appears to get cought deconstructing (or maybe resetting) itself; I click the "do it" button and the dialog gets smaller but still has most of the form visible then it hangs there for a second and then disappears
[16:26] <gary_poster> huh
[16:26] <gary_poster> yeah, a card in "display tweaks" sounds appropriate.
[16:26] <benji> k
[16:26] <gary_poster> For all of those, feel free to make a bug as well
[16:26] <benji> k
[16:26] <gary_poster> but a card is my top priority
[16:27] <gary_poster> Thank you benji, those are good catches.
[16:28] <benji> I'm good at being picky.
[16:28] <gary_poster> :-)
[16:31] <gary_poster> gmb, look for "parent_message" in Bug.index_message in lib/lp/bugs/model/bug.py.
[16:31] <gary_poster> looks like source
[16:31] <gmb> Aha.
[16:31] <gmb> Thanks.
[16:32] <gary_poster> np
[17:08]  * gary_poster was trying to figure out js test failures but then saw the MP
[17:28]  * gary_poster lunches
[18:42] <bac> benji: what is the URL that the edit link should point to?
[18:44] <gary_poster> +subscriptions on the bug target bac.  I *think* it is on bugs.*
[18:45] <bac> gary_poster: ok, that's what i figured
[18:45] <gary_poster> actually it is on all
[18:45] <gary_poster> or at least https://bugs.launchpad.net/launchpad/+subscriptions and https://launchpad.net/launchpad/+subscriptions
[18:47] <gary_poster> cool
[18:52] <benji> bac: sorry, was finishing up lunch... https://bugs.staging.launchpad.net/manuel/+subscriptions
[18:52] <bac> benji: thanks
[18:59] <benji> bac: I have a couple little things to do and then I'll be ready for some pillar divvying
[19:01] <bac> benji: ok
[19:48] <gary_poster> Be prepared for shock and awe: I actually filed my expenses!  It's amazing.
[20:01] <benji> bac: heh, cool
[20:12] <gary_poster> On kanban card, "Administrivia" cards are now "Miscellaneous" cards
[21:36] <bac> benji: fwiw, the refactoring you suggested last week (create 'add_subscribe_link) made today's work much easier.
[21:37] <benji> bac: cool
[21:37] <benji> I guess I'm not entirely full of hot air. ;)
[22:10] <benji> Reason #61 I like telecommuting: Even a home body like me can look forward to going outside on a nice spring day.