| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2021-02-04 | |||
| 02:21:10 | chenke | should we need register it again in cyborg/conf/devices file? | |
| 02:24:01 | xinranwang | chenke: I dont think so | |
| 02:24:28 | brinzhang_ | xinranwang: get your mean, it's ok for me | |
| 02:24:33 | chenke | just config it in cyborg.conf is ok? | |
| 02:24:36 | brinzhang_ | and there is another question physical_device_mappings = physnet1:eth2;eth3 | |
| 02:24:50 | xinranwang | we can add x810-static in enabled-nic-types | |
| 02:24:52 | brinzhang_ | does we use ";" instead of "," | |
| 02:25:02 | brinzhang_ | s/does/do | |
| 02:25:48 | xinranwang | yeah. I paste the neutron config format in my reply | |
| 02:26:11 | xinranwang | I prefer to keep the same format as neutron | |
| 02:26:23 | brinzhang_ | IMO, we should use "," to config the options, in nova, cinder they are all use ',' | |
| 02:26:34 | chenke | oh. your means we just add x810-static in enabled-nic-types, and then config [x810-static] section is ok, right? | |
| 02:26:41 | brinzhang_ | I think ";" is too old config | |
| 02:26:59 | xinranwang | chenke: yes. and we can add x810-static in enabled-nic-types. it will register implictly | |
| 02:27:28 | brinzhang_ | We should follow the habits of most people | |
| 02:27:35 | xinranwang | brinzhang_: I will check it | |
| 02:27:38 | chenke | good catch brin. ";" and ","，I didn't find it before. | |
| 02:28:20 | chenke | xinranwang, seems “;” need to ensure about it. | |
| 02:28:29 | xinranwang | what if we have 2 mapping | |
| 02:28:32 | brinzhang_ | xinranwang: thanks | |
| 02:29:01 | brinzhang_ | another thing is the report tests, I think we should maintain it in the docs too. https://wiki.openstack.org/wiki/Cyborg/TestReport/IntelNic | |
| 02:29:23 | brinzhang_ | because we dont have the CI to check the driver | |
| 02:29:23 | xinranwang | eg GTPv1:eth3;eth2 | |
| 02:29:28 | chenke | resource_provider_inventory_defaults = allocation_ratio:1.0,min_unit:1,step_size:1,reserved:0 | |
| 02:29:41 | chenke | I saw this config in neutron ovs.conf | |
| 02:30:07 | xinranwang | GTPv1:eth3;eth2，GTPV2：eth0;eth1 | |
| 02:30:43 | xinranwang | brinzhang_: what docs | |
| 02:31:00 | brinzhang_ | such as the driver matrix | |
| 02:31:59 | brinzhang_ | https://docs.openstack.org/cyborg/latest/reference/support-matrix.html | |
| 02:32:15 | xinranwang | what if we have GTPv1:eth3;eth2，GTPV2：eth0;eth1. you prefer GTPv1:eth3,eth2;GTPV2：eth0,eth1.right? | |
| 02:32:21 | brinzhang_ | what do you think? let maintain this int he driver matrix, is it ok? | |
| 02:33:16 | xinranwang | brinzhang_: driver matrix is ok for me | |
| 02:33:27 | chenke | let's first help xinran ensure the conf. | |
| 02:33:48 | xinranwang | chenke: yeah. thanks haha | |
| 02:35:09 | xinranwang | i see neutron excluded use ; for 1:N mapping | |
| 02:35:33 | xinranwang | *excluded_device | |
| 02:35:55 | xinranwang | that's why I? use ; | |
| 02:36:43 | xinranwang | sorry I use cellphone. so many typos... | |
| 02:37:46 | brinzhang_ | I think that is an exception, I suggest using , to instead of ; | |
| 02:38:32 | xinranwang | if we have 2 mapping, what seperator you prefer | |
| 02:38:49 | xinranwang | GTPv1:eth3,eth2;GTPV2：eth0,eth1.right? | |
| 02:38:55 | brinzhang_ | yes | |
| 02:38:58 | xinranwang | ok | |
| 02:39:09 | brinzhang_ | chenke: is that your concern too? | |
| 02:39:34 | xinranwang | it's ok for me | |
| 02:39:52 | chenke | Yes.You remind me. I didn't see any sample about xinran's situation. | |
| 02:40:12 | chenke | In my opinion. I think GTPv1:eth3,eth2;GTPV2：eth0,eth1 is ok. | |
| 02:40:15 | xinranwang | never mind | |
| 02:40:18 | xinranwang | lol | |
| 02:40:46 | xinranwang | just want got an agreement. the change is small | |
| 02:41:05 | chenke | yes. we are a team. lol. | |
| 02:41:21 | brinzhang_ | yeah, this is the first thing need to be changed | |
| 02:41:46 | xinranwang | so 2 things need improve. the config format and driver matrix doc | |
| 02:42:01 | brinzhang_ | another is the docs of the driver report "test result" add to the driver matrix | |
| 02:42:08 | brinzhang_ | xinranwang: yes | |
| 02:42:35 | brinzhang_ | chenke: do you have another issue need to be changed in next patch? | |
| 02:42:43 | xinranwang | ok. will update it by today. | |
| 02:43:02 | chenke | just one question about the main test | |
| 02:43:06 | brinzhang_ | xinranwang: thanks | |
| 02:43:47 | chenke | https://review.opendev.org/c/openstack/cyborg/+/758942/8/cyborg/tests/unit/accelerator/drivers/nic/intel/prepare_test_data.py | |
| 02:44:55 | chenke | this main part code. If we didn't call it directly. I think this part willnot be execute. | |
| 02:45:01 | chenke | xinranwang | |
| 02:46:07 | xinranwang | it is for tox test | |
| 02:46:40 | xinranwang | how co it is not excute? | |
| 02:46:56 | xinranwang | ... why it is not excute | |
| 02:47:34 | xinranwang | it is for create a fake sysfs. because both fpga | |
| 02:47:56 | chenke | oh? for tox test. I need to check | |
| 02:47:58 | xinranwang | and nic driver use sysfs to discover. instead of lspci | |
| 02:48:57 | chenke | if __name__ == "__main__": | |
| 02:49:27 | chenke | this part I will talk with you after meeting. | |
| 02:49:35 | xinranwang | yes. sure | |
| 02:50:17 | xinranwang | #topic AoB | |
| 02:50:40 | xinranwang | do you guys have anything to bring up? | |
| 02:50:54 | chenke | nothing from my side. | |
| 02:51:11 | chenke | will +2 after your update it. | |
| 02:53:03 | xinranwang | hah thanks chenke | |
| 02:53:30 | chenke | seems our meeting time is full enough. | |
| 02:53:46 | swp20 | nothing from myside | |
| 02:54:02 | xinranwang | if nothing else, let's wrap up the meeting. pls keep discuss offline if any question | |
| 02:54:39 | xinranwang | oh, next week is the Chinese New year. so no meeting next week | |
| 02:54:54 | xinranwang | also the week after next week | |
| 02:55:25 | chenke | cool | |
| 02:55:39 | xinranwang | and happy Chinese New Year!!!!!!! | |
| 02:55:47 | chenke | hahaha | |
| 02:56:15 | chenke | Happy Chinese New Year! Happy a new progress for us in 2021. | |
| 02:57:18 | xinranwang | lol | |
| 02:57:36 | xinranwang | see you next year | |
| 02:57:45 | xinranwang | #endmeeting | |
| 02:57:47 | openstack | Meeting ended Thu Feb 4 02:57:45 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 02:57:48 | openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-02-04-02.05.html | |
| 02:57:49 | openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-02-04-02.05.txt | |
| 02:57:50 | chenke | bye | |
| 02:57:51 | openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-02-04-02.05.log.html | |
| 02:59:04 | swp20 | Happy Chinese New Year! | |
| 02:59:07 | swp20 | bye | |
| 08:45:26 | openstackgerrit | Xinran WANG proposed openstack/cyborg master: Add intel NIC driver https://review.opendev.org/c/openstack/cyborg/+/758942 | |
| 09:59:12 | openstackgerrit | Xinran WANG proposed openstack/cyborg-specs master: Improve the nic driver spec according to current implementation. https://review.opendev.org/c/openstack/cyborg-specs/+/774058 | |
| #openstack-cyborg - 2021-02-07 | |||
| 08:45:39 | openstackgerrit | Merged openstack/cyborg master: Add intel NIC driver https://review.opendev.org/c/openstack/cyborg/+/758942 | |
| #openstack-cyborg - 2021-02-15 | |||
| 08:37:59 | openstackgerrit | Wenping Song proposed openstack/cyborg master: Add NVMe SSD driver https://review.opendev.org/c/openstack/cyborg/+/723310 | |
| #openstack-cyborg - 2021-02-16 | |||
| 02:13:19 | openstackgerrit | Wenping Song proposed openstack/cyborg master: support configuring multiple vgpu types in cyborg https://review.opendev.org/c/openstack/cyborg/+/764450 | |