Wednesday, 2019-10-23

« Tuesday, 2019-10-22 Index Thursday, 2019-10-24 »
*** rcernin has joined #openstack-lbaas00:05
*** yamamoto has quit IRC00:34
*** yamamoto has joined #openstack-lbaas00:55
openstackgerritAdam Harwell proposed openstack/octavia-tempest-plugin master: Allow amp-list test to run in a busy environment  https://review.opendev.org/60738202:12
*** goldyfruit has joined #openstack-lbaas02:25
rm_workjohnsom / cgoncalves: https://review.opendev.org/#/c/688548/ / https://review.opendev.org/#/c/688546/02:32
rm_work^_^02:32
rm_workneed to start poking at the priority review list again soon02:33
rm_workI just cleaned it up a bit02:33
rm_worknot sure I got everything though02:33
rm_workI ... am going to try to make it to the meeting tomorrow. :D02:38
openstackgerritMerged openstack/octavia-dashboard master: Fix dashboard certificates skipping pkcs12 bundles  https://review.opendev.org/68375102:45
*** ramishra has joined #openstack-lbaas03:11
*** ricolin has joined #openstack-lbaas03:16
*** yamamoto has quit IRC03:33
openstackgerritMerged openstack/octavia-tempest-plugin master: Fix positional formatting and add skip check  https://review.opendev.org/67316804:13
*** yamamoto has joined #openstack-lbaas04:22
*** ajay33 has joined #openstack-lbaas04:34
*** yamamoto has quit IRC05:05
*** yamamoto has joined #openstack-lbaas05:08
rm_workcgoncalves: https://review.opendev.org/#/c/682501/ :D05:29
rm_workpretty plz05:29
rm_workso annoying05:29
openstackgerritMerged openstack/octavia master: [Trivial]Add missing ws between words  https://review.opendev.org/68997405:33
rm_workcgoncalves: https://review.opendev.org/#/c/665103/ this too ^_^05:58
*** yamamoto has quit IRC06:01
*** ccamposr has joined #openstack-lbaas06:07
*** ccamposr__ has joined #openstack-lbaas06:09
*** ccamposr has quit IRC06:12
*** yamamoto has joined #openstack-lbaas06:14
*** yamamoto has quit IRC06:26
*** yamamoto has joined #openstack-lbaas06:37
*** rcernin has quit IRC06:37
*** yamamoto has quit IRC06:49
*** gcheresh_ has joined #openstack-lbaas06:51
*** maciejjozefczyk has joined #openstack-lbaas06:52
*** yamamoto has joined #openstack-lbaas06:55
*** trident has quit IRC06:55
*** trident has joined #openstack-lbaas07:00
*** AlexStaf has joined #openstack-lbaas07:08
openstackgerritOpenStack Proposal Bot proposed openstack/octavia-dashboard master: Imported Translations from Zanata  https://review.opendev.org/69048107:17
rm_workcgoncalves: i see you got one of those -- but this one pretty plzzzz https://review.opendev.org/#/c/682501/07:20
rm_workhmm i guess i'm not sure if anyone but me (with core) has an OSX machine to test on tho >_>07:21
*** yamamoto has quit IRC07:24
*** yamamoto has joined #openstack-lbaas07:36
*** ivve has joined #openstack-lbaas07:39
*** yamamoto has quit IRC07:55
*** yamamoto has joined #openstack-lbaas07:57
openstackgerritDat Le proposed openstack/octavia stable/rocky: Support to create amphora with customized metadata  https://review.opendev.org/69049808:06
*** rpittau|afk is now known as rpittau08:07
*** tkajinam has quit IRC08:10
*** yamamoto has quit IRC08:17
*** yamamoto has joined #openstack-lbaas08:20
*** yamamoto has quit IRC08:26
*** yamamoto has joined #openstack-lbaas08:26
*** yamamoto has quit IRC08:32
*** yamamoto has joined #openstack-lbaas08:33
*** yamamoto has quit IRC08:37
*** yamamoto has joined #openstack-lbaas08:38
openstackgerritMerged openstack/octavia master: Do not set log format when 'no log' is set  https://review.opendev.org/68793608:40
*** pcaruana has joined #openstack-lbaas09:02
*** ataraday_ has joined #openstack-lbaas09:04
*** yamamoto has quit IRC09:14
*** yamamoto has joined #openstack-lbaas09:14
*** maciejjozefczyk has quit IRC09:20
openstackgerritMerged openstack/octavia master: Allow IPv6 health network in devstack  https://review.opendev.org/66510309:20
*** maciejjozefczyk has joined #openstack-lbaas09:20
*** rcernin has joined #openstack-lbaas09:23
*** psachin has joined #openstack-lbaas09:46
*** psachin has quit IRC09:52
*** yamamoto has quit IRC10:17
*** pcaruana has quit IRC10:41
*** ramishra has quit IRC11:32
*** AlexStaf has quit IRC11:43
*** AlexStaf has joined #openstack-lbaas11:53
*** yamamoto has joined #openstack-lbaas12:05
*** yamamoto has quit IRC12:08
*** goldyfruit has quit IRC12:14
openstackgerritAnn Taraday proposed openstack/octavia master: Add option to set default ssl ciphers in haproxy  https://review.opendev.org/68533712:21
openstackgerritAnn Taraday proposed openstack/octavia master: Jobboard based controller  https://review.opendev.org/64740612:24
openstackgerritAnn Taraday proposed openstack/octavia master: Use retry for AmphoraComputeConnectivityWait  https://review.opendev.org/66279112:41
openstackgerritAnn Taraday proposed openstack/octavia master: Transition l7policy flows to dicts  https://review.opendev.org/66597712:43
openstackgerritAnn Taraday proposed openstack/octavia master: Transition l7rule flows to dicts  https://review.opendev.org/66817312:44
*** yamamoto has joined #openstack-lbaas12:48
*** takamatsu has joined #openstack-lbaas12:48
*** vishalmanchanda has joined #openstack-lbaas12:52
*** pcaruana has joined #openstack-lbaas12:53
*** yamamoto has quit IRC13:02
*** rcernin has quit IRC13:24
*** goldyfruit has joined #openstack-lbaas13:25
*** ramishra has joined #openstack-lbaas13:48
*** ajay33 has quit IRC13:49
*** ivve has quit IRC14:05
*** yamamoto has joined #openstack-lbaas14:33
*** ataraday_ has quit IRC14:35
*** yamamoto has quit IRC14:35
*** numans has quit IRC14:43
*** gcheresh_ has quit IRC14:56
*** pcaruana has quit IRC14:57
*** yamamoto has joined #openstack-lbaas15:07
*** yamamoto has quit IRC15:11
*** maciejjozefczyk has quit IRC15:16
*** goldyfruit has quit IRC15:46
*** yamamoto has joined #openstack-lbaas15:52
*** ataraday_ has joined #openstack-lbaas15:58
rm_work#startmeeting Octavia16:00
openstackMeeting started Wed Oct 23 16:00:01 2019 UTC and is due to finish in 60 minutes.  The chair is rm_work. Information about MeetBot at http://wiki.debian.org/MeetBot.16:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
*** openstack changes topic to " (Meeting topic: Octavia)"16:00
openstackThe meeting name has been set to 'octavia'16:00
johnsomo/16:00
rm_worko/16:00
colin-hi16:00
haleybhi16:01
*** yamamoto has quit IRC16:01
rm_work#topic Announcements16:02
*** openstack changes topic to "Announcements (Meeting topic: Octavia)"16:02
rm_workQueens is transitioning to Extended Maintenance!16:02
rm_work#link http://lists.openstack.org/pipermail/openstack-discuss/2019-October/010161.html16:02
rm_workAny other announcements today?16:03
cgoncalveshi16:03
ataraday_hi16:03
johnsomI have proposed a 1.11.0 python-octaviaclient release: https://review.opendev.org/69038616:04
johnsom#link https://review.opendev.org/69038616:04
johnsomThere is no real functional changes, it is just to get PDF docs going and fix a help string bug that was breaking python-openstackclient docs.16:04
johnsomSo no need for people to rush and update.16:04
rm_workcool16:05
johnsomThat is the only announcement I have16:05
rm_workk16:05
rm_workanyone else?16:05
rm_work#topic Brief progress reports / bugs needing review16:06
*** openstack changes topic to "Brief progress reports / bugs needing review (Meeting topic: Octavia)"16:06
rm_workI started to refresh our Priority etherpad16:06
rm_work#link https://etherpad.openstack.org/p/octavia-priority-reviews16:06
johnsomI have been focused on a series of issues related to certificate handling. One of those patches has merged, the other I posted yesterday:16:07
johnsom#link https://review.opendev.org/69044416:07
johnsomI also fixed an issue with the gates Friday related to version 2.4 of networkx (taskflow uses this). I have also submitted a patch to Taskflow to prepare it for the 2.x series of networkx.16:08
johnsom#link https://review.opendev.org/68961116:08
ataraday_Working on taskflow changes queue... Checking errors that job octavia-v2-dsvm-scenario-amphora-v2 got on them - if there are relevent etc..16:08
*** maciejjozefczyk has joined #openstack-lbaas16:09
ataraday_Looking forward reviews for #link https://review.opendev.org/#/c/689128/16:09
johnsomI have a few more TLS related patches to work on, then hopefully moving back to fixing the failover flow.16:09
rm_workataraday_: how close are those patches to us being able to merge the whole set?16:10
johnsomataraday_ Done, makes sense16:10
rm_worki was kinda poking at them the other day16:10
rm_workwould love to get moving on those, and stuff tends to move faster once we get the base work merged16:11
ataraday_all changes ready for review :)16:11
ataraday_I rebased jobboard controller on the top of all refactor, so when extra change will be merged - like retry and tempest and job will be green - we can merge16:13
rm_workcool16:13
rm_workmight try to get through those soon16:14
ataraday_Retry change #link https://review.opendev.org/#/c/662791/16:14
rm_workI just have one bugfix i'd like to see merged16:14
rm_work#link https://review.opendev.org/#/c/688548/16:14
rm_workand its dep in octavia-lib16:15
rm_workalright, moving on...16:16
rm_work#topic Open Discussion16:16
*** openstack changes topic to "Open Discussion (Meeting topic: Octavia)"16:16
rm_workhow is this meeting time for folks? any chance it could be a couple hours later? or is this pushing it?16:16
johnsomFYI, we have an agenda etherpad for the Shanghai PTG:16:17
johnsom#link https://etherpad.openstack.org/p/octavia-shanghai-U-ptg16:17
colin-i am flexible16:17
johnsomI am fine with a later time16:17
ataraday_for me this time is fine, but I can do later as well.16:18
cgoncalvesquestion would be what time and possibly different date. doodle?16:18
rm_workwell, yes, but mostly would like... 2 hours later for instance be workable for folks?16:19
cgoncalvesfine for me during non-DST (starting next week)16:20
rm_workk, i can make a doodle but not sure what other times i'd like to put on there16:22
rm_workmight just be this time and that time :D16:22
cgoncalvesI'd say add as many time slots as possible16:25
rm_workalright, anything else?16:25
johnsomFYI, I have -2'd this feature patch to stable/rocky:16:25
johnsom#link https://review.opendev.org/#/c/69049816:25
cgoncalvesfeature patch and straight to rocky? hmmmm16:25
johnsomRight, it also references a blueprint that doesn't exist16:26
rm_workprobably running a rocky cloud16:26
rm_workwouldn't mind some of that if it were proposed to master :)16:26
johnsomWell, I wonder if they know how Octavia works. Monitoring amphora directly is....  questionable value.16:27
johnsomBut, yes, I commented that it could be proposed to master16:27
rm_workright16:27
johnsomFeel free to add additional comments, etc.16:27
rm_workbut sometimes they get lost16:27
rm_workand it'd be nice info to have stamped on them :)16:27
rm_workanywho, yes, can't be on rocky16:27
maciejjozefczykjohnsom, rm_work hey! can you please take a look on this one? https://review.opendev.org/#/c/672264/ For now all the tests have ROUND_ROBIN algorithm hardcoded. That makes OVN tests not possible to successfully run (all the tests will be failing). That would be great to start running OVN gate first. Can we do a refactor of this issue later?16:28
maciejjozefczykI know thats not mona lisa, but that could unblock us, folks  :)16:29
cgoncalvesmaciejjozefczyk, octavia-tempest-plugin is branchless, hence there's no time-constraint (i.e. Usurri feature freeze)16:31
johnsomI think the other cores should comment. I still stand behind my comment that adding a config option for which algorithm is tested is counter productive for the test suite in general.16:31
cgoncalvesplus, your patch also adds a new option. this means that sooner or later we would need to deprecate which comes at a cost16:32
rm_worktechnically not ALL options get deprecated :D16:32
johnsomYeah, deprecation on a branchless repo is... painful16:33
cgoncalvesplus^2, agreeing with johnsom that we should test as many algorithms as possible. we don't want to run reconfigure and run tempest multiple times to test different algos16:33
maciejjozefczykcgoncalves, johnsom so that would need a refactor of whole octavia-tempest-plugin16:34
rm_workyeah, might just want to have tests for each and skip them based on what's available?16:34
johnsomrm_work That is what I proposed16:34
rm_workso we can do a provider list16:34
rm_workand detect that way which ones we can run16:34
johnsomrm_work We already have a config for "provider"16:34
rm_workah16:34
johnsom#link https://github.com/openstack/octavia-tempest-plugin/blob/master/octavia_tempest_plugin/config.py#L9616:35
maciejjozefczykBut its not only about skipping tests that are not accurate for given provider driver, for example this part: https://review.opendev.org/#/c/672264/9/octavia_tempest_plugin/tests/test_base.py is about checking if members are balanced16:36
johnsomWhat I proposed is for each test that is using RR, we create another test that uses the "SOURCE_IP_PORT", then do a skip check for each based on the configured provider.16:36
johnsomSince OVN doesn't support RR and amphora doesn't support "SOURCE_IP_PORT"16:37
rm_workyep, seems sane to me16:37
rm_workthough that still makes it impossible to test multiple in one run16:38
johnsomYes, a "check member balanced" will need to be created for the "SOURCE_IP_PORT" algorithm. Or the test to at least be appropriate to test that algorithm.16:38
rm_workwe COULD do something smart and inspect providers and flavors and see if we should try testing with multiple flavors or something, or allow that to be configured16:38
rm_workbut at least just adding both tests would be a start16:39
maciejjozefczykSo what about api tests? Like this? https://review.opendev.org/#/c/672264/9/octavia_tempest_plugin/tests/api/v2/test_member.py16:39
*** ccamposr__ has quit IRC16:39
johnsomrm_work Why would it? It would run all the tests that apply to that provider. The config option proposed is the only one that would limit it to only testing one per run.16:39
rm_workno, you linked to the config option already merged that sets what provider to use16:40
rm_workwhich is a single provider16:40
johnsomrm_work Yes, test runs are validate against one "provider" at a time. The proposed patch would make test runs only valid for one provider and one LB algorithm.16:41
rm_workyeah, but what you proposed still just runs one algorithm test16:41
rm_workit does pick it correctly16:41
johnsomrm_work Run per-provider seems valid given the other infrastructure requirements for them.16:41
rm_workbut if we want to run BOTH, we'd need to detect flavors, etc16:41
johnsomrm_work I think we have confused you.16:42
rm_work[09:33:25] cgoncalves:plus^2, agreeing with johnsom that we should test as many algorithms as possible. we don't want to run reconfigure and run tempest multiple times to test different algos16:42
johnsomrm_work We should leave the provider config. That is valid. I'm just saying the algorithm tests should run all algorithms that are valid for a given provider.16:42
rm_work[09:38:17] rm_work:though that still makes it impossible to test multiple in one run16:42
rm_work[09:36:59] johnsom:What I proposed is for each test that is using RR, we create another test that uses the "SOURCE_IP_PORT", then do a skip check for each based on the configured provider.16:43
rm_work^^ that's two16:43
rm_workone for each16:43
rm_workand requires two runs and a reconfigure inbetween16:43
rm_workit's technically possible to test both in one run16:43
rm_workif there exist flavors to allow it16:43
johnsomrm_work only because OVN doesn't support anything other than "SOURCE_IP_PORT". For amphora, one run should test multiple algorithms that it supports. Unlike the proposed patch.16:44
johnsomrm_work No, this has nothing to do with flavors.16:44
maciejjozefczykjohnsom, I don't understand. How?16:44
maciejjozefczykjohnsom, Looks like everything is hardcoded, no?16:44
rm_workyeah i'm talking less about the proposed patch, and more about what we WANT to see16:45
rm_workjohnsom: flavors are the only way to actually run with multiple providers at the same time right now16:45
rm_workso yes, it has to do with flavors?16:45
johnsommaciejjozefczyk That is what I am proposing instead of your patch.16:45
johnsomrm_work We don't want to run multiple providers in one test run. That is not what I am proposing and I think it is a bad idea.16:46
cgoncalvesrm_work, disregard test of multiple providers in same run16:46
johnsomrm_work We want one test run per provider, each running all of the algorithm tests it can.16:46
rm_workk, then that requires us to duplicate our test a bunch of times16:46
rm_workbecause right now it just tests RR16:47
rm_worknot just add SOURCE_IP16:47
cgoncalveswe want to have the ability to test multiple algorithms for the same provider in one run. e.g. test RR and LEAST_CONNECTION for a given provider16:47
johnsomrm_work Right, today we only have tests for one algorithm. I am saying we should fix that.16:47
cgoncalves+116:47
johnsomcgoncalves +116:47
maciejjozefczyk+1 cgoncalves16:47
cgoncalvesotherwise it also comes at a cost of exploding our CI job matrix16:47
rm_workmaciejjozefczyk: +116:47
cgoncalvesseems we're all in agreement, in a round-robin fashion :D16:48
maciejjozefczykjohnsom, rm_work so how we could do it? inspect provider algorithm and then generate classes dynamically based on current code? that would have only changed unsed algorithm?16:48
rm_worknah, just write the test a bunch16:48
rm_workas long as it's fairly decomposed, it won't be a ton of code reuse16:48
rm_workand then do skiptests depending on the config value for provider16:49
rm_workwe COULD try to be fancy and use one test and detect the provider and switch out the algorithm+test_func based on a dictionary of options16:49
rm_workbut16:49
rm_work¯\_(ツ)_/¯16:50
johnsomWell, that is an implementation detail. There would be a run for each algorithm and a validation for each algorithm. So, maybe separate tests is easier/better.16:50
haleybmaciejjozefczyk: can you just create sub-classes with a single line selecting the algorithm, then @skip some?  i think that's what ^^^ is ?16:50
rm_workyes16:50
maciejjozefczykhum, so we have ~17 testfiles here https://review.opendev.org/#/c/672264/; for every class we're just inherit with given subclass? It will produce a lot of new classes16:50
*** rpittau is now known as rpittau|afk16:50
rm_workjust another method16:51
rm_worknot another full class, I think16:51
rm_worki mean, I guess you could do it that way too O_o16:51
* johnsom wonders if supporting the tempest suite for third party drivers is a good idea....16:51
rm_workwell, for a ton of tests (everything but traffic, pretty much) the only change is literally which default algorithm to use16:52
rm_workso you *could* at a very high level (base test class?) just detect provider and select a "default algorithm"16:52
rm_workand then for the tests where it *matters*, create copies and skip the ones that don't run16:53
johnsomYou would still want per-algorithm tests though16:53
rm_worknot for a ton of them16:53
johnsomright, we said the same thing at the same time-ish16:53
rm_workmost tests don't actually CARE about the algorithm, they just provide it so it will be valid create syntax16:54
maciejjozefczykrm_work, like an API tests16:54
rm_workyes, like all API tests16:54
maciejjozefczykmaybe not all,16:54
cgoncalvesyeah but now we'd be actually testing the algorithm16:54
johnsomIt's really too bad there isn't support for RR in OVN16:54
maciejjozefczykjohnsom, yeah...16:54
rm_workwell, we already have exactly one test for algorithm testing16:54
rm_workthe traffic-ops test16:54
rm_workthat's the one that'd actually benefit from having copies for each alg16:55
cgoncalvesright16:55
rm_workbut the majority of tests just need to fill that field with SOMETHING valid16:55
rm_workso they can do their real testing which is "does a member create work or ERROR"16:55
johnsomAgreed, something valid16:55
cgoncalvesalgorithm validation with traffic is important also for certification purposes16:56
rm_workso yes, for 99% of tests, selecting a default algorithm in a base class based on the provider would work16:56
johnsomyep16:56
rm_workthen for the traffic tests, you'd want to actually put in the work to make them validate each algorithm16:56
rm_workand skip as appropriate16:56
johnsomWell, maybe a bit more than 99%, but yes.16:56
maciejjozefczykrm_work +116:56
maciejjozefczykrm_work I own you a beer in Shanghai16:57
johnsomrm_work Yep. That is what I proposed in a comment on that patch.16:57
rm_worklol yeah i guess i'll take it since johnsom isn't there :D16:57
maciejjozefczykjohnsom, you too :)16:57
johnsomlol16:57
cgoncalveshe can take the week after Shanghai :P16:57
maciejjozefczykahh. :P16:57
maciejjozefczykah, right, Barcelona16:57
maciejjozefczyk;)16:57
rm_workk well, we went through the meeting time pretty well there16:58
rm_workany parting comments?16:58
rm_workwe've got one more meeting before the summit16:58
rm_workthen will probably cancel the meeting during the summit, per usual16:59
johnsom+116:59
rm_workalright16:59
rm_workthanks for coming folks!16:59
rm_work#endmeeting16:59
*** openstack changes topic to "Discussions for OpenStack Octavia | Priority bug review list: https://etherpad.openstack.org/p/octavia-priority-reviews"16:59
openstackMeeting ended Wed Oct 23 16:59:54 2019 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:59
openstackMinutes:        http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-10-23-16.00.html16:59
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-10-23-16.00.txt16:59
openstackLog:            http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-10-23-16.00.log.html17:00
maciejjozefczykThanks!17:00
*** ataraday_ has quit IRC17:07
cgoncalvesjohnsom, rm_work: would you disagree that https://review.opendev.org/#/c/688546/ is not a backport candidate as it touches API?17:10
rm_workyeah, that might be true...17:12
johnsomcgoncalves Yeah, this one is nasty as it does change the API on providers. I considered asking for a compatibility method for the old signature.17:12
*** ricolin has quit IRC17:55
*** vishalmanchanda has quit IRC17:59
*** ricolin has joined #openstack-lbaas18:03
*** ricolin has quit IRC18:25
*** pcaruana has joined #openstack-lbaas18:38
openstackgerritMichael Johnson proposed openstack/octavia master: Fix listeners with SNI certificates  https://review.opendev.org/69044418:39
*** pcaruana has quit IRC18:44
*** maciejjozefczyk has quit IRC18:50
rm_workjohnsom: i think i'm gonna poke at building `--wait` for the loadbalancer client today18:56
rm_workor --poll or ... whatever is most like what other services use18:56
rm_workI think --wait is right18:56
rm_workyeah, that's what nova uses18:57
rm_workany objections?18:57
rm_workwould simplify a lot of stuff because i can't count the number of times i've rewritten the polling logic in various places18:57
rm_workjust poll at a sane interval and return the final LB state -- and code 0 for success, 1 for failure18:58
rm_workanywho, brb loonch18:58
rm_workProbably also `--wait-timeout <seconds>`19:00
johnsomSounds good to me19:01
johnsomHow often do you plan to poll?19:02
rm_workHmm... I'll look at what nova does, but I was thinking 2s or so19:06
rm_workSpeed will be better too on internal polling because it won't have to go through all the keystone auth stuff every single time19:06
johnsomYep19:12
*** gcheresh_ has joined #openstack-lbaas19:18
*** abaindur has joined #openstack-lbaas19:21
johnsomIf someone is bored and has cycles, we could go through and remove the six stuff for py2719:28
*** gcheresh_ has quit IRC19:59
*** gcheresh_ has joined #openstack-lbaas20:18
*** gcheresh_ has quit IRC20:31
rm_workooooooooooooh21:08
rm_workoooooh21:08
rm_workme21:08
rm_workme me me21:08
rm_workI love deleting cruft21:08
rm_workit's one of my favorite activities :D21:08
rm_workbut ... i guess since it's fairly simple, someone else could probably do it as low-hanging fruit...21:09
johnsomI'm itching for it to go, but I'm a bit busy right now with TLS SNI stuffs21:09
rm_work... except maybe some of the unicode parts lol21:09
johnsomJust about to have an SNI tempest scenario. Handy to test my new patch... grin21:09
rm_workhow about I do it on top of your TLS work? unless you were hoping it was gone21:09
rm_workso you don't have to deal with py2 unicode :21:10
rm_work:D21:10
johnsomSure, this should be pretty close to good: https://review.opendev.org/690444 (it at least passes the existing TLS tests)21:10
johnsomIt's going to be one of those "touch everything" patches we should merge quickly IMO21:11
*** henriqueof has quit IRC21:15
openstackgerritMerged openstack/octavia master: Fix some plug.py unit tests that broke on OSX  https://review.opendev.org/68250121:27
johnsomrm_work ^^^^ There you go fruit fanboy21:29
rm_work:P21:29
rm_workyes, i am pleased21:29
rm_workfor the record, I despise Apple as a company, but their macbook hardware is just... better.21:30
rm_workor it was, until this generation :/21:30
*** goldyfruit has joined #openstack-lbaas21:38
*** yamamoto has joined #openstack-lbaas21:59
*** ccamposr has joined #openstack-lbaas22:02
*** yamamoto has quit IRC22:03
*** goldyfruit has quit IRC22:04
*** ramishra has quit IRC22:07
*** ccamposr has quit IRC22:17
*** rcernin has joined #openstack-lbaas22:33
*** tkajinam has joined #openstack-lbaas23:02
*** abaindur has quit IRC23:03
*** abaindur has joined #openstack-lbaas23:04
*** yamamoto has joined #openstack-lbaas23:26
« Tuesday, 2019-10-22 Index Thursday, 2019-10-24 »

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