opendevreview | Merged openstack/neutron-lib master: Remove unused reno from test requirements https://review.opendev.org/c/openstack/neutron-lib/+/906984 | 07:40 |
---|---|---|
opendevreview | yatin proposed openstack/neutron master: [DNM][functional] ovs-vswitchd debug log https://review.opendev.org/c/openstack/neutron/+/907037 | 07:40 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [UT] Change number of ``utcnow`` call number in ``test_update_timpestamp`` https://review.opendev.org/c/openstack/neutron/+/907040 | 08:30 |
ralonsoh | ykarel, slaweq ^^ please | 08:30 |
ralonsoh | this is blocking the CI now | 08:31 |
ykarel | ralonsoh, ohkk | 08:33 |
opendevreview | yatin proposed openstack/neutron master: [DNM][functional] ovs-vswitchd debug log https://review.opendev.org/c/openstack/neutron/+/907037 | 08:37 |
slaweq | ralonsoh +2 +W | 10:08 |
ralonsoh | thanks both! | 10:14 |
opendevreview | Merged openstack/neutron master: [UT] Change number of ``utcnow`` call number in ``test_update_timpestamp`` https://review.opendev.org/c/openstack/neutron/+/907040 | 10:27 |
opendevreview | Fernando 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/+/905504 | 10:37 |
opendevreview | Michel 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/+/907057 | 13:32 |
opendevreview | Merged openstack/neutron stable/zed: Don't set port capabilities for OVS HW offloading https://review.opendev.org/c/openstack/neutron/+/904273 | 13:48 |
sahid | o/ 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 avail | 13:55 |
sahid | any particular point that we have to take into account when using this option | 13:55 |
ralonsoh | sahid, 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 |
ralonsoh | https://rodolfo-alonso.com/reduce-dhcp-agent-redundancy | 14:16 |
sahid | ralonsoh: 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 |
sahid | where I have configured it to be 2 with dhcp_agents_per_network | 14:23 |
sahid | If I do remove one agent to a given network | 14:23 |
sahid | then restart the agent that I have just removed to the given network | 14:24 |
sahid | It get back assigned to the agent | 14:24 |
sahid | so 3 back... | 14:24 |
ralonsoh | this is a server config option, not agent | 14:24 |
ralonsoh | you need to restart the Neutron API | 14:24 |
sahid | I don't touch the dhcp_agents_per_network | 14:25 |
ralonsoh | sorry, what? | 14:25 |
opendevreview | Michel 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/+/907057 | 14:25 |
sahid | :-) I will try to be more clear sorry for that | 14:26 |
sahid | In our env dhcp_agents_per_network is configured to 2 | 14:26 |
sahid | when I pick a network and list the dhcp agents attached I can see 3 | 14:26 |
sahid | when I remove a dhcp agent from that network | 14:26 |
sahid | that work well I can see 2 agent for the network | 14:27 |
sahid | now if I restart the agent that I just have removed | 14:27 |
sahid | the agent is re attached to the network | 14:27 |
sahid | and when I list I can see 3 agents on that network | 14:28 |
ralonsoh | so you never changed this value? was always 2? | 14:28 |
sahid | In that env I guess I can confirm it always has been 2 | 14:29 |
ralonsoh | sahid, so check this value in all your running Neutron API and try restarting them | 14:30 |
sahid | but 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 server | 14:30 |
sahid | ralonsoh: yes looks like something around | 14:31 |
ralonsoh | yes, this is only a Neutron server config option | 14:31 |
ralonsoh | not agent | 14:31 |
sahid | ++ i will check that | 14:31 |
opendevreview | Michel 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/+/907068 | 14:38 |
sahid | we should have it in [DEFAULT], right? no need of having a specific network_scheduler_driver? (this option is not set) | 14:39 |
sahid | ralonsoh: ^ (a gentle ping) | 14:40 |
ralonsoh | one sec (in a meeting) | 14:40 |
sahid | sure sorry | 14:41 |
opendevreview | Michel 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/+/907057 | 14:41 |
sahid | hum I did not get all the code but like that it seems that the option is not used when using routed provider | 14:51 |
sahid | https://github.com/openstack/neutron/blob/master/neutron/scheduler/dhcp_agent_scheduler.py#L79 | 14:51 |
opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: Re-use new methods and make code cleaner https://review.opendev.org/c/openstack/ovn-bgp-agent/+/907073 | 14:52 |
opendevreview | Michel 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/+/907068 | 14:57 |
sahid | hum actually that does not look to be in relation... | 14:58 |
opendevreview | Michel 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/+/907073 | 15:03 |
opendevreview | Michel 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/+/907075 | 15:14 |
opendevreview | Michel 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/+/907075 | 15:17 |
haleyb | ralonsoh: i guess you didn't see https://review.opendev.org/c/openstack/neutron/+/906918 :-/ | 15:18 |
ralonsoh | haleyb, ups... sorry! I started today this morning with this bug without checking gerrit. My bad, sorry | 15:22 |
haleyb | np, i will rebase | 15:23 |
ralonsoh | sahid, is is under [DEFAULT] | 15:23 |
ralonsoh | haleyb, if you rebase, you can remove my code. Yours is going in a different direction | 15:25 |
haleyb | ralonsoh: i just rebased and running tests... i think we both found where the issue was in context() | 15:26 |
ralonsoh | yes | 15:27 |
opendevreview | Brian Haley proposed openstack/neutron master: Fix test_update_timestamp unit test failure https://review.opendev.org/c/openstack/neutron/+/906918 | 15:27 |
opendevreview | Michel 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/+/907068 | 16:28 |
opendevreview | Michel 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/+/906111 | 16:33 |
opendevreview | Michel 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/+/907073 | 16:40 |
opendevreview | Merged openstack/neutron master: Set trunk parent port as access port in ovs to avoid loop https://review.opendev.org/c/openstack/neutron/+/905125 | 17:33 |
opendevreview | Merged openstack/neutron stable/2023.2: Make get_ports RPC method common for the DHCP and Metadata agent https://review.opendev.org/c/openstack/neutron/+/905906 | 17:58 |
opendevreview | Merged openstack/neutron stable/2023.1: Make get_ports RPC method common for the DHCP and Metadata agent https://review.opendev.org/c/openstack/neutron/+/906296 | 17:58 |
opendevreview | Rodolfo 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/+/907142 | 19:58 |
opendevreview | Rodolfo 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/+/907143 | 19:58 |
opendevreview | Rodolfo 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/+/907144 | 19:58 |
opendevreview | Merged openstack/neutron master: [ovn] AZs distribution in L3 port scheduler https://review.opendev.org/c/openstack/neutron/+/892604 | 21:23 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!