| *** brinzhang_ has joined #openstack-cyborg | 00:32 | |
| *** brinzhang0 has quit IRC | 00:36 | |
| *** brinzhang_ has quit IRC | 01:31 | |
| *** brinzhang_ has joined #openstack-cyborg | 01:31 | |
| *** links has joined #openstack-cyborg | 02:29 | |
| *** efried has quit IRC | 02:57 | |
| *** Yumeng has joined #openstack-cyborg | 02:59 | |
| *** efried has joined #openstack-cyborg | 03:01 | |
| *** s_shogo has joined #openstack-cyborg | 03:02 | |
| Yumeng | hi xinranwang__, s_shogo swp20 brinzhang_ | 03:05 |
|---|---|---|
| brinzhang_ | hi Yumeng | 03:05 |
| swp20 | hi Yumeng | 03:05 |
| xinranwang__ | Hi all | 03:05 |
| *** chenke has joined #openstack-cyborg | 03:05 | |
| Yumeng | hi chenke | 03:06 |
| s_shogo | Hi all | 03:06 |
| chenke | hi all | 03:06 |
| swp20 | Hi all | 03:06 |
| chenke | hi yumeng. shogo. | 03:06 |
| chenke | hi wenping. | 03:06 |
| chenke | I leave some comments on your patches. | 03:06 |
| swp20 | yeah, i see. | 03:06 |
| Yumeng | s_shogo: I asked haibin to join our meeting today. he will join soon. you guys can discuss Program API things ：） | 03:07 |
| swp20 | thanks chenke. | 03:07 |
| s_shogo | Yumeng great! | 03:07 |
| Yumeng | ok. let's start | 03:07 |
| Yumeng | #startmeeting openstack-cyborg | 03:07 |
| openstack | Meeting started Thu Aug 27 03:07:58 2020 UTC and is due to finish in 60 minutes. The chair is Yumeng. Information about MeetBot at http://wiki.debian.org/MeetBot. | 03:07 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 03:08 |
| *** openstack changes topic to " (Meeting topic: openstack-cyborg)" | 03:08 | |
| openstack | The meeting name has been set to 'openstack_cyborg' | 03:08 |
| Yumeng | #topic Roll call | 03:08 |
| *** openstack changes topic to "Roll call (Meeting topic: openstack-cyborg)" | 03:08 | |
| Yumeng | #info Yumeng | 03:08 |
| swp20 | #info swp20 | 03:08 |
| s_shogo | #info s_shogo | 03:08 |
| xinranwang__ | #info xinranwang__ | 03:08 |
| brinzhang_ | #info brinzhang_ | 03:09 |
| *** shaohe_feng has joined #openstack-cyborg | 03:09 | |
| Yumeng | hi shaohe_feng | 03:10 |
| shaohe_feng | hi all | 03:10 |
| shaohe_feng | hi Yumeng, morning | 03:10 |
| Yumeng | cool, we have so many people today | 03:11 |
| brinzhang_ | Hi all, we should have a aggrement of the tempary report result of the 3rd-party driver | 03:11 |
| Yumeng | #topic Agenda | 03:11 |
| *** openstack changes topic to "Agenda (Meeting topic: openstack-cyborg)" | 03:11 | |
| Yumeng | brinzhang_: yes, that's one of the topic I want to mention | 03:12 |
| brinzhang_ | where to save the tempart results | 03:12 |
| brinzhang_ | yeah ^^ | 03:12 |
| *** haibin-huang has joined #openstack-cyborg | 03:12 | |
| Yumeng | as discussed in Intel QAT driver patch: https://review.opendev.org/#/c/725821/7//COMMIT_MSG | 03:14 |
| brinzhang_ | I am ok with the qat driver patch, and the inspur fpga driver patch, just with the temparory reulst | 03:14 |
| xinranwang__ | do we have a template wiki to show test result? | 03:14 |
| brinzhang_ | xinranwant_: may we should have to create a tempate for this | 03:15 |
| brinzhang_ | the etherpad is not save, because it can be modified by everyone | 03:16 |
| xinranwang__ | IMO, the format of test result is not a big deal. Just to show the driver works. | 03:16 |
| brinzhang_ | s/save/safe | 03:16 |
| chenke | #info chenke | 03:17 |
| swp20 | xinranwang__: your format of test result is good enough. | 03:18 |
| brinzhang_ | xinranwang_: not exactly all, we shoul keep the readable of the result, so the format shuold be beautiful ASAP | 03:18 |
| Yumeng | xinranwang__: about the template, I'm fine with the structure of your report. I think that make sense to showt the report results and provision results | 03:19 |
| xinranwang__ | I remember it was a website call paste.openstack.org, things like that. | 03:19 |
| Yumeng | xinranwang__: just one small question, we can add openstack accelerator device list to show the report result in cyborg side. | 03:19 |
| xinranwang__ | does anyone know that website> | 03:19 |
| brinzhang_ | xinranwang_: paste.openstack.org is also a temparory place too | 03:20 |
| brinzhang_ | I am trend to use wiki | 03:21 |
| xinranwang__ | brinzhang_: does everyone can modify it? | 03:21 |
| Yumeng | brinzhang_, xinranwang__: I also have the concern that etherpad and paste.openstack.org are temparory. anyobe can modify it | 03:21 |
| xinranwang__ | Yumeng: ok, sure | 03:21 |
| brinzhang_ | xinranwang_: no, but it will be lost with a long time | 03:22 |
| xinranwang__ | brinzhang_: ok, got it. | 03:22 |
| Yumeng | we can create a wiki page to record the driver report result and then add the link to the driver-support table on doc page https://docs.openstack.org/cyborg/latest/reference/support-matrix.html#driver-support | 03:22 |
| xinranwang__ | It seems a wiki page is the better one | 03:23 |
| Yumeng | what do you think? | 03:23 |
| brinzhang_ | we can use wiki, but how do we let the user or operator know the result exist in wiki? | 03:23 |
| xinranwang__ | We can mention this wiki page in cyborg doc | 03:23 |
| brinzhang_ | agree, and we can add a column as "temparory result", marked when it merged | 03:24 |
| shaohe_feng | the https://docs.openstack.org/cyborg/latest/reference/support-matrix.html#driver-support should add item to show the link | 03:24 |
| shaohe_feng | yes | 03:25 |
| Yumeng | yes, agree | 03:25 |
| brinzhang_ | such as, Added it in Victoria (Auguest, 2020) | 03:25 |
| Yumeng | yes, we can do that. | 03:26 |
| brinzhang_ | and past the wiki test result link | 03:26 |
| xinranwang__ | lol, like a biography for the driver | 03:26 |
| Yumeng | yep =_< | 03:27 |
| brinzhang_ | ok, let do it by this way, after meeting, I will send an email to the ML, that everyone should do, if they are not have the third-party CI | 03:27 |
| swp20 | aggree | 03:27 |
| Yumeng | brinzhang_: cool. | 03:28 |
| Yumeng | about the driver implementation, I have another quesion to discuss | 03:28 |
| Yumeng | VENDOR_MAPS format of GPU,FPGA are different, should we keep consistent? | 03:29 |
| Yumeng | FPGA is like this https://github.com/openstack/cyborg/blob/master/cyborg/accelerator/drivers/fpga/base.py#L23 | 03:29 |
| Yumeng | while GPU is like https://github.com/openstack/cyborg/blob/master/cyborg/accelerator/drivers/gpu/base.py#L26 | 03:29 |
| swp20 | we can merge them in one dict. | 03:30 |
| brinzhang_ | move this to a common file? | 03:31 |
| xinranwang__ | Yumeng: what do you mean by difference | 03:32 |
| brinzhang_ | VENDER_MAPS = ['GPU': {'0x8086': 'intel', ..}, 'FPGA': {'','', ...}] | 03:32 |
| brinzhang_ | how about like this? that we can maintain this in one place | 03:32 |
| xinranwang__ | Just wondering why we should do this. Is there any gap in current implementation? | 03:34 |
| brinzhang_ | not array, a dict. VENDER_MAPS = {'GPU': {'0x8086': 'intel', ..}, 'FPGA': {'','', ...}} | 03:34 |
| brinzhang_ | xinranwang_: I am not sure what Yumeng want todo, but from the maintain accelerator devices type and vendor, that we can merge this, because we can easy to know what devices that we can support | 03:36 |
| brinzhang_ | but as you said, is it make sense? It make sense to me. ^ | 03:37 |
| Yumeng | xinranwang__: initially, this is a downstream problem I found in baremetal support. ironic report Nvidia GPU vendor_id as hexadecimal int, while we accept vendor_id as string | 03:38 |
| *** wangzhh has joined #openstack-cyborg | 03:38 | |
| xinranwang__ | brinzhang_: https://github.com/openstack/cyborg/blob/cde5c3421d03722696269eb45ac148afd9838042/cyborg/common/constants.py#L90 we can this to show all supported resources. | 03:38 |
| Yumeng | here by difference I mean '0x8086' and '10de' | 03:38 |
| xinranwang__ | Yumeng: ok, that's the problem. | 03:38 |
| xinranwang__ | It comes from the discovery method. Intel drivers all discover devices from sysfs which contains the hex number. | 03:39 |
| Yumeng | seems different, I not yet look into the details. | 03:39 |
| Yumeng | but should we keep consistent, or just let this go and let any driver do their own way | 03:40 |
| xinranwang__ | AFAIK, the gpu driver's result comes from lspci's output. | 03:40 |
| swp20 | Inspur FPGA is '1db4' like GPU. | 03:41 |
| Yumeng | yes | 03:41 |
| swp20 | yeah also use lscpi command. | 03:41 |
| xinranwang__ | So the problem is that if driver use lspci, the output is now in hex format. We may need to translate it. | 03:42 |
| Yumeng | sounds good. | 03:42 |
| Yumeng | we can rethink it and make decision later. not in a hurry. | 03:43 |
| xinranwang__ | Yes, we should leave some time to program api discussion. | 03:43 |
| xinranwang__ | ;) | 03:43 |
| brinzhang_ | do we need to maintain a list for the supported device type? we know every product maybe need a driver | 03:44 |
| Yumeng | YES, We don't have much time left, I wanna leave some time for haibin-huang s_shogo, and shaohe_feng to discuss Program API | 03:44 |
| s_shogo | yes, that's welcome >> Program API discussion | 03:45 |
| shaohe_feng | hello | 03:45 |
| haibin-huang | ok | 03:45 |
| haibin-huang | hello shogo | 03:46 |
| s_shogo | hello haibin-huang | 03:46 |
| s_shogo | Is ok to begin discussion about the program API? | 03:46 |
| haibin-huang | yes | 03:47 |
| s_shogo | Thanks, | 03:47 |
| haibin-huang | so, what is your question? | 03:48 |
| s_shogo | haibin-huang , shaohe, xinran : Let me confirm, what type FPGA do you use for the test of programming API?(N3000?) | 03:48 |
| s_shogo | IMHO, in my concern, to use N3000 with Cyborg, that needs suited driver for N3000 and async program algorithm for that, before trying programming API. | 03:48 |
| haibin-huang | yes, N3000 | 03:49 |
| shaohe_feng | we need aysnc | 03:49 |
| shaohe_feng | s/aysnc/async | 03:50 |
| s_shogo | OK,t I got it. | 03:51 |
| s_shogo | As we discussed in previous IRC, I thinks that needs new driver and async, and that seems to be out of scope of this Program API patch.. | 03:52 |
| haibin-huang | I think we should consider two points, one is reboot fpga, one is get program status | 03:52 |
| s_shogo | Are there any plan for modifying driver and cyborg mechanism for N3000? | 03:52 |
| haibin-huang | we have implentation draft for it | 03:55 |
| haibin-huang | but, we don't know when we can push is upstream | 03:55 |
| haibin-huang | the Draft is not ready to push it upstream | 03:57 |
| s_shogo | Ok, I got it. | 03:58 |
| s_shogo | So that is better to be reviewed as the target. | 03:58 |
| s_shogo | Of course, the programming method for N3000 is important,that should be discussed as another patch and IRC. | 03:58 |
| s_shogo | I would like to settle out the program patch work as current scope, once. | 04:00 |
| shaohe_feng | OK, but can you add a new API for program status check, just add a UnimplemenationExcetpion(" unsport") ? | 04:01 |
| shaohe_feng | just define the API | 04:01 |
| shaohe_feng | we can add a the implementation later. | 04:02 |
| s_shogo | umm, to add a new API for program check is agree, | 04:03 |
| shaohe_feng | OK, thanks | 04:06 |
| s_shogo | Thanks, shaohe-feng :) | 04:08 |
| Yumeng | s_shogo，shaohe_feng：cool. since we don't have much time left for victoria, I think we should keep the scope of program API patch focused. and add other features later. | 04:09 |
| s_shogo | Yumeng Thanks, I agree that. | 04:09 |
| Yumeng | Thanks haibin-huang for coming to join the discussion. | 04:10 |
| s_shogo | Thanks haibin-huang, shaohe-feng | 04:10 |
| Yumeng | Thanks s_shogo and shaohe_feng, program API is not easy work. | 04:11 |
| *** shaohe_feng has quit IRC | 04:11 | |
| Yumeng | time is up for today. I will bring up the left topics(PTG things) on wechat where necessary | 04:12 |
| Yumeng | Thank you all for coming, and I'll see you all again next week | 04:13 |
| Yumeng | bye | 04:13 |
| swp20 | bye | 04:13 |
| brinzhang_ | bye | 04:13 |
| Yumeng | #endmeeting | 04:13 |
| *** openstack changes topic to "Pending patches (Meeting topic: openstack-cyborg)" | 04:13 | |
| openstack | Meeting ended Thu Aug 27 04:13:39 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 04:13 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-08-27-03.07.html | 04:13 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-08-27-03.07.txt | 04:13 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-08-27-03.07.log.html | 04:13 |
| s_shogo | bye | 04:14 |
| *** s_shogo has quit IRC | 04:14 | |
| *** haibin-huang has quit IRC | 05:22 | |
| *** chenke has quit IRC | 06:11 | |
| openstackgerrit | YumengBao proposed openstack/cyborg master: keep GPU driver's deployable_name unique https://review.opendev.org/748366 | 07:09 |
| *** brinzhang_ has quit IRC | 07:31 | |
| *** brinzhang_ has joined #openstack-cyborg | 07:32 | |
| *** brinzhang_ has quit IRC | 08:10 | |
| *** brinzhang_ has joined #openstack-cyborg | 08:11 | |
| openstackgerrit | Merged openstack/cyborg master: Optimize microversion support https://review.opendev.org/746282 | 08:59 |
| openstackgerrit | Wenping Song proposed openstack/cyborg master: Add instance project_id for arq patch https://review.opendev.org/738427 | 09:14 |
| openstackgerrit | Wenping Song proposed openstack/cyborg master: Add instance project_id for arq patch https://review.opendev.org/738427 | 09:20 |
| openstackgerrit | Xinran WANG proposed openstack/cyborg master: Add a new driver for Intel QAT card https://review.opendev.org/725821 | 09:51 |
| openstackgerrit | Xinran WANG proposed openstack/cyborg master: Add a new driver for Intel QAT card https://review.opendev.org/725821 | 09:56 |
| openstackgerrit | Xinran WANG proposed openstack/cyborg master: keep FPGA driver's deployable_name unique https://review.opendev.org/748408 | 10:37 |
| *** wangzhh has quit IRC | 10:48 | |
| *** Yumeng has quit IRC | 11:07 | |
| *** efried has quit IRC | 11:45 | |
| *** efried has joined #openstack-cyborg | 13:23 | |
| *** links has quit IRC | 14:53 | |
| openstackgerrit | zhoulinhui proposed openstack/cyborg master: Remove all usage of six library https://review.opendev.org/725928 | 15:46 |
| *** dustinc has quit IRC | 16:07 | |
| *** xinranwang__ has quit IRC | 17:54 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!