Wednesday, 2018-03-28

*** salv-orlando has joined #openstack-oslo00:05
*** salv-orlando has quit IRC00:10
*** yamamoto has joined #openstack-oslo00:12
*** openstack has joined #openstack-oslo00:14
*** ChanServ sets mode: +o openstack00:14
*** yamamoto has quit IRC00:17
*** dave-mccowan has joined #openstack-oslo00:21
*** spilla has joined #openstack-oslo00:30
*** yamamoto has joined #openstack-oslo00:38
*** yamamoto has quit IRC00:45
*** tovin07_ has joined #openstack-oslo00:47
*** yamamoto has joined #openstack-oslo00:56
*** bobh has joined #openstack-oslo01:05
*** gyankum has joined #openstack-oslo01:12
*** yamamoto has quit IRC01:14
*** bobh has quit IRC01:17
openstackgerritmelissaml proposed openstack/futurist master: Update links in README  https://review.openstack.org/54838001:40
*** bobh has joined #openstack-oslo01:45
*** bobh has quit IRC01:53
*** salv-orlando has joined #openstack-oslo02:07
*** salv-orlando has quit IRC02:12
*** AlexeyAbashkin has joined #openstack-oslo02:12
*** yamamoto has joined #openstack-oslo02:15
*** AlexeyAbashkin has quit IRC02:16
*** yamamoto has quit IRC02:21
*** yamamoto has joined #openstack-oslo02:37
*** yamamoto has quit IRC03:18
*** dave-mccowan has quit IRC03:55
*** jaosorior has joined #openstack-oslo03:56
*** salv-orlando has joined #openstack-oslo04:08
*** salv-orlando has quit IRC04:13
*** yamamoto has joined #openstack-oslo04:19
openstackgerritMerged openstack/oslo.cache master: Change memcache_socket_timeout to a float  https://review.openstack.org/51935304:24
*** yamamoto has quit IRC04:25
*** namnh has joined #openstack-oslo04:27
*** links has joined #openstack-oslo04:46
*** lpetrut has joined #openstack-oslo04:55
*** chhagarw has joined #openstack-oslo05:03
*** salv-orlando has joined #openstack-oslo05:09
*** salv-orlando has quit IRC05:09
*** salv-orlando has joined #openstack-oslo05:09
*** chhagarw has quit IRC05:12
*** yamamoto has joined #openstack-oslo05:21
*** lpetrut has quit IRC05:26
*** e0ne has joined #openstack-oslo05:26
*** lpetrut has joined #openstack-oslo05:27
*** yamamoto has quit IRC05:27
*** msimonin has quit IRC05:28
*** msimonin has joined #openstack-oslo05:28
*** msimonin has quit IRC05:28
*** msimonin has joined #openstack-oslo05:28
*** msimonin has quit IRC05:29
*** msimonin has joined #openstack-oslo05:29
*** msimonin has quit IRC05:30
*** msimonin has joined #openstack-oslo05:30
*** msimonin has quit IRC05:31
*** msimonin has joined #openstack-oslo05:31
*** msimonin has quit IRC05:31
*** msimonin has joined #openstack-oslo05:32
*** msimonin has quit IRC05:32
*** msimonin has joined #openstack-oslo05:32
*** e0ne has quit IRC05:33
*** msimonin has quit IRC05:33
*** lpetrut has quit IRC05:39
*** e0ne has joined #openstack-oslo05:50
*** lpetrut has joined #openstack-oslo05:52
*** e0ne has quit IRC06:10
*** yamamoto has joined #openstack-oslo06:23
*** yamamoto has quit IRC06:28
*** jaosorior has quit IRC06:29
*** msimonin has joined #openstack-oslo06:31
*** msimonin has quit IRC06:35
*** pcaruana has joined #openstack-oslo06:38
*** lpetrut has quit IRC06:58
*** jaosorior has joined #openstack-oslo07:10
*** ianychoi has quit IRC07:10
*** rcernin has quit IRC07:15
*** pooja_jadhav has quit IRC07:21
*** salv-orlando has quit IRC07:22
*** pooja_jadhav has joined #openstack-oslo07:22
*** salv-orlando has joined #openstack-oslo07:22
*** msimonin has joined #openstack-oslo07:23
*** yamamoto has joined #openstack-oslo07:25
*** tesseract has joined #openstack-oslo07:25
*** msimonin has quit IRC07:26
*** msimonin has joined #openstack-oslo07:26
*** salv-orlando has quit IRC07:27
*** yamamoto has quit IRC07:30
*** shardy has joined #openstack-oslo07:38
*** salv-orlando has joined #openstack-oslo07:45
*** salv-orlando has quit IRC07:54
*** salv-orlando has joined #openstack-oslo07:54
*** lucas-afk is now known as lucasagomes07:56
*** salv-orlando has quit IRC07:59
*** AlexeyAbashkin has joined #openstack-oslo08:00
*** rcernin has joined #openstack-oslo08:24
*** yamamoto has joined #openstack-oslo08:26
*** e0ne has joined #openstack-oslo08:27
*** yamamoto has quit IRC08:30
*** oanson has quit IRC08:34
*** oanson has joined #openstack-oslo08:36
*** lpetrut has joined #openstack-oslo08:40
*** rcernin has quit IRC08:42
openstackgerritwanghui proposed openstack/futurist master: Update links in README  https://review.openstack.org/55719508:43
*** kashyap has left #openstack-oslo08:46
*** lpetrut has quit IRC08:47
*** lpetrut has joined #openstack-oslo08:47
*** shardy has quit IRC09:10
*** shardy has joined #openstack-oslo09:12
*** yamamoto has joined #openstack-oslo09:27
*** yamamoto has quit IRC09:32
*** lpetrut_ has joined #openstack-oslo09:41
*** vstinner has joined #openstack-oslo09:49
*** salv-orlando has joined #openstack-oslo09:58
*** namnh has quit IRC10:03
*** tovin07_ has quit IRC10:04
*** ianychoi has joined #openstack-oslo10:15
*** sambetts|afk is now known as sambetts10:18
*** yamamoto has joined #openstack-oslo10:29
*** yamamoto has quit IRC10:34
*** ansmith has quit IRC10:37
*** pbourke has quit IRC10:44
*** pbourke has joined #openstack-oslo10:45
*** salv-orl_ has joined #openstack-oslo10:53
*** salv-orlando has quit IRC10:56
*** gyankum has quit IRC10:56
*** salv-orlando has joined #openstack-oslo10:58
*** salv-orl_ has quit IRC10:58
*** links has quit IRC11:04
*** salv-orlando has quit IRC11:05
*** links has joined #openstack-oslo11:05
*** salv-orlando has joined #openstack-oslo11:06
*** dave-mccowan has joined #openstack-oslo11:09
*** shardy is now known as shardy_lunch11:24
*** yamamoto has joined #openstack-oslo11:31
*** yamamoto has quit IRC11:35
*** salv-orl_ has joined #openstack-oslo11:36
*** salv-orlando has quit IRC11:41
*** raildo has joined #openstack-oslo11:51
*** lucasagomes is now known as lucas-hungry11:55
*** bobh has joined #openstack-oslo12:06
*** edmondsw has joined #openstack-oslo12:06
*** chhagarw has joined #openstack-oslo12:08
openstackgerritJan Vondra proposed openstack/oslo.log master: Add log handler for Sentry  https://review.openstack.org/51894912:09
*** ansmith has joined #openstack-oslo12:11
*** msimonin has quit IRC12:11
*** lpetrut_ has quit IRC12:15
*** msimonin has joined #openstack-oslo12:18
*** eck` is now known as eck`gone12:22
*** yamamoto has joined #openstack-oslo12:22
*** bobh has quit IRC12:34
*** kgiusti has joined #openstack-oslo12:34
*** msimonin has quit IRC12:54
*** lucas-hungry is now known as lucasagomes12:55
*** salv-orlando has joined #openstack-oslo12:56
*** salv-or__ has joined #openstack-oslo12:57
*** salv-orl_ has quit IRC12:58
*** msimonin has joined #openstack-oslo13:00
*** msimonin has quit IRC13:01
*** salv-orlando has quit IRC13:02
*** salv-or__ has quit IRC13:02
*** salv-orlando has joined #openstack-oslo13:03
*** eck`gone is now known as eck`13:03
*** msimonin has joined #openstack-oslo13:04
*** lbragstad has joined #openstack-oslo13:04
*** salv-orlando has quit IRC13:08
*** spilla has joined #openstack-oslo13:30
*** salv-orlando has joined #openstack-oslo13:53
*** songjian has joined #openstack-oslo13:55
*** songjian has quit IRC13:55
*** janzian has joined #openstack-oslo13:59
*** yassine has quit IRC14:00
*** songjian has joined #openstack-oslo14:01
*** yassine has joined #openstack-oslo14:02
*** ianychoi_ has joined #openstack-oslo14:33
*** ianychoi has quit IRC14:36
*** shardy_lunch is now known as shardy14:53
*** kevinbenton has quit IRC14:58
bnemecdhellmann: It looks like there are some problems with the config location changes and other projects' unit tests: http://logs.openstack.org/12/557012/1/check/cross-nova-py27/37b2a7c/job-output.txt.gz#_2018-03-27_21_41_10_03763515:02
bnemecFrom the constraint update: https://review.openstack.org/#/c/557012/15:02
*** links has quit IRC15:03
bnemecI'm not sure what the deal with the cinder duplicate opt is.15:03
*** kevinbenton has joined #openstack-oslo15:06
bnemecOh, the duplicate opt is because the _set_location member doesn't match.15:07
bnemecWe probably need to filter that out in __eq__15:07
*** pcaruana has quit IRC15:09
*** gyankum has joined #openstack-oslo15:11
openstackgerritClaudiu Belu proposed openstack/oslotest master: mock: Properly patch mock.MagicMock  https://review.openstack.org/55736815:20
*** songjian has quit IRC15:20
dhellmannbnemec : ack15:24
dhellmannbnemec : is there an easy way for me to run those tests locally?15:25
dhellmannnm, I have it15:25
dhellmannat least I have the tests running15:25
bnemecI think you would just need to pip install the new oslo.config into one of the projects' tox venv.15:26
dhellmannyeah, I was confused and thinking these were something other than just nova's tests15:26
*** bobh has joined #openstack-oslo15:31
*** bobh has quit IRC15:36
*** bobh has joined #openstack-oslo15:43
*** bobh has quit IRC15:48
*** bobh has joined #openstack-oslo15:53
*** bobh has quit IRC15:58
*** bobh has joined #openstack-oslo16:00
*** bobh has quit IRC16:05
*** bobh has joined #openstack-oslo16:08
*** lucasagomes is now known as lucas-afk16:13
*** msimonin has quit IRC16:21
*** msimonin has joined #openstack-oslo16:21
*** msimonin has quit IRC16:25
*** lpetrut has quit IRC16:45
*** kgiusti has left #openstack-oslo16:50
*** e0ne has quit IRC17:01
*** AlexeyAbashkin has quit IRC17:08
*** sambetts is now known as sambetts|afk17:13
dhellmannbnemec : the issue with that one nova test is the way it mocks out os.path.exists17:13
bnemecYeah, I saw mock.patch in the traceback so I figured it had something to do with that.17:15
dhellmannI'm willing to bet that's the issue with all of them17:15
dhellmannnova/tests/unit/image/test_glance.py mocks open ffs17:15
* bnemec has written unit tests that mock open :-|17:16
dhellmannyou have to be very careful mocking builtins and stdlib stuff like that17:16
dhellmannI guess this deserves a ML thread17:16
dhellmannmaybe some of the nova folks have ideas for what to do here17:16
bnemecYeah17:17
dhellmanndid any other projects have similar issues with oslo.config master?17:18
dhellmannbnemec : http://lists.openstack.org/pipermail/openstack-dev/2018-March/128833.html17:27
*** yamamoto has quit IRC17:29
bnemecLet's see, cinder was duplicate opt, neutron is also duplicate opt.  Those were the only other ones tested on the requirements change.17:29
*** lpetrut has joined #openstack-oslo17:30
dhellmannok, I think that's probably a thing we can fix in our code but let me try17:33
*** shardy has quit IRC17:35
*** tesseract has quit IRC17:35
dhellmannbnemec : in cinder there are actually 2 different instances of zfssa_rest_timeout17:37
bnemecdhellmann: Yeah, but as long as they match that should be allowed.17:38
dhellmannthat's debatable17:38
bnemecI assume they do since the cinder tests were passing before.17:38
bnemecWell, that was the previous behavior of oslo.config.17:38
dhellmannthe intent behind detecting that they are different is to avoid having 2 copies of the definition in 2 places17:38
bnemecI suspect we need to fix this anyway in case an opt gets registered, modified in some way, then registered again at a later time.17:40
bnemecI think that would fail because the location member would differ.17:40
dhellmannif it's the same code path it should be fine17:40
dhellmannbut I guess the path of least resistance here is to allow it17:40
dhellmannhow about a warning?17:40
bnemecYeah, changing that behavior would require another major version bump, I think.17:41
bnemecI'm fine with a warning.17:41
*** yamamoto has joined #openstack-oslo17:46
*** lpetrut has quit IRC18:02
*** yamamoto has quit IRC18:12
*** pblaho has quit IRC18:12
*** chhagarw has quit IRC18:16
*** lpetrut has joined #openstack-oslo18:16
*** e0ne has joined #openstack-oslo18:19
*** yamamoto has joined #openstack-oslo18:21
*** yamamoto has quit IRC18:26
openstackgerritDoug Hellmann proposed openstack/oslo.config master: ignore location when comparing options for duplicate registration  https://review.openstack.org/55746718:29
dhellmannoh, I forgot the warning, hang on18:30
dhellmannah, nevermind, that's just likely to spew lots of warnings to the logs that operators can't do anything about18:31
dhellmannbnemec : ^^18:31
*** gyankum has quit IRC18:36
*** yamamoto has joined #openstack-oslo18:36
*** gyankum has joined #openstack-oslo18:40
*** yamamoto has quit IRC18:41
*** gyankum has quit IRC18:48
*** yamamoto has joined #openstack-oslo18:52
*** AlexeyAbashkin has joined #openstack-oslo18:54
*** lpetrut has quit IRC18:54
*** yamamoto has quit IRC18:56
*** AlexeyAbashkin has quit IRC18:59
*** msimonin has joined #openstack-oslo19:00
*** yamamoto has joined #openstack-oslo19:17
*** yamamoto has quit IRC19:17
*** pblaho has joined #openstack-oslo19:25
*** yamamoto has joined #openstack-oslo19:28
*** yamamoto has quit IRC19:32
*** bobh has quit IRC19:37
*** yamamoto has joined #openstack-oslo19:43
*** yamamoto has quit IRC19:47
*** yamamoto has joined #openstack-oslo19:57
*** yamamoto has quit IRC19:57
*** spilla has quit IRC19:59
*** bobh has joined #openstack-oslo20:01
*** ansmith has quit IRC20:03
*** eck` is now known as eck`gone20:38
*** nicolasbock has joined #openstack-oslo20:39
*** e0ne has quit IRC20:48
*** cdent has joined #openstack-oslo20:50
*** raildo has quit IRC20:51
cdentI'm pretty sure I've found a bug with oslo.config 6.0.0 but before I register a bug I thought I'd sanity check. A script I have which takes a --config-file argument that worked just fine 5.2.0 takes a _very_ long to complete its imports20:52
cdentThat ^ ring any bells dhellmann  ?20:53
dhellmannnot really20:53
dhellmannI'd have to look at what changed between those versions20:54
dhellmannoh, it could be the new location stuff20:54
cdentseems to be near /usr/local/lib/python3.6/dist-packages/oslo_config/cfg.py20:55
cdentsorry, left out the line number: line 814, in _get_caller_detail20:55
dhellmannyeah20:55
dhellmannso that's the stuff that probes where the option is coming from20:55
dhellmannso that we can say "this option is defined in foo.py and set in $file.ini"20:55
dhellmannthat's the thing jay got when he asked for the ability to tell if the user had set an option :-)20:56
cdentah20:56
cdentfigures20:56
dhellmannis that code in a patch I can download?20:56
dhellmannoops, wrong window again20:57
cdenthave you checked with jay that he actually still want/needs that because last I checked (if it is allocation ratio stuff) that's stalled out20:57
dhellmannI have not20:57
*** yamamoto has joined #openstack-oslo20:57
cdentI'll dig a little closer in that chunk of code and see if I can figure out what's going on a bit more and register a bug. launchpad or storyboard?20:58
bnemecStill launchpad at the moment21:02
cdentthanks21:02
*** yamamoto has quit IRC21:03
*** vstinner has left #openstack-oslo21:04
cdentdhellmann, bnemec : https://bugs.launchpad.net/oslo.config/+bug/175968921:12
openstackLaunchpad bug 1759689 in oslo.config "In 1.6.0 startup can be extremely slow when checking many config items" [Undecided,New]21:12
dhellmann1.6.0 or 6.0.0?21:12
dhellmanneither way it sounds like we want to turn off the file location stuff21:13
cdentsorry, yeah, will fix it, have too many versions in my head21:14
cdentyeah, seems so21:15
dhellmanncdent : if you look at oslo_config/cfg.py around line 807 you should see a _get_caller_detail() function. If you short-circuit that function to always return None does it fix the performance?21:16
cdentlemme try21:16
dhellmannif so I can propose that as a fix quickly21:16
cdentyes, it does: it goes from a time of 1m27 to  0m121:17
dhellmannwell21:17
dhellmannstand by then21:18
cdent:)21:18
openstackgerritDoug Hellmann proposed openstack/oslo.config master: disable stack inspection when setting option values  https://review.openstack.org/55751321:19
dhellmannbnemec , cdent ^^21:19
dhellmannwe could possibly keep the location for set_defaults() calls because those are less frequent21:20
dhellmannbut this was quicker to fix the immediate problem21:20
dhellmannwe could also do something like use an environment variable to turn it on/off21:20
dhellmannsince the location in code is really only useful for debugging21:21
cdentthat seems like a reasonable plan21:21
* cdent enjoys running code without upper constraints21:22
openstackgerritDoug Hellmann proposed openstack/oslo.config master: use environment variable to control file location probing  https://review.openstack.org/55751621:27
dhellmannok, there we go21:27
dhellmannprobably needs docs21:27
*** ansmith has joined #openstack-oslo21:27
dhellmannand I think I'm going to quit for the day while I'm ahead21:28
*** e0ne has joined #openstack-oslo21:29
cdentdhellmann is smarter than cdent21:33
*** cdent has quit IRC21:37
*** edmondsw has quit IRC21:43
-openstackstatus- NOTICE: the zuul web dashboard will experience a short downtime as we roll out some changes - no job execution should be affected21:52
*** pblaho has quit IRC21:53
*** harlowja has joined #openstack-oslo21:53
*** e0ne has quit IRC21:55
*** yamamoto has joined #openstack-oslo21:59
*** yamamoto has quit IRC22:04
*** bobh has quit IRC22:17
*** rcernin has joined #openstack-oslo22:28
*** msimonin has quit IRC22:38
*** pblaho has joined #openstack-oslo22:48
*** pblaho has quit IRC23:00
*** yamamoto has joined #openstack-oslo23:00
openstackgerritDoug Hellmann proposed openstack/oslo.config master: disable stack inspection when setting option values  https://review.openstack.org/55751323:04
openstackgerritDoug Hellmann proposed openstack/oslo.config master: use environment variable to control file location probing  https://review.openstack.org/55751623:04
*** yamamoto has quit IRC23:06
*** bobh has joined #openstack-oslo23:16
*** salv-orlando has quit IRC23:21
*** salv-orlando has joined #openstack-oslo23:22
*** salv-orlando has quit IRC23:27
-openstackstatus- NOTICE: Zuul has been restarted to update to the latest code; existing changes have been re-enqueued, you may need to recheck changes uploaded in the past 10 minutes23:58

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