| opendevreview | Sergey Kraynev proposed openstack/octavia master: Add option to limit recursion in to_data_model https://review.opendev.org/c/openstack/octavia/+/883063 | 04:57 |
|---|---|---|
| dnegreira | Hi there o/ I am trying to backport a previous merged fix: https://review.opendev.org/c/openstack/octavia/+/894547 - seems that the CI is failing and doesn't seem related to what I am backporting. Should I re-trigger again the `recheck` ? Any other suggestion? | 07:25 |
| gthiemonge | dnegreira: Hi, I'll take a look, the jobs look broken, AFAIR we may have to pin tox on victoria | 07:32 |
| dnegreira | gthiemonge: ack, thx, lmk if I should pin tox, and which version on my PR as well. | 07:33 |
| gthiemonge | ok, there's already a fix | 07:35 |
| gthiemonge | https://review.opendev.org/c/openstack/octavia/+/888707 | 07:35 |
| gthiemonge | I'm approving it without a 2nd CR+2 | 07:35 |
| dnegreira | awesome. I'll keep an eye on it. Should I try to `recheck` on my PR when that's finalized ? | 07:37 |
| gthiemonge | dnegreira: yeah you can recheck when 888707 is merged, but please don't use "bare rechecks", always add a reason for the recheck (like "recheck gate is fixed"), TC is doing statistics with bare rechecks and they point projects that use them :D | 07:41 |
| dnegreira | gthiemonge: gotcha, will do that from now on :) thanks for the heads-up and merging the gate fixes. | 07:44 |
| opendevreview | Merged openstack/octavia stable/victoria: [stable-only] Cap virtualenv/setuptools https://review.opendev.org/c/openstack/octavia/+/888707 | 09:13 |
| opendevreview | Lukas Piwowarski proposed openstack/octavia-tempest-plugin master: [WIP] Add backup member tests https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/897564 | 09:46 |
| opendevreview | Merged openstack/octavia master: Disable seqdiag in the spec docs https://review.opendev.org/c/openstack/octavia/+/897519 | 10:48 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Reduce duration of failovers with amphora in ERROR https://review.opendev.org/c/openstack/octavia/+/893536 | 11:39 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Fix amphorae in ERROR during the failover https://review.opendev.org/c/openstack/octavia/+/893612 | 11:39 |
| opendevreview | Merged openstack/octavia master: Fix timeout duration in start_vrrp_service during failovers https://review.opendev.org/c/openstack/octavia/+/893537 | 12:10 |
| opendevreview | Omer Schwartz proposed openstack/octavia-tempest-plugin master: Add TERMINATED_HTTPS listener API tests https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/893066 | 12:12 |
| opendevreview | Merged openstack/octavia stable/victoria: Fix barbican client with application credentials/trusts https://review.opendev.org/c/openstack/octavia/+/894547 | 13:32 |
| opendevreview | Merged openstack/octavia stable/ussuri: Fix barbican client with application credentials/trusts https://review.opendev.org/c/openstack/octavia/+/894548 | 14:24 |
| opendevreview | Merged openstack/octavia master: Reduce duration of failovers with amphora in ERROR https://review.opendev.org/c/openstack/octavia/+/893536 | 15:35 |
| opendevreview | Merged openstack/octavia master: Retry to set loadbalancer prov status on failures https://review.opendev.org/c/openstack/octavia/+/896383 | 16:16 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily while we restart it for a combined runtime and platform upgrade | 16:33 | |
| opendevreview | Merged openstack/octavia master: Fix amphorae in ERROR during the failover https://review.opendev.org/c/openstack/octavia/+/893612 | 18:05 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!