Friday, 2020-03-20

*** TMM has quit IRC00:36
*** TMM has joined #openstack-lbaas00:36
openstackgerritMichael Johnson proposed openstack/octavia-tempest-plugin master: Add skip_if_not_implemented to the service client  https://review.opendev.org/71400300:38
*** spatel has joined #openstack-lbaas00:38
openstackgerritMichael Johnson proposed openstack/octavia-tempest-plugin master: Add skip_if_not_implemented to the service client  https://review.opendev.org/71400300:44
*** nicolasbock has quit IRC01:07
*** spatel has quit IRC01:10
*** sapd1 has joined #openstack-lbaas01:46
*** sapd1 has quit IRC01:48
*** sapd1 has joined #openstack-lbaas01:48
*** rcernin has quit IRC02:03
*** rcernin has joined #openstack-lbaas02:03
dawzonIs there a way to force tox to use a local version of octavia-lib?  I need my modified local version to pass the tests but it always uses 1.5.002:22
johnsomdawzon: the easiest way is to install the new one over the tox installed version02:48
johnsomCd .tox/pep8/bin02:49
johnsomSource activate02:49
johnsomYour prompt will change, you are now in the pep8 virtualenv.02:49
johnsomYou can the pip uninstall and install the new one02:50
johnsomdeactivate will exit the virtualenv02:50
johnsomThere is one for each tox environment02:50
johnsomThere are other ways, but this will work for you02:51
dawzonThanks!  So I'm guessing that in order for my patch to pass zuul I'll have to get the octavia-lib patch merged first02:52
johnsomYes02:53
johnsomYou can try using the depends-on tag in the commit message, but that doesn’t always work02:54
*** yamamoto has joined #openstack-lbaas03:01
*** yamamoto has quit IRC03:13
*** psachin has joined #openstack-lbaas03:26
*** yamamoto has joined #openstack-lbaas04:05
*** yamamoto has quit IRC04:12
*** vishalmanchanda has joined #openstack-lbaas05:21
*** rcernin is now known as rcernin|brb05:30
*** pckizer has quit IRC05:44
*** yamamoto has joined #openstack-lbaas06:13
*** yamamoto has quit IRC06:36
*** gcheresh has joined #openstack-lbaas07:31
*** psachin has quit IRC07:41
*** sapd1_x has joined #openstack-lbaas07:42
*** rpittau|afk is now known as rpittau07:57
*** maciejjozefczyk has joined #openstack-lbaas08:00
openstackgerritMerged openstack/octavia master: Fix load balancer update with provider filtered params  https://review.opendev.org/69558908:06
*** gcheresh has quit IRC08:07
openstackgerritCarlos Goncalves proposed openstack/octavia stable/train: Fix load balancer update with provider filtered params  https://review.opendev.org/71404308:09
*** sapd1_x has quit IRC08:17
*** yamamoto has joined #openstack-lbaas08:54
*** yamamoto has quit IRC08:58
openstackgerritCarlos Goncalves proposed openstack/octavia stable/train: Fix load balancer update with provider filtered params  https://review.opendev.org/71404309:20
openstackgerritCarlos Goncalves proposed openstack/octavia stable/stein: Fix load balancer update with provider filtered params  https://review.opendev.org/71405809:29
*** ramishra has quit IRC09:38
*** ramishra has joined #openstack-lbaas09:46
*** ramishra has quit IRC09:50
*** ramishra has joined #openstack-lbaas09:50
*** yamamoto has joined #openstack-lbaas09:57
*** yamamoto has quit IRC10:11
*** yamamoto has joined #openstack-lbaas10:21
sapd1I'm creating L7 policy for a listener. There is no error in Octavia-api and octavia-worker. But the configuration of haproxy inside amphora wasnt changed.10:25
*** nicolasbock has joined #openstack-lbaas11:03
kklimondahmm, should pool members have operating_status set to ERROR when healthmonitor check fails due to unexpected return code?11:07
kklimondaIs there a way for users to see the cause of error somehow?11:07
cgoncalveskklimonda, yes, pool members should be in ERROR in such case you described. see https://docs.openstack.org/api-ref/load-balancer/v2/#status-codes11:27
cgoncalveskklimonda, no way to for users to see the cause of error via octavia api but they could reproduce the check themselves directly against the member11:28
kklimondathanks, that's less than ideal but such is life :)11:28
kklimonda(I know, patches welcome :D)11:29
cgoncalvessapd1, I have not tested L7 policies in months. may be a bug maybe introduced in the single-process listeners work, just a possibility of regression11:29
cgoncalvessapd1, we do have traffic scenario tests for L7 policies so disregard what I said above11:32
cgoncalveshttps://github.com/openstack/octavia-tempest-plugin/blob/master/octavia_tempest_plugin/tests/scenario/v2/test_traffic_ops.py11:32
*** rpittau is now known as rpittau|bbl11:37
*** yamamoto has quit IRC11:51
sapd1cgoncalves, That was my fault. Sorry.11:52
cgoncalvessapd1, no problem. good to have your input that it works ;)11:58
*** ramishra has quit IRC12:06
*** ramishra has joined #openstack-lbaas12:28
*** gcheresh has joined #openstack-lbaas12:35
*** gcheresh has quit IRC12:59
*** yamamoto has joined #openstack-lbaas13:10
*** gcheresh has joined #openstack-lbaas13:10
*** yamamoto has quit IRC13:16
*** yamamoto has joined #openstack-lbaas13:24
*** yamamoto has quit IRC13:29
*** TrevorV has joined #openstack-lbaas13:37
-openstackstatus- NOTICE: Gerrit (review.opendev.org) will be down for maintenance starting at 14:00 (in less than 20 mins), probably until 15:00 UTC13:43
*** yamamoto has joined #openstack-lbaas13:43
*** rpittau|bbl is now known as rpittau13:52
*** yamamoto has quit IRC13:55
-openstackstatus- NOTICE: The Gerrit service on review.opendev.org is offline for maintenance until 15:00 UTC http://lists.opendev.org/pipermail/service-announce/2020-March/000001.html14:07
*** ChanServ changes topic to "The Gerrit service on review.opendev.org is offline for maintenance until 15:00 UTC http://lists.opendev.org/pipermail/service-announce/2020-March/000001.html"14:07
*** yamamoto has joined #openstack-lbaas14:17
*** yamamoto has quit IRC14:18
*** yamamoto has joined #openstack-lbaas14:20
*** yamamoto has quit IRC14:21
*** TrevorV has quit IRC14:42
*** TrevorV has joined #openstack-lbaas14:45
*** ChanServ changes topic to "Discussions for OpenStack Octavia | Priority bug review list: https://etherpad.openstack.org/p/octavia-priority-reviews"15:04
-openstackstatus- NOTICE: Gerrit maintenance is concluded at this time and requested renames have been performed15:04
openstackgerritBrian Haley proposed openstack/octavia-tempest-plugin master: Use same flake8 extensions as Octavia  https://review.opendev.org/71414215:16
*** gcheresh has quit IRC15:44
*** vishalmanchanda has quit IRC16:40
*** openstack has joined #openstack-lbaas16:58
*** ChanServ sets mode: +o openstack16:58
*** maciejjozefczyk has quit IRC17:59
rm_workkklimonda: if you use log offloading, and expose the haproxy logs to the users, then they can look and see :)18:03
cgoncalvesah, good point. I forgot that option18:05
rm_workspeaking of which, today I try to fix the UDP HMs18:07
rm_workT_T18:07
johnsomWe have docs for that! <grin> https://docs.openstack.org/octavia/latest/admin/log-offloading.html#tenant-flow-logs18:12
* johnsom feels like he is stuck in a skit for Portlandia. Put a bird on it!18:12
rm_workportlandia is so eerily accurate i had to stop watching it because i have visited portland a lot and it's just ... like watching real life happen18:14
johnsomIt is very true. Especially the early seasons18:14
johnsomhttps://youtu.be/G-ppr9LDaQE18:15
rm_workMy favorite was their NPR thing18:21
rm_workhttps://www.youtube.com/watch?v=YsufMtUOXtI18:22
johnsomlol, yeah, that is a good one too18:22
rm_workthe blue apron reference was just ... perfect18:23
rm_workdo you know know to make a catchall for a log handler?18:27
rm_worktried: qualname = sqlalchemy.*18:27
rm_worknot working18:27
rm_workah because root_logger overrides it18:30
rm_workerr, logger_root18:30
johnsomIt depends on the logger used and the OS overrides18:32
johnsomIf you look at our amphora template: https://github.com/openstack/octavia/blob/master/octavia/common/jinja/logging/templates/10-rsyslog.conf.template18:32
johnsomI use *.* stop to block logging to the local disk (if that setting is enabled)18:33
johnsomAnther way is to define a input or output with no filters.18:33
johnsomhttps://github.com/openstack/octavia/blob/master/devstack/etc/rsyslog/10-octavia-log-offloading.conf18:34
johnsomOur devstack config ^^ defines two inputs and filters on facility and severity.18:34
rm_workoh i mean python18:34
johnsomOh, ha18:34
johnsomYeah, less familiar with that18:34
rm_workwe use these split log configs for every service18:35
rm_workand it's overriding `debug=true` in the main config18:35
rm_workAH got it, fff, forgot to add my new logger to the logger list18:37
johnsomEh gads. I'm like six levels deep of _methods here.18:45
johnsomMe things this stuff needs to be pulled out of test_base.py18:46
johnsomAlso because I might be on a path of adding a few more. sigh.18:47
*** KeithMnemonic has joined #openstack-lbaas19:06
KeithMnemonicjohnsom Hi Michael how are you doing. I hope you are holding up well. Do you have a minute or two for a few quick questions on db migration for the octavia db. specifically here  https://github.com/openstack/octavia/tree/stable/pike/octavia/db/migration/alembic_migrations/versions  this is not a question about pike specifically but more general19:09
rm_workdaily plug for https://review.opendev.org/#/q/topic:az-tweaks+status:open19:21
rm_workjohnsom: could you also respond on https://review.opendev.org/#/c/711298/ ?19:22
johnsomKeithMnemonic Hi, caught me at lunch (back now). I am well and hope you are as well.  What can I answer about migrations?19:44
openstackgerritAdam Harwell proposed openstack/octavia master: Support HTTP and TCP checks in UDP healthmonitor  https://review.opendev.org/58918019:45
rm_work^^ rebase19:45
rm_workshould be good to go though, deployed in my cloud and seems to work :D19:45
KeithMnemonici sort of figured it out, i have a customer with two files in the versions dir and have no idea where they came from. they are not upstream and not in any of the suse or old hos repos19:46
KeithMnemonicthese are causing his db failure19:46
johnsomAh, well then, "not our problem" lol19:46
KeithMnemonicthey are basically "noop"s as they do nothing other than a version change19:46
johnsomYeah, well, they would need to make sure the chain is still intact.19:46
KeithMnemonicbut not sure if they got these from some vendor that hooks into octavia or what19:47
johnsomThat would be a horrible idea and a sin itself, lol19:47
KeithMnemonicdoes this version ever ring a bell to you "82ca3350c2a3"19:47
johnsomNope, we never had it upstream19:48
johnsomhttps://www.irccloud.com/pastebin/bkgr9mTt/19:49
johnsomThat is all I have19:49
johnsomWonder if they mixed neutron-lbaas migrations into Octavia? Bad bad bad idea of course19:50
johnsomNope, not in the last neutron-lbaas repo I have laying around either19:51
johnsomNo hits in codesearch.openstack.org either, so likely a customer science experiment19:51
johnsomrm_work So, my experience with OSC has been that --long means everything, no matter how ugly it gets.19:53
johnsomrm_work That is why I raised the flag.19:54
rm_workhmm19:54
rm_workok19:54
johnsomWe can of course poll others for their thoughts.19:54
rm_worki kinda dislike that but i see the point19:54
johnsomYou could add "--more" lol.....19:54
johnsom"--bit-more" "--some" "--my-fav"19:55
johnsomLet me look at "openstack network list --long" and compare19:55
johnsomHmm, yeah, ok, so maybe I am remembering wrong. --long on network list is only three lines of column titles(normal is two). certainly not all of the columns, though guessing if that is just because no one updated the OSC plugin or not.19:58
johnsomIt just seems odd to have an intermediary option with no "give me all of them" that would be nice for filtering via OSC.19:59
johnsomWe can't do "--long" and "--all" because --all has other meaning19:59
johnsomhttps://docs.openstack.org/python-openstackclient/train/contributor/command-options.html#additional-fields20:01
KeithMnemonicjohnsonm i think we figured it out, if you run octavia-db-manage revision it seems to create them20:02
KeithMnemonicand messes up the history,...20:02
johnsomKeithMnemonic "revision" *is* the command to create a *new* revision..... lol20:03
johnsomrevision            Create a new revision file.20:03
KeithMnemonicyou needed a laugh today i bet20:04
johnsomlol, yes20:04
openstackgerritMerged openstack/octavia-tempest-plugin master: Use same flake8 extensions as Octavia  https://review.opendev.org/71414220:06
johnsomupgrade, head, and check_migration are probably the only commands a non-developer type would run20:07
johnsomMaybe current and history20:07
johnsomAh, I guess "head" is the old command. "current" replaced it20:08
rm_workhmmmmmmmmm21:19
KeithMnemonicjohnsom we fixed it, thanks and take care21:19
johnsomo/21:19
*** TrevorV has quit IRC21:27
rm_workjohnsom: hmm, DB goes to R/O -- lots of LBs go to ERROR. thoughts?21:36
rm_workI saw some failover notifications, which semi-makes-sense? except, i thought we fixed that issue; and also, if the DB is R/O it wouldn't even get as far as marking the LB in PENDING21:36
rm_workright?21:36
johnsomHow can they be marked ERROR if the DB is read only?21:37
rm_workright21:37
rm_workwell i mean, it came back21:37
rm_worki'm worried that we disable healthchecks for some amount of time on a DB outage, but not if it's still readable?21:38
johnsomYeah, so, the safety we put in only detects a DB missing, not read only. So, if it exists, but is read only the safety "wait for a few minutes" will not fire21:38
rm_workyeah :/21:38
rm_workso then the second it comes back up21:38
rm_workeverything tried to go failover, i assume21:38
johnsomRight, HMs will get busy21:38
rm_workis it possible it tried to failover both backup and master at the same time?21:39
johnsomNo21:39
rm_workwhich would obviously fail currently -- not sure if we have guards in place for that21:39
johnsomhttps://github.com/openstack/octavia/blob/master/octavia/controller/healthmanager/health_manager.py#L11521:40
johnsomYeah, first one in locks the LB, so only one amp per lb at a time21:40
johnsomBut, remember, the first one might fail, go error, and the second one come back around....21:40
openstackgerritMerged openstack/octavia master: Add oslo middleware healthcheck to Octavia API  https://review.opendev.org/71112721:43
openstackgerritMerged openstack/octavia stable/train: Remove the barbican "Grant access" from cookbook  https://review.opendev.org/71353521:43
openstackgerritMerged openstack/octavia stable/stein: Remove the barbican "Grant access" from cookbook  https://review.opendev.org/71353621:43
openstackgerritMerged openstack/octavia stable/rocky: Remove the barbican "Grant access" from cookbook  https://review.opendev.org/71353721:43
rm_workhmmm looks like we're seeing port detach timeouts which might account for it21:44
rm_worka lot of LBs failed over due to the DB R/O thing21:44
rm_workand then ... about half of them had one amp do that21:44
johnsomYeah, if the DB is RO, nova is probably hosed too21:45
johnsomThe new failover flow will help with that for sure.21:45
rm_workwtf i'm getting plugging errors NOW tho21:56
rm_workthe DB RO thing was like, a day ago21:56
openstackgerritMerged openstack/octavia master: Add a periodic image build job  https://review.opendev.org/70639321:58
*** KeithMnemonic has quit IRC22:02
*** nicolasbock has quit IRC22:24
*** hongbin has joined #openstack-lbaas22:33
*** lxkong has quit IRC23:23
*** lxkong has joined #openstack-lbaas23:24
*** TMM has quit IRC23:25
*** TMM has joined #openstack-lbaas23:25
*** irclogbot_3 has quit IRC23:27
*** irclogbot_1 has joined #openstack-lbaas23:28
*** guilhermesp_ has joined #openstack-lbaas23:36
*** guilhermesp has quit IRC23:37
*** guilhermesp_ is now known as guilhermesp23:37
openstackgerritAdam Harwell proposed openstack/octavia master: Fix padding logic for UDP health daemon  https://review.opendev.org/71424023:57

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!