| *** s_shogo has joined #openstack-cyborg | 00:34 | |
| *** yuki_t__ has joined #openstack-cyborg | 02:30 | |
| *** yuki_t__ is now known as yuki_t | 02:30 | |
| s_shogo | I joined this team as successor of Ikuo Otani from NTT Corp, thank you. | 02:54 |
|---|---|---|
| *** Biwei has joined #openstack-cyborg | 03:00 | |
| *** Yumeng has joined #openstack-cyborg | 03:03 | |
| *** Sundar has joined #openstack-cyborg | 03:03 | |
| Sundar | #startmeeting openstack-cyborg | 03:04 |
| openstack | Meeting started Wed Jul 3 03:04:11 2019 UTC and is due to finish in 60 minutes. The chair is Sundar. Information about MeetBot at http://wiki.debian.org/MeetBot. | 03:04 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 03:04 |
| *** openstack changes topic to " (Meeting topic: openstack-cyborg)" | 03:04 | |
| openstack | The meeting name has been set to 'openstack_cyborg' | 03:04 |
| Sundar | #topic Roll call | 03:04 |
| *** openstack changes topic to "Roll call (Meeting topic: openstack-cyborg)" | 03:04 | |
| Sundar | #info SUndar | 03:04 |
| Yumeng | #info Yumeng | 03:04 |
| Biwei | #info Biwei | 03:04 |
| *** xinranwang has joined #openstack-cyborg | 03:04 | |
| Sundar | Hi Yumeng, Biwei, xinranwang | 03:05 |
| Sundar | Let us wait a min for others | 03:05 |
| Yumeng | Hi sundar Biwei and xinran | 03:05 |
| xinranwang | Hi all | 03:05 |
| xinranwang | welcome back Yumeng | 03:06 |
| Yumeng | ^ ^ thanks xinran | 03:06 |
| Sundar | #topic Spec review | 03:06 |
| *** openstack changes topic to "Spec review (Meeting topic: openstack-cyborg)" | 03:06 | |
| s_shogo | Hi I joined this team as successor of Ikuo Otani from NTT Corp, thank you. | 03:07 |
| Sundar | Hi s_shogo | 03:07 |
| Sundar | Welcome to Cyborg | 03:07 |
| Yumeng | welcome s_shogo | 03:07 |
| Sundar | Nothing much on the agenda today except for completing the spec reviews: https://review.opendev.org/#/q/status:open+project:openstack/cyborg-specs+owner:Sundar+after:2019-04-01 | 03:08 |
| yikun | #info yikun | 03:08 |
| Sundar | Please review and provide comments and/or a +1/+2 | 03:09 |
| Sundar | s_shogo: Hope you have got the information from Ikuo on the activities that NTT took up: Python 3 migration and Cyborg client | 03:10 |
| Sundar | #topic AoB | 03:12 |
| *** openstack changes topic to "AoB (Meeting topic: openstack-cyborg)" | 03:12 | |
| Sundar | Please continue the spec review. Any thing else to discuss today? | 03:12 |
| *** wangzhh has joined #openstack-cyborg | 03:14 | |
| wangzhh | Hi all. | 03:15 |
| Yumeng | I just reviewed the tempest patch https://review.opendev.org/#/c/667231/ we may need add a zuul job to run it. | 03:15 |
| Sundar | Hi wangzhh | 03:15 |
| Yumeng | I can add it today. | 03:16 |
| yikun | A question on the example on fake driver slot: https://review.opendev.org/#/c/665318/3/cyborg/accelerator/drivers/fake.py@42 | 03:16 |
| Sundar | yikun: Please see https://review.opendev.org/gitweb?p=openstack/cyborg.git;f=nova-integ/demo_setup.sh;hb=refs/changes/20/626420/24#l80 | 03:17 |
| Sundar | Yumeng: Thanks | 03:18 |
| yikun | For now, we already use the string style slot like "0000:0c:00.0" in exist GPU/FPGA patch, so do you think we should change these to json something like "{domain:xx, slot:xx}" | 03:18 |
| Sundar | yikun:Yes, that is what we concluded in our last IRC meeting: http://eavesdrop.openstack.org/irclogs/%23openstack-cyborg/%23openstack-cyborg.2019-06-19.log.html#t2019-06-19T03:32:48 | 03:19 |
| yikun | ok, sorry to miss that, then we should change it in GPU/FPGA and all other related patch, right? | 03:20 |
| xinranwang | wangzhh: do we need this pci_addr string to name deployables in gpu driver | 03:20 |
| yikun | and then we do NOT support string style slot anymore? only json format will be supported, right? | 03:21 |
| Sundar | yikun: Yes | 03:22 |
| Sundar | xinranwang:: The PCI string is for controlpath_id and attach_handle, not deployables. | 03:22 |
| yikun | OK, It's okay to me, Let's see other idea, xinranwang, wangzhh, | 03:22 |
| wangzhh | xintanwang: Yes. Checked it just now. | 03:22 |
| wangzhh | *xinranwang | 03:23 |
| yikun | : ) | 03:23 |
| xinranwang | Sundar: I mean “this string is used to name the deployable” | 03:23 |
| wangzhh | Yes. | 03:23 |
| xinranwang | wangzhh: ok, thanks. | 03:23 |
| wangzhh | Just name of deployable. | 03:23 |
| yikun | wangzhh: 'r' 't' is very close I know | 03:24 |
| yikun | https://review.opendev.org/#/c/665318/3/cyborg/accelerator/drivers/fake.py@34 | 03:24 |
| Sundar | yikun: I'll reply in that patch too. | 03:25 |
| yikun | Yes, so "driver_dep.name = pci.get('device', '') + '_' + pci["slot"]" the deploy name still should be "device_strstylepciaddr" like "device_0000_00_000" | 03:25 |
| Sundar | A deployable may have more than one VF. Then which PCI address would we give it? | 03:27 |
| Sundar | I mean, what name would be give it? | 03:27 |
| yikun | not sure, wangzhh do you have any idea for GPU deployable name? | 03:28 |
| xinranwang | it's gpu's case, in FPGA driver, currently we do not name deployable using pci addr. | 03:29 |
| xinranwang | But I am not sure how OPAE will report VFs' name | 03:29 |
| yikun | As I know, the deployable logic model is useless in GPU, so I guess the device name can be changed to other. | 03:31 |
| yikun | anyway, we need wangzhh to confirm. | 03:31 |
| Sundar | We may use the deployable name as part of the Placement RP name. So, should we have some uniform naming convention across devices? | 03:32 |
| wangzhh | <ComputeNodeName>_<NumaNodeName>_ <Device_name>_<Device_address> | 03:32 |
| wangzhh | yikun, Can we named it like this? | 03:32 |
| yikun | deployable name? | 03:33 |
| wangzhh | Yes. | 03:33 |
| Sundar | By device address, do you mean ctrl path id? | 03:34 |
| yikun | whatif we do not have the numa? what the name looks like? | 03:34 |
| yikun | Sundar: I guess it is | 03:35 |
| wangzhh | yikun, we can set a default numa node value, such as 'default_numa'. | 03:37 |
| Sundar | I think we had this discussion a long time ago. Must be in some IRC log | 03:37 |
| Sundar | If there is more than one deployable but only one ctrlpath_id, I think we considered some solution for that. | 03:38 |
| Sundar | Let's try to find the old discussion. | 03:39 |
| Sundar | Meanwhile, can we all make it a priority to get the specs reviewed? | 03:39 |
| Sundar | Milestone 2 is approaching fast | 03:39 |
| yikun | anyway the name is not my concern, so back to original question, json style is ok to me if no one is against, I will change it to json style | 03:40 |
| yikun | Sundar: thanks for reminder, I will review the spec today | 03:41 |
| xinranwang | yikun Sundar I'm ok with json, I can change it in FPGA driver too | 03:41 |
| Sundar | Thanks, yikun, xinranwang | 03:42 |
| wangzhh | Me too. | 03:42 |
| yikun | wangzhh: so could change the GPU one, and I will address fake and ascend patch | 03:42 |
| yikun | and next topic, :) | 03:42 |
| Sundar | Thanks, all | 03:44 |
| Sundar | Anything else for today? | 03:44 |
| xinranwang | I have one question about test related code | 03:44 |
| Sundar | xinranwang: sure | 03:45 |
| xinranwang | should we form a class like TestDeployableClass with test_list, test_get inside? | 03:45 |
| yikun | it's for test Deployable obj? | 03:46 |
| Sundar | I'd say yes. We are doing that for v2 (thanks to Biwei): like https://review.opendev.org/#/c/626057/12/cyborg/tests/unit/api/controllers/v2/test_device_profiles.py | 03:46 |
| Sundar | But v1 is not following that pattern. Not sure if we should spend the time to fix v1 tests. | 03:47 |
| yikun | ok, it's the api test | 03:47 |
| xinranwang | I have an impression that I see some thing like TestList() class somewhere | 03:47 |
| yikun | xinranwang: what's your expected? it's okay tome | 03:48 |
| Sundar | xinranwang: Yes, some v1 API tests do that: https://review.opendev.org/gitweb?p=openstack/cyborg.git;f=cyborg/tests/unit/api/controllers/v1/_test_accelerators.py;hb=refs/changes/57/626057/12#l68 | 03:49 |
| xinranwang | Ah, v1 api test uses TestList() class. I think it's better to do like https://review.opendev.org/#/c/626057/12/cyborg/tests/unit/api/controllers/v2/test_device_profiles.py | 03:49 |
| xinranwang | should we change v1, or just let it be : | 03:50 |
| Sundar | IMHO, no point fixing v1 code, let alone v1 tests | 03:51 |
| xinranwang | ok, sure. Just want make sure about this. | 03:52 |
| xinranwang | I have no other questions | 03:52 |
| Sundar | Anything else for today? | 03:53 |
| yikun | https://review.opendev.org/#/c/660548/ | 03:53 |
| yikun | one more topic from me, I will add the "aichip" type in this patch as we discussed in PTG before. | 03:53 |
| Sundar | Sure, yikun. | 03:53 |
| yikun | OK, thanks~ | 03:54 |
| Sundar | I can give a +2, if others don't need more time | 03:54 |
| yikun | OK, sure | 03:54 |
| yikun | I have no other topic then | 03:55 |
| Sundar | All done for today, then? | 03:56 |
| Sundar | Thank you all. Let's complete the specs by next week. Have a good day! | 03:57 |
| Sundar | #endmeeting | 03:57 |
| *** openstack changes topic to "Pending patches (Meeting topic: openstack-cyborg)" | 03:57 | |
| openstack | Meeting ended Wed Jul 3 03:57:14 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 03:57 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-07-03-03.04.html | 03:57 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-07-03-03.04.txt | 03:57 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-07-03-03.04.log.html | 03:57 |
| *** Biwei has quit IRC | 03:58 | |
| wangzhh | Thx Sundar, Bye all. | 03:59 |
| *** links has joined #openstack-cyborg | 04:15 | |
| *** Sundar has quit IRC | 04:23 | |
| *** links has quit IRC | 05:38 | |
| *** links has joined #openstack-cyborg | 05:46 | |
| *** jraju__ has joined #openstack-cyborg | 05:50 | |
| *** links has quit IRC | 05:51 | |
| *** wangzhh has quit IRC | 06:04 | |
| *** yuki_t has quit IRC | 06:06 | |
| openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Remove the old generic driver https://review.opendev.org/658986 | 06:22 |
| openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add the new generic driver https://review.opendev.org/658987 | 06:22 |
| openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add the fake driver https://review.opendev.org/665318 | 06:22 |
| *** Yumeng has quit IRC | 06:22 | |
| *** s_shogo has quit IRC | 06:38 | |
| openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Remove the old generic driver https://review.opendev.org/658986 | 07:18 |
| openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add the new generic driver https://review.opendev.org/658987 | 07:18 |
| openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add AICHIP and GENERIC type in DB and obj https://review.opendev.org/660548 | 07:18 |
| openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add Huawei Ascend driver https://review.opendev.org/660874 | 07:18 |
| *** helenafm has joined #openstack-cyborg | 07:21 | |
| *** xinranwang has quit IRC | 07:24 | |
| openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add Huawei Ascend driver https://review.opendev.org/660874 | 08:04 |
| openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add the fake driver https://review.opendev.org/665318 | 08:06 |
| openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add the fake driver https://review.opendev.org/665318 | 08:06 |
| openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add the fake driver https://review.opendev.org/665318 | 08:07 |
| *** links has joined #openstack-cyborg | 08:10 | |
| *** jraju__ has quit IRC | 08:11 | |
| *** Yumeng has joined #openstack-cyborg | 08:21 | |
| *** helenafm has quit IRC | 09:00 | |
| *** helenafm has joined #openstack-cyborg | 09:03 | |
| openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add Huawei Ascend driver https://review.opendev.org/660874 | 09:05 |
| *** helenafm has quit IRC | 09:15 | |
| openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Fix re defination of GPU https://review.opendev.org/660353 | 09:18 |
| *** helenafm has joined #openstack-cyborg | 09:20 | |
| *** helenafm has quit IRC | 09:34 | |
| *** helenafm has joined #openstack-cyborg | 09:41 | |
| *** Yumeng has quit IRC | 09:57 | |
| openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add Huawei Ascend driver https://review.opendev.org/660874 | 10:07 |
| openstackgerrit | Xinran WANG proposed openstack/cyborg master: Placement report https://review.opendev.org/659233 | 10:31 |
| openstackgerrit | Xinran WANG proposed openstack/cyborg master: Change attach_info, cpid_info from sting to a dict. https://review.opendev.org/668868 | 11:23 |
| *** jraju__ has joined #openstack-cyborg | 11:44 | |
| *** links has quit IRC | 11:45 | |
| *** jraju__ has quit IRC | 12:35 | |
| *** jraju__ has joined #openstack-cyborg | 12:42 | |
| *** jraju__ has quit IRC | 13:47 | |
| *** helenafm has quit IRC | 15:56 | |
| *** openstackgerrit has quit IRC | 17:04 | |
| *** jaypipes has quit IRC | 20:34 | |
| *** openstackgerrit has joined #openstack-cyborg | 21:31 | |
| openstackgerrit | Sundar Nadathur proposed openstack/cyborg-specs master: Specification for Cyborg APIs Version 2. https://review.opendev.org/658263 | 21:31 |
| openstackgerrit | Sundar Nadathur proposed openstack/cyborg-specs master: Specification for instance operations with accelerators. https://review.opendev.org/605237 | 22:07 |
| *** efried is now known as efried_pto | 22:45 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!