| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2020-02-20 | |||
| 03:24:00 | brinzhang_ | shaohe_feng: I think it should wait for Sundar in, otherwise anyone does not to say something, as I know, Sundar want to complete it | |
| 03:24:28 | brinzhang_ | Although he will go to new jobs. | |
| 03:24:35 | shaohe_feng | OK, next. | |
| 03:24:51 | xinranwang | we are still waiting for Sundar's reply | |
| 03:26:05 | shaohe_feng | Other patch do you want to discuss? | |
| 03:26:23 | Yumeng | chenke proposed this tiny fix https://review.opendev.org/#/c/708367/ | |
| 03:26:30 | shaohe_feng | Or we can go through the status of pending patch | |
| 03:26:41 | Yumeng | I reviewed this, it's ready to W+1 | |
| 03:26:53 | chenke | ye. We can go through the pending patch. | |
| 03:26:55 | brinzhang_ | Some cleanup job, please review https://review.opendev.org/#/c/707332/ | |
| 03:26:59 | Yumeng | pls help to W+1 | |
| 03:27:22 | Li_Liu | just did | |
| 03:27:26 | chenke | A lot of patches need to be review. | |
| 03:27:32 | shaohe_feng | Thanks Li_Liu | |
| 03:28:13 | Yumeng | aha. Thanks! also cyborgclient patcheshttps://review.opendev.org/#/q/owner:shogo.saito.ac%2540hco.ntt.co.jp+status:open | |
| 03:28:36 | Yumeng | patches https://review.opendev.org/#/q/owner:shogo.saito.ac%2540hco.ntt.co.jp+status:open | |
| 03:29:37 | chenke | +1 | |
| 03:29:39 | xinranwang | cyborg client looks good to me. I have tested it | |
| 03:29:43 | s_shogo | Thank you for your mention about the client and sdk patches. | |
| 03:29:53 | Li_Liu | will take a look today | |
| 03:30:11 | chenke | I also tested shogo's patch. LGTM. | |
| 03:30:30 | shaohe_feng | good work on the client and sdk . | |
| 03:30:42 | shaohe_feng | let we speed up the review on them. | |
| 03:31:09 | shaohe_feng | brinzhang_ you clean up looks good. | |
| 03:31:29 | brinzhang_ | shaohe_feng: thanks | |
| 03:33:12 | brinzhang_ | https://review.opendev.org/#/c/701518/4/cyborg/tests/unit/accelerator/drivers/gpu/test_utils.py@101 | |
| 03:33:53 | brinzhang_ | This code I want to know why add [], chenke can, you give some description? | |
| 03:34:31 | brinzhang_ | It's a separate line of code, why add []? | |
| 03:34:56 | chenke | That's ok. I tested it. | |
| 03:35:29 | chenke | [] is a magical function | |
| 03:35:45 | openstackgerrit | Merged openstack/cyborg-specs master: Fix some minor concept and typo https://review.opendev.org/708367 | |
| 03:35:55 | Yumeng | haha. brinzhang_: append included within []. seems the same as extend. | |
| 03:36:05 | shaohe_feng | I see Dan submit a patch for a provider improvement | |
| 03:36:29 | Yumeng | let me show you the results later. | |
| 03:36:39 | Yumeng | I will ping you in wechat. | |
| 03:36:50 | shaohe_feng | We can have a look at this patch. | |
| 03:36:54 | brinzhang_ | Yumeng, chenke, yes, that why I want to ask, if so, why do for x in y: xxx | |
| 03:37:17 | shaohe_feng | xinranwang you are a provider expert, also help to review it. | |
| 03:37:40 | brinzhang_ | From Dan's patch I found some error in exception, I will submit a patch, and fixed some errors | |
| 03:37:56 | openstackgerrit | Merged openstack/python-cyborgclient master: v2 API Support(using openstack-sdk) https://review.opendev.org/681391 | |
| 03:38:59 | shaohe_feng | ||
| 03:39:23 | xinranwang | shaohe_feng: what patch are you talking about, nova provider config patch ? | |
| 03:39:59 | brinzhang_ | https://review.opendev.org/708726 and https://review.opendev.org/708728 | |
| 03:40:01 | shaohe_feng | we have discuss to improve devstack before. | |
| 03:40:40 | brinzhang_ | and there are sean's patch base on | |
| 03:40:43 | shaohe_feng | brinzhang_ yes, thanks | |
| 03:41:46 | shaohe_feng | sean mooney submit a serial patches. | |
| 03:42:11 | xinranwang | brinzhang_: thanks, will look into them | |
| 03:42:25 | shaohe_feng | many of them about devstack plugin. | |
| 03:42:42 | shaohe_feng | we have found many issues devstack plugin | |
| 03:43:04 | shaohe_feng | but they are pending for a half years. | |
| 03:44:22 | shaohe_feng | such as https://review.opendev.org/#/c/693784/ | |
| 03:45:22 | shaohe_feng | also we have discuss the fake drive improvement | |
| 03:46:06 | shaohe_feng | https://review.opendev.org/#/c/697011/ | |
| 03:46:16 | shaohe_feng | they are pending. | |
| 03:46:36 | shaohe_feng | any one also want to talk about your patches? | |
| 03:48:12 | shaohe_feng | Am I online? | |
| 03:48:30 | chenke | Sure. | |
| 03:49:19 | shaohe_feng | OK, any one else? | |
| 03:49:25 | shaohe_feng | one/more | |
| 03:49:43 | chenke | Shogo's another patch:https://review.opendev.org/#/c/679914/ | |
| 03:50:00 | chenke | Aslo need review. +1 | |
| 03:50:22 | s_shogo | Thanks chenke, That is also important for the client. | |
| 03:50:38 | Yumeng | will do. | |
| 03:51:17 | s_shogo | That needs +1 from openstacksdk folks. It may needs contact them , from me via e-mail and so on. | |
| 03:51:31 | Li_Liu | Sure | |
| 03:51:48 | Li_Liu | Will review the SDK patches | |
| 03:51:56 | chenke | Nice. | |
| 03:53:08 | s_shogo | Thank you Yumeng and Li_Liu . | |
| 03:53:53 | shaohe_feng | s_shogo any more patches need to be discussed here? | |
| 03:54:15 | s_shogo | shaohe_feng that's ok. thanks. | |
| 03:54:32 | shaohe_feng | OK, thanks for you hard work. | |
| 03:54:36 | shaohe_feng | go ahead. | |
| 03:54:47 | Yumeng | s_shogo: my pleasure.:) thanks for all the efforts! | |
| 03:55:21 | shaohe_feng | any more? | |
| 03:55:59 | shaohe_feng | if no, is every one interested in timeout issue? | |
| 03:57:31 | shaohe_feng | ^ brinzhang_ chenke | |
| 03:58:02 | brinzhang_ | shaohe_feng: I have nothing, later will send the fixed patch to wechat | |
| 03:58:04 | chenke | I had saw your patch. | |
| 03:58:04 | Yumeng | maybe lunch time for them. ^^ | |
| 03:58:13 | chenke | We can chat on the weixin. | |
| 03:58:14 | brinzhang_ | thanks all | |
| 03:58:43 | shaohe_feng | if no, please help to review the serial patches https://review.opendev.org/#/c/708655/7 | |
| 03:58:59 | shaohe_feng | I have give a enough information in the patches | |
| 03:59:14 | brinzhang_ | Yumeng: yes, it time lunch :) | |
| 03:59:17 | shaohe_feng | about the history and the root cause | |
| 03:59:32 | shaohe_feng | OK, let's end the meeting. | |
| 03:59:36 | shaohe_feng | have good lunch | |
| 03:59:42 | chenke | Ok. | |
| 03:59:49 | chenke | bye~ | |
| 04:00:05 | shaohe_feng | Thanks All. | |
| 04:00:18 | shaohe_feng | #endmeeting | |
| 04:00:20 | openstack | Meeting ended Thu Feb 20 04:00:18 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 04:00:21 | openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-02-20-03.15.html | |
| 04:00:23 | openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-02-20-03.15.txt | |
| 04:00:24 | openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-02-20-03.15.log.html | |
| 04:04:13 | openstackgerrit | Brin Zhang proposed openstack/cyborg master: Fixed define incorrectly of the exc msg https://review.opendev.org/708796 | |
| #openstack-cyborg - 2020-02-22 | |||
| 10:22:38 | openstackgerrit | YumengBao proposed openstack/cyborg master: Add cpid_type to db schema https://review.opendev.org/709286 | |
| #openstack-cyborg - 2020-02-24 | |||
| 06:47:19 | openstackgerrit | YumengBao proposed openstack/cyborg master: Update cpid_type allowed values in db schema https://review.opendev.org/709286 | |
| 09:43:26 | openstackgerrit | YumengBao proposed openstack/cyborg master: Update cpid_type allowed values in db schema https://review.opendev.org/709286 | |