| opendevreview | Takashi Kajinami proposed openstack/tap-as-a-service master: Squash tass.ini and taas_plugin.ini https://review.opendev.org/c/openstack/tap-as-a-service/+/929814 | 01:09 |
|---|---|---|
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Add initial mypy job https://review.opendev.org/c/openstack/neutron/+/929864 | 03:26 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: Use tuple type for SUPPORTED_LOGGING_TYPES https://review.opendev.org/c/openstack/neutron/+/929865 | 03:26 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/* https://review.opendev.org/c/openstack/neutron/+/929866 | 03:26 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: neutron/db/l3* https://review.opendev.org/c/openstack/neutron/+/929867 | 03:26 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: use dataclass for AgentConsumer https://review.opendev.org/c/openstack/neutron/+/929868 | 03:26 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: neutron/extensions/* https://review.opendev.org/c/openstack/neutron/+/929869 | 03:26 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: neutron/agent/* https://review.opendev.org/c/openstack/neutron/+/929870 | 03:26 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: neutron/objects/* https://review.opendev.org/c/openstack/neutron/+/929871 | 03:26 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Add initial mypy job https://review.opendev.org/c/openstack/neutron/+/929864 | 03:30 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Add initial mypy job https://review.opendev.org/c/openstack/neutron/+/929864 | 03:33 |
| opendevreview | Takashi Kajinami proposed openstack/tap-as-a-service master: Use generated config files in devstack https://review.opendev.org/c/openstack/tap-as-a-service/+/929872 | 05:37 |
| opendevreview | Takashi Kajinami proposed openstack/tap-as-a-service master: Squash tass.ini and taas_plugin.ini https://review.opendev.org/c/openstack/tap-as-a-service/+/929814 | 05:44 |
| opendevreview | Takashi Kajinami proposed openstack/tap-as-a-service master: Use generated config files in devstack https://review.opendev.org/c/openstack/tap-as-a-service/+/929872 | 05:44 |
| opendevreview | Takashi Kajinami proposed openstack/tap-as-a-service master: Use generated config files in devstack https://review.opendev.org/c/openstack/tap-as-a-service/+/929872 | 05:47 |
| opendevreview | Merged openstack/neutron-vpnaas stable/2023.2: Remove reference to devstack-gate https://review.opendev.org/c/openstack/neutron-vpnaas/+/923050 | 07:22 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use neutron-lib constants IPv4_ANY and IPv6_ANY https://review.opendev.org/c/openstack/neutron/+/929883 | 07:58 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-sfc master: [sqlalchemy-20] Remove CI job "openstack-tox-py310-with-sqlalchemy-main-sfc" https://review.opendev.org/c/openstack/networking-sfc/+/929884 | 08:03 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-bagpipe master: [sqlalchemy-20] Remove CI job "networking-bagpipe-openstack-tox-py310-with-sqlalchemy-main" https://review.opendev.org/c/openstack/networking-bagpipe/+/929886 | 08:05 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-bgpvpn master: [sqlalchemy-20] Remove CI job "networking-bgpvpn-openstack-tox-py310-with-sqlalchemy-main" https://review.opendev.org/c/openstack/networking-bgpvpn/+/929887 | 08:07 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-dynamic-routing master: [sqlalchemy-20] Remove CI job "neutron-dynamic-routing-openstack-tox-py310-with-sqlalchemy-main" https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/929888 | 08:08 |
| opendevreview | Takashi Kajinami proposed openstack/tap-as-a-service master: Use generated config files in devstack https://review.opendev.org/c/openstack/tap-as-a-service/+/929872 | 08:08 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-fwaas master: [sqlalchemy-20] Remove CI job "neutron-fwaas-openstack-tox-py310-with-sqlalchemy-main" https://review.opendev.org/c/openstack/neutron-fwaas/+/929889 | 08:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: [sqlalchemy-20] Remove CI job "neutron-vpnaas-openstack-tox-py310-with-sqlalchemy-main" https://review.opendev.org/c/openstack/neutron-vpnaas/+/929890 | 08:12 |
| ralonsoh | lajoskatona, haleyb hello! Trivial clean-up stuff: https://review.opendev.org/q/topic:%22remove_sqlalchemy_20_main_jobs%22 | 08:14 |
| lajoskatona | ralonsoh: checking, thanks | 08:15 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Docs: Add link to OVN from deplyoments page https://review.opendev.org/c/openstack/neutron/+/929895 | 08:45 |
| opendevreview | Takashi Kajinami proposed openstack/tap-as-a-service master: Use generated config files in devstack https://review.opendev.org/c/openstack/tap-as-a-service/+/929872 | 08:56 |
| *** liuxie is now known as liushy | 09:06 | |
| opendevreview | Takashi Kajinami proposed openstack/neutron-vpnaas master: Load core neutron.conf in ovn-vpn-agent https://review.opendev.org/c/openstack/neutron-vpnaas/+/929898 | 09:32 |
| opendevreview | Merged openstack/neutron-vpnaas stable/2023.1: Remove reference to devstack-gate https://review.opendev.org/c/openstack/neutron-vpnaas/+/923051 | 09:33 |
| opendevreview | Takashi Kajinami proposed openstack/tap-as-a-service master: Use generated config files in devstack https://review.opendev.org/c/openstack/tap-as-a-service/+/929872 | 10:03 |
| opendevreview | Takashi Kajinami proposed openstack/tap-as-a-service master: Use generated config files in devstack https://review.opendev.org/c/openstack/tap-as-a-service/+/929872 | 10:11 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ml2/ovs: add log message for ports without VLAN tags during OVS restore https://review.opendev.org/c/openstack/neutron/+/929907 | 11:36 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ml2/ovs: add log message for ports without VLAN tags during OVS restore https://review.opendev.org/c/openstack/neutron/+/929908 | 11:36 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ml2/ovs: change log level from DEBUG to WARNING for port deletion during binding https://review.opendev.org/c/openstack/neutron/+/929909 | 11:36 |
| noonedeadpunk | as promised - bug report: https://bugs.launchpad.net/neutron/+bug/2081173 | 11:36 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ml2/ovs: add log message for ports without net-id during OVS restore https://review.opendev.org/c/openstack/neutron/+/929908 | 11:37 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ml2/ovs: change log level from DEBUG to WARNING for port deletion during binding https://review.opendev.org/c/openstack/neutron/+/929909 | 11:37 |
| opendevreview | Merged openstack/neutron-fwaas-dashboard master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/neutron-fwaas-dashboard/+/929083 | 12:08 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: refactor: don't serialize all switches on each lb checked https://review.opendev.org/c/openstack/neutron/+/929774 | 12:33 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: refactor: refactor: slightly more explicit return value https://review.opendev.org/c/openstack/neutron/+/929792 | 12:33 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: refactor: unindent some indents in _handle_lb_fip_cmds https://review.opendev.org/c/openstack/neutron/+/929793 | 12:33 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: refactor: don't calculate list of attached lbs for every lb https://review.opendev.org/c/openstack/neutron/+/929794 | 12:33 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: refactor: split out code to verify a lb member into a function https://review.opendev.org/c/openstack/neutron/+/929795 | 12:33 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: refactor: construct member dict in one go https://review.opendev.org/c/openstack/neutron/+/929796 | 12:33 |
| opendevreview | yatin proposed openstack/neutron master: Handle EndpointNotFound in nova notifier https://review.opendev.org/c/openstack/neutron/+/929920 | 13:09 |
| opendevreview | Takashi Kajinami proposed openstack/os-ken master: Bump hacking https://review.opendev.org/c/openstack/os-ken/+/929921 | 13:12 |
| opendevreview | Merged openstack/neutron-vpnaas master: [sqlalchemy-20] Remove CI job "neutron-vpnaas-openstack-tox-py310-with-sqlalchemy-main" https://review.opendev.org/c/openstack/neutron-vpnaas/+/929890 | 13:14 |
| opendevreview | Merged openstack/networking-bgpvpn master: [sqlalchemy-20] Remove CI job "networking-bgpvpn-openstack-tox-py310-with-sqlalchemy-main" https://review.opendev.org/c/openstack/networking-bgpvpn/+/929887 | 14:01 |
| opendevreview | Merged openstack/neutron-dynamic-routing master: [sqlalchemy-20] Remove CI job "neutron-dynamic-routing-openstack-tox-py310-with-sqlalchemy-main" https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/929888 | 14:03 |
| PrzemekK | Why there is no repo bgpvpn-dashboard like all dashboards ? 'horizon-plugin-blazar-dashboard': { 'type': 'url', 'location': ('$tarballs_base/openstack/blazar-dashboard/' 'blazar-dashboard-${openstack_branch}.tar.gz')}, | 14:06 |
| PrzemekK | How to change above code just download https://github.com/openstack/networking-bgpvpn/tree/master/bgpvpn_dashboard | 14:07 |
| opendevreview | Merged openstack/networking-sfc master: [sqlalchemy-20] Remove CI job "openstack-tox-py310-with-sqlalchemy-main-sfc" https://review.opendev.org/c/openstack/networking-sfc/+/929884 | 14:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Change the load method of SG rule "default_security_group" https://review.opendev.org/c/openstack/neutron/+/929941 | 14:28 |
| ralonsoh | noonedeadpunk, haleyb ^ please check this | 14:29 |
| ralonsoh | and also https://bugs.launchpad.net/neutron/+bug/2081087/comments/8 | 14:29 |
| haleyb | ralonsoh: did you see https://review.opendev.org/c/openstack/neutron/+/929851 ? | 14:30 |
| ralonsoh | yes and I commented that in c#8 | 14:31 |
| ralonsoh | we need to do this in batches | 14:31 |
| ralonsoh | to benchmark each change | 14:31 |
| ralonsoh | we can' | 14:31 |
| ralonsoh | we can't push this patch in a single shot | 14:31 |
| ralonsoh | for simple relationships, joined is fine. Actually could be faster | 14:32 |
| ralonsoh | the query is more complex but you issue one single query | 14:32 |
| noonedeadpunk | ralonsoh: yeah, I checked that yesterday and it did the job nicely with 2024.1 constraints in | 14:33 |
| haleyb | ralonsoh: ack, and i think that's why i left the 'joined' when i did the 'subquery' change. i think 'selectin' has been around since 1.3 btw | 14:33 |
| ralonsoh | yeah, I'm not sure but we can check when backporting to 2024.1 | 14:34 |
| noonedeadpunk | I haven't seen it in docs for sqlalchemy though, but it seemed to work <2.0 | 14:34 |
| noonedeadpunk | as I had 1.4.41 in my venv | 14:35 |
| haleyb | neutron-ovn-rally-task did seem slower in my patch, i just didn't know how confident i was since i didn't know if we could consider each run "equal" | 14:36 |
| noonedeadpunk | oh yes, that depends on the nodepool provider a lot.... | 14:37 |
| noonedeadpunk | btw.... I have a weird and completely different issue with OVN I don't know how to track even. so if someone can help out with basic OVN debugging or point to some docs - that would be nice :) so - | 14:38 |
| noonedeadpunk | I do have 2 VLAN providers, one of which is just regular bond0, but second one is actually a MACVLAN interface connected to eth0 (as eth0 is a standalone interface with SSH access) | 14:39 |
| noonedeadpunk | When I manually create a tagged vlan on this MACVLAN and assign IP - it works nicely. But as you might get - I don't have connectivity inside VMs once they're on different nodes | 14:40 |
| opendevreview | yatin proposed openstack/neutron unmaintained/wallaby: [DNM] Check sdk job with lioadm helper https://review.opendev.org/c/openstack/neutron/+/929944 | 14:41 |
| noonedeadpunk | What I'm trying to find - is how to actually check if flows are passing to the correct vlan provider? As networks are named differently, inside ovn I see switch having proper tag, but I'm not getting how tag is being added and at what point | 14:41 |
| noonedeadpunk | and I guess it could smth to do with MACVLAN I'm using. I tried to add just eth0 to the bridge (while loosing SSH connection), but that did not help obviously right away | 14:44 |
| noonedeadpunk | So I guess I'm just a bit confused, that I don't see tagged interfaces directly in the bridge. but I don't see for any of VLAN providers while one working as designed. So was thinking if just second provider does not work somehow and OVN still tries to pass through the first one | 14:45 |
| noonedeadpunk | I think I can workaround this through flat netwokrk ofc, as I 'm having just 2 vlans for the second provider.... | 14:46 |
| haleyb | ralonsoh: selectin was added in 1.2, but looks like 1.3 had improvements as did 1.4 ? either way there is a limit to how far back we can use it | 14:53 |
| ralonsoh | haleyb, in 2024.1 we use 1.4.x, so it will work too (the backport) | 14:53 |
| haleyb | perfect | 14:53 |
| PrzemekK | Why there is repo bgpvpn-dashboard ? https://bugs.launchpad.net/bgpvpn/+bug/2081188 | 15:04 |
| PrzemekK | there is no | 15:04 |
| haleyb | PrzemekK: there is a bgpbpn_dashboard subdir in the networking-bgpvpn repo, although i can't say i'm familiar with it | 15:40 |
| haleyb | and horizon seems to consume it? | 15:40 |
| haleyb | https://docs.openstack.org/networking-bgpvpn/latest/user/horizon.html | 15:41 |
| PrzemekK | yeah old link how it looks in gui https://youtu.be/LCDeR7MwTzE?t=993 | 15:41 |
| PrzemekK | but its hard to download it when there is no tarball 'horizon-plugin-blazar-dashboard': { 'type': 'url', 'location': ('$tarballs_base/openstack/blazar-dashboard/' 'blazar-dashboard-${openstack_branch}.tar.gz')}, | 15:42 |
| haleyb | sorry, i don't have a good answer | 15:46 |
| opendevreview | Brian Haley proposed openstack/neutron master: Use py312 for all neutron jobs https://review.opendev.org/c/openstack/neutron/+/929824 | 15:50 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Correct logic error when associating FIP with OVN LB https://review.opendev.org/c/openstack/neutron/+/921663 | 16:02 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: refactor: don't serialize all switches on each lb checked https://review.opendev.org/c/openstack/neutron/+/929774 | 16:26 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: refactor: refactor: slightly more explicit return value https://review.opendev.org/c/openstack/neutron/+/929792 | 16:26 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: refactor: slightly more explicit return value https://review.opendev.org/c/openstack/neutron/+/929792 | 16:27 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: refactor: unindent some indents in _handle_lb_fip_cmds https://review.opendev.org/c/openstack/neutron/+/929793 | 16:27 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: refactor: don't calculate list of attached lbs for every lb https://review.opendev.org/c/openstack/neutron/+/929794 | 16:28 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: refactor: split out code to verify a lb member into a function https://review.opendev.org/c/openstack/neutron/+/929795 | 16:28 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: refactor: construct member dict in one go https://review.opendev.org/c/openstack/neutron/+/929796 | 16:28 |
| ihrachys | ykarel: wonder if you could give me a quick tip how to define a new job for mypy here: https://review.opendev.org/c/openstack/neutron/+/929864 zuul syntax is not my strongest side :) | 16:29 |
| haleyb | ihrachys: there was a neutron-lib patch, don't remember the status | 16:32 |
| ihrachys | yeah I thought I copied the thing but not really I guess; the setup of zuul jobs is not identical there I think, template names etc. | 16:32 |
| haleyb | ah i see you found it | 16:33 |
| ihrachys | I mean... I guess I can just learn how to test it locally, I'm just so demotivated to learn this thing... :) | 16:33 |
| ihrachys | haleyb: I was at one point trying to align some patches to make that neutron-lib thing going, but there was no interest. I'm doing this in neutron now 1) not to step on others' toes but make the ball rolling and 2) because I've seen simple bugs in the tree that wouldn't have happened if we'd do typing hints earlier | 16:34 |
| haleyb | i don't really know how to test it either | 16:37 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Fix a potential race in ovn idl initialization https://review.opendev.org/c/openstack/neutron/+/929811 | 17:43 |
| opendevreview | Merged openstack/networking-bagpipe master: [sqlalchemy-20] Remove CI job "networking-bagpipe-openstack-tox-py310-with-sqlalchemy-main" https://review.opendev.org/c/openstack/networking-bagpipe/+/929886 | 18:21 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.2: Change the load method of SG rule "default_security_group" https://review.opendev.org/c/openstack/neutron/+/929966 | 19:37 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.1: Change the load method of SG rule "default_security_group" https://review.opendev.org/c/openstack/neutron/+/929967 | 19:37 |
| ralonsoh | haleyb, hello! | 19:41 |
| ralonsoh | if you have some minutes: https://review.opendev.org/c/openstack/neutron-lib/+/927820 | 19:41 |
| ralonsoh | thanks in advance | 19:41 |
| ralonsoh | no rush, this will be for 2025.1 anyway | 19:41 |
| haleyb | ralonsoh: ack, added to my queue | 19:51 |
| opendevreview | Merged openstack/neutron master: Correct logic error when associating FIP with OVN LB https://review.opendev.org/c/openstack/neutron/+/921663 | 20:03 |
| opendevreview | Merged openstack/neutron-fwaas master: Account for iptables-save output spacing differences https://review.opendev.org/c/openstack/neutron-fwaas/+/929658 | 20:16 |
| opendevreview | Merged openstack/neutron master: Handle EndpointNotFound in nova notifier https://review.opendev.org/c/openstack/neutron/+/929920 | 22:21 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!