| gmann | slaweq: ralonsoh this is green now https://review.opendev.org/c/openstack/neutron-lib/+/874133 | 00:10 |
|---|---|---|
| *** gibi_pto is now known as gibi | 08:04 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: [OVS] Allow custom ethertype traffic in the ingress table https://review.opendev.org/c/openstack/neutron/+/877585 | 09:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: [OVS] Allow custom ethertype traffic in the ingress table https://review.opendev.org/c/openstack/neutron/+/877586 | 09:18 |
| *** elodilles_pto is now known as elodilles | 09:26 | |
| opendevreview | Yamato Tanaka proposed openstack/neutron master: doc: state that O flag can be 0 in dhcpv6-stateful https://review.opendev.org/c/openstack/neutron/+/877601 | 09:33 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add end to end test for QosExtension https://review.opendev.org/c/openstack/neutron/+/877603 | 09:35 |
| slaweq | lajoskatona mlavalle hi, can You maybe check https://review.opendev.org/c/openstack/neutron-lib/+/874133 also? | 10:01 |
| *** EugenMayer46 is now known as EugenMayer4 | 10:02 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [OVS] Allow custom ethertype traffic in the ingress table https://review.opendev.org/c/openstack/neutron/+/877605 | 10:15 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: [OVS] Allow custom ethertype traffic in the ingress table https://review.opendev.org/c/openstack/neutron/+/877607 | 10:16 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: [OVS] Allow custom ethertype traffic in the ingress table https://review.opendev.org/c/openstack/neutron/+/877608 | 10:16 |
| opendevreview | Lajos Katona proposed openstack/neutron-tempest-plugin master: Add job definitions for 2023.1 (Antelope) branch https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/877613 | 10:45 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: Delete the PB level registers when deleting the duplicated PB https://review.opendev.org/c/openstack/neutron/+/877493 | 10:45 |
| opendevreview | Lajos Katona proposed openstack/neutron-tempest-plugin master: Add job definitions for 2023.1 (Antelope) branch https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/877613 | 10:48 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Replace "tenant_id" with "project_id" in IPAM engine https://review.opendev.org/c/openstack/neutron/+/877533 | 11:29 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Explicitly define the fixed IPs for the metadata port https://review.opendev.org/c/openstack/neutron/+/877528 | 11:33 |
| opendevreview | Merged openstack/neutron-lib stable/zed: Do not set system scope on context https://review.opendev.org/c/openstack/neutron-lib/+/874133 | 11:37 |
| ralonsoh | slaweq, hey!! Good news: "neutron-tempest-plugin-openvswitch-enforce-scope-new-defaults-zed" is passing. Lots of thanks. | 11:49 |
| ralonsoh | I have one concern about the CI results (still in progress) | 11:49 |
| slaweq | ralonsoh hi, great news :) | 11:49 |
| slaweq | thx | 11:49 |
| ralonsoh | the tests are taking from 2x up to 5x the time | 11:49 |
| ralonsoh | for example: | 11:50 |
| ralonsoh | neutron_tempest_plugin.scenario.test_multicast.MulticastTestIPv4.test_multicast_between_vms_on_same_network [231.032486s] | 11:50 |
| ralonsoh | neutron_tempest_plugin.scenario.test_multicast.MulticastTestIPv4.test_multicast_between_vms_on_same_network [1904.600389s] | 11:50 |
| slaweq | ouch | 11:50 |
| ralonsoh | the second is the current job execution | 11:50 |
| slaweq | but only in Zed or master too? | 11:50 |
| ralonsoh | zed | 11:50 |
| slaweq | ok, I will investigate that | 11:50 |
| slaweq | but not today | 11:51 |
| ralonsoh | no rush | 11:51 |
| ralonsoh | in any case, I'll send a mail saying that we support sRBAC in Zed | 11:51 |
| slaweq | ++ | 11:55 |
| slaweq | thx a lot | 11:55 |
| slaweq | yes, this issue can be treated as bug in Zed and we can fix it later | 11:55 |
| opendevreview | Lajos Katona proposed openstack/neutron-tempest-plugin master: Add job definitions for 2023.1 (Antelope) branch https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/877613 | 12:29 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [OVS] Allow custom ethertype traffic in the ingress table https://review.opendev.org/c/openstack/neutron/+/877605 | 12:33 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron-specs master: Active-active L3 Gateway with Multihoming https://review.opendev.org/c/openstack/neutron-specs/+/870030 | 12:35 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [DNM] Add rate-limiting to metadata agents https://review.opendev.org/c/openstack/neutron/+/858879 | 13:29 |
| otherwiseguy | ralonsoh: So, uh, the ovsdbapp functional test job for at least the last month has not been running functional tests--it's running unit tests: https://zuul.opendev.org/t/openstack/build/c9aad570a9a64594aa7b08ad1ee7d899/logs | 13:38 |
| ralonsoh | otherwiseguy, ? tox-functional is working | 13:50 |
| ralonsoh | what I'm missing? | 13:51 |
| otherwiseguy | It's passing, but it's running *unit* tests, not functional tests. | 13:51 |
| otherwiseguy | and has been that way for at least a month (no logs before that) | 13:51 |
| ralonsoh | otherwiseguy, no way... let me check what happened | 13:52 |
| otherwiseguy | thanks. I'm digging through the openstack-zuul-jobs, but haven't found it yet. There was this big change: https://opendev.org/openstack/openstack-zuul-jobs/commit/6d85fd8399ed6b9f2358412945cd6683989662cd | 13:54 |
| otherwiseguy | but it was 5 months ago... | 13:54 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: fix functional https://review.opendev.org/c/openstack/ovsdbapp/+/877661 | 14:01 |
| * otherwiseguy looks | 14:03 | |
| otherwiseguy | the weird thing was it was actually running stuff from .tox/functional and building ovn, etc. And then just ran unit tests. | 14:03 |
| ralonsoh | we still need to fix the tox.ino testeenv definition | 14:03 |
| ralonsoh | this is not going to work | 14:03 |
| otherwiseguy | it's almost like it's ignoring OS_TEST_PATH=./ovsdbapp/tests/functional | 14:13 |
| ralonsoh | right and I don't know why | 14:13 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: fix functional https://review.opendev.org/c/openstack/ovsdbapp/+/877661 | 14:15 |
| ralonsoh | otherwiseguy, ^^ with PS2, py3 is failing, functional passing. I need to know how to properly override OS_TEST_PATH in functional | 14:25 |
| otherwiseguy | well that certainly pissed off the unit tests, but that's weird. | 14:25 |
| * otherwiseguy is still watching the functional run | 14:25 | |
| otherwiseguy | yeah, there go the functional tests. | 14:26 |
| ykarel | ralonsoh, can follow neutron, there it works fine i.e unit test path set in .stestr.conf | 14:26 |
| ralonsoh | right | 14:26 |
| ykarel | seems the behavior is happening with newer tox | 14:26 |
| ralonsoh | I was looking at this now | 14:26 |
| ralonsoh | ykarel, thanks! | 14:26 |
| otherwiseguy | (although I would prefer not having to type tox -e functional-py39 at the CLI) :) | 14:27 |
| * otherwiseguy glares at tox | 14:27 | |
| otherwiseguy | ralonsoh++ thanks for jumping right on this. sorry to interrupt what you were already working on. | 14:28 |
| ralonsoh | otherwiseguy, no problem at all | 14:28 |
| ralonsoh | once merged, I'll backport it | 14:28 |
| ralonsoh | (seems that FT are passing, so we are safe) | 14:28 |
| ykarel | yes functional-py39 change shouldn't be needed | 14:29 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: [CI] Set the default OS_TEST_PATH in the stestr conf file https://review.opendev.org/c/openstack/ovsdbapp/+/877661 | 14:31 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp stable/2023.1: [CI] Set the default OS_TEST_PATH in the stestr conf file https://review.opendev.org/c/openstack/ovsdbapp/+/877632 | 14:54 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Add new function ls_get_localnet_ports https://review.opendev.org/c/openstack/ovsdbapp/+/873853 | 15:21 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Delete sg rule which remote is the deleted sg https://review.opendev.org/c/openstack/neutron/+/876716 | 15:35 |
| lajoskatona | ralonsoh, slaweq, haleyb: Hi, could you check https://review.opendev.org/c/openstack/neutron/+/876716 ? this is related to https://bugs.launchpad.net/neutron/+bug/2008712 | 15:38 |
| ralonsoh | sure | 15:39 |
| lajoskatona | ralonsoh, slaweq, haleyb: I just pushed PS4 to change how OVN deletes the rules which has remote group id, but to tell the truth as it worked previously well I am not sure if it is a good idea to change it :-) | 15:40 |
| haleyb | lajoskatona: on my list :) | 15:41 |
| lajoskatona | ralonsoh, slaweq, haleyb: so if you have time please check if PS3-PS4 change is a good odea or not | 15:41 |
| lajoskatona | haleyb: thanks | 15:41 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovsdbapp master: Add new function ls_get_localnet_ports https://review.opendev.org/c/openstack/ovsdbapp/+/873853 | 15:48 |
| ralonsoh | froyo_, https://review.opendev.org/c/openstack/ovn-octavia-provider/+/876585 | 16:03 |
| ralonsoh | please check this patch | 16:03 |
| ralonsoh | to release the last RC version tomorrow | 16:04 |
| froyo_ | this one is pending of this another https://review.opendev.org/c/openstack/ovn-octavia-provider/+/877462 | 16:04 |
| ralonsoh | apprived | 16:06 |
| ralonsoh | approved | 16:06 |
| froyo_ | bcafarel, could you review this last one ^ | 16:06 |
| ralonsoh | froyo_, this patch is merged | 16:06 |
| froyo_ | right! my fault! | 16:06 |
| froyo_ | rechecking right now | 16:07 |
| ralonsoh | bcafarel, slaweq https://review.opendev.org/q/project:openstack/neutron+status:open+branch:stable/2023.1 | 16:07 |
| ralonsoh | please, review these patches | 16:07 |
| ralonsoh | to push the final RC for antelope tomorrow | 16:07 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/2023.1: Remove HM uuiid from LB external_ids when the HM's pool is deleted https://review.opendev.org/c/openstack/ovn-octavia-provider/+/876585 | 16:07 |
| slaweq | ralonsoh sure, checking now | 16:16 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: Fix metadata agent intermittent test failures https://review.opendev.org/c/openstack/neutron/+/877535 | 16:17 |
| slaweq | ralonsoh I just approved those which had +2 from bcafarel too and I +2 one which didn't have other +2 yet :) | 16:20 |
| ralonsoh | slaweq, thanks! | 16:21 |
| slaweq | ralonsoh: yw | 16:21 |
| bcafarel | ralonsoh: checking that last one right now | 16:21 |
| ralonsoh | thanks | 16:21 |
| opendevreview | Bernard Cafarelli proposed openstack/neutron stable/yoga: [OVS] Allow custom ethertype traffic in the ingress table https://review.opendev.org/c/openstack/neutron/+/877608 | 16:28 |
| opendevreview | Bernard Cafarelli proposed openstack/neutron stable/wallaby: [OVS] Allow custom ethertype traffic in the ingress table https://review.opendev.org/c/openstack/neutron/+/877607 | 16:30 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add end to end test for QosExtension https://review.opendev.org/c/openstack/neutron/+/877603 | 17:36 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add end to end test for QosExtension https://review.opendev.org/c/openstack/neutron/+/877603 | 17:40 |
| opendevreview | Merged openstack/neutron stable/2023.1: [OVN] Use the BW values retrieved from ``get_port_qos`` https://review.opendev.org/c/openstack/neutron/+/876806 | 17:45 |
| opendevreview | Merged openstack/neutron stable/2023.1: [OVN] Change oslo config options entry point for the OVN agent https://review.opendev.org/c/openstack/neutron/+/876825 | 17:45 |
| opendevreview | Rodolfo Alonso proposed openstack/ovn-octavia-provider stable/2023.1: Add new FTs for health monitoring basic operations https://review.opendev.org/c/openstack/ovn-octavia-provider/+/876586 | 17:53 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [DNM] Add rate-limiting to metadata agents https://review.opendev.org/c/openstack/neutron/+/858879 | 18:02 |
| opendevreview | Merged openstack/neutron stable/zed: Delete the PB level registers when deleting the duplicated PB https://review.opendev.org/c/openstack/neutron/+/877484 | 18:06 |
| opendevreview | Merged openstack/neutron stable/yoga: Delete the PB level registers when deleting the duplicated PB https://review.opendev.org/c/openstack/neutron/+/877485 | 18:09 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovsdbapp master: Add new function ls_get_localnet_ports https://review.opendev.org/c/openstack/ovsdbapp/+/873853 | 18:14 |
| opendevreview | Merged openstack/neutron stable/2023.1: [OVS] Allow custom ethertype traffic in the ingress table https://review.opendev.org/c/openstack/neutron/+/877585 | 18:15 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2023.1: Remove HM uuiid from LB external_ids when the HM's pool is deleted https://review.opendev.org/c/openstack/ovn-octavia-provider/+/876585 | 18:30 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: WIP [ovn] Drop use of LR OVN_GW_NETWORK_EXT_ID_KEY https://review.opendev.org/c/openstack/neutron/+/877712 | 19:24 |
| opendevreview | Arnaud Morin proposed openstack/neutron master: Set default value for local_ip https://review.opendev.org/c/openstack/neutron/+/877716 | 20:18 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2023.1: Add new FTs for health monitoring basic operations https://review.opendev.org/c/openstack/ovn-octavia-provider/+/876586 | 20:19 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: WIP Because I am trying to figure out best way to write test for this. Fix ACL sync when default sg group is created https://review.opendev.org/c/openstack/neutron/+/875989 | 20:37 |
| opendevreview | Brian Haley proposed openstack/neutron master: Add support for multiple IPv6 metadata addresses https://review.opendev.org/c/openstack/neutron/+/876903 | 22:31 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [DNM] Add rate-limiting to metadata agents https://review.opendev.org/c/openstack/neutron/+/858879 | 23:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!