Posted	Nick	Remark
#openstack-cyborg - 2018-01-24
15:03:02	zhipeng	let's wait for a few minutes for others to show up
15:05:59	zhipeng	guys if you are here for the meeting
15:06:09	zhipeng	please use #info to identify yourself
15:06:33	helloway	#info helloway
15:06:55	Dutch	#info Dutch
15:07:20	Amol	#info amol
15:07:47	Vipparthy	#info Vipparthy
15:07:59	zhipeng	Amol which company are you from ?
15:09:51	Amol	Hi Zhipeng . I am from nvxltech
15:10:16	zhipeng	ah thx, same with Vipparthy right ?
15:10:25	Amol	Yup
15:11:54	zhipeng	cool :)
15:12:10	zhipeng	Ok I see Intel folks flowing in, let's start the discussion
15:12:21	zhipeng	#topic Queens Release Final Push
15:13:00	dolpher	Hi
15:13:38	zhipeng	#info Generic Driver
15:13:43	zhipeng	#link https://review.openstack.org/525057
15:13:58	shaohe_feng_	hello all
15:14:01	zhipeng	crushil have you look at the comments from shaohe_feng_ and others ?
15:15:06	crushil	zhipeng, yup. I'm working on the patch rn. Will push a new patchset by this week
15:15:46	zhipeng	I have to cut the release branch Monday next week, so the window is pretty tight :)
15:16:15	zhipeng	I will need your workflow+1 during the weekend crushil :P
15:16:32	crushil	Hmmm. Ok
15:16:43	crushil	I will be online on the weekend
15:16:47	zhipeng	This is the final frenzy kicks in
15:16:52	zhipeng	gr8t thx :)
15:16:57	zhipeng	okey moving on
15:17:07	zhipeng	#info Intel FPGA driver
15:17:19	zhipeng	#link https://review.openstack.org/531129
15:17:54	zhipeng	Li_Liu has shaohe_feng_ addressed your comments ?
15:18:57	Li_Liu	I havn't had the chance to look at the new patch yet
15:19:05	Li_Liu	Doing it now
15:19:22	zhipeng	okey
15:19:46	zhipeng	shaohe_feng_ other than the comments you received, code-wise is your patch ready ?
15:19:52	Dutch	I can also add some input to help keep things vendor neutral
15:20:05	zhipeng	Dutch absolutely
15:20:39	shaohe_feng_	zhipeng: I have test it on our platform, it can works.
15:21:06	shaohe_feng_	zhipeng: but maybe we still need to introduce rootwraper
15:21:07	zhipeng	okey then here are my suggestions:
15:21:27	zhipeng	no rootwrapper for queens :)
15:21:33	zhipeng	we need to discuss that in Dublin
15:21:47	zhipeng	So here are my suggestions at the patch :
15:22:02	shaohe_feng_	the program need root for both intel and xilinx
15:22:05	zhipeng	again due to the tight release window, we will have to land this patch this weekend
15:22:17	dolpher	Li_Liu, Dutch: current implementation in fpga/intel is specific to intel FPGA, for Xilinx, we may need to create a directory of fpga/Xilinx
15:22:44	zhipeng	as I understand the comments are mostly on vendor neutral, as dolpher just mentioned
15:22:44	Li_Liu	dolpher, I agree
15:23:00	dolpher	we need to make the interface generic in the end.
15:23:06	zhipeng	so I would suggest to change the patch accordingly to reflect that this is an Intel FPGA driver
15:23:15	Li_Liu	only issue I have is in cyborg/accelerator/drivers/fpga/util.py
15:23:26	zhipeng	and let's try to harmonize it in the long run
15:23:40	zhipeng	poor gerrit
15:24:22	shaohe_feng_	gerrit need rest :)
15:25:27	zhipeng	anyway, shaohe_feng_ you captured my suggestion ?
15:26:14	shaohe_feng_	yes
15:26:24	zhipeng	great :)
15:26:29	zhipeng	next item
15:26:45	zhipeng	#info Justin leftovers: conductor interaction with Placement
15:26:51	zhipeng	not sure if zhuli is here
15:27:32	zhipeng	but as I have discussed with zhuli and Li_Liu offline previously, we will need to quick modify and copy-paste of nova-sched report.py
15:28:00	zhipeng	and also an extension at the agent level to implement the discover function in full
15:28:22	zhipeng	This is also another big hole we need to fix in a short time
15:28:36	zhipeng	shaohe_feng_ can your team help on the agent side ?
15:29:07	zhipeng	what we need is an equal counterpart of "resource_tracker.py" in cyborg
15:29:19	shaohe_feng_	OK
15:29:32	zhipeng	which I assume is the discover interface that is very basic at the moment
15:29:57	shaohe_feng_	yes.
15:30:33	zhipeng	okey thx :) zhuli and Li_Liu will coordinate on the conductor side
15:30:38	zhipeng	moving on
15:30:40	Li_Liu	sure
15:30:44	zhipeng	#info fix on devstack
15:31:10	shaohe_feng_	resource_tracker.py will check resource periodically
15:32:12	zhipeng	shaohe_feng_ we can simplify that for queens
15:32:23	zhipeng	and write to db instead of a provider-tree like cache
15:32:33	zhipeng	but anyways, back to the topic
15:32:43	zhipeng	#link https://review.openstack.org/536251
15:32:56	zhipeng	another good patch from Jinghan to make devstack working
15:33:16	zhipeng	crushil if you thinks it ok you could just land it :)
15:34:55	crushil	zhipeng, Ok. Will look at it later today
15:35:15	zhipeng	gr8t thx :)
15:35:50	zhipeng	okey folks that is all we need to do for Queens, I see Nova is still panicking on landing patches, so we are not alone :)
15:36:05	zhipeng	let's try to wrap it up before next Mon
15:36:18	dolpher	:)
15:36:49	zhipeng	release is always driving people insane
15:37:53	zhipeng	#action crushil to update the generic driver patch, and work with Howard to land all the remaining patches
15:38:21	crushil	I thought the release data was feb 4th?
15:38:24	zhipeng	#action shaohe_feng_ to update the patch to reflect the patch is Intel FPGA driver
15:38:44	zhipeng	crushil that is like hard cutof, we have to run through the CIs and other stuff
15:38:47	zhipeng	docs ...
15:38:53	crushil	Hmmm
15:39:08	zhipeng	so I left out a week of time for more panicking stuff lol
15:39:11	crushil	Ok, so are we cutting RC1, RC2, etc.
15:39:12	crushil	?
15:39:18	zhipeng	not for this release
15:39:22	crushil	Ok
15:39:29	crushil	We need to start doing that from Rocky
15:39:30	zhipeng	but for Rocky we are gonna use the MS1,MS2,...
15:39:37	crushil	Ok, makes sense
15:39:42	zhipeng	yes, so that things are not so in chaos
15:40:39	zhipeng	#action zhuli Li_Liu and shaohe_feng_ works on the Placement interaction patch (report and discovery)

Earlier   Later