| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2020-08-06 | |||
| 03:01:22 | chenke | hi all~ | |
| 03:01:47 | Yumeng | hi chenke | |
| 03:02:04 | chenke | hi mengmeng. | |
| 03:06:03 | swp20 | Hi Yumeng, chenke. | |
| 03:06:12 | chenke | hi wenping | |
| 03:06:23 | xinranwang__ | hi all | |
| 03:07:44 | Yumeng | hi swp20 xinranwang__ and s_shogo | |
| 03:07:45 | swp20 | hi xinran, shogo. | |
| 03:07:53 | s_shogo | Hi all | |
| 03:08:07 | Yumeng | is brinzhang0 around? | |
| 03:08:22 | Yumeng | hi brinzhang_ | |
| 03:08:36 | brinzhang_ | hi Yumeng | |
| 03:08:50 | Yumeng | great! let's get started | |
| 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: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 | |