Monday, 2018-06-04

« Sunday, 2018-06-03 Index Tuesday, 2018-06-05 »
*** longkb has joined #openstack-lbaas00:35
*** longkb has quit IRC00:42
*** openstackgerrit has joined #openstack-lbaas01:14
openstackgerritAdam Harwell proposed openstack/octavia-tempest-plugin master: Create api+scenario tests for l7rules  https://review.openstack.org/57048501:14
openstackgerritAdam Harwell proposed openstack/octavia-tempest-plugin master: Create scenario tests for amphora  https://review.openstack.org/57125101:15
*** hongbin has joined #openstack-lbaas01:29
*** kiennt26 has joined #openstack-lbaas01:39
openstackgerritAdam Harwell proposed openstack/octavia master: Fix listener update race with default pool removal  https://review.openstack.org/57199701:53
*** kiennt26 has quit IRC01:54
*** atoth has quit IRC01:57
*** yamamoto has joined #openstack-lbaas02:05
*** sapd has joined #openstack-lbaas02:36
*** annp has joined #openstack-lbaas02:47
openstackgerritzhulingjie proposed openstack/octavia master: Fix some typos for Octavia  https://review.openstack.org/57141903:04
*** ivve has joined #openstack-lbaas04:17
*** kiennt26 has joined #openstack-lbaas04:23
*** hongbin has quit IRC04:44
*** kobis has joined #openstack-lbaas04:55
*** kobis has quit IRC04:59
openstackgerritAdam Harwell proposed openstack/octavia master: Fix listener update race with default pool removal  https://review.openstack.org/57199705:07
openstackgerritAdam Harwell proposed openstack/octavia-tempest-plugin master: Create api+scenario tests for l7policies  https://review.openstack.org/57048205:08
openstackgerritAdam Harwell proposed openstack/octavia-tempest-plugin master: Create api+scenario tests for l7rules  https://review.openstack.org/57048505:08
openstackgerritAdam Harwell proposed openstack/octavia-tempest-plugin master: Create scenario tests for amphora  https://review.openstack.org/57125105:08
openstackgerritAdam Harwell proposed openstack/octavia master: Fix listener update race with default pool removal  https://review.openstack.org/57199705:10
*** nmanos has joined #openstack-lbaas05:22
*** kobis has joined #openstack-lbaas05:25
*** ivve has quit IRC05:26
*** links has joined #openstack-lbaas05:38
*** nmanos has quit IRC06:31
*** ispp has joined #openstack-lbaas06:53
*** velizarx has joined #openstack-lbaas07:01
*** pcaruana has joined #openstack-lbaas07:12
*** pcaruana is now known as pcaruana|worksho07:14
*** sapd_ has quit IRC07:15
*** sapd_ has joined #openstack-lbaas07:15
*** tesseract has joined #openstack-lbaas07:16
*** yamamoto has quit IRC07:24
*** nmagnezi has joined #openstack-lbaas07:32
*** kiennt26 has quit IRC07:34
*** AlexeyAbashkin has joined #openstack-lbaas07:35
*** sapd has quit IRC07:38
*** sapd has joined #openstack-lbaas07:38
*** nmanos has joined #openstack-lbaas07:40
*** tesseract-RH has joined #openstack-lbaas07:45
*** tesseract has quit IRC07:48
*** tesseract-RH has quit IRC07:48
*** tesseract has joined #openstack-lbaas07:49
*** ispp has quit IRC07:56
*** ispp has joined #openstack-lbaas07:58
*** rcernin has quit IRC08:13
*** rpittau has joined #openstack-lbaas08:13
*** nmanos has quit IRC08:15
openstackgerritAdit Sarfaty proposed openstack/octavia master: Use object instead of object id in the drivers delete callbacks  https://review.openstack.org/57197408:15
*** nmagnezi has quit IRC08:15
*** nmagnezi has joined #openstack-lbaas08:16
*** nmanos has joined #openstack-lbaas08:16
*** yamamoto has joined #openstack-lbaas08:25
*** AlexeyAbashkin has quit IRC08:27
*** AlexeyAbashkin has joined #openstack-lbaas08:27
*** yamamoto has quit IRC08:33
*** ispp has quit IRC08:47
*** bzhao__ has quit IRC08:49
*** ispp has joined #openstack-lbaas08:49
*** sapd has quit IRC08:58
*** sapd has joined #openstack-lbaas08:59
*** salmankhan has joined #openstack-lbaas09:08
*** salmankhan has quit IRC09:14
*** yamamoto has joined #openstack-lbaas09:27
*** kobis has quit IRC09:36
*** kobis has joined #openstack-lbaas09:38
*** kobis has quit IRC09:42
*** sapd_ has quit IRC09:45
*** sapd_ has joined #openstack-lbaas09:45
*** kobis has joined #openstack-lbaas09:46
*** ispp has quit IRC10:11
*** yamamoto has quit IRC10:26
*** tesseract has quit IRC10:41
openstackgerritJan Zerebecki proposed openstack/neutron-lbaas master: Improve speed of loading LBs from DB  https://review.openstack.org/56836110:41
*** tesseract has joined #openstack-lbaas10:44
*** sapd has quit IRC10:45
*** ispp has joined #openstack-lbaas10:47
openstackgerritJan Zerebecki proposed openstack/neutron-lbaas master: Improve speed of loading Pools from DB  https://review.openstack.org/57098610:53
openstackgerritJan Zerebecki proposed openstack/neutron-lbaas master: Remove eager loading of Listener relations  https://review.openstack.org/57059611:00
openstackgerritJan Zerebecki proposed openstack/neutron-lbaas master: Remove eager loading of L7Policy relations  https://review.openstack.org/57102611:00
*** yamamoto has joined #openstack-lbaas11:26
*** yamamoto has quit IRC11:35
*** atoth has joined #openstack-lbaas11:44
*** yamamoto has joined #openstack-lbaas12:18
*** openstackgerrit has quit IRC12:34
*** amuller has joined #openstack-lbaas12:44
*** velizarx has quit IRC12:48
*** samccann has joined #openstack-lbaas13:00
*** samccann has joined #openstack-lbaas13:00
*** samccann has quit IRC13:00
*** samccann has joined #openstack-lbaas13:01
*** kobis has quit IRC13:02
*** yamamoto has quit IRC13:16
*** yamamoto has joined #openstack-lbaas13:16
*** beisner is now known as beisner-sick13:22
*** kobis has joined #openstack-lbaas13:26
*** ispp has quit IRC13:27
*** ispp has joined #openstack-lbaas13:28
*** yamamoto has quit IRC13:36
*** yamamoto has joined #openstack-lbaas13:36
*** openstackgerrit has joined #openstack-lbaas14:02
openstackgerritJan Zerebecki proposed openstack/neutron-lbaas master: Improve speed of loading LBs from DB  https://review.openstack.org/56836114:02
openstackgerritJan Zerebecki proposed openstack/neutron-lbaas master: Improve speed of loading Pools from DB  https://review.openstack.org/57098614:03
*** nmanos has quit IRC14:03
*** ivve has joined #openstack-lbaas14:03
*** ispp has quit IRC14:06
*** ispp has joined #openstack-lbaas14:08
*** links has quit IRC14:10
*** kobis has quit IRC14:17
*** kobis has joined #openstack-lbaas14:20
*** kobis has quit IRC14:26
*** fnaval has joined #openstack-lbaas14:31
*** yboaron has quit IRC14:36
*** ispp has quit IRC14:37
*** ispp has joined #openstack-lbaas14:45
*** kobis has joined #openstack-lbaas14:53
*** kobis has quit IRC14:58
*** ispp has quit IRC15:00
*** tesseract-RH has joined #openstack-lbaas15:01
*** tesseract has quit IRC15:02
*** ivve has quit IRC15:02
openstackgerritCarlos Goncalves proposed openstack/octavia master: Add grenade support  https://review.openstack.org/54965415:11
*** ispp has joined #openstack-lbaas15:17
*** ispp has quit IRC15:18
*** pcaruana|worksho has quit IRC15:35
*** AlexeyAbashkin has quit IRC15:45
*** yamamoto has quit IRC15:55
*** KeithMnemonic has joined #openstack-lbaas15:57
*** harlowja has joined #openstack-lbaas16:28
*** ianychoi has quit IRC16:32
*** tesseract-RH has quit IRC16:34
*** ianychoi has joined #openstack-lbaas16:48
*** pcaruana|worksho has joined #openstack-lbaas16:51
*** yamamoto has joined #openstack-lbaas16:55
rm_workjohnsom: ah you already reviewed my fix16:56
johnsomYeah, trying to catch up on all the activity over the weekend16:57
*** yamamoto has quit IRC17:03
*** kobis has joined #openstack-lbaas17:09
*** harlowja has quit IRC17:10
kobiscan anyone take a look at https://review.openstack.org/#/c/571974/17:13
kobisbefore we move on to work on the update methods?17:13
kobisthanks :)17:14
rm_worki thought we were going to just change the original patches to match the new spec, but i guess not :P17:14
rm_worki suppose they're getting big enough already17:14
kobisoh we were trying to do something useful other than overloading more tasks on you guys17:14
openstackgerritAdam Harwell proposed openstack/octavia master: Use object instead of object id in the drivers delete callbacks  https://review.openstack.org/57197417:15
rm_workneed to see that running on the newest test chain17:15
rm_work^^ thus rebase17:15
kobisok17:15
*** pcaruana|worksho is now known as pcaruana17:18
*** kobis has quit IRC17:26
*** kobis has joined #openstack-lbaas17:27
*** AlexeyAbashkin has joined #openstack-lbaas17:33
*** Alexey_Abashkin has joined #openstack-lbaas17:36
*** AlexeyAbashkin has quit IRC17:37
*** Alexey_Abashkin is now known as AlexeyAbashkin17:37
*** samccann has quit IRC17:46
johnsomkobis Do you guys mind if I work on that patch today?17:49
*** harlowja has joined #openstack-lbaas17:49
kobisnot at all - you want to build on top of Adit's?17:50
johnsomYeah, I guess since you did the deletes I can just create another one behind it that starts the updates.17:51
johnsomI was thinking we can do the "follow the sun" thing on this to get it done faster17:51
kobisok - unless you have something more important - if so we can complete the updates tomorrow17:51
johnsomHmm, good point. I could work on the migration tool today instead.17:52
kobisare the status update callbacks ready btw?17:53
johnsomI just know that the amphora driver is going to need the update changes too and will be a bit of work.17:53
kobisobviously but we can work on these too I guess17:53
kobisthen we can complete our driver17:53
johnsomOk17:55
rm_workhopefully we can merge the tests and through your spec update today17:59
rm_worktests looked good now last i checked18:00
johnsomYeah, running through the tempest chain now.18:03
johnsomrm_work What do you think about the HM driver?18:03
rm_workneed to finish reviewing, but if it's passing, it should be good18:04
johnsomDoes the L7rules patch just need a recheck? I see it had some failures18:04
rm_workhmmm let me see18:08
rm_workerr wut, i thought this was fixed18:09
rm_workyeah i guess a recheck18:09
rm_worklol you already got it18:09
*** AlexeyAbashkin has quit IRC18:09
rm_workk18:09
rm_worki need to disappear for a bit but i'll review that whole chain after lunch18:09
*** kobis has quit IRC18:10
*** kobis has joined #openstack-lbaas18:13
*** kobis has quit IRC18:32
openstackgerritMichael Johnson proposed openstack/octavia master: DNM: Re-enable KVM on OVH hosts  https://review.openstack.org/54905418:42
*** kobis has joined #openstack-lbaas18:53
*** kobis has quit IRC18:54
openstackgerritMichael Johnson proposed openstack/octavia master: DNM: Re-enable KVM on OVH hosts  https://review.openstack.org/54905419:14
*** harlowja has quit IRC19:29
*** ivve has joined #openstack-lbaas19:31
rm_workjohnsom: how do you get tempest up correctly with your devstack? I tried enabling the tempest service, and adding an `enable_plugin` line for octavia-tempest-plugin19:42
rm_workbut it does not seem to be set up right to run tempest19:43
rm_workoh, maybe just need to copy the config from ~stack/tempest/etc/tempest.conf to /etc/tempest/19:45
johnsomI just enable tempest in devstack19:45
johnsomENABLED_SERVICES+=,tempest19:45
johnsomlocalrc19:45
rm_workyeah19:45
rm_worki did that19:45
johnsomThen I have been manually installing the plugin19:45
johnsompip install .19:45
rm_workah19:45
rm_workyeah i mean, adding the enable_plugin line does that too19:46
rm_worksame thing19:46
johnsomThen from the tempest directory I run "tox -e all-plugin -- octavia_tempest_plugin"19:46
johnsomWhich I think is the "old way" but it works19:46
rm_workooh, you run from the tempest dir19:46
rm_workinteresting, ok19:46
rm_workthat's prolly it19:46
rm_workyeah seems like just copying the generated conf from inside the tempest dir to /etc/tempest works19:47
rm_workso i guess i just need to update my script to symlink that19:47
*** harlowja has joined #openstack-lbaas19:59
*** amuller has quit IRC19:59
*** kobis has joined #openstack-lbaas20:05
*** kobis has quit IRC20:08
xgerman_^^ can we put that in a readme20:14
johnsomhttps://github.com/openstack/octavia-tempest-plugin20:15
johnsomIt is20:15
xgerman_sweet20:15
*** sapd_ has quit IRC20:17
rm_workyeah i just expected to not have to be in the tempest dir20:18
*** atoth has quit IRC20:20
openstackgerritMerged openstack/octavia master: Fix listener update race with default pool removal  https://review.openstack.org/57199720:20
johnsomrm_work Quick fix on this one: https://review.openstack.org/#/c/57125120:21
rm_worklol k20:22
xgerman_mmh20:24
rm_worksuch a nitpick ;P20:24
johnsomGuess it's one of  those days20:25
*** sapd has joined #openstack-lbaas20:40
openstackgerritMerged openstack/octavia-tempest-plugin master: Create api+scenario tests for l7policies  https://review.openstack.org/57048220:43
openstackgerritAdam Harwell proposed openstack/octavia-tempest-plugin master: Create scenario tests for amphora  https://review.openstack.org/57125120:47
xgerman_rm_work: shodl we backport https://review.openstack.org/57199720:48
xgerman_?20:48
rm_workno20:49
rm_workthat was introduced with the provider patches20:49
rm_workwhich made just about everything "racey"20:49
rm_work(with the amp driver)20:49
xgerman_ah, got it —20:50
rm_worki'm retesting the L7Rules patch now... i am not seeing how it was failing on that other patch20:51
rm_workit's like it wasn't running the new code20:51
rm_workeven on rechecks :/20:51
rm_workah nm it was fine on the recheck20:51
rm_workso yeah, I am good for merging all the way up to cleanup20:52
rm_workjohnsom: do have a couple questions if you could clarify some stuff for me on the cleanup patch20:53
johnsomSure, go20:55
rm_worki posted them20:55
johnsomAh, ok, will look20:55
rm_workjust letting you know20:55
*** pcaruana has quit IRC20:59
rm_workxgerman_: I think this one is good: https://review.openstack.org/#/c/567059/ and the tests merged now :)21:01
rm_workand the next one we both +2'd already, but the tests need to merge also: https://review.openstack.org/#/c/570485/21:01
xgerman_ok, looking21:02
rm_worki'm particularly happy with the l7rules traffic scenarios :P21:02
rm_workfinally some real testing that that stuff *works*21:03
*** harlowja has quit IRC21:04
xgerman_mmh,  we set PENDING_DELEYE nut none of the other pending states21:07
xgerman_https://review.openstack.org/#/c/567059/13/octavia/api/v2/controllers/l7policy.py21:07
rm_workno, we do21:07
rm_workhttps://review.openstack.org/#/c/567059/13/octavia/api/v2/controllers/l7policy.py@25521:08
rm_workat least there21:08
xgerman_ah, ok — hard to see21:09
rm_workhmmm21:09
rm_workthough maybe not in the actual create?21:09
rm_workerr well no, that should already be set21:09
* rm_work looks for the UPDATE one21:09
rm_workoh nm that is it21:09
rm_workit just looked like it was in graph_create for a sec21:10
xgerman_I installed the UDP stuff … let’ see21:12
*** ivve has quit IRC21:26
*** harlowja has joined #openstack-lbaas21:36
*** rcernin has joined #openstack-lbaas22:03
*** fnaval has quit IRC22:23
johnsomFYI, Zun looking to work on docker amps: https://storyboard.openstack.org/#!/story/200211722:24
johnsomThere was a mailing list email about this as well22:24
xgerman_sweet22:30
*** yamamoto has joined #openstack-lbaas22:34
*** yamamoto has quit IRC22:38
rm_workoh we didn't +A https://review.openstack.org/#/c/567037/ yet22:46
rm_workI thought that already merged somehow22:46
rm_workjohnsom: https://review.openstack.org/#/c/567037/11/octavia/api/v2/controllers/health_monitor.py@278 so am I right that if the driver raises an exception, we'd roll back to ACTIVE?22:47
johnsomCorrect, with the new model errors roll back to the previous state and the error is raised for the requester22:48
rm_workk22:48
johnsomYeah, I removed a huge class of provisioning_status "ERROR" cases22:49
rm_workif we can get a +A on the HM one, we'll be able to merge most everything today22:50
rm_workwell, except the cleanup i guess22:50
rm_workoh and for some reason the l7rule tests have +A's around them but not ON them :P22:51
johnsomDon't look at me for that one22:52
« Sunday, 2018-06-03 Index Tuesday, 2018-06-05 »

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