Wednesday, 2020-03-04

« Tuesday, 2020-03-03 Index Thursday, 2020-03-05 »
*** vishalmanchanda has joined #openstack-lbaas00:11
*** yamamoto has joined #openstack-lbaas00:23
*** yamamoto has quit IRC00:28
johnsomI swear I should change my password to "assert_called_once_with", sigh.   I have used that sooooo many times in the last few weeks.00:34
*** dayou has quit IRC00:36
*** dayou has joined #openstack-lbaas00:37
*** spatel has joined #openstack-lbaas00:43
*** spatel has quit IRC00:48
openstackgerritMichael Johnson proposed openstack/octavia master: WIP - Refactor the failover flows  https://review.opendev.org/70531701:03
johnsomSadly more to go... But, down to just the network drivers. The end is in sight.01:04
*** spatel has joined #openstack-lbaas01:14
*** sapd1 has joined #openstack-lbaas01:21
*** yamamoto has joined #openstack-lbaas01:55
*** rcernin has quit IRC02:02
*** threestrands has joined #openstack-lbaas02:06
*** yamamoto has quit IRC02:09
*** yamamoto has joined #openstack-lbaas02:09
*** hongbin has joined #openstack-lbaas02:50
*** rcernin has joined #openstack-lbaas02:52
openstackgerritLuke Tollefson proposed openstack/octavia master: WIP Add parameters for configuration of TLS protocols  https://review.opendev.org/71115403:14
johnsomluketollefson: Woot!03:16
*** psachin has joined #openstack-lbaas03:33
openstackgerritSteven Glasford proposed openstack/octavia-lib master: WIP Add the four different tls and their ciphersuite  https://review.opendev.org/71115703:57
*** spatel has quit IRC04:05
*** spatel has joined #openstack-lbaas04:05
*** spatel has quit IRC04:39
*** hongbin has quit IRC04:54
*** haleyb is now known as haleyb|away05:05
*** yamamoto has quit IRC05:56
*** yamamoto has joined #openstack-lbaas06:11
*** gcheresh_ has joined #openstack-lbaas06:13
*** vishalmanchanda has quit IRC06:31
*** threestrands has quit IRC06:34
*** threestrands has joined #openstack-lbaas06:35
*** threestrands has quit IRC06:36
*** threestrands has joined #openstack-lbaas06:36
*** threestrands has quit IRC06:37
*** threestrands has joined #openstack-lbaas06:38
*** threestrands has quit IRC06:39
*** gcheresh_ has quit IRC06:51
*** vishalmanchanda has joined #openstack-lbaas06:55
*** gcheresh_ has joined #openstack-lbaas06:57
*** happyhemant has joined #openstack-lbaas07:17
*** yamamoto has quit IRC07:25
*** ccamposr has joined #openstack-lbaas07:55
*** yamamoto has joined #openstack-lbaas08:01
*** gcheresh_ has quit IRC08:04
*** yamamoto has quit IRC08:06
*** tesseract has joined #openstack-lbaas08:09
*** maciejjozefczyk has joined #openstack-lbaas08:14
*** rpittau|afk is now known as rpittau08:16
openstackgerritAnn Taraday proposed openstack/octavia master: [Amphorav2] Fix noop driver case  https://review.opendev.org/70969608:22
*** yamamoto has joined #openstack-lbaas08:32
*** gcheresh_ has joined #openstack-lbaas08:32
*** Erdosip has quit IRC08:42
*** ccamposr has quit IRC08:53
*** ccamposr has joined #openstack-lbaas08:53
*** ramishra has quit IRC08:57
*** ramishra has joined #openstack-lbaas09:03
*** tkajinam has quit IRC09:30
*** yamamoto has quit IRC09:44
*** yamamoto has joined #openstack-lbaas10:22
*** yamamoto has quit IRC10:27
*** rcernin has quit IRC11:08
*** sapd1 has quit IRC11:21
*** rpittau is now known as rpittau|bbl11:32
*** psachin has quit IRC11:33
*** psachin has joined #openstack-lbaas11:37
*** psachin has quit IRC11:49
*** openstackstatus has joined #openstack-lbaas11:50
*** ChanServ sets mode: +v openstackstatus11:50
*** yamamoto has joined #openstack-lbaas12:27
*** yamamoto has quit IRC12:28
*** vishalmanchanda has quit IRC12:31
*** xgerman has quit IRC12:54
*** xgerman has joined #openstack-lbaas12:55
*** yamamoto has joined #openstack-lbaas13:01
*** yamamoto has quit IRC13:06
*** jamesdenton has quit IRC13:07
*** nicolasbock has joined #openstack-lbaas13:11
*** NobodyCam has quit IRC13:24
*** NobodyCam has joined #openstack-lbaas13:25
*** vishalmanchanda has joined #openstack-lbaas13:36
*** rpittau|bbl is now known as rpittau13:44
*** yamamoto has joined #openstack-lbaas14:06
*** sapd1 has joined #openstack-lbaas14:14
*** psachin has joined #openstack-lbaas14:16
*** Wellie has joined #openstack-lbaas14:20
WellieHello14:20
Wellieis it possible to equip the LB with a WAF?14:20
*** spatel has joined #openstack-lbaas14:24
*** spatel has quit IRC14:29
*** nicolasbock has quit IRC14:51
*** ccamposr has quit IRC14:54
*** ccamposr has joined #openstack-lbaas14:54
johnsomWellie Hi, for the Amphora driver it is on the road map, but not available at this time. For provider drivers, it can be implemented via the Octavia flavors, but is less than ideal.14:57
johnsomWellie Right now we just offer ACLs.14:57
kevinzHi all! Is it possible for an admin Loadbalancer to connect admin subnet and an other-project subnet?14:59
kevinzto make two network connected, "other-project" instance can use the service / http exposed from admin subnet workloads15:00
*** psachin has quit IRC15:00
johnsomkevinz With Octavia you can select any subnet that is visible to the tenant for the VIP or the members servers.15:04
*** nicolasbock has joined #openstack-lbaas15:04
*** nicolasbock has quit IRC15:10
*** gcheresh_ has quit IRC15:31
*** psachin has joined #openstack-lbaas15:41
*** TrevorV has joined #openstack-lbaas15:46
*** yamamoto has quit IRC15:46
*** yamamoto has joined #openstack-lbaas15:47
*** yamamoto has quit IRC15:50
*** sapd1 has quit IRC15:52
openstackgerritMaciej Józefczyk proposed openstack/octavia-tempest-plugin master: Run reuse connection test only if protocol is HTTP or TCP  https://review.opendev.org/71125515:53
*** psachin has quit IRC15:56
*** ataraday_ has joined #openstack-lbaas16:00
johnsom#startmeeting Octavia16:01
openstackMeeting started Wed Mar  4 16:01:33 2020 UTC and is due to finish in 60 minutes.  The chair is johnsom. Information about MeetBot at http://wiki.debian.org/MeetBot.16:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:01
*** openstack changes topic to " (Meeting topic: Octavia)"16:01
openstackThe meeting name has been set to 'octavia'16:01
johnsom#chair rm_work16:01
openstackCurrent chairs: johnsom rm_work16:01
johnsomHi everyone. I thought rm_work was planning to be there today....16:02
ataraday_hi16:02
johnsom#topic Announcements16:02
*** openstack changes topic to "Announcements (Meeting topic: Octavia)"16:02
*** Trevor_V has joined #openstack-lbaas16:02
johnsomI have two reminders on our up coming deadlines16:02
johnsomOctavia-lib final Ussuri release is week of 3/3016:02
cgoncalveshi16:02
rm_worko/ sorry16:03
rm_workmeeting ran over16:03
johnsomRemember that octavia-lib must release before feature freeze. So we need to make sure we land our octavia-lib patches sooner rather than later.16:03
johnsomThat is just weeks away16:03
johnsomFor our other repos, the feature freeze MS3 is coming up as well16:04
johnsomOther projects feature freeze is week of 4/616:04
johnsomrm_work I will hand it over to you. I just wanted to remind everyone of the upcoming deadlines16:04
*** psachin has joined #openstack-lbaas16:05
*** TrevorV has quit IRC16:06
rm_workah kk16:06
rm_workok then... any other announcements?16:08
rm_workmoving on...16:08
rm_work#topic Brief progress reports / bugs needing review16:08
*** openstack changes topic to "Brief progress reports / bugs needing review (Meeting topic: Octavia)"16:08
ataraday_Remind of jobboard #link https://review.opendev.org/#/c/647406/16:09
rm_work#link https://review.opendev.org/#/q/status:open+project:openstack/octavia+branch:master+topic:az-tweaks16:09
rm_workPLEASE FOR THE LOVE OF ALL THAT IS HOLY16:09
johnsomI am making good progress on the failover patch. Most of the tests are now implemented. I'm just down to the network driver changes now.16:10
rm_workthese are not SO complex, and we need them to land for AZ support to not be completely broken16:10
ataraday_Small fix for noop related to amphorav2 #link https://review.opendev.org/#/c/709696/16:10
cgoncalvesI reviewed the first on the chain already... :P16:11
ataraday_And just pointing that test change with making amphorav2 default https://review.opendev.org/#/c/697213/ passing all tests :)16:11
johnsomSadly, this has become a large patch due to the test requirements and the gaps we had in testing for the previous implementation(s)16:11
rm_work\o/16:11
cgoncalvesrm_work, scenario test coverage?16:11
johnsomAlso of note, I'm not sure if any of the NDSU students are here for the meeting, but they have started to post some patches.16:12
johnsomPlease help them out with reviews as you can.16:12
rm_workcgoncalves: yeah, the scenario tests actually haven't merged yet for AZs -- they are not fully accurate... unfortunately I am not sure how to make them work properly -- they may need at least two-node testing to function, too...16:12
rm_workI am hoping that is more reliable now and I can get those fixed up with some help maybe from someone who understands how nova/neutron ops work :D16:12
rm_workok, does anyone else have any specific topics today?16:15
rm_work#topic Open Discussion16:15
*** openstack changes topic to "Open Discussion (Meeting topic: Octavia)"16:15
johnsomYou missed an agenda item16:16
rm_workah, johnsom you had something on the provider driver docs right?16:16
rm_workah was that in the agenda?16:16
johnsom#topic Updating the available drivers page16:16
*** openstack changes topic to "Updating the available drivers page (Meeting topic: Octavia)"16:16
johnsomYes, remember agendas?16:16
johnsomlol16:16
rm_workno :D16:16
rm_workliterally forgot that was a thing16:16
rm_work(I just load up the last meeting log, and copy/paste the topics in order)16:17
johnsomRecently we have had a number of comments/requests/confusion about the available provider drivers for Octavia16:17
johnsomI setup a page:16:17
johnsom#link https://docs.openstack.org/octavia/latest/admin/providers.html16:18
johnsomTo allow folks to "advertise" the availability of drivers, but sadly no one has updated it.16:18
johnsomSo, given the amount of confusion going on, I have proposed a patch updating it with the drivers that are public and I am aware of:16:18
johnsom#link https://review.opendev.org/#/c/711109/16:19
johnsomThe rendered page is here:16:19
johnsom#link https://9f709e7d1e4d7533935c-7291d6e818a7c847826cc66aee2194e8.ssl.cf5.rackcdn.com/711109/1/check/openstack-tox-docs/b3b7974/docs/admin/providers/index.html16:19
johnsomI wanted to raise this at the meeting to see if anyone had any concerns/questions/updates for this page.16:19
johnsomMany of these have been available for a few releases now...16:20
rm_workI think this is a good route forward, personally... Gives a framework and examples, and then should let provider vendors themselves propose updates as they wish very easily with minimal friction16:20
cgoncalvesI'm receptive to this idea. thank you, johnsom16:21
cgoncalvesthe rendered page looks good16:21
johnsomOk, thanks for the feedback. The proposal is up for review.16:22
johnsom#topic Open Discussion16:23
*** openstack changes topic to "Open Discussion (Meeting topic: Octavia)"16:23
johnsomUgh, chrome just exploded and I now have two of all of my windows.....16:24
rm_work(I'm a little salty about A10 being alphabetically first above the only actually in-tree driver, but it is what it is :D)16:24
johnsomYeah, well, I wanted to be fair, so alpha sorted16:24
rm_workyeah i get it16:24
johnsomOther topics today?16:27
*** yamamoto has joined #openstack-lbaas16:27
rm_worknothing on my end16:27
johnsomI'm really pushing hard to have the tests done this week. I will post here when I think we are ready to start reviews. There will probably be some additional clean up after I call it ready for review, but getting close to being done-ish.16:28
johnsomOh, also, I am out of office on Friday and will likely not have connectivity, so I will not be around until Monday.16:30
rm_workalright, if that's it, thanks for coming ya'll, catch you next week :)16:31
rm_workataraday_: will try to look at those patches you linked today16:32
rm_work#endmeeting16:32
*** openstack changes topic to "Discussions for OpenStack Octavia | Priority bug review list: https://etherpad.openstack.org/p/octavia-priority-reviews"16:32
openstackMeeting ended Wed Mar  4 16:32:35 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:32
openstackMinutes:        http://eavesdrop.openstack.org/meetings/octavia/2020/octavia.2020-03-04-16.01.html16:32
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/octavia/2020/octavia.2020-03-04-16.01.txt16:32
openstackLog:            http://eavesdrop.openstack.org/meetings/octavia/2020/octavia.2020-03-04-16.01.log.html16:32
openstackgerritHamza Zafar proposed openstack/python-octaviaclient master: add --all argument to stats show command  https://review.opendev.org/71127016:32
*** yamamoto has quit IRC16:33
ataraday_rm_work, thank you!16:34
openstackgerritHamza Zafar proposed openstack/python-octaviaclient master: add --all argument to stats show command  https://review.opendev.org/71127016:37
*** ataraday_ has quit IRC16:39
openstackgerritMerged openstack/octavia master: Update the available provider drivers list.  https://review.opendev.org/71110916:47
openstackgerritAdam Harwell proposed openstack/octavia stable/train: Fix batch member update error on empty change list  https://review.opendev.org/71127516:49
*** maciejjozefczyk_ has joined #openstack-lbaas16:51
*** maciejjozefczyk has quit IRC16:52
*** psachin has quit IRC16:53
*** maciejjozefczyk_ has quit IRC16:56
*** gcheresh_ has joined #openstack-lbaas16:58
*** gcheresh_ has quit IRC17:04
*** openstackgerrit has quit IRC17:20
*** Trevor_V has quit IRC17:21
*** TrevorV has joined #openstack-lbaas17:22
*** rpittau is now known as rpittau|afk17:44
dawzonIs it possible to use multiple lines for an option in octavia.conf?  Wondering if I can make OpenSSL ciphers strings a bit less ugly17:46
johnsomdawzon We could, with the multistr. That might limit the ability for folks to cut/paste the string from references like OWASP however.17:54
johnsomrm_work What do you think?17:54
*** ccamposr has quit IRC17:54
rm_workhmm, need to see an example of what that looks like17:54
*** ccamposr has joined #openstack-lbaas17:55
rm_workBut i will say that if the alternative is stuff like "certline1\nline2\nline3\n" we really don't have a choice, that's unacceptable in practice T_T17:55
JayFFor something like a certificate, why would it be specified inline in a conf vs the config option being a path to it17:56
JayFif it's not a separate file, it's going to be a heck of a lot tougher to rotate it17:57
rm_workah sorry yeah i am in the wrong mental mode17:57
rm_workthey're talking about just the cipher lists17:57
JayFAh, that makes more sense, but I still think external files can make sense for some things like that :)17:57
rm_workliterally right now staring at one of those \n certs in code >_<17:57
rm_workand wanting to stab something17:57
JayFturn the monitor off and step away (with the button, not with the dagger)17:58
JayFlol17:58
johnsomWe are talking about the default "openssl style" ciphers list setting.18:03
johnsomIt will likely look like this:18:03
johnsomTLS_AES_256_GCM_SHA384:TLS_CHACHA20_POLY1305_SHA256:TLS_AES_128_GCM_SHA256:DHE-RSA-AES256-GCM-SHA384:DHE-RSA-AES128-GCM-SHA256:ECDHE-RSA-AES256-GCM-SHA384:ECDHE-RSA-AES128-GCM-SHA256:DHE-RSA-AES256-SHA256:DHE-RSA-AES128-HA256:ECDHE-RSA-AES256-SHA384:ECDHE-RSA-AES128-SHA25618:04
johnsomI.e. the standard format you see everywhere18:04
JayFnice18:06
rm_workyeah it's just super long... but actually *doesn't* contain any newlines, right?18:08
rm_workso not sure how multiline helps us18:08
*** openstackgerrit has joined #openstack-lbaas18:09
openstackgerritAdam Harwell proposed openstack/python-octaviaclient master: Add --long to amphora-list to show more columns  https://review.opendev.org/71129818:09
rm_work^^ long time annoyance, trivially simple fix apparently, now i'm kinda feeling dumb because my scripted workaround actually took longer to do than fixing the client code <_<18:10
johnsomRight it would just be to make the config look pretty by breaking it into multiple config settings.18:10
johnsomPersonally I would rather have it one string so I can cut/paste from other sources for that string18:11
rm_workyeah, how would it be broken up even?18:15
johnsomoslo config has a multistropt that lets you have the same key on multiple lines. It then gives you a list back I think.18:16
rm_workoh so like, split on colon18:19
rm_workand literally one key a dozen+ times18:19
johnsomYeah, I guess. It just seems messy18:19
rm_workagreed18:19
johnsomdawzon So, two cores vote for leaving it a single string.18:19
johnsomI would propose it that way in the patch. Maybe others will comment in the patch, but I think you are likely good with a string.18:20
*** nicolasbock has joined #openstack-lbaas18:24
openstackgerritMichael Johnson proposed openstack/octavia stable/train: Add a periodic image build job  https://review.opendev.org/70639418:27
*** nicolasbock has quit IRC18:29
*** gcheresh_ has joined #openstack-lbaas18:29
*** nicolasbock has joined #openstack-lbaas18:30
johnsomrm_work https://storyboard.openstack.org/#!/story/200737118:30
johnsomI think that was some of your handy work18:31
johnsomOh, well, with some of mine over the top.... hmmm18:32
johnsomOh, they are trying to load individual secrets instead of a container.....18:34
johnsomI will respond18:36
johnsomOh, no, they are using a container18:36
johnsomhmm18:36
johnsomrm_work So, cert_secret = connection.secrets.get(secret_ref=cert_ref) will not error if they passed an href to a container?18:38
rm_workuhhh it should18:38
rm_work40418:38
johnsomYeah, that is what I thought18:39
rm_workunless we're eating the 404 somewhere18:39
johnsomNope: https://review.opendev.org/#/c/683254/3/octavia/certificates/manager/barbican.py18:39
johnsomDoes it just return None now and not raise?18:40
rm_workuhhh wtf18:40
johnsomAlso WTF: https://storyboard.openstack.org/#!/story/200737218:41
rm_worklol yes18:41
rm_worksome misunderstanding it seems18:41
johnsomI don't think we ever had those stats18:42
rm_workwe did not in octavia at least18:42
* rm_work shrugs18:42
johnsomMaybe v1? before my time18:42
johnsomI will look at the stats one, if you track down the bbq one18:42
rm_workpfft that's not fair :D18:43
rm_workthe stats one is just "-2, no, we don't have these stats"18:43
johnsomAren't you core in bbq?18:43
rm_worklol no18:43
rm_workalso not core in Barbican18:43
rm_workthough they keep threatening to do it :D18:43
rm_workanyway, i can look a bit at the barbican issue18:46
rm_workbut not exactly at this precise moment18:46
*** gcheresh_ has quit IRC18:50
johnsomOk18:52
johnsomSo that stats thing must have been downstream changes that were never pushed upstream. neutron-lbaas had less stats than Octavia does.18:53
*** tesseract has quit IRC18:55
*** ccamposr has quit IRC18:56
rm_worklol18:57
rm_worki am amazed you even bothered to look, because it doesn't matter whether n-lbaas had it or not -- we're not that project18:57
johnsomWell, we made a commitment to be backward compatible, so ... Honoring that18:57
rm_workuhh i mean18:58
rm_workthose stats *don't exist* AFAIU18:58
rm_worknot sure what we could do18:58
rm_workah i guess some of them do18:58
rm_workI guess we COULD add a lot of these18:59
johnsomNone the less, I couldn't find that in the neutron-lbaas code or the python-neutronclient. the neutron client just returns whatever it finds in the results, so it must be a custom neutron-lbaas they were using.18:59
johnsomYeah, it would be a RFE for more stats. Plus, likely want/need the metrics re-work18:59
rm_workugh why won't storyboard link me from story to gerrit, if the link exists18:59
johnsomWell, for the stats patch, the gerrit doesn't have the task # in it, so it didn't link19:00
rm_workah did they MANUALLY set it to "Review"? lol19:00
johnsomyep19:00
*** gcheresh_ has joined #openstack-lbaas19:01
rm_workso I mean, `connection.secrets.get(secret_ref=cert_ref)` SHOULD raise a NotFound19:02
rm_worknot just return None <_<19:02
rm_workthat'd be the real bug19:02
johnsomIt *seems* like it did.....19:02
rm_workand that's barbican client side19:02
rm_workyeah it SHOULD19:02
rm_workI swear it did19:02
johnsomMe too19:03
rm_workcommented on the --all19:05
rm_workmaybe it's for stats that return from some 3rd-party provider?19:05
rm_workdo any of them re-use our DB?19:06
rm_workor our stats call?19:06
openstackgerritHamza Zafar proposed openstack/python-octaviaclient master: add --all argument to stats show command  https://review.opendev.org/71127019:06
johnsomWell, stats always came direct from the neutron/octavia DB19:06
johnsomThat allowed aggregate stats queries19:06
rm_workso, the code itself doesn't seem bad? and without any additional columns ... it just ... works the same as what we have now I believe19:07
rm_workit might actually be fine to do it this way...19:07
rm_workthough i don't think it will *do* anything different without deeper changes in our API/DB19:08
rm_worki'm just -1 for being confused, not -2 for now19:08
johnsomYeah, I commented that this will require an RFE on the story19:09
johnsomI think someone asked for "more stats" at the last summit but we never caught what stats they wanted. Maybe this is the list?19:10
cgoncalvesI'm looking at test_amphora.py and wondering why do we create two LBs: one at resource setup, another for the only test method. what am I missing?19:20
cgoncalveshttps://github.com/openstack/octavia-tempest-plugin/blob/master/octavia_tempest_plugin/tests/scenario/v2/test_amphora.py19:20
johnsomlist tests?19:21
cgoncalvespardon?19:22
johnsomWell, you need three to test the API "list" command with sort options right?19:22
cgoncalvesyou can still do LB and amphora list and show on environments with 1 LB only19:22
cgoncalvessorry, I don't follow. I need three LBs?19:23
johnsomWell, I can't speak for the amp tests, but for load balancer list for example: https://github.com/openstack/octavia-tempest-plugin/blob/master/octavia_tempest_plugin/tests/api/v2/test_load_balancer.py#L24619:24
*** dmellado has quit IRC19:24
johnsomIf you don't have at least three, you can't test the ascending and descending sort orders.19:25
*** dmellado has joined #openstack-lbaas19:25
cgoncalvesnot something we test in test_amphora.py today anyway19:25
johnsomhttps://github.com/openstack/octavia-tempest-plugin/blob/master/octavia_tempest_plugin/tests/api/v2/test_load_balancer.py#L36319:26
cgoncalvestest_load_balancer.py != test_amphora.py19:26
johnsomYeah, like I said, I don't know on the amphora tests, I don't think I authored those19:26
johnsomThe other thought would be RBAC tests, but I don't think that applies there in scenario either.19:28
johnsomYeah, maybe the filtering test there at the bottom, but two should do for that I think19:30
johnsomAh, yeah, ok, so it is only creating two. I expect so that if you query by LB you only get the amps for that LB and not bleed over from the other LBs. That would be my guess19:32
johnsomIs it me, or are those two test at the bottom the same?19:33
cgoncalvesthey are not19:34
cgoncalvesthe first one queries for the LB created at resource setup, the second one queries for the LB created by the test method itself19:34
cgoncalvesso, same test but on different resources19:35
johnsomAh, self vs. not19:35
cgoncalvesright19:35
johnsommulti-tasking19:35
johnsomSo, I guess that is why.19:35
openstackgerritCarlos Goncalves proposed openstack/octavia-tempest-plugin master: WIP: fix test_amphora.py  https://review.opendev.org/71131619:37
cgoncalvespushed what I started working on this morning19:37
cgoncalvesit simplifies the test, creating only one load balancer19:37
johnsomSo two questions there.19:38
johnsomIt will now fail if it's on a spares cloud.19:38
johnsomYou dropped the filtering test.....19:38
cgoncalveshow so?19:38
cgoncalvesI did not19:38
cgoncalvesI moved it instead ;)19:38
cgoncalvesL10719:39
johnsomSo it's not testing a list of all amphora?19:39
johnsomjust a filtered list?19:40
cgoncalvesah, oops (spare pool)19:40
cgoncalvesI should keep the assert greater equal19:41
cgoncalvesa filtered list still tests the list API no?19:41
johnsomWell, only for filtered results19:46
johnsomdevils advocate here19:46
openstackgerritCarlos Goncalves proposed openstack/octavia-tempest-plugin master: WIP: fix test_amphora.py  https://review.opendev.org/71131619:51
cgoncalvesworks now with spare pools19:51
*** gcheresh_ has quit IRC19:52
openstackgerritCarlos Goncalves proposed openstack/octavia-tempest-plugin master: WIP: fix test_amphora.py  https://review.opendev.org/71131619:58
*** spatel has joined #openstack-lbaas20:26
*** spatel has quit IRC20:31
*** abaindur has joined #openstack-lbaas20:32
dawzonWhat is the difference between setting a pool's protocol to HTTPS and setting tls_enabled to True?  I notice that I can set tls_enabled while still having the protocol be HTTP20:45
*** gcheresh_ has joined #openstack-lbaas21:24
*** gcheresh_ has quit IRC21:30
*** KeithMnemonic has joined #openstack-lbaas21:37
*** yamamoto has joined #openstack-lbaas21:43
*** yamamoto has quit IRC21:49
johnsomYeah, this is an unfortunate thing we inherited. HTTPS is pass through, so basically just TCP21:58
johnsomNo encryption, etc.21:58
*** nicolasbock has quit IRC22:35
*** rcernin has joined #openstack-lbaas22:39
*** TrevorV has quit IRC22:55
*** tkajinam has joined #openstack-lbaas22:59
*** threestrands has joined #openstack-lbaas23:28
« Tuesday, 2020-03-03 Index Thursday, 2020-03-05 »

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