| opendevreview | LiZekun proposed openstack/octavia-tempest-plugin master: Remove unicode literal from code https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/844733 | 01:25 |
|---|---|---|
| opendevreview | LiZekun proposed openstack/octavia-tempest-plugin master: Update python testing as per zed cycle teting runtime https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/844734 | 01:25 |
| opendevreview | LiZekun proposed openstack/octavia-tempest-plugin master: remove unicode literal from code https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/844736 | 01:25 |
| opendevreview | Michael Johnson proposed openstack/octavia master: Failover stop threshold / circuit breaker https://review.opendev.org/c/openstack/octavia/+/656811 | 02:46 |
| *** ysandeep|out is now known as ysandeep | 05:22 | |
| opendevreview | tianhao.chen proposed openstack/octavia master: Support selecting volume type when creating LB https://review.opendev.org/c/openstack/octavia/+/855312 | 08:40 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/yoga: Ignore status update on deleted objects in driver-agent https://review.opendev.org/c/openstack/octavia/+/855626 | 09:36 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/xena: Ignore status update on deleted objects in driver-agent https://review.opendev.org/c/openstack/octavia/+/855627 | 09:36 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/wallaby: Ignore status update on deleted objects in driver-agent https://review.opendev.org/c/openstack/octavia/+/855628 | 09:37 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/victoria: Ignore status update on deleted objects in driver-agent https://review.opendev.org/c/openstack/octavia/+/855630 | 09:40 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/ussuri: Ignore status update on deleted objects in driver-agent https://review.opendev.org/c/openstack/octavia/+/855631 | 09:40 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/train: Ignore status update on deleted objects in driver-agent https://review.opendev.org/c/openstack/octavia/+/855632 | 09:40 |
| *** ysandeep is now known as ysandeep|afk | 09:49 | |
| opendevreview | Tom Weininger proposed openstack/octavia master: Update doc about failover circuit breaker https://review.opendev.org/c/openstack/octavia/+/855642 | 10:39 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Update doc about failover circuit breaker https://review.opendev.org/c/openstack/octavia/+/855642 | 10:42 |
| *** ysandeep|afk is now known as ysandeep | 11:08 | |
| opendevreview | Tom Weininger proposed openstack/octavia-tempest-plugin master: Add scenario tests for failover circuit breaker https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/837878 | 12:15 |
| opendevreview | Merged openstack/octavia stable/train: Apply openstack-selinux policies in Centos amphorae https://review.opendev.org/c/openstack/octavia/+/853999 | 13:30 |
| opendevreview | Merged openstack/octavia stable/ussuri: Apply openstack-selinux policies in Centos amphorae https://review.opendev.org/c/openstack/octavia/+/853998 | 13:45 |
| opendevreview | Merged openstack/octavia stable/victoria: Apply openstack-selinux policies in Centos amphorae https://review.opendev.org/c/openstack/octavia/+/853997 | 14:07 |
| opendevreview | Merged openstack/octavia stable/yoga: Apply openstack-selinux policies in Centos amphorae https://review.opendev.org/c/openstack/octavia/+/853994 | 14:08 |
| opendevreview | Merged openstack/octavia stable/wallaby: Apply openstack-selinux policies in Centos amphorae https://review.opendev.org/c/openstack/octavia/+/853996 | 14:08 |
| opendevreview | Merged openstack/octavia stable/xena: Apply openstack-selinux policies in Centos amphorae https://review.opendev.org/c/openstack/octavia/+/853995 | 14:14 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Prevent incorrect reschedule of resumed tasks with jobboard https://review.opendev.org/c/openstack/octavia/+/838438 | 14:15 |
| *** ysandeep is now known as ysandeep|out | 14:34 | |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Prevent incorrect reschedule of resumed tasks with jobboard https://review.opendev.org/c/openstack/octavia/+/838438 | 14:53 |
| johnsom | tweining Maybe I misunderstood the docs about FAILOVER_STOPPED amps resuming automatic failover. | 14:57 |
| tweining | johnsom: don't worry, I think it's fine. | 14:58 |
| johnsom | Well, this could be a pretty simple fix to put it back. | 14:58 |
| tweining | hm, I'm not sure. I tested that version already. | 14:59 |
| johnsom | gthiemonge What do you think? It is likely just changing the list of acceptable states and some test adjustments. | 15:02 |
| gthiemonge | well, I liked the auto-recover behavior in the previous patchset.. but I think it's up to tweining to decide | 15:05 |
| tweining | I liked it too. johnsom if you think you can fix it easily please do so then | 15:06 |
| johnsom | Ok, yeah, looking back I can see benefit. I have updated the code, just looking at the tests now. | 15:06 |
| tweining | there's a scenario that I updated today as well https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/837878 | 15:07 |
| johnsom | sqlalchemy became a bit of a grind yesterday, so it took longer than I expected. It kept giving me back tuples instead of ORM objects. | 15:07 |
| tweining | that scenario test also tested that automatic failover. I can change it back so you can test it more easily. | 15:08 |
| opendevreview | Michael Johnson proposed openstack/octavia master: Failover stop threshold / circuit breaker https://review.opendev.org/c/openstack/octavia/+/656811 | 15:09 |
| johnsom | There we go | 15:09 |
| gthiemonge | ok, I'm testing it too | 15:10 |
| tweining | ok, I'll update the docs back | 15:10 |
| gthiemonge | tweining: do we still need this update? | 15:13 |
| tweining | only wrt. to the failover_threshold value recommendation, but I'll add it to the original patch | 15:14 |
| gthiemonge | ok | 15:15 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Failover stop threshold / circuit breaker https://review.opendev.org/c/openstack/octavia/+/656811 | 15:16 |
| opendevreview | Tom Weininger proposed openstack/octavia-tempest-plugin master: Add scenario tests for failover circuit breaker https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/837878 | 15:17 |
| johnsom | I have re-applied my +2 | 15:18 |
| tweining | thanks to both of you. I need to go now. have a nice weekend. | 15:20 |
| gthiemonge | tweining: thanks, you too! | 15:23 |
| johnsom | tweining If you are good with the change, re-apply your +2 | 15:24 |
| gthiemonge | ok it works as expected, +2 from me | 15:26 |
| gthiemonge | and W+1 | 15:26 |
| opendevreview | Merged openstack/python-octaviaclient master: Allow creating/showing LBs with Additional VIPs https://review.opendev.org/c/openstack/python-octaviaclient/+/664473 | 15:26 |
| opendevreview | Merged openstack/octavia master: Update zuul queue configuration https://review.opendev.org/c/openstack/octavia/+/842564 | 16:30 |
| opendevreview | Merged openstack/octavia master: Fix ignored [nova] service_name https://review.opendev.org/c/openstack/octavia/+/820034 | 16:51 |
| opendevreview | Merged openstack/octavia master: Enable taskflow retry feature when waiting for compute https://review.opendev.org/c/openstack/octavia/+/816535 | 16:57 |
| opendevreview | Merged openstack/octavia master: Fix PING health-monitor with recent haproxy releases https://review.opendev.org/c/openstack/octavia/+/836149 | 17:11 |
| opendevreview | Merged openstack/octavia master: Failover stop threshold / circuit breaker https://review.opendev.org/c/openstack/octavia/+/656811 | 20:33 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!