*** xgerman has joined #openstack-lbaas | 00:03 | |
johnsom | Yeah, I see. I have no idea why we need to get the logger.... | 00:03 |
---|---|---|
johnsom | The test is a bogus test anyway. | 00:04 |
*** bharath has quit IRC | 00:07 | |
*** xgerman_ has joined #openstack-lbaas | 00:08 | |
*** xgerman has quit IRC | 00:08 | |
*** madhu_ak has quit IRC | 00:19 | |
*** SumitNaiksatam has joined #openstack-lbaas | 00:30 | |
*** SumitNaiksatam has quit IRC | 00:36 | |
*** SumitNaiksatam has joined #openstack-lbaas | 00:37 | |
dougwig | I'm on my phone, but you can use my +2 and rage merge. | 00:48 |
*** Varun_Lodaya has quit IRC | 00:55 | |
*** mwang2_ has quit IRC | 01:00 | |
rm_you | heh | 01:14 |
rm_you | johnsom: so are you saying "wontfix, don't need the logger bit"? | 01:15 |
rm_you | it just doesn't make sense | 01:15 |
johnsom | I asked xgerman the author of the spec to comment because I don't see the value. | 01:15 |
rm_you | but if that's the dealio, then fine i guess | 01:15 |
rm_you | but like | 01:15 |
rm_you | why bother setting the logger | 01:16 |
johnsom | Excatly | 01:16 |
rm_you | which is the whole point of that patch? :P | 01:16 |
rm_you | so then, it doesn't matter if this merges? | 01:16 |
rm_you | or, could change this patch to "remove logger functions"? :P | 01:16 |
rm_you | i need to go look at the interface spec | 01:17 |
rm_you | to see why that function is even there | 01:17 |
*** vivek-ebay has quit IRC | 01:22 | |
*** madhu_ak has joined #openstack-lbaas | 01:31 | |
*** xgerman_ has quit IRC | 01:32 | |
*** Youcef has quit IRC | 02:00 | |
rm_you | I just noticed there's a talk called "Autoscaling Openstack Natively with Heat, Ceilometer and LBaaS" and I don't know any of the names on it T_T | 02:30 |
rm_you | heh, there's also a talk titled "The Openstack Summit talk selection process is broken" | 02:32 |
blogan | and that talk got accepted | 02:36 |
blogan | ironic | 02:36 |
blogan | i should make a talk titled "If you see this talk at the summit, the talk selection process is broken" | 02:37 |
rm_you | heh | 02:46 |
*** haigang has joined #openstack-lbaas | 02:56 | |
*** haigang has quit IRC | 02:58 | |
*** vivek-ebay has joined #openstack-lbaas | 02:58 | |
*** haigang has joined #openstack-lbaas | 02:59 | |
*** vivek-ebay has joined #openstack-lbaas | 03:00 | |
*** woodster_ has quit IRC | 03:00 | |
*** Tiancheng has joined #openstack-lbaas | 03:02 | |
*** vivek-ebay has quit IRC | 03:09 | |
*** haigang has quit IRC | 03:33 | |
*** vivek-ebay has joined #openstack-lbaas | 03:58 | |
openstackgerrit | Aishwarya Thangappa proposed openstack/neutron-lbaas: Modified the tempest.config file https://review.openstack.org/172299 | 04:01 |
*** Varun_Lodaya has joined #openstack-lbaas | 04:08 | |
*** vivek-ebay has quit IRC | 04:11 | |
*** crc32 has joined #openstack-lbaas | 04:27 | |
*** vivek-ebay has joined #openstack-lbaas | 04:45 | |
*** vivek-ebay has quit IRC | 04:48 | |
*** woodster_ has joined #openstack-lbaas | 04:57 | |
*** sbfox has joined #openstack-lbaas | 05:01 | |
*** Varun_Lodaya has quit IRC | 05:05 | |
*** Varun_Lodaya has joined #openstack-lbaas | 05:07 | |
*** Varun_Lodaya has quit IRC | 05:18 | |
*** sbfox has quit IRC | 05:26 | |
*** crc32 has quit IRC | 05:33 | |
*** madhu_ak has quit IRC | 05:51 | |
*** haigang has joined #openstack-lbaas | 06:24 | |
*** mwang2_ has joined #openstack-lbaas | 06:56 | |
*** woodster_ has quit IRC | 07:00 | |
*** mwang2_ has quit IRC | 07:01 | |
*** apuimedo has joined #openstack-lbaas | 07:02 | |
*** SumitNaiksatam has left #openstack-lbaas | 08:01 | |
*** Miouge has joined #openstack-lbaas | 08:18 | |
*** Miouge has quit IRC | 09:00 | |
*** Miouge has joined #openstack-lbaas | 09:02 | |
*** Miouge has left #openstack-lbaas | 09:02 | |
*** kiran_ has joined #openstack-lbaas | 09:24 | |
*** kiran_ is now known as kiran-r | 09:24 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron-lbaas: Lazy load certificate manager plugin https://review.openstack.org/172357 | 09:44 |
*** amotoki has joined #openstack-lbaas | 09:51 | |
*** Tiancheng has quit IRC | 09:53 | |
*** amotoki has quit IRC | 10:18 | |
*** amotoki has joined #openstack-lbaas | 10:25 | |
*** haigang has quit IRC | 10:29 | |
*** amotoki_ has joined #openstack-lbaas | 10:31 | |
*** amotoki has quit IRC | 10:31 | |
*** amotoki has joined #openstack-lbaas | 10:31 | |
*** amotoki_ has quit IRC | 10:35 | |
*** Miouge has joined #openstack-lbaas | 11:04 | |
*** Miouge has quit IRC | 11:07 | |
*** amotoki_ has joined #openstack-lbaas | 11:58 | |
*** amotoki has quit IRC | 12:01 | |
*** amotoki_ has quit IRC | 12:03 | |
*** amotoki has joined #openstack-lbaas | 12:04 | |
*** amotoki has quit IRC | 12:08 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron-lbaas: Lazy load certificate manager plugin https://review.openstack.org/172357 | 12:15 |
*** rm_you| has joined #openstack-lbaas | 12:23 | |
*** rm_you has quit IRC | 12:25 | |
*** woodster_ has joined #openstack-lbaas | 12:37 | |
openstackgerrit | Trevor Vardeman proposed stackforge/octavia: Update PUT methods to send update through queue https://review.openstack.org/170989 | 13:32 |
*** john-davidge has joined #openstack-lbaas | 13:47 | |
*** xgerman_ has joined #openstack-lbaas | 13:54 | |
*** TrevorV|Home has joined #openstack-lbaas | 14:02 | |
*** mestery is now known as mestery_afk | 14:13 | |
*** kiran-r has quit IRC | 14:26 | |
*** Varun_Lodaya has joined #openstack-lbaas | 14:40 | |
*** vivek-ebay has joined #openstack-lbaas | 14:40 | |
*** vivek-ebay has quit IRC | 14:45 | |
*** vivek-ebay has joined #openstack-lbaas | 14:45 | |
*** ajmiller has joined #openstack-lbaas | 14:46 | |
*** mestery_afk is now known as mestery | 15:06 | |
*** jorgem has joined #openstack-lbaas | 15:36 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron-lbaas: Switch from neutron.common.log.log to oslo_log.helpers.log_method_call https://review.openstack.org/172484 | 15:38 |
*** mlavalle has joined #openstack-lbaas | 15:46 | |
*** enikanorov has quit IRC | 15:55 | |
*** jorgem has quit IRC | 16:02 | |
*** jorgem has joined #openstack-lbaas | 16:06 | |
*** madhu_ak has joined #openstack-lbaas | 16:13 | |
*** madhu_ak has quit IRC | 16:19 | |
*** amotoki has joined #openstack-lbaas | 16:23 | |
*** jorgem has quit IRC | 16:39 | |
openstackgerrit | Michael Johnson proposed stackforge/octavia: Removing a log parameter from the noop driver https://review.openstack.org/172237 | 16:41 |
johnsom | rm_work blogan I have updated https://review.openstack.org/#/c/172237/3 based on your comments, which I agree with.... | 16:42 |
johnsom | xgerman_ if you could have a look as well | 16:43 |
*** ajmiller_ has joined #openstack-lbaas | 16:44 | |
*** ajmiller has quit IRC | 16:46 | |
*** ajmiller_ is now known as ajmiller | 16:47 | |
*** madhu_ak has joined #openstack-lbaas | 16:49 | |
*** mwang2_ has joined #openstack-lbaas | 16:52 | |
xgerman_ | looking | 16:54 |
openstackgerrit | Merged stackforge/octavia: Removing a log parameter from the noop driver https://review.openstack.org/172237 | 17:01 |
*** john-davidge has quit IRC | 17:04 | |
*** vivek-ebay has quit IRC | 17:09 | |
*** jorgem has joined #openstack-lbaas | 17:10 | |
*** Miouge has joined #openstack-lbaas | 17:12 | |
*** mwang2_ has quit IRC | 17:14 | |
*** apuimedo has quit IRC | 17:21 | |
*** vivek-ebay has joined #openstack-lbaas | 17:23 | |
*** Varun_Lodaya has quit IRC | 17:26 | |
blogan | johnsom: looks good, beat me to the +A though | 17:29 |
johnsom | Yep, thanks! | 17:29 |
*** vivek-ebay has quit IRC | 17:30 | |
*** vivek-ebay has joined #openstack-lbaas | 17:42 | |
*** vivek-ebay has quit IRC | 17:42 | |
*** mestery has quit IRC | 17:43 | |
*** mestery has joined #openstack-lbaas | 17:43 | |
blogan | johnsom, xgerman_: hwo would yall feel about putting the keepalived element in the disk_imagecreate script? | 17:44 |
xgerman_ | probably ok | 17:45 |
johnsom | It needs to get there at some point. Personally I don't want to take that on right now as I am focused on controller/demo | 17:45 |
johnsom | So, go for it if you have the cycles | 17:46 |
blogan | johnsom: thats fine, i can add it in, ive already added it in, but need to make it more useful | 17:46 |
johnsom | Though finishing up the network driver would be cool <grin> | 17:46 |
blogan | well more customizeable | 17:46 |
xgerman_ | sure, sounds good in that case | 17:46 |
xgerman_ | we will review | 17:46 |
blogan | johnsom: lol yeah thats what im getting done right now | 17:46 |
johnsom | Excellent | 17:47 |
blogan | johnsom: actually ahd it done last night and then my computer froze and deleted the test file and i hadn't committed it yet | 17:47 |
johnsom | That sucks. Yeah, I am doing some strange things in the network task tests since I don't have any drivers to load and the stevedore code has to be commented out. | 17:48 |
johnsom | Min is going to hook us up with some noop drivers soon | 17:48 |
blogan | btw ive changed that bind_ip to vrrp_ip, ha_ip will be the ip that floats between the active passive pairs | 17:48 |
blogan | which the more traditional ways | 17:49 |
blogan | yeah the noop drivers will be good | 17:50 |
xgerman_ | yeah, naming is key for us not to get confused ;-) | 17:50 |
xgerman_ | also w elikely need to set up the haproxy in a way that the sahre the ssl sessions | 17:50 |
johnsom | Yeah, do we have the peers stuff in the template yet? | 17:50 |
blogan | xgerman_: well originally i was trying to get it more generic, but in this case i think we just have to do it | 17:51 |
*** SumitNaiksatam has joined #openstack-lbaas | 17:51 | |
xgerman_ | and I am not sure how well keepalived works in a neutron network - the system might start to flip-flop a lot | 17:52 |
blogan | i don't think so, we also need to change the bind call in the template for the active/passive so when the ha_ip doesn't exist, it can still pick it up | 17:52 |
blogan | xgerman_: it might but it'll be good enough for a demo | 17:52 |
xgerman_ | totally | 17:52 |
blogan | do yall know if there are any issues with haproxy binding to an ip with the transparent flag? | 17:53 |
xgerman_ | bedis? | 17:53 |
blogan | bedis doesn't like us anymore | 17:54 |
xgerman_ | what did you guys do? Told him that Texan frog legs are better then French? | 17:54 |
johnsom | lol | 17:54 |
blogan | i would never say that! | 17:54 |
blogan | i did tell him texas time was better than french time though, but its also better than west coast time | 17:55 |
blogan | im pretty sure that is a fact | 17:55 |
blogan | look it up | 17:55 |
xgerman_ | I don't think it's called Texas time BTW | 17:55 |
blogan | shhh | 17:56 |
blogan | dont tell the rest of the country | 17:56 |
blogan | or world | 17:56 |
*** jorgem has quit IRC | 17:57 | |
openstackgerrit | Michael Johnson proposed stackforge/octavia: Implements Octavia Controller Worker https://review.openstack.org/151496 | 17:58 |
*** bharath has joined #openstack-lbaas | 17:58 | |
*** amotoki has quit IRC | 18:02 | |
*** Varun_Lodaya has joined #openstack-lbaas | 18:08 | |
*** Miouge has quit IRC | 18:12 | |
*** jorgem has joined #openstack-lbaas | 18:14 | |
*** vivek-ebay has joined #openstack-lbaas | 18:28 | |
*** mlavalle has quit IRC | 18:33 | |
*** crc32 has joined #openstack-lbaas | 18:33 | |
rm_you| | cool, looks like the logger situation worked itself out :) | 18:37 |
*** jorgem has quit IRC | 18:48 | |
*** enikanorov_ has joined #openstack-lbaas | 18:59 | |
*** jorgem has joined #openstack-lbaas | 19:04 | |
*** mlavalle has joined #openstack-lbaas | 19:06 | |
*** crc32 has quit IRC | 19:13 | |
*** crc32 has joined #openstack-lbaas | 19:20 | |
*** madhu_ak has quit IRC | 19:28 | |
*** sbfox has joined #openstack-lbaas | 19:29 | |
*** raj_ has joined #openstack-lbaas | 19:32 | |
*** Miouge has joined #openstack-lbaas | 19:33 | |
*** Miouge has quit IRC | 19:35 | |
openstackgerrit | min wang proposed openstack/neutron-lbaas: Admin API tempest for healthmonitor https://review.openstack.org/171011 | 19:36 |
*** SumitNaiksatam has quit IRC | 19:38 | |
*** SumitNaiksatam has joined #openstack-lbaas | 19:38 | |
openstackgerrit | Brandon Logan proposed stackforge/octavia: Added neutron allowed address pairs network driver https://review.openstack.org/167441 | 19:42 |
openstackgerrit | min wang proposed openstack/neutron-lbaas: Admin API tempest for healthmonitor https://review.openstack.org/171011 | 19:42 |
*** rm_you| is now known as rm_you | 19:44 | |
openstackgerrit | min wang proposed openstack/neutron-lbaas: Admin API tempest for healthmonitor https://review.openstack.org/171011 | 19:44 |
*** crc32 has quit IRC | 19:47 | |
*** sbfox has quit IRC | 19:52 | |
*** sbfox has joined #openstack-lbaas | 20:06 | |
*** mlavalle has quit IRC | 20:07 | |
*** Aish has joined #openstack-lbaas | 20:15 | |
openstackgerrit | Brandon Logan proposed stackforge/octavia: Added post_vip_plug method to the amphora driver interface https://review.openstack.org/172556 | 20:15 |
openstackgerrit | Brandon Logan proposed stackforge/octavia: Added neutron allowed address pairs network driver https://review.openstack.org/167441 | 20:15 |
openstackgerrit | Brandon Logan proposed stackforge/octavia: Added vrrp_ip and ha_ip to amphora models https://review.openstack.org/172558 | 20:15 |
blogan | ^^ all of those ready for review | 20:17 |
xgerman_ | k | 20:17 |
rm_you | k | 20:22 |
blogan | whoops need to make one change to the network driver | 20:23 |
*** mestery has quit IRC | 20:24 | |
openstackgerrit | Brandon Logan proposed stackforge/octavia: Added neutron allowed address pairs network driver https://review.openstack.org/167441 | 20:28 |
openstackgerrit | Trevor Vardeman proposed stackforge/octavia: Update PUT methods to send update through queue https://review.openstack.org/170989 | 20:30 |
*** TrevorV|Home has quit IRC | 20:31 | |
blogan | johnsom do you run python 2.6? | 20:32 |
johnsom | No | 20:32 |
johnsom | 2.7 only | 20:32 |
blogan | when you run the diskiamge-create script you dont get complaints about argparse not installed? | 20:33 |
*** mestery has joined #openstack-lbaas | 20:33 | |
johnsom | That was a recent change (my pip list still shows it). Al posted a fix for that I think | 20:33 |
blogan | yeah its in his devstack plugin review | 20:34 |
johnsom | https://review.openstack.org/#/c/167796/12/diskimage-create/diskimage-create.sh | 20:34 |
johnsom | Yeah, not a good place, but known issue | 20:34 |
ajmiller | blogan johnsom would it be helpful to put that in a separate patch? | 20:34 |
blogan | wonder why it shows in your pip freeze | 20:34 |
johnsom | It changed somewhere between when I wrote the disk image stuff and onw | 20:35 |
blogan | ajmiller: its not a big deal to me as ive just been removing it, but if you feel like it | 20:35 |
johnsom | now | 20:35 |
johnsom | ubuntu@maindev:~$ pip freeze | 20:35 |
blogan | ajmiller: eh might as well do it bc that can get in quick and the devstack plugin is probably a ways out | 20:35 |
johnsom | argparse==1.2.1 | 20:35 |
xgerman_ | blogan, ajmiller +1 | 20:35 |
ajmiller | OK. I can do that pretty quickly. | 20:36 |
xgerman_ | actually I have my +2 ready ;-) | 20:36 |
ajmiller | LOL... | 20:36 |
johnsom | I could put it in the existing disk image patch up for review if you like | 20:36 |
blogan | my python has argparse 1.1 | 20:36 |
blogan | maybe you have an upgraded version? | 20:36 |
johnsom | I have no idea why it is in mine and not others. | 20:36 |
blogan | johnsom: yeah do that, and could you add another thing real quick | 20:36 |
ajmiller | johnsom, if you have a diskiamge patch going already, go ahead. | 20:36 |
johnsom | Ok, I will take it | 20:37 |
johnsom | What have you got blogan? | 20:37 |
blogan | johnsom: added a comment on the review, easier to post it there than here | 20:37 |
blogan | unless there are problems with that setting, which i wouldnt be surprised if there is | 20:38 |
rm_you | so, I am confused about the "create_health_monitor" and similar things | 20:39 |
rm_you | for create_amphora, it creates an amphora and returns an ID, like I would expect | 20:39 |
rm_you | but the rest of the creates in that controller_worker.py TAKE ids, and don't return what i'd expect | 20:39 |
johnsom | blogan I will do a quick bit of research on that setting and add it, NP | 20:40 |
rm_you | how/why are the "create" methods not actually creating the objects? how do they already exist in the DB? | 20:40 |
blogan | johnsom: thanks, this would allow not having to change the jinja configs bind template, but if this is a big deal we'll ahve to do that | 20:40 |
blogan | rm_you: once the controller worker gets an id to an object, it shoudl already exist in the db | 20:41 |
xgerman_ | yep | 20:41 |
blogan | the api will insert htem into the db | 20:41 |
johnsom | rm_you This is the set of questions I had at the weekly meeting. My understanding is that something upstream of the controller worker (api server, consumer worker, etc.) will put those in the DB and hand me the ID for reference | 20:41 |
rm_you | ok, interesting | 20:41 |
xgerman_ | and then the worker updates with SUCCESS or ERROR | 20:41 |
xgerman_ | in the DB | 20:41 |
rm_you | so it's not REALLY a create? more like a "middle_step_of_health_monitor_setup"? | 20:42 |
blogan | is ist a create in db method or create in the backend? | 20:42 |
xgerman_ | it "creates" it outside the DB | 20:42 |
blogan | depends on the context | 20:42 |
johnsom | It's a tell the amphora to go create things | 20:42 |
rm_you | hmm ok | 20:42 |
rm_you | yeah i guess that makes sense | 20:42 |
johnsom | I was right there with you on Wednesday.... | 20:42 |
rm_you | doesn't look like you responded to my question about fetch_all() being used in create_amphora | 20:43 |
blogan | i think the confusion yall have is assuming neutron_lbaas api is the frontend api, but assume octavia is standalone | 20:43 |
xgerman_ | don't confuse us blogan | 20:43 |
rm_you | can create_amphora return multiple results, or not?\ | 20:44 |
xgerman_ | neutron lbaas v2 will drive databases and trains | 20:44 |
blogan | what about datatrains? | 20:44 |
xgerman_ | rm_you that might just be from my early prototype where I simply used that | 20:44 |
xgerman_ | you are likely right | 20:44 |
xgerman_ | we should grab only the result we wanr | 20:44 |
johnsom | rm_you No, I will fix that. It was legacy code from the hack-a-thon | 20:44 |
rm_you | kk | 20:45 |
johnsom | It was a valid comment | 20:45 |
xgerman_ | +1 | 20:45 |
rm_you | and on ID placement, does anyone mind if we ... fix that? I guess would need to update the spec | 20:45 |
rm_you | but it's super awkward having ID not be the first arg, from my experience | 20:46 |
*** mlavalle has joined #openstack-lbaas | 20:46 | |
xgerman_ | ok, we should do that | 20:46 |
johnsom | You can update the spec, Jorge's code, and I will re-order controller worker. That is if we care enough | 20:46 |
rm_you | i care enough | 20:46 |
rm_you | I'll go do that now | 20:46 |
xgerman_ | ok, go for it then rm_you | 20:46 |
johnsom | Ok, just ping me in irc when you have a patchset up | 20:46 |
blogan | is this on the updates? | 20:47 |
blogan | that the id is not the first arg? | 20:47 |
blogan | rm_you: ^ | 20:47 |
rm_work | yeah | 20:48 |
rm_work | is that not odd to you? | 20:48 |
blogan | i told jorge to fix that the other day, he's been procrastinating | 20:48 |
rm_work | heh | 20:48 |
rm_work | I'll fix the spec, and tell jorge to get off his arse and fix his code :P | 20:48 |
blogan | i jsut did | 20:48 |
blogan | i told him it'd take him 10 secs | 20:48 |
*** fnaval has joined #openstack-lbaas | 20:48 | |
johnsom | blogan Does this nonlocal work for ipv6? https://gist.github.com/aw/1008793 | 20:49 |
blogan | johnsom: never tried that, but the comments definitely sound like it would be the same thing | 20:50 |
blogan | but for ipv6 | 20:50 |
johnsom | I REALLY want us to be able to use ipv6 addresses throughout Octavia | 20:50 |
blogan | me too, im pretty sure we will need to | 20:50 |
xgerman_ | +1 | 20:50 |
blogan | johnsom: if that doesn't work for ipv6, then we could build the template for haproxy to bind in transparent mode | 20:53 |
johnsom | this may have been why I always created a non-arp lo interface for the floating (vrrp floating) ip | 20:53 |
blogan | for ipv6? | 20:53 |
johnsom | Whatever address I passed between hosts. | 20:54 |
johnsom | v6 or v4 | 20:54 |
johnsom | I'm going to put in the v4 setting. When we get a chance to test ipv6 we can put a fix in then | 20:54 |
blogan | well just setting this one field will work for the demo, we can investigate the ipv6 afterwards | 20:55 |
blogan | yeah, thanks | 20:55 |
rm_work | johnsom: for those unused params like "health_monitor_updates" etc | 20:56 |
rm_work | johnsom: are those … dicts? | 20:57 |
rm_work | or are they DB objects? | 20:57 |
rm_work | or what? | 20:57 |
johnsom | rm_work that is what I am told. I would love to see the code/spec that builds them | 20:57 |
blogan | johnsom: http://marc.info/?l=linux-netdev&m=132369656811468&w=2 | 20:57 |
rm_work | heh yeah I was hoping someone would know so i didn't have to dig through | 20:57 |
blogan | not sure if that has merged | 20:57 |
johnsom | blogan I will look | 20:58 |
blogan | it was rejected :( | 21:00 |
johnsom | Yeah, it's not there | 21:00 |
*** madhu_ak has joined #openstack-lbaas | 21:02 | |
openstackgerrit | Michael Johnson proposed stackforge/octavia: Updating the disk image builder code https://review.openstack.org/170578 | 21:03 |
johnsom | blogan xgerman_ ^^^^ | 21:04 |
xgerman_ | yep | 21:04 |
openstackgerrit | Adam Harwell proposed stackforge/octavia: Fix arg order for Update functions in Controller-Worker https://review.openstack.org/172568 | 21:06 |
rm_work | ^^ need to confirm what the update args are for -- expecting to fix the docstrings and add a patchset once we agree for sure :P | 21:06 |
*** mestery has quit IRC | 21:07 | |
*** mestery has joined #openstack-lbaas | 21:08 | |
johnsom | rm_work do you know who is writing the code that will create/populate those updates fields? | 21:08 |
rm_work | nope | 21:08 |
rm_work | err, unless it's carlos? | 21:08 |
rm_work | would it be the queue producer? | 21:08 |
johnsom | It could be. blogan do you know? | 21:08 |
blogan | the code that is sending to the queue? | 21:09 |
blogan | then yeh sthat is carlos's review | 21:09 |
xgerman_ | queue producer | 21:09 |
blogan | https://review.openstack.org/#/c/149079/ | 21:09 |
xgerman_ | though it confused me since he was referring an API | 21:10 |
xgerman_ | but I guess I should +2 it now as we know it's use | 21:10 |
blogan | well teh api will call this cdoe | 21:10 |
blogan | and the code will send to the queue | 21:10 |
blogan | xgerman_: hold off on the +2 | 21:10 |
xgerman_ | too late | 21:11 |
blogan | well we may have to make another review then | 21:11 |
xgerman_ | yep | 21:11 |
xgerman_ | also let's refer to the thing claling it as lbaas v2 octavia driver | 21:11 |
xgerman_ | api makes me think octavia api | 21:12 |
blogan | it is octavia api! | 21:12 |
rm_work | honestly, we may want to do that with a few of the larger reviews up now -- just get them merged and do some follow-up patches | 21:12 |
blogan | neutron-lbaas's octavia driver will call the octavia api for every operation | 21:12 |
xgerman_ | huh? we can't make it call our queue? | 21:13 |
xgerman_ | that feels cleaner to me... | 21:13 |
blogan | i thought we've always viewed octavia as workign standalone from neutron-lbaas | 21:13 |
openstackgerrit | German Eichberger proposed stackforge/octavia: Implements the haproxy amphora agent api server https://review.openstack.org/160034 | 21:13 |
xgerman_ | ok, that's were our views differ | 21:13 |
*** sbfox has quit IRC | 21:14 | |
xgerman_ | I thought separating with the queue would be good enough | 21:14 |
xgerman_ | so octavia is stand alone for everybody who posts straight in a queue | 21:14 |
rm_work | heh | 21:16 |
rm_work | I really want Octavia to be COMPLETELY standalone | 21:16 |
rm_work | always have | 21:16 |
blogan | yeah but the receiving side of the queue assumes it can't get stacked updates, so it requires an api to not allow multiple updates until the first update finishes | 21:16 |
rm_work | personally I find the neutron-lbaas frontend a bit T_T but I live with it | 21:16 |
johnsom | I like the idea that it has the option of being stand alone | 21:16 |
openstackgerrit | Merged stackforge/octavia: Implementation for bp/api-to-oslo-messing-handler https://review.openstack.org/149079 | 21:16 |
rm_work | yeah, and since the rest of the neutron-lbaas drivers work is to call an API, it stays in-line with that | 21:17 |
blogan | it being standalone is more in line with it being another vendor really | 21:17 |
rm_work | ^^ jinx | 21:17 |
xgerman_ | ok, I have a different opinion but that should not stop anybody... | 21:19 |
xgerman_ | so carry on | 21:19 |
rm_work | xgerman_: well, "our opinion" leads to it being the most permissive/flexible, so yeah | 21:19 |
rm_work | I think it shouldn't make it WORSE to have additional flexibility :P | 21:20 |
* rm_work knocks on wood | 21:20 | |
blogan | its just another layer to go through | 21:20 |
xgerman_ | yep, but layers which don't exist don't require maintenanc, don't have bugs, ... | 21:21 |
xgerman_ | but as I said I am ok with it... | 21:21 |
blogan | if we relied on teh neutron-lbaas api being the frontend, then the octavia controller worker would need hooks int o the neutron db, which to me is a bad idea to require access to the neutron db | 21:21 |
xgerman_ | we can't just put stuff on the queue on work on them in order? | 21:22 |
xgerman_ | or collapse them if needed | 21:22 |
xgerman_ | (like rug does) | 21:22 |
*** mestery_ has joined #openstack-lbaas | 21:22 | |
blogan | order may not be deterministic, so we'd have to deal wiht all of that | 21:22 |
blogan | which inthe grand scheme of things, a lbaas api is not going to be a high request api so i dont think it is worth that trouble | 21:23 |
blogan | well it may be high request for reads, but posts, puts, and deletes, i dont think so | 21:23 |
blogan | well i know so from our current CLB offering | 21:24 |
xgerman_ | agreed -- we have an offering as well :-) | 21:24 |
*** mestery has quit IRC | 21:25 | |
*** mestery_ is now known as mestery | 21:26 | |
openstackgerrit | Michael Johnson proposed stackforge/octavia: Implements Octavia Controller Worker https://review.openstack.org/151496 | 21:36 |
blogan | xgerman: would it be better if that docstring said it returned octavia.common.data_models.LoadBalancer? | 21:36 |
blogan | xgerman_: ^ | 21:36 |
xgerman_ | yep | 21:37 |
blogan | xgerman_: because i was more implying it was a class name, not spelling it | 21:37 |
blogan | but the description I think should still have Load balancer, since it is two words | 21:37 |
xgerman_ | ok | 21:37 |
xgerman_ | yeah, just confused me | 21:39 |
openstackgerrit | Merged stackforge/octavia: Updating the disk image builder code https://review.openstack.org/170578 | 21:40 |
openstackgerrit | Madhusudhan Kandadai proposed openstack/neutron-lbaas: Introduced admin_api tempest tests https://review.openstack.org/171832 | 21:44 |
openstackgerrit | Brandon Logan proposed stackforge/octavia: Added post_vip_plug method to the amphora driver interface https://review.openstack.org/172556 | 21:46 |
openstackgerrit | Merged openstack/neutron-lbaas: Add HaproxyNSDriver to lbaas entry points https://review.openstack.org/171127 | 21:46 |
blogan | xgerman: i have a text file that i've been using to test the network driver, amphora driver, and compute driver out against devstack to make sure its doing everything correctly | 21:47 |
blogan | xgerman_: i'm gong to wikify it soon | 21:47 |
*** bharath has quit IRC | 21:47 | |
*** bharath has joined #openstack-lbaas | 21:47 | |
xgerman_ | awesome | 21:47 |
johnsom | Cool, I just built a devstack this week to try Octavia out on. I have been testing against HP cloud so far | 21:48 |
xgerman_ | neat | 21:50 |
*** bharath has quit IRC | 21:53 | |
*** bharath has joined #openstack-lbaas | 21:53 | |
*** jorgem has quit IRC | 22:11 | |
*** bharath has quit IRC | 22:25 | |
*** bharath has joined #openstack-lbaas | 22:25 | |
*** raj_ has quit IRC | 22:48 | |
openstackgerrit | Al Miller proposed stackforge/octavia: Add devstack plugin for octavia https://review.openstack.org/167796 | 22:56 |
*** ajmiller has quit IRC | 23:34 | |
*** xgerman_ has quit IRC | 23:35 | |
openstackgerrit | min wang proposed stackforge/octavia: Add network noop driver https://review.openstack.org/172601 | 23:35 |
*** mlavalle has quit IRC | 23:48 | |
*** sbfox has joined #openstack-lbaas | 23:53 | |
openstackgerrit | min wang proposed stackforge/octavia: Add network noop driver https://review.openstack.org/172601 | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!