| opendevreview | Takashi Kajinami proposed openstack/octavia master: Add auth options to [neutron] section https://review.opendev.org/c/openstack/octavia/+/930470 | 00:55 |
|---|---|---|
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Add zookeeper backend for jobboard in devstack https://review.opendev.org/c/openstack/octavia/+/862671 | 06:00 |
| noonedeadpunk | oh, yes, would make sense to bump zookeeper version to latest stable | 06:45 |
| noonedeadpunk | btw I tried to double limits for RequestThrottler without any good result | 06:47 |
| noonedeadpunk | and also - this doesn't seem to affect basic functionality, just control plane is not redundant I assume, as there're nothing being published inside zookeper namespace. | 06:48 |
| noonedeadpunk | (but namespace is reliably re-created) | 06:48 |
| noonedeadpunk | (if I drop it) | 06:48 |
| gthiemonge | noonedeadpunk: the zookeeper jobs passed, but I see kazoo backtraces in the logs | 08:57 |
| gthiemonge | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_22c/862671/11/check/octavia-v2-dsvm-scenario-non-traffic-ops-jobboard-zookeeper/22c7ded/controller/logs/screen-o-cw.txt | 08:57 |
| gthiemonge | ERROR taskflow.jobs.backends.impl_zookeeper kazoo.exceptions.ConnectionLoss | 08:58 |
| noonedeadpunk | yup, exactly what I see as well. | 08:59 |
| gthiemonge | 2024-09-26 06:33:33,497 - WARN [NIOServerCxn.Factory:0.0.0.0/0.0.0.0:2181:NIOServerCnxn@383] - Exception causing close of session 0x100000d1b39002e: Connection reset by peer | 09:00 |
| noonedeadpunk | Oh. Can I make a DNM cherry-pick to 2023.1? As I'm quite sure I didn't see that before upgrade, but would be nice to confirm with CI | 09:00 |
| gthiemonge | yeah sure, maybe removes the non necessary jobs from zuul/project.yaml | 09:00 |
| noonedeadpunk | ++ sure | 09:01 |
| noonedeadpunk | actually I think your original version awas avbout for 2023.1 :D | 09:02 |
| noonedeadpunk | aha, and that's also with the old zookeeper | 09:02 |
| opendevreview | Seunghun Lee proposed openstack/octavia master: Fix verification of private CA signed certificates https://review.opendev.org/c/openstack/octavia/+/930219 | 09:05 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/octavia master: [DNM] Test zookeeper backend for 2023.1 https://review.opendev.org/c/openstack/octavia/+/862671 | 09:12 |
| noonedeadpunk | crap | 09:12 |
| noonedeadpunk | sorry | 09:12 |
| noonedeadpunk | I'm 100% sure I checked out on 2023.1 | 09:12 |
| noonedeadpunk | ah, I probably used checkout /o\ | 09:13 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/octavia master: Add zookeeper backend for jobboard in devstack https://review.opendev.org/c/openstack/octavia/+/862671 | 09:14 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/octavia stable/2023.1: [DNM] Add zookeeper backend for jobboard in devstack https://review.opendev.org/c/openstack/octavia/+/930544 | 09:19 |
| opendevreview | Merged openstack/octavia master: compat with cryptography 43 https://review.opendev.org/c/openstack/octavia/+/928889 | 14:17 |
| noonedeadpunk | gthiemonge: hey! sorry, have another question, maybe you have some recollections about octavia-dashboard as well?:) | 16:16 |
| noonedeadpunk | I was wondering what's this condition is actually doing? https://opendev.org/openstack/octavia-dashboard/blame/commit/7941df2f78153ab4b4c9452862df0fc3da780795/octavia_dashboard/static/dashboard/project/lbaasv2/workflow/pool/pool.html#L60 | 16:16 |
| noonedeadpunk | as it feels that the purpose of it to disable protocol selection for Pool on LB creation page | 16:17 |
| noonedeadpunk | so to create PROXYV2, for isntance, you have to create pool separately from rest of things | 16:17 |
| gthiemonge | noonedeadpunk: hmm it seems the I only changed the indentation of this chunk | 16:23 |
| gthiemonge | yeah it's weird, the pool protocol selection is disabled in the "Create Load Balancer" form | 16:27 |
| johnsom | Yeah, I don't know on that either | 16:29 |
| johnsom | It's something Jacky added six years ago | 16:31 |
| noonedeadpunk | hehe | 16:32 |
| noonedeadpunk | I dropped that if and proto appeared | 16:33 |
| noonedeadpunk | I assume idea was to match listener to smth, or dunno... | 16:33 |
| noonedeadpunk | Will suggest patch then :) | 16:33 |
| johnsom | Right, that is my guess as well, that it would automatically pick a pool proto | 16:34 |
| noonedeadpunk | like proxy proto is a corner case? | 16:34 |
| noonedeadpunk | which was added way after original code | 16:35 |
| johnsom | It may not have existed back then | 16:35 |
| johnsom | Either way, letting people pick is the right path forward | 16:35 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/octavia-dashboard master: Enable protocol selection for Pool during LB creation https://review.opendev.org/c/openstack/octavia-dashboard/+/930616 | 16:40 |
| opendevreview | Michael Johnson proposed openstack/octavia-tempest-plugin master: Use the same ssl module httpx uses https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/893607 | 16:44 |
| noonedeadpunk | btw, am I right that due to no meaningful horizon dashboard for barbican it's basically not possible to create tls-terminated LB over horizon? | 16:50 |
| noonedeadpunk | as there're tls_ciphers field, but actually no way to supply chain or barbican secret? | 16:57 |
| noonedeadpunk | ah. damn, stupid me :) | 17:01 |
| noonedeadpunk | I'm blind, haven't spotted another meny which kicks off when terminated https is in | 17:02 |
| opendevreview | Merged openstack/octavia master: Fixed retry functions with tenacity 8.5.0 https://review.opendev.org/c/openstack/octavia/+/924126 | 17:07 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!