| *** liuxie is now known as liushy | 03:04 | |
| slaweq | haleyb: hi, just FYI, together with elvira1 we have RFE to discuss: https://bugs.launchpad.net/neutron/+bug/2138746 - can it be added to the drivers meeting agenda for this or next week? | 08:29 |
|---|---|---|
| ralonsoh | lajoskatona, elvira hi! I have a question about https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/973626/10/zuul.d/master_jobs.yaml#363 | 09:35 |
| ralonsoh | why taas should be running only in OVS? | 09:36 |
| lajoskatona | ralonsoh: tap-services and tap-flows are working only with OVS currently and SRIOV | 09:45 |
| ralonsoh | lajoskatona, but I don't understand, we added support for ovn | 09:48 |
| ralonsoh | https://github.com/openstack/tap-as-a-service/commit/a2523e1e29f8f0142fa6e13eeda69af3e9cc59b5 | 09:48 |
| lajoskatona | ralonsoh: only for tap-mirrors which works both with native OVS and OVN, and now elvira extended the OVN part with the both direction which was added alter to the OVN core itself | 10:00 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Enable tap-as-a-service OVN extension https://review.opendev.org/c/openstack/neutron/+/973920 | 10:04 |
| ralonsoh | lajoskatona, but there are specific tests for this | 10:04 |
| ralonsoh | I don't know why TestTaaSTrafficScenarios shouldn't be executed | 10:04 |
| lajoskatona | ralonsoh: the naming of the scenario modules not that straight forward, scenario/test_taas.py i.e. has test for sriov, test_traffic_impact.py for tap_services and tap_flows on OVS, and test_tap_mirror is for tap_mirrors executed on OVS and OVN | 10:16 |
| lajoskatona | ralonsoh: at least that is my memory, perhaps I have to revisit these tests and jobs for them | 10:16 |
| ralonsoh | ooook, I think that deserves at least a note in the OVS taas extension | 10:17 |
| ralonsoh | just to be aware of this | 10:17 |
| ralonsoh | lajoskatona, can you push this patch for n-t-p? just to have this info | 10:18 |
| ralonsoh | elvira, ^^ | 10:18 |
| ralonsoh | I'll +2 your patch now | 10:18 |
| opendevreview | Merged openstack/neutron master: Bump oslo.config to 10.2.0 https://review.opendev.org/c/openstack/neutron/+/972491 | 10:25 |
| opendevreview | Merged openstack/neutron master: Fix ``LogicalRouterPortEvent`` when a LR is deleted https://review.opendev.org/c/openstack/neutron/+/973584 | 10:25 |
| elvira | Hi, just read all the conversation. Thanks a lot for the inputs! is there anything I should change from the patches? I already removed taas from enabled extensions in neutron ovn | 10:28 |
| elvira | I think maybe the post-config in ovn job is not needed anymore since I guess taas tests will be skipped anyway in OVN, not sure what you think ralonsoh | 10:30 |
| ralonsoh | elvira, maybe it could be nice to have a comment in the taas plugin in the OVS job | 10:33 |
| ralonsoh | in another patch | 10:33 |
| elvira | ralonsoh: to note that the post_config is needed for that? or to say that we only support taas extension in ML2/OVS? | 10:35 |
| ralonsoh | the last, that taas is only for OVSA | 10:36 |
| ralonsoh | OVS* | 10:36 |
| elvira | ok! will do | 10:36 |
| elvira | I will also remove the post-config in OVN. I can always put it back if something fails but I think that was failing only on taas tests iiuc | 10:37 |
| wilkmarcin | ralonsoh: I'd appreciate it if you shared your comments in https://review.opendev.org/c/openstack/neutron/+/971876, whenever you have time. Thank you in advance | 10:39 |
| ralonsoh | wilkmarcin, let me check that today. I would also want terry wilson to check it too | 10:40 |
| wilkmarcin | Thank you, ralonsoh | 10:42 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron-tempest-plugin master: Add both direction tap-as-a-service scenario test https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/973626 | 11:17 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron-tempest-plugin master: Add comment highlighting taas is ML2/OVS only https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/974040 | 11:19 |
| opendevreview | Maor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Fix allows multi value configuration options https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/974041 | 11:28 |
| opendevreview | Alexey Stupnikov proposed openstack/neutron master: Use RFC 5398 reserved AS numbers in Neutron documentation https://review.opendev.org/c/openstack/neutron/+/974042 | 11:28 |
| *** ralonsoh_ is now known as ralonsoh | 12:26 | |
| haleyb | slaweq, elvira: i will have to read the docs on PVLAN but i think you can add to this week drivers agenda, i might be out next friday | 14:49 |
| haleyb | slaweq: and while i have your attention can you look at https://review.opendev.org/c/openstack/neutron/+/971628 - neutron-lib bump | 14:50 |
| haleyb | or lajoskatona ^^ | 14:50 |
| lajoskatona | haleyb: cool, I wf+1-ed it | 14:54 |
| opendevreview | Merged x/whitebox-neutron-tempest-plugin master: Fix allows multi value configuration options https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/974041 | 15:05 |
| ralonsoh | haleyb, hello! Now in the correct channel | 15:09 |
| ralonsoh | this is about https://review.opendev.org/c/openstack/neutron/+/973866 | 15:09 |
| ralonsoh | You are right, this is the fix that solves the problem in the driver | 15:09 |
| ralonsoh | but you know sometimes these fixes take sometime to land in some D/S repositories | 15:10 |
| haleyb | ralonsoh: ah yes, we had tracked that down a few months back during a fire drill with a customer | 15:10 |
| ralonsoh | and it could happen that the customer doesn't have the most updated drivers | 15:10 |
| ralonsoh | so this is why I'm suggesting this patch | 15:10 |
| haleyb | ralonsoh: the reason we didn't go with a neutron change is there are other places in the code that just always poke the interface and don't check state | 15:11 |
| ralonsoh | I can add a TODO to remove it in some releases | 15:11 |
| ralonsoh | haleyb, but for SR-IOV this is the only method that changes the link state | 15:11 |
| ralonsoh | and nova-cpu is not doing it | 15:11 |
| haleyb | plus we found the offending commit from intel | 15:11 |
| ralonsoh | haleyb, so you don't think this should go into the neutron repo? | 15:12 |
| ralonsoh | even with a TODO to remove it? | 15:12 |
| slaweq | haleyb sure, I will add it to the agenda then | 15:14 |
| slaweq | and sorry for not checking earlier this patch but I was in the meeting | 15:14 |
| slaweq | and lajoskatona already approved it :) | 15:14 |
| slaweq | haleyb are you still using https://wiki.openstack.org/wiki/Meetings/NeutronDrivers for the drivers meeting? | 15:15 |
| haleyb | ralonsoh: i'm on the fence since i was involved in the triage of the driver, i'll explain... | 15:15 |
| haleyb | there was a change to the i40e driver to make sure things were (re)initialized properly, and while it broke this case, it seemed that not blindly setting link UP might potentially cause an issue in the driver | 15:18 |
| haleyb | we worked with intel on the fix and verified in a partner cloud | 15:18 |
| haleyb | i think the neutron change would have worked too, but since the i40e driver was seemingly the only one broken we went with the targeted change | 15:19 |
| haleyb | plus we could release the kernel quicker | 15:19 |
| ralonsoh | haleyb, ok, I'll push this patch only in our D/S repos | 15:19 |
| ralonsoh | I'll abandon it | 15:19 |
| haleyb | ralonsoh: ack | 15:22 |
| slaweq | ok, I've added it to the On Demand agenda in https://wiki.openstack.org/wiki/Meetings/NeutronDrivers#Agenda | 15:28 |
| slaweq | thx haleyb | 15:28 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron-tempest-plugin master: Add comment highlighting taas is ML2/OVS only https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/974040 | 15:31 |
| opendevreview | sean mooney proposed openstack/os-vif master: Add TAP device pre-creation support for OVS/OVN https://review.opendev.org/c/openstack/os-vif/+/971231 | 16:08 |
| opendevreview | Merged openstack/neutron master: Bump neutron-lib to 3.23.0 https://review.opendev.org/c/openstack/neutron/+/971628 | 17:04 |
| opendevreview | Merged openstack/neutron master: bgp: Detect BGP bridge creation https://review.opendev.org/c/openstack/neutron/+/964425 | 18:15 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Keep bridge mappings consistent https://review.opendev.org/c/openstack/neutron/+/964429 | 18:17 |
| opendevreview | Lajos Katona proposed openstack/neutron master: [eventlet-removal] Handle systemctl stop for DHCP agent https://review.opendev.org/c/openstack/neutron/+/967902 | 18:53 |
| opendevreview | Lajos Katona proposed openstack/neutron master: WIP: [eventlet]: Make ovs-agent stop again https://review.opendev.org/c/openstack/neutron/+/967105 | 18:53 |
| opendevreview | Brian Haley proposed openstack/neutron master: Use project_id in HANetworkConcurrentDeletion() https://review.opendev.org/c/openstack/neutron/+/974247 | 20:13 |
| opendevreview | Brian Haley proposed openstack/neutron-lib master: Remove tenant_id from HANetworkConcurrentDeletion() https://review.opendev.org/c/openstack/neutron-lib/+/974248 | 20:16 |
| opendevreview | Brian Haley proposed openstack/neutron master: Use project_id key in port plugin code https://review.opendev.org/c/openstack/neutron/+/972982 | 20:40 |
| opendevreview | Brian Haley proposed openstack/neutron master: Stop using tenant_* in context objects https://review.opendev.org/c/openstack/neutron/+/974253 | 21:03 |
| cardoe | TheJulia, slaweq, haleyb: The PVLAN stuff actually layers really nicely/easily with the VXLAN stuff that TheJulia and I have been working on. Because the private VLAN or Vi is just a lower segment to the Vp which is the upper segment. | 21:58 |
| cardoe | Cisco's own docs reference each other. | 22:00 |
| cardoe | The spec and the docs that I submitted can cover PVLAN with a simple sed. | 22:01 |
| cardoe | The ML2 driver that I submitted to the neutron repo and TheJulia has now moved over to networking-baremetal can cover PVLAN by changing the config. | 22:02 |
| cardoe | We'd love to work with you folks. | 22:05 |
| opendevreview | Brian Haley proposed openstack/neutron-lib master: Remove tenant_id from HANetworkConcurrentDeletion() https://review.opendev.org/c/openstack/neutron-lib/+/974248 | 22:29 |
| haleyb | cardoe: great, can you sync with elvira and/or slaweq on it? they were going to present at drivers meeting this week | 22:29 |
| cardoe | Yep. | 22:32 |
| cardoe | We can also update that data center networking doc to include an example of this as well. | 22:32 |
| cardoe | Unless what you're really looking for is doing this with VM then it wouldn't make sense for that doc. | 22:32 |
| * haleyb shrugs | 22:33 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!