Monday, 2021-04-26

*** sapd1 has quit IRC00:44
*** mchlumsky has joined #openstack-lbaas01:19
*** zzzeek has quit IRC02:48
*** zzzeek has joined #openstack-lbaas02:50
*** sapd1 has joined #openstack-lbaas03:16
*** vishalmanchanda has joined #openstack-lbaas04:52
*** gcheresh has joined #openstack-lbaas05:04
*** yamamoto has quit IRC05:07
*** strigazi has quit IRC05:37
*** strigazi has joined #openstack-lbaas05:39
*** luksky has joined #openstack-lbaas06:09
*** luksky has quit IRC06:17
*** luksky has joined #openstack-lbaas06:18
*** rpittau|afk is now known as rpittau07:12
*** openstack has joined #openstack-lbaas07:53
*** ChanServ sets mode: +o openstack07:53
*** openstackgerrit has quit IRC08:11
marlincjohnsom: https://gist.github.com/Marlinc/f60efb44df457b8131404be0daae853c08:18
zigoHi. Could I get a 2nd view on that one?08:21
zigohttps://review.opendev.org/c/openstack/octavia/+/65649108:21
zigoI don't think the -1 is relevant, and I've been carying the patch in the Debian package for more than a year now ...08:22
*** lxkong has quit IRC08:24
*** emccormick has quit IRC08:25
gthiemongezigo: yeah I agree. LGTM08:27
*** emccormick has joined #openstack-lbaas08:29
*** lxkong has joined #openstack-lbaas08:30
gthiemongemarlinc: it's weird: the octavia management network is missing from the 'addresses' attribute in server list08:40
*** parallax has joined #openstack-lbaas08:57
marlincYea gthiemonge that's the issue I'm running into. When I check the 'action logs' in Horizon then I can see the octavia user creating the instance, attaching the tenant network and then detaching the management network09:24
marlincI hope enabling debugging will provide more information on what's causing that09:28
*** strigazi has quit IRC11:05
*** sapd1 has quit IRC11:20
*** servagem has joined #openstack-lbaas12:12
*** marlinc has quit IRC12:48
*** marlinc has joined #openstack-lbaas12:59
*** gregraka has joined #openstack-lbaas13:05
*** marlinc has quit IRC13:23
*** marlinc has joined #openstack-lbaas13:23
*** sapd1 has joined #openstack-lbaas13:25
*** vishalmanchanda has quit IRC13:41
*** openstackgerrit has joined #openstack-lbaas13:49
openstackgerritGregory Thiemonge proposed openstack/octavia master: Spare pool removal  https://review.opendev.org/c/openstack/octavia/+/78772213:49
openstackgerritGregory Thiemonge proposed openstack/octavia master: Spare pool removal  https://review.opendev.org/c/openstack/octavia/+/78772213:50
*** vishalmanchanda has joined #openstack-lbaas14:22
*** openstackgerrit has quit IRC14:23
*** openstackgerrit has joined #openstack-lbaas14:43
openstackgerritMohammed Naser proposed openstack/octavia stable/ussuri: Fix empty Batch Member Update to unlock objects  https://review.opendev.org/c/openstack/octavia/+/78719014:43
openstackgerritMohammed Naser proposed openstack/octavia stable/train: Fix empty Batch Member Update to unlock objects  https://review.opendev.org/c/openstack/octavia/+/78801514:47
openstackgerritCarlos Gonçalves proposed openstack/octavia stable/victoria: Fix empty Batch Member Update to unlock objects  https://review.opendev.org/c/openstack/octavia/+/78801814:55
marlincAbout the issue where the management interface gets detached, I haven't figured out why its happening yet but I have gathered some debug information: https://gist.github.com/Marlinc/70a3b38b0dc9bc84c6af25d8a324d9ca15:23
marlincNot sure what "Disabling health monitoring on amphora" is, going to look into that15:27
*** spatel has joined #openstack-lbaas15:28
spateljohnsom quick question does octavia support TLS based load-balancing ?15:29
johnsommarlinc Just a tip, if you are asking for help, please stay around 20-30 minutes. It was my weekend too, but jumped online to offer help.15:30
johnsomspatel Yes15:30
spatelperfect! thanks15:30
johnsomspatel Pass through and via offloading on the LB15:30
marlincjohnsom, yea normally I do, I just installed ZNC again so I can use multiple IRC clients to make that easier. I had irssi open on a remote machine so I didn't get a notification15:31
spateljohnsom do i need to tell octavia to perform TLS load-balancing ?15:31
johnsomspatel You specify it when creating your listeners.  This section and down in the docs: https://docs.openstack.org/octavia/latest/user/guides/basic-cookbook.html#deploy-a-non-terminated-https-load-balancer15:31
spatel+115:33
johnsommarlinc Can you provide the settings in octavia.conf [controller_worker] amp_boot_network_list , if you have any Octavia availability zones configured, and a subnet show for "some-test-network"15:34
marlincjohnsom, wait a minute I did not add the management network to amp_boot_network_list because its the only network in the project so I expected it to work because its the default. Is Octavia removing it because it didn't explicitly add that interface?15:35
johnsomAre you saying you are trying to use the same network for your lb-mgmt-net and your VIP network?15:36
marlincNo what I meant to say is that the service project in which Octavia is creating instances only has one network (the lb-mgmt-net network) so Nova is auto attaching that network to the Octavia Amphora instances15:38
johnsomVIP and member networks can be any project. lb-mgmt-net needs to be owned by the octavia account. Nova isn't auto-attaching it, we are telling nova to attache the lb-mgmt-net to the instance.15:40
marlincYea that's I think what's going wrong, I did not tell Octavia to add the lb-mgmt-net network (by setting it in amp_boot_network_list)15:41
marlincamp_boot_network_list is currently not set at all in [controller_worker]15:41
johnsomYeah, that is a required setting15:42
johnsomIt must match your lb-mgmt-net15:42
openstackgerritGreg Rakauskas proposed openstack/octavia master: Edits for "Basic Load Balancing Cookbook"  https://review.opendev.org/c/openstack/octavia/+/78764315:51
*** rpittau is now known as rpittau|afk15:55
*** mnaser has joined #openstack-lbaas15:55
marlincFound the logic that is was removing the network because it didn't expect it: https://github.com/openstack/octavia/blob/master/octavia/controller/worker/v2/tasks/network_tasks.py#L9415:58
marlincI think I'm going to open a review for a small change that introduces some debug logs indicating that Octavia ia removing interfaces it doesn't expect15:59
marlincSo as a operator you know why Octavia is removing them and you can be pointed to amp_boot_network_list15:59
johnsomWell, since you probably used the same for the VIP, when you manually triggered that failover, it also probably removed it.15:59
johnsomThat is probably not helpful in this situation. A better approach is probably adding an error and exit if the lb-mgmt-net is not defined.16:01
johnsomNetworks in that list are already excluded from that delta calculation16:01
*** xgerman has joined #openstack-lbaas16:01
marlincWhat I think happened is that Octavia was sending a empty network list to Nova when creating the instance, this causes Nova to assume it has to add the default network (which was lb-mgmt-net). Then the Octavia reconciliation loop comes along (the delta calculation) and sees a unexpected network interface and thus removes it16:03
johnsomYeah, because the lb-mgmt-net wasn't on the boot network list16:04
johnsomArguably nova should have just failed to boot the instance.16:04
marlincYea so giving an error when amp_boot_network_list isn't set is probably a good idea16:04
johnsomOr booted it with no network at all, which might have happened and the VIP network came along16:04
marlincAnd maybe some debugging information in that reconciliation loop so you know what its trying to change the state to16:05
johnsomI would vote against that as it will be fairly noisy for a very rare configuration issue. That delta calculation runs often with member add/remove, etc.16:06
johnsomWhat deployment tool did you use that it didn't set that for you?16:07
marlincI'm using openstack-helm together with Terraform to deployed the OpenStack tooling so I have to set that manually based on the network that Terraform creates16:08
marlincWas just thinking of a log message when it actually needs to remove a network, not some log message that always shows that there's nothing to removed16:09
*** nicolasbock has joined #openstack-lbaas16:09
johnsomYep16:09
marlincI have had multiple times with OpenStack components where they didn't really log anything even with the highest level of debugging so I had to go through the code to figure out what was wrong16:10
marlincSo having some log message that indicates that indicates that something's wrong would be very useful16:10
johnsomIf you used the helm chart for Octavia, it might be worth opening a bug for the helm team to make sure the lb-mgmt-net gets configured16:10
marlincThe issue is not with the Helm chart but rather with the Terraform logic I wrote around it so that's my fault16:11
marlincThe Helm chart itself doesn't set anything, it just sets up the deployments etc in Kubernetes. In the official openstack-helm tooling there's tons of bash scripts around Helm to provide that information to the Helm chart16:12
johnsomI am just saying I won't vote for it as I don't think it is addressing the real problem. It's completely unrelated to the root cause and will lead people in the wrong direction of fixing the problem.16:12
johnsomWe get a bit of feedback that we log too much already, lol.16:13
marlincYea I understand I'll leave it, also I wasn't talking INFO or WARNING logging but DEBUG level, I assume people don't run with DEBUG in production16:14
johnsomMany do. I even know of one distro that runs with TRACE on, which is bizarre to me.16:15
marlincOkay well, haha16:16
*** QG has joined #openstack-lbaas16:17
QGHello16:17
marlincHey there16:18
QGI have a question about the octavia plugin for Horizon, I made a small patch to be able to check if the Octavia endpoint is present in the region, do you think I should make a blueprint?16:19
johnsomQG No, a blueprint (we have been migrated to Storyboard, so don't use launchpad) should not be necessary for that. It wouldn't need an RFE either. At most a story (bug) in my opinion.16:21
QGjohnsom: ok i will create a story bug thank you16:22
johnsomNP16:24
marlincjohnsom, anyway thank you for the pointed in the right direction16:28
johnsomSure, NP16:29
johnsomOh joy, the concrete grinding outside has started. This should be a productive day16:30
gthiemongeFYI openstack-tox-pep8 fails on master16:34
johnsomgregraka Was just asking me about that. New linter package I assume16:38
johnsompylint...16:40
johnsomgthiemonge Are you on it, or should I work on a patch?16:41
gthiemongejohnsom: I'll work on it ;-)16:42
johnsomThanks16:42
gregrakaThanks, gthiemonge++ !! and Johnsom++ !!16:43
*** QG is now known as INFO16:52
*** INFO is now known as QG16:57
openstackgerritGregory Thiemonge proposed openstack/octavia master: Fix pylint errors  https://review.opendev.org/c/openstack/octavia/+/78804717:11
gthiemonge^ I choose to not disable the new checker17:12
johnsomCool. Interesting, those cases really don't make sense to use "with" IMO, so excluding them is the right answer IMO.17:14
openstackgerritQuentin GROLLEAU proposed openstack/octavia-dashboard master: Look for load-balancer endpoints in catalog  https://review.opendev.org/c/openstack/octavia-dashboard/+/78805017:28
QGgthiemonge thank you so much for the respond on storyboard, i was on stable/rocky branch so i didn't have the patch17:35
QGHowever, I looked in the existing patches but I did not find one17:36
johnsomRocky is end of life, so maybe that is why it didn't get the patch17:37
QGbut I can try to backport it ?17:37
johnsomSure, just note that there will not be any more rocky releases created17:38
johnsomThe patch will also likely not merge17:38
QGohhhhh17:38
QGOK thank you johnsom17:39
johnsomhttps://github.com/openstack/octavia/tree/rocky-eol17:39
johnsomIt was marked EOL in July last year17:40
QGk, so i put the gerrit patch as abandoned, how can i close the story as it is no longer relevant ?17:43
johnsomQC You can mark it as "Invalid"17:47
QGThank you johnsom and sorry about it18:01
johnsomNo worries18:01
*** QG has quit IRC18:06
*** vishalmanchanda has quit IRC18:21
*** stand has joined #openstack-lbaas18:42
marlincHow can I delete or failover a Amphora instance when I get the the load balancer gets marked as immutable: Load Balancer b83bcf0a-a36d-4751-91c1-6acd677c2c30 is immutable and cannot be updated. (HTTP 409) (Request-ID: req-23cfe4e2-a066-40ea-8c46-d3dcbe7ae535)19:30
johnsomYou need to wait for it to finish and become mutable19:31
johnsomHow long depends on your settings.19:31
marlincDoes it also become mutable again when the load balancer is in an ERROR state? After some timeout for example?19:34
johnsomYes19:37
marlincCool then I'll wait a bit longer!19:38
johnsomWhen the load balancer is in a PENDING_ state, one of the controllers has ownership and is working to finish your requested action. It will retry actions if nova or neutron are failing up to the retry settings in the config. It will either move to ONLINE or ERROR.19:39
marlincIs this retrying supposed to be visible in the worker? I did see an error that it got from Nova but after that it went completely quiet since about half an hour ago now19:42
johnsomWith debug you will see repeated retry messages. Make sure to check all of the controller logs.19:44
*** zzzeek has quit IRC19:44
*** zzzeek has joined #openstack-lbaas19:47
*** gregraka has quit IRC20:04
*** zzzeek has quit IRC20:09
marlincDefinitely have some more testing to do tomorrow, also with Nova20:16
*** zzzeek has joined #openstack-lbaas20:16
marlincI see Octavia nicely adding a interface for the tenant network to the Amphora instance, you can see it when viewing info about the server but the VM doesn't actually see a new interface20:18
marlincBut that's probably something I have to check in Nova not Octavia20:18
*** gcheresh has quit IRC20:18
johnsomIt does, it's in a network namespace.20:18
johnsomWe isolate tenant traffic for security reasons20:18
marlincOh interesting, I did see the interface quite a few times but that was probably was while it was still configuring things20:19
marlincAh yes I see it20:19
marlincVery nice engineering, like it!20:20
marlincThanks again20:36
*** zzzeek has quit IRC20:40
*** zzzeek has joined #openstack-lbaas20:41
*** zzzeek has quit IRC20:46
*** zzzeek has joined #openstack-lbaas20:48
*** gregraka has joined #openstack-lbaas21:27
*** spatel has quit IRC22:01
*** zzzeek has quit IRC22:42
*** zzzeek has joined #openstack-lbaas22:43
*** rcernin has joined #openstack-lbaas23:09

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!