Thursday, 2021-02-04

*** xinranwang has joined #openstack-cyborg01:59
*** chenke has joined #openstack-cyborg02:00
xinranwanghi all02:01
swp20hi xinran02:01
chenkehi02:01
xinranwangbrin will be late02:04
xinranwangso let's start02:04
chenkeok02:04
xinranwang#startming02:04
xinranwang#startmeeting openstack_cyborg02:05
openstackMeeting started Thu Feb  4 02:05:16 2021 UTC and is due to finish in 60 minutes.  The chair is xinranwang. Information about MeetBot at http://wiki.debian.org/MeetBot.02:05
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.02:05
*** openstack changes topic to " (Meeting topic: openstack_cyborg)"02:05
openstackThe meeting name has been set to 'openstack_cyborg'02:05
xinranwang#topic rollcall02:05
*** openstack changes topic to "rollcall (Meeting topic: openstack_cyborg)"02:05
xinranwang#info xinranwang02:05
swp20#info swp2002:05
chenke#info chenke02:05
xinranwang#topic vgpu implementation02:06
*** openstack changes topic to "vgpu implementation (Meeting topic: openstack_cyborg)"02:06
xinranwangdo you have some thing want bring up about vgpu02:06
chenkeyumeng have a new work now.02:07
swp20nothing from myside02:07
swp20wait yument response02:07
swp20yumeng02:07
xinranwangyumeng has left zte now?02:08
chenkeyes.02:08
chenkethe day before yesterday.02:09
xinranwangoh.02:09
xinranwangso does anyone hand over the vgpu staff02:09
chenkemaybe recently, I have no time to update her patch because some office task. sorry.02:10
chenkebecause of another project task.02:10
xinranwangit's ok02:10
xinranwangis it possible that swp20 can update the patch according to the test report.  do you have time to do this?02:12
swp20ok i will try.02:13
xinranwangswp20:  thanks! we will continue review this. hope we can land it in this release. lol02:14
swp20np02:15
chenkecool.02:15
brinzhang_hi all02:15
xinranwanghi brinzhang_02:15
chenkehi brin.02:16
xinranwang#topic nic driver02:16
*** openstack changes topic to "nic driver (Meeting topic: openstack_cyborg)"02:16
brinzhang_^^ pls continue02:16
xinranwangthanks for your connent s on nic driver.  I have update it.  please check if I answered your question02:17
xinranwang*comment02:17
swp20i have CR+2 for nic driver02:17
chenkeI had check.02:17
chenkeabout the main code , I din't understand your comment.02:17
brinzhang_xinranwang: we have concern of the config of enable_drivers in https://review.opendev.org/c/openstack/cyborg/+/758942/7/cyborg.conf.sample#6202:17
brinzhang_if there is not intel nic that we can write a driver for it02:18
xinranwangI am not able to accese to gerrit now02:18
brinzhang_but if there is also a new nic for intel and it's not x710, how does it to config?02:19
brinzhang_xinranwang: nothing, you know what I said02:19
xinranwangit should be in another section. like [x810-static]02:20
xinranwangand nic driver will read this section according to the device id discovered02:21
chenkeshould we need register it again in cyborg/conf/devices file?02:21
xinranwangchenke: I dont think so02:24
brinzhang_xinranwang: get your mean, it's ok for me02:24
chenkejust config it in cyborg.conf is ok?02:24
brinzhang_and there is another question physical_device_mappings = physnet1:eth2;eth302:24
xinranwangwe can add x810-static in enabled-nic-types02:24
brinzhang_does we use ";" instead of ","02:24
brinzhang_s/does/do02:25
xinranwangyeah. I paste the neutron config format in my reply02:25
xinranwangI prefer to keep the same format as neutron02:26
brinzhang_IMO, we should use "," to config the options, in nova, cinder they are all use ','02:26
chenkeoh. your means we just add x810-static in enabled-nic-types, and then config [x810-static] section is ok, right?02:26
brinzhang_I think ";" is too old config02:26
xinranwangchenke: yes. and we can add x810-static in enabled-nic-types. it will register implictly02:26
brinzhang_We should follow the habits of most people02:27
xinranwangbrinzhang_:  I will check it02:27
chenkegood catch brin. ";" and ",",I didn't find it before.02:27
chenkexinranwang, seems “;”  need to ensure about it.02:28
xinranwangwhat if we have 2 mapping02:28
brinzhang_xinranwang: thanks02:28
brinzhang_another thing is the report tests, I think we should maintain it in the docs too. https://wiki.openstack.org/wiki/Cyborg/TestReport/IntelNic02:29
xinranwangeg GTPv1:eth3;eth202:29
brinzhang_because we dont have the CI to check the driver02:29
chenkeresource_provider_inventory_defaults = allocation_ratio:1.0,min_unit:1,step_size:1,reserved:002:29
chenkeI saw this config in neutron ovs.conf02:29
xinranwang GTPv1:eth3;eth2,GTPV2:eth0;eth102:30
xinranwangbrinzhang_: what docs02:30
brinzhang_such as the driver matrix02:31
brinzhang_https://docs.openstack.org/cyborg/latest/reference/support-matrix.html02:31
xinranwangwhat if we have  GTPv1:eth3;eth2,GTPV2:eth0;eth1.  you prefer GTPv1:eth3,eth2;GTPV2:eth0,eth1.right?02:32
brinzhang_what do you think? let maintain this int he driver matrix, is it ok?02:32
xinranwangbrinzhang_: driver matrix is ok for me02:33
chenkelet's first help xinran ensure the conf.02:33
xinranwangchenke:  yeah. thanks haha02:33
xinranwangi see neutron excluded use ; for 1:N mapping02:35
xinranwang*excluded_device02:35
xinranwangthat's why I? use ;02:35
xinranwangsorry  I use cellphone.  so many typos...02:36
brinzhang_I think that is an exception, I suggest using , to instead of ;02:37
xinranwangif we have 2 mapping, what seperator you prefer02:38
xinranwangGTPv1:eth3,eth2;GTPV2:eth0,eth1.right?02:38
brinzhang_yes02:38
xinranwangok02:38
brinzhang_chenke: is that your concern too?02:39
xinranwangit's ok for me02:39
chenkeYes.You remind me. I didn't see any sample about xinran's situation.02:39
chenkeIn my opinion. I think GTPv1:eth3,eth2;GTPV2:eth0,eth1 is ok.02:40
xinranwangnever mind02:40
xinranwanglol02:40
xinranwangjust want got an agreement. the change is small02:40
chenkeyes. we are a team. lol.02:41
brinzhang_yeah, this is the first thing need to be changed02:41
xinranwangso 2 things need improve. the config format and driver matrix doc02:41
brinzhang_another is the docs of the driver report "test result" add to the driver matrix02:42
brinzhang_xinranwang: yes02:42
brinzhang_chenke: do you have another issue need to be changed in next patch?02:42
xinranwangok. will update it by today.02:42
chenkejust one question about the main test02:43
brinzhang_xinranwang: thanks02:43
chenkehttps://review.opendev.org/c/openstack/cyborg/+/758942/8/cyborg/tests/unit/accelerator/drivers/nic/intel/prepare_test_data.py02:43
chenkethis main part code. If we didn't call it directly. I think this part willnot be execute.02:44
chenkexinranwang02:45
xinranwangit is for tox test02:46
xinranwanghow co it is not excute?02:46
xinranwang...  why it is not excute02:46
xinranwangit is for create a fake sysfs. because both fpga02:47
chenkeoh?  for tox test. I need to check02:47
xinranwangand nic driver use sysfs to discover. instead of lspci02:47
chenkeif __name__ == "__main__":02:48
chenkethis part I will  talk with you after meeting.02:49
xinranwangyes. sure02:49
xinranwang#topic AoB02:50
*** openstack changes topic to "AoB (Meeting topic: openstack_cyborg)"02:50
xinranwangdo you guys have anything to bring up?02:50
chenkenothing from my side.02:50
chenkewill +2 after your update it.02:51
xinranwanghah thanks chenke02:53
chenkeseems our meeting time is full enough.02:53
swp20nothing from myside02:53
xinranwangif nothing else, let's wrap up the meeting. pls keep discuss offline if any question02:54
xinranwangoh,  next week is the Chinese New year. so no meeting next week02:54
xinranwangalso the week after next week02:54
chenkecool02:55
xinranwangand happy Chinese New Year!!!!!!!02:55
chenkehahaha02:55
chenkeHappy Chinese New Year! Happy a new progress for us in 2021.02:56
xinranwanglol02:57
xinranwangsee you next year02:57
xinranwang#endmeeting02:57
*** openstack changes topic to "Pending patches (Meeting topic: openstack-cyborg)"02:57
openstackMeeting ended Thu Feb  4 02:57:45 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)02:57
openstackMinutes:        http://eavesdrop.openstack.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-02-04-02.05.html02:57
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-02-04-02.05.txt02:57
chenkebye02:57
openstackLog:            http://eavesdrop.openstack.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-02-04-02.05.log.html02:57
swp20Happy Chinese New Year!02:59
swp20bye02:59
*** links has joined #openstack-cyborg04:36
*** xinranwang has quit IRC05:39
*** chenke has quit IRC06:14
*** openstackgerrit has joined #openstack-cyborg08:45
openstackgerritXinran WANG proposed openstack/cyborg master: Add intel NIC driver  https://review.opendev.org/c/openstack/cyborg/+/75894208:45
openstackgerritXinran WANG proposed openstack/cyborg-specs master: Improve the nic driver spec according to current implementation.  https://review.opendev.org/c/openstack/cyborg-specs/+/77405809:59
*** songwenping_ has joined #openstack-cyborg12:51
*** swp20 has quit IRC12:54
*** songwenping__ has joined #openstack-cyborg14:01
*** songwenping_ has quit IRC14:05
*** Underknowledge has quit IRC14:17
*** Underknowledge1 has joined #openstack-cyborg14:17
*** Underknowledge1 is now known as Underknowledge14:18
*** openstack has joined #openstack-cyborg17:24
*** ChanServ sets mode: +o openstack17:24
*** songwenping_ has joined #openstack-cyborg18:12
*** songwenping__ has quit IRC18:14
*** efried has quit IRC23:38
*** efried has joined #openstack-cyborg23:38

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!