| johnsom | Ah, ok, if you don't have an OpenStack account, you can always file a but like you did and we can make the needed changes. | 00:00 |
|---|---|---|
| Purandar | I get redirected from launchpad to wiki.openstack after that when it just provides prompt to login, nothing happens clicking on log in | 00:02 |
| johnsom | Hmm, not sure what to tell you on that. | 00:04 |
| rm_you | yeah, launchpad(bug) account should be the same account as the wiki :/ | 00:15 |
| Purandar | but for some reason it doesn't allow to login after redirection from launchpad site | 00:17 |
| rm_you | try directly to https://wiki.openstack.org/w/index.php?title=Special:OpenIDLogin&returnto=Network/LBaaS/docs/how-to-create-tls-loadbalancer | 00:19 |
| xgerman | blogan and dougwig if one of you can look at the cascade delete LBaaS stuff please ;-) | 00:20 |
| Purandar | tried but same behavior, it first lands me to launchpad then get redirected to wiki.openstack and still displays log in button again, with message above saying 'All users need a nickname; you can choose one from the options below. ' | 00:23 |
| Purandar | but there is nothing below to select other than log in & cancel button. | 00:23 |
| rm_you | try clearing your browser cache / delete cookies | 00:29 |
| *** ducttape_ has joined #openstack-lbaas | 00:32 | |
| *** piet has joined #openstack-lbaas | 00:34 | |
| *** ajmiller has quit IRC | 00:43 | |
| Purandar | tried, but didn't work either. | 00:46 |
| *** Aish has quit IRC | 00:49 | |
| *** ducttape_ has quit IRC | 00:55 | |
| *** minwang2 has quit IRC | 01:01 | |
| *** madhu_ak has quit IRC | 01:07 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: WIP - delete lbaasv2 agent driver https://review.openstack.org/286380 | 01:12 |
| openstackgerrit | Banashankar k proposed openstack/octavia: Adding the topology info at the time of LB AMP association https://review.openstack.org/286365 | 01:17 |
| *** ducttape_ has joined #openstack-lbaas | 01:20 | |
| *** ducttape_ has quit IRC | 01:21 | |
| *** ducttape_ has joined #openstack-lbaas | 01:21 | |
| *** piet has quit IRC | 01:29 | |
| *** mixos has joined #openstack-lbaas | 01:29 | |
| *** paco20151113 has joined #openstack-lbaas | 01:40 | |
| *** fawadkhaliq has quit IRC | 01:42 | |
| *** yamamoto has joined #openstack-lbaas | 01:45 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: WIP - delete lbaasv2 agent driver https://review.openstack.org/286380 | 01:48 |
| *** ducttape_ has quit IRC | 01:49 | |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Add maximum number of l7rules per l7policy https://review.openstack.org/285160 | 01:51 |
| *** ducttape_ has joined #openstack-lbaas | 01:54 | |
| *** ducttape_ has quit IRC | 01:58 | |
| *** manishg has quit IRC | 01:58 | |
| *** ducttape_ has joined #openstack-lbaas | 02:00 | |
| *** woodster_ has quit IRC | 02:06 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: WIP - nuke lbaas v1 https://review.openstack.org/286381 | 02:08 |
| *** ducttape_ has quit IRC | 02:11 | |
| *** bana_k has quit IRC | 02:14 | |
| *** bhaargavi has quit IRC | 02:14 | |
| *** paco20151113 has quit IRC | 02:18 | |
| *** paco20151113 has joined #openstack-lbaas | 02:20 | |
| *** bana_k has joined #openstack-lbaas | 02:22 | |
| *** ducttape_ has joined #openstack-lbaas | 02:22 | |
| *** ducttape_ has quit IRC | 02:32 | |
| *** ducttape_ has joined #openstack-lbaas | 02:35 | |
| *** yamamoto has quit IRC | 02:36 | |
| *** bana_k has quit IRC | 02:45 | |
| *** ducttape_ has quit IRC | 02:45 | |
| *** Purandar has quit IRC | 02:46 | |
| *** kevo has quit IRC | 02:47 | |
| *** prabampm1 has quit IRC | 02:49 | |
| *** manishg has joined #openstack-lbaas | 02:57 | |
| blogan | i merged it? | 03:22 |
| *** yamamoto has joined #openstack-lbaas | 03:22 | |
| rm_you | I did +1 with a note that i was testing it in devstack and you followed with a +A :P | 03:22 |
| rm_you | no worries | 03:22 |
| blogan | i do not remember this at all | 03:23 |
| blogan | i wasn't even around a computer at that time | 03:23 |
| rm_you | blogan: https://review.openstack.org/#/c/284690/ | 03:23 |
| blogan | oh right before i left | 03:23 |
| blogan | ok i need my core status revoked, i dont remember reviewing that | 03:23 |
| rm_you | lol | 03:24 |
| blogan | i mean i reviewed it before | 03:24 |
| johnsom | blogan You bought it now.... | 03:24 |
| blogan | i dont remember +A'ing it | 03:24 |
| rm_you | heh | 03:24 |
| johnsom | I actually think it was fine myself. | 03:24 |
| rm_you | yeah it was fine | 03:24 |
| blogan | well that scares me | 03:25 |
| johnsom | I just really hate that our scenario gate is screwed up so bad | 03:25 |
| blogan | usually +A's are a memorable event | 03:25 |
| blogan | johnsom: i do too, i really do | 03:25 |
| johnsom | It was 1:24 in the afternoon, liquid lunch? | 03:25 |
| johnsom | Just kidding | 03:25 |
| blogan | i wish | 03:26 |
| blogan | i think i've been traumatized by internal stuff that i've just blocked any memories for the past month | 03:26 |
| blogan | or however long i've been on this, i can't remember | 03:26 |
| blogan | i think xgerman wants me to review the cascade delete patch | 03:27 |
| blogan | i'm not sure though | 03:27 |
| johnsom | Yes, that would be good | 03:27 |
| blogan | if only he gave me a link | 03:27 |
| blogan | i guess i cant now | 03:27 |
| johnsom | https://review.openstack.org/268237 | 03:27 |
| johnsom | and | 03:27 |
| johnsom | https://review.openstack.org/284340 | 03:27 |
| blogan | i think he realized i was in a +A fugue state | 03:27 |
| blogan | and wanted to take advantage | 03:28 |
| blogan | ohi was kidding about needing a link | 03:28 |
| johnsom | Just helping | 03:28 |
| blogan | another thing i dont understand about the scenarios, sometimes it seems to run 4 tests other times 7 or 8 | 03:28 |
| johnsom | Gives me bonus points by promoting his patches.... | 03:28 |
| johnsom | grin | 03:28 |
| xgerman | blogan how might you get the idea I want you to review something? | 03:29 |
| blogan | xgerman: ESP? | 03:29 |
| *** neelashah has joined #openstack-lbaas | 03:29 | |
| *** johnsom_ has joined #openstack-lbaas | 03:29 | |
| *** johnsom__ has joined #openstack-lbaas | 03:30 | |
| blogan | seriously though, why does some scenario test jobs run 7 tests and others run 4 | 03:30 |
| xgerman | no idea though I think you signed me up for writing some | 03:31 |
| blogan | xgerman: not scenario tests | 03:31 |
| johnsom__ | Doesn't it stop where it failed out? | 03:31 |
| blogan | just api ones | 03:31 |
| blogan | johnsom__: shoudln't, but it wouldn't run the test if it fails in the setup | 03:32 |
| johnsom__ | That is different than what I have seen. To me it looks like it bombs out at the first failure (test block) and then spews all of the not-so-useful steps | 03:33 |
| *** johnsom_ has quit IRC | 03:33 | |
| blogan | hmm, but then the ones that fail have 8 failures, whereas the ones that seem to have some passing ones have 3 passes, and 1 failure (session persistence) | 03:34 |
| blogan | i would expect the 8 failures ot just be 1, but it could be at a certain point where its failed that breaks it all | 03:35 |
| blogan | like resource_setup | 03:35 |
| johnsom__ | Yeah, I know that my lbaas patch will fix one random failure path where the HTTP connect goes sideways and we weren't catching the right exception | 03:35 |
| blogan | some of these other errors are just the VM not going active in time | 03:36 |
| blogan | bc of vt-x | 03:36 |
| blogan | or lack of | 03:36 |
| johnsom__ | Yeah, I think so too. But ten minutes??? geez | 03:37 |
| johnsom__ | I looked at the dstat logs last night. I see we are causing them to swap too, so I'm sure that doesn't help either | 03:38 |
| blogan | xgerman: will sni and L7 be deleted from the db with the cascade? | 03:39 |
| *** fnaval has quit IRC | 03:39 | |
| xgerman | sin won’t be deleted (there is some bug I filed) | 03:41 |
| xgerman | sni will never be deleted right now | 03:42 |
| blogan | xgerman: so sni will be a follow up patch? | 03:42 |
| xgerman | sin is broken today | 03:42 |
| xgerman | try to delete it :-) | 03:42 |
| blogan | i did | 03:42 |
| xgerman | with/without cascading delete | 03:42 |
| blogan | delete from lbaas_sni; | 03:42 |
| blogan | works just fine :) | 03:42 |
| xgerman | yeah, that still works | 03:42 |
| *** fawadkhaliq has joined #openstack-lbaas | 03:43 | |
| blogan | i know im being a jack ass | 03:43 |
| blogan | xgerman: what about l7? | 03:43 |
| xgerman | that cascades on listeners so should be good | 03:43 |
| blogan | xgerman: models say sni should cascade too | 03:44 |
| *** yuanying has quit IRC | 03:44 | |
| blogan | a sqlalchemy cascade, as opposed to a mysql cascade | 03:44 |
| xgerman | mmh, maybe somebody fixed it or I was blind | 03:46 |
| blogan | gonna test it out real quick :) | 03:46 |
| blogan | xgerman: you might have just looked for the cascade in the mysql table schema, not the sqlalchemy model | 03:46 |
| blogan | i don't think we've ever set the table definition to do a cascade | 03:46 |
| blogan | just the models so sqlalchemy takes care of it | 03:46 |
| xgerman | mmh, I might have looked on SNI but the cascade is on listener | 03:47 |
| blogan | yep | 03:47 |
| blogan | the relationship is | 03:47 |
| xgerman | well, the heavy lifting was to get cascade actually invoked in the driver | 03:47 |
| blogan | yeah, i was just making sure it was covered | 03:48 |
| xgerman | yep, that’s good | 03:48 |
| xgerman | I coded that in a few afternoons so speed was at the essence | 03:48 |
| xgerman | but Al said it works fist & glove with the Horizon Panels — which was my goal ;-) | 03:49 |
| blogan | xgerman: think i found a problem, though i may just do a follow up patch myself to fix it | 03:51 |
| xgerman | I am curious — what did I break? | 03:53 |
| *** fnaval has joined #openstack-lbaas | 03:53 | |
| blogan | its the agent code | 03:53 |
| blogan | so no one will notice lol | 03:53 |
| xgerman | lyep | 03:55 |
| xgerman | isn’t that something dougwig is removing anyway | 03:55 |
| blogan | xgerman: no! | 03:55 |
| blogan | xgerman: well its up for a discussion tomorrow | 03:55 |
| xgerman | so if I vote right I won’t have to fix it ;- | 03:56 |
| blogan | i could push up a follow-up patch real quick | 03:57 |
| blogan | xgerman: i reviewed, put a couple comments | 03:57 |
| blogan | you tell me if you think its worth a respin, honor system :) | 03:57 |
| xgerman | looking | 03:58 |
| xgerman | yeah, I guess I should respin those are two potential problems | 03:59 |
| blogan | xgerman: or push up a new patch real quick that fixes them | 04:00 |
| blogan | i mean | 04:00 |
| blogan | a new review | 04:00 |
| xgerman | on it | 04:00 |
| blogan | hate to wipe out al's +2 | 04:01 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/octavia: Updated from global requirements https://review.openstack.org/286968 | 04:03 |
| xgerman | yep, me, too | 04:06 |
| *** yuanying has joined #openstack-lbaas | 04:07 | |
| openstackgerrit | German Eichberger proposed openstack/neutron-lbaas: Adds Cascade option for load balancer drivers https://review.openstack.org/268237 | 04:11 |
| xgerman | ok, here we go | 04:11 |
| *** neelashah has quit IRC | 04:17 | |
| blogan | xgerman: :( | 04:18 |
| blogan | isn't it supposed to be plugin.db.delete_loadbalancer_cascade? and not the super method? | 04:18 |
| blogan | xgerman: yeah i think thats wrong | 04:19 |
| xgerman | :-( | 04:20 |
| xgerman | true | 04:21 |
| blogan | xgerman: adding comments | 04:22 |
| blogan | well i cna just tell you here | 04:22 |
| blogan | lol | 04:22 |
| xgerman | yeah, I will just do your changes… I know what I vote for tomorrow | 04:23 |
| blogan | just change that db delete call and the super back to delete and it'll be good | 04:24 |
| *** ducttape_ has joined #openstack-lbaas | 04:25 | |
| *** manishg has quit IRC | 04:26 | |
| openstackgerrit | German Eichberger proposed openstack/neutron-lbaas: Adds Cascade option for load balancer drivers https://review.openstack.org/268237 | 04:31 |
| xgerman | ok, another spin | 04:31 |
| *** ducttape_ has quit IRC | 04:31 | |
| *** neelashah has joined #openstack-lbaas | 04:37 | |
| xgerman | blogan hope this is now right | 04:41 |
| *** Purandar has joined #openstack-lbaas | 04:50 | |
| *** links has joined #openstack-lbaas | 04:51 | |
| *** neelashah has quit IRC | 04:51 | |
| *** fawadkhaliq has quit IRC | 04:58 | |
| *** manishg has joined #openstack-lbaas | 05:03 | |
| *** amotoki has joined #openstack-lbaas | 05:16 | |
| blogan | xgerman: yeah just realized there's a whole part to the agent driver that needs to have the cascade delete functionality | 05:22 |
| xgerman | oh, no... | 05:23 |
| blogan | xgerman: too much to do for this review | 05:23 |
| xgerman | agreed and the agent might die tomorrow | 05:23 |
| blogan | xgerman: meaning, not worth not doing it in your review, would be part of either 1) removing agent driver 2) fixing agent driver to get it up-to-date | 05:23 |
| xgerman | yep | 05:24 |
| *** piet has joined #openstack-lbaas | 05:26 | |
| *** ducttape_ has joined #openstack-lbaas | 05:27 | |
| *** kevo has joined #openstack-lbaas | 05:28 | |
| *** ducttape_ has quit IRC | 05:32 | |
| *** pcaruana has joined #openstack-lbaas | 05:36 | |
| *** piet has quit IRC | 05:42 | |
| reedip | sbalukoff: ping | 05:43 |
| sbalukoff | reedip: Pong | 05:43 |
| sbalukoff | What's up, dude? | 05:43 |
| reedip | sbalukoff : Hey, just saw your bug | 05:44 |
| *** manishg has quit IRC | 05:44 | |
| reedip | sbalukoff: was thinking if https://review.openstack.org/#/c/255093/21/neutronclient/neutron/v2_0/lb/v2/pool.py solves it? | 05:44 |
| sbalukoff | I've got it fixed. :) Just running through tox locally before committing. | 05:44 |
| sbalukoff | reedip: This isn't a CLI bug. | 05:44 |
| reedip | sbalukoff: Ohh, lol ... sorry , sure | 05:44 |
| sbalukoff | It's in the neutron-lbaas DB update code. | 05:44 |
| reedip | sablukoff: thats what I wanted to know | 05:44 |
| sbalukoff | But! Let me look at that again... | 05:44 |
| reedip | sbalukoff : sure :) | 05:45 |
| sbalukoff | It turns out we didn't provide a good way to set session persistence to None in the API... | 05:45 |
| sbalukoff | So I had to add a session persistence 'NONE' type, which tells the DB layer just to delete the session persistence record... | 05:45 |
| sbalukoff | Let me make sure that'll work with your API update. | 05:45 |
| sbalukoff | Ahh-- yes that will still work with your API code. :) | 05:46 |
| *** pcaruana has quit IRC | 05:49 | |
| *** rcernin has joined #openstack-lbaas | 05:51 | |
| *** johnsom__ has quit IRC | 05:51 | |
| openstackgerrit | LIU Yulong proposed openstack/neutron-lbaas: Restrict delete v2 pool if it has binding healthmonitor https://review.openstack.org/179684 | 06:00 |
| openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: WIP - nuke lbaas v1 https://review.openstack.org/286381 | 06:13 |
| reedip | sbalukoff : sorry, confused a bit :) | 06:13 |
| sbalukoff | reedip: It may be more informative when I upload my patch which fixes the bug I found. :) | 06:13 |
| reedip | sbalukoff : hehe .. sure, will wait for your update :) | 06:14 |
| sbalukoff | reedip: (I wanted to add a couple unit tests to make sure this bug doesn't come back if someone makes updates...) | 06:14 |
| sbalukoff | Hence the reason I haven't uploaded it yet... | 06:14 |
| *** manishg has joined #openstack-lbaas | 06:15 | |
| openstackgerrit | Stephen Balukoff proposed openstack/neutron-lbaas: Fix delete session_persisnces with pool update bug https://review.openstack.org/287004 | 06:19 |
| sbalukoff | ... dangit it commit message! | 06:19 |
| sbalukoff | Fixing that now... | 06:19 |
| *** manishg has quit IRC | 06:19 | |
| openstackgerrit | Stephen Balukoff proposed openstack/neutron-lbaas: Fix delete of session_persistence with pool update https://review.openstack.org/287004 | 06:20 |
| dougwig | blogan: dude, you imported the v1 namespace driver in the v2 driver?!? :) | 06:22 |
| *** ducttape_ has joined #openstack-lbaas | 06:28 | |
| reedip | sbalukoff : Just saw the code | 06:29 |
| reedip | sbalukoff: so if a user wants to clear the session persistence, what should he/she do? | 06:30 |
| reedip | sbalukoff: in the CLI | 06:30 |
| sbalukoff | neutron lbaas-pool-update pool1 --session_persistence type=dict type=NONE | 06:30 |
| sbalukoff | Because right now we don't have a good way of setting --session_persistence to an empty dictionary. | 06:31 |
| reedip | sbalukoff: cant we keep a simple option --no-session-persistence ? | 06:31 |
| sbalukoff | reedip: Do we have that option? | 06:31 |
| sbalukoff | And if so, yes, we can. | 06:31 |
| sbalukoff | Just make sure that the pool update dictionary contains: session_persistence: None | 06:32 |
| reedip | sbalukoff: not yet | 06:32 |
| sbalukoff | And that will work too. | 06:32 |
| *** ducttape_ has quit IRC | 06:32 | |
| blogan | xgerman: pep8 issue | 06:32 |
| blogan | dougwig: what? | 06:32 |
| sbalukoff | I wrote the update code to have a couple different ways to set the session persistence to None... | 06:32 |
| sbalukoff | Because I didn't want to break CLI backward-compatibility, per se. | 06:33 |
| *** amotoki has quit IRC | 06:33 | |
| reedip | sbalukoff: Just thinking that instead of such a large option, we can just use a simple option ( --clear-session-persistence ) which would do the job | 06:33 |
| dougwig | blogan: the number of places that v2 reaches into v1 is fascinating. | 06:33 |
| sbalukoff | reedip: Sure, that's fine and easier to understand, eh. | 06:33 |
| reedip | sbalukoff: instead of a more complicated type=dict type=None | 06:33 |
| reedip | sbalukoff: yep, thats why floating it here :) | 06:33 |
| *** amotoki has joined #openstack-lbaas | 06:34 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: WIP - nuke lbaas v1 https://review.openstack.org/286381 | 06:34 |
| sbalukoff | Note that if you update you patch to do that, you'll need to do a Depends-On: with my patch. | 06:34 |
| sbalukoff | (Unless my patch merges really quickly... which will probably happen as this is a 'critical' but in Octavia.) | 06:35 |
| sbalukoff | bug | 06:35 |
| *** armax has quit IRC | 06:35 | |
| blogan | dougwig: i have to say i dont remember doing that, but it had to be me, i need to look at it again | 06:35 |
| dougwig | critical butt | 06:35 |
| dougwig | diff the last two patchsets in the nuke review to see the overlap | 06:35 |
| dougwig | i'm guilty of the unit test one. | 06:35 |
| *** amotoki has quit IRC | 06:36 | |
| openstackgerrit | German Eichberger proposed openstack/neutron-lbaas: Adds Cascade option for load balancer drivers https://review.openstack.org/268237 | 06:38 |
| *** prabampm has joined #openstack-lbaas | 06:38 | |
| xgerman | blogan — tonight clearly is not my night | 06:39 |
| xgerman | going to bed to avid more disasters... | 06:39 |
| xgerman | avid=avoid | 06:39 |
| reedip | sbalukoff : Sure, will put up a patch with depends-on. This would however be a separate patch, not the one I am working on, as this is a new option | 06:39 |
| sbalukoff | Oh, right. | 06:40 |
| blogan | dougwig: im going to say you're guilty of both and call it night just bc | 06:40 |
| reedip | sablukoff: this would also be transitioned to OpenstackClient, so its better we introduce such changes early, so that the transition is smoother | 06:40 |
| sbalukoff | So for now, to clear session_persistence on a pool, people will just have to use the long, ugly method. ;) | 06:40 |
| sbalukoff | reedip: Agreed. | 06:40 |
| reedip | sablukoff : -clear-session-persistence is also a big word... would --clear-session work better ? | 06:41 |
| sbalukoff | 'session' is too ambiguous. | 06:43 |
| sbalukoff | 'session-persistence' has a specific meaning for pools. | 06:43 |
| reedip | sbalukoff: so --clear-session-persistence / --no-session-persistence would be a better option | 06:46 |
| sbalukoff | Yep. | 06:46 |
| sbalukoff | It's long, but at least it's clear what it means. | 06:46 |
| *** Purandar has quit IRC | 06:47 | |
| reedip | sbalukoff: Ok, putting up the patch today, with the depends on. Will add you as a reviewer | 06:47 |
| sbalukoff | Ok! | 06:48 |
| reedip | :) | 06:48 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron-lbaas: Fix model vs migration mismatches for L7 rules and policies https://review.openstack.org/287014 | 06:51 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron-lbaas: Fix model vs migration mismatches in nsxv tables https://review.openstack.org/287015 | 06:51 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron-lbaas: Add model sync test https://review.openstack.org/287016 | 06:51 |
| sbalukoff | Oh, hey! Nice! | 06:59 |
| *** mixos has quit IRC | 07:04 | |
| *** nmagnezi has joined #openstack-lbaas | 07:12 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: WIP - nuke lbaas v1 https://review.openstack.org/286381 | 07:13 |
| *** amotoki has joined #openstack-lbaas | 07:14 | |
| *** manishg has joined #openstack-lbaas | 07:15 | |
| *** manishg has quit IRC | 07:20 | |
| *** evgenyf has joined #openstack-lbaas | 07:22 | |
| *** ducttape_ has joined #openstack-lbaas | 07:28 | |
| *** ducttape_ has quit IRC | 07:33 | |
| *** amotoki has quit IRC | 07:35 | |
| *** bana_k has joined #openstack-lbaas | 07:36 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron-lbaas: Add model sync test https://review.openstack.org/287016 | 07:41 |
| blogan | dougwig: you embellish too much, there wasn't that much v2 reaching into v1 | 07:42 |
| *** amotoki has joined #openstack-lbaas | 07:43 | |
| *** amotoki has quit IRC | 07:48 | |
| *** bana_k has quit IRC | 08:00 | |
| *** pcaruana has joined #openstack-lbaas | 08:02 | |
| *** manishg has joined #openstack-lbaas | 08:16 | |
| *** eezhova has joined #openstack-lbaas | 08:21 | |
| openstackgerrit | Merged openstack/neutron-lbaas: LBaaS: make use of neutron_lib exceptions https://review.openstack.org/286878 | 08:21 |
| *** manishg has quit IRC | 08:21 | |
| *** eranra has joined #openstack-lbaas | 08:23 | |
| reedip | sbalukoff : In case you are awake, https://review.openstack.org/287035 | 08:26 |
| *** ducttape_ has joined #openstack-lbaas | 08:29 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron-lbaas: Fix model vs migration mismatches in nsxv tables https://review.openstack.org/287015 | 08:30 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron-lbaas: Add model sync test https://review.openstack.org/287016 | 08:30 |
| *** kobis has joined #openstack-lbaas | 08:31 | |
| *** ducttape_ has quit IRC | 08:34 | |
| *** kevo has quit IRC | 08:50 | |
| *** ihrachys has joined #openstack-lbaas | 08:51 | |
| *** manishg has joined #openstack-lbaas | 09:17 | |
| *** manishg has quit IRC | 09:21 | |
| *** jschwarz has joined #openstack-lbaas | 09:31 | |
| *** lane_kong has joined #openstack-lbaas | 09:41 | |
| *** lane_kong is now known as lynn | 09:45 | |
| *** reedip is now known as reedip_away | 09:46 | |
| *** lynn is now known as Guest29970 | 09:46 | |
| *** Guest29970 is now known as lynn_kong | 09:46 | |
| *** evgenyf has quit IRC | 09:57 | |
| *** paco20151113 has quit IRC | 10:02 | |
| *** lynn_kong is now known as Larrie | 10:12 | |
| *** Larrie is now known as larrie | 10:12 | |
| *** evgenyf has joined #openstack-lbaas | 10:14 | |
| *** amotoki has joined #openstack-lbaas | 10:18 | |
| *** larrie is now known as larrie_kong | 10:28 | |
| *** ducttape_ has joined #openstack-lbaas | 10:31 | |
| *** ducttape_ has quit IRC | 10:35 | |
| *** evgenyf has quit IRC | 10:37 | |
| *** ihrachys has quit IRC | 10:47 | |
| *** evgenyf has joined #openstack-lbaas | 11:03 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron-lbaas: Add tenant-ID index for L7 tables again https://review.openstack.org/287111 | 11:03 |
| *** manishg has joined #openstack-lbaas | 11:18 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron-lbaas: Fix model vs migration mismatches for L7 rules and policies https://review.openstack.org/287014 | 11:22 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron-lbaas: Fix model vs migration mismatches in nsxv tables https://review.openstack.org/287015 | 11:22 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron-lbaas: Add model sync test https://review.openstack.org/287016 | 11:22 |
| *** manishg has quit IRC | 11:23 | |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Fix session_persistence deletion bug https://review.openstack.org/287118 | 11:25 |
| *** ducttape_ has joined #openstack-lbaas | 11:32 | |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Fix session_persistence deletion bug https://review.openstack.org/287118 | 11:32 |
| *** ducttape_ has quit IRC | 11:36 | |
| *** yamamoto has quit IRC | 11:41 | |
| *** ihrachys has joined #openstack-lbaas | 11:52 | |
| *** doug-fish has joined #openstack-lbaas | 11:56 | |
| *** manishg has joined #openstack-lbaas | 12:19 | |
| *** manishg has quit IRC | 12:23 | |
| *** yamamoto has joined #openstack-lbaas | 12:24 | |
| *** yamamoto_ has joined #openstack-lbaas | 12:25 | |
| *** yamamoto has quit IRC | 12:29 | |
| *** kiran-r has joined #openstack-lbaas | 12:29 | |
| *** rtheis has joined #openstack-lbaas | 12:29 | |
| *** ducttape_ has joined #openstack-lbaas | 12:32 | |
| *** ducttape_ has quit IRC | 12:37 | |
| doug-fish | we still have the dashboard patch https://review.openstack.org/#/c/280287/ awaiting review | 12:37 |
| doug-fish | xgerman: if you have time ^ these are the changes recommended by your group | 12:38 |
| *** yamamoto_ has quit IRC | 12:43 | |
| *** TrevorV has joined #openstack-lbaas | 12:49 | |
| *** fawadkhaliq has joined #openstack-lbaas | 12:54 | |
| *** yamamoto has joined #openstack-lbaas | 12:55 | |
| *** links has quit IRC | 12:59 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron-lbaas: Fix model vs migration mismatches for L7 rules and policies https://review.openstack.org/287014 | 13:03 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron-lbaas: Fix model vs migration mismatches in nsxv tables https://review.openstack.org/287015 | 13:03 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron-lbaas: Add model sync test https://review.openstack.org/287016 | 13:03 |
| *** ducttape_ has joined #openstack-lbaas | 13:09 | |
| *** manishg has joined #openstack-lbaas | 13:20 | |
| *** fawadkhaliq has quit IRC | 13:21 | |
| *** manishg has quit IRC | 13:25 | |
| *** Bjoern_ has joined #openstack-lbaas | 13:28 | |
| *** yamamoto has quit IRC | 13:28 | |
| *** Bjoern_ has quit IRC | 13:34 | |
| *** ducttape_ has quit IRC | 13:35 | |
| *** neelashah has joined #openstack-lbaas | 13:37 | |
| *** piet has joined #openstack-lbaas | 13:38 | |
| *** yamamoto has joined #openstack-lbaas | 13:39 | |
| *** yamamoto has quit IRC | 13:39 | |
| *** yamamoto has joined #openstack-lbaas | 13:54 | |
| TrevorV | Am I going to be lucky enough to find anyone that can help me with a quick neutron lbaas question? | 13:54 |
| *** doug-fish has quit IRC | 13:55 | |
| *** yamamoto has quit IRC | 14:02 | |
| *** yamamoto has joined #openstack-lbaas | 14:07 | |
| *** doug-fish has joined #openstack-lbaas | 14:13 | |
| *** ducttape_ has joined #openstack-lbaas | 14:17 | |
| *** ducttape_ has quit IRC | 14:17 | |
| *** manishg has joined #openstack-lbaas | 14:21 | |
| *** yamamoto has quit IRC | 14:23 | |
| *** manishg has quit IRC | 14:25 | |
| *** yamamoto has joined #openstack-lbaas | 14:44 | |
| *** alejandrito has joined #openstack-lbaas | 14:49 | |
| xgerman | and back: blogan, dougwig, etc: https://review.openstack.org/#/c/268237/ | 14:51 |
| *** doug-fish has quit IRC | 14:52 | |
| *** ducttape_ has joined #openstack-lbaas | 14:53 | |
| xgerman | TrevorV shoot | 14:53 |
| *** doug-fish has joined #openstack-lbaas | 14:54 | |
| TrevorV | xgerman so I got a -2 from Armando on the neutron lbaas single-create | 14:54 |
| xgerman | yep, this is how he rolls | 14:54 |
| TrevorV | But in spite of that (which seems "excessive" but oh well) I was still trying to fix what I've already added on what brandon had | 14:54 |
| xgerman | making friends + influencing people | 14:54 |
| *** prabampm has quit IRC | 14:55 | |
| TrevorV | In running tests, I was trying to figure out where in the code it throws the 400 back for "bad parameters" in the request body | 14:55 |
| TrevorV | In this case, "listeners" is not allowed in a LB post | 14:55 |
| TrevorV | Do you know what file I should start in? | 14:55 |
| TrevorV | Brandon kept saying "plugin" but every time I put a debug point in those methods it never stops there, and throws the 400 | 14:56 |
| TrevorV | So I can't find the entry point | 14:56 |
| xgerman | is the 400 coming from Octavia then it would be that “plugin" | 14:56 |
| *** doug-fish has quit IRC | 14:56 | |
| TrevorV | As far as I know it never gets there | 14:56 |
| *** doug-fish has joined #openstack-lbaas | 14:57 | |
| xgerman | ok | 14:57 |
| TrevorV | Since I've dropped debug points all over the "create_loadbalancer" and "create_lb_graph" methods that we've added | 14:57 |
| xgerman | the other we stuff happens in extensions | 14:58 |
| xgerman | loadbalancerv2.py | 14:58 |
| *** mixos has joined #openstack-lbaas | 14:59 | |
| TrevorV | Alright, so would I drop a debug point on "get_resources" as the first point in the API? | 14:59 |
| TrevorV | Looks that way. Perfect, thanks xgerman | 15:00 |
| xgerman | you are welcome -- | 15:00 |
| *** mixos has quit IRC | 15:13 | |
| *** yamamoto has quit IRC | 15:17 | |
| *** yamamoto has joined #openstack-lbaas | 15:18 | |
| *** ajmiller has joined #openstack-lbaas | 15:18 | |
| *** yamamoto has quit IRC | 15:18 | |
| *** manishg has joined #openstack-lbaas | 15:19 | |
| *** yamamoto has joined #openstack-lbaas | 15:21 | |
| dougwig | TrevorV: that's a procedural -2, as code compete has passed. As he mentioned, if it's done but just needs reviews, we can ask for an FFE. If it's sitting in gerrit as a WIP and only partially done (the rest via "bugfix"), it's likely not going to make Mitaka. | 15:26 |
| TrevorV | dougwig I figured, just -2 felt strong. I wasn't complaining so much as "surprised" | 15:27 |
| *** woodster_ has joined #openstack-lbaas | 15:27 | |
| dougwig | TrevorV: well, there are two kinds of -2. One is used administratively as a notice to other cores. The other means roughly, "go fuck yourself", near as I can tell. And the latter tends to make the former feel harsh. | 15:27 |
| TrevorV | Exactly. The only time I heard of -2 being used at all was when neutron stomped that one change a while back | 15:28 |
| TrevorV | I forget which company was trying to completely overhaul neutron or whatever but you guys just... nuked it. | 15:28 |
| TrevorV | Part of me was going to comment in an irritated fashion about "-2-ing something that we want just because it can't get in RIGHT NOW" but then I figured it wasn't the same severity as I've attributed to it. | 15:29 |
| openstackgerrit | Kobi Samoray proposed openstack/octavia: [WIP] Implement custom header support for Octavia https://review.openstack.org/257901 | 15:34 |
| TrevorV | dougwig its all good, I was hoping to get this in for Mitaka, but not knowing neutron lbaas very well, it was/is a long shot anyway. | 15:38 |
| TrevorV | At least we got Octavia in | 15:38 |
| *** eranra_ has joined #openstack-lbaas | 15:38 | |
| *** piet has quit IRC | 15:39 | |
| openstackgerrit | Kobi Samoray proposed openstack/octavia: [WIP] Implement custom header support for Octavia https://review.openstack.org/257901 | 15:40 |
| *** eranra has quit IRC | 15:41 | |
| *** yamamoto has quit IRC | 15:42 | |
| *** yamamoto has joined #openstack-lbaas | 15:43 | |
| *** piet has joined #openstack-lbaas | 15:43 | |
| *** eranra_ has quit IRC | 15:43 | |
| *** yamamoto has quit IRC | 15:43 | |
| *** yamamoto has joined #openstack-lbaas | 15:45 | |
| *** kobis has quit IRC | 15:47 | |
| *** manishg has quit IRC | 15:57 | |
| *** piet has quit IRC | 15:57 | |
| *** Purandar has joined #openstack-lbaas | 15:59 | |
| *** Purandar has quit IRC | 16:01 | |
| *** piet has joined #openstack-lbaas | 16:02 | |
| *** Purandar has joined #openstack-lbaas | 16:02 | |
| *** pcaruana has quit IRC | 16:05 | |
| *** fawadkhaliq has joined #openstack-lbaas | 16:06 | |
| *** piet has quit IRC | 16:08 | |
| *** armax has joined #openstack-lbaas | 16:11 | |
| *** ctracey_ has joined #openstack-lbaas | 16:14 | |
| *** fawadkhaliq has quit IRC | 16:14 | |
| *** [1]evgenyf has joined #openstack-lbaas | 16:14 | |
| *** larrie_kong has quit IRC | 16:15 | |
| *** amitry has quit IRC | 16:15 | |
| *** ctracey has quit IRC | 16:15 | |
| *** larrie_kong has joined #openstack-lbaas | 16:15 | |
| *** larrie_kong has quit IRC | 16:15 | |
| *** larrie_kong has joined #openstack-lbaas | 16:15 | |
| *** amitry has joined #openstack-lbaas | 16:15 | |
| *** ctracey_ is now known as ctracey | 16:16 | |
| *** evgenyf has quit IRC | 16:18 | |
| *** [1]evgenyf has quit IRC | 16:20 | |
| *** yamamoto has quit IRC | 16:42 | |
| *** Purandar has quit IRC | 16:43 | |
| *** Purandar has joined #openstack-lbaas | 16:54 | |
| *** Purandar has quit IRC | 16:56 | |
| *** Purandar has joined #openstack-lbaas | 16:57 | |
| *** manishg has joined #openstack-lbaas | 16:58 | |
| xgerman | thanks blogan - I am still embarrassed about those mistakes... | 17:04 |
| blogan | xgerman: lol we've all made worse mistakes | 17:04 |
| *** minwang2 has joined #openstack-lbaas | 17:06 | |
| *** minwang2 has quit IRC | 17:18 | |
| *** fawadkhaliq has joined #openstack-lbaas | 17:21 | |
| *** kobis has joined #openstack-lbaas | 17:21 | |
| *** nmagnezi has quit IRC | 17:22 | |
| *** Bjoern_ has joined #openstack-lbaas | 17:23 | |
| ajmiller | doug-fish ping | 17:26 |
| *** kiran-r has quit IRC | 17:28 | |
| *** jschwarz has quit IRC | 17:30 | |
| *** minwang2 has joined #openstack-lbaas | 17:34 | |
| *** minwang2 has quit IRC | 17:35 | |
| *** yamamoto has joined #openstack-lbaas | 17:43 | |
| *** bana_k has joined #openstack-lbaas | 17:44 | |
| *** kevo has joined #openstack-lbaas | 17:44 | |
| doug-fish | ajmiller: hi | 17:46 |
| *** yamamoto has quit IRC | 17:50 | |
| *** crc32 has joined #openstack-lbaas | 17:58 | |
| *** minwang2 has joined #openstack-lbaas | 17:59 | |
| *** kobis has quit IRC | 17:59 | |
| openstackgerrit | Banashankar k proposed openstack/octavia: Adding the topology info at the time of LB AMP association https://review.openstack.org/286365 | 18:03 |
| *** madhu_ak has joined #openstack-lbaas | 18:09 | |
| *** fnaval has quit IRC | 18:14 | |
| *** ihrachys has quit IRC | 18:14 | |
| *** jschwarz has joined #openstack-lbaas | 18:27 | |
| openstackgerrit | Stephen Balukoff proposed openstack/neutron-lbaas: Fix delete of session_persistence with pool update https://review.openstack.org/287004 | 18:27 |
| sbalukoff | Mornin' folks! | 18:27 |
| barclaac | Hi dougwig: German's been filling me in about the Octavia meeting this morning. I'm interested in what our migration story is if someone is using an LBaaS v1 or v2 haproxy agent from a Liberty based release? | 18:28 |
| barclaac | Of course, the rest of you can pile on as well ;-) | 18:28 |
| sbalukoff | If LBaaSv1: Burn it with fire. | 18:29 |
| barclaac | sbalukoff: noted ;-) | 18:29 |
| sbalukoff | :) | 18:29 |
| sbalukoff | Just tryin' to help. :) | 18:29 |
| dougwig | barclaac: ha. well, way back in kilo i wanted v1 and v2 to run in parallel, so it could be a lazy migration. but the group's decision was to punt and make that an operator problem. which it still is. | 18:30 |
| barclaac | I still don't see an F5 driver for v2 so removal in M would seem problematic (on a personal, non customer-focused basis, let me grab the gasoline ;-) | 18:30 |
| *** doug-fish has quit IRC | 18:30 | |
| *** doug-fish has joined #openstack-lbaas | 18:31 | |
| dougwig | barclaac: f5 doesn't have a v1 driver in tree either. | 18:31 |
| barclaac | I think my view is that we're pretty late in M to make this decision. I'd fully support to do it my N-1 | 18:31 |
| xgerman | F5 showed up one day and talked about V2… I think they have a driver or are close to it | 18:31 |
| sbalukoff | Er... hasn't it always been up to the vendors to write their own drivers? | 18:31 |
| barclaac | I'm not disagreeing there. A cross I have to bear. | 18:31 |
| dougwig | barclaac: which discussion are we having? removal of the agent driver, or removal of v1? | 18:31 |
| *** Aish has joined #openstack-lbaas | 18:31 | |
| xgerman | v1 | 18:32 |
| barclaac | Actually both. Talk me through the scenario. A customer has Liberty based v2 haproxy and upgrades to Mitake based v2 Octavia. What does that look like? | 18:32 |
| sbalukoff | dougwig: Were you hoping to get your patches in for M? I thought the idea was to prime the pump to get rid of them for N the moment we're in that cycle.. | 18:32 |
| barclaac | ^^ +1 | 18:32 |
| dougwig | in my mind we have to decide on both: keep, separate, or nuke, and then decide the timeline for agent is m or n, and the timeline for v1 is n (or later, for either.) | 18:32 |
| dougwig | i have already -2'ed myself on v1 until N. i was expecting a conversation about the agent; i'm open to either, **but it is broken right now**, so either we fix it for m or take some other action? | 18:33 |
| barclaac | So the liberty based v2 haproxy install will suddenly break on upgrade as the haproxies are no longer present? | 18:33 |
| barclaac | I think sbalukoff should fix the v1 agent :-p | 18:34 |
| dougwig | barclaac: i'd agree that would make it too soon. i'm hoping we get a volunteer with some cycles. | 18:34 |
| barclaac | I would be good for his soul | 18:34 |
| sbalukoff | It's probably worth noting that none of the new features (L7, shared-pools, get-me-a-LB, cascade-delete) include agent driver updates as it's no longer the reference... | 18:34 |
| sbalukoff | barclaac: Am I that easily baited? | 18:34 |
| dougwig | sbalukoff: and that is fine. it fails the basic v2 tests, though. | 18:34 |
| xgerman | yeah, I think agent is dead | 18:34 |
| *** doug-fis_ has joined #openstack-lbaas | 18:34 | |
| dougwig | refer to the non-voting namespace job. | 18:34 |
| *** doug-fish has quit IRC | 18:35 | |
| barclaac | So even though it's bitrotted and broken, that's for the new stuff. Can we accept it working for non-new features because an L user wouldn't have been able to configure them anyway? | 18:35 |
| dougwig | yep, as long as someone gets the job working. we can't ship it as is. | 18:36 |
| barclaac | How much work do you estimate? | 18:36 |
| dougwig | i don't know. it's had six months to rot. i'd guess 1-5 days. | 18:37 |
| *** fnaval has joined #openstack-lbaas | 18:37 | |
| barclaac | Can we treat it as a "bug fix" after M3? | 18:37 |
| dougwig | might be as simple as some test exclusions. | 18:37 |
| barclaac | that was kind of my thinking | 18:37 |
| dougwig | barclaac: yes, but i'd rather we not ship an rc with a busted driver. though that's just my preference. | 18:37 |
| xgerman | +1 | 18:38 |
| barclaac | sigh - you've got me on that one. | 18:38 |
| sbalukoff | +1 | 18:38 |
| *** doug-fis_ has quit IRC | 18:39 | |
| barclaac | OK. I hear you all. Let me go and do some research here (aka figure out if my product guy will be pissed at me ;-) | 18:40 |
| barclaac | If he will, it's likely that we'll resource the fixes. | 18:40 |
| *** neelashah has quit IRC | 18:40 | |
| *** piet has joined #openstack-lbaas | 18:40 | |
| sbalukoff | In other words, you'll tell xgerman to go do it? ;) | 18:41 |
| barclaac | That happened about 3 mins ago :-D | 18:41 |
| sbalukoff | Haha! | 18:42 |
| dougwig | lol | 18:42 |
| xgerman | sympathy guys? | 18:42 |
| dougwig | none. | 18:42 |
| barclaac | Mind you, if my product guy can live with it I'm more than happy to gather around the pyre with the rest of you. | 18:42 |
| sbalukoff | xgerman: I feel for you. | 18:42 |
| dougwig | i suspect that we need to keep it for M, just due to deprecation policies. | 18:42 |
| barclaac | yup - he's got a really shitty boss | 18:42 |
| *** alejandrito has quit IRC | 18:43 | |
| blogan | my opinion is that since we're going to probably remvoe v1, those on v1 who are using the agent will want somethign similar for v2 and if we remove the agent that destroys that | 18:45 |
| *** ajmiller_ has joined #openstack-lbaas | 18:45 | |
| dougwig | i know a lot of folks on v1, but no one willingly on the agent. what does your customer base look like for v1? | 18:46 |
| blogan | dougwig: so you know people who are on the v1 agent but are ready to move to octavia? is that what you mean | 18:46 |
| xgerman | they should be | 18:47 |
| dougwig | blogan: that would imply that i know folks running the v1 agent in production. | 18:47 |
| dougwig | i'm aware of many with v1 in production. but... | 18:47 |
| *** crc32 has quit IRC | 18:48 | |
| *** ajmiller has quit IRC | 18:48 | |
| blogan | dougwig: our private cloud product does not support v1, but i'm more worried about those that are, i'm assuming some people are running the v1 agent in production, and removing the v2 agent would be a big FU to them | 18:48 |
| blogan | and we've already done one big FU with not allowing v1 and v2 together | 18:49 |
| *** crc32 has joined #openstack-lbaas | 18:52 | |
| openstackgerrit | Merged openstack/neutron-lbaas: Adds Cascade option for load balancer drivers https://review.openstack.org/268237 | 18:56 |
| *** bana_k has quit IRC | 18:59 | |
| *** Bjoern_ is now known as Bjoern_zZzZzZzZ | 18:59 | |
| *** Bjoern_zZzZzZzZ is now known as Bjoern_ | 19:03 | |
| *** bana_k has joined #openstack-lbaas | 19:03 | |
| *** neelashah has joined #openstack-lbaas | 19:08 | |
| *** doug-fish has joined #openstack-lbaas | 19:12 | |
| *** ajmiller_ is now known as ajmiller | 19:15 | |
| barclaac | dougwig: OK. Our product mgrs strong preference would be for us to fix the haproxy agent in M and remove it in N. If we resource that can you live with it? | 19:17 |
| *** crc32 has quit IRC | 19:18 | |
| sbalukoff | I'm OK with that. | 19:19 |
| dougwig | barclaac: absolutely, i would consider that ideal. | 19:19 |
| *** bana_k has quit IRC | 19:19 | |
| sbalukoff | Sorry, xgerman. | 19:20 |
| barclaac | xgerman's busy doing our sprint planning right now. A perfect time to add new work! | 19:20 |
| sbalukoff | Haha! | 19:22 |
| dougwig | barclaac, xgerman: note this patch that fixes it get tests running: https://review.openstack.org/#/c/286332/ | 19:22 |
| dougwig | grammar fail, but you get the idea. | 19:23 |
| barclaac | thx dougwig | 19:23 |
| barclaac | when's m3 close? | 19:23 |
| dougwig | door has already closed, we're in feature exception and bugfix territory now. | 19:23 |
| sbalukoff | blogan: Did my comments on the neutron-lbaas session_persistence update fix patch make sense? | 19:27 |
| *** neelashah has quit IRC | 19:29 | |
| *** neelashah has joined #openstack-lbaas | 19:30 | |
| *** fnaval has quit IRC | 19:32 | |
| *** doug-fis_ has joined #openstack-lbaas | 19:35 | |
| *** doug-fish has quit IRC | 19:36 | |
| *** evgenyf has joined #openstack-lbaas | 19:44 | |
| *** neelashah1 has joined #openstack-lbaas | 19:45 | |
| *** doug-fish has joined #openstack-lbaas | 19:46 | |
| *** neelashah has quit IRC | 19:47 | |
| *** Purandar has quit IRC | 19:47 | |
| *** doug-fis_ has quit IRC | 19:47 | |
| *** ducttape_ has quit IRC | 19:48 | |
| *** ducttape_ has joined #openstack-lbaas | 19:48 | |
| *** neelashah has joined #openstack-lbaas | 19:51 | |
| *** neelashah1 has quit IRC | 19:54 | |
| openstackgerrit | Merged openstack/octavia: Updated from global requirements https://review.openstack.org/286968 | 19:55 |
| *** Purandar has joined #openstack-lbaas | 19:55 | |
| *** bhaargavi has joined #openstack-lbaas | 19:55 | |
| doug-fish | any word on cascading delete of load balancers? is it merged or is there a patch I can track? | 19:57 |
| johnsom | Octavia meeting starting soon on #openstack-meeting-alt | 19:58 |
| *** bana_k has joined #openstack-lbaas | 19:58 | |
| xgerman | merged | 19:59 |
| xgerman | doug-fish merged | 19:59 |
| blogan | sbalukoff: yes but i still disagree to making that change just bc the client doesn't support setting session_persistence to null | 19:59 |
| *** pothole has joined #openstack-lbaas | 19:59 | |
| doug-fish | xgerman: woohoo! | 20:00 |
| blogan | sbalukoff: i think other calls have had similar issues, and they've done stuff that woudl be akin to --no-session-persistence or --remove-session-persistence | 20:00 |
| *** ducttape_ has quit IRC | 20:02 | |
| *** piet has quit IRC | 20:03 | |
| *** jschwarz has quit IRC | 20:12 | |
| *** ducttape_ has joined #openstack-lbaas | 20:13 | |
| *** ducttape_ has quit IRC | 20:14 | |
| *** ducttape_ has joined #openstack-lbaas | 20:15 | |
| *** pothole has quit IRC | 20:15 | |
| *** piet has joined #openstack-lbaas | 20:17 | |
| *** doug-fis_ has joined #openstack-lbaas | 20:17 | |
| *** doug-fish has quit IRC | 20:18 | |
| *** Purandar has quit IRC | 20:26 | |
| *** doug-fish has joined #openstack-lbaas | 20:31 | |
| *** doug-fis_ has quit IRC | 20:33 | |
| *** rcernin has quit IRC | 20:36 | |
| *** evgenyf has quit IRC | 20:45 | |
| xgerman | https://review.openstack.org/#/c/286413/5 — this is just a ridiculous buerocracy | 20:49 |
| xgerman | it’s like we are moving HP/IBM processes into OpenStack :-( | 20:50 |
| *** neelashah has quit IRC | 20:51 | |
| elarson | xgerman: those are really just release notes right? | 20:55 |
| xgerman | not sure what this form is good for... | 20:56 |
| *** neelashah has joined #openstack-lbaas | 20:56 | |
| elarson | that's what it looked like. it seems better to use more rigor in commit messages, ensuring blueprints and what have you are properly linked and then generate it. | 20:58 |
| elarson | but what do I know! | 20:58 |
| xgerman | same here — I do what I am told. If it takes more forms then lines of code so be it | 20:59 |
| *** neelashah has quit IRC | 21:09 | |
| *** doug-fish has quit IRC | 21:09 | |
| *** doug-fish has joined #openstack-lbaas | 21:15 | |
| *** neelashah has joined #openstack-lbaas | 21:16 | |
| *** doug-fis_ has joined #openstack-lbaas | 21:16 | |
| openstackgerrit | German Eichberger proposed openstack/octavia: Adds the Cascade Deelete REST API doc https://review.openstack.org/287443 | 21:18 |
| *** Purandar has joined #openstack-lbaas | 21:19 | |
| *** doug-fish has quit IRC | 21:20 | |
| *** _ducttape_ has joined #openstack-lbaas | 21:21 | |
| *** doug-fish has joined #openstack-lbaas | 21:23 | |
| *** doug-fis_ has quit IRC | 21:25 | |
| *** ducttape_ has quit IRC | 21:25 | |
| dougwig | sbalukoff: can you edit the l7 section of https://review.openstack.org/#/c/286413 ? | 21:28 |
| sbalukoff | dougwig: Sure. When does it need to be done? | 21:29 |
| dougwig | sbalukoff: is today possible? | 21:30 |
| sbalukoff | Sure; Engaged in something at the moment, so as long as "today" means "before midnight" then yes. | 21:30 |
| sbalukoff | I can do it sooner if it's really urgent. | 21:30 |
| *** neelashah1 has joined #openstack-lbaas | 21:31 | |
| *** neelashah2 has joined #openstack-lbaas | 21:33 | |
| xgerman | I think it is — I put my toe in and got it bitten off | 21:33 |
| *** neelashah has quit IRC | 21:34 | |
| sbalukoff | dougwig: Er... I'm confused as to what I'm supposed to do. I gather I need to write some thing describing L7 stuff. But I have no idea how I'm supposed to contribute it to this. | 21:34 |
| *** neelashah has joined #openstack-lbaas | 21:34 | |
| openstackgerrit | Trevor Vardeman proposed openstack/neutron-lbaas: WIP - Get Me A LB https://review.openstack.org/257201 | 21:35 |
| *** doug-fis_ has joined #openstack-lbaas | 21:35 | |
| *** neelashah1 has quit IRC | 21:35 | |
| sbalukoff | dougwig: Do I just upload a new patch set? | 21:35 |
| sbalukoff | Or just drop comments in? | 21:35 |
| *** Aish has quit IRC | 21:35 | |
| johnsom | sbalukoff there is a template file that explains | 21:36 |
| johnsom | I would guess it's comments, or they are going to have fun flattening it | 21:36 |
| *** Aish has joined #openstack-lbaas | 21:36 | |
| *** neelashah2 has quit IRC | 21:37 | |
| *** neelashah1 has joined #openstack-lbaas | 21:37 | |
| sbalukoff | Heh! | 21:37 |
| sbalukoff | Ok. | 21:37 |
| *** doug-fish has quit IRC | 21:38 | |
| *** neelashah has quit IRC | 21:41 | |
| xgerman | you can look at my back-and-forth with arms on cascading_delete | 21:41 |
| xgerman | and my previous vent sbalukoff | 21:42 |
| *** doug-fish has joined #openstack-lbaas | 21:43 | |
| sbalukoff | xgerman: Ok. :P | 21:45 |
| *** doug-fis_ has quit IRC | 21:46 | |
| *** piet has quit IRC | 21:49 | |
| xgerman | dougwig where do we keep the LBaaS V2 docs and CLI docs? | 21:52 |
| *** armax has quit IRC | 21:55 | |
| *** armax has joined #openstack-lbaas | 21:59 | |
| *** bana_k has quit IRC | 22:00 | |
| *** bana_k has joined #openstack-lbaas | 22:01 | |
| *** manishg has quit IRC | 22:02 | |
| *** manishg has joined #openstack-lbaas | 22:04 | |
| *** doug-fis_ has joined #openstack-lbaas | 22:10 | |
| *** doug-fish has quit IRC | 22:11 | |
| *** rtheis has quit IRC | 22:25 | |
| *** neelashah1 has quit IRC | 22:29 | |
| *** neelashah has joined #openstack-lbaas | 22:31 | |
| dougwig | xgerman: with the neutron Api and cli docs. | 22:39 |
| xgerman | ok, links? | 22:39 |
| *** doug-fis_ has quit IRC | 22:39 | |
| *** doug-fish has joined #openstack-lbaas | 22:42 | |
| *** doug-fis_ has joined #openstack-lbaas | 22:44 | |
| *** manishg has quit IRC | 22:45 | |
| *** manishg has joined #openstack-lbaas | 22:46 | |
| *** doug-fish has quit IRC | 22:47 | |
| *** doug-fis_ has quit IRC | 22:48 | |
| *** manishg_ has joined #openstack-lbaas | 22:53 | |
| *** manishg has quit IRC | 22:53 | |
| *** neelashah has quit IRC | 22:55 | |
| openstackgerrit | Trevor Vardeman proposed openstack/neutron-lbaas: WIP - Get Me A LB https://review.openstack.org/257201 | 22:56 |
| *** TrevorV has quit IRC | 22:56 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron-lbaas: Revert "Adds Cascade option for load balancer drivers" https://review.openstack.org/287494 | 23:02 |
| dougwig | xgerman: let's respin the cascade delete patch tomorrow with some tweaks. i can help if you're short on time. | 23:05 |
| xgerman | I did it exactly as you guys wanted at the midcycle | 23:05 |
| xgerman | and now Horizon relies on it working exactly that way | 23:06 |
| xgerman | None, of that extension stuff was my idea... | 23:07 |
| dougwig | blogan: talk to me about overloading the delete endpoint. | 23:07 |
| dougwig | xgerman: extension stuff is good. but modifying existing endpoints is supposed to be restricted to additive things like adding properties. changing semantic behavior is a no-no. blogan, did you have a reason for that? if not, we can respin quickly with separate endpoints. | 23:12 |
| *** yamamoto_ has joined #openstack-lbaas | 23:12 | |
| dougwig | and of course, i'm about to board a plane. sigh. | 23:12 |
| xgerman | well, this is what I have been told to do at the mid cycle — | 23:16 |
| *** ajmiller has quit IRC | 23:17 | |
| *** ajmiller has joined #openstack-lbaas | 23:18 | |
| *** kevo has quit IRC | 23:19 | |
| openstackgerrit | Merged openstack/neutron-lbaas: Adds Cascade Delete for LoadBalancers to Octavia Driver https://review.openstack.org/284340 | 23:20 |
| rm_work | grats | 23:22 |
| rm_work | or is that good or not | 23:22 |
| xgerman | rm_work you are missing the sh*tstorm | 23:23 |
| xgerman | ? | 23:23 |
| rm_work | apparenty | 23:23 |
| rm_work | *apparently | 23:23 |
| rm_work | I've been on internal stuff today | 23:23 |
| *** yamamoto_ has quit IRC | 23:24 | |
| johnsom | rm_work https://review.openstack.org/287494 | 23:24 |
| xgerman | I poked the bear... | 23:24 |
| rm_work | :/ | 23:24 |
| rm_work | i thought we were going to fix issues in bugfix patches this week/next | 23:25 |
| rm_work | does it break things THAT badly? | 23:25 |
| dougwig | i want a revert button on the revert, so i can revert the revert. | 23:25 |
| blogan | dougwig: im back | 23:25 |
| blogan | dougwig: i brought up the concern of it using the same endpoint adn the driver determines what to do at the midcycle, pretty sure you were okay with it too | 23:25 |
| dougwig | blogan: are non-additive semantic changes based on an extension really okay? i'm pretty sure that's not what i meant. | 23:27 |
| dougwig | blogan: now i'm asking our resident neutron api core (you). :) | 23:27 |
| blogan | dougwig: that was basically my concern with it, it would definitely hurt cloud inter-op, but at the midcycle that seemed to be okay | 23:29 |
| blogan | dougwig: i dont know if there's some rule against it | 23:29 |
| blogan | dougwig: seems like that could be an unwritten unknown rule | 23:29 |
| dougwig | it violates the additive rule, for sure. | 23:30 |
| blogan | dougwig: remember the options were: query parameter on the delete call (not possible), a /loadbalancers/{lb_id}/cascade_delete call which got shot down because it would either require a POST to delete something or a DELETE on a resource that doesn't seem to exist | 23:31 |
| openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: Revert "Adds Cascade option for load balancer drivers" https://review.openstack.org/287494 | 23:33 |
| dougwig | ok, that's super super weird. | 23:33 |
| blogan | dougwig: what is additive could be subjective too | 23:33 |
| blogan | what hapened? | 23:34 |
| xgerman | we did the DELETE on a non existent resource in Ocatvia | 23:34 |
| dougwig | applying the old patch on top of the revert fails in a really weird way. | 23:34 |
| xgerman | but I still think for neutron labs cascade is the right way and all deletes should work that way | 23:34 |
| dougwig | bumped it out, boarding plane | 23:37 |
| xgerman | Don’t they have wifi so we will hear from him soon ;-) | 23:39 |
| *** Purandar has quit IRC | 23:40 | |
| openstackgerrit | Stephen Balukoff proposed openstack/neutron-lbaas: Fix delete of session_persistence with pool update https://review.openstack.org/287004 | 23:41 |
| dougwig | Turboprop. No wifi | 23:43 |
| *** bana_k has quit IRC | 23:44 | |
| *** bana_k has joined #openstack-lbaas | 23:45 | |
| *** kevo has joined #openstack-lbaas | 23:45 | |
| johnsom | Nice | 23:45 |
| sbalukoff | ...I should probably prepare a release notes for that patch, given the CLI stuff we were talking about earlier. | 23:45 |
| johnsom | sbalukoff This is the patch we are likely not going forward with right? https://review.openstack.org/#/c/287118/ | 23:46 |
| sbalukoff | johnsom: Once I get it updated, we should go forward with that. | 23:46 |
| sbalukoff | There's one for neutron-lbaas and one for Octavia | 23:46 |
| sbalukoff | The session_persistence update bug affected both projects, but in different ways. | 23:47 |
| johnsom | Ok, so this isn't the NONE one, or there were multiple things there? | 23:47 |
| sbalukoff | Probably part of the reason it's been a pain to troubleshoot... | 23:47 |
| sbalukoff | Both of them have the session_persistence NONE type... I'm removing that from them now. | 23:47 |
| sbalukoff | The one I just uploaded has it removed. | 23:47 |
| johnsom | Oh, ok | 23:47 |
| sbalukoff | I need to do the Octavia side too... | 23:47 |
| johnsom | I am going to mark it WIP on the priority review etherpad | 23:48 |
| sbalukoff | But before I do I want to add a release note to that neutron-lbaas patch mentioning the CLI problem. | 23:48 |
| sbalukoff | johnsom: Oh, it should be out of WIP later today, unless I get waylaid by something else. | 23:48 |
| *** Purandar has joined #openstack-lbaas | 23:49 | |
| openstackgerrit | Stephen Balukoff proposed openstack/neutron-lbaas: Fix delete of session_persistence with pool update https://review.openstack.org/287004 | 23:54 |
| sbalukoff | Ok, that ^^^ should be ready for review. | 23:54 |
| sbalukoff | Working on updating the Octavia patch now.... | 23:54 |
| *** ajmiller has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!