| ralonsoh | hi folks, I'm fast approving https://review.opendev.org/q/I9c62d429bb819336da05055fecd08e3816986bf8 in order to unblock the stable CI gate | 08:15 |
|---|---|---|
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Fix jobs after tox4 upgrade https://review.opendev.org/c/openstack/ovn-octavia-provider/+/868476 | 08:25 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-fwaas master: Fix tox.ini with tox4 https://review.opendev.org/c/openstack/neutron-fwaas/+/868965 | 08:46 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-sfc master: Fix tox.ini for tox4 https://review.opendev.org/c/openstack/networking-sfc/+/866971 | 08:47 |
| opendevreview | Arkady Shtempler proposed openstack/neutron-tempest-plugin master: Adds use of "Designate API" into "_verify_dns_records" method https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/859871 | 09:06 |
| bcafarel | ralonsoh++ | 09:20 |
| sahid | o/ | 09:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-fwaas master: Fix tox.ini with tox4 https://review.opendev.org/c/openstack/neutron-fwaas/+/868965 | 10:08 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: Fix tox.ini with tox4 https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/868993 | 10:09 |
| ralonsoh | eolivare, ^^ | 10:12 |
| ralonsoh | let's see if that solves the issue with pep8 job | 10:12 |
| eolivare | ralonsoh, let's see, thanks! | 10:18 |
| ralonsoh | slaweq, hi! please check | 10:22 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/868993 | 10:22 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron-fwaas/+/868965 | 10:22 |
| ralonsoh | https://review.opendev.org/c/openstack/networking-sfc/+/866971 | 10:22 |
| ralonsoh | thanks!! | 10:22 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-sfc master: Fix tox.ini for tox4 https://review.opendev.org/c/openstack/networking-sfc/+/866971 | 10:24 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Fix jobs after tox4 upgrade https://review.opendev.org/c/openstack/ovn-octavia-provider/+/868476 | 10:34 |
| opendevreview | Merged openstack/neutron stable/xena: Limit tox version to <4 https://review.opendev.org/c/openstack/neutron/+/868517 | 10:57 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Delete the PB level registers when deleting the duplicated PB https://review.opendev.org/c/openstack/neutron/+/868301 | 11:05 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Support for minimum bandwidth rules in tunnelled networks https://review.opendev.org/c/openstack/neutron/+/860639 | 11:14 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [ovn]support read chassis update time from nb_cfg_timestamp https://review.opendev.org/c/openstack/neutron/+/868498 | 11:16 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [ovn]neutron agent show real heartbeat_timestamp https://review.opendev.org/c/openstack/neutron/+/868499 | 11:16 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: Fix handling the restart of ovn-controllers https://review.opendev.org/c/openstack/neutron/+/868263 | 11:17 |
| opendevreview | Merged openstack/neutron stable/yoga: Limit tox version to <4 https://review.opendev.org/c/openstack/neutron/+/868516 | 11:17 |
| opendevreview | Merged openstack/neutron stable/zed: Limit tox version to <4 https://review.opendev.org/c/openstack/neutron/+/868515 | 11:17 |
| opendevreview | Merged openstack/networking-bgpvpn master: Tox4: add allowlist_externals where necessary https://review.opendev.org/c/openstack/networking-bgpvpn/+/868072 | 11:34 |
| ralonsoh | froyo, hi! | 11:36 |
| ralonsoh | you should backport https://review.opendev.org/c/openstack/ovn-octavia-provider/+/868476 to z and x | 11:36 |
| froyo | ralonsoh hi! | 11:36 |
| froyo | yeah! | 11:37 |
| ralonsoh | you have patches on those branches that are failing now | 11:37 |
| froyo | give some minutes | 11:37 |
| ralonsoh | just a heads-up | 11:37 |
| ralonsoh | sure | 11:37 |
| froyo | thx | 11:37 |
| tobias-urdin | ralonsoh: i'm trying to understand something here, for a HA router we are not tracking any qg or qr interface in track_interfaces in keepalived.conf just the ha interface where vrrp traffic is sent | 11:40 |
| tobias-urdin | i assume that's on purpose, but imo couldn't we track other interfaces there as well, we would need to reload on changes to the router which is why I assume it's not done yet | 11:41 |
| tobias-urdin | stumbled upon a router that became primary but the qg interface was down due to some unknown reason causing it to blackhole traffic | 11:41 |
| ralonsoh | tobias-urdin, those interfaces should be configured and ready before the keepalived process is spawned | 11:43 |
| ralonsoh | in any case, if you propose a solid use case and a solution for this, we can always consider that | 11:44 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/zed: Fix jobs after tox4 upgrade https://review.opendev.org/c/openstack/ovn-octavia-provider/+/868997 | 11:50 |
| ralonsoh | tobias-urdin, btw, are you aware of https://review.opendev.org/q/c52029c39aa824a67095fbbf9e59eff769d92587? | 11:50 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/yoga: Fix jobs after tox4 upgrade https://review.opendev.org/c/openstack/ovn-octavia-provider/+/868998 | 11:50 |
| ralonsoh | actually part of this functionality is being reverted | 11:51 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/xena: Fix jobs after tox4 upgrade https://review.opendev.org/c/openstack/ovn-octavia-provider/+/868999 | 11:51 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/wallaby: Fix jobs after tox4 upgrade https://review.opendev.org/c/openstack/ovn-octavia-provider/+/869000 | 11:56 |
| tobias-urdin | ralonsoh: ack, i wasn't aware of that one, will check | 11:58 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-fwaas master: Fix tox.ini with tox4 https://review.opendev.org/c/openstack/neutron-fwaas/+/868965 | 12:08 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: Move registration "ChassisBandwidthConfigEvent" to OvnSbIdl init https://review.opendev.org/c/openstack/neutron/+/868784 | 12:13 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add an env variable "PROCESS_TAG" in ``ProcessManager`` https://review.opendev.org/c/openstack/neutron/+/865018 | 12:24 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/wallaby: Fix jobs after tox4 upgrade https://review.opendev.org/c/openstack/ovn-octavia-provider/+/869000 | 12:25 |
| *** gthiemonge_ is now known as gthiemonge | 12:32 | |
| opendevreview | Merged openstack/neutron stable/ussuri: Limit tox version to <4 https://review.opendev.org/c/openstack/neutron/+/868520 | 13:25 |
| opendevreview | Bartosz Bezak proposed openstack/neutron master: [OVN] Fix availability zones changes check Functional test for network with external ports update added https://review.opendev.org/c/openstack/neutron/+/868746 | 13:54 |
| opendevreview | Bartosz Bezak proposed openstack/neutron master: [OVN] Fix availability zones changes check https://review.opendev.org/c/openstack/neutron/+/868746 | 14:13 |
| opendevreview | Merged openstack/neutron master: Add sanity check for keepalived gar on SIGHUP support https://review.opendev.org/c/openstack/neutron/+/868524 | 14:15 |
| opendevreview | Merged openstack/neutron master: [Fullstack] use common assert for ncat connection https://review.opendev.org/c/openstack/neutron/+/868676 | 14:23 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Fix tox.ini with tox4 https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/868993 | 14:24 |
| opendevreview | Merged openstack/neutron stable/victoria: Limit tox version to <4 https://review.opendev.org/c/openstack/neutron/+/868519 | 14:24 |
| froyo | ralonsoh, slaweq: could you review those backports https://review.opendev.org/q/Iea8e355cd18c51a00bdbe5225239965cfc1704d7, thx in advance | 15:44 |
| opendevreview | Bartosz Bezak proposed openstack/neutron master: [OVN] Fix availability zones changes check https://review.opendev.org/c/openstack/neutron/+/868746 | 15:44 |
| ralonsoh | froyo, sure, let me check | 15:45 |
| slaweq | froyo done | 15:46 |
| froyo | you are in sync! thank you both! | 15:47 |
| tobias-urdin | hm with this and the follow up reverted https://review.opendev.org/c/openstack/neutron/+/834162/4 | 15:48 |
| tobias-urdin | we should revert this as well right https://review.opendev.org/c/openstack/neutron/+/721799 | 15:48 |
| tobias-urdin | since now we can take advantage of actually finding out if an interface is dead now that they both are up (on active and standby) | 15:48 |
| tobias-urdin | see above, i got hit by that this morning | 15:49 |
| tobias-urdin | slaweq: what do you think? | 15:50 |
| slaweq | tobias-urdin it's probably good idea to revert it now | 15:51 |
| opendevreview | Tobias Urdin proposed openstack/neutron master: Revert "[L3 HA] Add "no_track" option to VIPs in keepalived config" https://review.opendev.org/c/openstack/neutron/+/869030 | 15:55 |
| tobias-urdin | missed that there was a config opt for that already, perhaps we just flip the default in master and let older versions handle it themselves in config | 16:01 |
| tobias-urdin | it's deprecated for removal tho | 16:01 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/wallaby: Fix jobs after tox4 upgrade https://review.opendev.org/c/openstack/ovn-octavia-provider/+/869000 | 16:13 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/zed: Fix jobs after tox4 upgrade https://review.opendev.org/c/openstack/ovn-octavia-provider/+/868997 | 16:39 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add an env variable "PROCESS_TAG" in ``ProcessManager`` https://review.opendev.org/c/openstack/neutron/+/865018 | 16:46 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [ovn]support read chassis update time from nb_cfg_timestamp https://review.opendev.org/c/openstack/neutron/+/868498 | 17:35 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [ovn]neutron agent show real heartbeat_timestamp https://review.opendev.org/c/openstack/neutron/+/868499 | 17:36 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: Fix handling the restart of ovn-controllers https://review.opendev.org/c/openstack/neutron/+/868263 | 17:36 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [ovn]support read chassis update time from nb_cfg_timestamp https://review.opendev.org/c/openstack/neutron/+/868498 | 18:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [ovn]neutron agent show real heartbeat_timestamp https://review.opendev.org/c/openstack/neutron/+/868499 | 18:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: Fix handling the restart of ovn-controllers https://review.opendev.org/c/openstack/neutron/+/868263 | 18:01 |
| opendevreview | Merged openstack/neutron stable/wallaby: Limit tox version to <4 https://review.opendev.org/c/openstack/neutron/+/868518 | 19:13 |
| opendevreview | Tobias Urdin proposed openstack/neutron master: Remove deprecated opt keepalived_use_no_track https://review.opendev.org/c/openstack/neutron/+/869012 | 19:25 |
| opendevreview | Tobias Urdin proposed openstack/neutron master: Track all interfaces in Keepalived https://review.opendev.org/c/openstack/neutron/+/869013 | 19:25 |
| opendevreview | Roberto Acosta proposed openstack/neutron master: [WIP] [RFE] [OVN] - IPv6 Distributed routing https://review.opendev.org/c/openstack/neutron/+/867513 | 20:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!