openstackgerrit | Aishwarya Thangappa proposed openstack/neutron-lbaas: Tempest tests using testscenarios for neutron-lbaas https://review.openstack.org/178883 | 00:30 |
---|---|---|
*** mlavalle has quit IRC | 00:32 | |
*** hitalia has quit IRC | 00:47 | |
*** Aish has quit IRC | 00:51 | |
*** fnaval has quit IRC | 00:59 | |
*** fnaval has joined #openstack-lbaas | 01:17 | |
*** rm_you| has joined #openstack-lbaas | 01:19 | |
*** rm_you has quit IRC | 01:20 | |
*** bharath has joined #openstack-lbaas | 01:49 | |
*** SumitNaiksatam has joined #openstack-lbaas | 01:52 | |
*** bharath has quit IRC | 01:53 | |
*** SumitNaiksatam has quit IRC | 02:03 | |
*** SumitNaiksatam has joined #openstack-lbaas | 03:39 | |
*** Aish has joined #openstack-lbaas | 04:24 | |
*** openstackgerrit has quit IRC | 05:22 | |
*** openstackgerrit has joined #openstack-lbaas | 05:22 | |
*** jschwarz has joined #openstack-lbaas | 05:35 | |
openstackgerrit | Phillip Toohill proposed stackforge/octavia: Add devstack plugin for octavia https://review.openstack.org/167796 | 05:40 |
*** kiran-r has joined #openstack-lbaas | 05:40 | |
*** kiran-r has quit IRC | 06:14 | |
*** [1]evgenyf has joined #openstack-lbaas | 06:33 | |
*** kiran-r has joined #openstack-lbaas | 06:40 | |
*** kiran-r has quit IRC | 06:46 | |
*** kiran-r has joined #openstack-lbaas | 06:55 | |
*** chlong has quit IRC | 07:42 | |
*** kobis has joined #openstack-lbaas | 07:49 | |
*** woodster_ has quit IRC | 07:50 | |
*** kiran-r has quit IRC | 08:03 | |
*** kobis has quit IRC | 08:31 | |
*** kobis has joined #openstack-lbaas | 08:31 | |
*** kiran-r has joined #openstack-lbaas | 08:56 | |
*** openstackgerrit_ has joined #openstack-lbaas | 10:16 | |
*** openstackgerrit_ has quit IRC | 10:31 | |
*** apuimedo has joined #openstack-lbaas | 10:32 | |
*** kiran-r has quit IRC | 11:15 | |
*** kiran-r has joined #openstack-lbaas | 11:37 | |
*** kiran-r has quit IRC | 11:54 | |
*** kiranr has joined #openstack-lbaas | 11:54 | |
*** woodster_ has joined #openstack-lbaas | 11:58 | |
*** kiranr has quit IRC | 11:59 | |
*** Aish has quit IRC | 13:35 | |
openstackgerrit | Shang Yong proposed openstack/neutron-lbaas: Fix a pid file name error Closes-bug: 1450474 https://review.openstack.org/179068 | 13:40 |
openstack | bug 1450474 in neutron "destory a loadbalancer will not kill haproxy process" [Undecided,New] https://launchpad.net/bugs/1450474 - Assigned to Brian Shang (brian-shang-dev) | 13:40 |
*** ajmiller has joined #openstack-lbaas | 13:52 | |
*** chlong has joined #openstack-lbaas | 13:56 | |
*** [1]evgenyf has quit IRC | 14:25 | |
*** evgenyf has joined #openstack-lbaas | 14:36 | |
*** fnaval has quit IRC | 14:42 | |
*** chlong has quit IRC | 14:53 | |
*** mlavalle has joined #openstack-lbaas | 14:53 | |
*** fnaval has joined #openstack-lbaas | 14:55 | |
*** fnaval has quit IRC | 14:58 | |
*** TrevorV_ has joined #openstack-lbaas | 15:00 | |
*** fnaval has joined #openstack-lbaas | 15:02 | |
*** TrevorV_ has quit IRC | 15:03 | |
*** jorgem has joined #openstack-lbaas | 15:05 | |
*** jschwarz_ has joined #openstack-lbaas | 15:07 | |
*** jschwarz has quit IRC | 15:07 | |
*** apuimedo has quit IRC | 15:10 | |
openstackgerrit | Phillip Toohill proposed stackforge/octavia: Update nova_driver to use amp_network https://review.openstack.org/178477 | 15:10 |
openstackgerrit | Phillip Toohill proposed stackforge/octavia: Update nova_driver to use amp_network https://review.openstack.org/178477 | 15:20 |
*** evgenyf has quit IRC | 15:22 | |
*** kobis has quit IRC | 15:27 | |
johnsom | blogan Are you going to poke adam to +2/+A the controller, or should I poke German? The oddness is German is a co-author | 15:27 |
* blogan pokes rm_work|away, rm_you| | 15:28 | |
blogan | he's not in the office yet | 15:28 |
blogan | if you can get german to do it go ahead | 15:28 |
johnsom | Hahahaaha, ok | 15:28 |
blogan | adam was fine with merging it yesterday too | 15:28 |
johnsom | Yeah. I'll see if I can find German. He is out at a bunch of meetings | 15:29 |
blogan | must be meeting season | 15:30 |
*** rm_work|away is now known as rm_work | 15:38 | |
openstackgerrit | Merged stackforge/octavia: Implements Octavia Controller Worker https://review.openstack.org/151496 | 15:40 |
*** mugu has left #openstack-lbaas | 15:40 | |
*** jorgem has quit IRC | 15:41 | |
johnsom | Cool stuff. Let me know when the queue consumer is rebased (maybe updated for new bin location), I will kick it through as well. | 15:48 |
johnsom | I am moving the open bugs in controller worker from our team's stash to launchpad. | 15:49 |
ptoohill | https://review.openstack.org/#/c/178477/ | 15:52 |
rm_work | ptoohill: commented | 15:58 |
rm_work | TrevorV: commented on https://review.openstack.org/#/c/170989/7 | 16:05 |
ptoohill | rm_work: There's error handling in the moethod that calls that one | 16:09 |
rm_work | ok | 16:10 |
rm_work | then that's the answer i needed | 16:10 |
rm_work | that's just one of my "pattern triggers" | 16:10 |
ptoohill | ;) | 16:10 |
rm_work | like seeing nested if statements | 16:10 |
ptoohill | Fair enough, That code was already there, I just updated it to use ID from config | 16:10 |
rm_work | want to comment on the code for posterity? | 16:11 |
rm_work | then I can +2 | 16:11 |
ptoohill | I did | 16:11 |
ptoohill | :) | 16:11 |
ptoohill | Well, now it's reviewed :P | 16:11 |
ptoohill | Though, if we ever plan on using get_amphora in other calls those would need error handling or the transform method needs updated. Right now, only one thiung calls it and handles the errors the way it needs to from what I can tell | 16:14 |
ptoohill | This patch really was just a way to get around having a networking name in the config. And, since the ID will already live there it was a hop, skip and jump to pull the required info | 16:16 |
ptoohill | Hey ajmiller, for the devstack plugin, we dont have it marked as WIP and am curious in your opinion what, if anything, else it needs? | 16:19 |
*** nmagnezi has joined #openstack-lbaas | 16:19 | |
ajmiller | ptoohill Mainly it just needs testing and validation with working control plane binaries. | 16:19 |
ptoohill | I made updates lastnight where it now calls the api built by setup.py and a few other tweaks | 16:19 |
ajmiller | OK, cool. | 16:19 |
ajmiller | I've been mostly on vacation since last Friday, so am a little behind on stuff. | 16:20 |
ptoohill | gotcha, Ive only tested the api, creating mgmt network and tempest config population. So, this review will probably hang out in review land for a while I assume? | 16:20 |
ptoohill | Ah, no worries :) | 16:20 |
ajmiller | Yes, it can hang out for a while. Should it be marked as WIP? | 16:21 |
ptoohill | I don't think so, from the things ive tested and looking at the other binary start ups it seems to be in good position for review. I was just curious if we needed anything else | 16:22 |
ajmiller | Cool. I don't think I have any outstanding items to do, other than testing and validation. | 16:23 |
ptoohill | I guess I do have a question | 16:24 |
ptoohill | Will the other binaries be installed with setup.py? | 16:24 |
ptoohill | If so, then we would need to change a few things | 16:25 |
ajmiller | which other binaries? | 16:25 |
*** Aish has joined #openstack-lbaas | 16:25 | |
ptoohill | the other workers, everything but api for this question | 16:25 |
ptoohill | the consumer, housekeeper etc | 16:26 |
ajmiller | Ah, I'm not sure about that. I would guess that they would, but that's probably a question for dougwig blogan johnsom xgerman | 16:26 |
ptoohill | ;) fair enough. If so, we should chaenge the 'bin_dir' to use what other projects are using which is: | 16:27 |
johnsom | I think they probably should be installed with setup.py. Not sure why they shouldn't | 16:27 |
ptoohill | $(get_python_exec_prefix) | 16:27 |
ptoohill | Just curious, as right now the plugin assume it will start from within the package. If we install everything with setup.py then we should call those binaries | 16:28 |
ptoohill | but, i cant really test those at the moment so its not a big deal | 16:28 |
ptoohill | Not a big deal right now, was just curious since I was messing with it this AM. We can look into it deeper later. | 16:29 |
ajmiller | ok, great. | 16:29 |
openstackgerrit | Trevor Vardeman proposed stackforge/octavia: Update PUT methods to send update through queue https://review.openstack.org/170989 | 16:34 |
*** bharath has joined #openstack-lbaas | 16:51 | |
*** madhu_ak has joined #openstack-lbaas | 16:55 | |
*** jorgem has joined #openstack-lbaas | 17:01 | |
*** jorgem has quit IRC | 17:01 | |
*** rm_work is now known as rm_work|away | 17:03 | |
*** Youcef has joined #openstack-lbaas | 17:04 | |
*** hitalia has joined #openstack-lbaas | 17:13 | |
*** jschwarz_ has quit IRC | 17:16 | |
*** nmagnezi has quit IRC | 17:21 | |
*** sbalukoff has quit IRC | 17:23 | |
*** SumitNaiksatam has quit IRC | 17:34 | |
*** johnsom_ has joined #openstack-lbaas | 17:39 | |
*** kbyrne has quit IRC | 17:47 | |
*** johnsom has quit IRC | 17:47 | |
*** jkoelker has quit IRC | 17:47 | |
*** ptoohill has quit IRC | 17:47 | |
*** jkoelker has joined #openstack-lbaas | 17:49 | |
*** ptoohill has joined #openstack-lbaas | 17:49 | |
*** kbyrne has joined #openstack-lbaas | 17:54 | |
*** SumitNaiksatam has joined #openstack-lbaas | 17:56 | |
*** johnsom_ is now known as johnsom | 17:57 | |
openstackgerrit | Madhusudhan Kandadai proposed openstack/neutron-lbaas: Tempest tests using testscenarios for neutron-lbaas https://review.openstack.org/178022 | 18:12 |
*** mlavalle has quit IRC | 18:18 | |
*** mwang2 has joined #openstack-lbaas | 18:19 | |
*** rm_work|away is now known as rm_work | 18:23 | |
*** sbalukoff has joined #openstack-lbaas | 18:28 | |
*** Youcef has quit IRC | 18:33 | |
*** rm_work is now known as rm_work|away | 18:35 | |
*** xgerman has joined #openstack-lbaas | 18:43 | |
*** rm_work|away is now known as rm_work | 18:48 | |
blogan | ptoohill, ajmiller: yeah setup.py should install those binaries | 19:25 |
ajmiller | ok, thanks for verifying blogan | 19:25 |
blogan | johnsom: i'm rebasing the queue consumer | 19:26 |
blogan | ajmiller: its not setup yet to do that though :) | 19:27 |
ajmiller | right | 19:27 |
*** madhu_ak_ has joined #openstack-lbaas | 19:30 | |
*** madhu_ak has quit IRC | 19:33 | |
openstackgerrit | Trevor Vardeman proposed stackforge/octavia: Update PUT methods to send update through queue https://review.openstack.org/170989 | 19:39 |
*** mwang2 has quit IRC | 19:43 | |
*** mwang2 has joined #openstack-lbaas | 19:48 | |
*** hitalia has quit IRC | 19:51 | |
*** bavanveen has joined #openstack-lbaas | 19:58 | |
johnsom | Cool | 20:00 |
*** madhu_ak has joined #openstack-lbaas | 20:02 | |
*** madhu_ak_ has quit IRC | 20:02 | |
*** hitalia has joined #openstack-lbaas | 20:07 | |
*** mlavalle has joined #openstack-lbaas | 20:15 | |
*** bavanveen has quit IRC | 20:32 | |
*** mwang2_ has joined #openstack-lbaas | 20:44 | |
*** mwang2 has quit IRC | 20:47 | |
*** madhu_ak has quit IRC | 20:48 | |
*** madhu_ak has joined #openstack-lbaas | 20:49 | |
johnsom | FYI, SSH driver bug: https://bugs.launchpad.net/octavia/+bug/1450630 | 20:56 |
openstack | Launchpad bug 1450630 in octavia "SSH driver doesn't update interfaces file correctly" [Critical,New] | 20:56 |
*** bharath has quit IRC | 21:04 | |
*** bharath has joined #openstack-lbaas | 21:05 | |
*** hitalia has quit IRC | 21:05 | |
openstackgerrit | Brandon Logan proposed stackforge/octavia: Implemented Queue Consumer https://review.openstack.org/149789 | 21:10 |
blogan | johnsom: ^ | 21:11 |
*** bharath has quit IRC | 21:11 | |
johnsom | I see it | 21:11 |
*** bharath has joined #openstack-lbaas | 21:11 | |
blogan | added the entry point in the setup.cfg too | 21:12 |
blogan | pbr doesn't like -'s in console_scripts | 21:13 |
johnsom | Cool, I have a fix for the lb update issue I will put up today. | 21:13 |
blogan | TrevorV: ssh driver bug above | 21:14 |
TrevorV | WUT?!?!? | 21:14 |
blogan | didn't mean to wake you | 21:14 |
blogan | well yeah i did | 21:14 |
*** bharath has quit IRC | 21:17 | |
*** hitalia has joined #openstack-lbaas | 21:17 | |
johnsom | blogan german has a question about the cmd directory | 21:27 |
blogan | shoot | 21:27 |
blogan | neutron and other projects started donig that | 21:28 |
johnsom | He put it on the queue consumer in gerrit | 21:28 |
johnsom | Yeah, I don't have the source to point him at | 21:28 |
TrevorV | johnsom I'll fix that ASAP, though it is a weird thing to happen. If you care to know, the "post_network_plug" was called before your interface was "DOWN" so the grep results were empty... Hence the issue you saw. | 21:32 |
johnsom | Ah, interesting. | 21:33 |
johnsom | Not something I'm doing wrong in the sequencing or something is it? | 21:33 |
TrevorV | Possibly. | 21:33 |
TrevorV | Not sure. | 21:33 |
TrevorV | If post_network_plug is called before the neutron interface attach is called, that could present that problem | 21:33 |
blogan | or if interface attach failed | 21:34 |
johnsom | GetPlumbedNetworks->CalculateDelta->PlugNetworks->AmphoraPostNetworkPlug->PlugVIP->.AmphoraPostVIPPlug | 21:35 |
johnsom | That is the sequence I have in the network subflow | 21:35 |
TrevorV | Yeah that looks correct, assuming the plug networks is completed successfully, as blogan pointed out | 21:35 |
blogan | ah i bet plug vip should go first before plug networks | 21:36 |
johnsom | I have them in nova and I see eth1 on the amp | 21:36 |
TrevorV | I was just discussing with blogan, so there should be two separate network subflows | 21:39 |
TrevorV | One like so: | 21:39 |
TrevorV | ->PlugVIP->.AmphoraPostVIPPlug | 21:39 |
TrevorV | wtf | 21:39 |
TrevorV | GetPlumbedNetworks->CalculateDelta->PlugNetworks->AmphoraPostNetworkPlug->PlugVIP->.AmphoraPostVIPPlug | 21:39 |
TrevorV | Damnit | 21:39 |
TrevorV | ha ha | 21:39 |
TrevorV | hold on | 21:39 |
TrevorV | First flow: | 21:39 |
TrevorV | GetPlumbedNetworks->CalculateDelta->PlugNetworks->AmphoraPostNetworkPlug | 21:40 |
TrevorV | Second flow: | 21:40 |
TrevorV | GetPlumbedNetworks->PlugVIP->.AmphoraPostVIPPlug | 21:40 |
blogan | so create lb should only do plug vip and post plug vip | 21:40 |
blogan | adding a member should do calculate delta, and then do plug network and post plug network | 21:41 |
johnsom | Why do they have to be separate? | 21:41 |
blogan | well plug vip should only happen on a create lb since we aren't allowing the ability to add multiple vips | 21:41 |
blogan | and the only time we'll ever need to plug a network is on a member addition | 21:42 |
johnsom | This is create lb flow | 21:42 |
blogan | yeah so create lb flow shouldn't need to do plug network bc the only network it will need to be plugged into is the vip network, which will be taken care of by the plug vip and post vip plug tasks | 21:43 |
johnsom | I don't think getplumbednetworks needs to be run before plugvip either. plugvip doesn't care about the networks | 21:44 |
johnsom | But it will need the listener network right? | 21:44 |
openstackgerrit | Brandon Logan proposed stackforge/octavia: Implemented Queue Consumer https://review.openstack.org/149789 | 21:47 |
blogan | listener network? | 21:48 |
blogan | there's just the vip network, and the member networks | 21:48 |
blogan | you mean listener as in octavia listener? or haproxy listener? | 21:48 |
johnsom | I agree, vip and member | 21:49 |
blogan | but yeah getplumbednetworks wouldn't need to be called either | 21:49 |
blogan | for the vip | 21:49 |
blogan | well the create lb flow | 21:49 |
johnsom | create lb should setup the vip network, right? | 21:50 |
xgerman | blogan - hi | 21:51 |
blogan | johnsom: yes | 21:52 |
blogan | xgerman: hi | 21:52 |
xgerman | yeah, I am cool with cmd (need to read up) but are we naming the commands then .py? | 21:52 |
xgerman | or like we started with bin without the .py suffix? | 21:52 |
johnsom | blogan When I am done with this lb update patch I will look at those network methods and see if my understanding of what each of them do is wrong | 21:52 |
dougwig | another neutron process change, that didn't hit in time for the octavia meeting. any existing octavia or lbaas core especially, take note: https://review.openstack.org/#/c/178846 | 21:53 |
johnsom | Are you tapping your fingers together and laughing dougwig? | 21:54 |
dougwig | i think i'm missing the reference? :) | 21:55 |
blogan | xgerman: bc pbr will automatically read the setup.cfg and it expects .py files to create the console_scripts | 21:55 |
xgerman | ok, got it | 21:55 |
*** bharath has joined #openstack-lbaas | 21:56 | |
xgerman | I have been in meetings the last three days — so my head hurts | 21:56 |
blogan | xgerman: my meetings have increased as well, thankfully i haven't been unfortunate enough to be in as many as you | 21:57 |
xgerman | well, three days all-day in a row is killing me... | 21:58 |
johnsom | xgerman Better you than me... Grin | 21:58 |
xgerman | (+ I need to live in the airport like dougwig) | 21:58 |
openstackgerrit | Merged stackforge/octavia: Implemented Queue Consumer https://review.openstack.org/149789 | 21:59 |
xgerman | johnsom Happy to take that bullet... | 21:59 |
johnsom | I have never been able to master the sleeping with eyes open thing... | 21:59 |
dougwig | johnsom: necessary skill for college. | 22:00 |
johnsom | I just sat in the back. The classes I didn't care about were large.... | 22:00 |
blogan | or even better, learn to pass the class without attending the class | 22:00 |
johnsom | blogan +1 | 22:01 |
blogan | always pissed me off when professors made attendance part of the grade | 22:01 |
rm_work | they were lazy and didn't want to test on everything they taught -- or else, they wanted to teach stuff you didn't actually need to know, and therefore wouldn't make sense to test on, but they REALLY wanted you to know it | 22:10 |
rm_work | mostly the latter is professors who just want to tell stories and force people to listen | 22:11 |
johnsom | Still wonder why I needed to know the name of the hormone that makes tobacoo plants grow until they die as an engineering student.... | 22:13 |
johnsom | True midterm question | 22:14 |
*** sbalukoff has quit IRC | 22:15 | |
dougwig | johnsom: do you still remember the name? | 22:15 |
johnsom | No, frankly I don't think I ever knew it | 22:15 |
johnsom | I did end up passing the class though... | 22:16 |
dougwig | johnsom: darn, we could've used it for our next naming argument. | 22:16 |
johnsom | Hahaha | 22:16 |
blogan | should have named amphora that, it woudl have been much more obvious | 22:21 |
rm_work | I hate all of you, but mostly blogan | 22:22 |
blogan | everyone mostly hates me | 22:22 |
blogan | my wife hated me so much she married me | 22:23 |
dougwig | we have to marry you if we hate you enough? oh my. | 22:26 |
*** chlong has joined #openstack-lbaas | 22:36 | |
blogan | doug logan, got a nice ring to it | 22:38 |
blogan | but you wouldn't get a nice rnig | 22:38 |
openstackgerrit | Michael Johnson proposed stackforge/octavia: Fixes load balancer create flow issue https://review.openstack.org/179277 | 22:44 |
*** clev is now known as clev-away | 22:48 | |
*** jorgem has joined #openstack-lbaas | 22:51 | |
*** jorgem has quit IRC | 22:58 | |
openstackgerrit | min wang proposed openstack/neutron-lbaas: Admin_state_up tempest test for health monitor https://review.openstack.org/178827 | 23:05 |
*** woodster_ has quit IRC | 23:10 | |
*** mlavalle has quit IRC | 23:11 | |
*** clev-away is now known as clev | 23:17 | |
openstackgerrit | Michael Johnson proposed stackforge/octavia: Fixes load balancer create flow issue https://review.openstack.org/179277 | 23:21 |
*** woodster_ has joined #openstack-lbaas | 23:22 | |
*** TrevorV|Home has joined #openstack-lbaas | 23:26 | |
*** rm_work is now known as rm_work|away | 23:27 | |
*** rm_work|away is now known as rm_work | 23:27 | |
*** rm_work is now known as rm_work|away | 23:29 | |
*** clev is now known as clev-away | 23:38 | |
*** TrevorV|Home has quit IRC | 23:43 | |
*** SumitNaiksatam has quit IRC | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!