14:03:20 <jokke_> #startmeeting glance
14:03:21 <openstack> Meeting started Thu Aug 22 14:03:20 2019 UTC and is due to finish in 60 minutes.  The chair is jokke_. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:03:22 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:03:24 <openstack> The meeting name has been set to 'glance'
14:03:27 <jokke_> #topic roll-call
14:03:31 <jokke_> o/
14:03:45 <jokke_> #link https://etherpad.openstack.org/p/glance-team-meeting-agenda
14:04:09 <abhishekk> o/
14:04:58 <jokke_> we won't have rosmaita here and I guess smcginnis neither
14:05:16 <abhishekk> yes
14:05:22 <jokke_> something something cinder mid-cycle
14:05:46 <abhishekk> :D
14:05:51 <jokke_> #topic release-updates
14:05:53 <jokke_> go for it
14:06:14 <abhishekk> I have created new etherpad to track down the important patches
14:06:26 <abhishekk> #link https://etherpad.openstack.org/p/glance-train-priority-patches
14:06:55 <abhishekk> I went through last meeting minutes and found that it got very messy to understand the current status
14:07:13 <abhishekk> so above etherpad will help us to track down the important patches
14:08:18 <abhishekk> Regarding periodic job we have 3 failures due to parser error
14:09:12 <jokke_> is it the same issue again, the parser can't handle the output volume?
14:09:56 <abhishekk> yes
14:10:55 <jokke_> gr8 :(
14:10:59 <abhishekk> that's it from me
14:11:12 <jokke_> k
14:11:15 <jokke_> thanks
14:11:38 <abhishekk> I have one solution, we can add config option and default set it to false
14:11:51 <abhishekk> if it is false it will not attache logs to the output
14:12:05 <abhishekk> else when set to true it will add those logs
14:12:26 <abhishekk> it adds glance-api, registry logs to each test which fails :(
14:13:11 <jokke_> hmm-m wondering if throwing out the last registry bits would solve that
14:13:51 <abhishekk> https://github.com/openstack/glance/blob/master/glance/tests/functional/__init__.py#L930
14:14:17 <abhishekk> https://github.com/openstack/glance/blob/master/glance/tests/functional/__init__.py#L1298
14:14:36 <abhishekk> I will try to avoid attaching registry logs, lets see
14:15:04 <jokke_> kk
14:15:08 <jokke_> #topic rocky-only cache-manage bug
14:15:32 <abhishekk> this is a straight fix
14:15:33 <jokke_> So we still seem to build the executable for cache-manage in rocky even it really doesn't exist
14:15:37 <jokke_> yup
14:15:54 <abhishekk> yes
14:16:22 <jokke_> #topic Open Discussion
14:16:25 <jokke_> ok
14:17:50 <jokke_> #link https://review.opendev.org/#/c/678015/
14:17:57 <abhishekk> I have not much to discuss, just stack of patches pending for reviews, 3-4 of them already has +2
14:19:35 <jokke_> so two things, a) Devstack is our 21st priority while production is like priority 1 b) are you really sure about the parameter names, as what you are saying is not what is documented for oslo_config
14:20:12 <abhishekk> jokke_, yes 100 %, I have applied it and checked
14:20:46 <abhishekk> with CONF.opts.config_dir it gives error and with CONF.config_dir it shows actual value
14:20:52 <abhishekk> same with config_file
14:21:00 <jokke_> ok, gr8 thnx ... I'm still fighting to get my test env building so good to know
14:21:18 <jokke_> will change those right now
14:21:27 <abhishekk> cool,
14:22:35 <jokke_> done, thanks again .... I will test and verify that it actually works once I get my test env running again
14:23:59 <abhishekk> great
14:24:09 <jokke_> So should be pretty self explanatory, but I wanted to minimize any hardcoding of that file path
14:25:09 <abhishekk> I have one doubt though, if CONF.reload_config_files()  is accessing CONF.config_file then will it return list including image-import.conf file?
14:25:23 <abhishekk> just give me a minute I will confirm
14:26:13 <jokke_> the oslo_config.ConfigOpts.find_file is actually decently smart. It looks for the file based on few criteria starting from the folder it's consuming other config files. So even if one use /x/y/z/glance-api.conf, has /x/y/z/glance-image-import.conf and /etc/glance/glance-image-import.conf it will return the /x/y/z/ path one
14:27:16 <abhishekk> it is
14:27:37 <jokke_> So we really mainly assume that as long as the file is in same folder as rest of the configs used, it should work
14:27:47 <abhishekk> right
14:28:05 <jokke_> unless it's listed on command line, in which case we know it's being parsed already
14:28:26 <jokke_> same thing when config-dir is used instead of config-file
14:29:25 <abhishekk> yes
14:29:29 <jokke_> abhishekk: so you tested that version and it works?
14:30:25 <abhishekk> jokke_, Not fully, I just found that the import-conf file is appended to config_file option
14:30:56 <jokke_> as it should, no?
14:31:17 <abhishekk> I need to revert systemd file to avoid reading of config_dir and just read api.conf
14:31:22 <abhishekk> jokke_, yes
14:31:56 <jokke_> ah, yes if you're using config_dir that should not do anything
14:33:07 <abhishekk> yes
14:33:18 <abhishekk> I will check it and update the same on the patch
14:34:04 <jokke_> if you're using --config-file(s) that should pick the glance-image-import.conf automatically if it exists either in the folder(s) you're picking the configs from or from the bunch of default folders we look for configs
14:37:58 <jokke_> but that's all from me. We can end the meeting and continue this offline
14:38:18 <abhishekk> yes
14:38:40 <abhishekk> we can continue offline
14:38:48 <jokke_> ok, thanks!
14:38:53 <jokke_> #endmeeting