16:00:03 <ildikov> #startmeeting cinder-nova-api-changes
16:00:04 <openstack> Meeting started Thu Aug 10 16:00:03 2017 UTC and is due to finish in 60 minutes.  The chair is ildikov. Information about MeetBot at http://wiki.debian.org/MeetBot.
16:00:05 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:00:07 <openstack> The meeting name has been set to 'cinder_nova_api_changes'
16:00:15 <ildikov> johnthetubaguy jaypipes e0ne jgriffith hemna mriedem patrickeast smcginnis diablo_rojo xyang1 raj_singh lyarwood jungleboyj stvnoyes
16:00:39 <jungleboyj> :-)  Lurking.  In 3 meetings right now.
16:00:50 * smcginnis is here but at ops midcycle
16:01:14 <ildikov> all the important people :)
16:01:22 <ildikov> should be a short one today
16:02:19 <jungleboyj> :-)
16:02:30 <jungleboyj> Everything is fixed.  The people celebrate.
16:02:32 <mriedem> o/
16:02:44 <ildikov> jungleboyj: hopefully getting close to that :)
16:02:50 <ildikov> ok, let's start
16:03:10 <ildikov> so we have some failures due to the uWSGI switch in Cinder
16:03:33 <ildikov> we will have a new client out after the lib-freeze, which should put us back to the right track
16:03:50 <ildikov> mriedem: any preference on how to handle if there are issue with the cinderclient?
16:04:25 <ildikov> mriedem: I mean whether log and fall back to the old flow or let the exception bubble up?
16:04:53 <mriedem> umm
16:05:04 <mriedem> no opinion at this time
16:05:22 <ildikov> ok, then I will leave the code as is for now
16:06:00 <ildikov> we are also working on to solve the 'blockdev' errors in the logs
16:06:36 <ildikov> jgriffith has some findings, I believe the connection_info is still not good enough for Nova's taste is our conclusion at the moment
16:07:28 <jgriffith> sorry... getting coffee
16:07:44 <ildikov> jgriffith: no worries, coffee is important :)
16:08:05 <ildikov> jgriffith: can you share some updates on the above?
16:08:09 <jgriffith> I have clean tests again, reworked the connection info data
16:08:49 <ildikov> jgriffith: I guess it means some changes on the Nova side
16:09:08 <ildikov> ?
16:09:13 <jgriffith> there was something going on with os-brick causing all sorts of retries and iscsi logouts, our "new" block_device attach method appeared to have some shortcomings as well
16:09:48 <jgriffith> so I reworked the attachmen_update translation stuff to match the old records, and reuse the legacy_attach in block_device.py
16:10:04 <jgriffith> now the encryption checks seem happy again, and we're getting stable iscsi connections
16:10:41 <jgriffith> anyway... I'll start cleaning things up and also look for any other issues and get the patch updated hopefully today
16:10:46 <jgriffith> if not should be tomorrow for sure
16:11:08 <ildikov> jgriffith: sounds good, thank you!
16:11:40 <stvnoyes> i also have a rv up to update that tempest test - https://review.openstack.org/#/c/492570
16:11:41 <ildikov> mriedem: if we touch the translation function, is it ok in the attach patch?
16:12:04 <mriedem> ildikov: the attach patch needs to be as clean as possible,
16:12:18 <mriedem> so anything that can be split into separate patches earlier in the series is better for review
16:12:28 <ildikov> mriedem: ok, noted, thanks
16:12:31 <ildikov> stvnoyes: thanks!
16:13:19 <stvnoyes> sorry but have to run out for a few minutes, I'll catch up later
16:13:37 <ildikov> stvnoyes: no worries, I don't think we have much more than this for today
16:14:23 <ildikov> mriedem: I guess if we have a new cinderclient and get the tests and test logs clean then we should have a good chance to get this merged before the PTG
16:14:31 <ildikov> mriedem: is my assumption correct?
16:15:24 <mriedem> heh
16:15:42 <mriedem> just worry about getting the stuff working and ready for review for when queens opens up
16:15:53 <mriedem> i don't really want to be having this conversation on the day of rc1 honestly
16:16:01 <mriedem> yes i'd like this all done before the PTG
16:16:12 <ildikov> fair enough
16:16:25 <ildikov> then that's it for today from my side
16:16:32 <ildikov> anything else from anyone to discuss?
16:17:34 <ildikov> ok, I take it as a no :)
16:17:58 <ildikov> let's get back to fixing things and let mriedem to deal with RC1
16:18:08 <smcginnis> Probaly enough at this point.
16:18:12 <ildikov> thanks everyone!
16:18:15 <ildikov> smcginnis: :)
16:18:24 <smcginnis> Thanks!
16:18:29 <ildikov> #endmeeting