*** doug-fish has joined #openstack-lbaas | 00:01 | |
sbalukoff | johnsom: Joy! | 00:04 |
---|---|---|
*** outofmemory is now known as reedip | 00:05 | |
*** armax has quit IRC | 00:19 | |
*** armax_ has joined #openstack-lbaas | 00:19 | |
*** doug-fish has quit IRC | 00:22 | |
*** doug-fish has joined #openstack-lbaas | 00:24 | |
*** doug-fish has quit IRC | 00:25 | |
*** doug-fish has joined #openstack-lbaas | 00:26 | |
*** doug-fish has quit IRC | 00:27 | |
*** doug-fish has joined #openstack-lbaas | 00:30 | |
openstackgerrit | Merged openstack/octavia: Adding "region and endpoint_type" parameters to barbican_acl.py https://review.openstack.org/272861 | 00:35 |
*** doug-fish has quit IRC | 00:40 | |
*** Varun_Lodaya has joined #openstack-lbaas | 00:42 | |
*** ajmiller has quit IRC | 00:44 | |
*** doug-fish has joined #openstack-lbaas | 00:45 | |
*** allan_h has joined #openstack-lbaas | 00:55 | |
*** bank_ has quit IRC | 01:00 | |
*** allan_h has quit IRC | 01:12 | |
*** jwarendt has quit IRC | 01:13 | |
*** doug-fish has quit IRC | 01:13 | |
*** Varun_Lodaya has quit IRC | 01:14 | |
johnsom | Well, removing the workaround passed the gates (whatever that really means) | 01:21 |
johnsom | https://review.openstack.org/#/c/277602/ | 01:21 |
*** crc32 has quit IRC | 01:23 | |
*** Aish has quit IRC | 01:24 | |
*** rooney has joined #openstack-lbaas | 01:24 | |
*** Aish has joined #openstack-lbaas | 01:24 | |
*** Aish has quit IRC | 01:26 | |
*** rooney has quit IRC | 01:29 | |
*** ducttape_ has joined #openstack-lbaas | 01:53 | |
*** yamamoto has joined #openstack-lbaas | 01:59 | |
*** ducttape_ has quit IRC | 02:05 | |
*** ducttape_ has joined #openstack-lbaas | 02:05 | |
*** _ducttape_ has joined #openstack-lbaas | 02:08 | |
*** ducttape_ has quit IRC | 02:09 | |
*** doug-fish has joined #openstack-lbaas | 02:14 | |
*** doug-fish has quit IRC | 02:19 | |
*** madhu_ak has quit IRC | 02:21 | |
*** yamamoto has quit IRC | 02:24 | |
*** doug-fish has joined #openstack-lbaas | 02:25 | |
*** doug-fish has quit IRC | 02:45 | |
*** doug-fish has joined #openstack-lbaas | 02:46 | |
*** doug-fish has quit IRC | 02:50 | |
*** armax_ is now known as armax | 02:54 | |
*** sbalukoff has quit IRC | 03:00 | |
*** _ducttape_ has quit IRC | 03:02 | |
*** ducttape_ has joined #openstack-lbaas | 03:02 | |
*** ducttape_ has quit IRC | 03:04 | |
*** ducttape_ has joined #openstack-lbaas | 03:05 | |
ptoohill | https://review.openstack.org/#/c/275558/ | 03:05 |
*** yamamoto has joined #openstack-lbaas | 03:12 | |
*** fnaval has quit IRC | 03:25 | |
*** doug-fish has joined #openstack-lbaas | 03:32 | |
*** ajmiller has joined #openstack-lbaas | 03:34 | |
*** rooney has joined #openstack-lbaas | 03:36 | |
*** ajmiller has quit IRC | 03:37 | |
*** fnaval has joined #openstack-lbaas | 03:37 | |
*** rooney has quit IRC | 03:40 | |
*** links has joined #openstack-lbaas | 03:41 | |
*** ducttape_ has quit IRC | 03:51 | |
*** ducttape_ has joined #openstack-lbaas | 03:53 | |
*** doug-fish has quit IRC | 03:56 | |
*** doug-fish has joined #openstack-lbaas | 03:56 | |
*** doug-fish has quit IRC | 03:57 | |
*** doug-fis_ has joined #openstack-lbaas | 04:00 | |
*** ducttape_ has quit IRC | 04:01 | |
*** doug-fis_ has quit IRC | 04:04 | |
*** ducttape_ has joined #openstack-lbaas | 04:09 | |
*** doug-fish has joined #openstack-lbaas | 04:09 | |
*** armax_ has joined #openstack-lbaas | 04:18 | |
*** armax has quit IRC | 04:21 | |
*** armax_ is now known as armax | 04:21 | |
*** sbalukoff has joined #openstack-lbaas | 04:30 | |
*** allan_h has joined #openstack-lbaas | 04:44 | |
*** ducttape_ has quit IRC | 05:06 | |
*** amotoki has joined #openstack-lbaas | 05:07 | |
*** amotoki_ has joined #openstack-lbaas | 05:09 | |
*** amotoki has quit IRC | 05:12 | |
*** amotoki_ has quit IRC | 05:15 | |
*** allan_h has quit IRC | 05:28 | |
*** rooney has joined #openstack-lbaas | 05:34 | |
*** doug-fish has quit IRC | 05:48 | |
*** amotoki has joined #openstack-lbaas | 05:49 | |
*** doug-fish has joined #openstack-lbaas | 05:55 | |
*** rooney has quit IRC | 06:03 | |
*** doug-fish has quit IRC | 06:08 | |
*** numans has joined #openstack-lbaas | 06:19 | |
*** armax has quit IRC | 06:49 | |
openstackgerrit | Reedip proposed openstack/neutron-lbaas: [WIP]Add Timeout on Listeners- DO NOT REVIEW https://review.openstack.org/273896 | 06:56 |
*** amotoki has quit IRC | 06:56 | |
*** amotoki has joined #openstack-lbaas | 06:58 | |
openstackgerrit | Stephen Balukoff proposed openstack/octavia: Update repos for L7 policy / methods https://review.openstack.org/265529 | 07:11 |
openstackgerrit | Stephen Balukoff proposed openstack/octavia: Add L7 database structures https://review.openstack.org/265430 | 07:11 |
openstackgerrit | Stephen Balukoff proposed openstack/octavia: Update repos for L7 rules / validations https://review.openstack.org/276643 | 07:11 |
openstackgerrit | Merged openstack/octavia: Do not use __builtin__ in python3 https://review.openstack.org/262516 | 07:26 |
openstackgerrit | Stephen Balukoff proposed openstack/octavia: Add L7 api - policies https://review.openstack.org/265690 | 07:29 |
openstackgerrit | Stephen Balukoff proposed openstack/octavia: Add L7 api - rules https://review.openstack.org/277718 | 07:35 |
*** doug-fish has joined #openstack-lbaas | 07:38 | |
*** doug-fis_ has joined #openstack-lbaas | 07:42 | |
*** doug-fish has quit IRC | 07:43 | |
*** doug-fis_ has quit IRC | 07:47 | |
openstackgerrit | Reedip proposed openstack/neutron-lbaas: Add Timeout on Listeners- DO NOT REVIEW https://review.openstack.org/273896 | 07:49 |
*** nmagnezi has joined #openstack-lbaas | 07:49 | |
openstackgerrit | Merged openstack/octavia: Reverts a workaround that is impacting amp updates https://review.openstack.org/277602 | 08:19 |
*** jschwarz has joined #openstack-lbaas | 08:43 | |
*** jschwarz has quit IRC | 09:23 | |
*** jschwarz has joined #openstack-lbaas | 09:26 | |
openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Get Me A LB https://review.openstack.org/257201 | 09:38 |
openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Get Me A LB https://review.openstack.org/257201 | 09:43 |
openstackgerrit | Stephen Balukoff proposed openstack/octavia: Add L7 controller worker flows and tasks https://review.openstack.org/277768 | 10:17 |
*** amotoki has quit IRC | 10:34 | |
*** amotoki has joined #openstack-lbaas | 10:49 | |
*** amotoki has quit IRC | 10:55 | |
*** yamamoto has quit IRC | 11:05 | |
*** yamamoto has joined #openstack-lbaas | 11:51 | |
*** rtheis has joined #openstack-lbaas | 12:23 | |
*** openstackgerrit_ has joined #openstack-lbaas | 12:39 | |
*** doug-fish has joined #openstack-lbaas | 12:45 | |
*** yamamoto has quit IRC | 12:46 | |
*** doug-fish has quit IRC | 12:47 | |
*** doug-fish has joined #openstack-lbaas | 12:48 | |
*** ducttape_ has joined #openstack-lbaas | 12:53 | |
*** numans has quit IRC | 13:02 | |
*** yamamoto has joined #openstack-lbaas | 13:03 | |
*** woodster_ has quit IRC | 13:06 | |
*** ducttape_ has quit IRC | 13:10 | |
*** yamamoto has quit IRC | 13:16 | |
*** links has quit IRC | 13:17 | |
*** yamamoto has joined #openstack-lbaas | 13:23 | |
*** numans has joined #openstack-lbaas | 13:29 | |
*** numans has quit IRC | 13:34 | |
*** openstackgerrit_ has quit IRC | 13:56 | |
*** numans has joined #openstack-lbaas | 14:12 | |
*** localloop127 has joined #openstack-lbaas | 14:15 | |
*** links has joined #openstack-lbaas | 14:16 | |
*** numans has quit IRC | 14:20 | |
*** ducttape_ has joined #openstack-lbaas | 14:21 | |
*** ducttape_ has quit IRC | 14:21 | |
*** amotoki has joined #openstack-lbaas | 14:21 | |
*** numans has joined #openstack-lbaas | 14:22 | |
*** amotoki has quit IRC | 14:31 | |
*** yamamoto has quit IRC | 14:32 | |
*** yamamoto has joined #openstack-lbaas | 14:33 | |
*** amotoki has joined #openstack-lbaas | 14:35 | |
*** amotoki has quit IRC | 14:37 | |
*** yamamoto has quit IRC | 14:37 | |
*** prabampm1 has quit IRC | 14:39 | |
*** amotoki has joined #openstack-lbaas | 14:41 | |
*** amotoki has quit IRC | 14:41 | |
*** nmagnezi has quit IRC | 14:49 | |
*** amit213 has quit IRC | 14:51 | |
*** links has quit IRC | 14:54 | |
*** TrevorV has joined #openstack-lbaas | 14:54 | |
*** Bjoern has joined #openstack-lbaas | 14:55 | |
*** cgross has quit IRC | 14:56 | |
*** cgross has joined #openstack-lbaas | 14:56 | |
*** TrevorV has quit IRC | 14:57 | |
*** TrevorV has joined #openstack-lbaas | 15:01 | |
*** ducttape_ has joined #openstack-lbaas | 15:01 | |
*** ducttape_ has quit IRC | 15:02 | |
*** ducttape_ has joined #openstack-lbaas | 15:02 | |
*** fnaval has quit IRC | 15:05 | |
*** yamamoto has joined #openstack-lbaas | 15:07 | |
*** TrevorV has quit IRC | 15:11 | |
*** TrevorV has joined #openstack-lbaas | 15:12 | |
*** barclaac has quit IRC | 15:13 | |
*** barclaac has joined #openstack-lbaas | 15:13 | |
*** neelashah has joined #openstack-lbaas | 15:13 | |
*** ajmiller has joined #openstack-lbaas | 15:17 | |
*** numans has quit IRC | 15:20 | |
*** piet has joined #openstack-lbaas | 15:22 | |
*** numans has joined #openstack-lbaas | 15:23 | |
*** prabampm has joined #openstack-lbaas | 15:29 | |
*** neelashah has quit IRC | 15:38 | |
*** numans has quit IRC | 15:43 | |
*** woodster_ has joined #openstack-lbaas | 15:45 | |
*** yamamoto has quit IRC | 15:48 | |
TrevorV | Anyone online, can we get some more attention on: | 16:06 |
TrevorV | https://review.openstack.org/#/c/277912 | 16:06 |
TrevorV | Its an openstack/requirements issue that is currently breaking devstack | 16:07 |
*** armax has joined #openstack-lbaas | 16:08 | |
johnsom | It looks like it has been +A'd | 16:13 |
TrevorV | Yeah, johnsom happened right after I reviewed it and linked it ha ha | 16:20 |
xgerman | I don’t have +A in that project ;-) | 16:24 |
*** _ducttape_ has joined #openstack-lbaas | 16:26 | |
*** jschwarz has quit IRC | 16:29 | |
*** ducttape_ has quit IRC | 16:29 | |
*** _ducttape_ has quit IRC | 16:36 | |
*** ducttape_ has joined #openstack-lbaas | 16:37 | |
*** piet has quit IRC | 16:40 | |
*** TrevorV has quit IRC | 16:40 | |
*** TrevorV has joined #openstack-lbaas | 16:42 | |
*** neelashah has joined #openstack-lbaas | 16:44 | |
*** piet has joined #openstack-lbaas | 16:46 | |
*** neelashah has quit IRC | 16:54 | |
*** Aish has joined #openstack-lbaas | 16:59 | |
*** minwang2 has joined #openstack-lbaas | 16:59 | |
kevinbenton | xgerman: hey, how do you set a default flavor? | 17:03 |
kevinbenton | xgerman: or is it even possible? | 17:03 |
*** fnaval has joined #openstack-lbaas | 17:03 | |
xgerman | in that case you would just use the default provider | 17:03 |
xgerman | jwarendt is working on some more documentation :-) | 17:03 |
xgerman | flavor is basically to put an end user readable name on providers + some additional flecibility | 17:04 |
kevinbenton | so then a created thing might not have a flavor at all then? | 17:04 |
*** neelashah has joined #openstack-lbaas | 17:07 | |
*** neelashah1 has joined #openstack-lbaas | 17:08 | |
kevinbenton | xgerman: ^^ | 17:09 |
xgerman | not necessarily | 17:10 |
*** piet has quit IRC | 17:10 | |
*** jwarendt has joined #openstack-lbaas | 17:11 | |
*** neelashah has quit IRC | 17:11 | |
kevinbenton | xgerman: so the lookup logic is: lookup service provider for flavor, if no flavor, use default service provider | 17:13 |
johnsom | kevinbenton jwarendt is now online in this channel. He can probably answer you questions | 17:13 |
jwarendt | kevinbenton: Correct | 17:13 |
jwarendt | That is current logic. | 17:13 |
kevinbenton | i see, thanks | 17:14 |
*** bana_k has joined #openstack-lbaas | 17:14 | |
kevinbenton | jwarendt: load balancer API doesn't allow choosing a service provider directly, does it? | 17:16 |
kevinbenton | jwarendt: it's always indirect via a flavor | 17:16 |
kevinbenton | right? | 17:16 |
xgerman | no, we let you choose it directly | 17:16 |
jwarendt | kevinbenton: Still has the --provider flag as well, for explicit provider | 17:16 |
jwarendt | Predated flavors. | 17:17 |
kevinbenton | jwarendt: other than legacy support, do you think that has a good use case? | 17:17 |
xgerman | operators? | 17:18 |
kevinbenton | xgerman: why wouldn't they use a flavor? | 17:18 |
jwarendt | I prefer the flavor operator-controlled indirection, obviously. But flavor requires setup unless we automate the flavor creation/profile bindings based on providers. | 17:18 |
kevinbenton | makes sense | 17:19 |
xgerman | kevinbenton they might be evaluating a provider to turn it into a flavor — this allows for a quicker turn around | 17:19 |
kevinbenton | xgerman: is it not possible to add a flavor but only have operators be able to use it? | 17:19 |
ptoohill | https://review.openstack.org/#/c/275558/7 | 17:20 |
kevinbenton | like admin-only flavors or something? | 17:20 |
xgerman | I don;t think we have that functionality | 17:20 |
*** allan_h has joined #openstack-lbaas | 17:20 | |
kevinbenton | xgerman: ok. Makes sense then | 17:21 |
kevinbenton | Deciding if l3 should allow that as well | 17:21 |
kevinbenton | Probably should for operator only stuff | 17:21 |
xgerman | yeah, we like to allow for that flexibility | 17:23 |
kevinbenton | jwarendt: so is a load balancers flavor and service provider stored in the schema? | 17:24 |
xgerman | I think we store both | 17:24 |
jwarendt | The flavor id is stored with the lb, as is the provider. | 17:25 |
kevinbenton | It would seem necessary since a flavor could have multiple servic profiles | 17:25 |
jwarendt | The flavor->service profile bindings are separate db structures; the lbaas only stores the flavor and the chosen provider. | 17:26 |
jwarendt | At present. | 17:26 |
kevinbenton | Right | 17:27 |
kevinbenton | Ok. That's all I needed for now | 17:27 |
kevinbenton | Thx | 17:27 |
xgerman | y.w. | 17:33 |
*** _cjones_ has joined #openstack-lbaas | 17:34 | |
*** fnaval has quit IRC | 17:34 | |
*** ajmiller has quit IRC | 17:39 | |
*** minwang2 has quit IRC | 17:40 | |
*** ajmiller has joined #openstack-lbaas | 17:42 | |
*** prabampm has quit IRC | 17:44 | |
*** fnaval has joined #openstack-lbaas | 17:58 | |
rm_work | is this one of our known intermittent failures? http://logs.openstack.org/91/259391/3/check/gate-neutron-lbaasv2-dsvm-scenario/295e7b5/console.html.gz#_2016-01-08_10_49_18_183 | 17:59 |
rm_work | could be the third one you mentioned that i haven't actually seen yet? | 18:00 |
blogan | so slooooowwww | 18:03 |
blogan | rm_work: yeah i've seen it | 18:04 |
*** minwang2 has joined #openstack-lbaas | 18:05 | |
*** woodster_ has quit IRC | 18:06 | |
johnsom | dougwig Thoughts on the comment here: https://review.openstack.org/#/c/274319/ | 18:15 |
dougwig | i was torn between thinking it was style and wondering if both can exist at the same time, and then being annoyed at having to go look, so fuck it. i think it's fine/readable as is. | 18:16 |
johnsom | Ok, yeah, I don't have enough background on that. | 18:16 |
johnsom | I will go solicit the +A then. | 18:16 |
johnsom | blogan ajmiller ptoohill https://review.openstack.org/#/c/274319/ Gate speedup | 18:17 |
ajmiller | johnsom: coincidentally I had just been looking at it. | 18:18 |
ajmiller | Done. | 18:18 |
*** Aish has quit IRC | 18:18 | |
johnsom | Cool, thanks | 18:18 |
*** madhu_ak has joined #openstack-lbaas | 18:19 | |
*** bana_k has quit IRC | 18:27 | |
*** Aish has joined #openstack-lbaas | 18:31 | |
*** ajmiller has quit IRC | 18:33 | |
*** ajmiller has joined #openstack-lbaas | 18:36 | |
*** bana_k has joined #openstack-lbaas | 18:38 | |
*** neelashah1 has quit IRC | 18:38 | |
rm_work | yeah i looked at ENABLE/DISABLE a while back and could never figure out exactly what advantage one has over the other, or how it chooses which to use if you used both? | 18:46 |
rm_work | I think it's style / convenience, blacklist vs. whitelist both supported <_< | 18:46 |
ptoohill | crhttps://review.openstack.org/#/c/275558/ | 18:47 |
ptoohill | https://review.openstack.org/#/c/275558/ | 18:47 |
TrevorV | johnsom I'm having an interesting issue here... | 18:53 |
johnsom | Ok. I have ~7 minutes | 18:54 |
rm_work | ptoohill: +2'd, but it's just rax on it right now, waiting for a +A from someone non-rax T_T | 18:54 |
TrevorV | My unit tests are failing, saying "<Class> object has no attribute 'assertRaises'" | 18:54 |
johnsom | Yeah, I will take a look later today (I hope) | 18:54 |
rm_work | ah TrevorV where is that | 18:54 |
TrevorV | Its on EVERY unit test I'm running right now in Pycharm | 18:54 |
rm_work | something probably isn't the right kind of mock/baseclass | 18:54 |
rm_work | ack | 18:54 |
rm_work | that seems bad | 18:54 |
rm_work | does it NOT do it in tox? | 18:54 |
TrevorV | However, when I run "tox -e py27" it throws a real error... | 18:54 |
rm_work | ah | 18:55 |
rm_work | your environment in pycharm may not be set correctly, sometimes it gets "off" somehow on specific runs | 18:55 |
rm_work | check the options and make sure it's using project default and that the project default is right | 18:55 |
openstackgerrit | Merged openstack/neutron-lbaas: Optimize API gate devstack build time https://review.openstack.org/274319 | 18:55 |
rm_work | sometimes mine get messed up randomly | 18:55 |
TrevorV | rm_work I'm not sure entirely where I should be looking though | 18:55 |
TrevorV | You mean project interpreter? | 18:55 |
TrevorV | Or Run/Debug configs? | 18:56 |
rm_work | click the name of the run up by the Play/Debug buttons | 18:56 |
rm_work | run/debug config | 18:56 |
rm_work | Edit Configurations... | 18:56 |
TrevorV | Yeah | 18:56 |
rm_work | is Python Interpreter set to Project Default? | 18:56 |
TrevorV | Its set to the .tox one | 18:56 |
rm_work | py2 or py3, out of curiosity | 18:56 |
TrevorV | py2 | 18:57 |
rm_work | well i am about to head in | 18:57 |
TrevorV | it doesn't have to be, that's just what I've done so far | 18:57 |
rm_work | so if you don't figure it out in like 30m | 18:57 |
rm_work | i'll take a look | 18:57 |
TrevorV | Kk | 18:57 |
rm_work | i have def seen that before | 18:57 |
rm_work | oh hey, voting open for Austin talks? | 18:57 |
TrevorV | I'm not sure. | 18:58 |
rm_work | bbl | 18:58 |
*** neelashah has joined #openstack-lbaas | 18:59 | |
TrevorV | kk | 18:59 |
*** Bjoern has quit IRC | 19:21 | |
TrevorV | sbalukoff hey man you around? | 19:22 |
dougwig | johnsom: for the octavia meeting this week, can we set aside time to go over all the blueprints/rfes/m-3 bugs for neutron-lbaas and octavia, and where we're at with them? I need to know which to kick into newton. | 19:27 |
johnsom | Ok | 19:27 |
johnsom | I think we have one blueprint in neutron for L7 | 19:27 |
TrevorV | Think I found the answer sbalukoff no hurry man | 19:28 |
TrevorV | Well now I'm seeing a ton of errors that don't quite make sense. | 19:32 |
TrevorV | All of a sudden these tests are getting "None" values for things that I doubt have had them for a while | 19:32 |
TrevorV | And a bunch of reports of 500 response codes... | 19:33 |
TrevorV | Is this due to the mock stuff you guys had issues with a while back? | 19:33 |
TrevorV | Nevermind, I'm just going to revert my simple changes and double check that it wasn't me | 19:34 |
*** rooney has joined #openstack-lbaas | 19:35 | |
TrevorV | Alright, looks like its something I did. | 19:36 |
TrevorV | Alrighty. Time to figure out what I did wrong. | 19:36 |
*** crc32 has joined #openstack-lbaas | 19:54 | |
mhayden | blogan: ah, you were right on the security group -- port 443 isn't working yet, but i'm still digging | 20:03 |
mhayden | blogan: if i do a curl within the network namespace for the LB, i can connect to port 80 *and* port 443 without issue | 20:04 |
*** neelashah has quit IRC | 20:05 | |
*** rcernin has joined #openstack-lbaas | 20:08 | |
*** neelashah has joined #openstack-lbaas | 20:08 | |
*** piet has joined #openstack-lbaas | 20:09 | |
blogan | mhayden: ah, just can't from the global namespace? | 20:11 |
blogan | err default namespace, whatever its called | 20:11 |
mhayden | i'm on another instance on the same vxlan | 20:11 |
mhayden | and i can reach the nodes directly on 443 without issues | 20:11 |
blogan | hmmm | 20:11 |
mhayden | the traffic *is* making it to the network namespace though | 20:11 |
mhayden | verified that with tcpdump | 20:12 |
*** piet has quit IRC | 20:13 | |
*** piet has joined #openstack-lbaas | 20:13 | |
mhayden | i think haproxy is choking on something | 20:14 |
mhayden | the strace looks odd | 20:14 |
*** Varun_Lodaya has joined #openstack-lbaas | 20:23 | |
blogan | mhayden: but if haproxy is able to handle the traffic from within the namespace, it should be working fine | 20:38 |
mhayden | hmm, so i backed out the members, pool and listener | 20:38 |
mhayden | then jammed them back in | 20:38 |
*** SumitNaiksatam has joined #openstack-lbaas | 20:38 | |
mhayden | it seems to work! | 20:38 |
blogan | just in the haproxy config? or through the api | 20:39 |
blogan | ? | 20:39 |
mhayden | API | 20:39 |
blogan | and it works totally now? | 20:39 |
mhayden | that was... odd | 20:39 |
blogan | sounds like a great user workflow, create all your stuff, then delete htem, then add them back | 20:39 |
blogan | works as intended! | 20:39 |
mhayden | WOOT | 20:41 |
mhayden | i'll retry and see what happens | 20:41 |
mhayden | the first time, i copied/pasted the neutron commands | 20:42 |
mhayden | the second time, i did it a little more slowly | 20:42 |
mhayden | i wonder if there's a timing bug in there somewhere | 20:42 |
mhayden | but i did find a possible bug -- i can delete members, pool, and listener without deleting the healthmonitor | 20:43 |
mhayden | but then the health monitor is in an undeletable state | 20:43 |
mhayden | because it has no pools attached | 20:43 |
blogan | mhayden: on liberty? | 20:44 |
mhayden | yes | 20:44 |
blogan | mhayden: seems like an obvious bug, wonder if its been fixed | 20:45 |
blogan | sounds kind of familiar, may just need a backport | 20:45 |
mhayden | blogan: i'll be glad to open a bug in LP | 20:45 |
blogan | mhayden: do it! | 20:45 |
mhayden | which project? | 20:45 |
blogan | neutron, tag with lbaas | 20:45 |
mhayden | in progress.. | 20:45 |
mhayden | https://bugs.launchpad.net/neutron/+bug/1450375 | 20:46 |
openstack | Launchpad bug 1450375 in neutron "cannot delete v2 healthmonitor if the hm-associated-pool was deleted first" [Undecided,In progress] - Assigned to LIU Yulong (dragon889) | 20:46 |
mhayden | that looks like it, actually | 20:46 |
*** piet has quit IRC | 20:46 | |
blogan | mhayden: well its in progress lol | 20:49 |
mhayden | haha | 20:50 |
blogan | time to review that sucker, just got resurrected | 20:50 |
blogan | 2 weeks ago... | 20:50 |
mhayden | woot | 20:50 |
mhayden | i would be glad to gander, but i'd need to get familiar with neutron's innards first | 20:51 |
blogan | mhayden: you won't if you value your soul | 20:51 |
*** crc32 has quit IRC | 20:56 | |
*** neelashah has quit IRC | 21:16 | |
mhayden | blogan: so the security group didn't appear to matter | 21:20 |
mhayden | but i'm going to tear it all down and rebuild it to verify | 21:21 |
blogan | mhayden: it may not, probably because that vip port doesn't get attached to a VM, so it does't have a host/hypervisor that manages security groups, but i coudl be wrong, i'm just making uneducated guesses here | 21:22 |
mhayden | gotcha | 21:22 |
blogan | but pretty sure its because the vip port is being used on some agent host that obviously wouldn't have a networking agent to manage those ip table rules | 21:23 |
TrevorV | Anyeone else see this issue and know a fix? https://gist.github.com/anonymous/4ba8be1a065e5962eb0a | 21:25 |
johnsom | New one to me | 21:26 |
johnsom | TrevorV http://stackoverflow.com/questions/12805044/python-console-importerror-pyunicodeucs4-fromstring | 21:28 |
johnsom | Did you see that? | 21:28 |
TrevorV | I did. | 21:28 |
TrevorV | johnsom I was looking through this initially, but there is no "fix" here. Just comments about the problem. | 21:30 |
TrevorV | I'm not compiling something, I'm trying to run tox -e py27 | 21:31 |
TrevorV | Well... that may be me misunderstanding exactly what's happening when I run that. | 21:31 |
TrevorV | I guess. | 21:31 |
johnsom | It sounds like the package was compiled with a different version of python | 21:31 |
johnsom | ptoohill Around? | 21:45 |
ptoohill | johnsom: I am | 21:45 |
johnsom | How do you feel about moving the jinja template stuff into controller/worker/utils? | 21:46 |
ptoohill | Totally ok with that, and was something I was thinking about | 21:46 |
johnsom | The tasks directory is really for the taskflow tasks | 21:46 |
ptoohill | Agreed, i was just following others setup | 21:46 |
johnsom | Cool. It would just keep our directory tree clean | 21:46 |
ptoohill | maybe at somepoint we can move all the other templates to a common spot | 21:47 |
johnsom | Ok, that is the only comment I had for the patch. +2 after that | 21:47 |
johnsom | Yeah, I'm open to starting a wider directory for that now if you're up for it | 21:47 |
ptoohill | Maybe a common/templates? | 21:48 |
johnsom | That was what I was thinking too | 21:48 |
ptoohill | Ill just put my stuff there for now | 21:48 |
johnsom | Start a trend.... | 21:48 |
ptoohill | :) | 21:48 |
ptoohill | I'll get that out | 21:48 |
*** neelashah has joined #openstack-lbaas | 21:49 | |
*** Bjoern has joined #openstack-lbaas | 21:51 | |
sbalukoff | johnsom: Since the next step in L7 is to update all the jinja templates for generating haproxy configs, should I move them there as well? | 21:52 |
johnsom | If you're up for the work, sure that would be cool | 21:52 |
sbalukoff | Eh... I've got to mess with pretty much everything there anyway. | 21:53 |
johnsom | I expect we can get ptoohill's patch in today, so you shouldn't have to do a dependency | 21:53 |
sbalukoff | Sweet. | 21:53 |
*** rooney has quit IRC | 21:54 | |
johnsom | sbalukoff What is the link for the L7 etherpad? I keep forgetting to add it to the meeting agenda | 21:54 |
sbalukoff | https://etherpad.openstack.org/p/lbaas-l7-todo-list | 21:55 |
ptoohill | Testing it out | 21:55 |
johnsom | thanks | 21:55 |
sbalukoff | TrevorV: What was the issue you were running into earlier about which you tried to ping me? (Sorry-- was out at a doctor's appointment earlier.) | 21:55 |
TrevorV | Ummm... I forget. | 21:56 |
TrevorV | You took too long | 21:56 |
sbalukoff | Yeah, I do that. | 21:56 |
ptoohill | You want to -1 the patch johnsom ? | 21:57 |
ptoohill | force me to do it :) | 21:57 |
johnsom | Sure | 21:57 |
johnsom | Done | 21:58 |
ptoohill | sbalukoff: If youre going through updating all the jinja configs it would be good to get them in a common place, if you don't mind. But dont think that would be a -1 if you didnt | 21:58 |
ptoohill | ty | 21:58 |
johnsom | You just want our conflict numbers to go up.... grin | 22:00 |
ptoohill | :) | 22:00 |
ptoohill | helping out where i can | 22:00 |
ptoohill | :P | 22:00 |
sbalukoff | ptoohill: Again, I don't think it'll be significantly more work than I already have to do there. :) | 22:02 |
ptoohill | :) You say that, then it turns into nightmare | 22:03 |
ptoohill | Just trying to jinx ya ;) | 22:03 |
sbalukoff | Heh! Thanks! | 22:04 |
ptoohill | No problem, glad I could help! | 22:04 |
ptoohill | :P | 22:05 |
johnsom | sbalukoff Do you want me to assign these shared pools related bugs to you? | 22:10 |
sbalukoff | johnsom: Sure! | 22:12 |
johnsom | Ok. I am going to tag them too. I just don't want them to get lost, etc. | 22:12 |
sbalukoff | Thanks. | 22:14 |
johnsom | dougwig Question about the octavia launchpad bugs. I see Martin Hickey tagging octavia bugs with "lbaas". Is octavia repo included in the neutron bug scrub now? | 22:18 |
openstackgerrit | Phillip Toohill proposed openstack/octavia: Allow user-data on amphora creation https://review.openstack.org/275558 | 22:19 |
ptoohill | Alrighty, That should be good | 22:19 |
johnsom | ptoohill Thanks, looking now | 22:19 |
ptoohill | i may have missed tests >< | 22:20 |
ptoohill | doh | 22:20 |
ptoohill | johnsom: Not ready | 22:20 |
*** localloop127 has quit IRC | 22:20 | |
ptoohill | tested manual, forgot about units | 22:20 |
johnsom | Helping to pad my review stats | 22:21 |
ptoohill | :) | 22:21 |
johnsom | You can set the topic to bug/1541231 as well. Not sure if it matters, but that is the format in the dev guide | 22:22 |
ptoohill | Yea, not sure why it doesnt pull that with it, i had a rebase conflict, while inside branch and then this happens | 22:23 |
ptoohill | and it even says it pushed to the branch, i dont get it | 22:24 |
ptoohill | * [new branch] HEAD -> refs/publish/master/bug-1541231 | 22:24 |
dougwig | johnsom: no, not to my knowledge | 22:24 |
johnsom | Ok, just trying to be an over-achiever then. | 22:24 |
ptoohill | it looks like the topic is right hee | 22:25 |
johnsom | dougwig For reference: https://bugs.launchpad.net/octavia/+bug/1543356 I removed the lbaas as I didn't want folks to confuse the octavia API vs. lbaas | 22:26 |
openstack | Launchpad bug 1543356 in octavia "Errors found for octavia api----get listener stats" [Medium,New] - Assigned to Stephen Balukoff (sbalukoff) | 22:26 |
*** ducttape_ has quit IRC | 22:31 | |
openstackgerrit | Phillip Toohill proposed openstack/octavia: Allow user-data on amphora creation https://review.openstack.org/275558 | 22:34 |
openstackgerrit | German Eichberger proposed openstack/neutron-lbaas: [WIP] Adds Cascade option for load balancer drivers https://review.openstack.org/268237 | 22:34 |
ptoohill | Ok johnsom, I think that's good now | 22:35 |
ptoohill | :) | 22:35 |
xgerman | blogan I need some tutoring on that. That LBaaS V2 code is a dungeon | 22:38 |
xgerman | we definitely need some devrefs | 22:38 |
blogan | xgerman: we sure do | 22:38 |
xgerman | well, I am lost — not sure why my tests fails? why plugin doesn’t have cascade... | 22:38 |
xgerman | might want to give a talk on how that works :-) | 22:39 |
openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Get Me A LB https://review.openstack.org/257201 | 22:40 |
blogan | xgerman: all of 2 people would show up to that talk | 22:41 |
xgerman | well, I might need some tutoring - then whatever we did at the mid cycle doesn’t pass unit tests | 22:42 |
johnsom | 3 people | 22:43 |
fnaval | i'll attend too. | 22:44 |
fnaval | hmmm. that was a sarcastic remark i think | 22:44 |
xgerman | guess it’s a crowd :-) | 22:44 |
xgerman | blogan webinar? | 22:44 |
johnsom | We all just like to watch blogan be uncomfortable presenting.... | 22:44 |
blogan | i think thats the real reason, xgerman really has cascade delete working | 22:45 |
xgerman | honestly, the inner working of LBaaS V2 are completely lost on me | 22:45 |
blogan | xgerman: its mainling the black magic that is neutron extension loading and service plugin loading | 22:45 |
blogan | mainly | 22:45 |
xgerman | black magic there we go - I was supposed to sacrifice a goat before running too? | 22:46 |
fnaval | yeah i was gonna ask about that. if we're working on 1 load balancer create call, should there be 1 load balancer delete (everything) call? | 22:46 |
blogan | xgerman: indeed | 22:46 |
*** rtheis has quit IRC | 22:46 | |
fnaval | and update too | 22:46 |
blogan | fnaval: that is what xgerman is doing and what every UI team wants | 22:46 |
blogan | update is low on priority | 22:46 |
fnaval | kk blogan thanks | 22:46 |
blogan | UI team should just use heat | 22:47 |
blogan | amirite? | 22:47 |
*** TrevorV has quit IRC | 22:49 | |
xgerman | bloagn sure — or ansible | 22:50 |
xgerman | blogan | 22:50 |
* johnsom throws something heavy and *hot* at blogan | 22:50 | |
xgerman | or somebody ought to start oslo-orchestration | 22:50 |
blogan | xgerman: do you have your code up? | 22:50 |
xgerman | yep, but it’s broken and I don’t know why | 22:50 |
blogan | link me | 22:50 |
xgerman | https://review.openstack.org/268237 [WIP] Adds Cascade option for load balancer drivers | 22:51 |
blogan | broken on neutron-server start or when trying to actually delete? | 22:51 |
xgerman | broken when delete in unit tests | 22:51 |
xgerman | it just adds the option | 22:51 |
xgerman | drivers are later | 22:51 |
xgerman | it will try cascade and i that fails do the old thing | 22:52 |
xgerman | to some degree | 22:52 |
xgerman | https://www.irccloud.com/pastebin/hUl6s0WU/ | 22:53 |
xgerman | this pains me | 22:53 |
xgerman | there must be some magic piece I am missing | 22:54 |
blogan | nothing is throwing the TypeError exception though | 22:55 |
xgerman | it sure did - I had like 30 failures in the unit tests without | 22:55 |
blogan | hmm | 22:55 |
xgerman | yeah, there must be another place we need to add cascade=True or False | 22:56 |
blogan | ill test it out real quick | 22:56 |
xgerman | tahnks | 22:56 |
xgerman | I hope neutron-lib will make that ll ten times cleaner | 22:57 |
blogan | bwahahaha | 22:57 |
blogan | neutron lib will make a lot of things cleaner but i don't think this will be one of them | 22:57 |
*** rcernin has quit IRC | 22:58 | |
xgerman | maybe pecan then? | 22:58 |
xgerman | or do we need another project? | 22:59 |
blogan | oh there's the problem | 23:04 |
blogan | need to add **kwargs to the _call_driver_operation method | 23:04 |
blogan | and pass that to the actualy driver method call | 23:04 |
blogan | xgerman: also its not using the noop drivers it appears | 23:08 |
blogan | xgerman: nvm too quick on that assertion | 23:08 |
xgerman | yeah, this is like a riverwalk with snakes instead of water | 23:09 |
ptoohill | on fire? | 23:10 |
ptoohill | that's a new feature | 23:10 |
blogan | alright unit test passes xgerman | 23:14 |
blogan | ill push up the changes | 23:15 |
xgerman | awesome!! | 23:15 |
xgerman | What did I do wrong? | 23:15 |
blogan | since no driver method had taken a kwarg before, the _call_driver_operation didn't expect one, so it needed to be able to do that | 23:15 |
blogan | then within the tests you create a load baalncer and listener, but those creation contexts (using with keyword) automatically delete themselves with that context is left, so needed to pass the no_delete kwarg so it doesn't try to clean itself up | 23:16 |
openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: [WIP] Adds Cascade option for load balancer drivers https://review.openstack.org/268237 | 23:16 |
xgerman | cool | 23:17 |
blogan | you'll have to change the logic of how you determine whether a driver supports the cascade and if it does call the old way | 23:17 |
xgerman | mmh, so TypeError no good | 23:17 |
xgerman | I guess the driver would ignore the kwarh and just fail since not cascading | 23:18 |
xgerman | that logic is in there | 23:18 |
blogan | TypeError was being thrown because tha tmethod wasn't expecting a keyword argument | 23:18 |
xgerman | ok, I remove type error and we are all good ;-) | 23:18 |
blogan | but old drivers that know about the cascade probably do not have their delete method taking **kwargs, so it'll fail on old drivers | 23:19 |
blogan | that dont know | 23:19 |
xgerman | TypeError to the rescue :-) | 23:20 |
blogan | TypeError is too vague! | 23:20 |
blogan | i guess it'll work damnit | 23:20 |
xgerman | well, https://hynek.me/articles/hasattr/ suggests to use TypeError instead of the more fancy stuff | 23:21 |
xgerman | or in his case AttributeError — guess I am oversimolifying | 23:21 |
*** yamamoto_ has joined #openstack-lbaas | 23:21 | |
blogan | i was thinking more that you could call a driver.cascade_delete method, and if it raises NotImplemented, then faill back to the old way | 23:22 |
xgerman | mmh, probably something to ask our resident vendor driver expert dougwig | 23:23 |
*** yamamoto_ has quit IRC | 23:23 | |
blogan | which sounds more appropriate, you'd just have to make the base load balancer manager have the method cascade_delete and raise NotImplemented | 23:23 |
blogan | so now every driver gets taht method for free, and only those who do implement it can | 23:24 |
xgerman | yeah, I can see that… also my way would work in a more ugly fashion... | 23:24 |
xgerman | Thanks, I can add that more clean way | 23:25 |
blogan | also, got other comments, ill add them to the review though | 23:25 |
xgerman | cool | 23:25 |
blogan | specifically about the extension loading | 23:25 |
xgerman | dang — guess I have my work cut out | 23:26 |
blogan | added to the previous PS bc i didnt feel like rewriting | 23:26 |
blogan | nah they're simple, just inside knowledge of the extension loading basically | 23:26 |
xgerman | oh, I was afraid I would have become like gorge and need to go back to programming school | 23:26 |
xgerman | gorge=jorgem | 23:27 |
blogan | gorge works | 23:27 |
blogan | jorgem was his programming name, his manager name is gorge | 23:27 |
xgerman | lol | 23:27 |
blogan | me pushign up that review auto-assigned me to that bug! | 23:33 |
blogan | xgerman: you tricked me! | 23:33 |
ptoohill | lol | 23:33 |
ptoohill | oh noes | 23:34 |
blogan | well i removed myself, but can't readd you xgerman, it can't find your name, nor can it find mine | 23:35 |
*** allan_h has quit IRC | 23:39 | |
*** neelashah has quit IRC | 23:44 | |
xgerman | bug without a name | 23:46 |
*** Bjoern has quit IRC | 23:46 | |
*** yamamoto_ has joined #openstack-lbaas | 23:50 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!