*** ysandeep|out is now known as ysandeep | 05:38 | |
*** ysandeep is now known as ysandeep|holiday | 06:18 | |
opendevreview | Gregory Thiemonge proposed openstack/octavia master: WIP Add an element for installing FRR daemon https://review.opendev.org/c/openstack/octavia/+/837147 | 06:48 |
---|---|---|
opendevreview | Gregory Thiemonge proposed openstack/octavia master: WIP Configure FRR endpoint in devstack https://review.opendev.org/c/openstack/octavia/+/837148 | 06:48 |
opendevreview | Gregory Thiemonge proposed openstack/octavia master: WIP Add constants for MULTI_ACTIVE LB topology https://review.opendev.org/c/openstack/octavia/+/837149 | 06:48 |
opendevreview | Gregory Thiemonge proposed openstack/octavia master: WIP Add BGP support in the amphora-agent https://review.opendev.org/c/openstack/octavia/+/837150 | 06:48 |
opendevreview | Gregory Thiemonge proposed openstack/octavia master: WIP Add BGP driver in Octavia controller https://review.opendev.org/c/openstack/octavia/+/837151 | 06:48 |
opendevreview | Gregory Thiemonge proposed openstack/octavia master: WIP Integrate BGP driver in the Amphora client https://review.opendev.org/c/openstack/octavia/+/837152 | 06:48 |
opendevreview | Gregory Thiemonge proposed openstack/octavia master: WIP Add MULTI_ACTIVE flows to the Octavia controller https://review.opendev.org/c/openstack/octavia/+/837153 | 06:48 |
opendevreview | Gregory Thiemonge proposed openstack/octavia master: Support haproxy peer table in MULTI_ACTIVE topo https://review.opendev.org/c/openstack/octavia/+/849275 | 06:48 |
opendevreview | Gregory Thiemonge proposed openstack/octavia master: Check the number of available ip addresses for MULTI_ACTIVE LBs https://review.opendev.org/c/openstack/octavia/+/849276 | 06:48 |
jrosser | is there a spec for this bgp stuff? it looks very different to how neutron/ovn approach the same sort of thing | 07:07 |
gthiemonge | jrosser: there's an old spec at https://docs.openstack.org/octavia/latest/contributor/specs/version1.1/active-active-l3-distributor.html | 07:35 |
gthiemonge | jrosser: these commits are based on this spec, except that ARM the "distributor" of the spec is a service that runs on the controller | 07:36 |
jrosser | like i say it's widly different from the neutron bgp agent | 07:41 |
jrosser | *wildly | 07:41 |
jrosser | gthiemonge: it might be worth talking to people who are already doing ipv6 with neutron about it, as they almost certainly already have neutron<>bgp setups | 07:50 |
gthiemonge | jrosser: that would be great | 07:50 |
jrosser | in my case there is no connectivity (physically no path) from the lb mgmt network to my internet routers, completely by design | 07:51 |
jrosser | the only useful place to run a bgp agent is on the network nodes where i have a dedicated peering vlan with the routers | 07:51 |
jrosser | this is possible with both the neutron and ovn bgp agents | 07:52 |
gthiemonge | ok I see... in my setup, the amphora is a bgp speaker (with bfd support), it helps to quickly disable a broken amphora and to send the traffic to the other amps, I don't know if neutron/ovn bgp agent could help us to do it. | 07:57 |
jrosser | well, i point to that really as a design pattern | 07:57 |
opendevreview | Gregory Thiemonge proposed openstack/octavia master: DNM/WIP Testing periodic openstack-octavia-amphora-image-centos-9-stream https://review.opendev.org/c/openstack/octavia/+/849291 | 09:33 |
opendevreview | Merged openstack/octavia stable/xena: Fix duplicate object error messages https://review.opendev.org/c/openstack/octavia/+/847556 | 09:56 |
opendevreview | Merged openstack/octavia stable/wallaby: Fix duplicate object error messages https://review.opendev.org/c/openstack/octavia/+/847557 | 09:56 |
opendevreview | Gregory Thiemonge proposed openstack/octavia master: DNM/WIP Testing periodic openstack-octavia-amphora-image-centos-9-stream https://review.opendev.org/c/openstack/octavia/+/849291 | 10:09 |
opendevreview | Tom Weininger proposed openstack/octavia master: Amphora vertical scaling optimization in Octavia https://review.opendev.org/c/openstack/octavia/+/849130 | 10:47 |
opendevreview | Merged openstack/octavia stable/yoga: Correct format of release note https://review.opendev.org/c/openstack/octavia/+/846616 | 11:23 |
opendevreview | Merged openstack/octavia stable/xena: Correct format of release note https://review.opendev.org/c/openstack/octavia/+/846617 | 11:24 |
opendevreview | Merged openstack/octavia stable/wallaby: Correct format of release note https://review.opendev.org/c/openstack/octavia/+/846619 | 11:28 |
opendevreview | Merged openstack/octavia stable/victoria: Correct format of release note https://review.opendev.org/c/openstack/octavia/+/846620 | 11:28 |
opendevreview | Merged openstack/octavia stable/ussuri: Correct format of release note https://review.opendev.org/c/openstack/octavia/+/846621 | 11:28 |
opendevreview | Merged openstack/octavia stable/train: Correct format of release note https://review.opendev.org/c/openstack/octavia/+/846622 | 11:28 |
opendevreview | Gregory Thiemonge proposed openstack/octavia-tempest-plugin master: Fix TLS*_METHOD for old pyopenssl releases https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/849356 | 12:36 |
opendevreview | Gregory Thiemonge proposed openstack/octavia stable/victoria: DNM Testing o-t-p fix for old branches https://review.opendev.org/c/openstack/octavia/+/849358 | 12:37 |
opendevreview | Merged openstack/octavia stable/yoga: Set sensible nf_conntrack_max value in amphora https://review.opendev.org/c/openstack/octavia/+/847539 | 13:30 |
opendevreview | Merged openstack/octavia stable/xena: Set sensible nf_conntrack_max value in amphora https://review.opendev.org/c/openstack/octavia/+/847540 | 13:36 |
opendevreview | Merged openstack/octavia stable/wallaby: Set sensible nf_conntrack_max value in amphora https://review.opendev.org/c/openstack/octavia/+/847541 | 13:37 |
opendevreview | Merged openstack/octavia stable/train: Set sensible nf_conntrack_max value in amphora https://review.opendev.org/c/openstack/octavia/+/847544 | 14:14 |
opendevreview | Gregory Thiemonge proposed openstack/octavia master: DNM/WIP Testing octavia-amphora-image-build-live-centos-9-stream https://review.opendev.org/c/openstack/octavia/+/849291 | 14:29 |
tweining | johnsom: hi. Could you please have a look at https://review.opendev.org/c/openstack/octavia/+/847723 again? gthiemonge added a comment there. I'd like to get it into OSP17.0 if possible. | 15:27 |
opendevreview | Merged openstack/octavia-tempest-plugin master: Fix TLS*_METHOD for old pyopenssl releases https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/849356 | 16:51 |
johnsom | tweining ack | 16:55 |
johnsom | gthiemonge tweining We should talk about that patch here. I think it needs a release note for sure. | 16:56 |
johnsom | The other concerns I have is test coverage, risk of breaking things, the static field lists. | 16:58 |
johnsom | I did a quick scan to see if there was something in those fields that didn't exist in the recent stable branches, but without test coverage we might miss something. | 16:58 |
johnsom | Oh, I see there is some test coverage, nevermind on that. | 16:59 |
johnsom | It was the __repr__ that I thought got missed, but the cover job is showing it covered. | 17:01 |
johnsom | rm_work What do you think about backporting https://review.opendev.org/c/openstack/octavia/+/847723 ? | 17:02 |
rm_work | hmmm | 21:43 |
rm_work | print changes seem fine | 21:43 |
rm_work | i am worried about having to go through and see what all might be "sensitive" | 21:43 |
rm_work | but honestly not sure what exactly we store that could be a risk to print? | 21:44 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!