Friday, 2015-08-14

*** SumitNaiksatam has quit IRC00:01
rm_workk got it i think00:03
rm_worknow the annoying random places that i missed :P00:03
rm_workthe REST driver was a little closer to accurate00:05
rm_worksubmitting a draft00:05
rm_workonce all tox finishes00:05
*** jorgem has quit IRC00:09
openstackgerritBanashankar k proposed openstack/neutron-lbaas: Octavia driver unit tests Added unit tests for the octavia driver.  https://review.openstack.org/21064400:11
rm_workalso there's some issues in py34 allowed_address_pairs driver that are being masked by exception handling00:14
rm_workpossibly just with the tests00:14
openstackgerritAdam Harwell proposed openstack/octavia: Correct usage and configuration of CertManager/Generator with Stevedore  https://review.openstack.org/21285800:15
rm_workxgerman: ^^00:15
*** minwang2 has quit IRC00:17
*** madhu_ak has quit IRC00:20
johnsomrm_work Is the test supposed to be for the ssh driver while changes are to rest driver?00:25
rm_workchanges are to both00:25
rm_workbut the rest driver was way closer to accurate already00:25
johnsomOh, ok00:25
rm_workso tests didn't need to change for it00:25
rm_workthe old classloader method was replaced with Stevedore00:26
rm_workbut never got fully removed00:26
*** haigang has joined #openstack-lbaas00:26
rm_workand ssh driver was just never using it at all00:26
*** mlavalle has quit IRC00:29
openstackgerritAdam Harwell proposed openstack/octavia: test_plug_vip technically not testing properly due to mock issue in py34  https://review.openstack.org/21286800:46
rm_workand this is an interesting one ^^00:46
rm_workrandomly caught it while running other tox tests00:46
rm_workname slightly misleading00:46
rm_workwasn't sure how to word it00:46
rm_workdamnit and the commit title is too long anyway00:48
openstackgerritAdam Harwell proposed openstack/octavia: test_plug_vip not testing properly in py34  https://review.openstack.org/21286800:50
rm_workfixed00:50
rm_workbbl00:50
*** haigang has quit IRC00:50
*** diogogmt has joined #openstack-lbaas00:52
*** iwi1 has quit IRC00:52
*** bharath has quit IRC00:53
*** vivek-ebay has quit IRC01:05
rm_youthough, technically i'm ALWAYS here01:27
rm_youso... there's that01:27
*** bana_k has quit IRC01:34
*** haigang has joined #openstack-lbaas01:39
*** fnaval has quit IRC01:41
*** minwang2 has joined #openstack-lbaas01:44
*** fnaval has joined #openstack-lbaas01:52
*** crc32 has quit IRC01:53
*** haigang has quit IRC01:56
*** haigang has joined #openstack-lbaas01:59
*** mikeymeitbual has quit IRC02:08
*** KunalGandhi has quit IRC02:46
*** enikanorov2 has quit IRC02:55
*** KunalGandhi has joined #openstack-lbaas03:10
*** crc32 has joined #openstack-lbaas03:28
crc32you there blogan?03:34
*** mixos has joined #openstack-lbaas03:35
*** mixos has left #openstack-lbaas03:36
*** crc32 has quit IRC03:36
*** crc32 has joined #openstack-lbaas03:40
*** KunalGandhi has quit IRC03:43
*** KunalGandhi has joined #openstack-lbaas03:43
*** crc32 has quit IRC03:44
*** KunalGan_ has joined #openstack-lbaas03:52
*** bharath has joined #openstack-lbaas03:54
*** KunalGandhi has quit IRC03:56
*** bharath has quit IRC03:58
*** diogogmt has quit IRC04:00
*** KunalGan_ has quit IRC04:09
*** enikanorov2 has joined #openstack-lbaas04:10
*** vivek-ebay has joined #openstack-lbaas04:19
*** vivek-ebay has quit IRC04:21
*** vivek-ebay has joined #openstack-lbaas04:27
*** KunalGandhi has joined #openstack-lbaas04:29
*** KunalGandhi has quit IRC05:00
*** numan has joined #openstack-lbaas05:01
*** vivek-ebay has quit IRC05:14
*** SumitNaiksatam has joined #openstack-lbaas05:21
*** vivek-ebay has joined #openstack-lbaas05:23
*** chlong has quit IRC05:41
*** minwang2 has quit IRC05:43
*** vivek-ebay has quit IRC05:43
*** vivek-ebay has joined #openstack-lbaas05:44
*** chlong has joined #openstack-lbaas05:44
*** haigang has quit IRC05:45
*** haigang has joined #openstack-lbaas05:46
*** haigang has quit IRC05:47
*** haigang has joined #openstack-lbaas05:47
*** vivek-ebay has quit IRC05:48
*** minwang2 has joined #openstack-lbaas05:48
*** minwang2 has quit IRC06:05
*** ganeshna has joined #openstack-lbaas06:44
rm_workblogan: don't suppose...06:53
rm_workactually, anyone else around?06:53
*** ganeshna_ has joined #openstack-lbaas06:58
*** ganeshna has quit IRC07:00
*** fnaval has quit IRC07:04
*** numan has quit IRC07:17
*** haigang has quit IRC07:21
*** chlong has quit IRC07:36
*** chlong has joined #openstack-lbaas07:39
*** bharath__ has joined #openstack-lbaas07:40
*** eezhova has quit IRC07:52
*** ganeshna_ has quit IRC07:54
*** ganeshna has joined #openstack-lbaas07:55
*** ganeshna has quit IRC08:16
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lbaas: Updated from global requirements  https://review.openstack.org/21226108:18
openstackgerritOpenStack Proposal Bot proposed openstack/octavia: Updated from global requirements  https://review.openstack.org/21226508:20
*** eezhova has joined #openstack-lbaas08:27
*** numan has joined #openstack-lbaas08:33
*** ganeshna has joined #openstack-lbaas08:45
*** fnaval has joined #openstack-lbaas09:05
*** fnaval has quit IRC09:09
*** ganeshna has quit IRC09:11
*** ganeshna has joined #openstack-lbaas09:12
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lbaas: Updated from global requirements  https://review.openstack.org/21226109:17
*** apuimedo has joined #openstack-lbaas09:17
rm_workerr, so, uhh, I was bored for a bit waiting on some stuff to finish, so I gave this a shot...09:45
openstackgerritAdam Harwell proposed openstack/octavia: Single Call Create (WIP)  https://review.openstack.org/21306609:45
rm_workgoing to start a devstack build using that patch and see what happens, rofl09:46
rm_workprolly just go to sleep while that happens... feel free to poke holes in that implementation before I wake up09:47
rm_workhmm, I wonder if anyone else had actually been working on that -- didn't see any patchsets though, so whatev :P09:51
rm_works/patchsets/CRs/09:51
rm_workonly took about an hour and a half, so no big deal either way09:52
*** haigang has joined #openstack-lbaas09:56
*** numan has quit IRC10:00
*** haigang has quit IRC10:03
*** numan has joined #openstack-lbaas10:16
*** numan has quit IRC10:20
openstackgerritAdam Harwell proposed openstack/octavia: Single Call Create (WIP)  https://review.openstack.org/21306610:27
*** numan has joined #openstack-lbaas10:32
*** numan has quit IRC10:40
rm_workoh, cool, it works10:41
*** bharath__ has quit IRC10:41
rm_workhurrah :P10:41
* rm_work goes to sleep10:42
*** ganeshna has quit IRC10:50
*** ganeshna has joined #openstack-lbaas10:51
*** ganeshna has quit IRC10:53
*** numan has joined #openstack-lbaas10:53
*** ganeshna has joined #openstack-lbaas10:53
*** haigang has joined #openstack-lbaas10:55
*** ganeshna_ has joined #openstack-lbaas11:14
*** ganeshna has quit IRC11:14
*** ganeshna has joined #openstack-lbaas11:20
*** ganeshna_ has quit IRC11:22
*** numan has quit IRC11:35
*** bharath_ has joined #openstack-lbaas11:42
*** bharath_ has quit IRC11:47
*** ganeshna_ has joined #openstack-lbaas11:57
*** ganeshna has quit IRC11:59
*** haigang has quit IRC12:05
*** ganeshna_ has quit IRC12:21
*** diogogmt has joined #openstack-lbaas13:05
*** eezhova has quit IRC13:08
*** eezhova has joined #openstack-lbaas13:10
*** KunalGandhi has joined #openstack-lbaas13:12
*** KunalGan_ has joined #openstack-lbaas13:14
*** iwi1 has joined #openstack-lbaas13:17
*** KunalGandhi has quit IRC13:17
*** diogogmt has quit IRC13:19
*** vivek-ebay has joined #openstack-lbaas13:28
*** bharath has joined #openstack-lbaas13:31
*** bharath has quit IRC13:36
*** fnaval has joined #openstack-lbaas13:45
*** ganeshna_ has joined #openstack-lbaas14:12
*** ganeshn__ has joined #openstack-lbaas14:14
*** ganeshna_ has quit IRC14:14
*** chlong has quit IRC14:19
*** vivek-eb_ has joined #openstack-lbaas14:20
*** vivek-ebay has quit IRC14:22
*** numan has joined #openstack-lbaas14:23
*** fnaval has quit IRC14:30
*** bharath has joined #openstack-lbaas14:32
*** bharath has quit IRC14:37
*** minwang2 has joined #openstack-lbaas14:38
*** alejandrito has joined #openstack-lbaas14:44
*** fnaval has joined #openstack-lbaas14:46
johnsomYou are a wild man....14:46
*** bharath has joined #openstack-lbaas14:47
*** numan has quit IRC14:49
*** fnaval has quit IRC14:50
openstackgerritMerged openstack/octavia: Updated from global requirements  https://review.openstack.org/21226514:52
*** fnaval has joined #openstack-lbaas14:53
*** diogogmt has joined #openstack-lbaas15:01
*** TrevorV has joined #openstack-lbaas15:05
*** mlavalle has joined #openstack-lbaas15:10
*** iwi1 has quit IRC15:12
*** KunalGan_ has quit IRC15:20
*** madhu_ak has joined #openstack-lbaas15:22
*** ganeshn__ has quit IRC15:25
*** openstackgerrit has quit IRC15:31
*** ganeshna has joined #openstack-lbaas15:31
*** openstackgerrit has joined #openstack-lbaas15:32
*** ganeshna_ has joined #openstack-lbaas15:39
*** ganeshn__ has joined #openstack-lbaas15:41
*** ganeshna has quit IRC15:42
*** ganeshna_ has quit IRC15:43
*** iwi has joined #openstack-lbaas15:50
*** woodster_ has joined #openstack-lbaas16:04
*** KunalGandhi has joined #openstack-lbaas16:16
*** enikanorov2 has quit IRC16:20
*** enikanorov2 has joined #openstack-lbaas16:26
*** SumitNaiksatam has quit IRC16:28
*** SumitNaiksatam has joined #openstack-lbaas16:30
*** madhu_ak has quit IRC16:31
*** madhu_ak has joined #openstack-lbaas16:31
*** minwang2 has quit IRC16:37
*** apuimedo has quit IRC16:39
*** bharath has quit IRC16:46
*** ganeshn__ has quit IRC16:54
kfox1111what does lbaasv1 mode https do? since it doesn't support termination?16:54
*** madhu_ak_ has joined #openstack-lbaas16:55
*** madhu_ak has quit IRC16:57
*** madhu_ak has joined #openstack-lbaas17:04
*** madhu_ak_ has quit IRC17:05
*** KunalGandhi has quit IRC17:07
johnsomIt creates a TCP configuration for the flows.17:08
*** madhu_ak_ has joined #openstack-lbaas17:09
*** madhu_ak has quit IRC17:10
*** crc32 has joined #openstack-lbaas17:11
*** crc32 has quit IRC17:13
*** minwang2 has joined #openstack-lbaas17:13
*** iwi has quit IRC17:19
*** crc32 has joined #openstack-lbaas17:27
*** madhu_ak has joined #openstack-lbaas17:32
*** ajmiller_ has joined #openstack-lbaas17:33
*** madhu_ak_ has quit IRC17:35
*** madhu_ak has quit IRC17:35
*** madhu_ak has joined #openstack-lbaas17:36
*** ajmiller has quit IRC17:36
*** madhu_ak_ has joined #openstack-lbaas17:38
*** madhu_ak has quit IRC17:41
openstackgerritBanashankar k proposed openstack/neutron-lbaas: Octavia driver unit tests Added unit tests for the octavia driver.  https://review.openstack.org/21064417:46
openstackgerritBanashankar k proposed openstack/neutron-lbaas: Adding TLS args to octavia driver  https://review.openstack.org/20968217:46
openstackgerritBanashankar k proposed openstack/neutron-lbaas: WIP - Octavia driver  https://review.openstack.org/17411417:46
*** bana_k has joined #openstack-lbaas17:46
*** bharath has joined #openstack-lbaas17:47
*** bharath has quit IRC17:52
*** bharath has joined #openstack-lbaas17:59
*** madhu_ak_ has quit IRC18:01
TrevorVjohnsom, you around today my man?18:05
johnsomYep.  After my extended standup today I'm going to do a test run with Octavia devstack and update the etherpad.18:06
johnsomAssuming that is what you want to ask about... grin18:06
johnsomSo, maybe an hour or so18:09
*** woodster_ has quit IRC18:10
TrevorVSounds good18:12
TrevorVThe failover shouldn't be complete yet18:12
TrevorVI haven't put in the listener/loadbalancer updates18:12
TrevorVHoping to get more of an idea of what needs done, ya know?18:12
johnsomYep.  That is why I'm going to do an end-to-end run with debug on and do some log parsing to make sure I see all of the tasks being run in normal flows.  That should give me something to compare with our list and make sure I don't miss something....18:14
TrevorVAwesome johnsom that'll be a HUGE help.18:14
*** SumitNaiksatam has quit IRC18:15
rm_workI did update my devstack script to be *current*, so it actually works now18:17
rm_workhttps://gist.github.com/rm-you/d7fbe613d525f12dc44718:17
rm_workthough might need to tweak which octavia ref it pulls down18:18
*** vivek-eb_ has quit IRC18:21
rm_workjohnsom: did you see my random CR from last night for single-call LB?18:21
johnsomYeah, I commented that you are a wild man....18:21
johnsomHaven't had a chance to review  though18:22
rm_workit ... "works"18:23
rm_workthough i didn't test every object cause i am not sure where some of that stuff goes18:23
rm_worklike health_monitors or TLS18:23
rm_workthose might not be in yet but shouldn't be hard to add if they aren't18:23
johnsomrm_work FYI, there is a octavia/devstack directory that has sample localrc and the plugin.sh script that runs with devstack.  We probably should get the barbican stuff in there.18:27
johnsomAlso, I have switched to IMAGE_URLS+=",http://download.cirros-cloud.net/0.3.3/cirros-0.3.3-x86_64-disk.img" for cirros18:27
johnsomIt's a new location and a much better image18:27
rm_workok18:31
rm_workupdated that18:40
TrevorVrm_work, http://www.dxracer.com/us/en-us/product/1/pc_gaming_chair/king_series/oh-kf00-nw-zero/18:45
crc322015-08-14 18:47:25.313 | 'pyroute2' is not in global-requirements.txt <-- is this what killed my ./stack.sh run?18:56
johnsomYes, you need to update your dev stack environment, specifically /opt/stack/requirements18:57
johnsomhttps://review.openstack.org/#/c/168764/ <-- Merged 8/9/1518:59
crc32git pulled18:59
TrevorVjohnsom, any luck?  Still in progress my man?19:06
johnsomHave the log, parsing now19:07
TrevorVCool.19:07
johnsomFirst one will be no spares path19:07
TrevorVAlright19:08
*** vivek-ebay has joined #openstack-lbaas19:08
*** crc32 has quit IRC19:25
*** minwang2 has quit IRC19:28
*** ajmiller_ has quit IRC19:28
*** minwang2 has joined #openstack-lbaas19:28
*** ajmiller has joined #openstack-lbaas19:29
*** enikanorov2 has quit IRC19:35
*** enikanorov2 has joined #openstack-lbaas19:49
*** crc32 has joined #openstack-lbaas19:50
TrevorVjohnsom, so that's the logs of a startup for a completely functioning load balancer?19:51
johnsomYes, through member creation19:52
johnsomI'm pulled into another meeting, but after this ends I'm going to pull that into the failover list as necessary19:53
TrevorVAlright johnsom thanks.  I'm not sure how much longer I'll be around.  Some family stuffs going on, but hopefully I'll still be around for a couple more hours19:54
johnsomOk, have a good weekend.  I kind of also want to do a diagram off of this, just to have handy19:55
xgermanjohnsom there is a way to turn task flow stuff into a diagram FYI19:56
xgermanHave a great weekend TrevorV19:56
johnsomYep19:57
TrevorVThat would be awesome!  Could use it in future docs as well to describe work-flow, y eah?19:57
TrevorVyeah***19:57
rm_youblogan pointed out that single-call should prolly happen mostly post-queue and my impl was pre-queue, so i'm going to start over :P20:00
rm_youthough that CR is still ... interesting in an academic sense, i think20:00
xgermanlet’s not get too hung up with things we don’t necessarily need for Liberty20:01
xgermanI am not even sure if Neutron will get the “get me a network” done20:01
rm_youyeah, i suppose so -- is there something else I could help with20:01
bloganxgerman: neutron-lbaas willg et the single create callf or sure, just prob not liberty unles rm_you does it20:01
rm_youi might look at the single-call in my off-time tho <_< which is a little weird since WFH my time all blurs together anyway >_<20:01
xgermanblogan agreed20:01
xgermanI am single middle trying to get us to become ref implementation in Liberty20:02
xgerman:-)20:02
xgermansingle mindedly20:02
rm_youyeah, what would help with that20:02
bloganrm_you: docs20:02
rm_youlol20:02
xgerman:-)20:02
* rm_you backs away slowly20:02
* rm_you backs away more quickly20:03
xgermanwe probably need to do more tests to find holes/bugs20:03
* rm_you sprints away20:03
* TrevorV notes rm_you sprinting backwards o_020:03
xgermanit worries me that dougwig’s driver hasn’t merged20:03
xgermanis there something we can do to expedite that20:04
xgerman?20:04
johnsomYeah, I am a bit freaked out over the driver20:04
dougwigxgerman: i think we need to get it merged in some form to let folks iterate.20:04
dougwigbut are dependent commits and Depends-On not able to fill the gap?20:04
*** alejandrito has quit IRC20:04
rm_youbana_k was working on it AFAIK?20:04
rm_youi saw more patchsets this morning20:05
xgermanyep20:05
xgermanand I like that stuff merged ASAP before we get into the crunch and the gate fails and...20:05
rm_youlol yeah20:05
johnsombana_k did some unit tests and is working on updating for the TLS stuff20:05
rm_youwell, i can't help with that, it's neutron-lbaas20:05
rm_you(can't help with getting it merged, don't have +2)20:06
*** vivek-ebay has quit IRC20:06
rm_youi could help WORK ON IT, but i think bana_k has it covered20:06
xgermanmaybe review and heckle pothole and blogan :-)20:06
johnsomrm_work A pass over this would help: https://review.openstack.org/#/c/201882/20:06
johnsomIt's next on my list of things to look at and work on20:06
rm_youah right20:06
rm_youi have stayed away from that since it still seemed to be in heavy development20:07
xgermanyep, that’s another thing we need to square away quickly20:07
rm_youis it stable?20:07
xgermanonly tests will tell20:07
blogandougwig, xgerman: we can get it merged, but if Liberty releases and it doesn't do proper status updating that would be disappoitning to everyone20:07
xgermanyeah, since rm_work was looking for work — is there something he can do to make that all awesome20:08
xgerman?20:08
johnsomrm_work I don't know yet.  It's been on my back burner a bit.  I have signed up to add some config file/data passing to it.20:08
bloganTrevorV: you going to be abelt o work on that piece? or anyone else?20:08
xgermanI am stuck in internal projects so my time budget is unpredictable20:08
bloganrm_you can if he wants to, not sure if he'll start backing away all the way to mexico though20:08
bloganxgerman: same, but i may have to force myself to do this piece20:09
xgermanthat’s not too far from your neck of the woods :-)20:09
rm_youxgerman: that has been me, but i'm in a holding pattern right now for internal stuff so i have more time20:09
rm_youbut might have to disappear again at a moment's notice <_<20:09
bloganyou've already disappeared20:09
rm_you<_<20:09
bloganyou're just a ghost in the machine now20:09
xgermanlol20:10
rm_youT_T20:10
rm_youspeaking of, afk for the next 45m20:10
xgermanok, so we need stats in that driver20:10
xgermanmmh20:10
rm_youlet me know if you decide on some coding task you need me on20:11
rm_youstats as in... bandwidth/connections/etc?20:11
bloganxgerman: stats in the octavia driver?20:11
rm_youor stats as in "status of member nodes"20:11
rm_youor something else20:11
xgermanajmiller was trying to make that more fancy but right now it’s just bytes in/out per lb20:11
bloganso both of those are going to be itneresting to accomplish20:11
rm_youi think that's all that is necessary20:11
rm_youstrictly speaking20:11
rm_youat least for initial launch20:12
bloganoctavia doesn't even have a get stats call yet20:12
rm_youit's just exposing data from haproxy20:12
ajmillerblogan rm_you xgerman Yes, I have finally gotten back to the listener stats patch.20:12
xgermanmmh, I though cc32’s latest UDPm incarnations ends that data20:12
rm_youdunno, i'll read through it later today20:13
xgermanso all we need to do is tally it up somewhere in the DB20:13
bloganyes just exposing the data but thats a bit more involved, whetehr we just ahve an api go directly to the haproxy instances or if we those stored in the db fromt he heartbeats, those arent in place20:13
rm_youdo need to afk for 30-45 tho right now20:13
ajmillerI still have fantasies of getting it into Liberty....20:13
*** woodster_ has joined #openstack-lbaas20:13
bloganxgerman: those stats aren't being inserted as far as i know20:13
xgermanyep, that code is missing20:13
xgermanand also the code to get it out20:14
blogannot even sure we have a table for that20:14
bloganyep20:14
bloganand then member status is going to be a huge deal for neutron-lbaas to get from octavia20:14
xgermanbut member status is in the DB20:14
bloganbc neutron-lbaas simply polling octavia's api for all the members is not scalable20:14
bloganxgerman: neutron-lbaas needs to have its db updated with what octavia has20:14
xgermanlet’s not overcomplicate for now. If we add some bulk call to get all memebers20:15
bloganwell just fyi, no other drivers (except the namespace driver) update the member statuses20:16
bloganin neutron-lbaas20:16
bloganas far as i can remember20:16
bloganbut unfortnately, the namespace driver does20:16
*** madhu_ak has joined #openstack-lbaas20:16
xgerman:-(20:16
blogandougwig: any thoughts on this?20:17
blogandougwig: specifically, should updating the member's operating status be a requirement to be the ref impl20:17
TrevorVblogan, sorry, were you asking if I was going to do the piece that updates the status up the tree throuh neutron lbaas?20:19
TrevorVI can def do that20:19
xgermanmmh, as bogan said member status won’t scale20:20
xgermanblogan20:20
bloganTrevorV: i mean the piece that makes the octavia commands async20:20
bloganxgerman: separate issue, octavia driver currently just gets a call to create an lb and passes that command to octavia, once octavia responds, the driver sets the provisioning status to ACTIVE, even though its PENDING_CREATE in octavia20:21
xgermanmmh20:21
bloganso a thread needs to be spawned that polls that particular lb until its active or ERROR (or timesout)20:21
blogani don't tink that will have a scaling issue since its just for one single lb (or listener, pool, etc) and its a short lived thread20:22
xgermanwell, can we use our queue to send back updates?20:22
bloganthe octavia queue?20:22
xgermanyep20:22
xgermanof course new queue but we could add a bakchannel20:23
bloganwe could do something like that, but that requires code in both projects, whereas just the polling requires code in neutron-lbaas20:23
xgermanwe also need to move potentially member status and stats20:23
bloganbut we will have to do something like that for a scalable solution to get the member statuses20:24
xgermanyep, my thinking as well20:24
bloganxgerman: yeah but i stil think polling for this piece (not the member status) is the better wya to go for now20:25
xgermanyeah, but if we have to create a queue anyway… just saying :-)20:25
blogantrue20:25
blogandepends on if we need the memebr status for liberty or not then20:26
johnsomI think it is a must.  How can users troubleshoot?20:26
bloganits a must for the product, i'm not convinced its a must for Liberty considering what is already in neutron-lbaas20:27
xgerman+120:27
xgermanalso one week is short20:27
xgermanactually two weeks20:27
blogani think we're all spread a little thin too unfortunately20:27
xgerman:-(20:28
bloganif only openstack just paid me20:28
bloganTrevorV: are you already working on this or should i start on it?20:29
TrevorVI don't know that I understand what you said blogan.  Isn't the status update the same thing your discussing at this point?20:29
TrevorVJust HOW that happens?20:29
xgermanwell, there is the thing where you ask Octavia to fire up a listener, lb , etc and then you fork a thread to make sure it actually happened20:30
bloganTrevorV: 2 issues, all members that octavia has may have their operating status's changed at any time in their lifecycle, so neutron-lbaas needs to be updated on all of those status changes20:30
bloganTrevorV: the issue I'm asking you to complete is the whole driver just setting the provisioning status to ACTIVE when octavia hasn't reported it as ACTIVE20:30
TrevorVOkay, so you're talking about neutron lbaas member health monitoring, but asking me if I'm going to propogate the appropriate status according to Octavia, and then updating accordingly with its actual status20:31
TrevorV?20:31
bloganTrevorV: just for a single entity, so a user creates an lb through neutron lbaas, neutron lbaas calls octavia to create the lb, neutron-lbaas then returns to the user that the lb is in PENDING_CREATE. meanwhile the octavia drvier spawned a thread that is polling octavia waiting for it to become ACTIVE, and once it does it sets the status in the DB20:35
bloganin the neturon-lbaas DB20:35
TrevorVYeah, that's what I was talking about the stuff you suggest I work on.  Yeah I can do that.20:36
TrevorVHave some more work in the failover stuff, but I can do both20:36
TrevorVI won't be able to really get to it today, I'm on babysitting duty tonight, but I can maybe on Sunday/Monday, yeah?20:36
xgermansure - sounds good. I am camping this weekend so won’t be off much help20:37
*** vivek-ebay has joined #openstack-lbaas20:37
xgermanwill see if I can write some stats code later today so we at least get that into the Octavia DB20:38
bloganTrevorV: i might get ocd on this and attempt to do it, just to see how hard it is, i'll let you know, unless you really want to do it20:38
bloganTrevorV: but it needs to be done ASAP, and you plan on doing it in another review correct?20:39
TrevorVin another review than... what?20:39
TrevorVIn a separate review than my delete fix review?20:39
*** vivek-eb_ has joined #openstack-lbaas20:40
bloganyes20:40
bloganif so then we can get it all merged20:40
TrevorVI can do it in the same review, just chalk it up to "driver optimizations" or something.20:41
*** vivek-ebay has quit IRC20:41
TrevorVWhat do you think?20:41
TrevorVIf YOU want to get at it, then by all means20:41
TrevorVYou'll probably get to it before I would20:42
bloganTrevorV: i'd rather get the deletes working in as well and not have to wait on this to get done20:42
TrevorVRight, but deletes are still "pending" until the octavia driver goes in20:42
bloganyes but is it pending bc the code is not done? or bc its waiting on the octavia driver?20:42
TrevorVdelete fixes are waiting for octavia driver20:45
bloganbut the code is good right?20:45
TrevorVI'm saying if that's pending for something specific you might have the appropriate time.20:45
TrevorVYeah20:45
bloganwell i'm talking about just getting the octavia driver merged right now20:45
bloganadn then the deletes20:45
bloganah but it needs more tests20:46
TrevorVThe deletes needs more tests?20:46
bloganno driver20:46
blogandougwig's20:46
bloganim gonig to work on that right now unless he has20:47
TrevorVOh yeah, bana_k was doing that I thought20:47
blogandougwig ?20:47
bloganohhh20:47
bloganbana_k: around?20:47
TrevorVCheck dependent reviews, blogan should be there already20:47
bloganjust tls and delete20:47
bloganoh nvm its after tls20:48
TrevorVGotcha.20:48
TrevorVYeah, idk20:48
TrevorVhah20:48
bloganoh good, thanks bana_k20:48
blogannot sure this should depend on the tls review20:48
bloganok time to do a bit of git magic20:49
TrevorVIt should if he plans to have unit tests for the tls review as well, right blogan ?20:49
bloganyeah but there arent any20:49
TrevorVOh.20:49
TrevorVWell.20:49
TrevorVMy b20:49
TrevorVAlright, I have to step away20:49
bloganso really this should depend on the octavia driver reveiw, and phil's tls review should have the unit test additions20:49
TrevorVjohnsom, I should have some changes up this weekend or Monday, earliest.  Sorry I can't get to it ATM :(20:49
TrevorV+1 blogan20:50
TrevorVHave a good weekend all!20:50
*** TrevorV has quit IRC20:50
johnsomTrevorV Sounds good.  We are all busy these days.  I'm starting on my work for crc32 now.20:50
bana_kblogan20:57
bana_kyes20:57
bana_ksorry was away20:57
bana_kI have unit tests for it.20:58
bloganbana_k: np, thanks for doing that20:58
bana_kI followed the other drivers unit tests cases20:58
bloganbana_k: but i dont think you need to be dependent on the tls reveiw20:58
bana_kand covered most of it.20:58
bloganbana_k: just the Octavia driver20:58
bana_kPlease someone take a look at it and let me know if thats all we want or needs some change.20:59
bloganbana_k: quick look over i did, it looks good but ill take a more in depth look here in a sec once i get the root driver review fixed (a pep8 issue)20:59
bana_kYes I am dependent on the tls changes, as the rest call args changes21:00
bana_kthanks blogan21:00
bana_kkiran took a loot at it and suggested couple of cosmetic changes21:01
bana_kso fixed them21:01
rm_workblogan: honestly i think all of those could be one review, the TLS Additions is an addon to a driver that isn't even merged yet21:01
rm_workwhy not just include it21:01
rm_workand typically tests are included in the original review, not a sub-review21:01
rm_workso like21:01
bloganrm_work: true21:02
rm_workyou could just pull down that chain and --reset down to the first commit, amend it, and submit it all as the original single CR21:02
bloganand trevor's is just 3 line change21:02
rm_workyeah21:02
bloganso 2 reviews can be rolled in21:02
bloganthe tests probably could too21:02
rm_workyes21:02
bloganjust would inflte the lines but meh it makes sense21:03
bloganbana_k: you ahve any changes in flight?21:03
rm_workalso, i am more a fan of a queue backchannel than polling for status updates, but i always choose the "better" solution over "quick"21:03
rm_workeven when sometimes we need quick21:03
rm_workbut that's still my vote21:03
bana_kno all are committed21:03
bloganrm_work: polling for member status or just the single entity upon a create, update, delete?21:03
bloganbana_k: okay i think i'm going to combien it all into one reveiw and give us all Co-Authorship21:04
rm_workany polling on the neutron-lbaas side21:04
bana_kok sounds good :)21:04
rm_workreally any status update in octavia should trigger adding an event to a queue that could be read by neutron-lbaas21:04
rm_workessentially a queue that is a status change event stream21:04
rm_workand we'd have a consumer on the neutron-lbaas side watching that stream and making updates in the neutron db21:05
bloganrm_work: it is the right way but thats going to be a bit more involved, unless you want to work on that piece and you're sure you get it done soonish21:05
rm_workto reflect the changes21:05
rm_workhmm21:05
rm_workI could look into it21:05
bloganbut the fact it requires code in both octavia and neutron-lbaas gives me pause for it being done before liberty21:05
rm_workjust need to figure out where to hook in on the octavia side21:05
rm_workwe have one week left? O_o21:05
blogan221:05
rm_workyeah might be tough but i think i could do it21:05
bloganwell trevor and/or i will do the polling piece as a backup plan21:05
rm_workthe curious part is actually not whether i can DO it, but whether we can get it merged <_<21:05
bloganoh yes it can be done, but can it be done well enough in time to be merged21:06
rm_worki'll take that as my task then21:06
rm_workqueue for status updates21:06
rm_workk21:06
bloganoslo_messaging hooray!21:06
rm_worki am still going to think of it as an "event stream"21:06
rm_workanywho, brb again21:06
bloganwell it'll probably be used for stats as well21:06
rm_workugh k21:07
rm_worknoted21:07
rm_workwill plan for that eventuality21:07
blogangreat21:07
*** jerrygb has joined #openstack-lbaas21:24
bana_kI am kinda free now. Let me know if there is any work for me21:25
*** vivek-eb_ has quit IRC21:26
*** vivek-ebay has joined #openstack-lbaas21:26
*** jerrygb has quit IRC21:30
*** vivek-eb_ has joined #openstack-lbaas21:41
*** vivek-ebay has quit IRC21:44
johnsombana_k If you have some time, testing out Octavia on devstack and filing bugs could be handy.  I've been working on that some this week too.21:53
johnsomOr there is always docs... grin21:53
openstackgerritBrandon Logan proposed openstack/neutron-lbaas: Octavia driver  https://review.openstack.org/17411422:19
blogandougwig, ajmiller, pothole, xgerman, johnsom, rm_work, bana_k: ^^22:22
ajmillerack22:23
bana_kI can take devstack testing :P22:26
johnsomack22:27
bana_kcan i use https://gist.github.com/rm-you/d7fbe613d525f12dc447 to launch a devstack ?22:28
bana_kfor testing22:28
bana_kblogan : ack22:29
johnsomYou can22:29
bana_kok cool.22:29
xgermanlooking22:30
johnsomI'm going to post one with the neutron_lbaas.conf commented line in it22:33
bana_kand on the other hand, whats need to be documented ?22:34
openstackgerritMichael Johnson proposed openstack/neutron-lbaas: Octavia driver  https://review.openstack.org/17411422:36
johnsomOk, now looks good to me22:37
xgermanbana_k there is some debrief thing I forgot how to find we need to add some better LBaaS chapter22:37
xgermanjohnsom they look identical? or is Friday afternoon catching up with me22:39
johnsomFriday is catching up to you, it has been a long week.  I modified neutron_lbaas.conf to have the octavia service string in it commented out22:39
xgermanyep, it is22:40
johnsomCrumb, and looking at it, I put it in the wrong section.22:40
johnsomJust a sec22:40
bana_koh ok. I can try to take a look at it.22:41
openstackgerritMichael Johnson proposed openstack/neutron-lbaas: Octavia driver  https://review.openstack.org/17411422:41
*** vivek-eb_ has quit IRC22:42
*** vivek-ebay has joined #openstack-lbaas22:42
*** ajmiller_ has joined #openstack-lbaas22:45
openstackgerritMichael Johnson proposed openstack/octavia: health manager service  https://review.openstack.org/16006122:47
*** ajmiller has quit IRC22:49
openstackgerritPengtao Huang proposed openstack/neutron-lbaas: change if to elif  https://review.openstack.org/21256022:49
*** Purandar has joined #openstack-lbaas23:04
openstackgerritAl Miller proposed openstack/neutron-lbaas: Filter get_pool_members to return members from the desired pool  https://review.openstack.org/21096823:05
Purandarhello23:09
*** purandarkakde has joined #openstack-lbaas23:16
*** Purandar has quit IRC23:18
openstackgerritSherif Abdelwahab proposed openstack/octavia: Amphora Flows and Service Drivers for Active Standby  https://review.openstack.org/20625223:19
xgermanPurandar hi23:23
rm_youbana_k: if you are doing testing for octavia, replace the refss/changes/##/######/# with the one from the patchset you want to test (on the enable_plugin octavia line)23:24
rm_youthe one in there now was my not-accurate single-call CR23:24
rm_youwhich you probably don't care about :P23:25
rm_you(line 16)23:25
bana_koh ok :). will change that accordingly23:25
bana_kthanks23:25
bana_kso any preference on which patch I should be testing. why to overlap the testing23:26
rm_youyou can remove the ref/*23:27
rm_youand just have the line be:23:27
rm_youenable_plugin octavia https://review.openstack.org/openstack/octavia23:27
xgermanrm_you I am a fan of the queue as well so I am happy to do some part of it23:27
rm_youthen it'll just test on octavia master23:27
rm_youxgerman: alright, i'm going to start taking a look, but pointers at where to hook in to the existing event flow would be good23:28
bana_kyes thats what we want?23:28
xgermanwell, if things are done right we need to hook into the HealthMixin and I need to write a stats mixin23:28
rm_youbasically just need to find where to shove a "enqueue_status_event(object_type, object_id, new_status)" call23:28
rm_youand that should be *it* on the octavia side23:29
rm_youpretty simple23:29
rm_youthen the neutron-lbaas side needs to have a consumer that watches the queue and does db updates23:29
xgermanyeah, a QueueHealthMixin and QueueStatsMixin23:29
xgermanyep23:29
rm_youI have "mixed feelings" about using a ton of mixins23:29
rm_youbut I guess that's alright23:29
xgermanwell, that way we can replace them with something else in prod23:30
xgerman(e.g. we will send stats straight into some ceilometer queue IHMO)23:30
rm_youtrue23:30
rm_youhttp://www.artima.com/weblogs/viewpost.jsp?thread=24634123:30
rm_youhttps://www.artima.com/weblogs/viewpost.jsp?thread=24648323:31
rm_youpart 1 and 223:31
xgermanwell, we can always make a health and stats driver23:32
rm_youlol23:32
rm_yousuch driver23:32
rm_youmany plugin23:32
rm_youwow23:32
rm_youi honestly don't have a problem either way -- the mixin option is less configurable though I think?23:33
rm_youbasically i'd like the code to throw a "status change event" and then have some other code (configurable) decide what to do with it23:33
rm_youso maybe plugin is more apt23:33
rm_youplugin/driver23:33
xgermanso here is the format of the message23:34
xgermanhttps://review.openstack.org/#/c/201882/29/octavia/controller/healthmanager/heartbeat_udp.py23:34
xgermanI will just use that and put it on the queue for now23:34
rm_youam I looking at the json thing in dorecv?23:34
rm_youah, are you going to do it?23:34
xgermancrc32 sends that to the controlelr with UDP I turn around and put it on the queue23:35
xgermanyep, I can do that23:35
rm_youxgerman: well again, i'd rather have it call out to some interface23:35
rm_youwhich then lets us configure what happens to it23:35
xgermanso a mixin is not an interface?23:36
rm_youeither sent to queue, or logged, or whatever23:36
rm_youwell23:36
rm_youunless I'm mistaken, to switch which option is used, you'd have to edit the code to use the different mixin23:36
rm_youright?23:36
rm_youso like a StatusChangeLogMixin vs. StatusChangeEventQueueMixin23:36
xgermannot necessarily23:36
rm_youyou'd have to actually switch which one the code inherits23:36
crc32johnsom said he would write code to inject an ampphora conf file instead of the status_sender.json. xgerman wants to stop using status_sender.json23:36
johnsomYep23:37
rm_youerr, what?23:37
rm_youi mean23:37
crc32sorry I heard json thing in dorescv and that you meant /etc/amphora/status_sender.json23:38
rm_youi don't know exactly what you're referring to, but i really don't care what the message looks like -- i just wanted to "handle" (log/queue/whatever) specific events, like status changes23:38
xgermanok, cool'23:38
rm_yousomewhere in here, octavia gets a notification that a status has changed on a member/vip/listener/whatever23:38
rm_youi just want to put a hook there23:39
xgermanyeah, I can handle the hooking23:39
rm_youk23:39
rm_youthen let me know when you have a CR?23:39
rm_youwere you going to do that TODAY?23:39
rm_youif not, then maybe just let me do it because i will probably be working on it this weekend while you are off camping23:40
rm_yousince i kinda want to take Monday off :P23:40
xgermanwell, if you write the receiving part I can fill in the sender Monday and we can test it Tuesday ;-)23:41
xgermanI might also be able to do it tonight...23:41
rm_youI'll prolly just do both sides if you don't start today23:41
rm_youlol23:41
rm_youi tend to just get focused and hammer stuff out23:41
rm_youit's fine23:41
rm_youplus i have a vision now :P23:42
xgermanhere is how mixing can be loaded dynamically: https://review.openstack.org/#/c/206252/11/octavia/amphorae/drivers/driver_base.py23:43
rm_youloool23:43
xgermanand I hope our visions overlap — aka we either use some driver or mixin to write to the queue23:43
rm_youso we use a mixture of dynamic mixins and stevedore drivers?23:43
rm_youthat's ... >_>23:43
rm_youjust got finished removing the other class-loading mechanism from the cert stuff23:44
rm_youi don't know if using BOTH is a great idea, but that's fine23:44
rm_youI'll look at both ways, but it'll be really similar either way23:44
rm_youmixins might be "easier"? maybe23:44
xgermannot sure — I speed that stuff out in http://octavia.io/review/master/specs/version0.5/amphora-driver-interface.html23:45
xgermanspeced23:45
rm_youalright23:45
rm_yousounds good then23:45
xgermancool23:46
rm_younot as familiar with this method but i'll figure it out23:46
rm_youdo you have a base abstract Mixin and then extend it for the actual Mixins?23:46
xgermanyep23:46
rm_youk23:46
rm_youyeah ok i see that from the VRRP example23:47
rm_youI'll give that a shot this weekend23:47
xgermanIf I get to it tonight I will send you an e-mail or something23:48
rm_youi'll be here23:49
rm_youthis is my gaming machine :P23:49
rm_youI am always online either here or rm_work because i never am away from a computer for more than about 15 minutes <_<23:49
xgermanget a life!23:50
rm_youlol23:50
*** ajmiller_ has quit IRC23:54
*** mixos has joined #openstack-lbaas23:57

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