Tuesday, 2017-07-18

openstackgerritMerged openstack/oslo.serialization master: Revert "Explicitly raise ValueError in to_primitive"  https://review.openstack.org/48439600:08
*** ansmith has joined #openstack-oslo00:26
*** boris-42__ has quit IRC00:27
*** dave-mccowan has quit IRC00:42
*** dave-mccowan has joined #openstack-oslo00:46
*** gcb has quit IRC00:53
*** rajathagasthya has quit IRC01:03
*** boden has quit IRC01:10
*** trungnv has quit IRC01:12
*** zhangguoqing has joined #openstack-oslo01:24
*** harlowja has quit IRC01:34
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.log master: Updated from global requirements  https://review.openstack.org/48456701:54
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.messaging master: Updated from global requirements  https://review.openstack.org/48456801:54
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.policy master: Updated from global requirements  https://review.openstack.org/48456901:55
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.reports master: Updated from global requirements  https://review.openstack.org/48457001:55
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.versionedobjects master: Updated from global requirements  https://review.openstack.org/48457101:55
openstackgerritOpenStack Proposal Bot proposed openstack/pycadf master: Updated from global requirements  https://review.openstack.org/47013701:55
openstackgerritOpenStack Proposal Bot proposed openstack/taskflow master: Updated from global requirements  https://review.openstack.org/48459101:58
*** dave-mccowan has quit IRC02:02
*** xgerman__ has joined #openstack-oslo02:06
*** samueldmq_ has joined #openstack-oslo02:06
*** ildikov_ has joined #openstack-oslo02:06
*** patrickeast_ has joined #openstack-oslo02:06
*** nikhil_ has joined #openstack-oslo02:06
*** nikhil_ is now known as Guest7866902:06
*** _d34dh0r53_ has joined #openstack-oslo02:12
*** toabctl_ has joined #openstack-oslo02:13
*** dhellmann has quit IRC02:13
*** patrickeast has quit IRC02:13
*** ildikov has quit IRC02:13
*** toabctl has quit IRC02:13
*** larainema has quit IRC02:13
*** d34dh0r53 has quit IRC02:13
*** samueldmq has quit IRC02:13
*** xgerman_ has quit IRC02:13
*** nikhil has quit IRC02:13
*** ildikov_ is now known as ildikov02:13
*** patrickeast_ is now known as patrickeast02:13
*** toabctl_ is now known as toabctl02:14
*** samueldmq_ is now known as samueldmq02:14
*** xgerman__ is now known as xgerman_02:14
*** Guest78669 is now known as nikhil02:14
openstackgerritMerged openstack/osprofiler master: Update .gitignore  https://review.openstack.org/48098202:16
*** dhellmann has joined #openstack-oslo02:19
*** gcb has joined #openstack-oslo02:19
*** dave-mccowan has joined #openstack-oslo02:26
*** trungnv has joined #openstack-oslo02:31
*** yamamoto has joined #openstack-oslo02:41
*** dave-mccowan has quit IRC02:44
*** sdague has quit IRC03:29
*** harlowja has joined #openstack-oslo03:40
*** zhangguoqing has quit IRC03:49
*** links has joined #openstack-oslo03:55
*** larainema has joined #openstack-oslo04:03
*** aselius has quit IRC04:17
*** chhavi has joined #openstack-oslo04:21
openstackgerritMerged openstack/oslo.log master: Updated from global requirements  https://review.openstack.org/48456704:28
*** rajathagasthya has joined #openstack-oslo04:31
openstackgerritMerged openstack/oslo.versionedobjects master: Updated from global requirements  https://review.openstack.org/48457104:41
openstackgerritMerged openstack/taskflow master: Updated from global requirements  https://review.openstack.org/48459104:44
*** nicolasbock has joined #openstack-oslo04:44
*** zhangguoqing has joined #openstack-oslo04:48
*** harlowja has quit IRC04:54
*** zzzeek has quit IRC04:55
*** bnemec has quit IRC05:13
*** nicolasbock has quit IRC05:19
*** nicolasbock has joined #openstack-oslo05:21
*** rcernin has joined #openstack-oslo05:29
*** rajathagasthya has quit IRC05:31
*** zzzeek has joined #openstack-oslo05:36
*** zzzeek has quit IRC05:37
*** zzzeek has joined #openstack-oslo05:38
*** lpetrut_ has joined #openstack-oslo06:11
openstackgerritLingyong Xu proposed openstack/oslotest master: Use assertIsNone(...) instead of assertIs(None,...)  https://review.openstack.org/48464206:49
openstackgerritMerged openstack/oslo.messaging master: Updated from global requirements  https://review.openstack.org/48456806:50
*** lpetrut_ has quit IRC07:02
openstackgerritMerged openstack/oslo.policy master: Updated from global requirements  https://review.openstack.org/48456907:07
openstackgerritMerged openstack/oslo.reports master: Updated from global requirements  https://review.openstack.org/48457007:15
*** lpetrut_ has joined #openstack-oslo07:17
*** tesseract has joined #openstack-oslo07:18
*** shardy has joined #openstack-oslo07:19
*** shardy has quit IRC07:21
*** lpetrut_ has quit IRC07:29
openstackgerritChangBo Guo(gcb) proposed openstack/oslotest master: Update URLs in documents according to document migration  https://review.openstack.org/48316907:32
openstackgerritMerged openstack/oslotest master: Update URLs in documents according to document migration  https://review.openstack.org/48316907:40
*** ralonsoh has joined #openstack-oslo07:41
*** gcb has quit IRC08:09
*** lucas-afk is now known as lucasagomes08:17
openstackgerritchenxu proposed openstack/osprofiler master: Add loading local static files option of template.html  https://review.openstack.org/48467108:36
*** yamamoto has quit IRC08:36
*** yamamoto has joined #openstack-oslo09:01
*** yamamoto has quit IRC09:02
*** yamamoto has joined #openstack-oslo09:04
*** zhangguoqing has quit IRC09:20
*** zhangguoqing has joined #openstack-oslo09:22
*** shardy has joined #openstack-oslo09:39
*** sambetts|afk is now known as sambetts09:46
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.cache master: Imported Translations from Zanata  https://review.openstack.org/48328509:47
*** Dinesh_Bhor has quit IRC09:50
*** zhangguoqing has quit IRC10:02
*** gcb has joined #openstack-oslo10:11
*** ralonsoh has quit IRC10:15
*** yamamoto has quit IRC10:18
*** Dinesh_Bhor has joined #openstack-oslo10:21
*** lpetrut_ has joined #openstack-oslo10:23
*** sdague has joined #openstack-oslo10:23
*** yamamoto has joined #openstack-oslo10:28
*** boden has joined #openstack-oslo10:31
*** yamamoto has quit IRC10:38
*** raildo has joined #openstack-oslo11:03
*** ralonsoh has joined #openstack-oslo11:14
*** dave-mccowan has joined #openstack-oslo11:17
*** lucasagomes is now known as lucas-hungry11:21
*** dave-mccowan has quit IRC11:23
*** ralonsoh_ has joined #openstack-oslo11:24
*** ralonsoh has quit IRC11:24
*** ansmith has quit IRC11:28
*** yamamoto has joined #openstack-oslo11:33
*** lpetrut__ has joined #openstack-oslo11:35
*** lpetrut_ has quit IRC11:37
*** janzian has joined #openstack-oslo11:52
*** yamamoto has quit IRC12:04
*** clayton has quit IRC12:08
*** clayton has joined #openstack-oslo12:09
*** yamamoto has joined #openstack-oslo12:19
*** lucas-hungry is now known as lucasagomes12:25
*** MVenesio has joined #openstack-oslo12:29
*** eck`gone is now known as eck`12:33
*** kgiusti has joined #openstack-oslo12:38
*** gordc has joined #openstack-oslo12:40
*** ansmith has joined #openstack-oslo12:44
*** ralonsoh_ is now known as ralonsoh12:52
*** eck` is now known as eck`gone12:59
*** bknudson has joined #openstack-oslo13:06
*** eck`gone is now known as eck`13:10
*** nicolasbock has quit IRC13:27
*** nicolasbock has joined #openstack-oslo13:28
*** nicolasbock has quit IRC13:32
*** nicolasbock has joined #openstack-oslo13:35
*** links has quit IRC13:38
*** pblaho has quit IRC13:44
*** kgiusti has quit IRC13:59
*** pblaho has joined #openstack-oslo14:00
*** kgiusti has joined #openstack-oslo14:01
openstackgerritMichael Bayer proposed openstack/oslo.db master: Rename idle_timeout to connection_recycle_time  https://review.openstack.org/33418214:23
*** ihrachys has joined #openstack-oslo14:25
*** bnemec has joined #openstack-oslo14:25
*** aselius has joined #openstack-oslo14:33
*** kgiusti has quit IRC14:48
openstackgerritMichael Bayer proposed openstack/oslo.db master: Rename idle_timeout to connection_recycle_time  https://review.openstack.org/33418215:07
*** kgiusti has joined #openstack-oslo15:14
*** yamamoto has quit IRC15:17
*** bknudson has left #openstack-oslo15:17
*** yamamoto has joined #openstack-oslo15:21
*** bknudson has joined #openstack-oslo15:21
openstackgerritDoug Hellmann proposed openstack/oslo.config master: handle option defaults that are not strings  https://review.openstack.org/48484815:26
*** bknudson has left #openstack-oslo15:28
*** yamamoto has quit IRC15:29
*** bknudson has joined #openstack-oslo15:31
*** rcernin has quit IRC15:32
openstackgerritDoug Hellmann proposed openstack/oslo.config master: handle option defaults that are not strings  https://review.openstack.org/48484815:36
dhellmanngcb, dims, bnemec : ^^ could use reviews. it would be good to include that in a release this week.15:37
dimsack dhellmann on it15:38
*** pblaho has quit IRC15:40
dimsdhellmann : when will we end up in the else condition in line 498? https://review.openstack.org/#/c/484848/2/oslo_config/types.py15:47
dhellmannlooking15:48
dhellmanndims : oh, good question15:48
dhellmannif the option is a list, but the default is given as a single int15:48
dhellmannI don't have a test for that15:48
bnemecdhellmann: I left a comment on the generator change too.15:49
dhellmannoh, and that's not a valid configuration15:49
dhellmannbnemec : yes, thanks, typo there15:51
bnemecA list of one item is not valid?15:51
dhellmannbnemec : a list of 1 item is. A single int not in a list is not.15:51
dhellmannI wonder if a single string default works15:52
bnemecSo value is always either a string or a list object, not a bare int?15:52
dhellmannit looks like it15:53
dhellmann stand by for a new patchset15:54
openstackgerritDoug Hellmann proposed openstack/oslo.config master: handle option defaults that are not strings  https://review.openstack.org/48484815:55
dimsdhellmann : LGTM when it goes green15:59
bnemecdhellmann: So I guess the thing there is that while the List type can take a type parameter, we only ever added the String type: https://github.com/openstack/oslo.config/blob/a8482a323b65dd5390c4ec375b3b4b50f66bb204/oslo_config/cfg.py#L136416:01
bnemecSo I'm not sure the else was wrong, it just wasn't necessary today.16:01
*** rcernin has joined #openstack-oslo16:02
bnemecActually, does Cinder have a custom opt type defined or something?16:02
bnemecOtherwise how would they be creating lists of non-strings?16:03
openstackgerritEric Harney proposed openstack/oslo.config master: Fix python 3.6 escape char warnings in strings  https://review.openstack.org/48487116:03
*** rajathagasthya has joined #openstack-oslo16:17
*** yamamoto has joined #openstack-oslo16:29
*** rajathag_ has joined #openstack-oslo16:33
*** rajathagasthya has quit IRC16:33
*** yamamoto has quit IRC16:37
*** lpetrut__ has quit IRC16:38
*** sambetts is now known as sambetts|afk16:57
*** ralonsoh has quit IRC17:00
*** yamamoto has joined #openstack-oslo17:07
*** yamamoto has quit IRC17:12
openstackgerritMichael Bayer proposed openstack/oslo.db master: Rename idle_timeout to connection_recycle_time  https://review.openstack.org/33418217:15
*** harlowja has joined #openstack-oslo17:24
*** lucasagomes is now known as lucas-afk17:25
*** nicolasbock has quit IRC17:35
*** nicolasbock has joined #openstack-oslo17:36
*** bknudson has quit IRC17:38
dhellmannzzzeek : I've found your issue with oslo.config; stand by for the fix17:40
zzzeekdhellmann: woop17:44
openstackgerritDoug Hellmann proposed openstack/oslo.config master: fix an issue with looking up deprecated option names in code  https://review.openstack.org/48489717:45
dhellmannzzzeek : ^^17:45
dhellmanngcb, dims, bnemec : here's another quick-fix for pike ^^17:45
zzzeekdhellmann: cool17:45
dhellmannzzzeek : you'll be far far better off in the long run if you disallow the use of config options as APIs, though, and fix those nova tests.17:46
zzzeekdhellmann: yeah how do i "disallow" that ?17:46
dhellmannzzzeek : well, at this point it's a policy issue17:47
dhellmannso you'd change the tests in nova, with a comment that they should never have been doing whatever they're doing17:47
dhellmannchanging the tests may mean adding some functions to oslo.db to support what they want17:47
dhellmannor a fixture, maybe?17:48
dhellmannI'd have to look at the actual tests to say exactly what to do17:48
openstackgerritDoug Hellmann proposed openstack/oslo.config master: handle option defaults that are not strings  https://review.openstack.org/48484817:50
*** bknudson has joined #openstack-oslo17:53
zzzeekdhellmann: nova-manage command does it: http://logs.openstack.org/82/334182/4/check/gate-tempest-dsvm-neutron-src-oslo.db-ubuntu-xenial/c1dcab4/logs/devstacklog.txt.gz17:54
zzzeekdhellmann: ive no clue what theyre trying to do there17:55
dhellmannwhatever it is, oslo.db should provide the implementation of get_db_conf17:55
*** tesseract has quit IRC17:58
*** deep-book-gk_ has joined #openstack-oslo18:12
*** deep-book-gk_ has left #openstack-oslo18:15
*** chhavi has quit IRC18:17
zzzeekdhellmann: enginefacade already supported configure(**dict(conf.database)) so for now I've proposed that.  but it's crunchtime so now I've annoyed nova by putting up gerrits18:38
dhellmannyeah, given that this is the library freeze week your patch to move those config options is not likely to go out in pike :-/18:38
zzzeekdhellmann: yah it's not a pike thing19:13
dhellmannzzzeek : ok, good19:13
*** lpetrut__ has joined #openstack-oslo19:24
*** bknudson has quit IRC19:25
*** zzzeek has quit IRC19:29
*** zzzeek has joined #openstack-oslo19:30
*** nicolasbock has quit IRC19:53
*** dave-mccowan has joined #openstack-oslo19:58
*** eck` is now known as eck`gone20:04
*** bknudson has joined #openstack-oslo20:05
*** Dinesh_Bhor has quit IRC20:17
*** dave-mccowan has quit IRC20:17
*** kgiusti has quit IRC20:21
*** dave-mccowan has joined #openstack-oslo20:24
*** ansmith has quit IRC20:24
openstackgerritDoug Hellmann proposed openstack/oslo.config master: fix an issue with looking up deprecated option names in code  https://review.openstack.org/48489720:25
*** dave-mccowan has quit IRC20:27
dimsdhellmann : i noticed that castellan does not use stevedore ... filed this one https://review.openstack.org/#/c/484015/20:34
*** shardy has quit IRC20:34
dhellmanndims : nice20:36
openstackgerritMerged openstack/oslo.config master: handle option defaults that are not strings  https://review.openstack.org/48484820:37
*** lpetrut__ has quit IRC20:38
*** eck`gone is now known as eck`20:41
*** raildo has quit IRC20:48
*** rcernin has quit IRC20:57
*** sdague has quit IRC21:02
*** MVenesio has quit IRC21:09
openstackgerritMerged openstack/oslo.db master: Log an exception when reconnect-to-disconnected occurs  https://review.openstack.org/48445521:12
openstackgerritMerged openstack/oslo.config master: Fix python 3.6 escape char warnings in strings  https://review.openstack.org/48487121:15
*** rajathag_ has quit IRC21:26
*** MVenesio has joined #openstack-oslo21:50
*** MVenesio has quit IRC21:50
*** gordc has quit IRC22:25
*** janzian has quit IRC22:40
*** boden has quit IRC22:42
*** bknudson has quit IRC22:52
*** ansmith has joined #openstack-oslo22:58
*** gcb has quit IRC23:02
*** dave-mccowan has joined #openstack-oslo23:29
*** dave-mcc_ has joined #openstack-oslo23:37
*** dave-mccowan has quit IRC23:38
*** dave-mcc_ has quit IRC23:45

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