| openstackgerrit | Michael Johnson proposed openstack/octavia master: Fix PROXYV2 pools https://review.opendev.org/756934 | 00:11 |
|---|---|---|
| openstackgerrit | Michael Johnson proposed openstack/python-octaviaclient master: Add support for PROXYV2 protocol https://review.opendev.org/756936 | 00:14 |
| johnsom | Opps | 00:15 |
| johnsom | I missed a few things | 00:15 |
| *** spatel has joined #openstack-lbaas | 00:23 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia-tempest-plugin master: Fix scenario tests issue with immutable LBs https://review.opendev.org/755777 | 00:24 |
| *** spatel has quit IRC | 00:28 | |
| *** zzzeek has quit IRC | 01:10 | |
| *** zzzeek has joined #openstack-lbaas | 01:11 | |
| *** armax has joined #openstack-lbaas | 01:29 | |
| *** armax has quit IRC | 02:00 | |
| openstackgerrit | lkk proposed openstack/octavia-tempest-plugin master: test amphora stats https://review.opendev.org/756975 | 02:22 |
| openstackgerrit | lkk proposed openstack/octavia-tempest-plugin master: test amphora stats https://review.opendev.org/756980 | 02:44 |
| *** psachin has joined #openstack-lbaas | 03:07 | |
| *** armax has joined #openstack-lbaas | 03:07 | |
| *** rcernin has quit IRC | 03:17 | |
| *** psachin has quit IRC | 03:40 | |
| *** armax has quit IRC | 03:40 | |
| *** psachin has joined #openstack-lbaas | 03:41 | |
| *** rcernin has joined #openstack-lbaas | 03:55 | |
| *** rcernin has quit IRC | 03:56 | |
| *** rcernin has joined #openstack-lbaas | 04:03 | |
| *** xgerman has quit IRC | 04:09 | |
| *** vishalmanchanda has joined #openstack-lbaas | 04:35 | |
| *** zzzeek has quit IRC | 05:02 | |
| *** zzzeek has joined #openstack-lbaas | 05:05 | |
| *** zzzeek has quit IRC | 05:42 | |
| *** zzzeek has joined #openstack-lbaas | 05:43 | |
| *** rpittau|afk is now known as rpittau | 05:43 | |
| johnsom | Welp, that didn’t help | 05:44 |
| openstackgerrit | lkk proposed openstack/octavia-tempest-plugin master: test delete amphora and remove unused local variable https://review.opendev.org/757002 | 05:56 |
| *** zzzeek has quit IRC | 06:11 | |
| *** zzzeek has joined #openstack-lbaas | 06:12 | |
| openstackgerrit | lkk proposed openstack/octavia-tempest-plugin master: test amphora stats https://review.opendev.org/756980 | 06:13 |
| *** gcheresh has joined #openstack-lbaas | 06:31 | |
| *** tkajinam has quit IRC | 06:42 | |
| *** tkajinam has joined #openstack-lbaas | 06:42 | |
| *** armax has joined #openstack-lbaas | 07:24 | |
| *** rcernin has quit IRC | 07:45 | |
| *** sapd1 has joined #openstack-lbaas | 07:50 | |
| *** gcheresh has quit IRC | 08:04 | |
| *** gcheresh has joined #openstack-lbaas | 08:14 | |
| *** ccamposr has joined #openstack-lbaas | 08:40 | |
| *** ccamposr__ has quit IRC | 08:43 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/octavia-dashboard stable/victoria: Imported Translations from Zanata https://review.opendev.org/757027 | 08:47 |
| openstackgerrit | Ann Taraday proposed openstack/octavia master: Fix amphorav2 bytes error https://review.opendev.org/738407 | 09:15 |
| openstackgerrit | Gregory Thiemonge proposed openstack/octavia-tempest-plugin master: Fix scenario tests issue with immutable LBs https://review.opendev.org/755777 | 09:19 |
| *** takamatsu has joined #openstack-lbaas | 09:37 | |
| openstackgerrit | lkk proposed openstack/octavia-tempest-plugin master: delete amphora and remove unused local variable https://review.opendev.org/757002 | 09:42 |
| gthiemonge | ataraday: we need to merge https://review.opendev.org/#/c/755777/ before merging https://review.opendev.org/#/c/753838/ | 09:44 |
| openstackgerrit | lkk proposed openstack/octavia-tempest-plugin master: wait for the loadbalancer to active in healthmonitor case https://review.opendev.org/757057 | 09:48 |
| *** rcernin has joined #openstack-lbaas | 10:50 | |
| *** zzzeek has quit IRC | 10:52 | |
| *** zzzeek has joined #openstack-lbaas | 10:54 | |
| *** tamas_erdei has quit IRC | 11:05 | |
| *** terdei has joined #openstack-lbaas | 11:06 | |
| *** zzzeek has quit IRC | 11:17 | |
| *** zzzeek has joined #openstack-lbaas | 11:18 | |
| *** ataraday_ has joined #openstack-lbaas | 11:22 | |
| *** rcernin has quit IRC | 11:31 | |
| *** ccamposr__ has joined #openstack-lbaas | 11:33 | |
| *** ccamposr has quit IRC | 11:34 | |
| *** psachin has quit IRC | 11:34 | |
| *** jamesdenton has quit IRC | 11:34 | |
| *** psachin has joined #openstack-lbaas | 11:40 | |
| *** jamesdenton has joined #openstack-lbaas | 11:40 | |
| *** sapd1 has quit IRC | 12:01 | |
| *** psachin has quit IRC | 12:31 | |
| *** servagem has joined #openstack-lbaas | 12:44 | |
| openstackgerrit | Carlos Goncalves proposed openstack/octavia-tempest-plugin master: Fix scenario tests issue with immutable LBs https://review.opendev.org/755777 | 12:59 |
| frickler | can anti-affinity ensure different AZs being used for HA pairs? the operator guide helpfully has "See our forthcoming HA guide for more details.", is there at least some draft of that? c.f. https://docs.openstack.org/octavia/latest/admin/guides/operator-maintenance.html#monitoring-octavia-control-plane | 13:02 |
| *** rpittau is now known as rpittau|afk | 13:03 | |
| *** TrevorV has joined #openstack-lbaas | 13:28 | |
| openstackgerrit | Carlos Goncalves proposed openstack/octavia-tempest-plugin master: Fix scenario tests issue with immutable LBs https://review.opendev.org/755777 | 13:59 |
| *** ataraday_ has quit IRC | 14:23 | |
| johnsom | frickler: It uses nova anti-affinity, so it is host and not AZs. | 15:08 |
| *** sapd1 has joined #openstack-lbaas | 15:42 | |
| cgoncalves | sigh, CI mirroring issues on the menu today | 15:43 |
| johnsom | Weee | 15:59 |
| *** xgerman has joined #openstack-lbaas | 16:05 | |
| *** ccamposr has joined #openstack-lbaas | 17:05 | |
| *** ccamposr__ has quit IRC | 17:07 | |
| johnsom | So, I am not sure I'm comfortable with the cascade delete change, but I just looked at the cleanup method, it is also using build_interval instead of check interval. That is probably why cleanup is so slow | 17:11 |
| *** sapd1 has quit IRC | 17:13 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia-tempest-plugin master: Attempt to improve tempest performance https://review.opendev.org/757158 | 17:22 |
| cgoncalves | though I agree changing to check_* from build_* makes sense, I'm not sure it will help much either where simple GETs are already taking so long | 17:53 |
| cgoncalves | I'd try to squash your patch in to Greg's | 17:54 |
| johnsom | I'm not a fan of the cascade delete option. | 17:55 |
| cgoncalves | ok. what are the reasons for not liking? | 17:55 |
| johnsom | Well, in some cases it's doing both, so no real win. | 17:56 |
| cgoncalves | which cases and doing both of what? DELETE? | 17:56 |
| johnsom | It still has the individual deletes and then at the end does the cascade delete. | 17:57 |
| johnsom | The LB delete will always be the last delete called as it was the first created in the class | 17:57 |
| cgoncalves | no, I removed the individual object deletes at class tear down | 17:58 |
| cgoncalves | it will just be one DELETE (LB delete cascade) | 17:58 |
| johnsom | https://review.opendev.org/#/c/755777/8/octavia_tempest_plugin/tests/api/v2/test_amphora.py | 17:58 |
| cgoncalves | right. that class actually doesn't need cascade delete. I added just for the sake of consistency | 17:59 |
| johnsom | Ah, I guess that one doesn't have any cleanup hooks | 17:59 |
| cgoncalves | https://review.opendev.org/#/c/755777/8/octavia_tempest_plugin/tests/api/v2/test_member.py | 17:59 |
| cgoncalves | take that as an example | 17:59 |
| cgoncalves | bunch of addClassResourceCleanups removed | 17:59 |
| johnsom | Yeah. I guess I just need to think about it for a while and decide if it's a good idea or not. Were there no non-class deletes? | 18:00 |
| cgoncalves | there are tons of addCleanup's | 18:02 |
| cgoncalves | so, yeah, there is still space for further improvements. my cascade delete change was a quick but effective one saving loads of time | 18:05 |
| johnsom | I have pingged opendev on the mirror issues | 18:06 |
| *** vishalmanchanda has quit IRC | 18:15 | |
| *** tkajinam has quit IRC | 18:25 | |
| *** armax has quit IRC | 18:25 | |
| *** armax has joined #openstack-lbaas | 18:30 | |
| *** armax has quit IRC | 18:45 | |
| openstackgerrit | Arkady Shtempler proposed openstack/octavia-tempest-plugin master: Adding two new tests: https://review.opendev.org/755326 | 18:49 |
| *** armax has joined #openstack-lbaas | 19:42 | |
| *** TrevorV has quit IRC | 19:42 | |
| *** zzzeek has quit IRC | 19:49 | |
| *** zzzeek has joined #openstack-lbaas | 19:51 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia-tempest-plugin master: Attempt to improve tempest performance https://review.opendev.org/757158 | 20:51 |
| *** servagem has quit IRC | 21:23 | |
| *** gcheresh has quit IRC | 22:01 | |
| *** armax has quit IRC | 22:16 | |
| *** takamatsu has quit IRC | 22:26 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!