Monday, 2022-07-11

*** ysandeep|out is now known as ysandeep05:38
*** ysandeep is now known as ysandeep|holiday06:18
opendevreviewGregory Thiemonge proposed openstack/octavia master: WIP Add an element for installing FRR daemon  https://review.opendev.org/c/openstack/octavia/+/83714706:48
opendevreviewGregory Thiemonge proposed openstack/octavia master: WIP Configure FRR endpoint in devstack  https://review.opendev.org/c/openstack/octavia/+/83714806:48
opendevreviewGregory Thiemonge proposed openstack/octavia master: WIP Add constants for MULTI_ACTIVE LB topology  https://review.opendev.org/c/openstack/octavia/+/83714906:48
opendevreviewGregory Thiemonge proposed openstack/octavia master: WIP Add BGP support in the amphora-agent  https://review.opendev.org/c/openstack/octavia/+/83715006:48
opendevreviewGregory Thiemonge proposed openstack/octavia master: WIP Add BGP driver in Octavia controller  https://review.opendev.org/c/openstack/octavia/+/83715106:48
opendevreviewGregory Thiemonge proposed openstack/octavia master: WIP Integrate BGP driver in the Amphora client  https://review.opendev.org/c/openstack/octavia/+/83715206:48
opendevreviewGregory Thiemonge proposed openstack/octavia master: WIP Add MULTI_ACTIVE flows to the Octavia controller  https://review.opendev.org/c/openstack/octavia/+/83715306:48
opendevreviewGregory Thiemonge proposed openstack/octavia master: Support haproxy peer table in MULTI_ACTIVE topo  https://review.opendev.org/c/openstack/octavia/+/84927506:48
opendevreviewGregory Thiemonge proposed openstack/octavia master: Check the number of available ip addresses for MULTI_ACTIVE LBs  https://review.opendev.org/c/openstack/octavia/+/84927606:48
jrosseris there a spec for this bgp stuff? it looks very different to how neutron/ovn approach the same sort of thing07:07
gthiemongejrosser: there's an old spec at https://docs.openstack.org/octavia/latest/contributor/specs/version1.1/active-active-l3-distributor.html07:35
gthiemongejrosser: these commits are based on this spec, except that ARM the "distributor" of the spec is a service that runs on the controller07:36
jrosserlike i say it's widly different from the neutron bgp agent07:41
jrosser*wildly07:41
jrossergthiemonge: it might be worth talking to people who are already doing ipv6 with neutron about it, as they almost certainly already have neutron<>bgp setups07:50
gthiemongejrosser: that would be great07:50
jrosserin my case there is no connectivity (physically no path) from the lb mgmt network to my internet routers, completely by design07:51
jrosserthe only useful place to run a bgp agent is on the network nodes where i have a dedicated peering vlan with the routers07:51
jrosserthis is possible with both the neutron and ovn bgp agents07:52
gthiemongeok 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
jrosserwell, i point to that really as a design pattern07:57
opendevreviewGregory Thiemonge proposed openstack/octavia master: DNM/WIP Testing periodic openstack-octavia-amphora-image-centos-9-stream  https://review.opendev.org/c/openstack/octavia/+/84929109:33
opendevreviewMerged openstack/octavia stable/xena: Fix duplicate object error messages  https://review.opendev.org/c/openstack/octavia/+/84755609:56
opendevreviewMerged openstack/octavia stable/wallaby: Fix duplicate object error messages  https://review.opendev.org/c/openstack/octavia/+/84755709:56
opendevreviewGregory Thiemonge proposed openstack/octavia master: DNM/WIP Testing periodic openstack-octavia-amphora-image-centos-9-stream  https://review.opendev.org/c/openstack/octavia/+/84929110:09
opendevreviewTom Weininger proposed openstack/octavia master: Amphora vertical scaling optimization in Octavia  https://review.opendev.org/c/openstack/octavia/+/84913010:47
opendevreviewMerged openstack/octavia stable/yoga: Correct format of release note  https://review.opendev.org/c/openstack/octavia/+/84661611:23
opendevreviewMerged openstack/octavia stable/xena: Correct format of release note  https://review.opendev.org/c/openstack/octavia/+/84661711:24
opendevreviewMerged openstack/octavia stable/wallaby: Correct format of release note  https://review.opendev.org/c/openstack/octavia/+/84661911:28
opendevreviewMerged openstack/octavia stable/victoria: Correct format of release note  https://review.opendev.org/c/openstack/octavia/+/84662011:28
opendevreviewMerged openstack/octavia stable/ussuri: Correct format of release note  https://review.opendev.org/c/openstack/octavia/+/84662111:28
opendevreviewMerged openstack/octavia stable/train: Correct format of release note  https://review.opendev.org/c/openstack/octavia/+/84662211:28
opendevreviewGregory Thiemonge proposed openstack/octavia-tempest-plugin master: Fix TLS*_METHOD for old pyopenssl releases  https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/84935612:36
opendevreviewGregory Thiemonge proposed openstack/octavia stable/victoria: DNM Testing o-t-p fix for old branches  https://review.opendev.org/c/openstack/octavia/+/84935812:37
opendevreviewMerged openstack/octavia stable/yoga: Set sensible nf_conntrack_max value in amphora  https://review.opendev.org/c/openstack/octavia/+/84753913:30
opendevreviewMerged openstack/octavia stable/xena: Set sensible nf_conntrack_max value in amphora  https://review.opendev.org/c/openstack/octavia/+/84754013:36
opendevreviewMerged openstack/octavia stable/wallaby: Set sensible nf_conntrack_max value in amphora  https://review.opendev.org/c/openstack/octavia/+/84754113:37
opendevreviewMerged openstack/octavia stable/train: Set sensible nf_conntrack_max value in amphora  https://review.opendev.org/c/openstack/octavia/+/84754414:14
opendevreviewGregory Thiemonge proposed openstack/octavia master: DNM/WIP Testing octavia-amphora-image-build-live-centos-9-stream  https://review.opendev.org/c/openstack/octavia/+/84929114:29
tweiningjohnsom: 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
opendevreviewMerged openstack/octavia-tempest-plugin master: Fix TLS*_METHOD for old pyopenssl releases  https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/84935616:51
johnsomtweining ack16:55
johnsomgthiemonge tweining We should talk about that patch here. I think it needs a release note for sure. 16:56
johnsomThe other concerns I have is test coverage, risk of breaking things, the static field lists.16:58
johnsomI 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
johnsomOh, I see there is some test coverage, nevermind on that.16:59
johnsomIt was the __repr__ that I thought got missed, but the cover job is showing it covered.17:01
johnsomrm_work What do you think about backporting https://review.opendev.org/c/openstack/octavia/+/847723 ?17:02
rm_workhmmm21:43
rm_workprint changes seem fine21:43
rm_worki am worried about having to go through and see what all might be "sensitive"21:43
rm_workbut 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/!