| opendevreview | Merged openstack/networking-bagpipe master: pyupgrade changes for Python3.9+ https://review.opendev.org/c/openstack/networking-bagpipe/+/933725 | 01:15 |
|---|---|---|
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: enable for most modules https://review.opendev.org/c/openstack/neutron/+/929866 | 02:02 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: DNM: run all ovs connections as leader-only=false https://review.opendev.org/c/openstack/neutron/+/921461 | 02:05 |
| opendevreview | Takashi Kajinami proposed openstack/os-vif master: Fix missing netifaces dependency https://review.opendev.org/c/openstack/os-vif/+/934197 | 02:25 |
| opendevreview | Merged openstack/neutron stable/2024.1: [OVN] Fix the revision number retrieval method https://review.opendev.org/c/openstack/neutron/+/934024 | 02:31 |
| opendevreview | Merged openstack/neutron stable/2023.2: [OVN] Fix the revision number retrieval method https://review.opendev.org/c/openstack/neutron/+/934025 | 02:31 |
| opendevreview | Merged openstack/neutron stable/2023.1: [OVN] Fix the revision number retrieval method https://review.opendev.org/c/openstack/neutron/+/934026 | 02:45 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: enable for all non-test modules https://review.opendev.org/c/openstack/neutron/+/929866 | 03:21 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.2: [OVN] Check LSP.up status before setting the port host info https://review.opendev.org/c/openstack/neutron/+/934201 | 07:09 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.1: [OVN] Check LSP.up status before setting the port host info https://review.opendev.org/c/openstack/neutron/+/934202 | 07:09 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: [OVN] Check LSP.up status before setting the port host info https://review.opendev.org/c/openstack/neutron/+/934203 | 07:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: [OVN] Check LSP.up status before setting the port host info https://review.opendev.org/c/openstack/neutron/+/934204 | 07:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Create the SG rules revision number registers https://review.opendev.org/c/openstack/neutron/+/933969 | 07:11 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] The Port_Group deletion implies the ACLs deletion https://review.opendev.org/c/openstack/neutron/+/933877 | 07:11 |
| ralonsoh | slaweq, hello! Please check https://review.opendev.org/c/openstack/neutron/+/934191 if you have 1 minute | 07:22 |
| ralonsoh | thanks! | 07:22 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: DNM: A glorious fight against excessive else branches https://review.opendev.org/c/openstack/neutron/+/934205 | 07:34 |
| opendevreview | Merged openstack/neutron master: Remove dhcp.PORTS attribute https://review.opendev.org/c/openstack/neutron/+/931993 | 07:42 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == [OVN] L3 plugin support for floating IP distributed attribues https://review.opendev.org/c/openstack/neutron/+/856955 | 07:53 |
| ralonsoh | haleyb, hello! if you have some minutes, please check https://review.opendev.org/c/openstack/neutron/+/933877 (and the parent patch) | 08:00 |
| ralonsoh | I need to rebase both once merged | 08:00 |
| ralonsoh | s/rebase/backport | 08:00 |
| opendevreview | liuyulong proposed openstack/neutron master: Add basical functionalities for metadata path extension https://review.opendev.org/c/openstack/neutron/+/881535 | 09:01 |
| opendevreview | liuyulong proposed openstack/neutron master: Add metadata path extension openflows https://review.opendev.org/c/openstack/neutron/+/888097 | 09:01 |
| opendevreview | liuyulong proposed openstack/neutron master: Fullstack case for metadata path https://review.opendev.org/c/openstack/neutron/+/888098 | 09:01 |
| opendevreview | liuyulong proposed openstack/neutron master: Add devstack plugin to enable ovs metadata_path https://review.opendev.org/c/openstack/neutron/+/928586 | 09:01 |
| opendevreview | liuyulong proposed openstack/neutron master: WIP: Fullstack case for metadata path with real curl https://review.opendev.org/c/openstack/neutron/+/934102 | 09:04 |
| yosef | Hi, there is a default security group, always will be created if you delete it, where does it get created? is there anyway to disable recreation of default sg? | 09:08 |
| ralonsoh | yosef, it is created when the first port is created on a network | 09:09 |
| ralonsoh | no, the default group is mandatory always | 09:09 |
| yosef | Thanks, it is interesting even listing sgs on an empty project will recreate it | 09:10 |
| ralonsoh | https://github.com/openstack/neutron/blob/master/neutron/db/securitygroups_db.py#L1142 | 09:12 |
| ralonsoh | also before the creation of a network in a project | 09:12 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: nit: Drop mention of postgresql https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/934174 | 09:45 |
| opendevreview | Merged openstack/neutron master: style: Clarify a check for mixed stateful/stateless SGs https://review.opendev.org/c/openstack/neutron/+/924087 | 09:45 |
| opendevreview | Lajos Katona proposed openstack/python-neutronclient master: vpnaas: show external_vx_ip for vpn service show https://review.opendev.org/c/openstack/python-neutronclient/+/934215 | 09:51 |
| s3rj1k | Hi all, can somebody look at the story here https://storyboard.openstack.org/#!/story/2011258? | 10:54 |
| ralonsoh | s3rj1k, we no longer use storyboard for bug/rfes in Neutron | 11:14 |
| ralonsoh | s3rj1k, please use https://bugs.launchpad.net/neutron | 11:14 |
| opendevreview | Takashi Kajinami proposed openstack/os-vif master: Fix missing netifaces dependency https://review.opendev.org/c/openstack/os-vif/+/934197 | 11:15 |
| s3rj1k | <ralonsoh> got it, thanks | 11:42 |
| opendevreview | Lajos Katona proposed openstack/neutron-vpnaas master: pyupgrade changes for Python3.9+ https://review.opendev.org/c/openstack/neutron-vpnaas/+/934232 | 11:48 |
| opendevreview | Merged openstack/neutron stable/2024.2: [OVN] Fix the revision number retrieval method https://review.opendev.org/c/openstack/neutron/+/934023 | 11:52 |
| s3rj1k | submitted to https://bugs.launchpad.net/neutron/+bug/2086776 | 12:10 |
| opendevreview | Lajos Katona proposed openstack/neutron-fwaas master: pre-commit: update versions https://review.opendev.org/c/openstack/neutron-fwaas/+/934238 | 12:58 |
| opendevreview | Lajos Katona proposed openstack/neutron-fwaas master: pyupgrade changes for Python3.9+ https://review.opendev.org/c/openstack/neutron-fwaas/+/934239 | 12:58 |
| opendevreview | Stephen Finucane proposed openstack/os-vif master: mypy: Add type annotations https://review.opendev.org/c/openstack/os-vif/+/745162 | 13:11 |
| opendevreview | Stephen Finucane proposed openstack/os-vif master: Integrate pre-commit https://review.opendev.org/c/openstack/os-vif/+/934240 | 13:11 |
| yosef | Hi everyone, a bit confused about vpnaas service, any insights? What is the difference between vpnaas and a simple private network? can you attach multiple subnets from different openstack clusters usings vpnaas? can this be interpreted same as the solution ovn interconnect introduces? and can you use it standalone? one side openstack and one side some other service like kubernetes? | 13:15 |
| opendevreview | Sharat Sharma proposed openstack/neutron master: Fix a spelling mistake in checks.py https://review.opendev.org/c/openstack/neutron/+/934241 | 13:25 |
| noonedeadpunk | yosef: yes, pretty much. basically you can connect any infra outside of openstack with a tenant network in openstack. | 14:09 |
| noonedeadpunk | one way of doing a hybrid setups between different cloud providers | 14:09 |
| noonedeadpunk | eventually I got a question as well.... | 14:09 |
| noonedeadpunk | has anybody seen a neutron-server (running with eventlet) consuming 100% of CPU constantly on all threads? | 14:10 |
| noonedeadpunk | and when I do strace to the process - I see it's really occupied with connections to OVN SB DB | 14:11 |
| noonedeadpunk | hm... an be actually related to ovn-octavia-provider as load gone with restart of octavia.... | 14:15 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == [OVN] L3 plugin support for floating IP distributed attribues https://review.opendev.org/c/openstack/neutron/+/856955 | 14:30 |
| ralonsoh | noonedeadpunk, can you identify what ovn-octavia-provider is doing? if this is stuck in any method | 14:31 |
| noonedeadpunk | ralonsoh: probably it's unrelated to octavia. As it seems that why load left - is because ovn nb db died | 14:36 |
| noonedeadpunk | and it for some reason attempts to bind to 6642 where sb db is running.... | 14:37 |
| noonedeadpunk | so I guess I messed up smth at the end, but it's actually second time this is happening | 14:39 |
| noonedeadpunk | on 2024.1 jsut in case. | 14:39 |
| noonedeadpunk | previous time I was able to solve that jsut after re-initiating ovn cluster and filling in db with neutron-ovn-db-sync-util afterwards | 14:40 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: OVN metadata agent additional_chassis detection https://review.opendev.org/c/openstack/neutron/+/934188 | 14:42 |
| haleyb | ralonsoh: can you check my comment in https://review.opendev.org/c/openstack/neutron/+/934241 ? imo we can remove this old cruft | 15:01 |
| ralonsoh | haleyb, let me check | 15:02 |
| ralonsoh | haleyb, ufff, this code should be removed now | 15:03 |
| haleyb | ralonsoh: thanks for confirming, when i saw Train i groaned | 15:03 |
| ykarel | elodilles, lajoskatona bcafarel if you can re workflow https://review.opendev.org/c/openstack/devstack/+/933738 , just moved ironic job nonvoting | 15:03 |
| * bcafarel looking | 15:04 | |
| ykarel | also yoga one https://review.opendev.org/c/openstack/devstack/+/933739 | 15:07 |
| lajoskatona | ykarel: thanks, looking | 15:07 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: OVN metadata agent additional_chassis detection https://review.opendev.org/c/openstack/neutron/+/934188 | 15:08 |
| opendevreview | Sharat Sharma proposed openstack/neutron master: The ovn_db_revision_check is not required anymore https://review.opendev.org/c/openstack/neutron/+/934241 | 15:13 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: [Fullstack] Use only one keepalived-state-change thread in L3 agent https://review.opendev.org/c/openstack/neutron/+/934248 | 15:14 |
| slaweq | haleyb ykarel ralonsoh ^^ small improvement of the resources usage in the fullstack tests, please check when You will have a minute | 15:14 |
| ralonsoh | sure | 15:15 |
| opendevreview | Sharat Sharma proposed openstack/neutron master: The ovn_db_revision_check is not required anymore https://review.opendev.org/c/openstack/neutron/+/934241 | 15:18 |
| noonedeadpunk | so once I got nb db running back - neutron-server resumed consuming 100% CPU.... | 15:19 |
| noonedeadpunk | and nothing on debug logs or anything | 15:19 |
| ralonsoh | noonedeadpunk, is the API reactive? can you do something in the CLI? "port list", for example | 15:21 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: OVN metadata agent additional_chassis detection https://review.opendev.org/c/openstack/neutron/+/934188 | 15:21 |
| noonedeadpunk | yup, and moreover it respect commands | 15:21 |
| noonedeadpunk | so I can assing fips, create ports, etc | 15:22 |
| noonedeadpunk | nothing is obviously wrong | 15:22 |
| ralonsoh | noonedeadpunk, I would suggest to limit the servers to one and reduce the neutron API to 1 thread, and try to debug this code | 15:22 |
| ralonsoh | trying to see what method is taking more time, thread, etc | 15:23 |
| ralonsoh | *API to 1 worker (not thread) | 15:23 |
| noonedeadpunk | I'm like 99% sure it's related with some sb db state.... I actually saw in logs that nodes exist in sb db, but not in neutron database | 15:25 |
| noonedeadpunk | wouldn't expect it to cause issues though | 15:26 |
| noonedeadpunk | also interesting that it involves RPCWorkers as well | 15:28 |
| noonedeadpunk | strace -p spamming with the same connection attempt to same sbdb host | 15:29 |
| noonedeadpunk | and then I see `EAGAIN (Resource temporarily unavailable)` in starce from time to time as well. | 15:29 |
| noonedeadpunk | but I can telnet easily.... | 15:30 |
| noonedeadpunk | and if I shut down this worker - another IP starts being used | 15:33 |
| noonedeadpunk | in case none available - load just goes away | 15:34 |
| noonedeadpunk | in another environment with mostly identical setup I've never seen anything like that | 15:38 |
| noonedeadpunk | interestingly, octavia-driver-agent - provider_agent -- ovn also consume 100% of cpu.... | 15:55 |
| noonedeadpunk | also would be nice to have some review on https://review.opendev.org/c/openstack/neutron/+/931495 :) | 16:24 |
| opendevreview | Brian Haley proposed openstack/neutron master: Start enforcing additional pylint checks https://review.opendev.org/c/openstack/neutron/+/934193 | 17:14 |
| noonedeadpunk | huh. once I removed all OVN LBs - neutron-server load has dissapeared.... | 17:18 |
| haleyb | noonedeadpunk: what there something possibly out of sync? i.e. running the neutron-ovn-db-sync-util showed something? just a thought | 17:41 |
| noonedeadpunk | ran it but changed nothing | 17:50 |
| noonedeadpunk | yeah, Iwas also thinking smth it out of sync | 17:50 |
| noonedeadpunk | why I actualy started looking into that, is because FIP<>VIP binding for OVN LB wasn't working | 17:51 |
| opendevreview | Merged openstack/neutron master: tests: always raise an explanation when status_int is wrong https://review.opendev.org/c/openstack/neutron/+/934162 | 18:45 |
| opendevreview | Merged openstack/neutron master: gate: squash mypy into pep8 job https://review.opendev.org/c/openstack/neutron/+/934191 | 19:36 |
| opendevreview | Merged openstack/neutron master: [OVN] Create the SG rules revision number registers https://review.opendev.org/c/openstack/neutron/+/933969 | 19:38 |
| opendevreview | Merged openstack/neutron-lib master: nit: Drop mentions of postgresql https://review.opendev.org/c/openstack/neutron-lib/+/934173 | 20:50 |
| opendevreview | Merged openstack/neutron master: Enable pylint useless-super-delegation check https://review.opendev.org/c/openstack/neutron/+/934192 | 21:02 |
| opendevreview | Merged openstack/neutron master: Add migration_cli.py to mypy exclude list https://review.opendev.org/c/openstack/neutron/+/932927 | 21:07 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Remove prefix delegation code https://review.opendev.org/c/openstack/neutron/+/934283 | 22:55 |
| opendevreview | Brian Haley proposed openstack/neutron master: Increase code coverage for OVN metadata port sync https://review.opendev.org/c/openstack/neutron/+/914922 | 22:58 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: tests: move test_db_base_plugin_v2.py to common https://review.opendev.org/c/openstack/neutron/+/934190 | 23:02 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Fix dependencies for mypy tox target https://review.opendev.org/c/openstack/neutron/+/934284 | 23:15 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: enable for all non-test modules https://review.opendev.org/c/openstack/neutron/+/929866 | 23:21 |
| ihrachys | haleyb: q: was https://review.opendev.org/c/openstack/neutron/+/932927 failing in gate? or just locally in a particular distro version? | 23:21 |
| ihrachys | afaiu noble == python3.12 while gate uses jammy, 3.10. maybe that's the difference. | 23:25 |
| ihrachys | yeah indeed; I can reproduce it in py312 tox env but not py311 with exact same version of mypy | 23:34 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: enable for all non-test modules https://review.opendev.org/c/openstack/neutron/+/929866 | 23:41 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: enable for all non-test modules https://review.opendev.org/c/openstack/neutron/+/929866 | 23:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!