| opendevreview | OpenStack Proposal Bot proposed openstack/neutron master: Imported Translations from Zanata https://review.opendev.org/c/openstack/neutron/+/838078 | 03:36 |
|---|---|---|
| opendevreview | liuyulong proposed openstack/neutron master: Meter flows and ovsdb action for ovs bridge https://review.opendev.org/c/openstack/neutron/+/816800 | 03:38 |
| opendevreview | liuyulong proposed openstack/neutron master: Support pps limitation for openvswitch agent https://review.opendev.org/c/openstack/neutron/+/816802 | 03:38 |
| opendevreview | liuyulong proposed openstack/neutron master: Fullstack tests of packet rate limit for ovs qos driver https://review.opendev.org/c/openstack/neutron/+/819418 | 03:38 |
| opendevreview | Sharon Koech proposed openstack/neutron-specs master: Add spec for Network cascade deletion https://review.opendev.org/c/openstack/neutron-specs/+/810822 | 05:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/victoria: [OVN] Correctly set dns_server in dhcpv4/v6 options https://review.opendev.org/c/openstack/neutron/+/838878 | 06:27 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/ussuri: [OVN] Correctly set dns_server in dhcpv4/v6 options https://review.opendev.org/c/openstack/neutron/+/838879 | 06:27 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-ovn stable/train: Do not announce any DNS resolver if "0.0.0.0" or "::" provided https://review.opendev.org/c/openstack/networking-ovn/+/838987 | 06:38 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-ovn stable/train: [OVN] Correctly set dns_server in dhcpv4/v6 options https://review.opendev.org/c/openstack/networking-ovn/+/838988 | 06:50 |
| lajoskatona | ralonsoh, slaweq, mlavalle, obondarev_: Hi, when you have some time please check the Network cascade deletion spec (for Outreachy): https://review.opendev.org/c/openstack/neutron-specs/+/810822 | 07:07 |
| ralonsoh | sure | 07:07 |
| obondarev_ | ++ | 07:16 |
| opendevreview | Sharon Koech proposed openstack/neutron-specs master: Add spec for Network cascade deletion https://review.opendev.org/c/openstack/neutron-specs/+/810822 | 07:45 |
| lajoskatona | ralonsoh, otherwiseguy: Hi, for ovsdbapp don't we need nw victoria release before closing victoria? https://review.opendev.org/c/openstack/releases/+/837947/3/deliverables/victoria/ovsdbapp.yaml | 08:09 |
| ralonsoh | give me 5 mins, in a meeting now | 08:09 |
| lajoskatona | ralonsoh, otherwiseguy we have these on top of last release: $ git log --no-merges --oneline 1.6.1...HEAD | 08:10 |
| lajoskatona | 36fa770 (HEAD -> stable/victoria, origin/stable/victoria) github no longer accepts git:// connections | 08:10 |
| lajoskatona | 7440859 Log what exception caused abort of the transaction | 08:10 |
| lajoskatona | e5a97ed tools: run functional tests against ovn/main branch | 08:10 |
| ralonsoh | lajoskatona, the ovs2.17 fix has not been backported to Victoria | 08:23 |
| ralonsoh | because this version should not used this OVS version | 08:23 |
| ralonsoh | I think we are good now with the current code | 08:24 |
| ralonsoh | (I'll add otherwiseguy to this patch) | 08:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: [L3][QoS] Remove router QoS binding when GW is removed https://review.opendev.org/c/openstack/neutron/+/839016 | 08:40 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [L3][QoS] Remove router QoS binding when GW is removed https://review.opendev.org/c/openstack/neutron/+/839017 | 08:40 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: [L3][QoS] Remove router QoS binding when GW is removed https://review.opendev.org/c/openstack/neutron/+/839018 | 08:40 |
| lajoskatona | ralnsoh: thanks | 08:42 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: [OVN] Handle OVN agents when "Chassis" register is deleted https://review.opendev.org/c/openstack/neutron/+/839022 | 09:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [OVN] Handle OVN agents when "Chassis" register is deleted https://review.opendev.org/c/openstack/neutron/+/839023 | 09:02 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: [OVN] Handle OVN agents when "Chassis" register is deleted https://review.opendev.org/c/openstack/neutron/+/839024 | 09:02 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/victoria: [OVN] Handle OVN agents when "Chassis" register is deleted https://review.opendev.org/c/openstack/neutron/+/839027 | 09:07 |
| opendevreview | Merged openstack/neutron master: [ovn][migration] Support migration to OVN from iptables firewall https://review.opendev.org/c/openstack/neutron/+/837566 | 10:02 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-ovn stable/train: Do not announce any DNS resolver if "0.0.0.0" or "::" provided https://review.opendev.org/c/openstack/networking-ovn/+/838987 | 11:06 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [sfc] Remove unused router related methods https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/836457 | 11:09 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [sfc] Remove _create_port & create_keypair https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/836491 | 11:09 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [fwaas] Remove unused methods https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/837273 | 11:09 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [bgpvpn] Remove unused scenario.manager methods https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/836950 | 11:09 |
| opendevreview | Merged openstack/networking-bgpvpn master: Register common cfg options from Neutron https://review.opendev.org/c/openstack/networking-bgpvpn/+/838840 | 11:54 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Fix request to OVN NB DB API https://review.opendev.org/c/openstack/ovn-octavia-provider/+/839055 | 12:13 |
| opendevreview | Merged openstack/networking-sfc master: Register common cfg options from Neutron https://review.opendev.org/c/openstack/networking-sfc/+/838839 | 12:22 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [sfc] Remove create_server & get_remote_client https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/836571 | 12:37 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [sfc] Remove _log_console_output & ping_ip_address https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/836578 | 12:37 |
| ykarel | ralonsoh, lajoskatona can you please check https://review.opendev.org/c/openstack/neutron/+/835786 and https://review.opendev.org/c/openstack/neutron/+/833620 when get a chance | 12:46 |
| ykarel | thx in advance | 12:46 |
| ralonsoh | sure | 12:48 |
| lajoskatona | ykarel: I will check it | 12:51 |
| *** dasm|off is now known as dasm | 13:16 | |
| otherwiseguy | lajoskatona: it really just depends on how far back we think someone is likely to try to run an old release of neutron with ovs as new as 2.17 (but less than probably 2.17.2 which will have restored backward compatibility) | 13:52 |
| otherwiseguy | I'd happily backport it as far back as we're doing backports. | 13:53 |
| *** Guest2246 is now known as prometheanfire | 13:54 | |
| lajoskatona | otherwiseguy: don't we have constraint for it? | 13:54 |
| lajoskatona | otherwiseguy: now I said (wrote) it we have: https://opendev.org/openstack/requirements/src/branch/stable/victoria/upper-constraints.txt#L175 | 13:55 |
| lajoskatona | otherwiseguy: I don't know if that is "official" recommendation for deployers or just something more used for CI | 13:55 |
| otherwiseguy | lajoskatona: so anyone running from pip in production is safe. :D | 13:55 |
| lajoskatona | otherwiseguy: nowadays I more think that pip causing more trouble, but you are right :-) | 13:56 |
| otherwiseguy | lajoskatona: I know wallabyh has 2.13.0, but at RH we'll be using 2.17 | 13:56 |
| opendevreview | Daniel Alvarez proposed openstack/neutron master: [WIP][DNM][OVN] Change the default firewall policy https://review.opendev.org/c/openstack/neutron/+/839066 | 13:59 |
| lajoskatona | otherwiseguy: do you think we need to backport this ovs workaround to Victoria then, or just let it as it is now? I can argue for both | 14:02 |
| otherwiseguy | lajoskatona: I'll check real quick that it is even needed. the update_tables method might not even be in victoria ovsdbapp (though I'd bet the .condition usage in neutron is) | 14:03 |
| otherwiseguy | lajoskatona: yeah, ovsdbapp is fine, neutron victoria would still crash with 2.17. So it's pretty simple to just backport the change to neutron, no other version bumps required for ovsdbapp, etc. | 14:05 |
| lajoskatona | otherwiseguy: thanks, so we have to release Neutron for victoria with the backport? | 14:09 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-ovn stable/train: [OVN] Pin OVS version for OVN master https://review.opendev.org/c/openstack/networking-ovn/+/839070 | 14:10 |
| otherwiseguy | lajoskatona: yeah. unless it's a lot of trouble. if so, then meh, it's probably not going to bite many people. | 14:10 |
| opendevreview | Terry Wilson proposed openstack/neutron stable/victoria: Fix setting table monitoring conditions https://review.opendev.org/c/openstack/neutron/+/838883 | 14:12 |
| lajoskatona | otherwiseguy: we have one week for it as I see: http://lists.openstack.org/pipermail/openstack-discuss/2022-April/028013.html :-) | 14:12 |
| otherwiseguy | I'll have the patch in in the next 5 mins. | 14:13 |
| opendevreview | Terry Wilson proposed openstack/neutron stable/victoria: Fix setting table monitoring conditions https://review.opendev.org/c/openstack/neutron/+/838883 | 14:13 |
| opendevreview | Terry Wilson proposed openstack/neutron stable/victoria: Fix setting table monitoring conditions https://review.opendev.org/c/openstack/neutron/+/838883 | 14:14 |
| lajoskatona | otherwiseguy: we need the full pack of backports then: https://review.opendev.org/q/I0503037b803a3c99fb7988bc20394c111ac456db | 14:17 |
| opendevreview | Merged openstack/neutron master: ovn migration: Fix check for stack name https://review.opendev.org/c/openstack/neutron/+/835786 | 14:29 |
| otherwiseguy | lajoskatona: running recheck on them now that ovsdbapp releases have been made. | 14:34 |
| * otherwiseguy has to go run an errand and will be back in an hour | 14:35 | |
| lajoskatona | otherwiseguy: ok, thanks, I will probably disappear also | 14:35 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-ovn stable/train: [OVN] Pin OVS/OVN versions in rally job https://review.opendev.org/c/openstack/networking-ovn/+/839070 | 14:35 |
| ralonsoh | froyo, ^ | 14:36 |
| ralonsoh | lucasagomes, jlibosva ^^ rally job is broken now. I'm pining OVN/OVS versions to something that for sure is working now | 14:37 |
| froyo | ralonsoh ok!, but there are other jobs using main | 14:37 |
| ralonsoh | yes but this is stable/train | 14:37 |
| ralonsoh | there is no need to use main in a stable rally job | 14:37 |
| jlibosva | ralonsoh: why do we use the commit hash for OVS? Can't we use release tag? | 14:37 |
| froyo | ralonsoh, you are right! | 14:38 |
| ralonsoh | jlibosva, no, we had this problem before | 14:38 |
| ralonsoh | one sec | 14:38 |
| ralonsoh | jlibosva, https://bugs.launchpad.net/neutron/+bug/1967472 | 14:38 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/836121 | 14:38 |
| ralonsoh | that was the recommended hash by dceara | 14:38 |
| jlibosva | ralonsoh: so why don't we use 22.03 OVN that won't need that OVS specific commit? | 14:39 |
| jlibosva | it would also be closer to the main branch | 14:39 |
| ralonsoh | jlibosva, in n-ovn train? | 14:39 |
| ralonsoh | ok, I can try it | 14:39 |
| ralonsoh | but I don't think we need this version in stable/train | 14:39 |
| jlibosva | ralonsoh: ah, it's train | 14:39 |
| jlibosva | sorry, I missed that | 14:39 |
| jlibosva | ralonsoh: I +1d because I'm not a stable core - but makes sense to me now | 14:40 |
| ralonsoh | thanks! | 14:40 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-ovn stable/train: Do not announce any DNS resolver if "0.0.0.0" or "::" provided https://review.opendev.org/c/openstack/networking-ovn/+/838987 | 14:57 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-ovn stable/train: [OVN] Correctly set dns_server in dhcpv4/v6 options https://review.opendev.org/c/openstack/networking-ovn/+/838988 | 14:59 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: [OVN] Pin OVS version for OVN main https://review.opendev.org/c/openstack/ovn-octavia-provider/+/839083 | 16:37 |
| mnaser | lajoskatona, slaweq: can i get a +W on https://review.opendev.org/c/openstack/neutron-vpnaas/+/838795/1 to get the neutron-vpnaas fixes in? | 19:53 |
| opendevreview | Mohammed Naser proposed openstack/neutron-vpnaas master: cleanup: drop un-used test-requirements.txt https://review.opendev.org/c/openstack/neutron-vpnaas/+/838808 | 19:54 |
| opendevreview | Jakub Libosvar proposed openstack/os-vif master: Check for hybrid plugging in OVS https://review.opendev.org/c/openstack/os-vif/+/839102 | 21:09 |
| *** dasm is now known as dasm|off | 22:03 | |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: Make the recheck policy more prominently visible https://review.opendev.org/c/openstack/neutron/+/839107 | 22:11 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: Make the recheck policy more prominently visible https://review.opendev.org/c/openstack/neutron/+/839107 | 22:13 |
| opendevreview | Jakub Libosvar proposed openstack/os-vif master: Check for hybrid plugging in OVS https://review.opendev.org/c/openstack/os-vif/+/839102 | 22:20 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!