Monday, 2014-08-11

openstackgerritMichael Bayer proposed a change to openstack/oslo.db: Consolidate sqlite and mysql event listeners  https://review.openstack.org/11315100:03
openstackgerritMichael Bayer proposed a change to openstack/oslo.db: Repair pysqlite transaction support  https://review.openstack.org/11315200:04
openstackgerritMichael Bayer proposed a change to openstack/oslo.db: Repair pysqlite transaction support  https://review.openstack.org/11315200:12
*** harlowja_at_home has quit IRC00:14
openstackgerritMichael Bayer proposed a change to openstack/oslo.db: Reorganize DbTestCase to use provisioning completely  https://review.openstack.org/11017000:15
openstackgerritMichael Bayer proposed a change to openstack/oslo.db: Use testr instance provisioning to lazily create databases  https://review.openstack.org/11048600:15
openstackgerritMichael Bayer proposed a change to openstack/oslo.db: Add transactional test support to provisioning  https://review.openstack.org/11315300:15
*** yamahata has joined #openstack-oslo01:42
*** harlowja_at_home has joined #openstack-oslo02:23
*** harlowja_at_home has quit IRC02:36
openstackgerritJames Carey  proposed a change to openstack/oslo-incubator: Add unicode coercion of logged messages to ContextFormatter  https://review.openstack.org/11213502:42
openstackgerritJames Carey  proposed a change to openstack/oslo-incubator: Correct coercion of logged message to unicode  https://review.openstack.org/11077202:42
*** noelbk has joined #openstack-oslo03:20
*** stevemar has joined #openstack-oslo03:25
*** jecarey has quit IRC04:09
*** HenryG_ has joined #openstack-oslo04:14
*** HenryG_ has quit IRC04:16
*** HenryG has quit IRC04:16
*** HenryG has joined #openstack-oslo04:18
*** HenryG is now known as HenryG_afk04:18
*** k4n0 has joined #openstack-oslo05:38
*** SridharG has joined #openstack-oslo05:45
*** abhijeetm has joined #openstack-oslo05:54
*** stevemar has quit IRC06:01
*** abhijeetm_ has joined #openstack-oslo06:10
openstackgerritOpenStack Proposal Bot proposed a change to openstack/oslo.utils: Imported Translations from Transifex  https://review.openstack.org/11317106:10
*** abhijeetm has quit IRC06:12
*** praneshp has joined #openstack-oslo06:16
*** abhijeetm_ has quit IRC06:25
*** ildikov has joined #openstack-oslo06:40
*** abhishekk has joined #openstack-oslo06:49
*** amotoki has joined #openstack-oslo07:20
*** ihrachyshka has joined #openstack-oslo07:28
*** jaosorior has joined #openstack-oslo07:29
openstackgerritgaryk proposed a change to openstack/oslo.vmware: Add in a retry decorator  https://review.openstack.org/10729507:32
*** AAzza_afk is now known as AAzza07:37
*** markmc has joined #openstack-oslo07:41
*** mrda is now known as mrda-away08:02
*** AAzza is now known as AAzza_afk08:11
openstackgerritBogdan Dobrelya proposed a change to openstack/oslo.messaging: Should not send replies for cast messages  https://review.openstack.org/10914308:39
openstackgerritBogdan Dobrelya proposed a change to openstack/oslo.messaging: Should not send replies for cast messages  https://review.openstack.org/10914308:41
*** k4n0 has quit IRC08:42
*** Alexei_987 has joined #openstack-oslo08:45
*** pblaho has joined #openstack-oslo08:48
*** k4n0 has joined #openstack-oslo08:59
*** abhishekk has quit IRC09:06
*** abhishekk has joined #openstack-oslo09:06
*** oomichi has quit IRC09:09
openstackgerritVlad Okhrimenko proposed a change to openstack/oslo.db: Add get_non_innodb_tables() to utils  https://review.openstack.org/10899409:25
*** SridharG has quit IRC09:26
*** SridharGaddam has joined #openstack-oslo09:26
*** praneshp has quit IRC09:29
*** SridharGaddam has quit IRC09:34
*** SridharG has joined #openstack-oslo09:36
*** jaosorior has quit IRC09:42
*** alexpilotti has joined #openstack-oslo09:52
*** linkid has joined #openstack-oslo10:03
linkidhi10:03
linkidI'm patching the bug https://bugs.launchpad.net/oslo/+bug/1354470 and I wonder if I need test for it10:03
*** pcm_ has joined #openstack-oslo10:04
linkidbecause if I need it, maybe I have to patch the assert_exception method too, to add some arguments10:05
linkidand then, do I have to do it in my wmse patch or do I have to submit another bug / patch ?10:06
*** pcm_ has quit IRC10:08
*** pcm_ has joined #openstack-oslo10:08
alexpilottimarkmc: hello10:23
markmcalexpilotti, hey10:23
alexpilottimarkmc: I have a spec proposal for Oslo: https://review.openstack.org/#/c/109863/10:24
alexpilottimarkmc: an RPC backend for the Windows Service Bus10:24
markmcalexpilotti, yep, had a quick look at it earlier this morning - been away for two weeks10:24
markmcalexpilotti, will try to leave some comments in a bit10:24
alexpilottimarkmc: np :-)10:24
alexpilottimarkmc: not an urgent matter!10:25
alexpilottimarkmc: the only question that requires a relatively timely answer is if we can push it for J-3 or postpone it to K-110:26
markmcalexpilotti, best to assume Kilo at this point, I think10:26
markmcalexpilotti, have you pushed an implementation anywhere yet?10:27
alexpilottimarkmc: I was waiting for comments on the BP10:27
alexpilottimarkmc: I can push the code tomorrow10:27
alexpilottimarkmc: just need to clean up some minor nits10:27
*** k4n0 has quit IRC10:27
markmcalexpilotti, that would be helpful while reviewing the spec10:28
alexpilottimarkmc: ok np, while I prepare the code, do you think you could levae some general initial comments?10:28
markmcalexpilotti, I'll try to soon, yes10:29
alexpilottimarkmc: like “no way, we are sick of all those RPC backends”? :-)10:29
markmcalexpilotti, nope, I'm open to any backends which can sanely implement the semantics of the API10:29
alexpilottimarkmc: just to see if it makes sense to go on with the coding effort in the first place :-)10:29
alexpilottimarkmc: ok, got it!10:30
alexpilottimarkmc: btw I talked about testing w dhellmann. We’ll move the Hyper-V CI from RabbitMQ to the Service Bus, so it’ll be fully tested there10:30
*** AAzza_afk is now known as AAzza10:31
markmcalexpilotti, great10:31
*** jaosorior has joined #openstack-oslo10:50
*** k4n0 has joined #openstack-oslo10:59
openstackgerritIhar Hrachyshka proposed a change to openstack/oslo.db: Set utf8 encoding for mysql and postgresql  https://review.openstack.org/11123611:14
openstackgerritIhar Hrachyshka proposed a change to openstack/oslo.db: Set utf8 encoding for mysql and postgresql  https://review.openstack.org/11123611:15
*** ihrachyshka has quit IRC11:30
*** jroll has quit IRC11:34
*** dims has joined #openstack-oslo11:39
*** dims is now known as Guest3793911:40
*** yamahata has quit IRC11:42
*** ihrachyshka has joined #openstack-oslo11:49
*** HenryG_afk is now known as HenryG12:01
*** Guest37939 is now known as dims_12:08
*** dims_ has quit IRC12:12
*** jroll has joined #openstack-oslo12:19
*** jroll has quit IRC12:24
*** jroll has joined #openstack-oslo12:32
*** dims_ has joined #openstack-oslo12:33
*** dims_ has quit IRC12:35
*** dims_ has joined #openstack-oslo12:35
*** dims_ has quit IRC12:36
*** dims_ has joined #openstack-oslo12:37
*** dims_ is now known as dims12:45
*** ihrachyshka has quit IRC12:51
*** gordc has joined #openstack-oslo12:52
*** ihrachyshka has joined #openstack-oslo12:53
*** jaosorior has quit IRC13:12
jd__dhellmann: should we be good with https://review.openstack.org/#/c/101911/ now?13:17
*** miqui has joined #openstack-oslo13:22
*** stevemar has joined #openstack-oslo13:37
*** jgrimm has quit IRC13:42
*** dims_ has joined #openstack-oslo13:47
*** dims has quit IRC13:48
*** abhishekk has quit IRC13:57
*** alexpilotti has quit IRC13:59
*** ildikov has quit IRC14:00
rpodolyakaany oslo cores around to review transition of oslo.db to using of oslo.utils - https://review.openstack.org/#/c/111701/ ?14:08
openstackgerritTimur Sufiev proposed a change to openstack/oslo-incubator: Enhance versionutils.deprecated to work with classes  https://review.openstack.org/10491614:18
*** jecarey has joined #openstack-oslo14:19
dims_rpodolyaka: ack.14:19
rpodolyakadims_: thanks!14:22
*** alexpilotti has joined #openstack-oslo14:23
*** markmcclain has joined #openstack-oslo14:28
*** k4n0 has quit IRC14:33
openstackgerritAndrey Kurilin proposed a change to openstack/oslo.db: Move to oslo.utils  https://review.openstack.org/11170114:34
*** tsekiyama has joined #openstack-oslo14:35
*** ihrachyshka_ has joined #openstack-oslo14:38
*** jaypipes has joined #openstack-oslo14:38
*** ihrachyshka has quit IRC14:40
*** jgrimm has joined #openstack-oslo14:44
dhellmannjd__: yes, I think we're ready to resume work on that. Note jeblair's comment about the changes to the acl file.14:46
*** Ish__ has joined #openstack-oslo15:01
jd__ah I missed that thanks dhellmann15:02
*** ihrachyshka_ has quit IRC15:03
*** AAzza is now known as AAzza_afk15:06
*** AAzza_afk is now known as AAzza15:12
*** zzzeek has joined #openstack-oslo15:18
*** pblaho has quit IRC15:51
openstackgerritA change was merged to openstack/oslo.utils: Imported Translations from Transifex  https://review.openstack.org/11317115:54
*** markmcclain has quit IRC15:56
openstackgerritMichael Bayer proposed a change to openstack/oslo.db: Implement a dialect-level function dispatch system  https://review.openstack.org/10943616:02
openstackgerritMichael Bayer proposed a change to openstack/oslo.db: Consolidate sqlite and mysql event listeners  https://review.openstack.org/11315116:06
openstackgerritMichael Bayer proposed a change to openstack/oslo.db: Use dialect dispatch for engine initiailization.  https://review.openstack.org/11044616:06
openstackgerritMichael Bayer proposed a change to openstack/oslo.db: Repair pysqlite transaction support  https://review.openstack.org/11315216:06
*** SridharG has quit IRC16:09
openstackgerritMichael Bayer proposed a change to openstack/oslo.db: Reorganize DbTestCase to use provisioning completely  https://review.openstack.org/11017016:10
openstackgerritMichael Bayer proposed a change to openstack/oslo.db: Use testr instance provisioning to lazily create databases  https://review.openstack.org/11048616:10
openstackgerritMichael Bayer proposed a change to openstack/oslo.db: Add transactional test support to provisioning  https://review.openstack.org/11315316:10
*** AAzza is now known as AAzza_afk16:13
*** HenryG has quit IRC16:30
*** openstack has joined #openstack-oslo16:32
*** alexpilotti has quit IRC16:38
*** HenryG has joined #openstack-oslo16:39
openstackgerritA change was merged to openstack/taskflow: add pre/post execute/retry callbacks to tasks  https://review.openstack.org/11230016:40
openstackgerritA change was merged to openstack/taskflow: Allow handlers to provide validation callables  https://review.openstack.org/10167916:40
*** markmc has quit IRC16:42
*** ildikov has joined #openstack-oslo16:48
*** alexpilotti has joined #openstack-oslo17:00
*** praneshp has joined #openstack-oslo17:22
*** alexpilotti has quit IRC17:29
krotscheckdhellmann: I figured out the config issue we ran into the other day. It turned out that the config file resolution order returned a file that didn’t have +r for the process user.17:29
krotschecki.e. oslo isn’t broken, our server was.17:29
dims_krotscheck: whew!17:29
*** markmcclain has joined #openstack-oslo17:30
krotscheckThough oslo.config might want to check permissions before completing config file resolution...?17:30
* krotscheck checks to see how hard that would be.17:30
dhellmannkrotscheck: do you think we should log a warning?17:34
krotscheckdhellmann: Depends. If it skips a file because of permissions then yes, it should definitely log a warning before moving on. If it just returns it though… I dunno, that seems more like a runtime exception to me.17:35
dhellmannkrotscheck: good point17:35
*** zzzeek_ has joined #openstack-oslo17:36
*** linkid has left #openstack-oslo17:38
*** alexpilotti has joined #openstack-oslo17:45
openstackgerritMichael Krotscheck proposed a change to openstack/oslo.config: _search_dirs only returns readable files  https://review.openstack.org/11332717:46
krotscheckI think that should do it.17:47
krotscheckSince it’s in oslo.config I feel pretty comfortable only checking for readability, though there might be edge cases for X_OK?17:47
dims_dhellmann: should i kick off a oslo.serialization release?17:55
dhellmanndims_: is the API for the library cleaned up?17:55
dhellmannkrotscheck: I think R_OK is probably good enough for this17:55
dims_dhellmann: will verify17:56
dhellmanndims_: let's make sure the docs are cleaned up and publishing properly, too17:56
dims_ack17:57
dhellmannkrotscheck: I left a couple of comments17:58
krotscheckdhellmann: Thanks17:58
dims_dhellmann: nothing in the specs about revamping API - http://git.openstack.org/cgit/openstack/oslo-specs/tree/specs/juno/graduate-oslo-serialization.rst17:58
dims_should we add oslo.i18n and oslo.utils dependency?17:59
dims_while we are at it17:59
dhellmanndims_: if the code in utils uses those libs, we should make sure the dependencies are set up before releasing18:01
dims_will do18:02
dhellmanndims_: I'll give the code in the repo a quick look to see what the public API surface is18:02
dims_thanks18:02
dhellmanndims_: oh, right, we just have json there now, don't we18:03
dims_right18:03
dhellmanndims_: some of those functions need docstrings, but otherwise I think it makes sense to leave everything in jsonutils and recommend importing it that way18:05
dhellmanndims_: so the public API face is fine as it is18:05
dims_cool18:05
*** harlowja_at_home has joined #openstack-oslo18:11
openstackgerritMichael Krotscheck proposed a change to openstack/oslo.config: _search_dirs only returns readable files  https://review.openstack.org/11332718:12
*** jecarey has quit IRC18:20
openstackgerritA change was merged to openstack/oslo.db: Move to oslo.utils  https://review.openstack.org/11170118:38
*** jecarey has joined #openstack-oslo18:43
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Add a pformat() failure method and use it in the conductor  https://review.openstack.org/11142618:43
*** zzzeek_ has quit IRC18:44
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Translate the engine runner into a well defined state-machine  https://review.openstack.org/10653518:45
*** zzzeek_ has joined #openstack-oslo18:45
*** zzzeek_ has left #openstack-oslo18:47
*** jecarey_ has joined #openstack-oslo18:48
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Ensure state machine can be frozen  https://review.openstack.org/10747918:49
*** zzzeek__ has joined #openstack-oslo18:49
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Add existing types to generated documentation  https://review.openstack.org/10693018:49
*** AAzza_afk is now known as AAzza18:50
*** jecarey has quit IRC18:51
*** jecarey_ has quit IRC18:51
*** jecarey has joined #openstack-oslo18:51
*** mrda-away is now known as mrda18:57
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo.serialization: Use oslo libraries - i18n and utils  https://review.openstack.org/11334718:59
openstackgerritA change was merged to openstack/pycadf: Enable PEP8 checks E128, E251 and E265  https://review.openstack.org/10906719:01
openstackgerritA change was merged to openstack/pycadf: revise readme with a project description  https://review.openstack.org/11049019:02
openstackgerritA change was merged to openstack/pycadf: add CONTRIBUTING doc  https://review.openstack.org/11049119:02
*** HenryG_ has joined #openstack-oslo19:04
*** HenryG has quit IRC19:05
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo.serialization: Use oslo libraries - i18n and utils  https://review.openstack.org/11334719:06
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo.serialization: Use oslo libraries - i18n and utils  https://review.openstack.org/11334719:07
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo.serialization: Fix docs build to run properly  https://review.openstack.org/11335119:12
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo.serialization: Use oslo libraries - i18n and utils  https://review.openstack.org/11334719:15
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo.serialization: Fix docs build to run properly  https://review.openstack.org/11335119:15
*** zzzeek has quit IRC19:16
*** zzzeek__ has quit IRC19:17
*** zzzeek__ has joined #openstack-oslo19:17
*** zzzeek__ is now known as zzzeek19:18
*** markmcclain has quit IRC19:25
*** AAzza is now known as AAzza_afk19:27
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo.serialization: Add some basic docstring(s)  https://review.openstack.org/11335619:38
*** flaper87|afk is now known as flaper8719:40
dims_dhellmann: there you go...lined up 3 reviews when you get a chance19:40
*** noelbk has quit IRC19:44
*** noelbk has joined #openstack-oslo19:46
*** noelbk has quit IRC19:47
dhellmanndims_: cool, I'll take a look19:53
*** markmcclain has joined #openstack-oslo19:56
beekneemechdims_: Shoot.  https://review.openstack.org/#/c/112990/20:10
*** beekneemech is now known as bnemec20:10
bnemecI forgot to reference the bp in the commit message.20:10
dims_ah duh20:10
dims_sorry20:10
bnemecNah, it's my bad.  There was no way to find those without looking at the pending reviews.20:11
dims_bnemec: so the _message reference in jsonutils is not really needed20:12
dims_bnemec: feel free to take over any of the oslo.ser reviews. did not realize you were back too. i just got back last night from vacation20:13
*** flaper87 is now known as flaper87|afk20:14
bnemecdims_: Yeah, we figured as much, which is why I went ahead with the graduation steps.20:14
*** miqui has quit IRC20:35
*** jecarey_ has joined #openstack-oslo20:55
*** flashgordon is now known as jogo20:56
*** yamahata has joined #openstack-oslo20:56
*** praneshp has quit IRC20:58
*** jecarey has quit IRC20:58
* bnemec wants to shoot lockutils in the head21:00
*** HenryG_ is now known as HenryG21:01
*** amotoki has quit IRC21:02
dims_bnemec: +1 :)21:07
harlowja_at_home;)21:11
harlowja_at_homesoon it will be pylockfile instead :-P21:11
harlowja_at_homedouble the fun, ha21:11
dims_known devil AND unknown god :)21:11
harlowja_at_home;)21:13
harlowja_at_home^ the circle of life, ha21:13
bnemecOh yes, also looking forward to the pylockfile refactoring. :-)21:13
harlowja_at_home:)21:14
harlowja_at_homei remember pylockfile (the last time i looked) being based on pid-based-files (aka likely needing alot of refactoring)21:16
*** Ish__ has quit IRC21:19
*** Ish__ has joined #openstack-oslo21:28
dhellmannharlowja_at_home: the plan is to move our lock stuff into pylockfile more or less as it is, adding a new API to the library but not removing the existing stuff. That way we have backwards compatibility for both and are less likely to break anything depending on the current behavior21:40
harlowja_at_homedhellmann,  ya, seems reasonable21:40
harlowja_at_home*as long as oslo.config stuff not there as another API to use pylockfile ;)21:41
* harlowja_at_home just like the simple predicatable apis :-/21:46
dhellmannharlowja_at_home: pylockfile won't need oslo.config21:48
harlowja_at_homewoot21:48
*** alexpilotti has quit IRC21:52
*** praneshp has joined #openstack-oslo22:04
*** praneshp_ has joined #openstack-oslo22:07
*** jecarey_ has quit IRC22:07
*** praneshp has quit IRC22:08
*** praneshp_ is now known as praneshp22:08
*** sheeprine has quit IRC22:10
*** Ish__ has quit IRC22:12
*** dims_ has quit IRC22:24
*** sheeprine has joined #openstack-oslo22:26
*** Ish__ has joined #openstack-oslo22:26
*** gordc has quit IRC22:31
*** yamahata has quit IRC22:31
*** jgrimm has quit IRC22:44
openstackgerritA change was merged to openstack/oslo-incubator: Updated from global requirements  https://review.openstack.org/11176422:45
dhellmanndims, bnemec: what did you guys decide about the competing change requests in oslo.utils?22:48
dhellmannsorry, I mean oslo.serialization22:48
bnemecdhellmann: I'd say go with dims version.  Mine would need an update because I forgot we included try_import in oslo.utils.22:52
*** bknudson has joined #openstack-oslo22:54
zzzeekwho likes to review heavy sqlalchemy patches on oslo.db23:05
zzzeekdont all jump at once23:07
*** harlowja_at_home has quit IRC23:08
*** dims_ has joined #openstack-oslo23:11
openstackgerritDoug Hellmann proposed a change to openstack/oslo.i18n: Add a test fixture for translatable strings  https://review.openstack.org/11339923:13
dhellmannbnemec: ok, do you want to go ahead and abandon yours, then?23:15
bnemecdhellmann: Should already be done23:15
dhellmannbnemec: ok, great23:15
*** dims_ has quit IRC23:16
*** jaypipes has quit IRC23:19
*** dims_ has joined #openstack-oslo23:20
*** dims_ has quit IRC23:27
*** dims_ has joined #openstack-oslo23:27
*** markmcclain has quit IRC23:27
*** dims_ has quit IRC23:52
*** harlowja_at_home has joined #openstack-oslo23:57

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