Tuesday, 2018-07-03

« Monday, 2018-07-02 Index Wednesday, 2018-07-04 »
*** threestrands_ has joined #openstack-oslo00:07
*** threestrands_ has quit IRC00:07
*** threestrands_ has joined #openstack-oslo00:07
*** threestrands_ has quit IRC00:08
*** threestrands_ has joined #openstack-oslo00:09
*** threestrands_ has quit IRC00:09
*** threestrands_ has joined #openstack-oslo00:09
*** threestrands has quit IRC00:10
*** threestrands_ has quit IRC00:10
*** threestrands_ has joined #openstack-oslo00:10
*** threestrands_ has quit IRC00:11
*** threestrands_ has joined #openstack-oslo00:12
*** threestrands_ has quit IRC00:13
*** threestrands_ has joined #openstack-oslo00:13
*** threestrands_ has quit IRC00:13
*** threestrands_ has joined #openstack-oslo00:13
*** threestrands_ has quit IRC00:14
*** threestrands_ has joined #openstack-oslo00:15
*** threestrands_ has quit IRC00:15
*** threestrands_ has joined #openstack-oslo00:15
*** threestrands_ has quit IRC00:16
*** threestrands_ has joined #openstack-oslo00:16
*** edmondsw has joined #openstack-oslo00:23
*** edmondsw has quit IRC00:28
openstackgerritMerged openstack/oslo.policy master: Fix requirements and convert to stestr  https://review.openstack.org/57916900:33
openstackgerritVu Cong Tuan proposed openstack/oslo.service master: Convert oslo.service to using stestr  https://review.openstack.org/57942001:09
openstackgerritVu Cong Tuan proposed openstack/oslo.service master: Remove unnecessary pyNN testenv  https://review.openstack.org/57768501:15
*** larainema has quit IRC02:14
openstackgerritVu Cong Tuan proposed openstack/oslo.concurrency master: Switch to stestr  https://review.openstack.org/57946702:27
openstackgerritVu Cong Tuan proposed openstack/oslo.cache master: Switch to stestr  https://review.openstack.org/57973702:37
openstackgerritVu Cong Tuan proposed openstack/oslo.context master: Switch to stestr  https://review.openstack.org/57973902:40
openstackgerritVu Cong Tuan proposed openstack/oslo.i18n master: Switch to stestr  https://review.openstack.org/57974002:44
*** larainema has joined #openstack-oslo02:44
openstackgerritVu Cong Tuan proposed openstack/oslo.messaging master: Switch to stestr  https://review.openstack.org/57974302:55
openstackgerritVu Cong Tuan proposed openstack/oslo.service master: Convert oslo.service to using stestr  https://review.openstack.org/57942003:01
openstackgerritVu Cong Tuan proposed openstack/oslo.service master: Remove unnecessary pyNN testenv  https://review.openstack.org/57768503:01
*** lpetrut has joined #openstack-oslo03:45
openstackgerritVu Cong Tuan proposed openstack/oslo.i18n master: Switch to stestr  https://review.openstack.org/57974004:18
openstackgerritVu Cong Tuan proposed openstack/oslo.middleware master: Switch to stestr  https://review.openstack.org/57975904:24
*** threestrands_ has quit IRC04:25
openstackgerritVu Cong Tuan proposed openstack/oslo.privsep master: Switch to stestr  https://review.openstack.org/57976004:27
*** lpetrut has quit IRC04:32
*** edmondsw has joined #openstack-oslo05:48
*** edmondsw has quit IRC05:53
*** lpetrut has joined #openstack-oslo06:10
openstackgerritVu Cong Tuan proposed openstack/oslo.context master: Switch to stestr  https://review.openstack.org/57973906:12
openstackgerritVu Cong Tuan proposed openstack/oslo.middleware master: Switch to stestr  https://review.openstack.org/57975906:22
openstackgerritVu Cong Tuan proposed openstack/oslo.reports master: Switch to stestr  https://review.openstack.org/57977106:26
openstackgerritVu Cong Tuan proposed openstack/oslo.rootwrap master: Switch to stestr  https://review.openstack.org/57977206:30
openstackgerritVu Cong Tuan proposed openstack/oslo.serialization master: Switch to stestr  https://review.openstack.org/57977306:32
openstackgerritVu Cong Tuan proposed openstack/oslo.tools master: Switch to stestr  https://review.openstack.org/57977506:36
openstackgerritVu Cong Tuan proposed openstack/oslo.utils master: Switch to stestr  https://review.openstack.org/57977606:39
*** nicolasbock has joined #openstack-oslo06:42
openstackgerritVu Cong Tuan proposed openstack/oslo.versionedobjects master: Switch to stestr  https://review.openstack.org/57977906:43
openstackgerritVu Cong Tuan proposed openstack/oslo.vmware master: Switch to stestr  https://review.openstack.org/57978006:47
*** openstackgerrit has quit IRC06:49
*** mmethot_ has joined #openstack-oslo06:54
*** mmethot has quit IRC06:55
*** rcernin has quit IRC06:55
*** _mmethot_ has joined #openstack-oslo07:06
*** tesseract has joined #openstack-oslo07:06
*** mmethot_ has quit IRC07:10
*** shardy has joined #openstack-oslo07:16
*** mmethot_ has joined #openstack-oslo07:19
*** peereb has joined #openstack-oslo07:21
*** _mmethot_ has quit IRC07:22
*** lpetrut has quit IRC07:31
*** edmondsw has joined #openstack-oslo07:36
*** oanson has quit IRC07:39
*** oanson has joined #openstack-oslo07:41
*** edmondsw has quit IRC07:41
*** lpetrut has joined #openstack-oslo07:44
*** mago_ has joined #openstack-oslo07:49
*** openstackgerrit has joined #openstack-oslo07:53
openstackgerritVu Cong Tuan proposed openstack/oslo.context master: Switch to stestr  https://review.openstack.org/57973907:53
*** rpittau has quit IRC08:02
*** rpittau has joined #openstack-oslo08:10
*** yassine_ has quit IRC08:21
*** yassine_ has joined #openstack-oslo08:31
openstackgerritVu Cong Tuan proposed openstack/oslo.serialization master: Switch to stestr  https://review.openstack.org/57977308:47
openstackgerritVu Cong Tuan proposed openstack/oslo.utils master: Switch to stestr  https://review.openstack.org/57977608:50
openstackgerritVu Cong Tuan proposed openstack/oslo.middleware master: Switch to stestr  https://review.openstack.org/57975908:52
*** cdent has joined #openstack-oslo08:56
openstackgerritVu Cong Tuan proposed openstack/oslo.vmware master: Switch to stestr  https://review.openstack.org/57978008:56
openstackgerritGaetan Semet proposed openstack-dev/pbr master: pbr tag command  https://review.openstack.org/56173108:58
openstackgerritVu Cong Tuan proposed openstack/oslo.rootwrap master: Switch to stestr  https://review.openstack.org/57977208:58
openstackgerritGaetan Semet proposed openstack-dev/pbr master: -s option for pbr info  https://review.openstack.org/57982209:02
openstackgerritVu Cong Tuan proposed openstack/oslo.reports master: Switch to stestr  https://review.openstack.org/57977109:03
*** cdent_ has joined #openstack-oslo09:04
*** cdent has quit IRC09:05
*** cdent_ is now known as cdent09:05
openstackgerritVu Cong Tuan proposed openstack/oslo.privsep master: Switch to stestr  https://review.openstack.org/57976009:06
openstackgerritVu Cong Tuan proposed openstack/oslo.i18n master: Switch to stestr  https://review.openstack.org/57974009:07
openstackgerritVu Cong Tuan proposed openstack/oslo.cache master: Switch to stestr  https://review.openstack.org/57973709:08
openstackgerritVu Cong Tuan proposed openstack/oslo.service master: Convert oslo.service to using stestr  https://review.openstack.org/57942009:10
openstackgerritVu Cong Tuan proposed openstack/oslo.service master: Remove unnecessary pyNN testenv  https://review.openstack.org/57768509:10
openstackgerritVu Cong Tuan proposed openstack/oslo.concurrency master: Switch to stestr  https://review.openstack.org/57946709:10
*** pbourke has quit IRC09:10
*** pbourke has joined #openstack-oslo09:12
openstackgerritVu Cong Tuan proposed openstack/oslo.versionedobjects master: Switch to stestr  https://review.openstack.org/57977909:14
*** edmondsw has joined #openstack-oslo09:25
*** edmondsw has quit IRC09:29
*** e0ne has joined #openstack-oslo09:33
*** sambetts|afk is now known as sambetts09:33
openstackgerritMichal Arbet proposed openstack/taskflow master: Fix code to support networkx > 1.0  https://review.openstack.org/57783309:34
*** lpetrut has quit IRC09:38
*** lpetrut has joined #openstack-oslo09:44
*** nicolasbock has quit IRC09:55
*** yassine_ has quit IRC09:58
openstackgerritJan Vondra proposed openstack/oslo.log master: Add log handler for Sentry  https://review.openstack.org/51894910:16
*** cdent has quit IRC10:25
openstackgerritMerged openstack/oslo.policy master: Pass dictionary as creds in policy tests  https://review.openstack.org/57899410:25
openstackgerritGaetan Semet proposed openstack-dev/pbr master: pbr tag command  https://review.openstack.org/56173111:00
*** yassine has joined #openstack-oslo11:45
*** yassine is now known as Guest6659311:46
*** ansmith has quit IRC11:57
*** raildo has joined #openstack-oslo12:10
*** yamamoto_ has quit IRC12:37
*** njohnston_ has joined #openstack-oslo12:38
*** njohnston has quit IRC12:39
*** njohnston_ has quit IRC12:43
*** njohnston has joined #openstack-oslo12:49
*** rmascena has joined #openstack-oslo12:51
*** raildo has quit IRC12:51
*** njohnston_ has joined #openstack-oslo12:52
*** yamamoto has joined #openstack-oslo12:53
*** njohnston has quit IRC12:53
*** yamamoto has quit IRC12:58
dhellmannbnemec : have a look at the last comment on https://review.openstack.org/#/c/560716/ -- I'm not sure what we might want to do there13:00
* dhellmann wishes we had not allowed python-style list options at all13:00
*** r-mibu has joined #openstack-oslo13:00
dhellmannmaybe we could just switch to yaml13:00
*** edmondsw has joined #openstack-oslo13:01
*** edmondsw has quit IRC13:06
*** yamamoto has joined #openstack-oslo13:08
*** ansmith has joined #openstack-oslo13:10
*** yamamoto has quit IRC13:13
*** rmascena__ has joined #openstack-oslo13:17
*** rmascena has quit IRC13:20
*** yamamoto has joined #openstack-oslo13:23
*** yamamoto has quit IRC13:28
bnemecdhellmann: You saw the followup on that?  It appears to only affect the generator, so it shouldn't be breaking anything.13:39
dhellmannbnemec : ok, good13:39
*** yamamoto has joined #openstack-oslo13:41
*** yamamoto has quit IRC13:41
*** njohnston_ is now known as njohnston13:56
*** bobh has joined #openstack-oslo13:57
*** namnh has joined #openstack-oslo13:58
*** lpetrut has quit IRC14:05
*** bobh has quit IRC14:18
*** ianychoi has quit IRC14:18
*** ianychoi has joined #openstack-oslo14:19
*** kgiusti has joined #openstack-oslo14:23
*** lpetrut has joined #openstack-oslo14:23
*** rmascena__ has quit IRC14:24
*** raildo has joined #openstack-oslo14:30
*** lpetrut has quit IRC14:38
openstackgerritMerged openstack/oslo.reports master: Switch to stestr  https://review.openstack.org/57977114:42
openstackgerritMerged openstack/oslo.limit master: Convert tox.ini to using stestr  https://review.openstack.org/57968514:45
*** yamamoto has joined #openstack-oslo14:50
*** edmondsw has joined #openstack-oslo14:51
mago_Hi zzzeek gentle ping, could you have a look at this issue again please : https://bugs.launchpad.net/oslo.db/+bug/1774544. Is it related to logging (has it changed as you suggested), or else how can we investigate further to understand the lost connections ? Many thanks14:53
openstackLaunchpad bug 1774544 in oslo.db "Upgrade pike to queens get `xxx DBConnectionError SELECT 1`" [Undecided,New]14:53
*** edmondsw has quit IRC14:55
*** yamamoto has quit IRC14:58
openstackgerritMerged openstack/oslo.rootwrap master: Switch to stestr  https://review.openstack.org/57977215:02
openstackgerritMerged openstack/oslo.cache master: Switch to stestr  https://review.openstack.org/57973715:02
openstackgerritMerged openstack/oslo.privsep master: Switch to stestr  https://review.openstack.org/57976015:04
*** peereb has quit IRC15:05
openstackgerritMerged openstack/oslo.i18n master: Switch to stestr  https://review.openstack.org/57974015:07
*** njohnston_ has joined #openstack-oslo15:08
*** njohnston has quit IRC15:08
*** lpetrut has joined #openstack-oslo15:09
*** njohnston_ has quit IRC15:26
*** njohnston has joined #openstack-oslo15:26
*** e0ne has quit IRC15:29
*** namnh has quit IRC15:44
*** namnh has joined #openstack-oslo15:44
*** nicolasbock has joined #openstack-oslo15:52
*** sileht has quit IRC15:53
*** sileht has joined #openstack-oslo15:55
*** Guest66593 has quit IRC15:58
*** namnh has quit IRC15:58
*** sileht has quit IRC15:59
*** sileht has joined #openstack-oslo16:01
*** shardy has quit IRC16:01
*** namnh has joined #openstack-oslo16:05
*** namnh has quit IRC16:10
*** tesseract has quit IRC16:20
*** lpetrut has quit IRC16:24
*** r-mibu has quit IRC16:29
*** mago_ has quit IRC16:47
*** yamamoto has joined #openstack-oslo16:50
*** yamamoto has quit IRC16:55
*** sambetts is now known as sambetts|afk17:11
*** dkehn has quit IRC17:26
*** dkehn has joined #openstack-oslo17:37
*** e0ne has joined #openstack-oslo17:45
openstackgerritKen Giusti proposed openstack/oslo.messaging master: No longer allow redundant calls to server start()  https://review.openstack.org/57993517:47
*** yamamoto has joined #openstack-oslo17:51
bnemeckgiusti: https://review.openstack.org/#/c/577964 is going to require a major version bump in oslo.messaging.  Do we have anything else we want to land before I do that?17:54
*** namnh has joined #openstack-oslo17:54
*** yamamoto has quit IRC17:56
*** namnh has quit IRC17:59
kgiustibnemec: I don't have anything queued up.  Maybe https://review.openstack.org/579935, but since the ordered task running prevented redundant calls to start I don't think this change is going to affect user visible behavior.18:04
kgiustiansmith: ^^18:04
bnemeckgiusti: Okay, I'm mostly just wondering if there's anything else you had queued up that would require a major release.18:04
bnemecI guess it's not a huge deal to do a major release, especially one for something this small, but I have a mental hangup on the word "major". :-)18:05
ansmithbnemec, kgiusti: nothing at my end18:05
bnemecOkay, thanks.18:08
*** cdent has joined #openstack-oslo18:13
*** njohnston has left #openstack-oslo18:21
*** njohnston has joined #openstack-oslo18:21
*** edmondsw has joined #openstack-oslo18:27
*** edmondsw has quit IRC18:32
*** yamamoto has joined #openstack-oslo18:52
*** yamamoto has quit IRC18:58
*** bobh has joined #openstack-oslo19:00
*** njohnston has quit IRC19:04
*** njohnston has joined #openstack-oslo19:04
*** bobh has quit IRC19:25
*** e0ne has quit IRC19:40
*** namnh has joined #openstack-oslo19:42
*** bobh has joined #openstack-oslo19:42
*** bobh has quit IRC19:46
*** bobh has joined #openstack-oslo19:47
*** namnh has quit IRC19:47
*** yamamoto has joined #openstack-oslo19:54
*** yamamoto has quit IRC20:00
*** ansmith has quit IRC20:03
*** bobh has quit IRC20:05
*** edmondsw has joined #openstack-oslo20:16
*** edmondsw has quit IRC20:21
*** gcb has joined #openstack-oslo20:31
lbragstaddhellmann: ping - i want to see if you have an opinion on something we were talking about the other day20:36
dhellmannblue!20:37
lbragstad42!20:37
dhellmannhut! hut! hike!20:37
* lbragstad trips on his shoe laces20:37
* dhellmann throws the ball through the goal posts20:37
lbragstadwe're getting the hang of it20:38
dhellmannI feel good about our progress20:38
lbragstadcan't be any worse than the Denver Broncos two super bowls ago20:38
*** kgiusti has left #openstack-oslo20:38
lbragstadooooooo20:38
dhellmannhold my beer20:38
lbragstadso you remember https://review.openstack.org/#/c/578995/720:39
dhellmannyes20:39
lbragstadapparently that's tripping in the gate for legit reasons20:39
dhellmannoh?20:39
lbragstadhttp://logs.openstack.org/95/578995/7/check/legacy-tempest-dsvm-neutron-src/ab54a98/logs/screen-keystone.txt.gz#_Jul_03_15_11_54_79890020:40
lbragstadso -keystone passes something other than a dictionary (go figure)20:40
lbragstadbut we actually use oslo.context to generate was we pass already20:40
lbragstadhttps://github.com/openstack/keystone/blob/master/keystone/middleware/auth.py#L22020:40
dhellmannah, yeah20:40
lbragstadthoughts on adding another case for the specific implementation of a dict?20:40
dhellmannI think we told the keystone team to stop doing that a while back20:40
dhellmannor maybe that was with a config option type20:40
dhellmannwhat would the case be?20:41
dhellmannwhy does keystone need their own special class for this?20:41
dhellmannoh, it's a deprecated thing out of oslo.context?20:41
dhellmannugh, yeah20:41
dhellmannwe should probably go ahead and support that case20:41
lbragstadhttps://github.com/openstack/oslo.context/blob/master/oslo_context/context.py#L313-L31720:41
dhellmannif you check collections.MutableMapping instead of dict that should work20:42
lbragstador we change middleware to call https://github.com/openstack/oslo.context/blob/master/oslo_context/context.py#L331 instead20:42
dhellmannalthough that may be too flexible20:42
dhellmannthe context has more in it than the policy values, I think?20:43
dhellmannI think that's why we added methods to get policy values and rpc values20:43
lbragstadthat makes sense20:43
lbragstadkmalloc: are you familiar with jamie's perspective on https://github.com/openstack/oslo.context/blob/master/oslo_context/context.py#L313-L31620:43
dhellmannthat _DeprecatedPolicyValues class is private to oslo.context, so we can't check for it in oslo.policy20:44
lbragstadright20:44
dhellmannso you'll have to look for the MutableMapping thing as a "dict-like" object20:44
lbragstadand hope we don't have a conflict with that in the future20:44
lbragstador it's not ambiguous20:44
dhellmannyeah, it's not ideal, but I think it's what we can do right now20:44
dhellmannat some point I would like to actually remove those policy values, and then we could stop checking for it20:45
dhellmannlbragstad : please add a long long comment explaining all of this in the policy patch20:45
lbragstadwill do - but remove the to_policy_values method you mean?20:45
dhellmannin oslo.policy just extend the check to allow a MutableMapping20:46
dhellmannthat should fix it, right?20:46
lbragstadright - when yuo said you'd like to remove "those policy values", what exactly were you referring to?20:46
dhellmannoh, sorry20:47
dhellmannin a separate thing later some day I want to remove _DeprecatedPolicyValues and the values it contains20:47
dhellmannthey're deprecated, we need to plan to remove them :-)20:47
kmallocrm -rf ;)20:47
kmalloc>.>20:47
kmallocbest plan ever20:47
kmalloclbragstad: and ftr, i never understood a lot of jamie's perspective on oslo_context20:49
kmalloci just rolled with it and figured he was right.20:49
lbragstadmaybe https://github.com/openstack/oslo.context/blob/master/oslo_context/context.py#L313-L316 is opaque to me20:49
lbragstadbecause there are things in there that don't seems like they should be deprecated?20:49
dhellmannI think some of those names were renamed20:51
dhellmannuser_id became user20:51
dhellmannetc.20:51
lbragstadhmm20:54
*** raildo has quit IRC20:54
*** yamamoto has joined #openstack-oslo20:56
kmallocthere has been a lot of shuffling of names in oslo_context20:56
lbragstadi guess i'm wondering if to_policy_values always needs to return a special implementation of dict20:56
kmallochm.20:57
kmallocmaybe20:57
lbragstadi guess having a vehicle there makes sense? maybe we need to know when it's safe to remove the "deprecated" ones20:58
dhellmannyeah, we haven't done the work to figure that out20:58
lbragstador if we have to support them forever20:58
kmallocwe might need to support them forever tbh20:58
kmallocwell it depends on what oslo_context's view on contract is20:59
*** yamamoto has quit IRC21:00
openstackgerritLance Bragstad proposed openstack/oslo.policy master: Teach Enforcer.enforce to deal with context objects  https://review.openstack.org/57899521:11
lbragstadok - that add the collection.MutableMapping check ^21:13
*** namnh has joined #openstack-oslo21:30
*** namnh has quit IRC21:34
*** yamamoto has joined #openstack-oslo21:57
*** nicolasbock has quit IRC22:00
*** yamamoto has quit IRC22:01
*** lbragstad is now known as lbragstad_50322:02
*** edmondsw has joined #openstack-oslo22:05
*** rcernin has joined #openstack-oslo22:09
*** edmondsw has quit IRC22:09
*** cdent has quit IRC22:23
*** dtruong has quit IRC22:30
*** namnh has joined #openstack-oslo22:41
*** namnh_ has joined #openstack-oslo22:46
*** namnh has quit IRC22:48
*** rcernin has quit IRC22:49
*** yamamoto has joined #openstack-oslo22:57
*** rcernin has joined #openstack-oslo23:02
*** yamamoto has quit IRC23:02
*** namnh_ has quit IRC23:15
*** rcernin_ has joined #openstack-oslo23:23
*** yassine has joined #openstack-oslo23:23
*** yassine is now known as Guest3844423:24
*** rcernin has quit IRC23:24
*** edmondsw has joined #openstack-oslo23:53
*** edmondsw has quit IRC23:57
*** yamamoto has joined #openstack-oslo23:58
« Monday, 2018-07-02 Index Wednesday, 2018-07-04 »

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