03:08:52 <Yumeng> #startmeeting openstack-cyborg 03:08:53 <openstack> Meeting started Thu Aug 6 03:08:52 2020 UTC and is due to finish in 60 minutes. The chair is Yumeng. Information about MeetBot at http://wiki.debian.org/MeetBot. 03:08:53 <brinzhang_> hi xinranwang_, swp20, chenke, good moning 03:08:54 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 03:08:56 <openstack> The meeting name has been set to 'openstack_cyborg' 03:08:59 <Yumeng> #topic Roll call 03:09:04 <Yumeng> #info Yumeng 03:09:16 <Yumeng> #topic Agenda 03:09:16 <brinzhang_> #info brinzhang_ 03:09:28 <xinranwang__> #info xinranwang__ 03:09:40 <swp20> #info swp20 03:09:52 <chenke> #info chenke 03:10:27 <Yumeng> Will be a quick meeting today 03:10:36 <Yumeng> progress review and one question from my side. 03:11:17 <Yumeng> #topic progress track 03:11:31 <s_shogo> #info s_shogo 03:11:40 <Yumeng> policy patch got comments from Ghanshyam(high priority): https://review.opendev.org/#/c/740542/ 03:11:56 <Yumeng> and I will fix the comments this week. 03:12:05 <brinzhang_> cool 03:12:20 <Yumeng> thanks brinzhang_ for asking gmann to help review 03:12:33 <brinzhang_> Yumeng: np 03:12:38 <Yumeng> I also asked keystone people to help review 03:12:59 <brinzhang_> yes, we should 03:13:12 <Yumeng> and next： nova operations support patches(high priority) need review: https://review.opendev.org/#/c/715326/, https://review.opendev.org/#/c/729563/ 03:14:14 <Yumeng> nova people think we should focus on operation support in this release, and we should. 03:15:40 <Yumeng> thanks brinzhang_, sean and swp20 for the patch. I spare more time on them. 03:15:58 <swp20> Yumeng: np 03:16:32 <brinzhang_> I think the evacuate patch is ok now, need to +2 from gibi 03:16:57 <swp20> all, pls review if have any spare time, tks. 03:17:45 <Yumeng> yes, always lack review.=: 03:18:07 <Yumeng> and programme API need refresh(high priority): https://review.opendev.org/#/c/698190/ 03:18:29 <Yumeng> I think this should be one of the high priority goals in V release. 03:18:51 <brinzhang_> we need to add the microversion of this patch 03:20:04 <Yumeng> emmm.. will that work cost much time or effort? 03:20:06 <brinzhang_> just need to add a decorator to check the microversion, it seems not difficult 03:20:51 <brinzhang_> and before in xinranwang_'s patch already contained the decorator, just move and debug for it, I think 03:21:00 <Yumeng> brinzhang_: alright. 03:21:06 <s_shogo> ok, I check that 03:21:14 <Yumeng> xinranwang__: what do you think? 03:21:53 <xinranwang__> Yes, program API need a new microversion check. 03:21:59 <brinzhang_> s_shogo: plse reference https://review.opendev.org/#/c/696860/8/cyborg/api/controllers/base.py@101 03:22:29 <xinranwang__> We need a decorator to check it. Please refer to my mircroversion previous patchset. 03:22:49 <xinranwang__> Thanks brinzhang_ for giving this link 03:23:35 <s_shogo> brinzhang_ xinranwang__ thanks for the link 03:23:37 <brinzhang_> I hope this decorator is sample enough for us to use, and also that we can inviate alex_xu and gmann to review, they are expert at API 03:23:38 <Yumeng> sounds good. 03:24:04 <brinzhang_> s/inviate/invite/ 03:24:34 <brinzhang_> xinranwang_, s_shogo: np 03:25:12 <Yumeng> brinzhang_: sure we can. 03:26:47 <Yumeng> and Maillist discussion on SRIOV SmartNic Support Spec: http://lists.openstack.org/pipermail/openstack-discuss/2020-August/016339.html , https://review.opendev.org/#/c/742785/ 03:28:32 <Yumeng> This spec may take long time to get ideas from nova and neutron and reach an agreement. 03:28:56 <brinzhang_> I dont think we can merge this spec in V release 03:29:52 <brinzhang_> first seen this spec just now, I have not reviewed into, but I know there are many content to discuss, refer previous ML 03:30:31 <Yumeng> yes, you'are right. not urgent, but important for us to start and continue response and resolve the open issues. 03:31:40 <Yumeng> I was thinking set it as a medium priority for this release, not urgent, what do you think ? 03:31:59 <brinzhang_> yes, if we can have an aggrement of this spec, I think it will be better in V release 03:33:15 <brinzhang_> nova spec will be freezed, I dont think we can have an aggrement before the deadline 03:33:43 <brinzhang_> but we can try 03:33:47 <Yumeng> yes, sean and gibi mentioned this in the ML. 03:33:59 <Yumeng> pls refer to http://lists.openstack.org/pipermail/openstack-discuss/2020-August/016339.html 03:34:07 <brinzhang_> xinranwang__, Yumeng, shaohe_feng 03:34:33 <Yumeng> they think we can continue work, but don't have to merge before spec freeze. 03:34:35 <brinzhang_> I have received the ML 03:34:57 <brinzhang_> yes, it is my mean too. 03:35:41 <brinzhang_> if we can make this spec is ready in V, that we can start to work in W realease, that will be merged fast 03:36:43 <Yumeng> yes, that's the best result!. let's try. 03:37:24 <xinranwang__> LOL 03:37:25 <Yumeng> ok. that's all for today's progress track. 03:37:45 <Yumeng> Is there any other patch need to discuss? 03:37:46 <xinranwang__> In next release, it might be the high priority 03:38:07 <Yumeng> yse xinranwang__ 03:38:42 <Yumeng> ok, the last question from my side. 03:38:53 <Yumeng> # topic AOB 03:39:01 <songwenping_> yes, aggree to release in next release. 03:39:13 <xinranwang__> Ah, please review QAT patch as well. Hope we can introduce some new device in this release too. :) 03:39:43 <Yumeng> I noticed we always lack of review. how to better review our patches? should we focus on several patches during a period? like setting high priority patches for each week? 03:39:49 <brinzhang_> My review work will be in next week ^ 03:41:40 <xinranwang__> Yes, we can set the priority of reviews according to each patches process. 03:42:10 <Yumeng> high priority should be based on severals rules: 1) ready to review 2) need to catch up an appropriate deadline or milestones 03:42:33 <Yumeng> 3) an appropriate priority 03:43:55 <brinzhang_> agree 03:44:04 <Yumeng> xinranwang__: yes, that's what I mean. We have few contributors to review many patchsetss. 03:45:09 <Yumeng> songwenping_, s_shogo, what do you think? 03:45:29 <xinranwang__> Sometimes I am a little bit confused about if the patch is ready for review. Maybe we could use wf-1 when it is not ready? 03:45:41 <songwenping_> yes, it's ok for me. 03:46:16 <s_shogo> that is good idea >> add priority 03:46:43 <Yumeng> agree, I would encourage to use wf-1 when it is not ready. 03:47:47 <songwenping_> sound good. 03:48:30 <Yumeng> ok. cool. so I will release review priority for each week in wechat group. 03:48:34 <brinzhang_> have launch now ^ goodbye all 03:49:01 <Yumeng> that's all fro today. goodbye brinzhang_ 03:49:07 <Yumeng> Thank you all. 03:49:17 <Yumeng> Have a good day! bye bye 03:49:22 <songwenping_> goodbye 03:49:26 <xinranwang__> bye 03:49:34 <s_shogo> bye 03:49:35 <Yumeng> #endmeeting