Tuesday, 2014-04-01

*** bknudson has joined #openstack-oslo00:31
*** markmcclain has quit IRC00:45
*** markmcclain has joined #openstack-oslo00:53
*** zhiyan_ is now known as zhiyan00:57
*** jecarey has joined #openstack-oslo01:30
openstackgerritLi Ma proposed a change to openstack/oslo.messaging: Logical error in blockless fanout of zmq  https://review.openstack.org/8431002:03
*** zhiyan has left #openstack-oslo02:04
*** zhiyan has joined #openstack-oslo02:04
*** arnaud has quit IRC02:12
*** arnaud has joined #openstack-oslo02:14
*** arnaud has quit IRC02:22
*** dims has quit IRC02:23
*** dims has joined #openstack-oslo02:24
*** markmcclain has quit IRC02:59
*** harlowja is now known as harlowja_away02:59
openstackgerritDoug Hellmann proposed a change to openstack/stevedore: fix link to entry point docs  https://review.openstack.org/8432403:01
*** jecarey_ has joined #openstack-oslo03:07
*** harlowja_away is now known as harlowja03:09
*** jecarey has quit IRC03:10
*** yamahata has joined #openstack-oslo03:13
*** jecarey__ has joined #openstack-oslo03:25
*** jecarey_ has quit IRC03:29
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Decrease extraneous logging  https://review.openstack.org/8426003:51
*** gcb has joined #openstack-oslo04:02
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Add a engine preparation stage  https://review.openstack.org/8399105:06
*** harlowja is now known as harlowja_away05:11
*** zhiyan is now known as zhiyan_05:42
*** flaper87|afk is now known as flaper8706:50
*** zhiyan_ is now known as zhiyan08:38
openstackgerritRoman Podoliaka proposed a change to openstack/oslo-incubator: Add a base test case for DB schema comparison  https://review.openstack.org/7408109:21
*** morganfainberg is now known as morganfainberg_Z09:46
*** zoresvit has joined #openstack-oslo09:50
*** ihrachys is now known as ihrachys|lunch09:59
*** yamahata has quit IRC10:24
*** ihrachys|lunch is now known as ihrachys11:24
openstackgerritJulia Varlamova proposed a change to openstack/oslo-incubator: Add index modifying methods  https://review.openstack.org/6568911:58
openstackgerritJulia Varlamova proposed a change to openstack/oslo-incubator: Add index modifying methods  https://review.openstack.org/6568912:05
openstackgerritDoug Hellmann proposed a change to openstack/oslo.config: Move py33 env before py2x  https://review.openstack.org/8442112:14
openstackgerritYuriy Taraday proposed a change to openstack/oslo.rootwrap: Add an option to run rootwrap as a daemon  https://review.openstack.org/8179812:14
ekarlsohow can I generate a initial migration script with the oslo alembic code ?12:15
*** dims has quit IRC12:22
*** markmcclain has joined #openstack-oslo12:28
openstackgerritStanislav Kudriashev proposed a change to openstack/taskflow: Use correct exception in the timing listener  https://review.openstack.org/8403212:29
*** dims_ has joined #openstack-oslo12:34
*** zoresvit has quit IRC12:51
*** zoresvit has joined #openstack-oslo12:59
*** zoresvit has quit IRC13:02
*** lbragstad has joined #openstack-oslo13:07
*** jecarey__ has quit IRC13:09
*** mriedem has joined #openstack-oslo13:15
*** markmcclain has quit IRC13:45
*** jecarey has joined #openstack-oslo13:55
*** arnaud__ has joined #openstack-oslo13:57
*** jecarey has quit IRC14:02
*** jecarey has joined #openstack-oslo14:03
*** markmcclain has joined #openstack-oslo14:21
*** lbragstad has quit IRC14:32
*** lbragstad has joined #openstack-oslo14:33
dhellmannharlowja_away: please ping me when you come online today so we can talk about a release for taskflow14:38
*** lbragstad has quit IRC14:40
*** zoresvit has joined #openstack-oslo14:43
openstackgerritgaryk proposed a change to openstack/oslo.vmware: VMware: treat cases when SOAP reply does not have a body  https://review.openstack.org/8446514:57
*** lbragstad has joined #openstack-oslo14:58
dhellmanncliff 1.6.0 released15:27
*** hartsocks has quit IRC16:05
*** hartsocks has joined #openstack-oslo16:07
*** arnaud__ has quit IRC16:38
dhellmannoslo.config 1.3.0 tagged16:42
*** arnaud has joined #openstack-oslo16:44
openstackgerritGordon Sim proposed a change to openstack/oslo.messaging: Add driver independent functional tests of the messaging API  https://review.openstack.org/8416416:48
dims_yay oslo.config!16:54
dhellmann:-)16:54
* dhellmann is tagging all the libs16:54
dhellmannoslosphinx 2.1.016:55
dhellmannoslotest 1.0.016:55
dims_nice!16:56
*** mriedem has quit IRC16:56
*** russell_h has joined #openstack-oslo17:12
russell_hanyone have any thoughts on this: https://review.openstack.org/#/c/84262/17:13
russell_hit seems like Nova already has this in their Contexts, so I wasn't sure if this was a deliberate omission or not17:13
*** harlowja_away is now known as harlowja17:14
harlowjadhellmann back17:14
harlowjaso taskflow is waiting on 2 reviews for 0.2, then i'm ready for that17:15
harlowjahttps://review.openstack.org/#/c/84032/ and https://review.openstack.org/#/c/83940/17:15
harlowjaboth ready to go, just need 1 more person to look at :-P17:15
harlowjathen tagging taskflow for 0.2 is fine with me17:15
*** mriedem has joined #openstack-oslo17:19
dhellmannharlowja: do you anticipate that happening soon? today?17:20
harlowjalet me ping another core, maybe he can just quickly look over those17:20
*** arnaud has quit IRC17:21
dhellmannharlowja: cool, thanks17:21
dhellmannrussell_h: I'm not sure if it was deliberate or not.17:22
*** zhiyan is now known as zhiyan_17:27
*** morganfainberg_Z is now known as morganfainberg17:31
bnemecRelease all the things! :-)17:49
bnemecrussell_h: dhellmann: That seems like a reasonable change, but we probably need to decide on what's going to happen with that file.17:50
bnemecI think we were going to discuss that at summit?17:50
*** arosen has joined #openstack-oslo17:50
bnemecAlso, it can't merge until all of our consuming projects are out of feature freeze (I think we're getting close?).17:50
dhellmannbnemec: either that or a ML thread17:50
dhellmannyes, ttx said we should be ready to tag and open things back up in a day or two17:51
arosenHello, I'm using the loopingcall module (https://github.com/openstack/oslo-incubator/blob/master/openstack/common/loopingcall.py) .  From time time time an exception is being raised within the method that DynamicLoopingCall is looping for me. When this occurs it causes loopingcall to stop calling the method for me.17:53
arosenI was wondering if there was anyway to avoid this and have it keep calling the method. I don't see a way to do that off hand currently.17:53
arosenhere's a sample trace for example: http://codepad.org/LZEfXugW17:53
dhellmannarosen: it looks like that's a desired behavior of looping call, but if you don't want it you can trap exceptions in your method so they are all caught17:55
*** arnaud has joined #openstack-oslo17:55
dhellmannbnemec: for the split config generator issue, can glance use the entry point discovery instead of file scanning, as a work-around?17:58
russell_hbnemec: so, it should be fine for a consuming project to just not set remote_address17:58
dhellmannbnemec: or maybe just specify separate files as args to the generator?17:59
russell_hbnemec: but I have no problem with waiting also18:00
bnemecdhellmann: It's possible.  I only have a passing familiarity with the entry point discovery so I'm not positive.18:00
dhellmannbnemec: ok, I'll be back online in a couple of minutes and ask markwash about it18:00
arosendhellmann:  thanks for confirming.18:01
bnemecrussell_h: Yeah, I don't think that's a problem.  It's just that technically context.py is going away, at least according to our current status.18:01
bnemecdhellmann: Sounds good, thanks.18:01
russell_hbnemec: ahh, gotcha :)18:01
bnemecrussell_h: But we've had some discussions lately that suggest we don't actually want to get rid of it.  Some refactoring may be in order though.18:01
harlowjadhellmann ok, those reviews are running through the gate checks, shall i make a little release notes for u after they merge?18:03
harlowjawe've been doing stuff like https://etherpad.openstack.org/p/TaskFlow-0.2.018:03
dhellmannharlowja: if you've already been doing that, it's fine, otherwise just a message to the -dev ML like what I did earlier today for the others will be good18:11
harlowjadhellmann ya, we've been doing a little bit of both18:12
harlowjaemail to the -dev ML with the etherpad18:12
dhellmannharlowja: that works18:12
openstackgerritTerry Howe proposed a change to openstack/cliff: Remove PrettyTable from documentation requirements  https://review.openstack.org/8451518:19
dhellmannbnemec: chatting with markwash in #openstack-glance, he doesn't think the config generator bug is blocking their release18:29
bnemecdhellmann: Oh, yeah I didn't mean to imply that it was blocking the release.  They have the manually maintained sample configs already.18:31
bnemecThe manual stuff is a problem though, especially with libs needing to register options.  There's no way for those to be updated right now.18:31
bnemec(a Juno problem though :-)18:31
dhellmannbnemec: ah, my misunderstanding then18:34
bnemecdhellmann: I probably shouldn't use the word "blocker" around release time unless I mean it. ;-)18:35
dhellmannheh18:35
*** markmcclain has quit IRC19:08
harlowjadhellmann do u know if there is a list of all the recommended icehouse versions/tags19:19
harlowjai've got one @ https://review.openstack.org/#/c/84305/ but probably is better list somewhere :-P19:19
dhellmannharlowja: not sure what you mean?19:19
harlowja^^ see above, might help make it make sense :)19:20
dhellmannharlowja: you might be the one making the list :-)19:21
dhellmannharlowja: ttx might have input19:22
harlowjadhellmann haha, k19:22
harlowja:)19:22
harlowjajosh's list, lol19:22
* dhellmann registers the domain to squat on it19:22
harlowja:(19:22
harlowjalol19:22
harlowjattx let me know if u get around, be nice to have said list somewhere i guess :-/19:23
ttxharlowja: o/19:30
harlowjattx howday19:31
harlowjattx so i was wondering if there is a more official list for something like https://review.openstack.org/#/c/84305/1/conf/origins/icehouse.yaml19:31
harlowjaor maybe that is the current offical list :-P19:31
ttxhmm, nope nothing "more" official19:32
harlowjak19:32
harlowjanp19:32
harlowjai don't mind it being the offical list ;) ha19:32
harlowja*basically anvil will suck down all those versions, and build rpms from them19:33
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Allow atoms to save their own state/result  https://review.openstack.org/8394019:36
harlowja*arg, one more rebase then 0.2 dhellmann19:36
*** yamahata has joined #openstack-oslo19:55
*** markmcclain has joined #openstack-oslo20:03
*** jecarey has quit IRC20:35
*** dims_ has quit IRC20:44
*** jecarey has joined #openstack-oslo20:46
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Avoid holding the state lock while notifying  https://review.openstack.org/8454620:59
morganfainbergdhellmann, ping - issues with openstack.common.db.sqlalchemy.migration and utf8 table check.21:01
morganfainbergdhellmann, keystone (i'm sure others) cannot migrate cleanly from an empty db because sql-a-migrate makes it's migration_version table with the default charset21:01
morganfainbergdhellmann, and http://git.openstack.org/cgit/openstack/keystone/tree/keystone/openstack/common/db/sqlalchemy/migration.py#n200 then bails out21:01
morganfainbergdhellmann, if my.cnf is changed to make the default charset utf8 in the server, the issue wont occur.  This seems like we should exclude the migrate_version table.21:02
morganfainbergdhellmann, this is on a clean precise install (ubuntu) and affects icehouse (milestone-proposed) as well as master.21:04
morganfainbergdhellmann, do we fix this across the board or do we document that utf8 charset must be default?21:04
morganfainbergttx, ping ^21:04
dhellmannmorganfainberg: hop in #openstack-meeting,this would be a good topic for the project meeting21:05
morganfainbergk21:05
dhellmannit's going to be a bit before we get to open discussion, I think, we just started21:05
dhellmannmorganfainberg: ^^21:05
morganfainbergyah np. i might have a phone call shortly21:05
morganfainbergdhellmann, so if i'm not here this breaks before any schema migrations occur since it checks the tables and the migration_version table is already there/at version 021:06
morganfainbergdhellmann, but you have the information here. (hopefully i'll not be yanked out)21:07
dhellmannmorganfainberg: ok, I'll bring it up if you're not there21:07
dhellmannmorganfainberg: you know, since we're not joining to that table, it probably makes sense to skip checking it21:07
dhellmannmorganfainberg: or fix the code that is creating it21:07
morganfainbergdhellmann, ++ that is my thought21:07
dhellmannI assume you mean the first suggestion21:08
morganfainbergdhellmann, whichever is the path of least resistence21:08
dhellmannmorganfainberg: yeah, if you want to submit a patch we can push it through tomorrow21:08
morganfainbergdhellmann, but releasing a new SQLA Migrate might make distributions cranky21:08
dhellmannugh, yeah21:08
morganfainbergdhellmann, since we would need to bump the minimum req up21:08
dhellmannmorganfainberg: is there a bug?21:08
dhellmannI mean, a ticket, filed?21:08
morganfainbergdhellmann, i think so. at least against keystone21:09
morganfainberglet me check21:09
dhellmannmorganfainberg: add oslo to it?21:09
morganfainbergdhellmann, https://bugs.launchpad.net/keystone/+bug/127900021:09
ttxmorganfainberg: yes, maybe raise it when I ask for blockers21:09
ttxin ~ 10 min21:09
morganfainbergdhellmann, let me make sure we're not lagging behind on sync...21:10
morganfainbergttx, k21:10
morganfainbergdhellmann, but it doens't look like we are at first glance21:10
bnemecI don't think you are.  This came up somewhere else too.21:10
bnemecGlance I think.21:10
morganfainbergbnemec, nod.21:10
bnemecBut since they're skipping it for this release anyway it didn't matter as much to them.21:10
morganfainbergbnemec, oh joy21:11
morganfainbergbnemec, well at least imo this appears to be a blocker for keystone. and potentially for anyone else using this code.21:11
bnemecmorganfainberg: Agreed.  I thought the other projects were okay because they had already forced utf8 conversions, but if it's happening before any migrations are run then obviously not. :-/21:12
morganfainbergbnemec, if you create the db with the right charset it shouldn't be an issue. but that seems about as subwonderful as using a my.cnf change21:14
bnemecmorganfainberg: Yeah.  This db charset stuff is so much fun. ;-)21:16
dhellmannmaybe we should just refuse to connect to the database at all21:17
dhellmannif the charset is wrong, that is21:17
dhellmannor, you know, ever -- databasing is hard21:17
morganfainbergdhellmann, LOL21:17
morganfainbergdhellmann, can we just remove databases?21:17
dhellmannmorganfainberg: we're already only eventually consistent, so it seems like we could21:18
dhellmannmaybe take a token-ring approach and just keep passing data around on the message bus?21:18
morganfainbergdhellmann, so, what you're saying is store everything in xml and rsync it eventually?21:18
dhellmann"this record isn't mine, send to the next service"21:18
morganfainbergdhellmann, nah message bus is too performant21:18
* dhellmann stops talking to morganfainberg because he said xml21:19
morganfainbergdhellmann, AHA! i know the secret!21:19
bnemeclol21:19
* dhellmann :-|21:20
bnemec"eventually consistent" where "eventually" is defined as "when mysql's defaults are not stupid"21:20
dhellmannhaha21:20
*** mriedem has quit IRC21:44
*** lbragstad has quit IRC21:45
morganfainbergdhellmann, ttx, https://bugs.launchpad.net/ironic/+bug/1301036 working on hunting down who is affected21:49
dhellmannmorganfainberg: excellent, thanks21:49
morganfainbergi know keystone and ironic are, those are tagged in already21:49
dhellmannmorganfainberg: I'll be going offline in about 10 min for about 90 min until I'm home21:49
morganfainbergdhellmann, no worries i'll prob get a patch in later today pacific time21:49
morganfainbergdhellmann, pending a phone call soon21:49
morganfainbergand meeting21:50
dhellmannmorganfainberg: sounds good, just pre-explaining delays in replies21:50
morganfainbergdhellmann, it's IRC i always assume people are AFK/idle/etc until they respond21:50
bnemecSo I think Glance is still relatively safe from this bug, but probably good to fix while we're fixing the others.21:53
bnemecThey don't have a migration to force utf8 on any tables, so...21:53
bnemecIn order for the utf8 check to ever pass, you would have to set it at the server level (or something similarly global).21:53
*** ruhe has quit IRC21:54
*** openstackgerrit has quit IRC21:54
bnemecStill an issue, but I think it would be a lot harder for someone to want the utf8 check and hit this migrate_version problem.21:54
*** ruhe has joined #openstack-oslo21:55
*** dims_ has joined #openstack-oslo22:06
*** dims_ has quit IRC22:18
bnemecdhellmann: So I rebased https://review.openstack.org/#/c/83759/6 and rebased my changes on that change so there isn't a rebase race going on in the config project.22:19
bnemecI'm waiting to push it until you get back though so I don't step on any changes you might have had in progress.22:20
harlowjadhellmann https://pypi.python.org/pypi/taskflow/0.2, sending out mail now :-P22:23
*** dims_ has joined #openstack-oslo22:36
*** markmcclain has quit IRC22:52
*** yamahata has quit IRC23:08
*** mriedem has joined #openstack-oslo23:09
morganfainbergdhellmann, bnemec, looks like keystone and heat are the only two affected by this migration issue23:11
morganfainbergfrom what i can tell based upon projects under the openstack/* namespace23:12
morganfainbergwill be working on fixing the oslo migration stuff shortly23:12
bnemecmorganfainberg: We should add glance too since they'll need to worry about it too at some point.23:12
bnemecAlthough like I said above I think it would be hard for a user to hit this there.23:13
morganfainbergbnemec, techically ironic will also, but they said they would sync again before23:26
morganfainbergbnemec, the icky part for keystone is this could break upgrades from havana -> icehouse23:26
morganfainbergnot sure about for heat23:26
morganfainbergbnemec, ok i'll see waht i can do to roll in the change here shortly23:27
morganfainbergbnemec, testing this is going to be "fun" *23:27
morganfainberg[*] = some value of fun likely that isn't "fun"23:27
bnemec:-)23:27
morganfainbergbnemec oh i need to still check nova23:30
morganfainberglet me do that23:30
morganfainbergbnemec, the migration stuff is all in oslo-incubator still right?23:33
morganfainbergbnemec, don't need to worry about oslo.db library?23:33
morganfainbergbnemec, or is that not a lib yet?23:33
dhellmannbnemec: go for it, I wasn't working on that patch23:48
dhellmannmorganfainberg: oslo.db isn't a lib, yet, we're waiting until after freeze to make that change23:52
morganfainbergdhellmann, cool, amkes this easier23:52
*** flaper87 is now known as flaper87|afk23:57

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