| opendevreview | ZhouHeng proposed openstack/neutron master: [ovn]Change LogicalSwitchPortUpdateUpEvent old conditions https://review.opendev.org/c/openstack/neutron/+/851997 | 00:54 |
|---|---|---|
| opendevreview | Merged openstack/neutron master: Fix return value in BaseScheduler class https://review.opendev.org/c/openstack/neutron/+/851800 | 01:05 |
| *** tweining|off is now known as tweining | 06:51 | |
| sahid | o/ | 07:00 |
| *** amoralej|off is now known as amoralej | 07:05 | |
| opendevreview | Weronika Sikora proposed openstack/neutron master: Avoid unnecessary db calls with l2pop https://review.opendev.org/c/openstack/neutron/+/852089 | 07:53 |
| opendevreview | ayenachew molla proposed openstack/neutron-tempest-plugin master: Delete tcp rule from a security group https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/848268 | 09:44 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DNM] [OVN][QoS] Add minimum bandwidth rule support to ML2/OVN https://review.opendev.org/c/openstack/neutron/+/842292 | 09:51 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: [ovn]Change LogicalSwitchPortUpdateUpEvent old conditions https://review.opendev.org/c/openstack/neutron/+/852130 | 09:59 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [ovn]Change LogicalSwitchPortUpdateUpEvent old conditions https://review.opendev.org/c/openstack/neutron/+/852131 | 10:00 |
| opendevreview | Merged openstack/neutron stable/yoga: Implement specific tracked resource count method per quota driver https://review.opendev.org/c/openstack/neutron/+/851915 | 10:15 |
| opendevreview | Merged openstack/neutron stable/xena: Implement specific tracked resource count method per quota driver https://review.opendev.org/c/openstack/neutron/+/851918 | 10:15 |
| opendevreview | Merged openstack/neutron stable/train: Add workaround for eventlet.greendns bug https://review.opendev.org/c/openstack/neutron/+/851537 | 10:15 |
| opendevreview | Merged openstack/neutron-dynamic-routing master: Fix missing functions in StaticScheduler https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/849178 | 10:41 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: [ovn]Change LogicalSwitchPortUpdateUpEvent old conditions https://review.opendev.org/c/openstack/neutron/+/852155 | 10:50 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/victoria: [ovn]Change LogicalSwitchPortUpdateUpEvent old conditions https://review.opendev.org/c/openstack/neutron/+/852156 | 10:51 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/ussuri: [ovn]Change LogicalSwitchPortUpdateUpEvent old conditions https://review.opendev.org/c/openstack/neutron/+/852157 | 10:51 |
| *** amoralej is now known as amoralej|lunch | 11:06 | |
| opendevreview | Merged openstack/neutron stable/ussuri: Port update will trigger less notifications to the DHCP agents https://review.opendev.org/c/openstack/neutron/+/851389 | 12:16 |
| *** elvira2 is now known as elvira | 12:30 | |
| *** amoralej|lunch is now known as amoralej | 12:30 | |
| opendevreview | Merged openstack/networking-bagpipe master: Remove "distutils" library https://review.opendev.org/c/openstack/networking-bagpipe/+/842369 | 12:33 |
| *** dasm|off is now known as dasm | 13:15 | |
| opendevreview | Merged openstack/neutron stable/train: [stable/train] Convert neutron-tempest-postgres-full to zuul v3 https://review.opendev.org/c/openstack/neutron/+/851512 | 14:25 |
| *** gibi is now known as gibi_pto | 15:37 | |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: db: add segment_index to the unique constraint https://review.opendev.org/c/openstack/neutron/+/841459 | 15:39 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: segment: enable multisegments support for host https://review.opendev.org/c/openstack/neutron/+/841838 | 15:39 |
| *** amoralej is now known as amoralej|off | 15:56 | |
| spatel | ralonsoh or anyone else available, this is where i stuck in my ovn-bgp-agent EVPN mode testing - https://paste.opendev.org/show/bLPjGNButUU2x3umEepp/ | 16:04 |
| spatel | following this doc - https://ltomasbo.wordpress.com/2021/06/25/openstack-networking-with-evpn/ | 16:04 |
| spatel | Do i need to add this by hand or agent will do ( ovs-vsctl add-port br-ex vrf-1001 ) ? | 16:05 |
| ralonsoh | spatel, I don't even know where this repo is hosted | 16:06 |
| ralonsoh | where can I find this code? | 16:06 |
| ralonsoh | I think you should ping Luis Tombaso | 16:06 |
| spatel | https://opendev.org/x/ovn-bgp-agent | 16:07 |
| spatel | This is the code of EVPN mode driver - https://opendev.org/x/ovn-bgp-agent/src/branch/master/ovn_bgp_agent/drivers/openstack/ovn_evpn_driver.py | 16:08 |
| ralonsoh | spatel, do you see in your logs this | 16:09 |
| ralonsoh | LOG.debug('Either "%s" or "%s" were not found or have an ' | 16:09 |
| ralonsoh | 'invalid value in the port %s ' | 16:09 |
| ralonsoh | 'external_ids %s', constants.OVN_EVPN_VNI_EXT_ID_KEY, | 16:09 |
| ralonsoh | constants.OVN_EVPN_AS_EXT_ID_KEY, port.logical_port, | 16:09 |
| ralonsoh | port.external_ids) | 16:09 |
| ralonsoh | yes, first line | 16:10 |
| spatel | Yes i can that in logs | 16:10 |
| ralonsoh | you didn't set "neutron_bgpvpn:as" | 16:10 |
| ralonsoh | in external_ids | 16:10 |
| ralonsoh | https://opendev.org/x/ovn-bgp-agent/src/branch/master/ovn_bgp_agent/drivers/openstack/utils/ovn.py#L251-L263 | 16:11 |
| spatel | hmm | 16:13 |
| spatel | i didn't set anything in external_ids | 16:13 |
| spatel | i am following doc and there are no instruction related that | 16:14 |
| ralonsoh | spatel, you did | 16:14 |
| ralonsoh | # At OVN NB DB | 16:14 |
| ralonsoh | # ovn-nbctl set logical_switch_port 5bc97046-6483-4e5a-98db-0e616acfac07 external_ids:"neutron_bgpvpn\:vni"=1001 | 16:14 |
| ralonsoh | # ovn-nbctl set logical_switch_port 64e99514-6e13-43c9-a11f-7861bf6f75a4 external_ids:"neutron_bgpvpn\:vni"=1001 | 16:14 |
| ralonsoh | I know there is nothing in the doc | 16:14 |
| ralonsoh | but please read the code | 16:14 |
| ralonsoh | https://opendev.org/x/ovn-bgp-agent/src/branch/master/ovn_bgp_agent/drivers/openstack/utils/ovn.py#L251-L263 | 16:14 |
| spatel | ohhh i thought you are talking about ovs .. sorry | 16:14 |
| spatel | I did set vni in extranal_ids | 16:14 |
| spatel | I only set neutron_bgpvpn:vni"="2001" so code saying i should add neutron_bgpvpn:as too.. hmm | 16:18 |
| spatel | Logs is saying Either "neutron_bgpvpn:vni" or "neutron_bgpvpn:as" so i thought it just required either vni or AS | 16:18 |
| spatel | i can set AS and see how it goes | 16:19 |
| spatel | ralonsoh that works... i can see it created vxlan-2001 interface and added vm ip in bgp expose | 16:23 |
| ralonsoh | perfect then | 16:23 |
| spatel | ralonsoh thank you for second eyes on that.. sorry i am not very expert in python coding so i miss understood that block :( | 16:26 |
| *** dasm is now known as dasm|off | 22:11 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!