Tuesday, 2019-07-30

*** threestrands has joined #openstack-lbaas00:10
rm_workI mean, what is that looking for? If it's an internal tempest test thing (like testing flavor creation) we don't need it to be configurable, we can just generate some random description00:12
rm_workAnd if it's testing something external... What?00:13
rm_workAnd wait, is that expecting it as the name or as the description? I'm confused00:14
johnsomIt is testing the flavor profile api.00:29
johnsomIt is an octavi.conf setting exposed via the api00:30
*** ysandeep has joined #openstack-lbaas00:32
*** yamamoto has joined #openstack-lbaas00:32
*** ricolin_ has joined #openstack-lbaas00:58
rm_workjohnsom: right so if it's testing that api... we can list, save the list, create one, list, expect the new one to be added in the list, get the new one explicitly, check it, delete it01:08
rm_workwe don't need to have the operator configure tempest to expect whatever is currently in their env?01:08
rm_workjust seems weirdly fragile01:08
rm_workto expect some exact existing *description*01:09
rm_workif you think it isn't, and you tell me right now "this is the way we should do it", then i'll change my vote, but it just feels awkward01:10
johnsomWe can’t control oslo config from tempest right?01:16
*** abaindur has quit IRC01:50
*** ysandeep has quit IRC02:36
johnsomrm_work how do we create an oslo_config record in the API process from tempest?02:40
rm_workErr? A flavor isny an oslo_config record?02:43
rm_work*isn't02:43
rm_workAren't they just DB entries with an API?02:44
johnsomWhat that test is looking at is static records configured in the octavia.conf02:45
rm_workAh I missed the line where you said it's an octavia.conf entry -- but... It isn't? Is it? Why would a flavor profile be a config record02:45
rm_workAll the flavor stuff is API created in the DB I thought02:45
rm_workAnd if it isn't... Why the heck isnt it?02:45
rm_workI feel like maybe I missed that patch chain or else totally missed understanding this part if I did review it...02:46
johnsomAh, no this is not in octavia.conf, I'm thinking of another patch. This is defined by the provider driver.02:47
johnsomIt's in the code here: https://github.com/openstack/octavia/blob/master/octavia/api/drivers/amphora_driver/flavor_schema.py02:49
rm_workOk so could you move that to a constant and use the constant as a default then02:50
rm_workIf it's really defined by us in code02:50
johnsomI made it a config as the amphora driver may not be enabled in the cloud under test02:50
rm_workDo we import other Octavia constants to use? Hmmm02:50
johnsomOther drivers will have other strings02:52
rm_workYeah so leave it as a config02:53
*** hongbin has joined #openstack-lbaas02:53
rm_workBut use the same constant as the default that actually defines it?02:53
rm_workOr else we just admit this is a little janky but we don't want to change it and I'll just +2 I guess, lol02:54
johnsomIt's two different repos, so it can't point to the same code.02:55
johnsomI agree it is a bit annoying, but I'm not sure how to test that API another way02:56
johnsomI guess we could blind check that there is "a" string in each field03:00
*** threestrands has quit IRC03:30
*** threestrands has joined #openstack-lbaas03:30
*** threestrands has quit IRC03:31
*** hongbin has quit IRC03:34
*** psachin has joined #openstack-lbaas03:37
*** ramishra has joined #openstack-lbaas04:00
*** abaindur has joined #openstack-lbaas04:34
openstackgerritHidekazu Nakamura proposed openstack/octavia master: Add install guide for Ubuntu  https://review.opendev.org/67284204:57
*** gcheresh has joined #openstack-lbaas04:59
*** ricolin__ has joined #openstack-lbaas05:21
*** ricolin_ has quit IRC05:24
*** yamamoto has quit IRC05:26
*** yamamoto has joined #openstack-lbaas05:27
*** gcheresh has quit IRC05:32
*** gcheresh has joined #openstack-lbaas05:38
*** vishalmanchanda has joined #openstack-lbaas05:53
*** ataraday has quit IRC06:04
*** maciejjozefczyk has joined #openstack-lbaas06:30
openstackgerritOpenStack Proposal Bot proposed openstack/octavia-dashboard master: Imported Translations from Zanata  https://review.opendev.org/67340906:45
*** abaindur has quit IRC06:52
*** rpittau|afk is now known as rpittau06:58
*** abaindur has joined #openstack-lbaas07:00
*** rcernin has quit IRC07:02
openstackgerritGregory Thiemonge proposed openstack/octavia master: Add support for monitor_{address,port} in UDP members  https://review.opendev.org/66130807:05
openstackgerritGregory Thiemonge proposed openstack/octavia master: Fixed pool and members status with UDP loadbalancers  https://review.opendev.org/66130907:05
openstackgerritGregory Thiemonge proposed openstack/octavia master: Fixed down server issue after reloading keepalived  https://review.opendev.org/66131007:05
*** lemko has joined #openstack-lbaas07:06
*** tesseract has joined #openstack-lbaas07:11
*** ccamposr has joined #openstack-lbaas07:33
*** ccamposr__ has joined #openstack-lbaas07:33
*** ccamposr has quit IRC07:33
*** ricolin__ is now known as ricolin07:44
*** ivve has joined #openstack-lbaas07:49
*** happyhemant has joined #openstack-lbaas08:12
*** abaindur has quit IRC08:15
*** ivve has quit IRC08:48
openstackgerritpengyuesheng proposed openstack/octavia-lib master: Blacklist sphinx 2.1.0 (autodoc bug)  https://review.opendev.org/67287808:59
*** tkajinam has quit IRC09:03
*** ivve has joined #openstack-lbaas09:17
rm_workjohnsom: sounds like "yeah yeah it's janky, +2 it already" which ... sure, I guess whatev :P09:42
*** sapd1_x has joined #openstack-lbaas10:02
*** sapd1_x has quit IRC10:22
openstackgerritGregory Thiemonge proposed openstack/octavia master: Add support for monitor_{address,port} in UDP members  https://review.opendev.org/66130810:32
openstackgerritGregory Thiemonge proposed openstack/octavia master: Fixed pool and members status with UDP loadbalancers  https://review.opendev.org/66130910:32
openstackgerritGregory Thiemonge proposed openstack/octavia master: Fixed down server issue after reloading keepalived  https://review.opendev.org/66131010:32
*** yamamoto has quit IRC10:32
rm_workcgoncalves: could you revisit https://review.opendev.org/662246 ?10:42
*** ataraday has joined #openstack-lbaas10:43
rm_workgthiemonge: cool, will revisit those patches later today10:52
openstackgerritMerged openstack/octavia-lib master: Add get methods to the driver-lib  https://review.opendev.org/66502710:55
openstackgerritMerged openstack/python-octaviaclient master: Adds "unset" action to the healthmonitor command  https://review.opendev.org/66224610:58
openstackgerritMerged openstack/python-octaviaclient master: Adds "unset" action to the flavor command  https://review.opendev.org/66234810:58
openstackgerritMerged openstack/python-octaviaclient master: Adds "unset" action to the l7policy command  https://review.opendev.org/66251810:59
openstackgerritMerged openstack/python-octaviaclient master: Adds "unset" action to the l7rule command  https://review.opendev.org/66254011:00
openstackgerritMerged openstack/python-octaviaclient master: Adds "unset" action to the quota command  https://review.opendev.org/66258811:00
cgoncalvesall unset patches merged11:01
*** lemko has quit IRC11:03
*** yamamoto has joined #openstack-lbaas11:12
*** yamamoto has quit IRC11:18
*** yamamoto has joined #openstack-lbaas11:19
rm_workcgoncalves: noice11:20
*** mkuf has joined #openstack-lbaas11:29
*** ramishra has quit IRC11:35
*** ramishra has joined #openstack-lbaas11:46
*** boden has joined #openstack-lbaas11:47
*** yamamoto has quit IRC11:57
*** yamamoto has joined #openstack-lbaas12:12
*** yamamoto has quit IRC12:17
*** psachin has quit IRC12:25
*** maciejjozefczyk has quit IRC12:25
*** maciejjozefczyk has joined #openstack-lbaas12:27
*** ricolin has quit IRC12:35
openstackgerritAnn Taraday proposed openstack/octavia master: Convert pool flows to use dicts  https://review.opendev.org/66538112:36
openstackgerritAnn Taraday proposed openstack/octavia master: Transition amphora flows to dicts  https://review.opendev.org/66889812:36
*** yamamoto has joined #openstack-lbaas12:37
openstackgerritMerged openstack/octavia master: Bump the openstackdocstheme extension to 1.20  https://review.opendev.org/67098612:38
openstackgerritAnn Taraday proposed openstack/octavia master: [WIP] Lb flows to dicts  https://review.opendev.org/67172512:39
openstackgerritAnn Taraday proposed openstack/octavia master: [WIP] Jobboard based controller  https://review.opendev.org/64740613:06
openstackgerritAnn Taraday proposed openstack/octavia master: Transition member flows to use dicts  https://review.opendev.org/65784213:11
*** gcheresh has quit IRC13:17
*** goldyfruit has joined #openstack-lbaas13:24
openstackgerritCarlos Goncalves proposed openstack/octavia stable/stein: Fix multi-listener load balancers  https://review.opendev.org/67351813:28
cgoncalvesjust a few file conflicts13:28
*** ysandeep has joined #openstack-lbaas13:33
*** yamamoto has quit IRC13:33
rm_worki am so ... concerned13:34
rm_workabout this backport13:34
rm_workbut I am always the one who is like "yeah whatever, we can backport that" so not going to complain :D13:35
openstackgerritSandeep Yadav proposed openstack/octavia master: Correcting typo in received spelling  https://review.opendev.org/67352113:35
*** yamamoto has joined #openstack-lbaas13:36
*** yamamoto has quit IRC13:36
*** gcheresh has joined #openstack-lbaas13:37
*** yamamoto has joined #openstack-lbaas13:37
*** yamamoto has quit IRC13:42
bcafarelfirst time I see a backport where the conflict lines list does not fit the view indeed :)14:02
*** rpittau is now known as rpittau|afk14:02
*** lemko has joined #openstack-lbaas14:09
*** ysandeep has quit IRC14:15
*** yamamoto has joined #openstack-lbaas14:19
*** Vorrtex has joined #openstack-lbaas14:25
*** yamamoto has quit IRC14:28
rm_work+4030, -286614:47
rm_worknice backport <_<14:48
rm_workvery simple14:48
colin-quite14:48
*** takamatsu has joined #openstack-lbaas14:54
*** gcheresh has quit IRC14:58
*** ricolin has joined #openstack-lbaas15:14
cgoncalvesis there such a thing as a test (not job) timeout in tempest?15:26
johnsomYes, I think so15:28
cgoncalvesok, that is the only possible reason I can find that would explain the scalability test entering cleanup -- https://review.opendev.org/#/c/669541/15:29
johnsomOh, there are a bunch of issues there. I marked it WIP15:30
johnsomYeah, that is hitting a timeout of some sort, but I haven't figured out which one / where.15:31
cgoncalveslooking for the same15:31
colin-is the new process named "driver-agent" similar to "worker" and "api"?15:46
johnsomIt is a partner process to the API. It should run co-located with the API. It is there to support provider drivers.15:47
colin-oh i phrased that ambiguously, i wanted to validate its name15:47
colin-the upgrade note was nice and clear about its role15:47
johnsomOh, on devstack it is o-da. In other places... I'm not sure what it is called off my head.15:48
colin-works for me15:49
johnsomoctavia-driver-agent15:50
johnsomThat looks to be the command name setup utils creates15:50
cgoncalvesin TripleO land, the container is named "octavia-driver-agent"15:50
colin-last Q: i didn't see a bind port for the driver_agent in https://docs.openstack.org/octavia/stein/configuration/configref.html#driver-agent15:51
colin-does it need to expose any tcp/udp listener?15:51
johnsomIt doesn't listen on any network ports right now. It communicates over Unix domain sockets and calls out to the DB15:51
colin-got it, thanks15:52
colin-i lied; last, last Q: if i'm running these in containers and add a volume to the driver-agent for /var/run/octavia/status.sock and the stats.sock, i'll also want the other four services (also in containers) to mount that volume (they run on a common compute) right?15:56
johnsomNo worries, questions are good.15:56
johnsomBonus points if you write up a FAQ. grin15:56
johnsomIt's best if you don't do per-socket-file as we will add sockets (lib-getter is coming soonish). The only processes that need those are the API and the driver-agent15:57
johnsomThe rest don't need access to that15:57
colin-i wondered which actually need it as i only saw the socket path mentioned in the dirver-agent config settings and not the other services15:58
colin-perhaps just all of /var/run/octavia then to the api and driver-agent containers15:59
johnsomcgoncalves https://opendev.org/openstack/tempest/src/branch/master/tempest/lib/base.py#L4715:59
johnsomcolin- In container speak-ish, the driver-agent is a sidecar to the API.16:00
johnsomI don't know, it seems like all of the container terms are overloaded and mis-used.16:01
cgoncalvesjohnsom, wow, nicely done!16:01
cgoncalveshttps://opendev.org/openstack/tempest/src/branch/master/tox.ini#L6116:01
cgoncalves20 minutes timeout16:02
johnsomSo the magic question is how fine grained we can adjust that. Do we really want to extend it for all of the tests? probably not16:03
johnsomMaybe we set the listener count to configurable, something smaller for the normal gates, then do a periodic that does 1,000's?16:06
johnsomIdeally we would test up to the 65k limit16:06
cgoncalves65k listeners? brave!!16:07
cgoncalvesprobably that is our best option, yes16:07
johnsomWell, it *should* work, so why not prove it16:07
johnsomYou know someone is going to do it.  Like some game developer.....16:08
johnsomgrin16:08
cgoncalvesah, doesn't the API support full LB tree create?16:08
cgoncalveslol, yeah, those guys ;)16:09
johnsomIt does. I thought about that. I had it do them individually to test this one failure case, but maybe a hybrid is the right answer.16:09
cgoncalvesalso LB delete cascade16:09
cgoncalvesmaybe we have good chances to create 65k listeners ;)16:10
johnsomcgoncalves Do you want to run with that test patch and propose how we should run it?16:13
cgoncalvesI sense a trap in that question16:15
* johnsom this is not the trap you are looking for.....16:15
johnsomI remember long ago you advised me to delegate more....16:16
cgoncalveswoah! https://review.opendev.org/#/c/673518/ passed at first run16:16
*** ajay33 has joined #openstack-lbaas16:19
ajay33Hi,16:19
johnsomajay33 Hi16:20
* ajay33 Can you let me know the use of line number 488 (dhclient -v o-hm0) ?16:21
johnsomAh, ok. So in devstack (other ways to deploy may not use this), we create an interface, o-hm0, on the lb-mgmt-net for the control plane processes to access the lb-mgmt-net.16:22
ajay33I am doing manual installation and current not getting IP of the above command (dhclient -v o-hm0) mentioned in pulgin.sh file16:22
johnsomThe way the devstack script creates the lb-mgmt-net is to have DHCP enabled on it. So, the "dhclient" call is configuring this o-hm0 port to get an IP address from the neutron DHCP process.16:23
johnsomCheck the subnet you are using for o-hm0. Make sure neutron has it as dhcp enabled.16:23
ajay33I am searching, how to check this and will let you know.16:28
ajay33JFYI, my neutron-dhcp agent is running fine.16:28
ajay33I just want to confirm one more this, the port I had created (using command mentioned on line number 368 of pugin.sh) is in DOWN state, Is this can be a possible reason for dhclient is not able to assign IP to my o-hm0?16:32
*** trident has quit IRC16:32
johnsomWhen you do an "openstack subnet show <id>" it should have a row for that16:35
johnsomAh, it should not be in a down state. That is likely your issue16:36
*** trident has joined #openstack-lbaas16:37
ajay33@johnsom, yes DHCP is enabled on my subnet. I had confirmed it using openstack subnet show  command16:39
cgoncalvescheck if the o-hm0 neutron port is bound with "openstack port show <PORT_ID>"16:43
cgoncalvesif it is not, it is likely that the o-hm0 is not attached to OVS with all the necessary information16:44
ajay33I had tried to update the port status using "neutron port-update" command,  but it returning "Cannot update read-only attribute status"16:44
openstackgerritMichael Johnson proposed openstack/octavia-tempest-plugin master: Adds provider flavor capabilities API tests  https://review.opendev.org/63111316:45
cgoncalvesajay33, right, you should not do that16:46
johnsomYeah, if it is admin-state-up True, that is all you can do from the API. You would need to figure out why the port is not able to come up in neutron.16:46
cgoncalveshttps://github.com/openstack/octavia/blob/master/devstack/plugin.sh#L37616:47
openstackgerritMichael Johnson proposed openstack/octavia-tempest-plugin master: Add a flavor to the load balancer CRUD scenarios  https://review.opendev.org/63135316:52
openstackgerritMichael Johnson proposed openstack/octavia-tempest-plugin master: Add amphora update service client and API test  https://review.opendev.org/63329516:53
openstackgerritMichael Johnson proposed openstack/octavia-tempest-plugin master: Add amphora failover API test  https://review.opendev.org/63361416:53
ajay33@cgoncalves, I had check the command "openstack port show 7a4f1d2b-197b-4a5b-a9a3-f3b0b83c1ef7 | grep "o-hm0"" and getting NO Output..16:53
ajay33Is my o-hm0 is not attached to OVS?16:53
johnsomOk, merge conflicts and rebases done on that chain16:53
*** goldyfruit has quit IRC16:53
*** shubjero has quit IRC16:57
*** shubjero has joined #openstack-lbaas16:59
cgoncalvesajay33, I'm not sure how you created the neutron port. in devstack at least, the neutron port does not contain the "o-hm0" string anywhere17:07
cgoncalvesajay33, do a port show and check for the binding parameter.17:07
cgoncalvesif you can, share the output as well as the 'ovs-vsctl add-port' command you used to create the port (if you're deploying similarly to how the devstack plugin does it)17:11
*** ricolin has quit IRC17:11
ajay33In my setup, I am using linux bridge with ml2.17:13
ajay33I will paste the output. of that port.17:14
cgoncalvesah, ok. I've never tried with ML2/Linuxbridge17:14
johnsomWith linux bride you don't follow the OVS steps. Double check that your bridge is up and you need to manually up the bridge interface.17:16
johnsomhttps://github.com/openstack/octavia/blob/master/devstack/plugin.sh#L37717:17
*** eandersson has joined #openstack-lbaas17:20
*** tesseract has quit IRC17:34
openstackgerritAndreas Jaeger proposed openstack/octavia-dashboard master: Update api-ref location  https://review.opendev.org/67360217:55
ajay33@johnsom my linux bridge is UP. I had confirmed this18:06
openstackgerritMerged openstack/octavia-dashboard master: Imported Translations from Zanata  https://review.opendev.org/67340918:09
openstackgerritMerged openstack/octavia-dashboard master: Update api-ref location  https://review.opendev.org/67360218:19
*** yamamoto has joined #openstack-lbaas18:27
*** yamamoto has quit IRC18:31
*** boden has quit IRC19:05
*** lemko has quit IRC19:07
*** boden has joined #openstack-lbaas19:09
openstackgerritMerged openstack/octavia master: Fix l7rule API handling of None updates  https://review.opendev.org/66256919:19
openstackgerritMerged openstack/octavia-lib master: Bump the openstackdocstheme extension to 1.20  https://review.opendev.org/67287919:19
*** abaindur has joined #openstack-lbaas19:44
*** abaindur has quit IRC19:46
*** abaindur has joined #openstack-lbaas19:46
*** ramishra has quit IRC20:15
*** boden has quit IRC20:15
openstackgerritMichael Johnson proposed openstack/octavia-lib master: Add constants to octavia-lib  https://review.opendev.org/67368120:29
*** vishalmanchanda has quit IRC20:32
*** numans has quit IRC20:36
openstackgerritMichael Johnson proposed openstack/octavia master: Fix a unit test for new octavia-lib  https://review.opendev.org/67368720:52
johnsom^^^ That one was my bad. We should merge it soonish20:52
*** Vorrtex has quit IRC21:25
*** maciejjozefczyk has quit IRC21:41
openstackgerritMichael Johnson proposed openstack/octavia-lib master: Add constants to octavia-lib  https://review.opendev.org/67368121:42
*** JayF has joined #openstack-lbaas21:45
*** ivve has quit IRC22:23
*** ajay33 has quit IRC22:25
*** yamamoto has joined #openstack-lbaas22:29
*** yamamoto has quit IRC22:33
*** tkajinam has joined #openstack-lbaas22:57
*** rcernin has joined #openstack-lbaas23:01
*** yamamoto has joined #openstack-lbaas23:11
openstackgerritMichael Johnson proposed openstack/octavia-lib master: Add constants to octavia-lib  https://review.opendev.org/67368123:23
openstackgerritMichael Johnson proposed openstack/octavia master: Lookup interfaces by MAC directly  https://review.opendev.org/67333723:40
openstackgerritMichael Johnson proposed openstack/octavia master: Fix L7 repository create methods  https://review.opendev.org/67315423:40
openstackgerritMichael Johnson proposed openstack/octavia master: Fix provider driver utils  https://review.opendev.org/67315523:40
johnsomThese dependency chains are getting insane23:44
openstackgerritMichael Johnson proposed openstack/octavia master: Add get method support to the driver-agent  https://review.opendev.org/66502923:44
openstackgerritMichael Johnson proposed openstack/octavia master: Add get method support to the driver-agent  https://review.opendev.org/66502923:47
openstackgerritMichael Johnson proposed openstack/octavia master: Move to using octavia-lib constants  https://review.opendev.org/67371223:52
johnsomThat was painful23:53
johnsomOk, so cleaned up a bit of tech debt...23:54
openstackgerritMichael Johnson proposed openstack/octavia-lib master: Blacklist sphinx 2.1.0 (autodoc bug)  https://review.opendev.org/67287823:58

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