rm_work | not sure if it's related to quotas | 00:00 |
---|---|---|
rm_work | don't have another devstack to test on yet | 00:00 |
*** chlong has quit IRC | 00:01 | |
rm_work | oh and FYI | 00:01 |
rm_work | 2017-01-12 23:53:36.558 | +lib/neutron_plugins/services/l3:_neutron_configure_router_v6:416 sudo ip -6 addr replace 2001:db8::2/64 dev br-ex | 00:01 |
rm_work | 2017-01-12 23:53:36.579 | RTNETLINK answers: Permission denied | 00:01 |
rm_work | that's where it dies with IPv6 disabled | 00:01 |
johnsom | Well, I did touch that code, but the exception handling for that case is still there. I'm going to try it | 00:01 |
johnsom | Hmm, interesting. That must be new-ish | 00:01 |
rm_work | asking in qa | 00:03 |
rm_work | ah | 00:04 |
johnsom | Well, neutron-lbaas handles it ok: | 00:04 |
rm_work | http://docs.openstack.org/developer/devstack/configuration.html#ip-version | 00:04 |
rm_work | I think that | 00:04 |
johnsom | neutron lbaas-member-create --subnet tenant-2-subnet --address 172.21.2.12 --protocol-port 80 pool1 | 00:04 |
johnsom | neutron CLI is deprecated and will be removed in the future. Use openstack CLI instead. | 00:04 |
johnsom | Member with address 172.21.2.12 and protocol_port 80 already present in pool e6bae357-aacb-46a5-bcb1-08dcabc08464 | 00:04 |
johnsom | Neutron server returns request_ids: ['req-c6b97221-e700-4e6a-b6ee-88e6977a1c0b'] | 00:04 |
rm_work | johnsom: oh, i'm going through octavia | 00:04 |
johnsom | Trying with native octavia next | 00:04 |
rm_work | curl -X POST -H Content-type:application/json -H "X-Auth-Token:$MYTOKEN" -d '{"ip_address": "216.58.192.174", "protocol_port": 80, "project_id": "'$MYPROJECT'"}' http://localhost:9876/v1/loadbalancers/${LB_ID}/listeners/${LISTENER1_ID}/pools/${POOL1_ID}/members | python -m json.tool | 00:05 |
johnsom | Yeah, I can repo. Let's see what I can do with this | 00:07 |
rm_work | I wonder why it works through n-lbaas? >_> | 00:09 |
rm_work | it should be issuing the same command | 00:09 |
johnsom | It's querying it's own DB | 00:10 |
johnsom | Different API code... | 00:10 |
rm_work | yeah but then it hits octavia with the same request, no? | 00:10 |
johnsom | Probably not, I bet there is a validator in the nlbaas API | 00:10 |
rm_work | what would it validate? | 00:11 |
rm_work | that the member isn't duplicate? | 00:11 |
johnsom | I am guessing here... | 00:11 |
rm_work | hmm | 00:11 |
rm_work | i mean, the neutron-lbaas octavia driver just sends the same REST requests that I am manually, AFAIK | 00:12 |
rm_work | the member create call isn't complex, it's literally IP and PORT | 00:12 |
rm_work | I mean, you HAVE to be right at some level, because obviously the result is different | 00:12 |
rm_work | disagreeing with that isn't really an option :/ | 00:12 |
*** fnaval has quit IRC | 00:13 | |
rm_work | but it doesn't make a lot of sense to me | 00:13 |
johnsom | I think it just gets caught before the call is made to octavia. | 00:13 |
rm_work | so, it lies? | 00:13 |
rm_work | "yeah, created your member fine!" *didn't do anything* | 00:13 |
johnsom | So, yeah, it's walking off the end of that if elif for duplicates. Existing code, probably never run. | 00:14 |
rm_work | which would also be a bug in n-lbaas IMO | 00:14 |
rm_work | it should return an error code | 00:14 |
johnsom | The duplicate column is "uq_member_pool_id_address_protocol_port" evidently | 00:14 |
rm_work | right | 00:15 |
rm_work | makes sense | 00:15 |
rm_work | so, does having the same member twice make sense? | 00:15 |
rm_work | it DOESN'T, right? | 00:15 |
johnsom | It does not make sense | 00:15 |
rm_work | right, you can just double the weight if you want to have it "twice" | 00:16 |
johnsom | I'm just going to simplify this exception handler | 00:16 |
rm_work | it can be a different CR | 00:16 |
johnsom | Right | 00:16 |
rm_work | quotas looks good from my testing otherwise | 00:16 |
*** cody-somerville has joined #openstack-lbaas | 00:17 | |
rm_work | I haven't gone through the code line-by-line yet tho :/ ugh | 00:17 |
rm_work | I need to grab breakfast before I can deal with that | 00:17 |
johnsom | Grin. Ok, I am going to post one with the fix for that. Probably should check why the functional test for members didn't catch it, will do that now too. | 00:18 |
johnsom | Ahhh, it passes functional test (there is one) because sqllite handles that differently. | 00:27 |
johnsom | Joy | 00:27 |
*** cody-somerville has quit IRC | 00:42 | |
rm_work | rofl | 00:42 |
rm_work | woo sqlite | 00:42 |
rm_work | alright, diving into this | 00:42 |
rm_work | umm, are these default quotas right? | 00:43 |
rm_work | 20 LBs 10 pools? lol | 00:43 |
rm_work | that's ... not useful | 00:43 |
rm_work | or is pools PER LB? | 00:43 |
rm_work | ... I didn't actually test that | 00:43 |
johnsom | Per project. | 00:43 |
johnsom | So, yeah, the defaults are bogus.... | 00:43 |
rm_work | OK so... you can have 20 LBs but only 10 of them can have pools? :P | 00:44 |
johnsom | Should we just set them unlimited? | 00:44 |
johnsom | Shared pools man.... | 00:44 |
johnsom | grin | 00:44 |
rm_work | can't share between LBs :P | 00:44 |
johnsom | Remember, I inherited this patch | 00:44 |
rm_work | right I know very well | 00:44 |
rm_work | I inherited it too | 00:44 |
rm_work | and managed to escape somehow :P | 00:44 |
rm_work | if only pools were sharable across LBs... | 00:44 |
johnsom | Should we just set them unlimited? | 00:44 |
johnsom | That is kind of how I lean | 00:45 |
rm_work | by default? ... maybe | 00:45 |
rm_work | or maybe it'd be safe to have LBs default to like 10 and everything ELSE be unlimited | 00:50 |
rm_work | but it's pretty arbitrary whatever number you use | 00:50 |
johnsom | Yeah, I'm going to take the backwards-compat clause and say unlimited for everyone.... | 00:51 |
openstackgerrit | Michael Johnson proposed openstack/octavia: Add quota support to Octavia https://review.openstack.org/360794 | 00:51 |
johnsom | Ok, fixed those | 00:51 |
rm_work | damn, same error with IP_VERSION=4 | 00:52 |
openstackgerrit | Merged openstack/octavia: Updated from global requirements https://review.openstack.org/419351 | 01:01 |
*** kevo has quit IRC | 01:27 | |
rm_work | johnsom: couple more questions | 01:36 |
rm_work | johnsom: on the quota review | 01:36 |
rm_work | (posted) | 01:36 |
johnsom | Sure | 01:36 |
*** cody-somerville has joined #openstack-lbaas | 01:40 | |
*** cody-somerville has quit IRC | 01:40 | |
*** cody-somerville has joined #openstack-lbaas | 01:40 | |
*** gongysh has joined #openstack-lbaas | 01:40 | |
johnsom | rm_work Can you take a quick look at my answers and give me feedback, then I will do another spin | 01:45 |
rm_work | OK then it's just the constants :P | 01:50 |
rm_work | then I'm good for +2 | 01:51 |
johnsom | I can change to docs over too if you think it's the right thing. | 01:52 |
rm_work | I think it's the right thing, but | 01:52 |
rm_work | I don't think it's your responsibility here :/ | 01:52 |
johnsom | Ok, well, let's do it now... Get-er-done | 01:53 |
rm_work | lol alright, fine by me :P | 01:53 |
rm_work | but it's super tedious | 01:53 |
rm_work | since i think a blind find/replace is too much | 01:53 |
johnsom | Yeah. The should the request examples still be single quotes? | 01:54 |
johnsom | I think so | 01:54 |
rm_work | err | 01:56 |
rm_work | like what line | 01:56 |
rm_work | ah | 01:57 |
rm_work | no | 01:57 |
rm_work | requests with single-quotes in JSON will fail | 01:57 |
*** yamamoto_ has joined #openstack-lbaas | 01:57 | |
johnsom | Great, now this too | 01:58 |
johnsom | http://logs.openstack.org/94/360794/64/check/gate-octavia-v1-dsvm-scenario-ubuntu-xenial-nv/c99268b/console.html#_2017-01-13_01_42_50_664328 | 01:58 |
rm_work | lol wut | 01:58 |
johnsom | A quick google comes up with neutron having the same problem. They solved it by turning debug off for the test. The code just has a TODO in it.... | 01:59 |
rm_work | that's AFTER the tests pass | 01:59 |
rm_work | what test even failed | 01:59 |
johnsom | It's not a test, it's the subunit stuff | 01:59 |
rm_work | are we TOO VERBOSE? | 01:59 |
rm_work | subunit can't parse all the crap we spit out? | 01:59 |
johnsom | https://github.com/testing-cabal/subunit/blob/master/python/subunit/v2.py#L204 | 02:00 |
johnsom | I don't know. There is voodoo in that stuff for me still | 02:00 |
rm_work | umm, lol | 02:01 |
rm_work | yeah huh | 02:01 |
johnsom | Looking at the testr file it's parsing, The biggest blocks I see are the response body stuff | 02:02 |
johnsom | Yeah, the log output from "test_load_balancer_tree_minimal" is lengthy with all of the req/resp text | 02:04 |
*** bana_k has quit IRC | 02:07 | |
*** ducttape_ has joined #openstack-lbaas | 02:19 | |
openstackgerrit | Joe Mills proposed openstack/neutron-lbaas: api test: centralize health monitor protocol https://review.openstack.org/419324 | 02:27 |
*** cody-somerville has quit IRC | 02:37 | |
rm_work | alright, I'll be +2 whenever the next patchset goes up | 02:40 |
rm_work | is the subunit fail on anything voting? | 02:41 |
rm_work | I'd kinda be for ... fixing it in a followup CR | 02:41 |
johnsom | Cool, just a few more minutes. Fixing some other inconsistencies. | 02:41 |
rm_work | looks like that was on the scenarios... | 02:41 |
johnsom | The listener TLS stuff in this doc is all messed up too... That will be for another day | 02:41 |
rm_work | ah k | 02:41 |
rm_work | sad :/ | 02:41 |
johnsom | Yeah, it's in non-voting scenario tests | 02:42 |
rm_work | I've had a reprieve from looking at TLS since we aren't supporting it here for MVP, but | 02:42 |
rm_work | I need to dig into that again eventually and get the weirdness sorted | 02:42 |
johnsom | Well, soon I'm going to take on our api ref (next week maybe) so I will have to straighten it out then | 02:43 |
openstackgerrit | Michael Johnson proposed openstack/octavia: Add quota support to Octavia https://review.openstack.org/360794 | 02:45 |
johnsom | Ok, there it is.... | 02:45 |
johnsom | I need to sign off for the evening. Catch you tomorrow | 02:45 |
*** yuanying has quit IRC | 02:52 | |
*** amotoki has quit IRC | 02:53 | |
*** ducttape_ has quit IRC | 02:59 | |
*** ducttape_ has joined #openstack-lbaas | 03:00 | |
*** ducttape_ has quit IRC | 03:04 | |
*** ducttape_ has joined #openstack-lbaas | 03:18 | |
*** gongysh has quit IRC | 03:29 | |
*** ducttape_ has quit IRC | 03:42 | |
*** ducttape_ has joined #openstack-lbaas | 03:42 | |
*** ducttape_ has quit IRC | 03:44 | |
*** ducttape_ has joined #openstack-lbaas | 03:44 | |
*** yuanying has joined #openstack-lbaas | 03:52 | |
*** links has joined #openstack-lbaas | 03:54 | |
*** ducttape_ has quit IRC | 04:08 | |
*** ducttape_ has joined #openstack-lbaas | 04:09 | |
*** ducttape_ has quit IRC | 04:11 | |
*** ducttape_ has joined #openstack-lbaas | 04:12 | |
*** armax has quit IRC | 04:17 | |
*** ducttape_ has quit IRC | 04:18 | |
*** amotoki has joined #openstack-lbaas | 04:46 | |
*** beardedeagle has joined #openstack-lbaas | 05:18 | |
*** beardedeagle has quit IRC | 05:24 | |
*** amotoki_ has joined #openstack-lbaas | 05:27 | |
*** amotoki_ has quit IRC | 05:28 | |
*** gongysh has joined #openstack-lbaas | 05:30 | |
*** sticker has quit IRC | 05:31 | |
*** sferna1_ has joined #openstack-lbaas | 05:34 | |
*** yamamoto_ has quit IRC | 05:37 | |
*** anilvenkata has joined #openstack-lbaas | 06:00 | |
*** yuanying has quit IRC | 06:06 | |
*** yuanying has joined #openstack-lbaas | 06:07 | |
*** yuanying has quit IRC | 06:11 | |
*** yuanying_ has joined #openstack-lbaas | 06:11 | |
*** yuanying_ has quit IRC | 06:12 | |
*** yuanying has joined #openstack-lbaas | 06:12 | |
*** yuanying has quit IRC | 06:17 | |
*** yamamoto_ has joined #openstack-lbaas | 06:21 | |
*** madhu_ak has joined #openstack-lbaas | 06:45 | |
*** gcheresh_ has joined #openstack-lbaas | 06:48 | |
*** gcheresh has joined #openstack-lbaas | 06:52 | |
*** gcheresh_ has quit IRC | 06:55 | |
*** gongysh has quit IRC | 06:57 | |
*** anilvenkata has quit IRC | 06:59 | |
*** madhu_ak has quit IRC | 07:06 | |
*** tesseract has joined #openstack-lbaas | 07:11 | |
*** gcheresh_ has joined #openstack-lbaas | 07:12 | |
*** gcheresh has quit IRC | 07:13 | |
*** yuanying has joined #openstack-lbaas | 07:19 | |
*** sindhu has quit IRC | 07:28 | |
*** ptoohill has quit IRC | 07:30 | |
*** dlundquist has quit IRC | 07:30 | |
*** mhayden has quit IRC | 07:30 | |
*** dlundquist has joined #openstack-lbaas | 07:33 | |
*** ptoohill has joined #openstack-lbaas | 07:34 | |
*** sindhu has joined #openstack-lbaas | 07:34 | |
*** mhayden has joined #openstack-lbaas | 07:36 | |
*** AlexeyAbashkin has joined #openstack-lbaas | 07:39 | |
*** gcheresh_ has quit IRC | 08:06 | |
*** kevo has joined #openstack-lbaas | 08:16 | |
*** eezhova has joined #openstack-lbaas | 08:18 | |
*** kevo has quit IRC | 09:07 | |
*** ri0 has joined #openstack-lbaas | 09:08 | |
*** kobis has quit IRC | 09:15 | |
*** amotoki has quit IRC | 09:18 | |
*** ri0 has quit IRC | 09:19 | |
openstackgerrit | Joe Mills proposed openstack/neutron-lbaas: scenario test: configure scenario test protocols https://review.openstack.org/418201 | 09:29 |
*** amotoki has joined #openstack-lbaas | 09:36 | |
*** openstackgerrit has quit IRC | 10:18 | |
*** ipsecguy has quit IRC | 10:52 | |
*** sferna1_ has quit IRC | 11:37 | |
*** ipsecguy has joined #openstack-lbaas | 11:51 | |
*** yamamoto_ has quit IRC | 12:11 | |
*** catintheroof has joined #openstack-lbaas | 12:30 | |
*** catintheroof has quit IRC | 12:31 | |
*** catintheroof has joined #openstack-lbaas | 12:32 | |
*** gcheresh_ has joined #openstack-lbaas | 12:55 | |
*** yamamoto has joined #openstack-lbaas | 13:04 | |
*** yamamoto_ has joined #openstack-lbaas | 13:05 | |
*** yamamoto has quit IRC | 13:09 | |
*** amotoki has quit IRC | 13:19 | |
*** ducttape_ has joined #openstack-lbaas | 13:31 | |
*** yamamoto_ has quit IRC | 13:37 | |
*** AlexeyAbashkin has quit IRC | 13:37 | |
*** yamamoto has joined #openstack-lbaas | 13:40 | |
*** AlexeyAbashkin has joined #openstack-lbaas | 13:40 | |
*** reedip_outofmemo has joined #openstack-lbaas | 13:43 | |
*** sferna1_ has joined #openstack-lbaas | 13:49 | |
*** gcheresh_ has quit IRC | 13:49 | |
*** ducttape_ has quit IRC | 13:55 | |
*** beardedeagle has joined #openstack-lbaas | 14:11 | |
*** yamamoto has quit IRC | 14:35 | |
*** chlong has joined #openstack-lbaas | 14:45 | |
*** amotoki has joined #openstack-lbaas | 14:49 | |
*** yamamoto has joined #openstack-lbaas | 14:50 | |
*** yamamoto has quit IRC | 14:54 | |
*** beagles is now known as beagles_afk | 14:54 | |
*** ducttape_ has joined #openstack-lbaas | 15:04 | |
*** ducttape_ has quit IRC | 15:09 | |
*** armax has joined #openstack-lbaas | 15:26 | |
*** fnaval has joined #openstack-lbaas | 15:33 | |
*** ducttape_ has joined #openstack-lbaas | 15:33 | |
*** jsheeren has joined #openstack-lbaas | 15:34 | |
*** jsheeren has left #openstack-lbaas | 15:37 | |
*** kobis has joined #openstack-lbaas | 16:29 | |
*** pcaruana has joined #openstack-lbaas | 16:36 | |
*** chlong has quit IRC | 16:38 | |
xgerman | o/ | 16:40 |
johnsom | Morning | 16:42 |
*** eezhova has quit IRC | 16:42 | |
*** chlong has joined #openstack-lbaas | 16:43 | |
*** chlong has quit IRC | 16:48 | |
*** AlexeyAbashkin has quit IRC | 16:49 | |
*** bana_k has joined #openstack-lbaas | 16:50 | |
*** reedip_outofmemo has quit IRC | 16:51 | |
rm_work | o/ | 16:59 |
rm_work | night | 16:59 |
*** chlong has joined #openstack-lbaas | 16:59 | |
johnsom | Turning in? | 17:06 |
*** chlong has quit IRC | 17:09 | |
*** bana_k has quit IRC | 17:15 | |
*** eezhova has joined #openstack-lbaas | 17:26 | |
*** eezhova has quit IRC | 17:38 | |
*** _ducttape_ has joined #openstack-lbaas | 17:41 | |
*** ducttape_ has quit IRC | 17:44 | |
*** kevo has joined #openstack-lbaas | 17:45 | |
*** eezhova has joined #openstack-lbaas | 17:49 | |
*** openstackgerrit has joined #openstack-lbaas | 18:11 | |
openstackgerrit | Merged openstack/octavia: Add quota support to Octavia https://review.openstack.org/360794 | 18:11 |
*** cody-somerville has joined #openstack-lbaas | 18:11 | |
*** links has quit IRC | 18:12 | |
*** bana_k has joined #openstack-lbaas | 18:15 | |
*** beardedeagle has quit IRC | 18:24 | |
*** _ducttape_ has quit IRC | 18:25 | |
*** pcaruana has quit IRC | 18:26 | |
*** ducttape_ has joined #openstack-lbaas | 18:26 | |
*** pcaruana has joined #openstack-lbaas | 18:39 | |
*** tesseract has quit IRC | 18:53 | |
openstackgerrit | Michael Johnson proposed openstack/octavia: Fix octavia multinode setup for flavor and image https://review.openstack.org/420133 | 18:55 |
*** openstack has joined #openstack-lbaas | 19:02 | |
*** pcaruana has quit IRC | 19:02 | |
*** eezhova has quit IRC | 19:05 | |
*** chlong has joined #openstack-lbaas | 19:14 | |
*** eezhova has joined #openstack-lbaas | 19:30 | |
*** ankur-gupta-f1 has left #openstack-lbaas | 19:34 | |
openstackgerrit | Michael Johnson proposed openstack/octavia: Fix octavia multinode setup for flavor and image https://review.openstack.org/420133 | 19:43 |
*** openstack has joined #openstack-lbaas | 20:01 | |
*** ankur-gupta-f2 has joined #openstack-lbaas | 20:01 | |
*** beardedeagle has joined #openstack-lbaas | 20:10 | |
*** openstackstatus has quit IRC | 20:27 | |
*** openstack has joined #openstack-lbaas | 20:31 | |
openstackgerrit | Shashank Kumar Shankar proposed openstack/octavia: [Super WIP] Introduce Octavia v2 API for pools https://review.openstack.org/405922 | 20:44 |
*** cody-somerville has quit IRC | 20:58 | |
*** cody-somerville has joined #openstack-lbaas | 21:54 | |
*** cody-somerville has quit IRC | 21:54 | |
*** cody-somerville has joined #openstack-lbaas | 21:54 | |
*** yamamoto has joined #openstack-lbaas | 21:56 | |
*** diltram has quit IRC | 21:56 | |
openstackgerrit | Sindhu Devale proposed openstack/octavia: Introduce Octavia v2 API for listeners https://review.openstack.org/405060 | 22:04 |
openstackgerrit | Shashank Kumar Shankar proposed openstack/octavia: Introduce Octavia v2 API for pools https://review.openstack.org/405922 | 22:05 |
openstackgerrit | Sindhu Devale proposed openstack/octavia: Align Octavia API to n-lbaasv2 for L7Policy https://review.openstack.org/406328 | 22:08 |
*** cody-somerville has quit IRC | 22:09 | |
openstackgerrit | Sindhu Devale proposed openstack/octavia: Align Octavia API to n-lbaasv2 for L7Rules https://review.openstack.org/406336 | 22:10 |
*** yamamoto has quit IRC | 22:11 | |
*** yamamoto has joined #openstack-lbaas | 22:13 | |
openstackgerrit | Shashank Kumar Shankar proposed openstack/octavia: [WIP] Introduce Octavia v2 API for Pools with Type, Model change https://review.openstack.org/420195 | 22:14 |
rm_work | johnsom: yep, but i'm back! | 22:14 |
rm_work | man, it's lively in here today | 22:14 |
rm_work | <_< | 22:14 |
johnsom | Yeah, we have rebooted the API work, so there should be lots of activity there.... | 22:14 |
*** yamamoto has quit IRC | 22:15 | |
johnsom | rm_work When you have a minute: https://review.openstack.org/420133 Nova puked on one of the tests, but the multinode passed in the first go round | 22:15 |
johnsom | We were getting multinode failures because the flavor wasn't in the secondary node conf | 22:16 |
rm_work | ah | 22:16 |
rm_work | kk | 22:16 |
rm_work | i was about to go through and help just fix merge conflicts on the large number of patches that fell over due to the quota merge | 22:17 |
*** yamamoto has joined #openstack-lbaas | 22:20 | |
*** yamamoto has quit IRC | 22:22 | |
*** yamamoto has joined #openstack-lbaas | 22:24 | |
rm_work | johnsom: yeah i see. this seems right | 22:25 |
rm_work | mind if I +2 with a recheck? | 22:25 |
johnsom | I already hit recheck, so just a +2 | 22:25 |
rm_work | ah k | 22:25 |
rm_work | in that case i'll just wait :P | 22:25 |
johnsom | Ok, fair enough. ~30 minutes | 22:26 |
*** yamamoto has quit IRC | 22:26 | |
*** ducttape_ has quit IRC | 22:27 | |
*** catintheroof has quit IRC | 22:38 | |
*** ducttape_ has joined #openstack-lbaas | 22:59 | |
*** beardedeagle has quit IRC | 23:09 | |
*** fnaval has quit IRC | 23:18 | |
*** chlong has quit IRC | 23:38 | |
*** eezhova has quit IRC | 23:39 | |
*** harlowja has quit IRC | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!