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

=== gmb` is now known as gmb
gary_posterbac 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
gmbok13:22
gary_postergmb, hey, how goes the db war?13:22
gary_posteroh, also, I'm going to use Skype13:23
gary_posterbarring any objections13:23
bac`none from me13:23
gmbgary_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 013:23
gary_postercool13:23
=== bac` is now known as bac
gmbSkype works for me.13:23
gary_posterheh13:23
gary_posterok cool13:23
gary_postergmb, did you happen to merge db-devel or something similar (given the upcoming deploy, maybe devel already has db-devel stuff)?13:24
* gary_poster can't qa the bits I need to 'cause qastaging is down. :-/13:25
gmbgary_poster: No. I fixed the failures and pushed and ran. When's the DB deploy? Wednesday AM?13:25
gary_postergmb, yeah, I think so.  I saw that devel is in RC until qa is finished13:26
gary_postermy interest is in making an MP though13:26
gmbAh.13:26
gmbgary_poster: I can merge db-devel and push now for you.13:26
gary_posterI have necessary changes both from your branch and from trunk-ish13:26
gary_posterawesome thank you13:26
* gary_poster starting to stare an kanban board...13:27
gary_posteryay, my stuff is qa'd by wgrant!13:28
gary_posterdanilos, benji, skype plz?13:30
danilosgary_poster, ack13:30
gary_posterthanks13:30
gmbgary_poster: lp:~gmb/launchpad/team-subscription-opt-out and lp:~gmb/launchpad/team-subscription-opt-out-apis now contain db-devel.13:30
gary_posteryay, thank you gmb13:31
benjigary_poster: there now13:31
gmbAAAAH.13:31
gmbgary_poster: Please try again.13:32
danilosgary_poster, I am here13:32
bacbenji: skype?13:40
benjibac: sure13:40
bacbenji: what is your skype id?13:41
benjibac: um... let me see13:41
benjibac: benji_york13:41
gary_posterbenji_york13:41
danilosbac, benji, gary_poster, gmb: fwiw, my refactoring branch does change the structural_subscriptions.js quite a bit, at least for the setup code13:55
bacdanilos: ok.  shouldn't affect my work13:55
gary_posterdanilos, yeah I was afraid of that. :-) I have a branch thst changes JS13:55
gary_posterI'll figure out what to do about it in a few minutes13:55
danilosgary_poster, I did a lot of sanity refactoring and got rid of that context-passing-around hack that we had as well in the process13:57
gary_posterdanilos, cool13:57
danilosgary_poster, I am hoping nobody gets affected too much though13:57
gary_poster:-) I'll see.  it was good stuff to do anyway.13:57
gmbgary_poster: I just submitted AWS expenses for March. Can you sign them off when you get a chance?13:59
gary_posterwill do gmb, thanks for heads up13:59
gmbnp14:00
gmbThanks14:00
bacfeature flag enabled for ~yellow on staging14:04
gary_postergmb, approved expenses.  https://code.launchpad.net/~gary/launchpad/muteteamsub-email/+merge/56154 is now ready when you are.14:06
gmbOoo, a trade of web forms.14:06
gmbOn it.14:06
gary_poster:-) thanks14:06
danilosgary_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 looking14:07
gary_posterapproved 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:09
danilosgary_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
danilosgary_poster, what particular bits are you changing?14:10
danilosgary_poster, it's for muting/unmuting links on the subscriptions list?14:11
gary_posterright14:11
gary_posterI doubt that will be affected much but14:11
gary_posterI wonder if the tests might be14:11
gary_poster(which I started, and then stopped because I wondered if I might be stomping on you)14:11
danilosgary_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 functions14:12
danilosgary_poster, iow, I only expect to be introducing new tests, and my changes are only from make_edit_handler and "below"14:13
gary_posterdanilos.  ack.  One exciting bit is that there are no tests for the edit form rendering, as you probably know14:13
gary_posterright, cool14:13
danilosgary_poster, yeah, and I think my refactor should help with some of that when we get to it14:13
gary_posterok, but since your changes are make_edit_handler and below...14:14
gary_posterthen you won't be writing tests for edit stuff now?14:14
gary_posterSo I can?14:14
gary_poster(without fear of duplicating work)14:14
danilosgary_poster, well, "only below" is probably not completely correct, but I am not doing any tests yet, no14:15
gary_posterok14:15
danilosgary_poster, if you are planning to do tests, however, I strongly suggest you merge my branch first14:15
gary_posterthen I'll start working on thos in just a few.  I have other stuff I should get to first14:15
danilosack14:15
gary_posterok, danilos, will do.14:16
gmbgary_poster: Your branch is r=me.14:40
gary_postercool, thanks gmb.  I have something else for your attention in just 1 sec...14:41
gmbok14:41
gary_postergmb, please take a glance at https://dev.launchpad.net/LEP/BetterBugSubscriptionsAndNotifications/FeatureReviewNotes14:47
gary_posterYou might be able to fill in a couple fo XXXs (or remove them as appropriate)14:47
gmbOk, will do.14:49
gmbgary_poster: I've removed a couple of XXXs and left the rest (that I didn't know much about) alone.15:12
gmbgary_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> <Ubunt15:13
gary_postergmb, ...sure.15:15
gary_posterand thank you15:16
gmbnp15:16
gary_posterdanilos, 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_postermost are very small15:19
gary_posterbut some not so much :-P15:19
gary_posterdanilos, nice yellow sub btw :-)15:20
gary_poster...not clear on what triggered mup to get excited there...15:21
benjigary_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> <Ubunt15: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> <Ubunt15:23
benjiI 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> <Ubunt15:23
gary_posterah true15:24
gary_posterheh15:24
danilosgary_poster, not sure what you mean with "yellow sub"?15:26
gary_posterdanilos, I though you might be source of icon change in https://launchpad.net/~yellow ?15:30
gary_posterSo, I should let jml and diogo get the new flag15:31
gary_posterI could add them to ~yellow15:31
gary_posterBut I'm inclined to ask jml if he wants a particular team to be added to the flag15:32
gary_posterany opinions?15:32
benjigary_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:37
gary_posterbenji, yeah.  I thought at that point it would be clear, and didn't want the clutter.15:39
benjik15:39
gary_posterdon't feel strongly about it, but OTOH would not be keen on a change unless we got people actually being confused or compaining about it15:39
gary_posterin 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_posterThen, 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_posterThen I'll get back to the UI branch, and hopefully have an MP waiting for gmb in the morning with tests.15:47
gary_posterFinally, 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.15:47
benjigary_poster: here are my QA items for structual subscriptions: http://pastebin.ubuntu.com/589310/; how should I handle them?16:11
gary_posterbenji, this is in regards to landing the client branch, right?16:13
benjiright16:13
gmbgary_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_posterbenji, ok.  first, if this is blocking any card from being qa-ok, mark it as qa-ok.  We are feature flagged.16:15
benjigary_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
benjimy main question is what should I do with this list of small things16:18
gary_posterbenji, yes.  List of small things: 1 sec.16:19
benjino rush16:19
gary_postergmb, 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_posterIf 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:22
gmbgary_poster: Okay, that sounds like as good a way to go as any. Thanks.16:23
* gmb refuels the tea tank before starting16:23
gary_poster:-)16:23
gary_posterbenji, 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_posterdialog hangs around oddly: I'm not clear on that one.16:24
gary_posterI don't knw what it means16:24
gary_posterMaybe make a new card?  Maybe try to describe it for me first here?  your call <shrug>16:24
gary_postertag text box: new card16:25
gary_posterstatus "Unknown": new card + I suggest an associated bug16:25
gary_posteroverlay start out closer to top of page: new display card16:25
benjigary_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 disappears16:25
gary_posterhuh16:26
gary_posteryeah, a card in "display tweaks" sounds appropriate.16:26
benjik16:26
gary_posterFor all of those, feel free to make a bug as well16:26
benjik16:26
gary_posterbut a card is my top priority16:26
gary_posterThank you benji, those are good catches.16:27
benjiI'm good at being picky.16:28
gary_poster:-)16:28
gary_postergmb, look for "parent_message" in Bug.index_message in lib/lp/bugs/model/bug.py.16:31
gary_posterlooks like source16:31
gmbAha.16:31
gmbThanks.16:31
gary_posternp16:32
* gary_poster was trying to figure out js test failures but then saw the MP17:08
* gary_poster lunches17:28
bacbenji: what is the URL that the edit link should point to?18:42
gary_poster+subscriptions on the bug target bac.  I *think* it is on bugs.*18:44
bacgary_poster: ok, that's what i figured18:45
gary_posteractually it is on all18:45
gary_posteror at least https://bugs.launchpad.net/launchpad/+subscriptions and https://launchpad.net/launchpad/+subscriptions18:45
gary_postercool18:47
benjibac: sorry, was finishing up lunch... https://bugs.staging.launchpad.net/manuel/+subscriptions18:52
bacbenji: thanks18:52
benjibac: I have a couple little things to do and then I'll be ready for some pillar divvying18:59
bacbenji: ok19:01
=== Ursinha is now known as Ursinha-lunch
gary_posterBe prepared for shock and awe: I actually filed my expenses!  It's amazing.19:48
benjibac: heh, cool20:01
gary_posterOn kanban card, "Administrivia" cards are now "Miscellaneous" cards20:12
bacbenji: fwiw, the refactoring you suggested last week (create 'add_subscribe_link) made today's work much easier.21:36
benjibac: cool21:37
benjiI guess I'm not entirely full of hot air. ;)21:37
benjiReason #61 I like telecommuting: Even a home body like me can look forward to going outside on a nice spring day.22:10
=== Ursinha-lunch is now known as Ursinha

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