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