Monday, 2019-10-14

*** ramishra has joined #openstack-lbaas02:19
*** psachin has joined #openstack-lbaas03:35
*** ricolin has joined #openstack-lbaas04:12
*** ricolin_ has joined #openstack-lbaas04:15
*** ricolin has quit IRC04:19
*** gcheresh_ has joined #openstack-lbaas06:27
*** pcaruana has joined #openstack-lbaas06:43
*** maciejjozefczyk has joined #openstack-lbaas06:46
*** gcheresh_ has quit IRC06:51
openstackgerritVishal Manchanda proposed openstack/octavia-dashboard master: Switch to official Ussuri jobs  https://review.opendev.org/68833607:02
*** tesseract has joined #openstack-lbaas07:04
*** rcernin has quit IRC07:11
*** rpittau|afk is now known as rpittau07:40
*** ivve has joined #openstack-lbaas07:46
*** ricolin_ is now known as ricolin08:41
frickleris anyone still fixing py3 issues in stable neutron-lbaas versions like this one in rocky? http://paste.openstack.org/show/783587/08:43
cgoncalvesfrickler, resources shifted to octavia long ago. barely a handful of folks still looking after neutron-lbaas and even less py3 in rocky. nevertheless, bug fixes are welcome to stein and earlier versions08:51
openstackgerritMerged openstack/octavia-dashboard master: Switch to official Ussuri jobs  https://review.opendev.org/68833608:51
fricklercgoncalves: o.k., I created a story for it as the first step. we'll probably also discuss speeding up the migration to octavia internally https://storyboard.openstack.org/#!/story/200671408:54
cgoncalvesfrickler, thanks for capturing this in storyboard!08:55
cgoncalvesfrickler, some resources you might find helpful for the migration:08:57
cgoncalveshttps://wiki.openstack.org/wiki/Neutron/LBaaS/Deprecation08:57
cgoncalveshttps://www.youtube.com/watch?v=jj4KMJPA0Pk08:57
*** trident has quit IRC09:23
*** trident has joined #openstack-lbaas09:28
*** salmankhan has joined #openstack-lbaas09:35
*** salmankhan has quit IRC09:37
*** ricolin has quit IRC09:41
openstackgerritMaciej Józefczyk proposed openstack/octavia-tempest-plugin master: Add an option to reuse connections  https://review.opendev.org/67297609:50
openstackgerritMaciej Józefczyk proposed openstack/octavia-tempest-plugin master: Handle NotImplemented flavor exception  https://review.opendev.org/67613509:54
openstackgerritJens Harbott (frickler) proposed openstack/neutron-lbaas stable/rocky: WIP: Misc Python 3.6 fixes  https://review.opendev.org/68837910:32
*** maciejjozefczyk has quit IRC10:40
*** maciejjozefczyk has joined #openstack-lbaas10:41
*** gcheresh_ has joined #openstack-lbaas10:41
openstackgerritAnn Taraday proposed openstack/octavia master: Add option to set default ssl ciphers in haproxy  https://review.opendev.org/68533710:46
*** ioni has joined #openstack-lbaas10:51
ionihi guys10:51
ioniopenstack ansible uses http://tarballs.openstack.org/octavia/test-images/test-only-amphora-x64-haproxy-ubuntu-bionic.qcow2 as image10:51
ioniis this image alright for production ?10:51
jrosserioni: openstack-ansible uses that image for CI tests11:06
ionifor production, what should i use?11:06
jrosseryou shoukd probably build your own11:07
ioniis there any documentation related to that?11:08
jrosserfrom an OSA perspective there is this https://docs.openstack.org/openstack-ansible-os_octavia/latest/configure-octavia.html#building-octavia-images11:09
jrosserbut i think that quite likley needs updating11:10
jrosseri'm sure there are octavia docs too11:10
ionijrosser, thanks11:11
*** gcheresh_ has quit IRC11:23
cgoncalvesioni, https://docs.openstack.org/octavia/latest/admin/amphora-image-build.html11:26
ionicgoncalves, thanks11:28
*** pcaruana has quit IRC11:34
*** pcaruana has joined #openstack-lbaas11:41
openstackgerritCarlos Goncalves proposed openstack/octavia master: DNM CentOS7 gate test  https://review.opendev.org/66546411:46
openstackgerritJens Harbott (frickler) proposed openstack/neutron-lbaas stable/stein: Fix lb stats model  https://review.opendev.org/68839111:52
fricklercgoncalves: ^^ the fix is easy it seems, but adding a unit test I discovered that the verification was also broken for py2 :-/11:53
openstackgerritGregory Thiemonge proposed openstack/octavia-tempest-plugin master: WIP Use VM as client in traffic operation scenario tests  https://review.opendev.org/68756211:53
sapd1_Hi. Anyone help me. I don't know how to change the query in Listener Models after add type column to listener_cidr table.12:01
fricklercgoncalves: does the migration guide mentioned in https://wiki.openstack.org/wiki/Neutron/LBaaS/Deprecation#How_do_I_migrate_to_Octavia.3F actually exist? I cannot find it in the current octavia docs12:03
cgoncalvesfrickler, whoops! IIRC we do not have a doc there, no. perhaps that promised document is the README file in https://github.com/openstack/neutron-lbaas/tree/stable/stein/tools/nlbaas2octavia12:09
cgoncalvesfrickler, one other migration tool is https://github.com/nmagnezi/nlbaas2octavia-lb-replicator12:10
fricklercgoncalves: o.k., I'll look at both, thanks for the links12:12
cgoncalvesfrickler, the former tool is periodically tested in CI -- http://zuul.openstack.org/builds?pipeline=periodic&project=openstack%2Fneutron-lbaas12:15
fricklercgoncalves: yes, I saw that job when looking into updating for py36 testing earlier. it does look a bit unstable, but will be a good guide for internal testing when we tackle the migration12:27
openstackgerritCarlos Goncalves proposed openstack/octavia master: DNM CentOS7 gate test  https://review.opendev.org/66546412:44
openstackgerritJens Harbott (frickler) proposed openstack/neutron-lbaas stable/stein: Fix lb stats model  https://review.opendev.org/68839112:46
openstackgerritAnn Taraday proposed openstack/octavia master: [WIP] Jobboard based controller  https://review.opendev.org/64740612:57
*** Guest31368 has joined #openstack-lbaas12:58
*** Guest31368 is now known as redrobot_13:00
*** redrobot_ is now known as redrobot13:01
*** AustinR has quit IRC13:01
*** AustinR has joined #openstack-lbaas13:15
*** openstackstatus has quit IRC13:15
*** AustinR has quit IRC13:22
*** AustinR has joined #openstack-lbaas13:23
*** gcheresh_ has joined #openstack-lbaas13:33
openstackgerritCarlos Goncalves proposed openstack/octavia master: DNM CentOS7 gate test  https://review.opendev.org/66546413:34
openstackgerritMerged openstack/octavia master: Clean up requirements.txt from tox.ini  https://review.opendev.org/68797813:43
*** gcheresh_ has quit IRC14:00
*** gcheresh_ has joined #openstack-lbaas14:01
*** ricolin has joined #openstack-lbaas14:13
*** gcheresh_ has quit IRC14:24
*** maciejjozefczyk has quit IRC15:05
*** ivve has quit IRC15:05
*** openstackstatus has joined #openstack-lbaas16:03
*** ChanServ sets mode: +v openstackstatus16:03
*** rpittau is now known as rpittau|afk16:06
*** ramishra has quit IRC16:36
*** AlexStaf has joined #openstack-lbaas16:59
*** AustinR has quit IRC17:03
*** schaney has quit IRC17:07
*** AlexStaf has quit IRC17:12
*** AlexStaf has joined #openstack-lbaas17:14
guilhermesphey there! Is this fixed?  http://paste.openstack.org/show/783716/17:16
guilhermespthis is stable/queens17:16
johnsomrm_work FYI, https://storyboard.openstack.org/#!/story/200671917:17
guilhermespwhen users attempet to list lb from horizon, it spanws this in horizon logs repeatedly17:17
guilhermespI can list lbs from cli btw17:17
johnsomguilhermesp Check the version of openstacksdk you have installed on your horizon system. It seems like there is a bad version of openstacksdk installed.17:18
guilhermespthanks for the reply johnsom I will take a look!17:20
johnsomIt's a guess. I haven't seen that error before.17:21
guilhermespyeah, at this point, i think is the best clue tbh17:22
johnsomHmm, it also kind of looks like you have the master version of the dashboard installed instead of the queens version.17:23
johnsomguilhermesp Yeah, I think that might be your problem. You have master/train octavia-dashboard. So it probably doesn't align to the openstacksdk version installed.17:25
*** gmann is now known as gmann_afk17:25
*** tesseract has quit IRC17:25
johnsomLooks like it could be the stein version too, that line hasn't changed in a while.17:26
guilhermespyeah I'm cheking right now the exact version we are running17:27
*** ricolin has quit IRC17:33
guilhermespyeah sorry johnsom our hjorizon instances runs stein17:34
guilhermesplet me check the openstacksdk version running in it17:34
johnsomShould be 0.27.0 according to the releases page: https://releases.openstack.org/stein/index.html#openstacksdk17:35
jrosserguilhermesp: you are probably still using the queens constraints file even though moving the horizon service sha forward17:36
guilhermespopenstacksdk==0.26.0 yeah we are running the wrong one17:36
johnsomThat is pretty close though. Not sure I would expect an error with that combo. hmmm17:37
guilhermespgood point jrosser17:37
jrosseryou can probably make a config override just for the horizon role to fix that17:37
guilhermespyeah that would do it jrosser17:38
guilhermespthanks in advance guys!17:38
*** AustinR has joined #openstack-lbaas17:40
*** maciejjozefczyk has joined #openstack-lbaas17:40
guilhermespargh, seems that upgrading the sdk didn't fix17:45
*** maciejjozefczyk has quit IRC17:49
johnsomYeah, I was not convinced that 0.26.0->0.27.0 would have a fix. Let me look up the patch that changed that line to see if that gives any clues as to why you are seeing that.17:52
*** ivve has joined #openstack-lbaas17:53
guilhermespappreciate that jrosser17:53
guilhermespops17:53
guilhermespjohnsom ( and jrosser too ) :P17:53
rm_workahh johnsom i'll look at that today if i have time17:55
rm_workkinda lame >_<17:55
rm_workwould expect that to work17:56
johnsomYeah, not sure what is up with that. Just moved the bug over17:56
rm_workwoah that bug isn't new? old enough to migrate from launchpad? >_>17:56
rm_workjust noticed that17:56
johnsomSomeone opened it in launchpad. There are tricks that still allow it unfortunately. Like open it against another project, then re-assign.17:57
johnsomguilhermesp I am puzzled...  https://github.com/openstack/openstacksdk/blob/0.27.0/openstack/load_balancer/v2/_proxy.py#L6818:00
guilhermespnot sure but we enabling a flat to be able to use octavia v1 api18:01
johnsomWell, v2 must be enabled for horizon to work18:02
johnsomv1 was only used by neutron-lbaas18:02
*** gcheresh_ has joined #openstack-lbaas18:03
guilhermespi will try to toggle that off and see what happens18:04
*** psachin has quit IRC18:05
johnsomguilhermesp Do you have both octavia-dashboard and neutron-lbaas-dashboard installed in horizon?18:06
rm_workugh I did a really bad QA job on batch members, that was an obvious edge case and I just didn't test for it18:13
rm_workoh, wait, no18:15
rm_workI take that back, i totally did test for this and it should work fine18:15
rm_workthough I could have named the test better18:15
rm_workah i see how that error could happen18:22
rm_workif you update with [] on a pool that already has no members18:27
rm_workoh, nm, more common than that.18:28
guilhermespseems that we dont johnsom  http://paste.openstack.org/show/783718/18:42
guilhermespbut I found this in neutron.conf, which I think we dont need18:43
guilhermespservice_plugins=router,neutron_vpnaas.services.vpn.plugin.VPNDriverPlugin,neutron_lbaas.services.loadbalancer.plugin.LoadBalancerPluginv218:43
guilhermespspecifically neutron_lbaas.services.loadbalancer.plugin.LoadBalancerPluginv218:43
*** gcheresh_ has quit IRC18:45
rm_workright, that shouldn't be there if you're using Octavia directly19:05
openstackgerritAdam Harwell proposed openstack/octavia-lib master: Batch member update needs pool_id explicitly  https://review.opendev.org/68854619:33
johnsomguilhermesp I see you have 3.0.0 deployed for octavia-dashboard. You will really want to grab 3.0.1 as it has an important patch for a 403 error on creating load balancers.19:36
openstackgerritBrian Haley proposed openstack/octavia master: Stop testing python 2  https://review.opendev.org/68737019:43
guilhermespjohnsom: seems that we were missing that commit https://github.com/openstack/octavia/commit/aa4f3aba179020c0e9e42e8c01e97d30749787e119:45
openstackgerritAdam Harwell proposed openstack/octavia master: Fix batch member update error on empty change list  https://review.opendev.org/68854819:47
rm_workjohnsom: ^^19:48
rm_worknot sure if we can backport the fix :/19:48
rm_workcan we do backports at all on octavia-lib?19:48
johnsomI will look after lunch19:49
johnsomSure, lib has the same stable rules19:49
zigoHi guys. After thinking a lot about the Amphora image, I found that a lot of it can just go into the octavia-amphora package ...19:50
rm_workzigo: ah, i had on my list for this week to look at the debian image building in DIB -- looks like it shouldn't be too bad to make it build based on the official debian cloud image as a base19:58
zigorm_work: Looking at things, really, there's almost nothing that belong to image building, everything can be packaged.19:59
rm_workyeah, should be very little19:59
zigoThe only thing that belongs there is "update-rc.d haproxy disable"20:00
rm_workjust make sure the DIB reqs are installed and run the image build command20:00
zigoEverything else, I've pushed it to the octavia-agent package itself.20:00
zigoI uploaded this packaging change to Experimental, then this will be pushed to Sid this week, when Train gets released.20:01
*** gmann_afk is now known as gmann20:01
zigoThen I'll push a merge request to the Debian image builder...20:01
zigorm_work: https://salsa.debian.org/openstack-team/services/octavia/commit/fe6be9d39974c4c1d76c3094c4d83c94922d427120:03
openstackgerritAdam Harwell proposed openstack/octavia master: Fix batch member update error on empty change list  https://review.opendev.org/68854820:03
rm_workyeah it'd really be much better to not have you manage our elements separately and duplicate that work / have to keep up with changes20:03
johnsomrm_work Why are you passing pool_id here? https://review.opendev.org/#/c/688548/2/octavia/api/drivers/amphora_driver/v2/driver.py20:25
zigorm_work: Don't you think you'd better ask each and every distro provider to package octavia-agent properly?20:25
*** AustinR has quit IRC20:25
rm_workjohnsom: to solve the bug20:25
rm_workzigo: no i do not20:26
*** pcaruana has quit IRC20:26
johnsomrm_work can't you just no-op if there are no members to pull the pool_id off of?20:26
*** AustinR has joined #openstack-lbaas20:26
rm_workjohnsom: at that point it's not yet a noop if members is empty20:26
zigorm_work: ?!?20:26
rm_workjohnsom: deleted members don't go in that list, so it might need to delete some, we don't know yet20:26
rm_workthat'20:26
rm_work*that's determined by the next stage20:27
johnsomHmm, seems like we should sent it pre-calculated20:27
johnsomYeah, ok, let me think about this....20:27
rm_workzigo: i believe that no distro should diverge from the officially proscribed build method, and duplicate a ton of work just to result in a more fragile build process20:28
rm_workwe maintain all of this stuff in elements and using a flexible multi-os build system for a reason20:28
zigorm_work: What you should be doing is provide the things of the elements to the distro so that they package stuff.20:29
rm_worki guess we agree to disagree20:29
zigorm_work: Because of what you did, I didn't even realize that these things are belonging to a package.20:29
zigoNow I do understand my mistake.20:29
zigoTrying to solve it at the amphora image build is the wrong approach.20:29
rm_workif you weren't trying to replicate the whole build process outside of our build system, it wouldn't be a problem either20:30
zigoIt would ...20:30
rm_worktrying to replicate all of our elements in your own build system is the wrong approach20:30
zigoThings that belongs in a package belongs in a package, that's it ! :)20:30
rm_worki think we fundamentally disagree20:31
rm_workso i guess i'll leave you to it, good luck20:31
zigo:)20:31
zigorm_work: What you are doing in your elements really is packaging work, you just don't realize it.20:34
zigoThen it becomes a big hack...20:34
johnsomI think I disagree as well.20:46
*** rcernin has joined #openstack-lbaas20:57
*** goldyfruit has joined #openstack-lbaas21:20
*** vesper11- has joined #openstack-lbaas21:53
*** vesper11 has quit IRC21:53
*** rcernin has quit IRC21:59
openstackgerritBrian Haley proposed openstack/octavia master: Stop testing python 2  https://review.opendev.org/68737022:00
*** trident has quit IRC22:04
*** trident has joined #openstack-lbaas22:07
*** ivve has quit IRC22:49
openstackgerritMerged openstack/octavia master: Use bandit block in tox.ini  https://review.opendev.org/68796322:50
*** rcernin has joined #openstack-lbaas23:07
*** tkajinam has joined #openstack-lbaas23:17
*** tkajinam_ has joined #openstack-lbaas23:17
*** tkajinam has quit IRC23:21

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