Tuesday, 2014-06-03

*** zhiyan_ is now known as zhiyan00:16
*** arnaud has joined #openstack-oslo00:29
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Rename the graph analyzer to analyzer  https://review.openstack.org/9738001:20
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Rename the graph analyzer to analyzer  https://review.openstack.org/9738001:22
*** morganfainberg is now known as morganfainberg_Z01:41
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Add a deprecation module  https://review.openstack.org/8705501:45
*** dims has quit IRC01:59
*** yamahata has joined #openstack-oslo02:02
*** praneshp has joined #openstack-oslo02:04
*** shakamunyi has quit IRC02:31
*** arnaud has quit IRC02:47
*** zhiyan is now known as zhiyan_02:50
*** rmcall has quit IRC02:55
*** zhiyan_ is now known as zhiyan03:03
*** osloooooo has joined #openstack-oslo03:04
*** praneshp has quit IRC03:47
openstackgerritzhangjl proposed a change to openstack/oslo.messaging: Message was sended to wrong node with use zmq as rpc_backend  https://review.openstack.org/8493804:29
*** praneshp has joined #openstack-oslo04:29
*** praneshp_ has joined #openstack-oslo04:32
*** praneshp has quit IRC04:34
*** praneshp_ is now known as praneshp04:34
*** harlowja is now known as harlowja_away04:48
*** zhiyan is now known as zhiyan_05:01
*** xiej has quit IRC05:08
*** xiej has joined #openstack-oslo05:08
*** praneshp has quit IRC05:38
*** praneshp has joined #openstack-oslo05:39
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Extract the state changes from the ensure storage method  https://review.openstack.org/9740306:00
openstackgerritChristian Berendt proposed a change to openstack/cliff: Use six.add_metaclass instead of __metaclass__  https://review.openstack.org/9740506:05
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Rename inject_task_args to inject_atom_args  https://review.openstack.org/9741006:19
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Rename inject_task_args to inject_atom_args  https://review.openstack.org/9741006:21
*** mrda is now known as mrda-away07:40
*** nacim has quit IRC07:43
*** nacim has joined #openstack-oslo07:43
*** praneshp has quit IRC07:50
*** stannie has joined #openstack-oslo07:56
*** viktors|afk is now known as viktors07:57
*** ihrachyshka has joined #openstack-oslo08:14
*** ihrachyshka has quit IRC08:16
*** ihrachyshka has joined #openstack-oslo08:16
*** cammann has quit IRC08:29
*** flaper87|afk is now known as flaper8708:41
openstackgerritChristian Berendt proposed a change to openstack/cliff: Use six.add_metaclass instead of __metaclass__  https://review.openstack.org/9740509:03
*** yfujioka has joined #openstack-oslo09:05
openstackgerritRoman Podoliaka proposed a change to openstack/oslo.db: Handle slave database connection in EngineFacade  https://review.openstack.org/9459309:15
openstackgerritRoman Podoliaka proposed a change to openstack/oslo.db: Make get_session() pass kwargs to a sessionmaker  https://review.openstack.org/9459209:15
*** bogdando has joined #openstack-oslo09:26
*** flaper87 is now known as flaper87|afk09:50
*** ihrachyshka has quit IRC09:55
*** amotoki has joined #openstack-oslo09:57
*** ihrachyshka has joined #openstack-oslo10:29
*** yamahata has quit IRC10:52
*** flaper87|afk is now known as flaper8711:04
*** dims_ has joined #openstack-oslo11:14
*** nacim has quit IRC11:36
*** yamahata has joined #openstack-oslo11:38
*** flaper87 is now known as flaper87|afk11:46
*** nacim has joined #openstack-oslo11:46
openstackgerritAndrey Kurilin proposed a change to openstack/oslo.db: Add eventlet.tpool.Proxy for DB API calls  https://review.openstack.org/9646711:55
*** amotoki has quit IRC12:05
*** zhiyan_ is now known as zhiyan12:21
*** zhiyan is now known as zhiyan_12:31
*** dims_ has quit IRC12:31
*** dims_ has joined #openstack-oslo12:32
*** nacim has quit IRC13:02
*** mriedem has joined #openstack-oslo13:03
openstackgerritKen Giusti proposed a change to openstack/oslo.messaging: An initial implementation of a 1.0 based messaging driver  https://review.openstack.org/7581513:03
*** openstackgerrit has quit IRC13:06
*** openstackgerrit has joined #openstack-oslo13:06
*** jecarey has quit IRC13:11
*** nacim has joined #openstack-oslo13:27
*** bknudson has joined #openstack-oslo13:32
*** jecarey has joined #openstack-oslo13:35
openstackgerritAndrey Kurilin proposed a change to openstack/oslo.db: Add eventlet.tpool.Proxy for DB API calls  https://review.openstack.org/9646713:42
*** shakamunyi has joined #openstack-oslo13:51
openstackgerritAndrey Kurilin proposed a change to openstack/oslo.db: Add eventlet.tpool.Proxy for DB API calls  https://review.openstack.org/9646713:52
*** ihrachyshka_ has joined #openstack-oslo13:59
*** ihrachyshka has quit IRC14:02
*** flaper87|afk is now known as flaper8714:17
*** sreshetnyak has joined #openstack-oslo14:18
*** dims_ has quit IRC14:49
viktorsbnemec: hi!14:50
bnemecviktors: Hey14:50
viktorsbnemec: can you please re-review patch https://review.openstack.org/#/c/93396/ (Remove common context usage from db model_query()) ?14:52
bnemecviktors: I was literally looking at that when you pinged me. :-)14:52
bnemecviktors: While you're here though, I'm looking at the example in the docstring and wondering where _get_project_id and _get_deleted come from.14:53
bnemecAre they functions that would exist in the projects already?14:53
viktorsbnemec: yes, I suppose that these functions should be implemented in a project, for example, because of different context format14:55
*** ihrachyshka_ has quit IRC14:56
viktorswe just should parse context (and maybe some another parameters) and pass the new values them to the model_query()14:56
openstackgerritAndrey Kurilin proposed a change to openstack/oslo.db: Add eventlet.tpool.Proxy for DB API calls  https://review.openstack.org/9646714:57
bnemecviktors: Ah, right, because we dropped a param and added support for lists of ids, so that would need to be done differently now.14:57
viktorsbnemec: yes, you right14:57
bnemecviktors: Hmm, but the deleted one won't allow all three possible states - you can pass it None or a value, but you can't not pass it, which triggers a different behavior as I recall.14:58
* viktors checking14:58
bnemecCan we make deleted a dict and pass it as **deleted, so if the dict is empty it won't be included?14:59
bnemec(I realize this is just sample code, but the better we make the sample the easier it should be for projects to adopt)15:00
bnemecviktors: Oh, forgot to ping you on ^15:02
*** yfujioka has quit IRC15:08
viktorsbnemec: hm... It's sounds good )15:10
dhellmannjogo: LGTM15:15
bnemecCrud, I never did get that last import grouping check into hacking. :-(15:23
* bnemec wonders what the status of his patch to add hours to the day is15:24
*** morganfainberg_Z is now known as morganfainberg15:43
*** ihrachyshka has joined #openstack-oslo15:55
*** ihrachyshka has quit IRC15:59
*** ihrachyshka has joined #openstack-oslo15:59
*** kmartin has joined #openstack-oslo16:00
*** flaper87 is now known as flaper87|afk16:03
*** dims_ has joined #openstack-oslo16:16
*** yamahata has quit IRC16:16
*** dims_ has quit IRC16:21
*** markmcclain has joined #openstack-oslo16:24
kmartinmarkmcclain: Can you help me with what the correct "notification_driver ="  setting should be for nova and cinder for the Icehouse or Juno release? I'm confused by it requiring the full path or just rpc, rpc2,etc.. as documented on https://wiki.openstack.org/wiki/Oslo/Messaging16:25
*** ihrachyshka has quit IRC16:27
*** sreshetnyak has quit IRC16:33
*** sreshetnyak has joined #openstack-oslo16:33
*** harlowja_away is now known as harlowja16:45
*** arnaud has joined #openstack-oslo16:59
dhellmannbnemec: I agree with dropping oslo.language and putting that file in oslo.utils instead. I'm not sure whether to +2 or -2 the spec for oslo.language. :-)17:14
*** arnaud has quit IRC17:15
bnemecdhellmann: I'll just abandon it and leave a note in the oslo.utils blueprint that it should be included.17:16
dhellmannbnemec: that works, and I'll update the bp in lp17:16
*** dims_ has joined #openstack-oslo17:17
dhellmannbnemec: oh, one question first, what does that do to the dependency graph?17:17
dhellmannbnemec: ok, oslo.language was the only thing oslo.utils depended on so it should be safe to combine them17:18
bnemecdhellmann: Ah, yeah that should be safe.  I think it only requires log and i18n.17:18
*** markmcclain has quit IRC17:21
jogodhellmann: thanks17:22
*** shakamunyi has quit IRC17:24
*** arnaud has joined #openstack-oslo17:25
dhellmannbnemec: I count importutils and excutils in oslo.language17:25
bnemecdhellmann: I thought we decided to leave importutils in incubator?17:26
dhellmannbnemec: I think both can just go in oslo.utils, though17:26
bnemecBecause it's deprecated by stevedore.17:26
dhellmannbnemec: that seems familiar17:26
dhellmannI think my notes are out of date17:26
bnemecdhellmann: That's what it says on https://etherpad.openstack.org/p/juno-oslo-release-plan17:26
*** ihrachyshka has joined #openstack-oslo17:28
dhellmannbnemec: I updated the graph on https://wiki.openstack.org/wiki/Oslo/Dependencies and the text in https://wiki.openstack.org/wiki/Oslo/GraduationStatus17:34
bnemecdhellmann: I'm still seeing oslo.language in https://wiki.openstack.org/w/images/c/c5/Oslo-libs.png17:36
dhellmannbnemec: flush your browser cache17:38
dhellmannor force a reload or whatever17:38
bnemecdhellmann: Oh, right.  Browser caching has bitten me enough recently that you'd think I would know by now. :-)17:39
bnemecWow, lotta stuff still using importutils.17:40
dhellmannbnemec: yeah, I don't know how accurate that list is any more17:44
dhellmannprobably still fairly accurate17:44
bnemecYeah, I know serialization is using it, but it's a pretty trivial case that would be easy to remove.17:46
dhellmannbnemec: jsonutils uses importutils.try_import(). Maybe that one function should go into oslo.utils?17:47
bnemecdhellmann: Probably.  It's just a shorthand version of try: except ImportError:, which is handy.17:51
* bnemec needs to play around with gerrit-dash-creator17:52
bnemecMy standard dashboard is borderline useless at this point.17:52
openstackgerritJoe Gordon proposed a change to openstack-dev/hacking: Update localization checks to understand separate catalogs  https://review.openstack.org/9758018:02
*** flaper87|afk is now known as flaper8718:06
*** praneshp has joined #openstack-oslo18:17
*** ajo|mac has joined #openstack-oslo18:42
*** praneshp has quit IRC19:02
*** praneshp has joined #openstack-oslo19:04
*** ihrachyshka has quit IRC19:10
*** ihrachyshka has joined #openstack-oslo19:10
openstackgerritJoe Gordon proposed a change to openstack-dev/hacking: Update localization checks to understand separate catalogs  https://review.openstack.org/9758019:16
openstackgerritDoug Hellmann proposed a change to openstack/oslo-incubator: Commit changes to local repository after exporting  https://review.openstack.org/9112519:36
openstackgerritDoug Hellmann proposed a change to openstack/oslo.i18n: Handle . and - in translation domains  https://review.openstack.org/9268219:42
openstackgerritDoug Hellmann proposed a change to openstack/oslo.i18n: Split up monolithic test file  https://review.openstack.org/9268119:42
*** mkoderer has quit IRC19:42
openstackgerritDoug Hellmann proposed a change to openstack/oslo-specs: add spec for remove-context-adapter  https://review.openstack.org/9587019:46
openstackgerritDoug Hellmann proposed a change to openstack/oslo-specs: add spec for remove-context-adapter  https://review.openstack.org/9587019:48
*** jecarey has quit IRC19:48
*** jecarey has joined #openstack-oslo19:48
openstackgerritDoug Hellmann proposed a change to openstack/oslo-specs: Add spec for app-agnostic-logging-parameters  https://review.openstack.org/9528119:48
openstackgerritDoug Hellmann proposed a change to openstack/oslo-specs: Add fix-import-cycle-log-and-versionutils spec  https://review.openstack.org/9527319:49
openstackgerritDoug Hellmann proposed a change to openstack/oslo-specs: Add graduate-oslo-i18n spec to juno  https://review.openstack.org/9500219:52
openstackgerritDoug Hellmann proposed a change to openstack/oslo-specs: add spec for graduate-oslo-log  https://review.openstack.org/9592919:52
dhellmannbnemec: I removed the headers from all of my specs, as you suggested19:54
*** flaper87 is now known as flaper87|afk20:00
harlowjadhellmann u might be interested in https://review.openstack.org/#/c/94061/ (someday this will go in)20:01
harlowjahttps://github.com/stackforge/doc820:01
openstackgerritTomoki Sekiyama proposed a change to openstack/oslo.rootwrap: Add ChainingRegExpFilter for prefix utilities  https://review.openstack.org/9733620:06
dhellmannharlowja: I'll take a look tomorrow (the rest of my day is meetings).20:09
harlowjanp20:09
dhellmannharlowja: does doc8 run as a sphinx plugin?20:09
harlowjanot currently, more as just another binary like flake820:09
harlowjaguess it could be a sphinx plugin (didn't think of that)20:09
harlowjafuture feature i guess (it should be easy to do this i think)20:11
*** mriedem has quit IRC20:19
*** mriedem has joined #openstack-oslo20:21
dhellmannharlowja: I'll have a look at what it does, but it may save some parsing20:22
*** jecarey has quit IRC20:28
*** rmcall has joined #openstack-oslo20:30
*** mriedem has quit IRC20:33
*** mriedem has joined #openstack-oslo20:34
*** arnaud has quit IRC20:35
*** erecio has quit IRC20:36
*** jecarey has joined #openstack-oslo20:40
*** arnaud has joined #openstack-oslo20:41
*** erecio has joined #openstack-oslo20:42
harlowjadhellmann ya, thats the goal, use some of the same docutils stuff as sphinx but use it for validaiton/checking...20:44
dhellmannharlowja: you know that sphinx can be told to error out on a warning, right?20:45
harlowjaya with -W right which is nice, and similar to i guess what a part of doc8 does20:47
dhellmannharlowja: pbr also exposes a switch you can set in setup.cfg20:50
harlowjainteresting, pbr does so much, lol20:52
dhellmannharlowja: yeah, it's our attempt to standardize so it touches lots of stuff20:56
harlowjasure, seems like it can run sphinx for u, which is nice, although validation (imho) seems like it should be a separate tool like flake8, instead of bundled20:57
harlowjaanyway, maybe can find it a home, if not thats ok also :)21:02
harlowja*didn't exactly take me that long to do the basics there ;)21:04
*** jecarey has quit IRC21:08
*** jecarey has joined #openstack-oslo21:08
*** dhellmann has quit IRC21:11
*** dims__ has joined #openstack-oslo21:13
*** dhellmann has joined #openstack-oslo21:13
*** dims_ has quit IRC21:15
jogodhellmann: is the oslo meeting every week?21:18
dhellmannjogo: yes, every friday 1600 UTC21:18
dhellmannjogo: that's a change from last cycle, where we only held meetings when we called them21:18
jogoohh the google cal thing says its on demand21:18
*** ihrachyshka has quit IRC21:24
dhellmannjogo: ah, I need to update that21:27
dhellmannjogo: wait, which google cal thing?21:27
dhellmannjogo: nm, I found it21:29
jogocool21:31
dhellmannjogo: we're violating DRY all over the wiki; sorry I didn't update this spot, too. :-/21:38
jogodhellmann: haha yeah those wiki's are not very DRY21:39
*** dims__ has quit IRC21:42
*** arnaud has quit IRC21:42
*** mriedem has left #openstack-oslo21:53
*** mrda-away is now known as mrda21:58
*** stannie has quit IRC21:59
*** rmcall has quit IRC21:59
*** rmcall has joined #openstack-oslo22:00
openstackgerritJoe Gordon proposed a change to openstack-dev/hacking: Update localization checks to understand separate catalogs  https://review.openstack.org/9758022:07
*** dims has joined #openstack-oslo22:12
*** bknudson has quit IRC22:21
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Remove check_doc.py and use doc8  https://review.openstack.org/9764022:26
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Remove check_doc.py and use doc8  https://review.openstack.org/9764022:30
*** rmcall has quit IRC22:37
morganfainbergdhellmann, ping re: hints for translation (_LE, _LW etc)22:49
morganfainbergdhellmann, is there one we're supposed to specifically use when we do log.execption() ?22:49
*** praneshp has quit IRC22:59
*** zhiyan_ is now known as zhiyan23:17
*** zhiyan is now known as zhiyan_23:18
*** ruhe has quit IRC23:20
*** ruhe2 has joined #openstack-oslo23:20
*** ruhe2 is now known as ruhe23:20
*** bknudson has joined #openstack-oslo23:39

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