06:04:35 <brinzhang_> #startmeeting cyborg
06:04:35 <opendevmeet> Meeting started Fri Oct 29 06:04:35 2021 UTC and is due to finish in 60 minutes.  The chair is brinzhang_. Information about MeetBot at http://wiki.debian.org/MeetBot.
06:04:35 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
06:04:35 <opendevmeet> The meeting name has been set to 'cyborg'
06:04:58 <brinzhang_> #topic roll call
06:05:02 <brinzhang_> #topic roll call
06:05:08 <brinzhang_> #info brinzhang
06:05:20 <songwenping__> #info songwenping
06:06:09 <brinzhang_> #topic specs proposed in Yoga
06:07:33 <brinzhang_> wenpingsong__: with the enable/disable device status spec, I left some comments, and you can check again
06:07:42 <brinzhang_> https://review.opendev.org/c/openstack/cyborg-specs/+/815460
06:08:17 <brinzhang_> do you have some questions about this topic?
06:08:38 <brinzhang_> vmaccel: hi are you around?
06:09:23 <songwenping__> i havenot look into your comment.
06:09:55 <brinzhang_> dont worry, you can review it after meeting
06:10:05 <brinzhang_> hi eric-xie
06:10:12 <songwenping__> i have the question about the show api return value.
06:10:56 <brinzhang_> wenpingsong__: you mean the attributes_list column?
06:11:03 <eric-xie> Hi all
06:12:11 <brinzhang_> hi, welcome join in our meeting
06:12:31 <songwenping__> yes, and num_accelerators etc.
06:13:28 <brinzhang> ah, num_accelerators also a new column will add with this feature
06:14:13 <brinzhang> and she is not describe in *propose changes*
06:15:16 <brinzhang> wenpingsong: "attributes_list" you can paste your local device show, then we trace it to
06:16:02 <songwenping> i think we shouldnot change the return value except is_maintaining related this feature.
06:17:17 <songwenping> eric-xie: you can attach the device show return value as my env is not available.
06:17:40 <songwenping> for this spec: v
06:17:42 <songwenping> https://review.opendev.org/c/openstack/cyborg-specs/+/815460
06:19:17 <brinzhang> as discussed on PTG, we should record the num of the device used by instance, if you dont add it, how to check the device is available or in-use?
06:20:15 <songwenping> we can add it in other feature
06:22:03 <eric-xie> songweiping: attach the return value here?
06:22:21 <songwenping> to the spec pls
06:23:59 <eric-xie> have added it.
06:25:42 <songwenping> thanks
06:27:35 <brinzhang> songwenping: no, we can talked under the spec, num_accelerators is useful for this feature, IMO, it should added in this spec, and it also describe in spec
06:30:54 <songwenping> ok, but there are also some extra values such as rp_uuid and bitstream_id.
06:31:56 <brinzhang> another spec is xilin driver spec need to reiew: https://review.opendev.org/c/openstack/cyborg-specs/+/803103
06:32:44 <brinzhang> songwenping: yeah, we should have a clear decision with disable/enable feature changes
06:33:14 <brinzhang> pls add your doubt on the spec
06:33:27 <songwenping> ok i donot know where she get the return value.
06:34:53 <brinzhang> eric-xie: xilin driver spec need to fix the pep8 failed
06:35:03 <eric-xie> I changed it and submitted it based on ptg's comments.
06:35:21 <eric-xie> OK, i will fix it as soon.
06:35:36 <brinzhang> ack, added it in my list
06:35:54 <songwenping> the content is LGTM
06:38:10 <brinzhang> ok
06:38:47 <brinzhang> another spec is pmem driver spec: https://review.opendev.org/c/openstack/cyborg-specs/+/799173
06:40:34 <opendevreview> Shuai Qian proposed openstack/cyborg master: support recreating mdevs after node reboots  https://review.opendev.org/c/openstack/cyborg/+/815935
06:40:34 <songwenping> eric-xie: but i still have some nits for your spec, i will comment later.
06:41:08 <eric-xie> songwenping: got it.
06:41:27 <songwenping> the pmem driver still have question about the data to report
06:41:57 <songwenping> as nova doesnot report the pmem as a rp
06:43:46 <songwenping> i have talked with xinran on wechat but has no response
06:44:24 <brinzhang> ack
06:44:30 <brinzhang> #topic pending patches
06:44:41 <brinzhang> cyborg project: https://review.opendev.org/q/project:openstack%252Fcyborg+status:open
06:44:46 <brinzhang> python-cyborgclient: https://review.opendev.org/q/project:openstack%252Fpython-cyborgclient+status:open
06:44:51 <brinzhang> cyborg-tempest-plugin: https://review.opendev.org/q/project:openstack%252Fcyborg-tempest-plugin+status:open
06:45:02 <brinzhang> songwenping, eric-xie:  pls review
06:46:08 <songwenping> ack
06:47:23 <eric-xie> ack
06:56:02 <songwenping> we can let this patch go:https://review.opendev.org/c/openstack/python-cyborgclient/+/809093
06:56:46 <songwenping> brinzhang: if there are nothing propose, we can end the meeting.
06:57:00 <brinzhang> yeah, let's ending
06:57:49 <brinzhang> #end meeting
06:57:54 <brinzhang> #endmeeting
06:58:29 <brinzhang> #endmeeting
06:58:41 <brinzhang> #endmeeting cyborg
07:04:27 <opendevmeet> brinzhang: Error: Can't start another meeting, one is in progress.  Use #endmeeting first.
07:04:39 <brinzhang> #endmeeting