Tuesday, 2015-03-31

*** sreshetn2 has quit IRC00:00
*** achanda_ has joined #openstack-oslo00:03
*** achanda has quit IRC00:07
*** gtt116 has joined #openstack-oslo00:07
boris-42harlowja_: I have document00:07
boris-42harlowja_: related to that00:08
boris-42harlowja_: but DB archiving stuff is crap00:08
boris-42and should be just removed from source code00:08
boris-42=)00:08
*** achanda_ has quit IRC00:08
boris-42harlowja_: https://docs.google.com/a/mirantis.com/document/d/1WGrIgMtWJqPDyT6PkPeZhNpej2Q9Mwimula8S8lYGV4/edit#heading=h.xeo58l4ryngk00:09
*** sigmavirus24 is now known as sigmavirus24_awa00:09
boris-42sputnik13: ^00:09
*** takedakn has quit IRC00:09
*** takedakn has joined #openstack-oslo00:10
sputnik13what's the purpose of allowing things to be restored...  an things realistically be restored?  volumes, images, etc once deleted how do you restore?00:12
harlowja_magic man00:12
harlowja_magic00:12
harlowja_lol00:12
sputnik13it's a lofty goal, but I feel like the database is the wrong place to implement something like that00:13
harlowja_no disagreement from me00:13
vipulYea you could never restore those .. if that was the intention then it's flawed00:13
sputnik13normally you have "order management" type systems that you can reach in to and cancel orders that are "in flight"00:13
vipuli do think that for logging/historical purposes.. there is some value00:13
boris-42sputnik13: we are not "deleting" resource instantly00:13
boris-42sputnik13: just hiding it and removing from DB to special table00:13
boris-42sputnik13: this is case where we need restorable stuff00:14
sputnik13that's what's being done in the database true, but the underlying resource is likely gone00:14
boris-42sputnik13: but this should be well configured and support instant deletion as well00:14
sputnik13so restoring the database entry seems to have limited use to me00:14
boris-42sputnik13: in current implentation with this soft deletion things are very bad00:14
boris-42sputnik13: we shouldn't have this "mark as deleted" column at all00:15
boris-42sputnik13: and should not actually delete resource*00:15
boris-42sputnik13: if we need to support restorable resources*00:15
sputnik13right so the elephant in the room is should restorable resources be supported00:16
sputnik13I think00:16
boris-42if we don't need that, then we need just to get rid of soft_deleted column and that is all00:16
boris-42sputnik13:  but archiving db is very very bad approach in any case00:16
sputnik13in order to have true support for restorations checkpointing needs to be supported and well coordinated across resources...  that's a massive undertaking00:17
sputnik13boris-42: agreed00:17
sputnik13damn gotta run and catch my bus :)00:17
boris-42sputnik13: based on my experience in OpenStack this task is just impossible00:17
sputnik13boris-42: we are in violent agreement :)00:17
boris-42sputnik13: so I would prefer just to get rid of soft_deleted column =)00:17
harlowja_calm down children, ha00:18
sputnik13+200:18
harlowja_oh; violent agreement, ha00:18
vipuleven that is a massive undertaking in openstack ;)00:18
*** spyyy has quit IRC00:18
*** sputnik13 has quit IRC00:18
harlowja_just requires lots of peace pipe smoking00:19
harlowja_*this is the way of openstack, ha00:19
*** spyyy has joined #openstack-oslo00:20
*** rossella_s has quit IRC00:20
*** rossella_s has joined #openstack-oslo00:20
*** spyyy has quit IRC00:21
vipulharlowja_: need to start passing that around a LOT more ;)00:21
harlowja_ya; we might smoke all of colorado00:21
harlowja_lol00:22
*** gtt116 has quit IRC00:22
harlowja_and finally get somewhere, lol00:22
*** takedakn has quit IRC00:26
*** tsekiyam_ has joined #openstack-oslo00:26
*** dims has joined #openstack-oslo00:27
*** mtanino_ has joined #openstack-oslo00:27
*** mtanino has quit IRC00:29
*** tsekiyama has quit IRC00:30
*** tsekiyam_ has quit IRC00:31
*** zzzeek has quit IRC00:44
openstackgerritBrant Knudson proposed openstack/oslo.config: Fix doc build warning  https://review.openstack.org/16912500:49
*** YorikSar has quit IRC00:55
openstackgerritJoshua Harlow proposed openstack/taskflow: Optionally do caching/updating of flowdetail in storage  https://review.openstack.org/16785700:57
openstackgerritBrant Knudson proposed openstack/oslo.config: Generate docs for constructor  https://review.openstack.org/16912701:06
openstackgerritJoshua Harlow proposed openstack/taskflow: Optionally do caching/updating of flowdetail in storage  https://review.openstack.org/16785701:08
*** ChuckC has quit IRC01:08
*** amotoki has joined #openstack-oslo01:12
*** amotoki has quit IRC01:13
*** haigang has joined #openstack-oslo01:13
openstackgerritBrant Knudson proposed openstack/oslo.config: Add deprecated_for_removal support to the config generator  https://review.openstack.org/16912901:16
openstackgerritJoshua Harlow proposed openstack/taskflow: Address concurrent mutation of sqlalchemy backend  https://review.openstack.org/9868901:25
*** cpg is now known as cpg|away01:30
*** cpg|away is now known as cpg01:32
*** ChuckC has joined #openstack-oslo01:34
*** tsekiyama has joined #openstack-oslo01:37
*** stevemar has joined #openstack-oslo01:39
*** tsekiyama has quit IRC01:41
*** amotoki has joined #openstack-oslo01:42
*** yamahata has joined #openstack-oslo01:47
*** amotoki has quit IRC01:47
*** amotoki has joined #openstack-oslo01:47
*** amotoki has quit IRC01:52
*** gtt116 has joined #openstack-oslo01:59
*** sputnik13 has joined #openstack-oslo02:04
*** sputnik13 has quit IRC02:06
*** sputnik13 has joined #openstack-oslo02:06
*** dims has quit IRC02:07
*** amrith is now known as _amrith_02:07
*** dims has joined #openstack-oslo02:08
*** sputnik13 has quit IRC02:11
*** sputnik13 has joined #openstack-oslo02:12
*** dims has quit IRC02:14
*** amotoki has joined #openstack-oslo02:28
*** achanda has joined #openstack-oslo02:34
*** sputnik13 has quit IRC02:35
*** amotoki has quit IRC02:43
*** crc32 has quit IRC02:44
*** achanda has quit IRC02:44
*** yamahata has quit IRC02:44
*** achanda has joined #openstack-oslo02:45
*** mtanino_ has quit IRC02:50
*** haigang has quit IRC02:57
gtt116Making oslo.messaging notifiers support TTL of notification, does it make sense?03:03
gtt116All the RPC messages' TTL is already set by oslo03:05
*** harlowja_ is now known as harlowja_away03:07
*** amotoki has joined #openstack-oslo03:10
*** bknudson has quit IRC03:16
*** amotoki has quit IRC03:16
*** haigang has joined #openstack-oslo03:22
*** haigang has quit IRC03:27
*** sputnik13 has joined #openstack-oslo03:33
*** amotoki has joined #openstack-oslo03:43
*** haigang has joined #openstack-oslo03:55
*** sputnik13 has quit IRC04:02
*** sputnik13 has joined #openstack-oslo04:03
*** achanda has quit IRC04:12
*** BrianShang_ has quit IRC04:13
*** BrianShang has joined #openstack-oslo04:15
*** achanda has joined #openstack-oslo04:19
*** sputnik13 has quit IRC04:20
*** sputnik13 has joined #openstack-oslo04:21
*** achanda has quit IRC04:27
*** sputnik13 has quit IRC04:30
*** achanda has joined #openstack-oslo04:30
*** sputnik13 has joined #openstack-oslo04:31
*** achanda has quit IRC04:40
*** achanda has joined #openstack-oslo04:40
*** achanda has quit IRC04:51
*** sputnik13 has quit IRC04:55
*** rpodolyaka1 has joined #openstack-oslo05:15
*** harlowja_at_home has joined #openstack-oslo05:19
*** achanda has joined #openstack-oslo05:22
*** harlowja_at_home has quit IRC05:25
*** dulek has joined #openstack-oslo05:29
*** rpodolyaka1 has quit IRC05:38
*** rpodolyaka1 has joined #openstack-oslo05:38
*** exploreshaifali has joined #openstack-oslo05:40
*** vigneshvar has joined #openstack-oslo05:43
*** vigneshvar_ has joined #openstack-oslo05:43
*** ajo has joined #openstack-oslo05:44
*** exploreshaifali has quit IRC05:53
*** achanda has quit IRC06:08
*** achanda has joined #openstack-oslo06:08
*** rpodolyaka1 has quit IRC06:11
*** rpodolyaka1 has joined #openstack-oslo06:16
*** haigang has quit IRC06:27
*** haigang has joined #openstack-oslo06:27
*** amotoki has quit IRC06:27
*** sreshetn2 has joined #openstack-oslo06:29
*** haigang has quit IRC06:32
*** BrianShang has quit IRC06:35
*** BrianShang has joined #openstack-oslo06:36
*** sreshetn2 has quit IRC06:38
*** BrianShang_ has joined #openstack-oslo06:41
*** sreshetn2 has joined #openstack-oslo06:44
*** BrianShang has quit IRC06:44
*** YorikSar has joined #openstack-oslo06:47
*** jamielennox is now known as jamielennox|away06:49
*** amotoki has joined #openstack-oslo06:53
*** BrianShang_ has quit IRC07:00
*** amotoki has quit IRC07:01
*** amotoki has joined #openstack-oslo07:02
*** BrianShang has joined #openstack-oslo07:02
*** liusheng has joined #openstack-oslo07:04
*** achanda has quit IRC07:05
*** ajo_ has joined #openstack-oslo07:07
*** ajo has quit IRC07:10
*** haigang has joined #openstack-oslo07:12
*** stevemar has quit IRC07:16
*** rpodolyaka1 has quit IRC07:17
*** vigneshvar has quit IRC07:17
*** vigneshvar_ has quit IRC07:17
openstackgerritAngus Lees proposed openstack/oslo.rootwrap: Avoid calling sudo just to change users  https://review.openstack.org/16284907:17
*** takedakn has joined #openstack-oslo07:21
*** haypo has quit IRC07:21
*** jaosorior has joined #openstack-oslo07:28
*** rushiagr_away is now known as rushiagr07:30
*** liusheng has quit IRC07:33
*** liusheng has joined #openstack-oslo07:34
*** rushiagr is now known as rushiagr_away07:47
*** YorikSar has quit IRC07:56
*** viktors|afk is now known as viktors08:01
*** liusheng has quit IRC08:03
*** liusheng has joined #openstack-oslo08:04
*** sreshetn2 has quit IRC08:06
*** spyyy has joined #openstack-oslo08:09
*** sreshetn2 has joined #openstack-oslo08:11
*** ndipanov has joined #openstack-oslo08:15
*** sreshetn2 has quit IRC08:19
*** e0ne has joined #openstack-oslo08:29
*** YorikSar has joined #openstack-oslo08:30
*** rushiagr_away is now known as rushiagr08:34
*** haigang has quit IRC08:38
*** e0ne has quit IRC08:45
*** sreshetn2 has joined #openstack-oslo08:47
*** SridharGaddam has joined #openstack-oslo08:50
-openstackstatus- NOTICE: CI Check/Gate pipelines currently stuck due to a bad dependency creeping in the system. No need to recheck your patches at the moment.08:54
*** ChanServ changes topic to "CI Check/Gate pipelines currently stuck due to a bad dependency creeping in the system. No need to recheck your patches at the moment."08:54
*** rushiagr is now known as rushiagr_away08:57
*** rushiagr_away is now known as rushiagr08:58
*** takedakn has quit IRC09:04
openstackgerritTiantian Gao proposed openstack/oslo.messaging: Add functional test for notify.logger  https://review.openstack.org/16922509:09
*** sreshetn2 has quit IRC09:12
openstackgerritTiantian Gao proposed openstack/oslo.messaging: Add some exclude directory flake8, and update gitignore  https://review.openstack.org/16922809:13
*** amotoki_ has joined #openstack-oslo09:20
*** sreshetn2 has joined #openstack-oslo09:24
openstackgerritTiantian Gao proposed openstack/oslo.messaging: Add functional test for notify.logger  https://review.openstack.org/16922509:28
*** sreshetn2 has quit IRC09:33
*** e0ne has joined #openstack-oslo09:39
*** jamielennox|away is now known as jamielennox09:41
*** ajo_ is now known as ajo09:42
*** jamielennox is now known as jamielennox|away09:47
*** cdent has joined #openstack-oslo09:52
*** liusheng has quit IRC10:00
*** liusheng has joined #openstack-oslo10:01
*** dims has joined #openstack-oslo10:03
*** achanda has joined #openstack-oslo10:06
*** sreshetn2 has joined #openstack-oslo10:09
*** achanda has quit IRC10:10
*** e0ne is now known as e0ne_10:19
*** e0ne_ is now known as e0ne10:23
*** e0ne is now known as e0ne_10:37
*** e0ne_ is now known as e0ne10:40
*** flaper87 has quit IRC11:10
*** _amrith_ is now known as amrith11:12
*** tsufiev_ has joined #openstack-oslo11:15
*** flaper87 has joined #openstack-oslo11:22
*** pblaho has joined #openstack-oslo11:24
*** flaper87 has quit IRC11:26
*** flaper87 has joined #openstack-oslo11:26
*** salv-orlando has quit IRC11:26
*** hogepodge has quit IRC11:27
*** cpg is now known as cpg|away11:34
*** salv-orlando has joined #openstack-oslo11:40
*** rushiagr is now known as rushiagr_away11:42
*** tsufiev_ is now known as tsufiev11:42
*** ChanServ changes topic to "#openstack-oslo"11:50
-openstackstatus- NOTICE: Check/Gate unstuck, feel free to recheck your abusively-failed changes.11:50
*** ihrachyshka has joined #openstack-oslo11:52
*** ujjain has quit IRC11:54
*** sreshetn2 has quit IRC11:54
*** rushiagr_away is now known as rushiagr11:54
*** sreshetn2 has joined #openstack-oslo11:56
*** ndipanov has quit IRC12:03
openstackgerritAngus Lees proposed openstack/oslo.rootwrap: Avoid calling sudo just to change users  https://review.openstack.org/16284912:06
*** kgiusti has joined #openstack-oslo12:18
*** amotoki_ has quit IRC12:20
*** amrith is now known as _amrith_12:21
*** dims has quit IRC12:25
*** dims has joined #openstack-oslo12:25
*** amotoki_ has joined #openstack-oslo12:34
*** amotoki_ has quit IRC12:34
*** jecarey has quit IRC12:40
*** e0ne is now known as e0ne_12:42
*** bknudson has joined #openstack-oslo12:42
*** e0ne_ is now known as e0ne12:43
*** gordc has joined #openstack-oslo12:50
*** ajo has quit IRC13:01
*** ajo has joined #openstack-oslo13:01
*** ajo has quit IRC13:01
*** hogepodge has joined #openstack-oslo13:05
*** e0ne is now known as e0ne_13:08
*** e0ne_ is now known as e0ne13:13
*** ujjain has joined #openstack-oslo13:17
*** mriedem_away is now known as mriedem13:20
*** rushiagr is now known as rushiagr_away13:29
*** jaypipes has quit IRC13:29
*** sreshetn2 has quit IRC13:35
*** sreshetn2 has joined #openstack-oslo13:36
*** jaypipes has joined #openstack-oslo13:43
*** sreshetn2 has quit IRC13:45
*** sreshetn2 has joined #openstack-oslo13:49
*** _amrith_ is now known as amrith13:51
*** amotoki_ has joined #openstack-oslo13:51
*** zzzeek has joined #openstack-oslo13:57
*** sigmavirus24_awa is now known as sigmavirus2413:59
*** rushiagr_away is now known as rushiagr13:59
*** jecarey has joined #openstack-oslo13:59
*** rpodolyaka1 has joined #openstack-oslo14:01
*** rpodolyaka1 has quit IRC14:01
*** rpodolyaka1 has joined #openstack-oslo14:01
*** sreshetn2 has quit IRC14:06
*** sreshetn2 has joined #openstack-oslo14:06
*** rpodolyaka1 has quit IRC14:07
*** ChuckC has quit IRC14:09
sdagueso ... https://github.com/eventlet/eventlet/pull/75 - yay!14:14
sdaguewhich means we should be able to deprecate a bit of oslo.log oslo.service in next cycle14:14
*** enikanorov has quit IRC14:23
*** enikanorov has joined #openstack-oslo14:27
*** daniel3_ has joined #openstack-oslo14:28
*** viktors has quit IRC14:30
*** zz_jgrimm is now known as jgrimm14:31
*** ChuckC has joined #openstack-oslo14:39
*** amotoki_ has quit IRC14:49
*** ndipanov has joined #openstack-oslo14:51
*** jaypipes has quit IRC14:53
*** sputnik13 has joined #openstack-oslo14:55
*** jungleboyj_afk has quit IRC14:57
*** dulek has quit IRC14:58
*** cdent has quit IRC15:02
dimssdague: very cool15:04
*** dims has quit IRC15:05
*** BrianShang_ has joined #openstack-oslo15:06
*** dims_ has joined #openstack-oslo15:08
*** marun has joined #openstack-oslo15:08
*** BrianShang has quit IRC15:08
marundhellmann: Hi!15:09
marundhellmann: I have a question about oslo_config15:09
marundhellmann: How common is it for projects to use oslo_config.cfg.CONF instead of a local ConfigMapping instance?15:09
*** sputnik13 has quit IRC15:10
*** tsekiyama has joined #openstack-oslo15:22
*** zigo__ is now known as zigo15:23
*** exploreshaifali has joined #openstack-oslo15:25
*** stevemar has joined #openstack-oslo15:29
*** BrianShang has joined #openstack-oslo15:36
*** BrianShang_ has quit IRC15:37
*** YorikSar has quit IRC15:37
elarsonusing oslo_config I'm installing some plugins via stevedore that have some config options that I want to add. I find that if no plugins are found and I call cfg.CONF.reload_config_files it gets an error. is that considered normal?15:39
* elarson would have thought that if no new options were added reload_config_files would essentially be a safe noop15:39
*** harlowja_at_home has joined #openstack-oslo15:42
*** harlowja_at_home has quit IRC15:44
*** daniel3_ has quit IRC15:47
*** daniel3_ has joined #openstack-oslo15:49
openstackgerritBen Nemec proposed openstack/oslo.config: Fix logging of deprecated opts with dest override  https://review.openstack.org/16939215:51
*** sreshetn2 has quit IRC15:52
openstackgerritKen Giusti proposed openstack/oslo.messaging: Properly deserialize received AMQP 1.0 messages  https://review.openstack.org/16939615:56
*** stpierre has joined #openstack-oslo15:58
*** e0ne is now known as e0ne_16:03
*** arnaud___ has joined #openstack-oslo16:08
*** e0ne_ has quit IRC16:08
*** sreshetn2 has joined #openstack-oslo16:14
*** e0ne has joined #openstack-oslo16:14
*** mtanino has joined #openstack-oslo16:27
*** yamahata has joined #openstack-oslo16:29
*** arnaud___ has quit IRC16:32
*** daniel3_ has quit IRC16:37
*** YorikSar has joined #openstack-oslo16:42
*** sputnik13 has joined #openstack-oslo16:45
*** harlowja_away is now known as harlowja_16:45
*** jaypipes has joined #openstack-oslo16:48
*** dims_ has quit IRC16:51
*** dims_ has joined #openstack-oslo16:51
*** e0ne is now known as e0ne_16:56
*** andreykurilin_ has joined #openstack-oslo17:05
*** e0ne_ is now known as e0ne17:06
*** daniel3_ has joined #openstack-oslo17:12
openstackgerritayoung proposed openstack/oslo.policy: Lists for Generic Checks  https://review.openstack.org/16904517:20
*** achanda has joined #openstack-oslo17:23
*** daniel3_ has quit IRC17:27
*** daniel3_ has joined #openstack-oslo17:28
*** e0ne has quit IRC17:33
openstackgerritayoung proposed openstack/oslo.policy: Lists for Generic Checks  https://review.openstack.org/16904517:36
*** sheeprine has quit IRC17:43
*** sreshetn2 has quit IRC17:46
*** sheeprine has joined #openstack-oslo17:50
*** pblaho has quit IRC17:55
*** mtanino has quit IRC17:56
*** cpg|away is now known as cpg18:03
*** jamielennox|away is now known as jamielennox18:04
*** e0ne has joined #openstack-oslo18:09
*** e0ne is now known as e0ne_18:18
*** ihrachyshka has quit IRC18:22
openstackgerritJoshua Harlow proposed openstack/taskflow: Make an attempt at having taskflow exceptions print causes better  https://review.openstack.org/16708718:23
*** e0ne_ has quit IRC18:23
*** stpierre has quit IRC18:23
openstackgerritJoshua Harlow proposed openstack/taskflow: Make an attempt at having taskflow exceptions print causes better  https://review.openstack.org/16708718:25
*** stpierre has joined #openstack-oslo18:26
*** e0ne has joined #openstack-oslo18:36
openstackgerritJoshua Harlow proposed openstack/taskflow: Allow ls() to list recursively (using breadth-first)  https://review.openstack.org/16736418:42
openstackgerritJoshua Harlow proposed openstack/taskflow: Allow ls() to list recursively (using breadth-first)  https://review.openstack.org/16736418:44
*** achanda has quit IRC18:49
*** jaosorior has quit IRC18:52
*** stevemar has quit IRC18:54
harlowja_jd__ dims_ a simple one if u got some time; might as well merge it for sanity sake18:55
harlowja_https://review.openstack.org/#/c/149174/18:55
harlowja_for ye olde python2.618:55
*** pblaho has joined #openstack-oslo18:55
*** sreshetn2 has joined #openstack-oslo18:56
jd__harlowja_: actually we don't gate py26 so I think we don't support it, you still want that?19:03
harlowja_oh19:03
harlowja_'gate-oslo.serialization-python26' ?19:03
harlowja_u sure :-P19:04
*** stevemar has joined #openstack-oslo19:04
jd__harlowja_: why isn't it sorted?19:06
* jd__ beats Jenkins19:06
harlowja_unsure19:06
harlowja_ya, i've seen its sort order change19:06
harlowja_must be some bug19:06
jd__harlowja_: well can I -1 because there's no test?19:06
harlowja_sureee19:07
harlowja_i add test19:07
*** stevemar has quit IRC19:08
*** stevemar has joined #openstack-oslo19:08
* harlowja_ beats self with stick19:10
harlowja_lol19:10
*** salv-orl_ has joined #openstack-oslo19:11
*** spyyy has quit IRC19:12
*** salv-orlando has quit IRC19:13
*** sreshetn2 has quit IRC19:14
*** rushiagr is now known as rushiagr_away19:14
sdaguestevemar: I just bumped - https://bugs.launchpad.net/nova/+bug/1437992 to critical on the oslo side, because it's a major regression in the caching strategy by the whole policy mechanism19:23
openstackLaunchpad bug 1437992 in oslo-incubator "policy file in policy.d will be reloaded every rest api call" [Critical,In progress] - Assigned to Eli Qiao (taget-9)19:23
stevemarsdague, yay19:23
stevemardhellmann, sigmavirus24 ^19:24
sdaguewe have a reasonable work around by deleting all our policy.d files in nova right now19:24
stevemargood thing eli has a patch for it already19:24
sigmavirus24thanks sdague19:24
sdaguehowever, it basically makes that entire mechanism unuseful19:24
sigmavirus24we've also deprecated policy dirs, no?19:25
stevemari'll look at it in a few minutes19:25
sdaguesigmavirus24: we've deprecated it being configurable19:25
sigmavirus24right, by which I only mean, this won't be a problem for long19:26
sdaguesigmavirus24: no, it will remain a problem19:26
sdaguethe deprecation was just that the directory will always be policy.d, not bubba-gump-shrimp.d19:26
*** achanda has joined #openstack-oslo19:27
sdaguebut the fact that the recursive policy file check is never cached will remain a problem19:27
*** alexpilotti has joined #openstack-oslo19:41
*** sreshetn2 has joined #openstack-oslo19:43
*** sheeprine has quit IRC19:44
dims_bubba-gump-shrimp.d - hahaha19:48
*** sheeprine has joined #openstack-oslo19:50
dims_harlowja_: related to what stevemar / sdague were talking about. does this look good to you? https://review.openstack.org/#/c/168819/19:52
dims_bnemec: ^19:52
harlowja_ah, caching19:52
harlowja_hmmm19:52
harlowja_i sometimes just want to delete that19:52
harlowja_lol19:52
openstackgerritMerged openstack/taskflow: fix renamed class to call super correctly  https://review.openstack.org/16905919:52
harlowja_*let the filesystem cache itself...19:52
harlowja_that review though sorta confuses me, ha19:53
harlowja_how does one even get a directory into the cache in the first place :-P19:54
harlowja_>>> x = '/homes/harlowja/dev/os/cloud-init'19:54
harlowja_>>> with open(x, 'rb') as fh:19:54
harlowja_...   b = fh.read()19:54
harlowja_...19:54
harlowja_Traceback (most recent call last):19:54
harlowja_  File "<stdin>", line 1, in <module>19:54
harlowja_IsADirectoryError: [Errno 21] Is a directory: '/homes/harlowja/dev/os/cloud-init'19:54
harlowja_...19:54
dims_good point harlowja_ that's not right19:57
harlowja_dims_ added my comments/question19:57
dims_harlowja_: guess he is trying to use the existing _FILE_CACHE to store info about directories20:02
harlowja_maybe20:02
harlowja_read_cached_file seems like the wrong function though, ha20:02
dims_right20:02
*** andreykurilin_ has quit IRC20:06
*** achanda has quit IRC20:20
*** alexpilotti has quit IRC20:21
dims_sigmavirus24: stevemar: left a counter proposal patch in Eli's review20:21
dims_sdague: ^20:21
sigmavirus24dims_: thanks20:22
*** BrianShang_ has joined #openstack-oslo20:24
harlowja_jd__ actually now that i think about it https://review.openstack.org/#/c/149174/ is sorta hard to add a test for; since it requires a py2.6 runtime while decoding but a 2.7 runtime while encoding (to even format it in the first place)20:26
jd__harlowja_: ok20:26
*** spyyy has joined #openstack-oslo20:27
*** BrianShang has quit IRC20:27
sigmavirus24dims_: I'm not certain that oslo.policy should need to handle the mtime information itself20:31
dims_sigmavirus24: overloading that function in fileutils is overkill. don't want to pay that tax long term20:31
sigmavirus24dims_: I agree20:32
stevemardims_, mtime is an interesting approac20:32
dims_stevemar: essentially i pruned down what eli was using in fileutils20:33
dims_and stuck it into Enforcer20:33
sigmavirus24Why aren't we introducing a separate function for directories?20:36
*** sreshetn1 has joined #openstack-oslo20:38
stevemarif we use dims_ approach, we don't need to make changes to fileutils20:38
dims_correct20:38
*** sreshetn2 has quit IRC20:39
sigmavirus24Note, say it had to be in fileutils ;)20:40
*** david-lyle has quit IRC20:40
sigmavirus24We can mimic the existing function in fileutils in oslo_policy just to keep the separation of concerns clean20:41
dims_sigmavirus24: right, if it had to be fileutils, we would have a separate set of methods to cache directory information.20:41
*** e0ne has quit IRC20:42
*** e0ne has joined #openstack-oslo20:42
sigmavirus24The other thing is that storing teh cache on the enforcer isn't going to fix this bug in all services20:42
sigmavirus24Not all services use a single global enforcer20:42
* sigmavirus24 is also not following this closely as he's on his phone and it's hard to keep stuff straight20:43
dims_sigmavirus24: then i'd advocate a global variable in the existing code outside the Enforcer20:43
sigmavirus24Also is this intended to be released for Kilo?20:44
dims_sigmavirus24: yes20:44
stevemarsigmavirus24, yeah, it's been tagged as critical now20:52
*** e0ne is now known as e0ne_20:52
*** e0ne_ is now known as e0ne20:52
*** amrith is now known as _amrith_20:56
dims_stevemar: want me to file a separate review or patch that one?21:00
stevemardims_, try a separate review21:00
dims_stevemar: ack21:01
stevemardims_, you can also pass in _policy_directory_cache - as a new param to _walk_through_policy_directory21:01
dims_ack21:02
stevemardims_, also, it's not really a cache :P21:10
stevemar_policy_dir_last_access ?21:10
*** achanda has joined #openstack-oslo21:14
*** ndipanov has quit IRC21:18
*** e0ne has quit IRC21:23
*** exploreshaifali has quit IRC21:25
*** jgrimm is now known as zz_jgrimm21:26
*** spyyy has quit IRC21:33
*** gordc has quit IRC21:34
*** sreshetn1 has quit IRC21:36
*** pblaho has quit IRC21:37
*** mriedem is now known as mriedem_away21:45
*** stevemar has quit IRC21:51
openstackgerritMerged openstack/taskflow: Make an attempt at having taskflow exceptions print causes better  https://review.openstack.org/16708721:56
*** harlowja_ is now known as harlowja_away22:04
*** harlowja_away is now known as harlowja_22:07
*** salv-orl_ has quit IRC22:09
*** salv-orlando has joined #openstack-oslo22:13
*** mtanino has joined #openstack-oslo22:15
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.policy: Avoid reloading policy files in policy.d for every call  https://review.openstack.org/16953522:18
*** jlvillal_ has joined #openstack-oslo22:19
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.policy: Avoid reloading policy files in policy.d for every call  https://review.openstack.org/16953522:19
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.policy: Avoid reloading policy files in policy.d for every call  https://review.openstack.org/16953522:20
*** harlowja_ has quit IRC22:21
dims_sigmavirus24: ^^^22:22
dims_looks like stevemar and harlowja_ are not around22:22
sigmavirus24I'm not a core =P22:22
* sigmavirus24 is looking though22:22
dims_sigmavirus24: no issues, just need eyes :)22:23
dims_too late for jd__ or flaper87, bnemec around?22:24
dims_thanks ian!22:25
*** harlowja has joined #openstack-oslo22:26
sigmavirus24yw!22:28
*** sreshetn1 has joined #openstack-oslo22:33
*** jecarey has quit IRC22:35
*** sreshetn1 has quit IRC22:37
*** jlvillal_ has quit IRC22:38
*** stpierre has quit IRC22:39
*** sigmavirus24 is now known as sigmavirus24_awa22:42
*** andreykurilin_ has joined #openstack-oslo22:45
openstackgerritJoshua Harlow proposed openstack/taskflow: Add a suspension engine section  https://review.openstack.org/16954822:46
*** daniel3_ has quit IRC22:47
openstackgerritJoshua Harlow proposed openstack/taskflow: Add a suspension engine section  https://review.openstack.org/16954822:48
*** jlvillal_ has joined #openstack-oslo22:48
openstackgerritJoshua Harlow proposed openstack/taskflow: Add a suspension engine section  https://review.openstack.org/16954822:49
*** jlvillal_ has quit IRC22:49
*** devlaps has joined #openstack-oslo22:50
*** salv-orlando has quit IRC22:51
*** devlaps has quit IRC22:54
*** salv-orlando has joined #openstack-oslo22:59
*** flwang has quit IRC23:00
*** _amrith_ is now known as amrith23:01
*** tsekiyam_ has joined #openstack-oslo23:04
*** tsekiyama has quit IRC23:07
*** tsekiyam_ has quit IRC23:08
*** jaypipes has quit IRC23:09
*** flwang has joined #openstack-oslo23:13
*** dims_ has quit IRC23:14
*** davideagnello has quit IRC23:15
*** sputnik13 has quit IRC23:15
*** dims_ has joined #openstack-oslo23:17
*** marun has quit IRC23:22
*** jaypipes has joined #openstack-oslo23:23
*** andreykurilin_ has quit IRC23:31
*** sreshetn1 has joined #openstack-oslo23:34
*** sreshetn1 has quit IRC23:38
*** zzzeek has quit IRC23:38
*** achanda has quit IRC23:41
*** sputnik13 has joined #openstack-oslo23:42
*** achanda has joined #openstack-oslo23:55

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