07:38:56 <brinzhang> #startmeeting openstack_cyborg
07:38:56 <opendevmeet> Meeting started Fri Jan 21 07:38:56 2022 UTC and is due to finish in 60 minutes.  The chair is brinzhang. Information about MeetBot at http://wiki.debian.org/MeetBot.
07:38:56 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
07:38:56 <opendevmeet> The meeting name has been set to 'openstack_cyborg'
07:39:17 <brinzhang> #role call
07:39:24 <brinzhang> #info brinzhang
07:39:36 <songwenping> #info songwenping
07:41:41 <brinzhang> #topic propose eric-xie as cyborg core
07:42:17 <eric-xie> #info eric-xie
07:43:05 <brinzhang> eric-xie,songwenping: I have proposed the maillist for proposing eric-xie as cyborg core, with response from zhipeng Huang, you have to more review for cyborg patches
07:43:45 <eric-xie> Is review limited to cyborg?
07:43:59 <songwenping> cool
07:44:01 <eric-xie> got it
07:44:23 <songwenping> cyborg-spec tempest-plugin and cyborgclient
07:44:43 <brinzhang> but it's a long time work, please invest more time in this
07:45:07 <eric-xie> Hope all submit more patches.
07:45:50 <songwenping> we can invite more guys to contribute.
07:45:54 <brinzhang> if not any negative votes, I will add you to core group too
07:46:32 <eric-xie> cool
07:46:34 <brinzhang> yes, eric-xie, you can invite your partner join the cyborg
07:47:17 <eric-xie> ok
07:47:34 <brinzhang> help new contributor to contribute community is a great thing too
07:47:58 <brinzhang> #topic xilinx driver
07:48:31 <brinzhang> https://review.opendev.org/c/openstack/cyborg/+/804212
07:49:01 <brinzhang> with this patch, IMO, it's ready to go, and you should fix songwenping's comment inline
07:49:39 <eric-xie> i rebased it, and modifing it by wenping's comment now
07:49:54 <brinzhang> cool
07:50:03 <eric-xie> will submit it later.
07:50:25 <brinzhang> after zuul +1, you can ping us in wechat or irc channel
07:50:44 <eric-xie> ok
07:50:58 <brinzhang> #topic vGPU support and OWNER_CYBORG trait
07:51:36 <brinzhang> songwenping: Is there some update with this feature?
07:52:10 <songwenping> nothing now
07:52:33 <songwenping> bauzas have removed the spec to Z
07:54:24 <brinzhang> ok, moved not removed :), let's continue
07:54:41 <songwenping> yeah moved
07:55:05 <brinzhang> #topic Fix the null value of `deployable_id` in arq
07:55:13 <brinzhang> https://review.opendev.org/c/openstack/cyborg/+/823892
07:55:51 <brinzhang> eric-xie, songwenping has left his confusing under this patch, please fix it
07:56:26 <eric-xie> ack
07:56:48 <brinzhang> and I saw some comment in PS6, do you debug in your env, is there some data with deployable_data?
07:57:47 <eric-xie> yes, i debuged it with 'print'. the deploy id is 1.
07:59:09 <brinzhang> eric-xie: let's wait for songwenping, his down
07:59:25 <brinzhang> hi Jorhson, welcome to join our meeting
08:00:10 <songwenping> sorry offline just now
08:01:03 <eric-xie> songwenping: i debuged it with 'print'. the deploy id is 1
08:01:03 <songwenping> i donnot think the worth to fill the deployable_id value in releasenote is proper. to show in API future?
08:01:20 <songwenping> i see the value is 1.
08:01:45 <songwenping> but the mock_save.return_value is wrong
08:02:34 <eric-xie> for releasenote, i agreed with songwenping that is unnessesary.
08:02:43 <songwenping> the return_value should include deployable_id
08:08:17 <eric-xie> i will confirm it later.
08:09:32 <songwenping> cool
08:09:47 <brinzhang> ok
08:10:37 <brinzhang> eric-xie: we have an aggrement, after you update, you can ping us to review
08:11:36 <eric-xie> ack
08:12:01 <songwenping> you can put the  mock_save.return_value afte apply_patch
08:12:12 <songwenping> and assign the obj_extarq to it.
08:12:50 <eric-xie> OK
08:12:51 <songwenping> or just remove it.
08:17:18 <brinzhang> #topic pending patch recreate mdev
08:17:19 <brinzhang> https://review.opendev.org/c/openstack/cyborg/+/815935
08:17:52 <brinzhang> songwenping, eric: with this feature what do you think?
08:18:45 <songwenping> seems reasonable but depend on the vgpu feature.
08:19:03 <brinzhang> IMO, it make sense, when vm reboot, that should recreate a mdev
08:20:05 <brinzhang> vGPU feature cannot merge recently, I mean we can accept it firstly, after vGPU feature support then to update
08:23:01 <songwenping> https://review.opendev.org/c/openstack/nova/+/810220 nova side has the same problem.
08:23:23 <songwenping> we are not hurry to merge in cyborg.
08:24:14 <brinzhang> ok, let's tracing this, and it's not very high priority to merge
08:24:21 <brinzhang> #topic AOB
08:24:45 <brinzhang> eric-xie, songwenping: do you have some topic to discuss?
08:25:33 <eric-xie> nothing from myside
08:26:20 <songwenping> nothing from myside
08:26:30 <brinzhang> ok, thanks all
08:26:40 <brinzhang> let's wrap our today's meeting
08:26:45 <brinzhang> bye
08:26:52 <brinzhang> #endmeeting openstack_cyborg