opendevreview | Michael Johnson proposed openstack/octavia-tempest-plugin master: DNM: Capture cirros console logs https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/877832 | 04:43 |
---|---|---|
johnsom | Hmmm, so I guess I broke the image builds on older releases: https://zuul.opendev.org/t/openstack/build/8caa4f281fb848ccbce91d0749fe7973 | 05:19 |
johnsom | I would have expected that uninstall element to not fail if the package isn't present.... | 05:19 |
johnsom | I will not be around tomorrow, so if someone wants to poke at this, feel free | 05:21 |
johnsom | We probably need a pkg-map for those versions | 05:28 |
opendevreview | Michael Johnson proposed openstack/octavia-tempest-plugin master: DNM: Capture cirros console logs https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/877832 | 05:30 |
gthiemonge | johnsom: np, i'm going to take a look at it | 06:45 |
gthiemonge | interesting... dnf doesn't fail if a package doesn't exist | 06:49 |
opendevreview | Gregory Thiemonge proposed openstack/octavia master: WIP Fix focal amphora image build https://review.opendev.org/c/openstack/octavia/+/877894 | 07:00 |
opendevreview | Gregory Thiemonge proposed openstack/octavia master: Fix focal amphora image build https://review.opendev.org/c/openstack/octavia/+/877894 | 07:19 |
kevko | Hello folks, i found that octavia is not correctly interpreting [ipv6_address]:port when brackets are used ... this looks like a bug because [ipv6]:port format looks like is standard syntax of definition ipv6 address and port combination | 09:02 |
kevko | am i correct ? | 09:02 |
kevko | i am talking about amphora agent which is parsing configuration file from /etc/octavia/amphora-agent.conf [health_manager] controller_ip_port_list | 09:03 |
gthiemonge | kevko: Hi, I think this is a bug, currently only ipv6:port is supported | 09:12 |
kevko | gthiemonge: I am not sure if you understand my, yes ipv6:port is working ...but valid format for ipv6 and port is [ipv6]:port escpecially because ip6 semicolons etc ... | 09:13 |
kevko | gthiemonge: this is working -> controller_ip_port_list = fc00:164a:b260:bfe4::6:5555, fc00:164a:b260:bfe4::7:5555, fc00:164a:b260:bfe4::8:5555 , this is not working controller_ip_port_list = [fc00:164a:b260:bfe4::6]:5555, [fc00:164a:b260:bfe4::7]:5555, [fc00:164a:b260:bfe4::8]:5555 | 09:15 |
kevko | but it should | 09:15 |
gthiemonge | yeah I understand, I would consider that it is a bug in Octavia, could you open a story in storyboard? | 09:18 |
gthiemonge | kevko: did you configure this field manually? or did you use a deployment tool? | 09:21 |
kevko | gthiemonge: kolla-ansible | 09:40 |
gthiemonge | kevko: ack, maybe the support of ipv6 management network is not tested with kolla-ansible | 09:43 |
kevko | gthiemonge: it's probably not, but kolla-ansible is configuring ipv6 ip and port in correct way ...octavia not .. | 09:54 |
kevko | gthiemonge: i will fix it and send a patch ... do octavia use storyboard instead of launchpad ? | 09:54 |
gthiemonge | kevko: yeah, we use storyboard | 09:59 |
skraynev | gthiemonge: hi. I forgot to ask about: https://review.opendev.org/c/openstack/octavia/+/877414 do we have a chance to backport this fix to stable/yoga? | 10:37 |
skraynev | of course after merge in master branch %) | 10:37 |
gthiemonge | skraynev: hi, yes, we will backport it | 10:40 |
skraynev | gthiemonge: awesome! thx. I will wait it. I hope, that Pr will get one more +2 soon :) | 10:42 |
kevko | gthiemonge: it would be nice to merge and backport in short period of time ..do you think this will be possible as this is simple fix ? .. i am currently deploying octavia on customer side and it would help me to get this patch into stable branches | 10:43 |
gthiemonge | kevko: ack, we will try to prioritize it, I'll backport it as soon at it is merged | 10:44 |
gthiemonge | kevko: which release do you use? | 10:44 |
kevko | gthiemonge: for now we are deploying yoga | 10:55 |
kevko | gthiemonge: Btw, we have another requirement needed from customer, they are using 3 AZs with 3 cephs and using glance with multistore support ..they wants this model for his HA needs... | 11:22 |
kevko | *their | 11:22 |
kevko | this configuration work well with nova, neutron,ovn ,cinder,glance ... | 11:22 |
kevko | gthiemonge: problem is they wants to spin amphoras on different AZs ..to have always amphoras on differents AZs (or let's say compute hosts --- without AZs usage) ... and this looks like impossible in octavia | 11:24 |
kevko | gthiemonge: what i found is that you can schedule loadbalancer to some AZ for example ...but this is not enough because there can be several issues with this AZ in case of network outage or whatever .. then your LB is dead ... | 11:25 |
kevko | gthiemonge: when i am reading my explanation ..it is little bit chaotic ..sorry for that :) ...simply said ..they want to spin amphoras on different hosts/azs to be sure that LB will not be down in case of AZ/host is down ...and also ensure that two amphoras will not be in same AZ/host ... | 11:26 |
kevko | gthiemonge: how can be this achieved ? some special property of flavor/image ? special scheduler in nova ? or combination with anti-affinity ? is this somehow configurable in octavia ? if nothing is possible ...can we implement solution ? | 11:28 |
gthiemonge | kevko: I think you need this feature that is still in review: https://review.opendev.org/c/openstack/octavia/+/558962 | 11:58 |
gthiemonge | kevko: otherwise I don't know, maybe some specific parameters in a Nova flavor would make the amphorae schedule in different AZs | 11:59 |
kevko | gthiemonge: wow, cool, can I somehow help to speed up process ? i have several test envs to test it ... | 11:59 |
gthiemonge | kevko: yeah, reviewing/testing the code would be helpful | 12:00 |
kevko | gthiemonge: if it is working i will backport it into our downstream repository... but it would be fine to have an information that this will be merged ... small changes between patchsets are not big problem as then i can just update our downstream git repos/images or update octavia to antelope if it will be present there | 12:01 |
kevko | gthiemonge: what i am trying to say ...sometimes in gerrit is very usefull change which is waiting for merge months ... | 12:02 |
kevko | gthiemonge: this change is there from 2018 year :/ | 12:09 |
gthiemonge | yeah it was started long time ago and was nearly abandonned but folks restored it during the last cycle, we hope we will merge it in Bobcat | 12:14 |
opendevreview | Gregory Thiemonge proposed openstack/octavia master: Fix failover when the last listener is deleted https://review.opendev.org/c/openstack/octavia/+/877842 | 12:45 |
opendevreview | Michal Arbet proposed openstack/octavia master: Fix octavia to accept [ipv6]:port https://review.opendev.org/c/openstack/octavia/+/877959 | 14:12 |
kevko | gthiemonge: https://review.opendev.org/c/openstack/octavia/+/877959 ipv6 port fix | 14:13 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!