Friday, 2014-12-19

openstackgerritMerged openstack/oslo.config: Stop sorting options on output  https://review.openstack.org/13648200:00
vishyharlowja_: so I’m seeing a new error where the first call message sent to any queue fails with a timeout00:00
vishyi don’t know if the message is not getting sent or the response isn’t coming back00:01
harlowja_uh ohs00:02
harlowja_sileht ^^^^^00:02
harlowja_; )00:02
harlowja_vishy how u make that happen?00:05
vishyhttp://paste.openstack.org/show/152944/00:05
vishyi think this was probably happening before00:06
vishybut the compute node only sends messages to conductor and network on a regular basis00:06
vishythey both happen in a periodic00:06
vishyso it just meant that it took an extra couple of minutes to recover00:06
vishybut the api server doesn’t periodically send messages to all nodes00:07
vishyso the first time it tries to send a message directly to a node it fails00:07
*** noelbk has quit IRC00:09
*** tsekiyama has joined #openstack-oslo00:10
openstackgerritChris Hoge proposed openstack/oslo.config: Added help generation for groups  https://review.openstack.org/14293500:13
*** dims__ has joined #openstack-oslo00:14
*** stevemar has joined #openstack-oslo00:22
harlowja_hmmm00:28
*** noelbk has joined #openstack-oslo00:28
harlowja_vishy could be a possiblity; basically it just gets stuck there00:28
vishythis is also per topic00:29
vishythe second message to the same topic works fine00:29
openstackgerritJoshua Harlow proposed openstack/taskflow: Add an example which shows how to send events out from tasks  https://review.openstack.org/14195100:31
*** sigmavirus24 is now known as sigmavirus24_awa00:48
*** sigmavirus24_awa is now known as sigmavirus2400:48
*** sigmavirus24 is now known as sigmavirus24_awa00:51
*** dims__ has quit IRC00:55
*** dims__ has joined #openstack-oslo00:55
*** zzzeek has quit IRC00:57
*** tsekiyam_ has joined #openstack-oslo00:58
*** dims__ has quit IRC00:59
*** dims__ has joined #openstack-oslo01:00
*** tsekiyama has quit IRC01:01
*** tsekiyam_ has quit IRC01:02
*** takedakn has joined #openstack-oslo01:03
openstackgerritJeremy Stanley proposed openstack-dev/pbr: Port in git sha changes from 0.10 line  https://review.openstack.org/14293101:05
openstackgerritJeremy Stanley proposed openstack-dev/pbr: Merge branch 'feature/0.10'  https://review.openstack.org/14293001:05
openstackgerritJeremy Stanley proposed openstack-dev/pbr: Only import sphinx during hook processing  https://review.openstack.org/14284001:05
openstackgerritJeremy Stanley proposed openstack-dev/pbr: Properly check for git before getting git dir  https://review.openstack.org/14284101:05
*** sabari is now known as zz_sabari01:07
*** zz_sabari is now known as sabari01:10
*** takedakn has quit IRC01:12
harlowja_dhellmann running "./oslo_release.sh "kilo" "0.6.0"  "ba62a9c7b" "taskflow"" (with the right commit) should be all i need to do for a release right? just want to double check :)01:28
harlowja_haven't used this new style before01:28
* harlowja_ wondering if i should wait till monday, ha01:28
*** mtanino has quit IRC01:35
*** salv-orlando has quit IRC01:35
openstackgerritJeremy Stanley proposed openstack-dev/pbr: Port in git sha changes from 0.10 line  https://review.openstack.org/14293101:37
openstackgerritJeremy Stanley proposed openstack-dev/pbr: Only import sphinx during hook processing  https://review.openstack.org/14284001:37
openstackgerritJeremy Stanley proposed openstack-dev/pbr: Properly check for git before getting git dir  https://review.openstack.org/14284101:37
*** oomichi has joined #openstack-oslo01:39
*** noelbk has quit IRC01:41
*** noelbk has joined #openstack-oslo01:43
*** zzzeek has joined #openstack-oslo01:44
*** zzzeek has quit IRC01:47
dims__harlowja_: looks right01:47
harlowja_kk01:47
harlowja_don't want to bork this up, ha01:48
dims__i hear ya01:49
*** amotoki has joined #openstack-oslo01:52
*** oomichi has quit IRC01:55
*** mriedem has joined #openstack-oslo02:02
*** noelbk has quit IRC02:14
harlowja_hmmm, damn, don't think it connected to launchpad right, lol02:16
harlowja_but i think it did the rest of it :-/02:16
harlowja_guess pushing a tag to gerrit happened02:16
harlowja_i'll have to run the rest by hand02:16
harlowja_hmmm, what a weird thing, pops up with elinks stuff that doesn't seem to work for my openid i think, lol02:24
*** sabari is now known as zz_sabari02:25
*** salv-orlando has joined #openstack-oslo02:36
harlowja_ok, think i did it, ha02:41
*** salv-orlando has quit IRC02:41
harlowja_alright02:48
harlowja_http://lists.openstack.org/pipermail/openstack-dev/2014-December/053292.html02:48
*** rushiagr_away is now known as rushiagr02:49
*** _amrith_ is now known as amrith03:13
*** jeckersb is now known as jeckersb_gone03:42
*** mriedem has quit IRC03:43
*** harlowja_ is now known as harlowja_away03:51
*** dims__ has quit IRC04:00
*** dims__ has joined #openstack-oslo04:01
*** rushiagr is now known as rushiagr_away04:02
*** dims__ has quit IRC04:05
*** takedakn has joined #openstack-oslo04:05
*** achanda has joined #openstack-oslo04:20
*** stevemar has quit IRC04:25
*** rushiagr_away is now known as rushiagr04:32
jecareydhellmann, my concern didn't get addressed in the oslo.i18n namespace patch that just merged.04:36
jecareyI put the comment in the wrong place; that probably didn't help.04:37
jecareyhttps://review.openstack.org/#/c/127323/6/oslo/i18n.py line 21 comment 3 should have been in oslo_i18n/_i18n.py04:38
*** liusheng has quit IRC04:43
*** liusheng has joined #openstack-oslo04:44
*** takedakn has quit IRC04:46
*** achanda has quit IRC04:48
*** achanda has joined #openstack-oslo04:49
*** achanda has quit IRC04:53
*** zz_sabari is now known as sabari04:59
*** sabari is now known as zz_sabari05:05
openstackgerritPeng Wu proposed openstack/oslo.i18n: Support contextual and plural form of gettext functions  https://review.openstack.org/13485005:10
*** rpodolyaka2 has joined #openstack-oslo05:17
*** noelbk has joined #openstack-oslo05:22
*** salv-orlando has joined #openstack-oslo05:26
*** harlowja_at_home has joined #openstack-oslo05:30
*** salv-orlando has quit IRC05:31
*** rpodolyaka2 has quit IRC05:35
harlowja_at_homealright, think i fixed up bugs/blueprints that weren't adjusted05:41
*** noelbk has quit IRC05:53
*** rpodolyaka2 has joined #openstack-oslo06:01
*** dims__ has joined #openstack-oslo06:09
*** vigneshvar has joined #openstack-oslo06:11
*** dims__ has quit IRC06:13
*** arnaud__ has joined #openstack-oslo06:19
openstackgerritJoshua Harlow proposed openstack/taskflow: Correctly trigger 'on_exit' of starting/initial state  https://review.openstack.org/14299406:23
*** k4n0 has joined #openstack-oslo06:30
*** pcaruana has joined #openstack-oslo06:49
*** zz_sabari is now known as sabari06:50
*** harlowja_at_home has quit IRC06:59
*** rpodolyaka2 has quit IRC07:09
*** rpodolyaka2 has joined #openstack-oslo07:13
*** rpodolyaka2 has quit IRC07:30
*** boris-42 has quit IRC07:33
*** rpodolyaka2 has joined #openstack-oslo07:36
*** andreykurilin has joined #openstack-oslo07:43
openstackgerritJoshua Harlow proposed openstack/tooz: README.rst tweaks  https://review.openstack.org/14300507:45
*** rpodolyaka2 has quit IRC07:45
openstackgerritJoshua Harlow proposed openstack/oslo.messaging: Add a thread + futures executor based executor  https://review.openstack.org/7091407:50
*** salv-orlando has joined #openstack-oslo07:58
*** exploreshaifali has joined #openstack-oslo08:02
*** salv-orlando has quit IRC08:03
*** andreykurilin has quit IRC08:08
*** e0ne has joined #openstack-oslo08:25
*** dtantsur|afk is now known as dtantsur08:28
*** arnaud__ has quit IRC08:33
*** arnaud__ has joined #openstack-oslo08:34
*** salv-orlando has joined #openstack-oslo08:35
*** exploreshaifali has quit IRC08:35
*** arnaud__ has quit IRC08:38
*** e0ne has quit IRC08:38
openstackgerritFrode Nordahl proposed openstack/oslo.messaging: Enable IPv6-support in libzmq by default  https://review.openstack.org/13807508:42
*** dulek has joined #openstack-oslo08:57
*** exploreshaifali has joined #openstack-oslo08:59
*** dtantsur is now known as dtantsur|brb09:00
*** subscope has joined #openstack-oslo09:05
*** exploreshaifali has quit IRC09:05
*** sabari is now known as zz_sabari09:11
*** exploreshaifali has joined #openstack-oslo09:14
*** yassine has joined #openstack-oslo09:32
*** david-ly_ has quit IRC09:32
*** david-lyle has joined #openstack-oslo09:33
*** e0ne has joined #openstack-oslo09:35
*** david-lyle has quit IRC09:38
*** dims__ has joined #openstack-oslo09:46
*** subscope has quit IRC09:48
*** dims__ has quit IRC09:50
*** viktors has joined #openstack-oslo09:53
openstackgerritMerged openstack/oslo.messaging: Fix incorrect attribute name in matchmaker_redis  https://review.openstack.org/12911409:53
*** exploreshaifali has quit IRC09:56
*** ihrachyshka has joined #openstack-oslo09:59
*** subscope has joined #openstack-oslo10:00
*** subscope has quit IRC10:03
*** exploreshaiflai has joined #openstack-oslo10:04
*** xek has joined #openstack-oslo10:05
*** xek has joined #openstack-oslo10:05
*** ozamiatin has joined #openstack-oslo10:13
*** xek has quit IRC10:14
*** subscope has joined #openstack-oslo10:14
*** xek has joined #openstack-oslo10:15
*** exploreshaiflai has quit IRC10:30
openstackgerritMerged openstack/tooz: README.rst tweaks  https://review.openstack.org/14300510:38
jd__hey guys I've a question about oslo.context10:50
jd__it now stores the context in a _request_store and you can use get_current() to get it10:50
jd__but I cannot see any usage of that in openstack.common.log or in oslo.log10:50
jd__do I miss something, or if a project switch to oslo.context it does not log the context anymore?10:51
jd__dhellmann: that's likely a question for you :)10:51
*** exploreshaiflai has joined #openstack-oslo10:59
*** vigneshvar has quit IRC11:10
*** vigneshvar has joined #openstack-oslo11:12
silehtjd__, this a memory leak too11:25
silehtis*11:25
jd__sileht: hum possibly, in which case?11:25
silehtjd__, oslo.messaging spawn a (green)thread per message received, nova store the context of the message in a threading.local, but the threading.local is never cleaned11:27
silehtI wonder if they are a automatic cleanup when the thread died of the threading.local object11:27
*** arnaud__ has joined #openstack-oslo11:34
*** dims__ has joined #openstack-oslo11:37
*** arnaud__ has quit IRC11:38
silehtjd__, the use case for log+context is log everything with the request_id11:42
*** e0ne is now known as e0ne_11:42
*** e0ne_ is now known as e0ne11:46
jd__sileht: I'd expect threading.local to be deleted once the thread is gone?11:48
silehtme too11:48
*** dtantsur|brb is now known as dtantsur12:03
*** dims__ has quit IRC12:05
*** exploreshaiflai has quit IRC12:07
*** dims__ has joined #openstack-oslo12:10
openstackgerritSergey Nikitin proposed openstack/oslo-specs: Adds specification for SQL profiler  https://review.openstack.org/14306612:14
*** kashyap has joined #openstack-oslo12:16
dims__jd__: flaper87: easy one - https://review.openstack.org/#/c/142654/ please take a look when you get a chance. just a readme update12:19
dims__we should be able to roll out oslo.log after that12:20
*** subscope has quit IRC12:21
*** jaypipes has quit IRC12:22
*** k4n0 has quit IRC12:23
*** vigneshvar has quit IRC12:23
* flaper87 clicks12:25
flaper87dims__: done12:25
* flaper87 can't wait to have oslo.log out there12:25
dims__thanks!12:25
dims__dhellmann: ping12:26
*** e0ne is now known as e0ne_12:27
*** vigneshvar has joined #openstack-oslo12:28
*** tedross has joined #openstack-oslo12:34
*** e0ne_ has quit IRC12:37
*** krotscheck has quit IRC12:46
*** krotscheck has joined #openstack-oslo12:47
*** alexpilotti has joined #openstack-oslo12:48
openstackgerritMerged openstack/oslo.log: Move files out of the namespace package  https://review.openstack.org/13462212:50
openstackgerritMerged openstack/oslo.log: Switch to oslo.context  https://review.openstack.org/14014712:50
*** subscope has joined #openstack-oslo12:51
kashyapflaper87, I can reproduce the issue: "2014-12-19 07:53:57.332 TRACE nova.compute.manager [instance: dc3f16fe-b899-457a-9da3-a97932dbf167] MessagingTimeout: Timed out waiting for a reply to message ID b6204fe6001d4f70a159f70e296b139412:54
kashyap"12:54
kashyapWith current git master (LIBS_FROM_GIT=oslo.messaging)12:55
kashyapNot sure if a bug is warranted. Uh, stumbled on this, as part my yak shaving exercise of the day12:56
kashyapThat seems to be the bug? - https://bugs.launchpad.net/oslo.messaging/+bug/133873212:57
openstackgerritMerged openstack/oslo.log: Enhance the README a bit  https://review.openstack.org/14265412:57
* kashyap files a new bug13:00
*** pblaho has joined #openstack-oslo13:04
*** alexpilotti has quit IRC13:06
kashyapOh wait, it's probably a PEBKAC13:07
kashyapI normally use Neutron, and this time I ended up using Nova network to keep the footprint low, so I'm missing (Nova network) 'n-net' from ENABLED_SERVICES13:08
*** andreykurilin has joined #openstack-oslo13:18
dims__jd__: looking at bugs and reviews for oslo.log, looks like your https://review.openstack.org/#/c/136842/ needs to be rebased13:19
*** e0ne has joined #openstack-oslo13:21
*** exploreshaifali has joined #openstack-oslo13:21
*** salv-orlando has quit IRC13:28
*** vigneshvar has quit IRC13:34
jd__dims__: yeah I've a few things to complete13:35
jd__dims__: did you see my earlier question about oslo.context?13:35
dims__oops no13:36
dims__don't see in scroll back13:37
dims__"flaper87 can't wait to have oslo.log out there"13:37
dims__thats the last one13:37
*** boris-42 has joined #openstack-oslo13:38
*** jeckersb_gone is now known as jeckersb13:40
*** kgiusti has joined #openstack-oslo13:43
*** kashyap has left #openstack-oslo13:54
dhellmanndims__: good morning14:01
*** achanda has joined #openstack-oslo14:06
*** amrith is now known as _amrith_14:10
*** dims__ has quit IRC14:12
*** dims__ has joined #openstack-oslo14:12
*** gordc has joined #openstack-oslo14:12
*** dims__ has quit IRC14:16
*** salv-orlando has joined #openstack-oslo14:29
*** jeckersb is now known as jeckersb_gone14:30
*** dulek has quit IRC14:31
*** salv-orlando has quit IRC14:34
*** jeckersb_gone is now known as jeckersb14:34
*** jaypipes has joined #openstack-oslo14:37
*** rushiagr is now known as rushiagr_away14:38
*** exploreshaifali has quit IRC14:39
*** jaypipes is now known as leakypipes14:41
*** achanda has quit IRC14:41
*** zzzeek has joined #openstack-oslo14:41
*** _amrith_ is now known as amrith14:51
ttxdhellmann: around? re: oslo-incubator tag15:04
*** _gryf has quit IRC15:04
*** e0ne is now known as e0ne_15:13
*** e0ne_ is now known as e0ne15:14
*** rushiagr_away is now known as rushiagr15:16
*** mtanino has joined #openstack-oslo15:17
*** stevemar has joined #openstack-oslo15:20
*** yassine has joined #openstack-oslo15:35
*** achanda has joined #openstack-oslo15:35
*** eezhova has quit IRC15:54
*** amrith is now known as _amrith_15:54
*** akamyshnikova has quit IRC15:54
*** sreshetnyak has quit IRC15:54
*** eezhova has joined #openstack-oslo15:58
*** akamyshnikova has joined #openstack-oslo15:58
*** pcaruana has quit IRC15:59
viktorsdhellmann: ping15:59
*** sreshetnyak has joined #openstack-oslo16:01
viktorsdhellmann: I have a very dummy question about the patch  https://review.openstack.org/#/c/138787/ - what is  the difference between tests in oslo.db/test/ and  oslo.db/oslo_db/tests/ directories ?16:11
*** tsekiyama has joined #openstack-oslo16:22
*** dims__ has joined #openstack-oslo16:24
*** dims__ has quit IRC16:24
*** dims__ has joined #openstack-oslo16:24
dims__dhellmann: can we get oslo.log 0.1.0 out? everything we need is in from what i can tell (may be except for https://review.openstack.org/#/c/136842/)16:25
dims__dhellmann: can you please remove your -2 from that one? ^^^16:25
*** exploreshaifali has joined #openstack-oslo16:26
dims__dhellmann: i can get the release out as i have some time today16:30
bnemecviktors: The tests in the top-level directory are for verifying the namespace compatibility works, the ones in oslo_db/tests are for testing the moved interface.16:30
*** bnemec is now known as beekneemech16:31
beekneemechSo I'm seeing http://logs.openstack.org/47/141147/6/check/gate-tempest-dsvm-largeops-src-oslo.rootwrap/e77419f/logs/screen-s-proxy.txt.gz16:32
beekneemechIs this a known issue?  It looks like the new setuptools doesn't like our development versions.16:32
*** boris-42 has quit IRC16:33
dims__y known isue16:33
dims__beekneemech: did you try a recheck?16:36
*** salv-orlando has joined #openstack-oslo16:36
beekneemechdims__: Not yet, I didn't want to blindly recheck in case this was a new symptom of all the setuptools changes.16:37
beekneemechSo is this expected to be fixed now then?16:37
dims__yep16:37
beekneemechCool, will go ahead and recheck then.16:37
beekneemech(if I can find the window with that change open...)16:38
beekneemechdims__: Thanks16:38
*** noelbk has joined #openstack-oslo16:43
*** rpodolyaka1 has joined #openstack-oslo16:48
viktorsbeekneemech: ok, thanks16:49
andreykurilinzzzeek, hi! Can you help me with strange error related to sqlalchemy in python 34 env?16:51
zzzeeksure16:51
andreykurilinzzzeek: I started process of porting Rally(https://github.com/stackforge/rally) to python 34. After several issues were fixed, I got next traceback - http://paste.openstack.org/show/151305/16:52
andreykurilinzzzeek: I have found that the problem is occured while using our BigMutableJSONEncodedDict class - https://github.com/stackforge/rally/blob/master/rally/db/sqlalchemy/types.py#L93-L9416:53
zzzeekyeah16:53
* beekneemech chuckles at BigMutableJSONEncodedDict16:54
beekneemechBecause it's Friday :-)16:54
zzzeekandreykurilin: ive seen this before but it was part of SQLA related to some typing changes and it was fixed16:54
zzzeeklet me try raw16:54
andreykurilinzzzeek: This error can be reproduced with patch-set #6 in https://review.openstack.org/#/c/141163/6 .16:55
zzzeekandreykurilin: im just going to put those types in a .py file and run it16:56
andreykurilinzzzeek: :)16:56
*** dims__ is now known as dimsum__16:56
*** kgiusti has quit IRC16:57
andreykurilinzzzeek:  maybe I found a fix for Rally code - https://review.openstack.org/#/c/143133/1/rally/db/sqlalchemy/types.py , but I'm interested in details of such issue and why it appears only in py34 env :)16:58
*** e0ne is now known as e0ne_16:58
zzzeekandreykurilin: well this error occurs in py27 as well16:58
andreykurilinzzzeek: hm...16:59
zzzeekhttp://paste.openstack.org/show/153225/16:59
*** e0ne_ is now known as e0ne16:59
zzzeekandreykurilin: its the TypeDecorator referring to a TypeDecorator that is too much for it to handle17:00
zzzeekandreykurilin: its a bug, not sure why you dont see it in py2k17:00
andreykurilinzzzeek: it's quite strange. all py26,py27 jobs are not failed17:01
*** e0ne has quit IRC17:01
andreykurilinalso, we use Rally in py27 everyday without any problems related to db17:02
*** achanda has quit IRC17:04
andreykurilinzzzeek: thanks for help17:04
*** achanda has joined #openstack-oslo17:07
*** dtantsur is now known as dtantsur|afk17:10
zzzeekandreykurilin: use this: http://paste.openstack.org/show/153226/17:11
zzzeekandreykurilin: fixed17:16
*** viktors is now known as viktors|afk17:22
*** achanda has quit IRC17:28
*** alexpilotti has joined #openstack-oslo17:28
*** achanda has joined #openstack-oslo17:28
*** achanda has quit IRC17:33
*** f13o has quit IRC17:36
*** achanda has joined #openstack-oslo17:40
*** exploreshaifali has quit IRC17:45
openstackgerritClark Boylan proposed openstack-dev/pbr: Move write_pbr_json to avoid issues with nose  https://review.openstack.org/14314617:47
*** salv-orlando has quit IRC17:50
vishysileht: ping17:56
openstackgerritClark Boylan proposed openstack-dev/pbr: Move write_pbr_json to avoid issues with nose  https://review.openstack.org/14314617:59
*** harlowja_away is now known as harlowja_18:05
*** pblaho has quit IRC18:06
jd__dimsum__:18:10
jd__[11:50:03]  <jd__>hey guys I've a question about oslo.context18:10
jd__[11:50:43]  <jd__>it now stores the context in a _request_store and you can use get_current() to get it18:10
jd__[11:50:54]  <jd__>but I cannot see any usage of that in openstack.common.log or in oslo.log18:10
jd__[11:51:07]  <jd__>do I miss something, or if a project switch to oslo.context it does not log the context anymore?18:10
jd__[11:51:19]  <jd__>dhellmann: that's likely a question for you :)18:10
dstaneki've created a wip decorator for keystone and was wondering if it would be a good fit for oslotest? the docs best describe the motivation: http://bit.ly/1zIlKD918:10
*** kgiusti has joined #openstack-oslo18:16
*** alexpilotti has quit IRC18:35
*** _amrith_ is now known as amrith18:36
*** e0ne has joined #openstack-oslo18:41
*** salv-orl_ has joined #openstack-oslo18:44
*** salv-orl_ has quit IRC18:47
*** salv-orlando has joined #openstack-oslo18:48
*** salv-orlando has quit IRC18:50
openstackgerritJeremy Stanley proposed openstack-dev/pbr: Move write_pbr_json to avoid issues with nose  https://review.openstack.org/14314618:51
openstackgerritJeremy Stanley proposed openstack-dev/pbr: Port in git sha changes from 0.10 line  https://review.openstack.org/14293118:51
openstackgerritJeremy Stanley proposed openstack-dev/pbr: Only import sphinx during hook processing  https://review.openstack.org/14284018:51
openstackgerritJeremy Stanley proposed openstack-dev/pbr: Properly check for git before getting git dir  https://review.openstack.org/14284118:51
*** salv-orlando has joined #openstack-oslo18:51
*** yassine has quit IRC18:54
*** alexpilotti has joined #openstack-oslo18:58
*** openstack has joined #openstack-oslo19:09
*** krotscheck_ is now known as krotscheck19:09
* harlowja_ is just happy taskflow new release didn't break anyone19:09
*** dougwig has joined #openstack-oslo19:09
*** openstackstatus has joined #openstack-oslo19:10
*** ChanServ sets mode: +v openstackstatus19:10
*** stevemar2 is now known as stevemar19:13
*** e0ne has quit IRC19:27
*** ihrachyshka has quit IRC19:38
*** achanda has quit IRC19:40
*** ihrachyshka has joined #openstack-oslo19:40
vishyharlowja_: believe this is the same bug: https://bugs.launchpad.net/oslo.messaging/+bug/133873219:43
vishyeasy repro steps there19:43
vishygoing to try that patch19:43
harlowja_kk19:44
vishyharlowja_: that fixes it!19:54
harlowja_sweet19:54
harlowja_seems like sileht was looking into that a little to19:54
vishyyeah19:55
vishythe redeclare fixes it for me19:55
vishyI didn’t declare anything durable19:55
vishyso I’m going to merge that patch for now19:55
vishyif sileht has a better solution in the future we will get it when we update to 1.6 i guess19:55
*** rushiagr is now known as rushiagr_away19:55
harlowja_cool19:56
*** stevemar has quit IRC20:03
*** stevemar has joined #openstack-oslo20:03
*** ihrachyshka has quit IRC20:05
*** alexpilotti has quit IRC20:15
*** e0ne has joined #openstack-oslo20:20
*** exploreshaifali has joined #openstack-oslo20:27
dhellmannjd__: the use of the built-in _local module in oslo.log to find the context in oslo_log/formatters.py needs to be replaced with a call to that function from oslo.context20:31
*** e0ne has quit IRC20:34
*** salv-orlando has quit IRC20:36
hogepodgedhellmann so I finally caught up with what you were saying with regards to group help. https://review.openstack.org/#/c/142935/20:49
*** jeckersb is now known as jeckersb_gone20:49
hogepodgeI thought the conf options would be loaded globally anyway, which isn't the case (TIL, or yesterday as it were)20:50
hogepodgeTo get at the group help the groups either need to be exposed through a new API call, or extending the current one. My concern is how to extend while maximizing compatibility with the current (group_name, [opt1, opt2, ...])20:51
hogepodgeJust allowing a group to be passed back instead of just a group name would cause problems for anything that expects a string. Maybe an optional addition to the tuple that is the group? So (group_name, [opt1, opt2,...], group). Regardless, I'm thinking it's going to need a spec.20:52
*** andreykurilin_ has joined #openstack-oslo20:53
*** raildo has joined #openstack-oslo20:59
*** stevemar has quit IRC21:01
*** andreykurilin_ has quit IRC21:02
*** andreykurilin_ has joined #openstack-oslo21:02
*** salv-orlando has joined #openstack-oslo21:03
dhellmannhogepodge: I was really only counting on the existing config generator consuming that API, but you're right we probably need to be more careful than that. A spec would be a good idea.21:05
dhellmannit'll give us a place to hash out the details21:05
hogepodgedhellmann ok, I wasn't sure how widespread use of it was. I was hoping that python just had a way to let you overload print or string for a class (I think ruby lets you do it), but apparently it's a 3 feature.21:06
hogepodgeWhere do oslo specs live?21:07
dhellmannhogepodge: you can define __str__ on a class, but you have to cast to str explicitly to take advantage of that21:07
dhellmannthere's an openstack/oslo-specs repository21:07
hogepodgeok, cool. Going offline. brb21:07
*** raildo has quit IRC21:07
*** ihrachyshka has joined #openstack-oslo21:11
*** exploreshaifali has quit IRC21:19
*** zz_sabari is now known as zz_zz_sabari21:22
*** jeckersb_gone is now known as jeckersb21:30
jd__dhellmann: so likely all the current project using oslo.context are broken right?21:45
dhellmannjd__: ?21:46
amrithdhellmann, ping!21:51
SlickNikdhellmann: had a quick question for you21:51
SlickNikregarding https://bugs.launchpad.net/trove/+bug/136618921:51
amrithSlickNik, ... beat you to it ...21:51
amrithSlickNik, I have to head out in a couple of minutes so if you can chat with dhellmann I'll catch up on the answer in scrollback.21:53
SlickNikDoes that specific issue affect oslo.concurrency, or only via oslo.utils?21:53
SlickNikamrith: no worries — catch you later.21:54
amrithbefore I run ... if I understand correctly, the issue is fixed in oslo.utils https://review.openstack.org/#/c/121657/21:56
amrithFor trove, which is still using processutils from oslo-incubator, the way to get the fix is to move to oslo.concurrency version of processutils which therefore get us the fix as that processutils uses oslo.utils which already has the fix we want.21:56
amrithand we do that in https://review.openstack.org/#/c/141081/21:56
amrithhope that makes sense, I'll read the scrollback when I get back. have a good weekend all.21:58
beekneemechamrith: It sounds like the problem is worked around in oslo.utils, but eventually there should be a fix in oslo.concurrency too.21:58
beekneemechBut I think the short answer is yes, moving to oslo.concurrency (and by extension oslo.utils) should take care of that bug.21:58
beekneemechdhellmann or jecarey can correct me if I'm understanding anything wrong.21:59
amrithBen, the fix in oslo.concurrecy seems moot to me. as the person who caused this issue (ahem, yes)21:59
amriththe fix is already in processutils in oslo.concurrency21:59
amrithyou are already depending on oslo.utils.strutils21:59
amrithand you are using mask_password from there.21:59
amrithas best as I can tell, there isn't another code path in oslo.concurrency.processutils that can end you up in a place where you attempt to convert a plain ascii string22:00
amrithother than if you go through mask_password.22:00
amrithand that is already fixed because you are using the strutils.mask_password and your strutils is oslo.utils.strutils.22:01
amrithisn't that right?22:01
amrithbeekneemech ^^22:03
*** zz_zz_sabari is now known as zz_zz_zz_sabari22:04
SlickNikAlso in that case, should this bug be marked invalid for oslo.concurrency?22:05
beekneemechamrith: Based on comment 6 in the bug I think the "fix" in mask_password wasn't the desired final fix, and it was planned to revisit this in Kilo.22:06
beekneemechamrith: SlickNik: See also https://review.openstack.org/#/c/121657/2/oslo/utils/strutils.py22:07
beekneemechBut in any case, if you're using oslo.concurrency I think you should be in good shape.22:08
beekneemechAny further changes should be transparent to consumers (I would think anyway).22:09
SlickNikbeekneemech: Ah, that clarifies things a bit — thanks! Also, that's our plan of action (i.e. use oslo.concurrency and hope the fixes will land in that module and we can stay oblivious).22:12
*** ajo_ has quit IRC22:13
beekneemechYep, sounds good22:15
*** dimsum__ has quit IRC22:23
*** andreykurilin_ has quit IRC22:29
*** kgiusti has quit IRC22:31
*** rpodolyaka2 has joined #openstack-oslo22:48
*** rpodolyaka2 has quit IRC22:59
openstackgerritMerged openstack/oslo-specs: Graduating Policy  https://review.openstack.org/14016123:00
*** ihrachyshka has quit IRC23:01
*** noelbk has quit IRC23:04
*** tedross has quit IRC23:09
*** ajo has joined #openstack-oslo23:10
*** gordc has quit IRC23:18
*** salv-orlando has quit IRC23:29
*** salv-orlando has joined #openstack-oslo23:31
*** noelbk has joined #openstack-oslo23:34
*** ajo has quit IRC23:40
*** dimsum__ has joined #openstack-oslo23:41
openstackgerritJoshua Harlow proposed openstack/oslo.concurrency: Add a reader/writer lock  https://review.openstack.org/13821723:42

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