Tuesday, 2024-05-14

opendevreviewMerged openstack/octavia stable/2023.2: Removing tips jobs on stable/2023.2  https://review.opendev.org/c/openstack/octavia/+/91804511:23
opendevreviewTom Weininger proposed openstack/octavia stable/2023.1: Removing tips jobs on stable/2023.1  https://review.opendev.org/c/openstack/octavia/+/91804711:46
opendevreviewTom Weininger proposed openstack/octavia stable/2023.1: Fix zuul error about invalid regex  https://review.opendev.org/c/openstack/octavia/+/91956411:46
opendevreviewTom Weininger proposed openstack/octavia stable/2023.1: Removing tips jobs on stable/2023.1  https://review.opendev.org/c/openstack/octavia/+/91804711:51
opendevreviewMerged openstack/octavia stable/2023.2: Fix neutron setting overrides  https://review.opendev.org/c/openstack/octavia/+/91767312:54
opendevreviewMerged openstack/octavia stable/2023.2: Fix incorrect masquerade rules in multivip LBs  https://review.opendev.org/c/openstack/octavia/+/91767413:32
danfaiHi, before I open a launchpad entry, I thought I ask here: Are there plans of adding agent-check functionality to healthmonitors in octavia for some more advanced backend management? e.g. https://www.haproxy.com/documentation/haproxy-configuration-tutorials/service-reliability/health-checks/#agent-checks 13:44
opendevreviewGregory Thiemonge proposed openstack/octavia stable/2023.1: Remove grenade jobs [stable/2023.1]  https://review.opendev.org/c/openstack/octavia/+/91957213:44
gthiemongedanfai: Hi, there's no plan for the moment, but that sounds like an interesting feature13:46
danfaiok thank you, I'll create a feature request in launchpad then13:54
opendevreviewTakashi Kajinami proposed openstack/octavia master: Use built-in json module  https://review.opendev.org/c/openstack/octavia/+/91727414:04
opendevreviewMerged openstack/octavia stable/2023.2: Fix health monitor information retrieval in API response  https://review.opendev.org/c/openstack/octavia/+/91767614:06
skraynevdanfai: hi. I am just wondering, what is the key difference this functionality from existing health monitors? AFAIK HM right now allows to send HTTP and validate expected return code. I just read the head of doc, which you referenced, so maybe I missed something important14:11
danfaiskraynev: the benefit is you can tell the haproxy e.g. how much load to put on the backends, to drain a member more slowly or in general have a more fine grained status reporting, rather than UP/DOWN14:18
skraynevdanfai: got it. thank you for explanation. now it makes sense for me14:20
opendevreviewTom Weininger proposed openstack/octavia master: Add typing information to remaining octavia code  https://review.opendev.org/c/openstack/octavia/+/90519514:22
opendevreviewTom Weininger proposed openstack/octavia master: Integrate mypy type checker  https://review.opendev.org/c/openstack/octavia/+/87974914:22
fricklerlots of pep8 failures in gate, not sure how that can happen14:27
johnsomfrickler Do you have a link?14:28
fricklerhttps://zuul.opendev.org/t/openstack/build/163c0e758dea4ec5bbb5ce4c4101583a for example14:29
frickleror more general https://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-pep8&project=openstack%2Foctavia&pipeline=gate&skip=014:30
johnsomThanks, I will have a look. We do leave the linters un-capped, so it's possible a linter updated14:30
skraynevdanfai: I figured out what happened. I open page and it was not redirected to mentioned section. And I read about health checks instead of agent-checks =)14:30
danfaiskraynev: yeah, "check" without agent is already supported :)14:31
tkajinamfrickler, johnsom seems like these are new errors from pylint. I guess pylint got a new release recently14:37
fricklerah, 3.2.0 released today, that sounds plausible14:38
fricklerI was just watching the gate pipeline because the sqla2.0 bump merged earlier and got a bit nervous, but then that's not related :)14:39
opendevreviewTakashi Kajinami proposed openstack/octavia master: Fix new pylint errors  https://review.opendev.org/c/openstack/octavia/+/91958714:39
johnsomYep, we will push a fix today14:39
johnsomOh, beat me to it14:39
tkajinam;-)14:41
johnsomtkajinam Thank you!14:41
tkajinamjohnsom, I'm wondering what would be the appropriate steps to remove upstart/sysvinit support. We can technically add deprecation warning but the init system is detected in amphora instances so the deprecation warning may appear only in amphora. I'm not too sure if operators may catch such logs.14:43
tkajinamprobably we can update the dib element to reject any operating systems which do not use systemd14:43
johnsomlol, I just poked Greg about: https://review.opendev.org/c/openstack/octavia/+/76291514:43
johnsomThere are not "supported" distros that use either sysvinit or upstart any longer14:44
tkajinamok then we can just directly remove these14:44
tkajinamtouching these init system switch always makes me think about that removal :-)14:45
johnsomYeah, they are long overdue to just remove14:46
tkajinamyeah14:47
opendevreviewMerged openstack/octavia stable/2023.2: Fix DB deadlock in quotas  https://review.opendev.org/c/openstack/octavia/+/91767815:05
opendevreviewMerged openstack/octavia stable/2023.2: When we failed to load pkcs12 cert print warning  https://review.opendev.org/c/openstack/octavia/+/91767915:06
opendevreviewTom Weininger proposed openstack/octavia master: Add typing information to remaining octavia code  https://review.opendev.org/c/openstack/octavia/+/90519515:51
opendevreviewTom Weininger proposed openstack/octavia master: Integrate mypy type checker  https://review.opendev.org/c/openstack/octavia/+/87974915:51
opendevreviewTom Weininger proposed openstack/octavia master: Add typing information to remaining octavia code  https://review.opendev.org/c/openstack/octavia/+/90519515:54
opendevreviewTom Weininger proposed openstack/octavia master: Integrate mypy type checker  https://review.opendev.org/c/openstack/octavia/+/87974915:54
opendevreviewTom Weininger proposed openstack/octavia master: Add typing information to octavia.api package  https://review.opendev.org/c/openstack/octavia/+/90519416:34
opendevreviewTom Weininger proposed openstack/octavia master: Add typing information to remaining octavia code  https://review.opendev.org/c/openstack/octavia/+/90519516:34
opendevreviewTom Weininger proposed openstack/octavia master: Integrate mypy type checker  https://review.opendev.org/c/openstack/octavia/+/87974916:34
opendevreviewMichael Johnson proposed openstack/octavia master: Fix listener update when using SRIOV VIP  https://review.opendev.org/c/openstack/octavia/+/91954621:37

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!