Monday, 2024-01-29

opendevreviewMerged openstack/neutron-lib master: Remove unused reno from test requirements  https://review.opendev.org/c/openstack/neutron-lib/+/90698407:40
opendevreviewyatin proposed openstack/neutron master: [DNM][functional] ovs-vswitchd debug log  https://review.opendev.org/c/openstack/neutron/+/90703707:40
opendevreviewRodolfo Alonso proposed openstack/neutron master: [UT] Change number of ``utcnow`` call number in ``test_update_timpestamp``  https://review.opendev.org/c/openstack/neutron/+/90704008:30
ralonsohykarel, slaweq ^^ please08:30
ralonsohthis is blocking the CI now08:31
ykarelralonsoh, ohkk08:33
opendevreviewyatin proposed openstack/neutron master: [DNM][functional] ovs-vswitchd debug log  https://review.opendev.org/c/openstack/neutron/+/90703708:37
slaweqralonsoh +2 +W10:08
ralonsohthanks both!10:14
opendevreviewMerged openstack/neutron master: [UT] Change number of ``utcnow`` call number in ``test_update_timpestamp``  https://review.opendev.org/c/openstack/neutron/+/90704010:27
opendevreviewFernando Royo proposed openstack/ovn-bgp-agent master: Add support to PF OVN LBs for NB Driver  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90550410:37
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Disable exposing remote_ips, when only the lrp prefix is sufficient  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90705713:32
opendevreviewMerged openstack/neutron stable/zed: Don't set port capabilities for OVS HW offloading  https://review.opendev.org/c/openstack/neutron/+/90427313:48
sahido/ i have question regarding the option dhcp_agents_per_network, (I have not yet checked the code) It seems that even if we set it to a value, a network is always scheduled on all the agents avail13:55
sahidany particular point that we have to take into account when using this option13:55
ralonsohsahid, if you have a network already scheduled and you change the value of "dhcp_agents_per_network" reducing the value, then you need to manually remove the redundancy 14:16
ralonsohhttps://rodolfo-alonso.com/reduce-dhcp-agent-redundancy14:16
sahidralonsoh: for that point i'm with you but it's not really what is happening for us. im my env I have many network and when i check the number of agents assigned for a given network I can see that is it assigned to all the agents (3) 14:23
sahidwhere I have configured it to be 2 with dhcp_agents_per_network14:23
sahidIf I do remove one agent to a given network14:23
sahidthen restart the agent that I have just removed to the given network14:24
sahidIt get back assigned to the agent14:24
sahidso 3 back...14:24
ralonsohthis is a server config option, not agent14:24
ralonsohyou need to restart the Neutron API14:24
sahidI don't touch the dhcp_agents_per_network14:25
ralonsohsorry, what?14:25
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Disable exposing remote_ips, when only the lrp prefix is sufficient  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90705714:25
sahid:-) I will try to be more clear sorry for that14:26
sahidIn our env dhcp_agents_per_network is configured to 214:26
sahidwhen I pick a network and list the dhcp agents attached I can see 314:26
sahidwhen I remove a dhcp agent from that network14:26
sahidthat work well I can see 2 agent for the network14:27
sahidnow if I restart the agent that I just have removed14:27
sahidthe agent is re attached to the network14:27
sahidand when I list I can see 3 agents on that network14:28
ralonsohso you never changed this value? was always 2?14:28
sahidIn that env I guess I can confirm it always has been 214:29
ralonsohsahid, so check this value in all your running Neutron API and try restarting them14:30
sahidbut give me a moment because you raised an interesting point I think we set this option in the dhcp agents where you seem to say we should set it in neutron server14:30
sahidralonsoh: yes looks like something around14:31
ralonsohyes, this is only a Neutron server config option14:31
ralonsohnot agent14:31
sahid++ i will check that14:31
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Add feature to check if SNAT disabled before exposing tenant networks  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90706814:38
sahidwe should have it in [DEFAULT], right? no need of having a specific network_scheduler_driver? (this option is not set) 14:39
sahidralonsoh: ^ (a gentle ping)14:40
ralonsohone sec (in a meeting)14:40
sahidsure sorry14:41
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Disable exposing remote_ips, when only the lrp prefix is sufficient  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90705714:41
sahidhum I did not get all the code but like that it seems that the option is not used when using routed provider14:51
sahidhttps://github.com/openstack/neutron/blob/master/neutron/scheduler/dhcp_agent_scheduler.py#L7914:51
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Re-use new methods and make code cleaner  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90707314:52
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Add feature to check if SNAT disabled before exposing tenant networks  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90706814:57
sahidhum actually that does not look to be in relation...14:58
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Update NB driver to re-use new methods and make code cleaner  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90707315:03
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Check for networks on router port in match_fn  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90707515:14
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Check for networks on router port in match_fn  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90707515:17
haleybralonsoh: i guess you didn't see https://review.opendev.org/c/openstack/neutron/+/906918 :-/15:18
ralonsohhaleyb, ups... sorry! I started today this morning with this bug without checking gerrit. My bad, sorry15:22
haleybnp, i will rebase15:23
ralonsohsahid, is is under [DEFAULT]15:23
ralonsohhaleyb, if you rebase, you can remove my code. Yours is going in a different direction15:25
haleybralonsoh: i just rebased and running tests... i think we both found where the issue was in context() 15:26
ralonsohyes15:27
opendevreviewBrian Haley proposed openstack/neutron master: Fix test_update_timestamp unit test failure  https://review.opendev.org/c/openstack/neutron/+/90691815:27
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Add feature to check if SNAT disabled before exposing tenant networks  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90706816:28
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Fix event handling for LSP and prefer the options.requested-chassis info  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90611116:33
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Update NB driver to re-use new methods and make code cleaner  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90707316:40
opendevreviewMerged openstack/neutron master: Set trunk parent port as access port in ovs to avoid loop  https://review.opendev.org/c/openstack/neutron/+/90512517:33
opendevreviewMerged openstack/neutron stable/2023.2: Make get_ports RPC method common for the DHCP and Metadata agent  https://review.opendev.org/c/openstack/neutron/+/90590617:58
opendevreviewMerged openstack/neutron stable/2023.1: Make get_ports RPC method common for the DHCP and Metadata agent  https://review.opendev.org/c/openstack/neutron/+/90629617:58
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2023.2: Set trunk parent port as access port in ovs to avoid loop  https://review.opendev.org/c/openstack/neutron/+/90714219:58
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2023.1: Set trunk parent port as access port in ovs to avoid loop  https://review.opendev.org/c/openstack/neutron/+/90714319:58
opendevreviewRodolfo Alonso proposed openstack/neutron stable/zed: Set trunk parent port as access port in ovs to avoid loop  https://review.opendev.org/c/openstack/neutron/+/90714419:58
opendevreviewMerged openstack/neutron master: [ovn] AZs distribution in L3 port scheduler  https://review.opendev.org/c/openstack/neutron/+/89260421:23

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