| opendevreview | Sergey Kraynev proposed openstack/octavia stable/2023.2: Handle empty delay on update healthmonitor https://review.opendev.org/c/openstack/octavia/+/915360 | 03:22 |
|---|---|---|
| opendevreview | Sergey Kraynev proposed openstack/octavia master: Add option to limit recursion in to_data_model https://review.opendev.org/c/openstack/octavia/+/883063 | 03:23 |
| skraynev | gthiemonge: could you please help with +2 for backport patches: https://review.opendev.org/c/openstack/octavia/+/915521 for all stable branches? and that you for the help with previous patches ;) | 06:19 |
| opendevreview | Takashi Kajinami proposed openstack/octavia-tempest-plugin master: Add stable/2024.1 jobs https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/915853 | 07:02 |
| opendevreview | Takashi Kajinami proposed openstack/octavia-tempest-plugin master: Add stable/2024.1 jobs https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/915853 | 07:02 |
| gthiemonge | skraynev: done | 07:10 |
| skraynev | gthiemonge: thank you. one more note: for PR https://review.opendev.org/c/openstack/octavia/+/915521 it's already second +2, could you help with adding "workflow" also, please? | 07:14 |
| tkajinam | skraynev, it shouldn't be merged until its backports to 2024.1 and 2023.2 are merged | 07:19 |
| skraynev | ok. no problem. I thought, that order does not matter. so then, I will wait. | 07:20 |
| tkajinam | technically order is not enforced by gerrit but I'd not recommend breaking it. We in the past hit multiple problems caused by the broken order | 07:21 |
| skraynev | tkajinam: sounds reasonable. it's not a problem to wait. now I just need to find second +2 for other backports :) | 07:27 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Updates for new Python runtimes in 2024.2 https://review.opendev.org/c/openstack/octavia/+/915970 | 07:32 |
| opendevreview | Merged openstack/octavia stable/2023.2: Fix negative or 0 limit parameter in pagination https://review.opendev.org/c/openstack/octavia/+/915538 | 07:38 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Updates for new Python runtimes in 2024.2 https://review.opendev.org/c/openstack/octavia/+/915970 | 07:40 |
| tweining | hey, is it my IRC client only, or do new changes no longer get announced here? I just pushed a change. | 07:43 |
| gthiemonge | tweining: opendevreview sent 2 messages for your pushes at 9:32 and 9:40 CEST | 07:47 |
| tweining | ok, so it's my client/Matrix then | 07:48 |
| tkajinam | tweining, seems you connection is unstable ? | 08:01 |
| tkajinam | according to my hexchat log, the bot sent notifications about your changes at 16:32:36 and 16:40:58 and you joined the channel after these (at 16:43:06) | 08:01 |
| tweining | yeah, I use the Matrix IRC bridge and unfortunately it doesn't seem to tell me when the connection is lost | 08:01 |
| tweining | I assume it reconnected/joined when I sent my message then. still, that is not good. | 08:04 |
| tkajinam | yeah | 08:19 |
| tkajinam | tweining, fyi. we need https://review.opendev.org/c/openstack/releases/+/915936 to unblock jobboard jobs with SQLAlchemy 2.0 | 08:19 |
| tweining | ah, ok. | 08:24 |
| manuvakery1 | Hi, is it possible to add pool members from different project? | 08:58 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Updates for new Python runtimes in 2024.2 https://review.opendev.org/c/openstack/octavia/+/915970 | 09:00 |
| tweining | manuvakery1: AFAIK there is no restriction. | 09:06 |
| manuvakery1 | tweining: these members are using different subnet | 09:07 |
| tweining | in that case health monitoring will probably not work | 09:12 |
| gthiemonge | manuvakery1: when a user adds a member, octavia validates that the user can use the subnet (if subnet_id is passed), if subnet_id belongs to another project and is not available/visible to the user, the Octavia API returns an error | 09:15 |
| manuvakery1 | gthiemonge: I added the pool member using terraform by providing only the ip address and port | 09:17 |
| manuvakery1 | the member is added but its in error state | 09:19 |
| tweining | yeah, the subnet_id is optional | 09:19 |
| tweining | the documentation of the api says "The member address does not necessarily need to be a member of the subnet_id subnet. Members can be routable from the subnet specified either via the default route or by using host_routes defined on the subnet." | 09:21 |
| tweining | https://docs.openstack.org/api-ref/load-balancer/v2/index.html#create-member | 09:21 |
| manuvakery1 | ok.. As per this its possible to to add members from other projects | 09:24 |
| gthiemonge | right, you only need a route to the subnet from one of the already plugged subnet (for instance the VIP) | 09:38 |
| opendevreview | Merged openstack/octavia stable/2023.1: Fix negative or 0 limit parameter in pagination https://review.opendev.org/c/openstack/octavia/+/915537 | 09:41 |
| opendevreview | Merged openstack/octavia stable/2023.2: Handle empty delay on update healthmonitor https://review.opendev.org/c/openstack/octavia/+/915360 | 09:41 |
| opendevreview | Merged openstack/octavia stable/2023.1: Handle empty delay on update healthmonitor https://review.opendev.org/c/openstack/octavia/+/915521 | 09:41 |
| skraynev | tweining: could you please add +2 and workflow for the last backport patch https://review.opendev.org/c/openstack/octavia/+/915359 ? I suppose it was accidentally missed. | 09:49 |
| tweining | skraynev: oops, yes. done | 09:50 |
| skraynev | thank you | 09:51 |
| opendevreview | Merged openstack/octavia stable/zed: Handle empty delay on update healthmonitor https://review.opendev.org/c/openstack/octavia/+/915522 | 10:01 |
| opendevreview | Merged openstack/octavia master: SQLAlchemy 2.0: Drop use of removed autoload https://review.opendev.org/c/openstack/octavia/+/915772 | 10:18 |
| opendevreview | Merged openstack/octavia stable/2024.1: Handle empty delay on update healthmonitor https://review.opendev.org/c/openstack/octavia/+/915359 | 12:02 |
| skraynev | tweining: gthiemonge : thank you for the merging all backport patches with fixes. the last request from my side: could you please review: https://review.opendev.org/c/openstack/octavia/+/883063 ? it waits some feedback | 12:15 |
| gthiemonge | skraynev: I will, but it may take some time to test/check/review it | 12:15 |
| skraynev | gthiemonge: awesome, please do not rush with it. I just wanted to remind about it | 12:19 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!