Tuesday, 2019-02-05

« Monday, 2019-02-04 Index Wednesday, 2019-02-06 »
openstackgerritMerged openstack/octavia stable/queens: Healthmanager shouldn't update NO_MONITOR members  https://review.openstack.org/61228000:13
rm_workoh did that JUST merge? lol00:30
johnsomQueens00:40
*** Swami has quit IRC00:47
*** celebdor has quit IRC00:57
*** yamamoto has joined #openstack-lbaas01:01
rm_workyeah but still <_<01:14
*** yamamoto has quit IRC01:52
*** yamamoto has joined #openstack-lbaas01:52
*** Dinesh_Bhor has joined #openstack-lbaas02:32
*** Dinesh_Bhor has quit IRC02:58
*** hongbin has joined #openstack-lbaas03:00
*** psachin has joined #openstack-lbaas03:04
*** Dinesh_Bhor has joined #openstack-lbaas03:04
*** hongbin has quit IRC03:34
*** Dinesh_Bhor has quit IRC03:54
johnsomWell, we are stalled yet again with a zuul breakage.04:20
johnsomhttp://logs.openstack.org/44/634344/2/check/octavia-v1-dsvm-scenario/cac81e3/job-output.txt.gz#_2019-02-05_01_42_53_44194004:20
johnsomThis patch broke zuul: Ica2397a50de1054f5d5ec81b54ab36fda9bdc33304:20
johnsomLooks like pretty much every patch will fail with a POST_FAILURE04:21
*** Dinesh_Bhor has joined #openstack-lbaas04:42
*** yamamoto has quit IRC04:49
openstackgerritMerged openstack/octavia stable/queens: Fix Octavia for host host routes  https://review.openstack.org/63341204:55
*** dims has quit IRC04:58
*** yamamoto has joined #openstack-lbaas05:23
*** yamamoto has quit IRC05:25
*** yamamoto has joined #openstack-lbaas05:26
*** gcheresh has joined #openstack-lbaas06:28
*** yamamoto has quit IRC06:31
*** yamamoto has joined #openstack-lbaas06:32
*** dims has joined #openstack-lbaas06:48
*** dims has quit IRC06:52
*** dims has joined #openstack-lbaas06:54
openstackgerritMerged openstack/octavia-dashboard master: Update hacking version  https://review.openstack.org/62848006:58
*** dims has quit IRC06:59
*** dims has joined #openstack-lbaas07:02
*** ccamposr has joined #openstack-lbaas07:08
*** ccamposr has quit IRC07:19
*** Emine has joined #openstack-lbaas07:43
*** pcaruana has joined #openstack-lbaas07:45
*** rpittau has joined #openstack-lbaas08:04
*** celebdor has joined #openstack-lbaas08:19
openstackgerritMerged openstack/octavia stable/queens: Create disabled members in haproxy  https://review.openstack.org/61229608:27
*** AlexStaf has joined #openstack-lbaas08:32
*** yamamoto has quit IRC09:01
*** celebdor has quit IRC09:09
*** celebdor has joined #openstack-lbaas09:11
openstackgerritReedip proposed openstack/octavia-tempest-plugin master: TrivialFix Check Monitor in Member Scenario Tests  https://review.openstack.org/63489109:16
*** ccamposr has joined #openstack-lbaas09:19
*** ccamposr has quit IRC09:24
*** Dinesh_Bhor has quit IRC09:35
*** Dinesh_Bhor has joined #openstack-lbaas09:36
*** yamamoto has joined #openstack-lbaas09:43
*** yamamoto has quit IRC09:43
*** yamamoto has joined #openstack-lbaas09:44
*** ccamposr has joined #openstack-lbaas10:09
*** ccamposr has quit IRC10:19
openstackgerritReedip proposed openstack/octavia-tempest-plugin master: TrivialFix: Change cls.skip to cls.skipException  https://review.openstack.org/63004910:27
*** salmankhan has joined #openstack-lbaas10:33
*** salmankhan1 has joined #openstack-lbaas10:36
*** salmankhan has quit IRC10:37
*** salmankhan1 is now known as salmankhan10:37
*** ccamposr has joined #openstack-lbaas10:40
openstackgerritReedip proposed openstack/octavia-tempest-plugin master: TrivialFix: Change cls.skip to cls.skipException and raise it  https://review.openstack.org/63004910:40
*** salmankhan1 has joined #openstack-lbaas10:43
*** salmankhan has quit IRC10:43
*** salmankhan1 is now known as salmankhan10:43
openstackgerritReedip proposed openstack/octavia-tempest-plugin master: TrivialFix Check Monitor in Member Scenario Tests  https://review.openstack.org/63489110:47
*** salmankhan1 has joined #openstack-lbaas10:48
*** salmankhan has quit IRC10:50
*** salmankhan1 is now known as salmankhan10:50
openstackgerritReedip proposed openstack/octavia-tempest-plugin master: TrivialFixes: Use cls.skipException and raise it  https://review.openstack.org/63004910:58
openstackgerritReedip proposed openstack/octavia-tempest-plugin master: Use cls.skipException and raise it  https://review.openstack.org/63004911:01
openstackgerritReedip proposed openstack/octavia-tempest-plugin master: Check Monitor in Member Scenario Tests  https://review.openstack.org/63489111:02
openstackgerritReedip proposed openstack/octavia-tempest-plugin master: Use cls.skipException and raise it  https://review.openstack.org/63004911:03
*** salmankhan has quit IRC11:07
*** salmankhan has joined #openstack-lbaas11:07
*** Dinesh_Bhor has quit IRC11:11
*** salmankhan has quit IRC11:12
*** Dinesh_Bhor has joined #openstack-lbaas11:13
*** salmankhan has joined #openstack-lbaas11:19
*** numans is now known as numans_afk11:30
*** ramishra has quit IRC11:45
*** numans_afk is now known as numans11:51
*** ramishra has joined #openstack-lbaas12:00
*** yamamoto has quit IRC12:01
*** yamamoto has joined #openstack-lbaas12:02
*** yamamoto has quit IRC12:03
*** ccamposr has quit IRC12:09
*** yamamoto has joined #openstack-lbaas12:10
*** yamamoto has quit IRC12:13
*** celebdor has quit IRC12:36
*** yamamoto has joined #openstack-lbaas12:38
*** yamamoto has quit IRC12:39
*** yamamoto has joined #openstack-lbaas12:39
openstackgerritMerged openstack/octavia stable/queens: Fix IPv6 in Active/Standby topology  https://review.openstack.org/63276312:43
*** yamamoto has quit IRC12:43
*** yamamoto has joined #openstack-lbaas12:43
*** trown|outtypewww is now known as trown13:02
openstackgerritReedip proposed openstack/octavia-tempest-plugin master: Check Monitor in Member Scenario Tests  https://review.openstack.org/63489113:16
*** Dinesh_Bhor has quit IRC13:30
*** celebdor has joined #openstack-lbaas13:49
*** celebdor has quit IRC13:49
*** celebdor has joined #openstack-lbaas13:50
*** psachin has quit IRC14:09
*** pckizer has quit IRC14:38
*** pckizer has joined #openstack-lbaas14:46
*** ccamposr has joined #openstack-lbaas14:51
*** AlexStaf has quit IRC15:06
*** Emine has quit IRC15:47
*** gcheresh has quit IRC16:04
*** yamamoto has quit IRC16:13
*** Emine has joined #openstack-lbaas16:19
*** yamamoto has joined #openstack-lbaas16:25
*** yamamoto has quit IRC16:25
*** yamamoto has joined #openstack-lbaas16:26
*** gcheresh has joined #openstack-lbaas16:30
*** yamamoto has quit IRC16:31
*** openstackgerrit has quit IRC16:35
*** yamamoto has joined #openstack-lbaas16:53
*** yamamoto has quit IRC16:53
*** fnaval has joined #openstack-lbaas17:01
*** ramishra has quit IRC17:15
*** openstackgerrit has joined #openstack-lbaas17:18
openstackgerritCarlos Goncalves proposed openstack/octavia-tempest-plugin master: Add a scenario test for spare pool  https://review.openstack.org/63498817:18
openstackgerritCarlos Goncalves proposed openstack/octavia-tempest-plugin master: Add a scenario test for spare pool  https://review.openstack.org/63498817:20
*** ccamposr has quit IRC17:20
cgoncalvestwo more +W in stable/queens and we can request a queens tag! :-)17:23
cgoncalvesok, make it 317:25
cgoncalveshttps://review.openstack.org/#/q/status:open+project:openstack/octavia+branch:stable/queens+AND+NOT+label:Verified%253C%253D0+AND+NOT+label:Workflow%253C0+AND+NOT+label:Code-Review%253C017:25
johnsomI think we can tag without the functional test fix. They still run as part of the coverage job, so they are running....17:30
*** mloza has quit IRC17:30
*** rpittau has quit IRC17:31
cgoncalvestrue17:31
*** yamamoto has joined #openstack-lbaas17:31
openstackgerritMichael Johnson proposed openstack/octavia master: Fix the amphora noop driver  https://review.openstack.org/63434417:34
*** yamamoto has quit IRC17:36
johnsomLooks like we have a proposed zuul job fix for the POST-FAILURE issue.  Testing it ^^^17:43
johnsomcgoncalves Other than a bunch of internal meetings today, working on the octavia-lib patch. Should have that cleaned up today.17:44
cgoncalvesjohnsom, cool. I'll be testing it shortly after17:57
cgoncalvesoctavia-lib distgit (RDO repo) is now live and .spec file is under review17:58
johnsomCool. Did you figure out that you need to add o-da to you local.conf?17:58
*** trown is now known as trown|lunch17:59
cgoncalvesI don't remember if I added it or not. I do remember though that it errored on manual run18:00
cgoncalvesI was also expecting to find a new systemd service18:01
*** celebdor has quit IRC18:03
*** dims has quit IRC18:06
*** gcheresh has quit IRC18:09
*** celebdor has joined #openstack-lbaas18:17
*** salmankhan has quit IRC18:17
*** dims has joined #openstack-lbaas18:26
johnsomYeah, the systemd service was because you didn't add the o-da to localrc/local.conf18:36
johnsomAnyhow, it did have other issues which I'm working on now.18:36
*** celebdor has quit IRC18:53
*** Swami has joined #openstack-lbaas18:55
*** dims has quit IRC19:16
rm_workWait what's o-da? Driver agent?19:22
*** dims has joined #openstack-lbaas19:25
*** gcheresh has joined #openstack-lbaas19:27
*** pcaruana has quit IRC19:27
*** trown|lunch is now known as trown19:37
*** gcheresh has quit IRC19:52
johnsomYes20:05
johnsomNew, New, New! Coming to you in Stein20:06
johnsomIt's how provider drivers will callback stats/status to us and have long running processes20:06
*** yamamoto has joined #openstack-lbaas20:07
*** yamamoto has quit IRC20:12
*** celebdor has joined #openstack-lbaas20:20
openstackgerritMerged openstack/octavia stable/queens: Fix prefix for vip_ipv6  https://review.openstack.org/63318320:38
*** trown is now known as trown|outtypewww21:14
*** salmankhan has joined #openstack-lbaas21:14
*** salmankhan1 has joined #openstack-lbaas21:17
*** salmankhan has quit IRC21:19
*** salmankhan1 is now known as salmankhan21:19
*** yamamoto has joined #openstack-lbaas21:29
nmagnezicgoncalves, before you create a queens tag look at --> https://review.openstack.org/#/q/status:open+project:openstack/octavia+branch:stable/queens21:33
*** yamamoto has quit IRC21:34
cgoncalvesoh, silly me digging in for why the spare pool job failed. flavors!21:34
cgoncalvesnmagnezi, anything in particular? because that is why I mentioned earlier, not to tag until we get 2 more in21:34
openstackgerritCarlos Goncalves proposed openstack/octavia-tempest-plugin master: Add a scenario test for spare pool  https://review.openstack.org/63498821:36
cgoncalvesjohnsom, ^ so let's tempest test your flavor spare pool patch21:37
nmagnezicgoncalves, nothing specific that I see, just in case there is something that will catch your eye and you wish to include21:38
nmagnezi:|)21:38
cgoncalvesnmagnezi, ok. we agreed earlier on waiting for 2 patches to be merged before tagging21:38
openstackgerritMerged openstack/octavia master: Fix the amphora noop driver  https://review.openstack.org/63434421:44
rm_workYeah I was still dubious about how the spares pool is going to work... But I didn't realize it would be completely broken without that patch <_<21:46
rm_workWhich I guess is the case?21:46
johnsomYeah, why haven't we merged that spares pool fix patch?21:46
* johnsom looks around the channel....21:46
rm_workLike, errors due to argument count21:47
rm_workUgh I guess I can merge it and we can figure out later if it can be improved21:47
cgoncalvesjohnsom, because we all were waiting for my spare pool job to test your patch :P21:47
rm_work^^ :D21:48
johnsomCan't we make those, yet additional, gates non-voting?21:49
cgoncalvesuntil proven stable or non-voting forever? if the latter, why?21:50
johnsomUntil proven stable21:52
johnsomNot that the whole zuul environment has been stable recently, but that is another issue.21:53
openstackgerritCarlos Goncalves proposed openstack/octavia-tempest-plugin master: Add a scenario test for spare pool  https://review.openstack.org/63498821:55
openstackgerritMerged openstack/octavia stable/queens: Fix missing print format error  https://review.openstack.org/63477821:57
openstackgerritMichael Johnson proposed openstack/neutron-lbaas master: Fix the neutron-lbaas-to-octavia-migration job  https://review.openstack.org/63505622:04
rm_workeugh the ipv6 prefix patch merged in queens but is still stuck in Rocky <_<22:06
rm_workgrenade gate is broken or buggy maybe?22:06
cgoncalvesrocky grenade is broken. I haven't had the time yet to check it properly22:11
cgoncalvesI thought it was due to missing of the 2 monitor fixes in queens. seems I was wrong22:11
openstackgerritMerged openstack/octavia stable/queens: Fix functional tests on stable/queens  https://review.openstack.org/63478622:12
cgoncalvessomething is erroing inside the amphora22:12
*** celebdor has quit IRC22:13
rm_workjohnsom: https://review.openstack.org/#/c/632594/6 had comments22:15
rm_workerr, i had comments. they are new22:15
openstackgerritMichael Johnson proposed openstack/neutron-lbaas master: Fix the neutron-lbaas-to-octavia-migration job  https://review.openstack.org/63505622:15
johnsomOk, will look22:16
rm_workcgoncalves: are you planning to look at the rocky grenade soon then?22:16
rm_workthis is a weird state now, with a patch missing in the middle, lol22:17
johnsomlol, 0.1 vs 0.5. I just grabbed the version from the other docs lines....22:18
rm_workyeah that one is just lulz22:19
rm_workthe other one is what i -1'd for22:19
johnsomOk, this one: AmpDrvrNotImplementedError22:19
rm_workerr22:19
rm_workother other one22:19
johnsomSo, I ran into the dumb docs system saying "ambiguous NotImplementedError" on like everything, so I had to come up with something.22:19
rm_workyeah but ... the abbreviation?22:20
rm_workwhy22:20
johnsomBecauseOurStringsAreAlreadyAnnoyinglyTooDarnLong22:20
rm_workreally neede those 2 cgars>+?22:20
rm_work*chars22:20
johnsomOh, (22:20
rm_worklol22:20
johnsomBecauseOurStringsAreAlreadyAnnoyinglyTooDarnLong)22:20
rm_workyou saved two characters by making one random error message not have vowels in one of its words >_>22:21
johnsomWell, technically more as amphora isn't spelled out either22:21
johnsom6 characters22:22
rm_workanyway, still wasn't the thing i -1'd for22:22
rm_worklol22:22
johnsomI will put the two back if you would like22:22
rm_workyes plz :P22:22
rm_work<322:22
johnsomNP22:22
johnsomrm_work lol, yes, looking at that now, that is painfully obvious22:24
johnsomThe code dup that is....22:24
johnsomJust checking that people are still reading the patches...  Yeah, that's it, yeah22:25
rm_workLol22:25
rm_workThe rest of your patches have been pretty good :D22:26
johnsomrm_work I even updated the top of the document: "For the22:32
johnsompurposes of this document, the initial version of this API shall be v0.1." to be v0.522:32
johnsomAs it does appear we never checked in 0.122:32
openstackgerritMichael Johnson proposed openstack/octavia master: Fix flavors support when using spares pool  https://review.openstack.org/63259422:49
johnsomrm_work ^^^ addresses your comments22:49
rm_workCool thanks 👍22:55
*** celebdor has joined #openstack-lbaas23:01
rm_worklool so many 0.1 refs23:03
rm_worki thought you meant like... one ;p23:03
johnsomNope, that issue long pre-dated me23:03
johnsomI think everyone just assumed it was an "example" as the sentence at the top kind of stated23:03
rm_workkk, happy with this version23:04
rm_workyeah i just know how copy/paste-happy people get23:05
johnsomGeez, it's been four years...  At least since the amp api was checked in23:05
rm_work:P23:05
johnsomWhat a wild ride eh?23:05
rm_workcgoncalves: did you ever get the unit tests to run/debug in VSCode?23:05
rm_workjohnsom: indeed23:05
rm_workwho knew i'd still be working on the same thing 5 years later at a third company, lol23:06
johnsomYeah. We figured we would have it done years ago.... lol However, we had a lot more people working on it then too.23:07
*** yamamoto has joined #openstack-lbaas23:10
openstackgerritMerged openstack/octavia stable/queens: Fix health manager performance regression  https://review.openstack.org/61738523:33
*** fnaval has quit IRC23:39
openstackgerritErik Olof Gunnar Andersson proposed openstack/neutron-lbaas master: [WIP] Improve performance on create and get  https://review.openstack.org/63507623:40
eanderssonjohnsom, that is the patch I mentioned in PM ^23:48
eanderssonMajor culprint for lbaas was that whenever it gets the provider, it would fetch the full load balancer just to get the provider name....23:49
eanderssonVery expensive when you have 100+ members :p23:49
« Monday, 2019-02-04 Index Wednesday, 2019-02-06 »

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