| *** yamamoto_ has quit IRC | 00:03 | |
| *** rm_work has quit IRC | 00:04 | |
| ptoohill | yay | 00:08 |
|---|---|---|
| johnsom | Fixed | 00:09 |
| ptoohill | The tests? I was about to look into it, dealing with a fire | 00:09 |
| johnsom | no, the bug issue | 00:09 |
| ptoohill | oh | 00:09 |
| ptoohill | :P | 00:09 |
| *** rm_work has joined #openstack-lbaas | 00:09 | |
| johnsom | ptoohill It sounds like tempest is broken due to oslo_config 3.6.0 | 00:11 |
| ptoohill | :/ | 00:11 |
| ptoohill | Awesome | 00:11 |
| *** armax has quit IRC | 00:17 | |
| *** ducttape_ has joined #openstack-lbaas | 00:18 | |
| johnsom | FYI, this is the bug for the gate breakage: https://bugs.launchpad.net/devstack/+bug/1543841 | 00:20 |
| openstack | Launchpad bug 1543841 in cliff "Invocation error: tempest verify-config: error: unrecognized arguments: -uro ../../../../tmp/tmp.FR5OHabb3Y" [Undecided,New] | 00:20 |
| johnsom | And proposed patch: | 00:21 |
| johnsom | https://review.openstack.org/#/c/278151/ | 00:21 |
| xgerman | blogan the NotImplementedError your proposed gets swallowed in _call_driver_operation | 00:47 |
| *** ducttape_ has quit IRC | 00:49 | |
| *** ducttape_ has joined #openstack-lbaas | 00:51 | |
| *** ducttape_ has quit IRC | 00:55 | |
| *** SumitNaiksatam has quit IRC | 00:56 | |
| *** ducttape_ has joined #openstack-lbaas | 01:01 | |
| *** armax has joined #openstack-lbaas | 01:02 | |
| *** minwang2 has quit IRC | 01:06 | |
| openstackgerrit | German Eichberger proposed openstack/neutron-lbaas: [WIP] Adds Cascade option for load balancer drivers https://review.openstack.org/268237 | 01:10 |
| *** ducttape_ has quit IRC | 01:11 | |
| *** Varun_Lodaya has quit IRC | 01:15 | |
| *** ducttape_ has joined #openstack-lbaas | 01:17 | |
| *** ducttape_ has quit IRC | 01:30 | |
| *** _cjones_ has quit IRC | 01:36 | |
| *** Aish has quit IRC | 01:37 | |
| *** shakamunyi has quit IRC | 01:40 | |
| *** madhu_ak has quit IRC | 01:45 | |
| *** bana_k has quit IRC | 01:46 | |
| *** piet has joined #openstack-lbaas | 01:50 | |
| *** piet has quit IRC | 02:05 | |
| *** jwarendt has quit IRC | 02:05 | |
| *** SumitNaiksatam has joined #openstack-lbaas | 02:09 | |
| *** ducttape_ has joined #openstack-lbaas | 02:12 | |
| *** yamamoto has joined #openstack-lbaas | 02:12 | |
| *** armax has quit IRC | 02:18 | |
| *** rooney has joined #openstack-lbaas | 02:28 | |
| *** yamamoto has quit IRC | 02:31 | |
| openstackgerrit | Reedip proposed openstack/neutron-lbaas: Add Timeout on Listeners- DO NOT REVIEW https://review.openstack.org/273896 | 02:32 |
| openstackgerrit | Reedip proposed openstack/neutron-lbaas: Add Timeout on Listeners https://review.openstack.org/273896 | 03:00 |
| *** links has joined #openstack-lbaas | 03:00 | |
| *** logan-- has joined #openstack-lbaas | 03:10 | |
| *** logan-- has quit IRC | 03:14 | |
| blogan | xgerman: modify that method to catch and reraise | 03:16 |
| *** logan- has quit IRC | 03:22 | |
| *** logan- has joined #openstack-lbaas | 03:23 | |
| *** yamamoto has joined #openstack-lbaas | 03:29 | |
| *** woodster_ has joined #openstack-lbaas | 03:32 | |
| *** rooney has quit IRC | 03:39 | |
| *** rooney has joined #openstack-lbaas | 03:42 | |
| *** ducttape_ has quit IRC | 04:02 | |
| openstackgerrit | Merged openstack/octavia: Allow user-data on amphora creation https://review.openstack.org/275558 | 04:02 |
| *** ducttape_ has joined #openstack-lbaas | 04:03 | |
| *** ducttape_ has quit IRC | 04:08 | |
| *** rooney has quit IRC | 04:10 | |
| xgerman | I figured you would say something like that… | 04:10 |
| *** amotoki has joined #openstack-lbaas | 04:17 | |
| *** amotoki has quit IRC | 04:17 | |
| *** ducttape_ has joined #openstack-lbaas | 04:21 | |
| *** Bjoern has joined #openstack-lbaas | 04:22 | |
| *** rooney has joined #openstack-lbaas | 04:24 | |
| *** ducttape_ has quit IRC | 04:31 | |
| *** rooney has quit IRC | 04:33 | |
| *** johnsom_ has joined #openstack-lbaas | 04:44 | |
| *** fnaval has quit IRC | 04:54 | |
| xgerman | http://blog.episodicgenius.com/post/neutron-address-scopes/ somebody ought to test if that works with our stuff | 05:01 |
| *** piet has joined #openstack-lbaas | 05:13 | |
| *** fnaval has joined #openstack-lbaas | 05:13 | |
| *** rooney has joined #openstack-lbaas | 05:15 | |
| *** bana_k has joined #openstack-lbaas | 05:21 | |
| *** Bjoern has quit IRC | 05:21 | |
| *** armax has joined #openstack-lbaas | 05:31 | |
| *** Bjoern has joined #openstack-lbaas | 05:31 | |
| *** woodster_ has quit IRC | 05:36 | |
| *** ianbrown_ has quit IRC | 05:44 | |
| *** ianbrown_ has joined #openstack-lbaas | 05:45 | |
| *** allan_h has joined #openstack-lbaas | 05:47 | |
| johnsom_ | So confused... How did this timeout after 50 seconds when the code is hard coded for 600... http://logs.openstack.org/08/272708/6/check/gate-neutron-lbaasv2-dsvm-scenario/759d000/console.html#_2016-02-09_22_19_23_055 | 05:48 |
| *** bana_k has quit IRC | 05:51 | |
| *** piet has quit IRC | 05:55 | |
| *** johnsom_ has quit IRC | 05:58 | |
| *** prabampm has joined #openstack-lbaas | 06:21 | |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: WIP: Add L7 jinja template updates https://review.openstack.org/278223 | 06:26 |
| *** Bjoern has quit IRC | 06:36 | |
| *** rooney has quit IRC | 06:38 | |
| *** bana_k has joined #openstack-lbaas | 06:42 | |
| *** armax has quit IRC | 06:48 | |
| *** ianbrown_ has quit IRC | 06:52 | |
| *** ianbrown_ has joined #openstack-lbaas | 06:53 | |
| *** cgross- has joined #openstack-lbaas | 07:14 | |
| *** rcernin has joined #openstack-lbaas | 07:14 | |
| *** cgross- is now known as lmiccini | 07:20 | |
| *** nmagnezi has joined #openstack-lbaas | 07:36 | |
| *** openstackgerrit has quit IRC | 07:47 | |
| *** openstackgerrit has joined #openstack-lbaas | 07:48 | |
| *** rcernin has quit IRC | 07:52 | |
| *** rcernin has joined #openstack-lbaas | 08:15 | |
| *** SumitNaiksatam has quit IRC | 08:19 | |
| *** prabampm has quit IRC | 08:52 | |
| *** prabampm has joined #openstack-lbaas | 08:55 | |
| *** reedip is now known as outofmemory | 08:57 | |
| *** numans has joined #openstack-lbaas | 09:02 | |
| *** kobis has joined #openstack-lbaas | 09:07 | |
| *** bana_k has quit IRC | 09:32 | |
| *** links has quit IRC | 11:56 | |
| *** yamamoto has quit IRC | 12:20 | |
| *** rtheis has joined #openstack-lbaas | 12:24 | |
| *** links has joined #openstack-lbaas | 12:36 | |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Add L7 jinja template updates https://review.openstack.org/278223 | 13:05 |
| *** ducttape_ has joined #openstack-lbaas | 13:06 | |
| *** ducttape_ has quit IRC | 13:21 | |
| *** yamamoto has joined #openstack-lbaas | 13:29 | |
| *** piet has joined #openstack-lbaas | 13:53 | |
| *** ducttape_ has joined #openstack-lbaas | 14:08 | |
| *** piet has quit IRC | 14:09 | |
| *** piet has joined #openstack-lbaas | 14:09 | |
| *** neelashah has joined #openstack-lbaas | 14:17 | |
| *** ducttape_ has quit IRC | 14:20 | |
| *** Bjoern has joined #openstack-lbaas | 14:25 | |
| *** Bjoern is now known as Bjoern_zZzZzZzZ | 14:26 | |
| *** links has quit IRC | 14:27 | |
| *** Bjoern_zZzZzZzZ is now known as Bjoern | 14:36 | |
| *** Bjoern is now known as Bjoern_zZzZzZzZ | 14:37 | |
| *** Bjoern_zZzZzZzZ is now known as Bjoern | 14:38 | |
| *** localloop127 has joined #openstack-lbaas | 14:52 | |
| *** Bjoern has quit IRC | 14:53 | |
| *** ducttape_ has joined #openstack-lbaas | 14:58 | |
| *** nmagnezi has quit IRC | 15:11 | |
| openstackgerrit | Trevor Vardeman proposed openstack/octavia: Get Me A Load Balancer API https://review.openstack.org/256974 | 15:20 |
| *** Bjoern has joined #openstack-lbaas | 15:22 | |
| *** woodster_ has joined #openstack-lbaas | 15:23 | |
| *** TrevorV has joined #openstack-lbaas | 15:23 | |
| *** rcernin has quit IRC | 15:33 | |
| *** yamamoto has quit IRC | 15:35 | |
| *** piet has quit IRC | 15:46 | |
| *** numans has quit IRC | 15:51 | |
| *** yamamoto has joined #openstack-lbaas | 15:54 | |
| *** Aish has joined #openstack-lbaas | 15:55 | |
| *** piet has joined #openstack-lbaas | 15:56 | |
| *** rcernin has joined #openstack-lbaas | 16:01 | |
| *** fnaval has quit IRC | 16:04 | |
| *** fnaval has joined #openstack-lbaas | 16:05 | |
| *** lmiccini is now known as lmiccini|gone | 16:07 | |
| *** fnaval has quit IRC | 16:09 | |
| *** kobis has quit IRC | 16:21 | |
| *** rcernin has quit IRC | 16:24 | |
| *** fnaval has joined #openstack-lbaas | 16:26 | |
| xgerman | blogan more plugin questions: | 16:27 |
| xgerman | neutron.common.exceptions.ExtensionsNotFound: Extensions not found: ['n-lbaasv2-cascade-delete']. | 16:27 |
| blogan | does your extension have that alias? | 16:27 |
| xgerman | yes | 16:27 |
| xgerman | https://review.openstack.org/#/c/268237/5/neutron_lbaas/extensions/cascade_delete.py | 16:28 |
| xgerman | is there some place I need to register that extension | 16:28 |
| xgerman | ? | 16:28 |
| blogan | oh whoops, change the extension class name to Cascade_delete | 16:28 |
| *** longstaff has joined #openstack-lbaas | 16:29 | |
| xgerman | testing... | 16:30 |
| xgerman | I guess that mechanism wa invented by the man “who shall not be named" | 16:31 |
| xgerman | https://en.wikipedia.org/wiki/Lord_Voldemort#/media/File:Lordvoldemort.jpg | 16:34 |
| blogan | indeed | 16:39 |
| blogan | xgerman: i'd also suggest renaming the extension module to loadbalancer_cascade_delete because when someone does a neutron ext-list, cascade_delete is not clear what it is for | 16:40 |
| blogan | which means you'd have to change the class name to Loadbalancer_cascade_delete | 16:40 |
| *** doug-fish has quit IRC | 16:40 | |
| xgerman | yep, understood | 16:40 |
| *** doug-fish has joined #openstack-lbaas | 16:41 | |
| *** doug-fish has quit IRC | 16:45 | |
| *** kobis has joined #openstack-lbaas | 16:47 | |
| openstackgerrit | German Eichberger proposed openstack/neutron-lbaas: [WIP] Adds Cascade option for load balancer drivers https://review.openstack.org/268237 | 16:49 |
| *** doug-fish has joined #openstack-lbaas | 16:51 | |
| xgerman | remade in blogan’s image ;-) | 16:53 |
| blogan | xgerman: if that's true, it'll be a colossal failure | 16:54 |
| xgerman | I will let dougwig decide ;-) | 16:55 |
| *** doug-fish has quit IRC | 16:57 | |
| *** _cjones_ has joined #openstack-lbaas | 16:58 | |
| *** jwarendt has joined #openstack-lbaas | 17:04 | |
| *** blogan_ has joined #openstack-lbaas | 17:05 | |
| *** blogan_ has quit IRC | 17:09 | |
| *** armax has joined #openstack-lbaas | 17:09 | |
| rm_work | it'd be LoadbalancerCascadeDelete if it's a class, right? | 17:13 |
| rm_work | or do neutron-extensions force us to badly name classes | 17:13 |
| *** amotoki has joined #openstack-lbaas | 17:16 | |
| *** amotoki has quit IRC | 17:16 | |
| *** _cjones_ has quit IRC | 17:23 | |
| *** _cjones_ has joined #openstack-lbaas | 17:27 | |
| *** doug-fish has joined #openstack-lbaas | 17:29 | |
| blogan | rm_work: yes, its forced | 17:29 |
| blogan | rm_work: and awful, but its how it is | 17:29 |
| ptoohill | And thats why we cant have nice things, that whole it is what it is mantra | 17:33 |
| ptoohill | Fix it all, now, blogan | 17:33 |
| blogan | pretty sure i could make the loading logic also look for correct class names | 17:35 |
| ptoohill | do all the things | 17:36 |
| blogan | but its a beautiful wart | 17:36 |
| *** piet has quit IRC | 17:36 | |
| ptoohill | warts arent beautiful, i had a withc help me get rid of mine | 17:36 |
| ptoohill | o.o | 17:36 |
| blogan | and thats how you met jenn eh? | 17:37 |
| ptoohill | lol | 17:37 |
| blogan | sick burn son | 17:37 |
| ptoohill | lol | 17:37 |
| ptoohill | im telling | 17:37 |
| blogan | thats fine, she calls me worse | 17:37 |
| ptoohill | lol | 17:38 |
| *** ducttape_ has quit IRC | 17:39 | |
| *** ducttape_ has joined #openstack-lbaas | 17:39 | |
| *** madhu_ak has joined #openstack-lbaas | 17:46 | |
| *** madhu_ak_ has joined #openstack-lbaas | 17:46 | |
| *** rcernin has joined #openstack-lbaas | 17:52 | |
| openstackgerrit | Merged openstack/neutron-lbaas: Clean up removed hacking rule from [flake8] ignore lists https://review.openstack.org/267404 | 17:58 |
| *** localloop127 has quit IRC | 18:00 | |
| *** armax has quit IRC | 18:01 | |
| *** Aish has quit IRC | 18:02 | |
| *** numans has joined #openstack-lbaas | 18:06 | |
| fnaval | hi http://docs.openstack.org/developer/octavia/api/octaviaapi.html | 18:07 |
| fnaval | im running into some issues with creating the tests | 18:07 |
| fnaval | apparently, the octavia API does not require the root tag in the body of requests that are made | 18:08 |
| fnaval | this is inconsistent with how it's done in neutron_lbaas where we explicitly send in the root tag of a request body | 18:09 |
| fnaval | could we do the same in octavia? | 18:09 |
| fnaval | in fact, I think all of the core openstack services have the root tag in the request and response bodies | 18:09 |
| ptoohill | neutron_lbaas/neutron also doesnt follow some general 'rules'. Its defined somewhere that the URI defines the resource | 18:09 |
| *** Bjoern is now known as Bjoern_zZzZzZzZ | 18:09 | |
| fnaval | i know for compute and glance, the root tag is needed | 18:10 |
| ptoohill | We can keep going down the wrong road, or start to make it better | 18:10 |
| ptoohill | We planned on making our two apis more consistent with it leaning towards octavia | 18:11 |
| fnaval | hmm.. this might block testing; i'll see if I can hack around it. but I'll file a bug then | 18:12 |
| ptoohill | And it looks like its even part of the api-wg | 18:12 |
| ptoohill | Still reading, but: https://specs.openstack.org/openstack/api-wg/guidelines/tags.html | 18:12 |
| ptoohill | Is supposed to be the rules were all supposed to be working towards | 18:12 |
| ptoohill | linked 'tags' but can back up | 18:13 |
| *** Aish has joined #openstack-lbaas | 18:13 | |
| ptoohill | hrmm | 18:14 |
| ptoohill | the other page shows something a bit different | 18:14 |
| ptoohill | so i guess we do want tags, which goes against rfc a bit | 18:14 |
| ptoohill | https://specs.openstack.org/openstack/api-wg/guidelines/representation_structure.html | 18:14 |
| ptoohill | well | 18:14 |
| ptoohill | yea, i guess so. Well fnaval there is a task out there somewhere to make apis more aligned. And well end up following the 'rules' | 18:15 |
| ptoohill | Though I disagree with this a bit, it is what it is? | 18:15 |
| *** Aish has quit IRC | 18:15 | |
| *** Aish has joined #openstack-lbaas | 18:16 | |
| fnaval | hmm.. yeah, it's kind blocking my testing with tempest at the moment | 18:16 |
| ptoohill | well yea, that probably wont be done for a while | 18:17 |
| ptoohill | why are you concerned with neutron-lbaas if youre testing octavia? | 18:17 |
| xgerman | yeah, how can you be blocked? | 18:17 |
| *** kobis has quit IRC | 18:17 | |
| ptoohill | You should be testing that api as it is, and well have to update it when we get to that point | 18:17 |
| *** bana_k has joined #openstack-lbaas | 18:18 | |
| xgerman | +1 | 18:18 |
| fnaval | i was hoping to use the same tempest client structure as in all the other projects | 18:18 |
| fnaval | but I'll have to hack around this one apparently | 18:19 |
| ptoohill | :/ | 18:19 |
| ptoohill | can thank blogan for this | 18:19 |
| ptoohill | :) | 18:19 |
| *** yamamoto has quit IRC | 18:19 | |
| fnaval | it's cool. either way, I can probably push through without it | 18:22 |
| *** Bjoern_zZzZzZzZ is now known as Bjoern | 18:22 | |
| fnaval | #bug 1544214 | 18:23 |
| openstack | bug 1544214 in octavia "Octavia: Root tags should be required for request and response bodies" [Undecided,New] https://launchpad.net/bugs/1544214 | 18:23 |
| fnaval | bbiab | 18:23 |
| *** armax has joined #openstack-lbaas | 18:44 | |
| *** neelashah has quit IRC | 18:50 | |
| *** yamamoto has joined #openstack-lbaas | 18:50 | |
| *** localloop127 has joined #openstack-lbaas | 18:56 | |
| *** localloop127 has quit IRC | 19:00 | |
| *** crc32 has joined #openstack-lbaas | 19:01 | |
| *** doug-fis_ has joined #openstack-lbaas | 19:02 | |
| *** doug-fi__ has joined #openstack-lbaas | 19:04 | |
| *** doug-fish has quit IRC | 19:05 | |
| *** doug-fis_ has quit IRC | 19:06 | |
| *** doug-fi__ has quit IRC | 19:09 | |
| *** neelashah has joined #openstack-lbaas | 19:10 | |
| *** barclaac has left #openstack-lbaas | 19:22 | |
| *** neelashah1 has joined #openstack-lbaas | 19:34 | |
| *** neelashah has quit IRC | 19:35 | |
| *** Bjoern has quit IRC | 19:38 | |
| TrevorV | Did I forget something somewhere in my assumption that when I do a query to retrieve a LB it would have all its children sorted out? | 19:42 |
| TrevorV | For example, if the LB I wanted to "GET" had listeners, the query to get an LB should populate its listeners right? | 19:42 |
| rm_work | err | 19:42 |
| rm_work | do you mean via the API | 19:42 |
| rm_work | or like | 19:42 |
| rm_work | internally | 19:42 |
| rm_work | internally I think that happens | 19:42 |
| TrevorV | I'm in the repositories file? have I confused myself? | 19:42 |
| rm_work | because of the SQLAlchemy model magic | 19:42 |
| sbalukoff | The API doesn't show that because you can get it from a known URL. | 19:42 |
| rm_work | but yeah the API doesn't | 19:43 |
| rm_work | but yes, you mean internally | 19:43 |
| rm_work | so yeah it should | 19:43 |
| TrevorV | yeah, well it doesn't. | 19:43 |
| sbalukoff | Really? | 19:43 |
| TrevorV | Not as far as I can see right now at my debug point.... | 19:43 |
| *** doug-fish has joined #openstack-lbaas | 19:43 | |
| TrevorV | Both of the objects I'm looking for exist, but neither has a reference to the other in their object structure... | 19:44 |
| rm_work | are you actually trying to access them | 19:44 |
| rm_work | or just look | 19:44 |
| rm_work | because the pycharm debugger may not SHOW those | 19:44 |
| rm_work | until you try to use them | 19:44 |
| rm_work | it's kinda magic | 19:44 |
| TrevorV | When running the functional test it throws a "none object has no 'id' field" type of error, and whilst in the code, it shows that its "none" | 19:45 |
| TrevorV | However, it "shouldn't" be none. | 19:45 |
| rm_work | hmmm | 19:45 |
| rm_work | so you do like | 19:45 |
| rm_work | lb = lb_repoget_loadbalancer() | 19:45 |
| rm_work | or whatever | 19:45 |
| rm_work | and then lb.listener.id ? | 19:45 |
| rm_work | i think it might be lb.listeners[0].id | 19:46 |
| rm_work | or something | 19:46 |
| rm_work | but yeah that should work | 19:46 |
| rm_work | i am not sure exactly how you're accessing it | 19:46 |
| TrevorV | db_pool.load_balancers | 19:46 |
| TrevorV | I just made that as a "watch" in pycharm | 19:46 |
| rm_work | err k hold on | 19:46 |
| rm_work | do you have a review you can link me | 19:46 |
| sbalukoff | db_pool.load_balancer | 19:46 |
| rm_work | and tell me what file/line | 19:46 |
| rm_work | ah | 19:47 |
| rm_work | yeah | 19:47 |
| rm_work | only one load_balancer on a pool | 19:47 |
| rm_work | which is ... ugh i still hate that | 19:47 |
| TrevorV | Yeah, so that didn't throw an error, but it still says "NoneType" | 19:47 |
| sbalukoff | Huh. | 19:47 |
| TrevorV | Yeah. | 19:47 |
| TrevorV | I'm a little lost at this point. | 19:47 |
| rm_work | review + file + line | 19:47 |
| rm_work | please? | 19:47 |
| sbalukoff | What does the database look like? | 19:47 |
| rm_work | so I can have some context | 19:47 |
| TrevorV | rm_work I don't have the changes I'm looking at UP in a review | 19:47 |
| TrevorV | ... You'd definitely have context if you were sitting next to me right now.... :P | 19:48 |
| sbalukoff | Heh! | 19:48 |
| TrevorV | *hint hint nudge nudge* | 19:48 |
| TrevorV | I'm still on brandon's single-create API review, for the base context. | 19:48 |
| TrevorV | I can push up a revision, but I don't even have a solid IDEA down yet, I'm still sorting out rebasing... | 19:48 |
| TrevorV | Which is a nightmare in and of itself | 19:49 |
| sbalukoff | Also, did this go through a to_dict() at some point? Because one of those transformations doesn't follow sqlalchemy links. | 19:49 |
| sbalukoff | Let me see which that is... | 19:49 |
| TrevorV | Not balukoff's fault, but balukoff's fault, ya know? | 19:49 |
| TrevorV | :P | 19:49 |
| rm_work | ok can you gist the function at least | 19:49 |
| sbalukoff | I'm a jerk! | 19:49 |
| TrevorV | Nah, I'm strictly in the repositories class right now | 19:49 |
| *** kobis has joined #openstack-lbaas | 19:49 | |
| fnaval | question: is the lbaasv2 extension required to run octavia? | 19:49 |
| TrevorV | https://gist.github.com/anonymous/c5122644bb06a109b4c8 | 19:50 |
| fnaval | TrevorV nodded at me and softly said yes | 19:50 |
| sbalukoff | That's right-- if you go to the data_model then .to_dict() on that, you lose your sqlalchemy relationships (by design) | 19:50 |
| rm_work | fnaval: no | 19:50 |
| ptoohill | fnaval: no | 19:51 |
| ptoohill | ^ | 19:51 |
| fnaval | ok | 19:51 |
| rm_work | fnaval: when testing if you are using my script to spin VMs you can remove everything referencing neutron-lbaas | 19:51 |
| rm_work | which would be one enable-plugin line and a couple of ENABLED_SERVICES lines | 19:51 |
| sbalukoff | TrevorV: which line of that is giving you problems? | 19:51 |
| rm_work | TrevorV: ok so which line is breaking | 19:52 |
| rm_work | yeah | 19:52 |
| fnaval | rm_work: ptoohill do upstream octavia gate jobs require the lbaasv2 extension? | 19:54 |
| TrevorV | I thought franklin was talking about CLI. To use neutron lbaas-loadbalancer-list and such you need lbaas v2 | 19:54 |
| TrevorV | But not to just hit the octavia API for scenario tests | 19:55 |
| TrevorV | I clarified with him | 19:55 |
| *** minwang2 has joined #openstack-lbaas | 19:55 | |
| johnsom | Agenda for today's meeting: https://wiki.openstack.org/wiki/Octavia/Weekly_Meeting_Agenda#Meeting_2016-02-10 | 19:55 |
| TrevorV | The LINES giving me problems are 11 and 19 | 19:55 |
| *** doug-fish has quit IRC | 19:55 | |
| TrevorV | in this case, the test I'm trying to fix has to do with line 19 | 19:55 |
| TrevorV | db_pool.load_balancer == NoneType | 19:55 |
| TrevorV | So it can't get the id off it | 19:56 |
| TrevorV | The commented out stuff is what USED to be there, and was throwing much more interesting errors. | 19:56 |
| TrevorV | The commented out lines couldn't even find a pool in the DB previously... o_0 | 19:56 |
| TrevorV | Between 8 and 10 there should be more commented out stuff, but I accidentally deleted it. | 19:57 |
| TrevorV | sbalukoff rm_work | 19:58 |
| sbalukoff | Right... so in the original code I wrote there, I get the pool_db in a different place because the query for each if statement is different. | 19:58 |
| TrevorV | Yeah, I saw immediately what you were doing, but testing for some reason wasn't finding any LB or Pool or anything. | 19:59 |
| rm_work | fnaval: i would assume no, we'd be hitting Octavia directly via requests or somesuch in the tests, so no reason Octavia scenario gate would need neutron-lbaas | 19:59 |
| TrevorV | Returned No object at all. | 19:59 |
| sbalukoff | Can you gist a dump of your database load balancer, listener and pool tables? | 19:59 |
| johnsom | Octavia meeting starting on #openstack-meeting-alt | 19:59 |
| TrevorV | to be honest sbalukoff I'm not entirely sure how to look at those when talking about in memory DBs | 20:00 |
| fnaval | rm_work: kk thanks for the clarification | 20:01 |
| *** doug-fish has joined #openstack-lbaas | 20:02 | |
| *** kevo has joined #openstack-lbaas | 20:08 | |
| *** blallau has joined #openstack-lbaas | 20:16 | |
| *** doug-fish has quit IRC | 20:18 | |
| *** doug-fish has joined #openstack-lbaas | 20:20 | |
| *** barclaac has joined #openstack-lbaas | 20:26 | |
| *** piet has joined #openstack-lbaas | 20:26 | |
| *** longstaff has quit IRC | 20:37 | |
| *** longstaff has joined #openstack-lbaas | 20:39 | |
| *** kevo has quit IRC | 20:43 | |
| *** kevo has joined #openstack-lbaas | 20:52 | |
| *** localloop127 has joined #openstack-lbaas | 20:54 | |
| *** localloop127 has quit IRC | 20:58 | |
| johnsom | dougwig 1541670 Our gates, except for scenario, no longer depend on DIB. For this bug, what is your ask? Setup an Octavia image gate in DIB project? | 21:00 |
| johnsom | https://bugs.launchpad.net/neutron/+bug/1541670 | 21:00 |
| openstack | Launchpad bug 1541670 in neutron "lbaas tests gate on dib" [High,Confirmed] - Assigned to Michael Johnson (johnsom) | 21:00 |
| *** kevo has quit IRC | 21:03 | |
| fnaval | #bug 172199 | 21:10 |
| openstack | bug 172199 in Inkscape "Use of native GUI components on Mac OSX and Windows" [Low,Won't fix] https://launchpad.net/bugs/172199 | 21:10 |
| fnaval | oops wrong | 21:11 |
| fnaval | #bug 1544290 | 21:11 |
| openstack | bug 1544290 in octavia "Octavia: Create load balancer operating status does not go ONLINE" [Undecided,New] https://launchpad.net/bugs/1544290 | 21:11 |
| johnsom | fnaval That is a good question | 21:13 |
| fnaval | johnsom: spoke with brandon/trevor/adam about it and they say that octavia should be ACTIVE/OFFLINE until a member is added. This is inconsistent with neutron-lbaas and I forsee some confusion with this in the future. | 21:14 |
| johnsom | Yep | 21:15 |
| xgerman | we should file a bug | 21:15 |
| johnsom | He did ^^^^ | 21:15 |
| mhayden | is it possible to remove an lbaasv2 load balancer from its associated agent, as you can do with the l3 agent? | 21:15 |
| johnsom | fnaval FYI, I also created an operating state of NO_MONITOR for pools with members but no health monitor. I added that to both Octavia and LBaaS | 21:17 |
| fnaval | johnsom: good to know - i'll take note of it | 21:18 |
| fnaval | johnsom: I don't see any mention of it in the octaviaapi docs; it would be good to have that info in there somewhere | 21:18 |
| johnsom | Hmm, do we have that status in the docs? | 21:19 |
| dougwig | johnsom: or make the scenario non-voting. either would work | 21:19 |
| *** blallau has quit IRC | 21:25 | |
| longstaff | Hi. I'm looking for help on the procedure for contributing a vendor (third party) LBaaS v2 plugin/driver to neutron-lbaas. I'm aware of the neutron project guidelines for contributing new extensions to Neutron (http://docs.openstack.org/developer/neutron/devref/contribute.html) but it's not clear to me if those guidelines apply to neutron-lbaas. Can anyone provide guidance? | 21:28 |
| *** rooney has joined #openstack-lbaas | 21:28 | |
| dougwig | hiya longstaff | 21:29 |
| longstaff | Hi | 21:29 |
| dougwig | file a wishlist bug and go to town, that's it. i'd recommend you host it out of tree with a stub in tree, but you can submit the entire thing if you wish (for now). | 21:30 |
| dougwig | longstaff: what package or vendor is this for? | 21:30 |
| dougwig | longstaff: and welcome. | 21:30 |
| longstaff | Thank you. This is for F5 Networks. We have a shim and unit test ready. | 21:31 |
| longstaff | I submitted an RFE, 1539717. Is that what's needed? | 21:32 |
| xgerman | welcome | 21:33 |
| longstaff | Thank you. | 21:33 |
| dougwig | longstaff: yep, that's good. if your code is ready, go ahead and submit it to gerrit with that bug id. | 21:35 |
| longstaff | Great. Thanks for your help. | 21:35 |
| *** ducttape_ has quit IRC | 21:39 | |
| *** fnaval_ has joined #openstack-lbaas | 21:39 | |
| *** dnovosel_ has joined #openstack-lbaas | 21:40 | |
| *** pai15- has joined #openstack-lbaas | 21:40 | |
| *** Dave_____ has joined #openstack-lbaas | 21:43 | |
| xgerman | dougwig while I have your ear: https://review.openstack.org/#/c/272720/ | 21:44 |
| *** markvan_ has joined #openstack-lbaas | 21:45 | |
| *** HenryG_ has joined #openstack-lbaas | 21:47 | |
| *** fnaval has quit IRC | 21:48 | |
| *** Dave has quit IRC | 21:48 | |
| *** pai15 has quit IRC | 21:48 | |
| *** HenryG has quit IRC | 21:48 | |
| *** ramishra has quit IRC | 21:48 | |
| *** mariusv has quit IRC | 21:48 | |
| *** eezhova has quit IRC | 21:48 | |
| *** hockeynut has quit IRC | 21:48 | |
| *** dnovosel has quit IRC | 21:48 | |
| *** markvan has quit IRC | 21:48 | |
| *** HenryG_ is now known as HenryG | 21:48 | |
| *** rooney has quit IRC | 21:48 | |
| *** hockeynut has joined #openstack-lbaas | 21:49 | |
| *** rooney has joined #openstack-lbaas | 21:50 | |
| *** HenryG has quit IRC | 21:52 | |
| *** kevo has joined #openstack-lbaas | 21:52 | |
| johnsom | blogan fnaval Am I having a senior moment, or are the statuses backwards here: http://www.octavia.io/review/master/api/octaviaapi.html | 21:52 |
| *** localloop127 has joined #openstack-lbaas | 21:55 | |
| *** eezhova has joined #openstack-lbaas | 21:55 | |
| *** rooney has quit IRC | 21:55 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-lbaas: Updated from global requirements https://review.openstack.org/278696 | 21:55 |
| fnaval_ | johnsom: ah yes they are aren't they | 21:56 |
| *** HenryG has joined #openstack-lbaas | 21:56 | |
| fnaval_ | let me do a fix for that | 21:56 |
| *** ramishra has joined #openstack-lbaas | 21:56 | |
| *** fnaval_ is now known as fnaval | 21:56 | |
| johnsom | fnaval_ Ok thanks! While you are there, can you add the NO_MONITOR? | 21:58 |
| johnsom | That is why I noticed, I was going to push a patch for you that adds it | 21:58 |
| fnaval | good catch - where does it go? | 21:58 |
| johnsom | Under the "true" operating status | 21:59 |
| *** localloop127 has quit IRC | 21:59 | |
| fnaval | johnsom: k, it will show: Operating StatusONLINE, OFFLINE, DEGRADED, ERROR, NO_MONITOR | 22:00 |
| johnsom | Yes, I think that is correct | 22:05 |
| johnsom | At least based off the constants.py file | 22:05 |
| *** rtheis has quit IRC | 22:05 | |
| *** rcernin has quit IRC | 22:07 | |
| fnaval | 278762 | 22:08 |
| fnaval | https://review.openstack.org/#/c/278762/ | 22:08 |
| rm_work | weird, why didn't the bot report that | 22:09 |
| fnaval | yea strange | 22:09 |
| johnsom | Yeah, I was wondering the same | 22:09 |
| * rm_work pokes openstackgerrit | 22:10 | |
| *** Guest15678 has joined #openstack-lbaas | 22:10 | |
| fnaval | well anyway - that review should be good to go | 22:12 |
| johnsom | Thanks! | 22:13 |
| fnaval | johnsom: np thank you too. =) | 22:14 |
| *** kev0 has joined #openstack-lbaas | 22:16 | |
| blogan | johnsom: yes they are | 22:17 |
| blogan | johnsom: i think fnaval confirmed alreayd though | 22:17 |
| *** armax has quit IRC | 22:19 | |
| fnaval | thanks for the quick review turnarounds! | 22:21 |
| *** woodster__ has joined #openstack-lbaas | 22:22 | |
| *** chlong_ has joined #openstack-lbaas | 22:22 | |
| *** kevo has quit IRC | 22:22 | |
| *** chlong has quit IRC | 22:22 | |
| *** woodster_ has quit IRC | 22:22 | |
| *** woodster__ is now known as woodster_ | 22:23 | |
| *** armax has joined #openstack-lbaas | 22:27 | |
| *** ducttape_ has joined #openstack-lbaas | 22:27 | |
| *** madhu_ak_ has quit IRC | 22:34 | |
| madhu_ak | fnaval: I posted comments for https://review.openstack.org/#/c/172199/28 | 22:34 |
| fnaval | madhu_ak: cool. thanks madhu_ak - I will review and address when I get a chance. | 22:47 |
| *** neelashah1 has quit IRC | 22:47 | |
| *** longstaff has quit IRC | 22:49 | |
| TrevorV | thanks sbalukoff for breaking changes... what a jerk face. | 22:52 |
| TrevorV | You added "lb_algorithm" to PoolPOST and it was already there. | 22:52 |
| TrevorV | :P | 22:52 |
| TrevorV | Not a breakage, just kidding with ya, but it does just overwrite a previously defined lb_algorithm field :D | 22:53 |
| *** localloop127 has joined #openstack-lbaas | 22:55 | |
| *** piet has quit IRC | 22:56 | |
| *** doug-fish has quit IRC | 22:59 | |
| *** localloop127 has quit IRC | 23:00 | |
| *** doug-fish has joined #openstack-lbaas | 23:00 | |
| *** armax has quit IRC | 23:02 | |
| *** TrevorV has quit IRC | 23:04 | |
| *** doug-fish has quit IRC | 23:05 | |
| sbalukoff | Oh? | 23:07 |
| rm_work | he was just poking fun about https://github.com/openstack/octavia/blame/master/octavia/api/v1/types/pool.py#L63 | 23:07 |
| sbalukoff | Huh... | 23:07 |
| rm_work | which i let through too so whatever :P | 23:08 |
| sbalukoff | Oh! That was probably a rebasing error. | 23:08 |
| rm_work | doesn't actually affect anything | 23:08 |
| *** doug-fis_ has joined #openstack-lbaas | 23:08 | |
| rm_work | yep | 23:08 |
| sbalukoff | Yeah-- half the reason I was so anxious to get that patch it was because rebasing it was a nightmare. | 23:09 |
| sbalukoff | So congratulations! I foisted that problem off on to y'all! ;) | 23:09 |
| sbalukoff | Anyway, is there a patchset to remove the redundant lb_algorithm in there? | 23:10 |
| *** doug-fis_ has quit IRC | 23:10 | |
| *** doug-fis_ has joined #openstack-lbaas | 23:11 | |
| sbalukoff | (that'd be a super simple ATC if anyone is still looking for it, eh.) | 23:11 |
| rm_work | yeah i think trevor will prolly just do it as part of his larger patch | 23:11 |
| rm_work | he's working on Single-LB-Call | 23:12 |
| sbalukoff | Wow! | 23:13 |
| sbalukoff | Taking over from Brandon on that? | 23:13 |
| sbalukoff | On that front: One of these days, someone should go through and clean up a bunch of the code smell issues that sonar complains about. | 23:14 |
| rm_work | yeah | 23:14 |
| rm_work | some of them are valid | 23:14 |
| johnsom | Some are not so valid | 23:14 |
| rm_work | some ... yeah | 23:14 |
| rm_work | not so | 23:14 |
| rm_work | might want to tweak the profiles as we find stuff that isn't valid | 23:15 |
| *** localloop127 has joined #openstack-lbaas | 23:15 | |
| *** Aish has quit IRC | 23:16 | |
| sbalukoff | ptoohill: Ping | 23:18 |
| *** Aish has joined #openstack-lbaas | 23:18 | |
| *** localloop127 has quit IRC | 23:22 | |
| *** ducttape_ has quit IRC | 23:24 | |
| rm_work | sbalukoff: for testing L7 stuff in devstack, where should I start with the CRs? like, should I grab all the way from the very end of the chain, or somewhere in the middle? | 23:24 |
| ptoohill | sbalukoff: pong | 23:24 |
| rm_work | obviously i can't test the db updates directly really | 23:25 |
| rm_work | like should I just check out https://review.openstack.org/#/c/278223/ and test everything from that point? | 23:25 |
| rm_work | (that looks like the end of the chain?) | 23:25 |
| sbalukoff | ptoohill: responded to your comment on my L7 jinja patch. Could you verify whether I'm understanding your correctly? | 23:25 |
| ptoohill | I just read that | 23:26 |
| sbalukoff | rm_work: Yes, work off the latest one. Also note that I've not had time to run this through scenario tests myself yet, so I'm expecting to find breakage. | 23:26 |
| ptoohill | i was thinking everything under commong/jinja, so.. | 23:26 |
| ptoohill | commona/jinja/haproxy/templater.py and common/jinja/templates/haproxy/templates.j2 | 23:26 |
| sbalukoff | That's fine. I was going for a name less dependent on the technology in use, but in all honesty it's very unlikely we'll use a templating system other than jinja. | 23:27 |
| rm_work | yeah i would vote we standardize on that | 23:27 |
| ptoohill | common/templates by itself didnt make a whole lot of sense, otherwise it was on the right track | 23:27 |
| rm_work | i don't think there's anything Jinja just *can't* do | 23:27 |
| sbalukoff | ptoohill: Ok, I'm not sure I follow. | 23:27 |
| sbalukoff | common/jinja/haproxy/templater.py | 23:27 |
| ptoohill | ok, i see your point though | 23:28 |
| sbalukoff | What is templater.py? | 23:28 |
| ptoohill | hmm | 23:28 |
| ptoohill | jinja_cfg.py in that case | 23:28 |
| ptoohill | so in my case i would have | 23:28 |
| ptoohill | common/jinja/user-data/user-data-templater.py | 23:28 |
| sbalukoff | Are suggesting I rename jinja_cfg.py to templater.py? | 23:28 |
| sbalukoff | (literally?) | 23:28 |
| ptoohill | no, lol | 23:29 |
| sbalukoff | Aah, ok. | 23:29 |
| ptoohill | those were just generic names | 23:29 |
| ptoohill | directory structure is in question here | 23:29 |
| sbalukoff | Right. | 23:29 |
| ptoohill | I see you point about 'nameless' templater | 23:29 |
| ptoohill | I suppose we could use something else, but those templates are designed for jinja | 23:29 |
| sbalukoff | So, I'm OK with common/jinja/<thing_we_are_templating>/stuff | 23:29 |
| sbalukoff | Where stuff consists of a templater and a 'templates' directory. | 23:30 |
| ptoohill | will they work for another templating system? maybe? most cases probably not? | 23:30 |
| ptoohill | but you would want common/templates to remain? | 23:30 |
| sbalukoff | It's not whether they will work with another templating system, it's whether we will want to use a different templating system. | 23:30 |
| sbalukoff | No! | 23:30 |
| rm_work | well i mean... if we did use a different system, we'd also need a new different file for that one too right? :P | 23:31 |
| ptoohill | yea | 23:31 |
| sbalukoff | I don't see us using anything other than jinja for templates for the forseeable future. :)_ | 23:31 |
| rm_work | it wouldn't be the same file | 23:31 |
| ptoohill | that's what im thinking | 23:31 |
| sbalukoff | Right. | 23:31 |
| ptoohill | I think were on the same page here | 23:31 |
| ptoohill | :) | 23:31 |
| sbalukoff | Ok, so.... then what I explained in my response comment is I think what we're describing. | 23:31 |
| sbalukoff | :) | 23:31 |
| ptoohill | OK, then yea, what you describe there works for me, a bit different then what i described, but this makes more sense. So yay! | 23:32 |
| ptoohill | :) | 23:32 |
| sbalukoff | Yay! | 23:32 |
| *** doug-fis_ has quit IRC | 23:33 | |
| ptoohill | Thank you sbalukoff | 23:33 |
| sbalukoff | No problem, eh! | 23:34 |
| sbalukoff | And thank you as well. | 23:34 |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Add L7 jinja template updates https://review.openstack.org/278223 | 23:39 |
| *** chlong_ has quit IRC | 23:46 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
