Tuesday, 2024-10-29

skraynevHello, I added release notes to PRs with bug fixes, could someone please review: https://review.opendev.org/c/openstack/octavia/+/933101 and https://review.opendev.org/c/openstack/octavia/+/933254 07:07
gthiemongeskraynev: added a comment to the first one07:10
skraynev@gthiemonge : thx. Actually I didn't think about it, because it was the old format before fix. potentially I could remove the old error message and print always the version without db_lb. wdyt ? another option to display ip_address, vip_subnet, vip_network from VIP db object.07:17
gthiemongeskraynev: IMHO a straightforward approach is to display the id of the LB (loadbalancer[constants.LOADBALANCER_ID]) which is the unique identifier of the VIP07:22
skraynevHm. It will solve the issue totally. without db_lb in message it will be a simple fix in error message. I will update PR07:30
gthiemongeskraynev: maybe add result[constants.VRRP_PORT_ID] too?07:33
skraynevsure, it's possible07:34
opendevreviewSergey Kraynev proposed openstack/octavia master: Do not fail on revert PlugVIPAmphora due undefined db_lb  https://review.opendev.org/c/openstack/octavia/+/93310108:12
skraynev@gthiemonge : updated, could you please take a look? I add info about VRRP_PORT_ID and want to make sure, that it matches with your expectations08:14
opendevreviewDenys Mishchenko proposed openstack/octavia master: Add spec for multiple AZ amphora placement  https://review.opendev.org/c/openstack/octavia/+/92357111:23
skraynev@gthiemonge comments are fixed and Pr are ready for review: https://review.opendev.org/c/openstack/octavia/+/933254 , https://review.opendev.org/c/openstack/octavia/+/93310112:21
servagemHello, I am trying to understand the default value for the listener connection limit option. IIUC, the default is set to -1, which represents an unlimited number of connections. However, when using HAProxy (in Amphorae), the default is set to 50000 connections instead. Is that correct?17:41
tkajinamhttps://github.com/openstack/octavia/blob/fd071d3391e5695e50748f74af4529186d63e472/octavia/common/config.py#L428-L43017:51
tkajinamservagem, probably this answers you question ? ^^^17:51
tkajinamhttps://docs.openstack.org/api-ref/load-balancer/v2/#id3517:52
tkajinamconnection_limit (Optional)17:52
tkajinam17:52
tkajinambody17:52
tkajinam17:52
tkajinaminteger17:52
tkajinam17:52
tkajinamThe maximum number of connections permitted for this listener. Default value is -1 which represents infinite connections or a default value defined by the provider driver.17:52
johnsomThis changed over time. HAProxy changed memory allocation, so we had to drop the -1 to 50,000 by default.17:54
johnsomI guess not all of the docs got updated17:55
servagemWhat confused me is the fact that it depends on the provider. For example, according to the API reference, it states 'unlimited,' but for the Amphora provider driver, the limit is 50k.17:56
servagemso, I am not sure if that is really expected or it is missing update the docs to 50k17:56
johnsomYes, that is true. It is provider specific17:57
servagemyep, got it18:03
servagemthank you tkajinam and johnsom 18:06
opendevreviewMerged openstack/octavia-tempest-plugin master: Remove support for Pyton 3.8 and older  https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/93327018:59
opendevreviewMichael Johnson proposed openstack/octavia master: Make keystone default roles the default RBAC  https://review.opendev.org/c/openstack/octavia/+/92958021:11

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