| rm_work | sbalukoff: are your L7 changes rebased on top of those other db model changes? | 00:02 |
|---|---|---|
| sbalukoff | Yep. | 00:02 |
| rm_work | it doesn't ... look like it? I don't know, i can't freaking tell anymore, the new gerrit layout is not clear about commit chains | 00:02 |
| sbalukoff | Yeah... | 00:02 |
| sbalukoff | Well.. they're... mostly rebased. ;) | 00:02 |
| rm_work | lol k | 00:02 |
| sbalukoff | The only difference is a commit message | 00:02 |
| sbalukoff | (Adding the Closes-Bug: tag) | 00:02 |
| blogan | reedip: ping | 00:03 |
| rm_work | well when you fix that I can +2 that one... i wonder if that affects anything else tho | 00:03 |
| sbalukoff | rm_work: I'm looking. :p | 00:05 |
| sbalukoff | that was a really good catch. | 00:05 |
| rm_work | that's one of the kinds of things my brain flags immediately just scanning through | 00:06 |
| rm_work | reassignments | 00:07 |
| johnsom | rm_work https://bugs.launchpad.net/octavia/+bug/1543269 Nova and Neutron client factories can return incorrect clients on second call | 00:07 |
| openstack | Launchpad bug 1543269 in octavia "Nova and Neutron client factories can return incorrect clients on second call" [High,New] | 00:07 |
| rm_work | johnsom: yeah? | 00:07 |
| johnsom | rm_work What is the impact of this? Something we need to target for Mitaka? | 00:07 |
| *** jorgem is now known as jorgem[away] | 00:07 | |
| rm_work | probably ASAP if anyone is using admin tenants for anything | 00:07 |
| rm_work | which I think we are internally... | 00:07 |
| rm_work | and probably most deployers will be... | 00:07 |
| rm_work | or rather, admin + non-admin | 00:07 |
| rm_work | IMO the quick fix is to just ... not cache | 00:08 |
| rm_work | the more in-depth fix is to cache on a lookup dict based on a tuple | 00:08 |
| rm_work | but | 00:08 |
| blogan | johnsom: for Newton, we need to start trying to allow a better way to manage the amphorae than the single mgmt net | 00:08 |
| rm_work | honestly it will probably be fine just not cacheing | 00:09 |
| rm_work | can't have cache invalidation issues if you don't cache | 00:09 |
| johnsom | blogan Don't disagree. Want to write up an RFE with a proposal? | 00:09 |
| johnsom | rm_work Ok, I am going to tag for mitaka for now | 00:09 |
| blogan | yeah i don't think we need to cache, if we're using the keystone session thats doing a good amount of caching for us, the ones we care about really | 00:09 |
| rm_work | so that's a super easy one if someone wants to pick it up | 00:10 |
| blogan | johnsom: i have one idea but i may have more by the end of the week | 00:10 |
| rm_work | anyone still looking for ATC? :P | 00:10 |
| johnsom | Ok, cool. | 00:10 |
| johnsom | me, me, oh just kidding | 00:10 |
| blogan | johnsom: actually, we can talk about it at tomorrows meeting | 00:11 |
| blogan | got an agenda up yet that i can add to? | 00:11 |
| johnsom | blogan Sure. The agenda might be a bit full, but go ahead. Yes, I posted one today | 00:11 |
| blogan | johnsom: our meetings hardly ever go the full time, if i'm a betting man we'll have time | 00:12 |
| johnsom | There is at least one shed involved | 00:12 |
| rm_work | BLUE | 00:12 |
| rm_work | I vote blue | 00:12 |
| johnsom | RED | 00:13 |
| rm_work | damnit johnsom, i don't know if we can be friends anymore | 00:13 |
| johnsom | fine | 00:13 |
| johnsom | Grin | 00:13 |
| rm_work | maybe PURPLE! :P | 00:13 |
| rm_work | lol, these docstrings: https://review.openstack.org/#/c/278223/5/octavia/common/jinja/haproxy/jinja_cfg.py | 00:14 |
| rm_work | totally avoiding the whole point of the pep8 requirements for docstrings | 00:14 |
| rm_work | T_T | 00:14 |
| johnsom | hahaha | 00:15 |
| rm_work | supposed to be enforcing a *summary line* followed by a blank line followed by details | 00:15 |
| rm_work | not ... "whelp it's forcing me to make a new line with a blank" | 00:15 |
| johnsom | Right. It is consistent with the above _transform_pool though | 00:15 |
| rm_work | yes | 00:15 |
| rm_work | but T_T | 00:15 |
| johnsom | copy/paste | 00:15 |
| rm_work | yeah | 00:15 |
| rm_work | i'm not going to -1, but i am going to be sad | 00:16 |
| rm_work | and http://shamenun.com/ | 00:16 |
| johnsom | Haha | 00:17 |
| johnsom | I want the crack of a ruler on someone's wrist too | 00:17 |
| rm_work | ah he also forgot to add "Bluebox, an IBM company" to a bunch of copyright headers :P | 00:19 |
| * rm_work trolls | 00:19 | |
| * johnsom throws the eraser at rm_work | 00:20 | |
| blogan | we should put a specific hacking check for that | 00:21 |
| blogan | johnsom: added 2 items to the agenda, if we dont get to htem, we dont get to them | 00:21 |
| blogan | i actually have another thing to talk about too, which kind of ties into all of this | 00:22 |
| xgerman | I really hate seeing two (c_ lines | 00:22 |
| blogan | what the hell is (c_ lines? | 00:22 |
| johnsom | blogan Add away | 00:22 |
| blogan | copyright? | 00:22 |
| xgerman | yep (c_ | 00:22 |
| * xgerman can’t type | 00:22 | |
| xgerman | copyright | 00:23 |
| xgerman | but we had that discussion... | 00:23 |
| blogan | johnsom: before i add that 3rd one, mind if we talk about it here fist? | 00:24 |
| johnsom | Sure. I added another option too | 00:25 |
| blogan | i can totally see it being either 1) i'm a moron and not thinking correctly or 2) a real problem | 00:25 |
| * johnsom thinks that is too easy.... | 00:25 | |
| blogan | so we have the possibility to have many controller workers | 00:25 |
| blogan | haha | 00:25 |
| johnsom | Right | 00:25 |
| blogan | how do we get those controller workers on the mgmt net wihtout tieing ourselves to a specific neutron plugin? | 00:26 |
| blogan | tying | 00:26 |
| xgerman | assume they can recieve routed traffic from there? | 00:26 |
| xgerman | how is an implementation detail | 00:27 |
| johnsom | Not following on the tying to neutron plugin | 00:27 |
| blogan | xgerman: well the controller worker needs an ip on that mgmt net, so theoretically, we'd need to create a port for it | 00:27 |
| johnsom | Yes | 00:27 |
| xgerman | you only need an ip for the health stuff | 00:28 |
| rm_work | that's ... an administration level thing, right? | 00:28 |
| rm_work | like, the deployer would set up a box/vm/whatever and connect it up | 00:28 |
| blogan | johnsom: kind of like right now, for single host, to get the heartbeats working i had to create an ovs interface that is on the mgmt network | 00:28 |
| *** yuanying_ has joined #openstack-lbaas | 00:28 | |
| rm_work | and then run the controller on it? | 00:28 |
| xgerman | (and even that’s debatable since you could just sue DNS, VIP, etc.) | 00:28 |
| xgerman | blogan, that’s an implementation detail… say I have a provide rent, connect a box and install the controller on that | 00:29 |
| johnsom | Yeah, whoever sets up the controllers will need to wire them up the mgmt lan they create for the conf | 00:29 |
| blogan | so we're saying deployers need to create those ports | 00:29 |
| xgerman | blogan, right | 00:29 |
| johnsom | Yes sir | 00:29 |
| rm_work | yep | 00:29 |
| blogan | and then they just create an interface with that port mac address and get the ip (dhcp or static) and thats all they need to do correct? | 00:30 |
| johnsom | cables don't plug themselves in the real world, so.... | 00:30 |
| xgerman | and if they like to use a UDP LB for the health messages more power to them | 00:30 |
| blogan | yes they do! | 00:30 |
| blogan | so i was going down the road of us having a mechanism that automatically creates all that for deployers | 00:31 |
| *** yuanying has quit IRC | 00:31 | |
| rm_work | Ansible? :P | 00:31 |
| rm_work | SaltStack? | 00:31 |
| xgerman | I think that’s what Octavia ansible is trying to do :-) | 00:31 |
| xgerman | heat? | 00:32 |
| blogan | no! | 00:32 |
| johnsom | contractors? | 00:32 |
| johnsom | fanatical support? | 00:32 |
| blogan | i'd like to actually test this out to make sure it works | 00:32 |
| xgerman | well, we are building a “reference architecture” internally | 00:33 |
| blogan | but that leads into my other option for not having the single mgmt met, having a mgmt net per lb, which has its flaws, one being the controller would need to also plug itself into each mgmt network | 00:33 |
| johnsom | Right, not good for scale or HA | 00:33 |
| blogan | xgerman: and interally we're basically just using a provider network so its not the same | 00:33 |
| xgerman | yeah, I think the mgmt net issue is something we need to think about though we also use a provider net | 00:34 |
| blogan | nope but we're not doing vrrp either, and scale isn't going to be a problem (if you were talking about us doing provider net) | 00:34 |
| xgerman | but we can always give a lit of nets and pick one at random | 00:35 |
| xgerman | I think the original plan was to be able to configure N mgmt nets which all would ropiute to controllers | 00:35 |
| xgerman | I am pretty sure one mgmt net per Lb will cause other scale problems | 00:35 |
| blogan | xgerman: well i think the better option would to maybe start thinking about clusters of controllers | 00:36 |
| blogan | so that every controller won't need to be hooked into every mgmt net | 00:36 |
| johnsom | blogan my comments were about mgmt net per lb | 00:36 |
| xgerman | well, if you use clusters, you can just have N complete Octavia insytalls | 00:36 |
| blogan | johnsom: ah okay, why wouldn't that be good for hA then? | 00:36 |
| xgerman | like 10 devtsacks ;-) | 00:36 |
| blogan | xgerman: lol i guess, but why have multiple apis? | 00:37 |
| johnsom | blogan you would have to plug those all into each and every controller. | 00:37 |
| xgerman | +1 | 00:37 |
| blogan | there'd be a scheduling need to decide what cluster should be used | 00:37 |
| johnsom | So, what is wrong with a controller net that routes with amphora networks? | 00:37 |
| blogan | johnsom: you mean the current setup? | 00:38 |
| xgerman | blogan you can always schedule with LBaaS V2; provider octavia1, octavia2 | 00:38 |
| blogan | xgerman: do flavors schedule or something? | 00:38 |
| *** ajmiller_ has joined #openstack-lbaas | 00:38 | |
| xgerman | today you can schedule on different providers | 00:39 |
| johnsom | blogan more than one "amphora" network that has a route to a "controllers" network. Plug controllers in controllers net, at boot time distribute amphora across the "amphora networks" | 00:39 |
| blogan | xgerman: yeah but there's not automatic scheduling, just what the user tells the api | 00:39 |
| xgerman | flavor just gives them a name — so instead of octavia1 you can say “gold" | 00:39 |
| xgerman | yep, wouldn’t be automatic | 00:39 |
| johnsom | Housekeeping could even add "amphora networks" as needed | 00:40 |
| blogan | johnsom: are you using amphora network in a generic way to encapsulate mgmt nets and ha nets? | 00:40 |
| johnsom | No | 00:41 |
| blogan | johnsom: then whats an amphora network? | 00:41 |
| *** ajmiller has quit IRC | 00:41 | |
| johnsom | New concept, network segments we plug amphora mgmt interface into, that via router has a path to "controllers network" | 00:42 |
| johnsom | Preferably IPv6, but that is probably just my mantra | 00:42 |
| blogan | ah okay i think i see what you mean now, using routers we'd be able to have one controller network to many amp networks | 00:42 |
| johnsom | Intended to get around to many instances on one neutron network | 00:43 |
| johnsom | blogan Bingo | 00:43 |
| blogan | well one network per controller | 00:43 |
| xgerman | who is using more than 200 lbs? | 00:43 |
| xgerman | and wouldn’t neutron up that number anyway? | 00:43 |
| johnsom | Why one network per controller? You can plug a bunch of them in the same network | 00:43 |
| blogan | xgerman: since yall got out of the public cloud space, just us | 00:43 |
| johnsom | Yeah, we had one customer with more than 200 lbs | 00:44 |
| xgerman | yeah, exactly... | 00:44 |
| * johnsom thinks because it was in beta and cheap | 00:44 | |
| blogan | johnsom: yeah true about the controllers being on one network, i was not thinking that out correctly | 00:45 |
| xgerman | I just don’t want us to over-engineer that bit | 00:45 |
| blogan | we'd never over-engineer anything, what do you mean??? | 00:46 |
| blogan | i kid | 00:46 |
| xgerman | yep | 00:46 |
| johnsom | Yeah, this is a bit of frosting on the cake. But if it's important (N)ow-ish, we (octavia team) could look at it. | 00:47 |
| johnsom | I thought you were just doing the provider network and calling it a day | 00:47 |
| blogan | nah i'd like to just have a plan or at least options | 00:47 |
| xgerman | I could get behind a lit of mgmt networks you can configure | 00:47 |
| xgerman | and they all plug into the controller net... | 00:47 |
| blogan | johnsom: we are but we also have a private cloud side that can't use that, and they bring up good points about the single mgmt net not being a good solution, which we all agreed in the beginning, we just needed something working | 00:48 |
| xgerman | but spinning up clusters and then scheduling between them... | 00:48 |
| blogan | yeah i think the controller network -> router -> amp netowrks is a good idea, though we've tied it to the cloud its being deployed on having L3 | 00:49 |
| *** yuanying_ has quit IRC | 00:49 | |
| blogan | xgerman: i wasn't suggesting we do the clusters right away! | 00:49 |
| xgerman | ok, glad we are on the same page ;-) | 00:49 |
| blogan | just a possible end goal | 00:49 |
| *** yuanying has joined #openstack-lbaas | 00:49 | |
| blogan | but this sounds like a better plan | 00:49 |
| johnsom | blogan This should be optional, so if no L3 routers you can still run like devstack or provider. I think that should be do-able | 00:50 |
| blogan | johnsom: agreed | 00:50 |
| *** SumitNaiksatam has quit IRC | 00:50 | |
| xgerman | well, if the interface is a int of networks the operator set up… it’s generic enough | 00:50 |
| xgerman | int=list | 00:51 |
| johnsom | Side note, does anybody know what we should do with this? https://bugs.launchpad.net/octavia/+bug/1541986 | 00:51 |
| openstack | Launchpad bug 1541986 in octavia " Implementing EventStreamer" [Undecided,New] | 00:51 |
| johnsom | It seems like it just assigned it back to us.... | 00:51 |
| xgerman | isn’t it done? | 00:51 |
| rm_work | err | 00:51 |
| rm_work | did the neutron piece merge? | 00:51 |
| johnsom | Yeah, but look at the content of the bug | 00:51 |
| rm_work | it did, right? | 00:51 |
| rm_work | oh right yeah that | 00:51 |
| johnsom | Dear bug triager. This bug was created since a commit was marked with DOCIMPACT. | 00:51 |
| xgerman | yeah, no docs | 00:52 |
| * xgerman wags finger | 00:52 | |
| johnsom | Those conf options are documented somewhere I think. Should I just change the title on this? | 00:53 |
| blogan | ah shit i forgot to take that DocImpact flag off since i made all the config options have a default | 00:53 |
| johnsom | invalid? | 00:54 |
| blogan | and i think if you default the config option you odn't need a DocImpact but i'm not entirely surea bout that | 00:54 |
| johnsom | blogan you want to close it? | 00:54 |
| blogan | johnsom: do you know | 00:54 |
| blogan | all these rules i can't keep track of | 00:54 |
| johnsom | I'm not either. Like I said, I think there is a doc with the options somewhere. | 00:54 |
| blogan | let me do some research real quick | 00:54 |
| johnsom | Thank you! | 00:55 |
| johnsom | blogan http://docs.openstack.org/liberty/config-reference/content/networking-plugin-lbaas.html | 00:57 |
| *** yuanying has quit IRC | 00:59 | |
| blogan | johnsom: isn't that autogenerated? | 00:59 |
| johnsom | It should be, but I'm pretty sure we updated it for liberty | 00:59 |
| blogan | well do we have an octavia equivalent? | 01:00 |
| johnsom | octavia.conf is at the bottom of that page | 01:00 |
| blogan | son of a, i see | 01:01 |
| *** doug-fish has quit IRC | 01:01 | |
| *** doug-fish has joined #openstack-lbaas | 01:02 | |
| johnsom | blogan, I am going to close this one invalid and open a "update docs for mitaka" bug | 01:03 |
| blogan | johnsom: okay | 01:03 |
| johnsom | Or dup it | 01:03 |
| blogan | when do we get to hire our technical writer? | 01:03 |
| johnsom | grin | 01:04 |
| johnsom | I do however know who updated it for Liberty and we might be able to bribe her | 01:04 |
| blogan | bribery is the best policy | 01:04 |
| *** yuanying has joined #openstack-lbaas | 01:06 | |
| johnsom | Ok, need to step away. I tagged 16 bugs with mitaka that we can discuss tomorrow. | 01:07 |
| openstackgerrit | Franklin Naval proposed openstack/octavia: Octavia: Basic LoadBalancer Scenario Test https://review.openstack.org/172199 | 01:08 |
| *** allan_h has quit IRC | 01:09 | |
| *** fnaval has quit IRC | 01:09 | |
| rm_work | kk | 01:11 |
| *** allan_h has joined #openstack-lbaas | 01:11 | |
| *** ducttape_ has joined #openstack-lbaas | 01:16 | |
| *** ducttape_ has quit IRC | 01:25 | |
| *** ducttape_ has joined #openstack-lbaas | 01:26 | |
| *** ducttape_ has quit IRC | 01:30 | |
| *** _cjones_ has quit IRC | 01:31 | |
| *** bana_k has quit IRC | 01:36 | |
| *** minwang2 has quit IRC | 01:44 | |
| *** doug-fish has quit IRC | 01:44 | |
| *** doug-fish has joined #openstack-lbaas | 01:44 | |
| *** doug-fis_ has joined #openstack-lbaas | 01:48 | |
| *** kev0 has quit IRC | 01:49 | |
| *** doug-fish has quit IRC | 01:49 | |
| *** doug-fis_ has quit IRC | 01:52 | |
| *** bana_k has joined #openstack-lbaas | 01:54 | |
| *** madhu_ak has quit IRC | 01:57 | |
| *** Aish has quit IRC | 01:59 | |
| *** yamamoto_ has joined #openstack-lbaas | 02:06 | |
| *** yamamoto_ has quit IRC | 02:09 | |
| *** ducttape_ has joined #openstack-lbaas | 02:23 | |
| *** fnaval has joined #openstack-lbaas | 02:28 | |
| *** Bjoern has joined #openstack-lbaas | 02:32 | |
| *** doug-fish has joined #openstack-lbaas | 02:38 | |
| *** doug-fish has quit IRC | 02:39 | |
| *** doug-fish has joined #openstack-lbaas | 02:40 | |
| *** ajmiller_ has quit IRC | 02:44 | |
| *** Aish has joined #openstack-lbaas | 02:51 | |
| *** Aish has left #openstack-lbaas | 02:51 | |
| *** doug-fish has quit IRC | 02:51 | |
| *** ducttape_ has quit IRC | 02:52 | |
| *** Bjoern has quit IRC | 03:04 | |
| *** Bjoern has joined #openstack-lbaas | 03:05 | |
| *** Aish has joined #openstack-lbaas | 03:06 | |
| *** Aish has quit IRC | 03:06 | |
| *** Bjoern has quit IRC | 03:09 | |
| *** yamamoto has joined #openstack-lbaas | 03:10 | |
| *** ducttape_ has joined #openstack-lbaas | 03:15 | |
| *** SumitNaiksatam has joined #openstack-lbaas | 03:20 | |
| openstackgerrit | Franklin Naval proposed openstack/octavia: Octavia: Basic LoadBalancer Scenario Test https://review.openstack.org/172199 | 03:20 |
| *** allan_h has quit IRC | 03:25 | |
| *** yuanying has quit IRC | 03:30 | |
| *** yuanying has joined #openstack-lbaas | 03:32 | |
| *** yuanying has quit IRC | 04:05 | |
| *** yuanying has joined #openstack-lbaas | 04:06 | |
| *** yuanying has quit IRC | 04:07 | |
| *** yuanying_ has joined #openstack-lbaas | 04:07 | |
| *** TrevorV has joined #openstack-lbaas | 04:18 | |
| *** ducttape_ has quit IRC | 04:30 | |
| *** ducttape_ has joined #openstack-lbaas | 04:33 | |
| *** links has joined #openstack-lbaas | 04:41 | |
| *** ducttape_ has quit IRC | 04:42 | |
| *** ducttape_ has joined #openstack-lbaas | 04:45 | |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Fix model update flows https://review.openstack.org/280478 | 04:47 |
| *** ducttape_ has quit IRC | 04:52 | |
| *** TrevorV has quit IRC | 04:56 | |
| *** bana_k has quit IRC | 04:57 | |
| *** links has quit IRC | 04:57 | |
| *** neelashah has joined #openstack-lbaas | 04:58 | |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Add L7 database structures https://review.openstack.org/265430 | 05:03 |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Update repos for L7 policy / methods https://review.openstack.org/265529 | 05:18 |
| *** neelashah has quit IRC | 05:20 | |
| *** armax has quit IRC | 05:23 | |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Update repos for L7 rules / validations https://review.openstack.org/276643 | 05:27 |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Add L7 api - policies https://review.openstack.org/265690 | 05:35 |
| openstackgerrit | Franklin Naval proposed openstack/octavia: Octavia: Basic LoadBalancer Scenario Test https://review.openstack.org/172199 | 05:43 |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Add L7 api - rules https://review.openstack.org/277718 | 05:44 |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Add L7 controller worker flows and tasks https://review.openstack.org/277768 | 05:53 |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Add L7 jinja template updates https://review.openstack.org/278223 | 06:04 |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Add L7 documentation https://review.openstack.org/278830 | 06:11 |
| sbalukoff | johnsom, blogan, rm_work: Ok! Fixes are in and rebased. The last patch in the chain is https://review.openstack.org/278830 Please run your smoke tests against that! | 06:12 |
| sbalukoff | In the mean time, it looks like jenkins is randomly failing on tests in the chain. That's... annoying. I'mma look closer into this to see if it's something I did, or if it's jenkins acting up again. | 06:13 |
| *** allan_h has joined #openstack-lbaas | 06:21 | |
| sbalukoff | mirror.iad.rax.openstack.org appears to be having issues... so packages aren't reliably downloading. And this is leading to the errors in Jenkins. | 06:22 |
| sbalukoff | Yeah, all the errors I'm seeing here (except the ongoing session_persistence bug) seem to be caused by this. | 06:24 |
| *** allan_h has quit IRC | 06:43 | |
| *** numans has joined #openstack-lbaas | 07:07 | |
| *** numans has quit IRC | 07:07 | |
| *** numans has joined #openstack-lbaas | 07:08 | |
| *** kobis has joined #openstack-lbaas | 07:21 | |
| *** bana_k has joined #openstack-lbaas | 07:31 | |
| *** kobis has quit IRC | 07:31 | |
| *** kobis has joined #openstack-lbaas | 07:37 | |
| *** kevo has joined #openstack-lbaas | 07:38 | |
| *** rcernin has joined #openstack-lbaas | 07:42 | |
| *** bana_k has quit IRC | 08:06 | |
| *** bochi-michael has joined #openstack-lbaas | 08:07 | |
| *** jwarendt has quit IRC | 08:25 | |
| *** kevo has quit IRC | 08:25 | |
| *** pcaruana has joined #openstack-lbaas | 08:42 | |
| *** jschwarz has joined #openstack-lbaas | 08:54 | |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Fix health monitor URL in API documentation https://review.openstack.org/281126 | 09:00 |
| *** eranra has joined #openstack-lbaas | 09:16 | |
| *** paco20151113 has joined #openstack-lbaas | 09:32 | |
| *** barclaac has quit IRC | 09:52 | |
| *** ramishra has quit IRC | 09:52 | |
| *** ctracey has quit IRC | 09:52 | |
| *** dougwig has quit IRC | 09:52 | |
| *** _laco has quit IRC | 09:52 | |
| *** redrobot has quit IRC | 09:52 | |
| *** amitry has quit IRC | 09:52 | |
| *** xgerman has quit IRC | 09:52 | |
| *** blogan has quit IRC | 09:52 | |
| *** mestery has quit IRC | 09:52 | |
| *** redrobot has joined #openstack-lbaas | 09:52 | |
| *** barclaac has joined #openstack-lbaas | 09:52 | |
| *** blogan has joined #openstack-lbaas | 09:53 | |
| *** redrobot is now known as Guest53162 | 09:53 | |
| *** mestery has joined #openstack-lbaas | 09:53 | |
| *** paco20151113 has quit IRC | 09:56 | |
| *** amitry has joined #openstack-lbaas | 10:00 | |
| *** xgerman has joined #openstack-lbaas | 10:01 | |
| *** ramishra has joined #openstack-lbaas | 10:01 | |
| *** ramishra has quit IRC | 10:03 | |
| *** jschwarz has quit IRC | 10:03 | |
| *** rcernin has quit IRC | 10:03 | |
| *** jschwarz has joined #openstack-lbaas | 10:03 | |
| *** rcernin has joined #openstack-lbaas | 10:04 | |
| *** bochi-michael has quit IRC | 10:05 | |
| *** dougwig has joined #openstack-lbaas | 10:06 | |
| *** _laco has joined #openstack-lbaas | 10:06 | |
| *** ctracey has joined #openstack-lbaas | 10:07 | |
| *** mestery has quit IRC | 10:08 | |
| *** SumitNaiksatam has quit IRC | 10:08 | |
| *** yamamoto has quit IRC | 10:08 | |
| *** cgross- has quit IRC | 10:08 | |
| *** clduser has quit IRC | 10:08 | |
| *** chlong has quit IRC | 10:08 | |
| *** ajo has quit IRC | 10:08 | |
| *** clduser has joined #openstack-lbaas | 10:08 | |
| *** cgross has joined #openstack-lbaas | 10:08 | |
| *** ajo has joined #openstack-lbaas | 10:08 | |
| *** mestery has joined #openstack-lbaas | 10:08 | |
| *** yamamoto has joined #openstack-lbaas | 10:08 | |
| *** chlong has joined #openstack-lbaas | 10:08 | |
| *** ramishra has joined #openstack-lbaas | 10:12 | |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Delete SSH amphora driver https://review.openstack.org/281172 | 10:29 |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Delete SSH amphora driver https://review.openstack.org/281172 | 10:31 |
| *** ctracey has quit IRC | 10:49 | |
| *** amitry has quit IRC | 10:49 | |
| *** hockeynut has quit IRC | 10:49 | |
| *** lmiccini has quit IRC | 10:49 | |
| *** logan- has quit IRC | 10:49 | |
| *** mhayden has quit IRC | 10:49 | |
| *** kfox1111 has quit IRC | 10:49 | |
| *** reedip has quit IRC | 10:49 | |
| *** hockeynut has joined #openstack-lbaas | 10:49 | |
| *** reedip has joined #openstack-lbaas | 10:50 | |
| *** logan- has joined #openstack-lbaas | 10:52 | |
| *** mariusv has quit IRC | 10:53 | |
| *** mhayden has joined #openstack-lbaas | 10:53 | |
| *** kfox1111 has joined #openstack-lbaas | 10:54 | |
| *** amitry has joined #openstack-lbaas | 10:56 | |
| *** lmiccini has joined #openstack-lbaas | 10:57 | |
| *** ctracey has joined #openstack-lbaas | 10:58 | |
| *** rcernin has quit IRC | 11:03 | |
| *** hockeynut has quit IRC | 11:04 | |
| *** ramishra has quit IRC | 11:04 | |
| *** xgerman has quit IRC | 11:04 | |
| *** blogan has quit IRC | 11:04 | |
| *** barclaac has quit IRC | 11:04 | |
| *** Guest53162 has quit IRC | 11:04 | |
| *** numans has quit IRC | 11:04 | |
| *** bedis_ has quit IRC | 11:04 | |
| *** prabampm has quit IRC | 11:04 | |
| *** thomasem has quit IRC | 11:04 | |
| *** kbyrne has quit IRC | 11:04 | |
| *** HenryG has quit IRC | 11:04 | |
| *** rm_work has quit IRC | 11:04 | |
| *** mugsie has quit IRC | 11:04 | |
| *** barclaac has joined #openstack-lbaas | 11:04 | |
| *** redrobot has joined #openstack-lbaas | 11:04 | |
| *** hockeynut has joined #openstack-lbaas | 11:04 | |
| *** bedis has joined #openstack-lbaas | 11:04 | |
| *** blogan has joined #openstack-lbaas | 11:04 | |
| *** redrobot is now known as Guest1856 | 11:04 | |
| *** thomasem has joined #openstack-lbaas | 11:04 | |
| *** mugsie has joined #openstack-lbaas | 11:05 | |
| *** prabampm has joined #openstack-lbaas | 11:05 | |
| *** numans has joined #openstack-lbaas | 11:05 | |
| *** HenryG has joined #openstack-lbaas | 11:05 | |
| *** rm_work has joined #openstack-lbaas | 11:06 | |
| *** kbyrne has joined #openstack-lbaas | 11:07 | |
| *** ramishra has joined #openstack-lbaas | 11:10 | |
| *** xgerman has joined #openstack-lbaas | 11:17 | |
| *** yamamoto has quit IRC | 11:32 | |
| *** kbyrne has quit IRC | 11:39 | |
| *** logan- has quit IRC | 11:39 | |
| *** ptoohill has quit IRC | 11:39 | |
| *** pck has quit IRC | 11:39 | |
| *** kbyrne has joined #openstack-lbaas | 11:40 | |
| *** ptoohill has joined #openstack-lbaas | 11:40 | |
| *** logan- has joined #openstack-lbaas | 11:42 | |
| *** pck has joined #openstack-lbaas | 11:43 | |
| *** thomasem has quit IRC | 11:45 | |
| *** thomasem has joined #openstack-lbaas | 11:46 | |
| *** rcernin has joined #openstack-lbaas | 11:49 | |
| *** HenryG has quit IRC | 11:50 | |
| *** clduser has quit IRC | 11:50 | |
| *** mdavidson has quit IRC | 11:50 | |
| *** pai15- has quit IRC | 11:50 | |
| *** crc32|znc has quit IRC | 11:50 | |
| *** clduser has joined #openstack-lbaas | 11:50 | |
| *** pai15 has joined #openstack-lbaas | 11:50 | |
| *** crc32|znc has joined #openstack-lbaas | 11:50 | |
| *** HenryG has joined #openstack-lbaas | 11:51 | |
| *** mdavidson has joined #openstack-lbaas | 11:53 | |
| *** doug-fish has joined #openstack-lbaas | 11:58 | |
| *** openstack has joined #openstack-lbaas | 12:05 | |
| *** johnsom has quit IRC | 12:07 | |
| *** pc_m has quit IRC | 12:07 | |
| *** pc_m has joined #openstack-lbaas | 12:07 | |
| *** johnsom has joined #openstack-lbaas | 12:08 | |
| *** piet has quit IRC | 12:12 | |
| *** piet has joined #openstack-lbaas | 12:13 | |
| *** amitry has joined #openstack-lbaas | 12:14 | |
| *** piet has quit IRC | 12:15 | |
| *** piet has joined #openstack-lbaas | 12:20 | |
| *** piet has quit IRC | 12:22 | |
| *** piet has joined #openstack-lbaas | 12:22 | |
| *** amitry has quit IRC | 12:24 | |
| *** numans has quit IRC | 12:24 | |
| *** ajo has quit IRC | 12:24 | |
| *** itsuugo has quit IRC | 12:24 | |
| *** markvan_ has quit IRC | 12:24 | |
| *** sbalukoff has quit IRC | 12:24 | |
| *** jorgem[away] has quit IRC | 12:24 | |
| *** jidar has quit IRC | 12:24 | |
| *** Kiall has quit IRC | 12:24 | |
| *** zigo has quit IRC | 12:24 | |
| *** markvan has joined #openstack-lbaas | 12:25 | |
| *** sbalukoff has joined #openstack-lbaas | 12:25 | |
| *** Kiall has joined #openstack-lbaas | 12:25 | |
| *** Kiall has quit IRC | 12:25 | |
| *** Kiall has joined #openstack-lbaas | 12:25 | |
| *** jorgem has joined #openstack-lbaas | 12:25 | |
| *** numans has joined #openstack-lbaas | 12:25 | |
| *** markvan has quit IRC | 12:25 | |
| *** markvan has joined #openstack-lbaas | 12:25 | |
| *** ajo has joined #openstack-lbaas | 12:25 | |
| *** numans has quit IRC | 12:25 | |
| *** numans has joined #openstack-lbaas | 12:25 | |
| *** mariusv has joined #openstack-lbaas | 12:25 | |
| *** ajo has quit IRC | 12:25 | |
| *** ajo has joined #openstack-lbaas | 12:25 | |
| *** itsuugo has joined #openstack-lbaas | 12:26 | |
| *** zigo has joined #openstack-lbaas | 12:26 | |
| *** mariusv is now known as Guest45229 | 12:26 | |
| *** piet has quit IRC | 12:26 | |
| *** yamamoto has joined #openstack-lbaas | 12:27 | |
| *** jidar has joined #openstack-lbaas | 12:27 | |
| *** yamamoto_ has joined #openstack-lbaas | 12:29 | |
| *** yamamoto has quit IRC | 12:31 | |
| *** hockeynut has quit IRC | 12:32 | |
| *** eranra has quit IRC | 12:32 | |
| *** openstackgerrit_ has quit IRC | 12:32 | |
| *** openstackgerrit has quit IRC | 12:32 | |
| *** Dave has quit IRC | 12:32 | |
| *** davidlenwell has quit IRC | 12:32 | |
| *** eranra has joined #openstack-lbaas | 12:32 | |
| *** eranra has quit IRC | 12:32 | |
| *** eranra has joined #openstack-lbaas | 12:32 | |
| *** davidlenwell has joined #openstack-lbaas | 12:33 | |
| *** Dave has joined #openstack-lbaas | 12:33 | |
| *** yamamoto_ has quit IRC | 12:33 | |
| *** openstackgerrit has joined #openstack-lbaas | 12:34 | |
| *** openstackgerrit_ has joined #openstack-lbaas | 12:34 | |
| *** hockeynut has joined #openstack-lbaas | 12:36 | |
| *** amitry has joined #openstack-lbaas | 12:41 | |
| *** rtheis has joined #openstack-lbaas | 12:43 | |
| *** yamamoto has joined #openstack-lbaas | 12:50 | |
| *** barclaac has quit IRC | 12:51 | |
| *** dougwig has quit IRC | 12:53 | |
| *** barclaac has joined #openstack-lbaas | 12:53 | |
| *** piet has joined #openstack-lbaas | 12:59 | |
| *** alejandrito has joined #openstack-lbaas | 13:02 | |
| *** piet has quit IRC | 13:05 | |
| *** piet has joined #openstack-lbaas | 13:06 | |
| *** piet has quit IRC | 13:08 | |
| *** ducttape_ has joined #openstack-lbaas | 13:11 | |
| *** Kiall has quit IRC | 13:18 | |
| *** Kiall has joined #openstack-lbaas | 13:19 | |
| *** yamamoto has quit IRC | 13:19 | |
| *** yamamoto has joined #openstack-lbaas | 13:25 | |
| *** ducttape_ has quit IRC | 13:27 | |
| *** yamamoto has quit IRC | 13:38 | |
| *** yamamoto has joined #openstack-lbaas | 13:43 | |
| *** doug-fish has quit IRC | 13:44 | |
| *** yamamoto has quit IRC | 13:50 | |
| *** Kiall has quit IRC | 13:54 | |
| *** yamamoto has joined #openstack-lbaas | 13:57 | |
| *** Kiall has joined #openstack-lbaas | 13:59 | |
| *** neelashah has joined #openstack-lbaas | 13:59 | |
| mhayden | i drafted some docs for LBaaS v2 in the networking guide: https://review.openstack.org/#/c/278128/ | 14:05 |
| mhayden | if anyone has any cycles available to weigh in there, i'd appreciate it! :) | 14:05 |
| *** dougwig has joined #openstack-lbaas | 14:11 | |
| *** jwarendt has joined #openstack-lbaas | 14:24 | |
| *** localloop127 has joined #openstack-lbaas | 14:26 | |
| *** doug-fish has joined #openstack-lbaas | 14:28 | |
| *** jwarendt has quit IRC | 14:29 | |
| *** dougwig has quit IRC | 14:33 | |
| *** yamamoto has quit IRC | 14:35 | |
| *** ducttape_ has joined #openstack-lbaas | 14:50 | |
| *** TrevorV has joined #openstack-lbaas | 14:55 | |
| *** piet has joined #openstack-lbaas | 15:01 | |
| *** Bjoern has joined #openstack-lbaas | 15:14 | |
| *** piet has quit IRC | 15:16 | |
| *** piet has joined #openstack-lbaas | 15:18 | |
| *** numans has quit IRC | 15:26 | |
| *** ajmiller has joined #openstack-lbaas | 15:26 | |
| *** neelashah has quit IRC | 15:27 | |
| *** neelashah has joined #openstack-lbaas | 15:30 | |
| *** piet has quit IRC | 15:52 | |
| *** piet has joined #openstack-lbaas | 15:53 | |
| *** piet has quit IRC | 16:06 | |
| *** _cjones_ has joined #openstack-lbaas | 16:08 | |
| *** _cjones_ has quit IRC | 16:08 | |
| *** piet has joined #openstack-lbaas | 16:12 | |
| *** fnaval has quit IRC | 16:13 | |
| TrevorV | sbalukoff are you online right now? | 16:15 |
| *** dougw2 has joined #openstack-lbaas | 16:15 | |
| *** piet has quit IRC | 16:17 | |
| *** piet has joined #openstack-lbaas | 16:17 | |
| *** dougwig has joined #openstack-lbaas | 16:18 | |
| *** dougw2 has quit IRC | 16:19 | |
| *** armax has joined #openstack-lbaas | 16:19 | |
| *** Guest46279 is now known as redrobot | 16:22 | |
| *** fnaval has joined #openstack-lbaas | 16:26 | |
| *** piet has quit IRC | 16:26 | |
| *** jwarendt has joined #openstack-lbaas | 16:27 | |
| *** pcaruana has quit IRC | 16:35 | |
| *** rm_work has quit IRC | 16:37 | |
| *** rm_work has joined #openstack-lbaas | 16:37 | |
| *** allan_h has joined #openstack-lbaas | 16:53 | |
| *** minwang2 has joined #openstack-lbaas | 17:09 | |
| *** madhu_ak has joined #openstack-lbaas | 17:18 | |
| rm_work | sbalukoff: your l7 chain ready? | 17:22 |
| *** localloo1 has joined #openstack-lbaas | 17:24 | |
| *** localloop127 has quit IRC | 17:26 | |
| *** jschwarz has quit IRC | 17:28 | |
| johnsom | rm_work From a message last night he said it was, top of chain: https://review.openstack.org/278830 | 17:29 |
| rm_work | kk | 17:30 |
| *** xgerman has quit IRC | 17:31 | |
| *** xgerman has joined #openstack-lbaas | 17:32 | |
| *** davidlenwell has quit IRC | 17:34 | |
| *** zigo has quit IRC | 17:34 | |
| *** hockeynut has quit IRC | 17:35 | |
| *** sbalukoff has quit IRC | 17:35 | |
| *** kevo has joined #openstack-lbaas | 17:35 | |
| *** dougw2 has joined #openstack-lbaas | 17:36 | |
| *** zigo has joined #openstack-lbaas | 17:36 | |
| *** davidlenwell has joined #openstack-lbaas | 17:37 | |
| *** hockeynut has joined #openstack-lbaas | 17:39 | |
| *** sbalukoff has joined #openstack-lbaas | 17:39 | |
| *** Aish has joined #openstack-lbaas | 17:47 | |
| sbalukoff | Hey folks! | 17:49 |
| sbalukoff | rm_work: Yep, L7 chain is ready. | 17:49 |
| sbalukoff | TrevorV: I'm online now. | 17:50 |
| *** kobis has quit IRC | 17:50 | |
| johnsom | Patching the libc fun on my devstack, then restacking with that L7 to kick the tires | 17:52 |
| *** eranra has quit IRC | 17:56 | |
| sbalukoff | Yay! | 17:56 |
| *** localloo1 has quit IRC | 17:57 | |
| rm_work | leaving libc bug in place, stacking with l7 patches | 17:59 |
| johnsom | Well, previous testing with shared pools left a bunch of ports/sec groups laying around and I was out of quota, so instead of bumping those I just decided to start fresh. | 18:00 |
| rm_work | i reformat every single run, so i don't care so much if someone hacks one of my random VMs lol | 18:01 |
| mhayden | added octavia to the docs patch if someone wants to verify my work -> https://review.openstack.org/#/c/278128/13 | 18:03 |
| *** woodster_ has joined #openstack-lbaas | 18:08 | |
| *** bana_k has joined #openstack-lbaas | 18:10 | |
| johnsom | mhayden Excellent, thanks! | 18:11 |
| mhayden | no problem | 18:11 |
| sbalukoff | mhayden: Having a look now! | 18:15 |
| * mhayden woots! | 18:15 | |
| mhayden | i enjoy writing documentation, but i'm still wrapping my head around octavia | 18:16 |
| mhayden | but i plan on being in the meeting today -- i heard blogan was bringing punch and pie | 18:22 |
| sbalukoff | Sweet! | 18:23 |
| johnsom | Mmm, pie | 18:23 |
| johnsom | Well, L7 created the haproxy.cfg ok this time, so improvement! | 18:31 |
| *** rcernin has quit IRC | 18:36 | |
| *** neelashah has quit IRC | 18:38 | |
| blogan | mhayden: i'm bringing piss and vinegar | 18:40 |
| blogan | i mean honey and sugar | 18:40 |
| blogan | always get those mixed up | 18:40 |
| mhayden | hmm, i guess everyone has their own tastes | 18:40 |
| mhayden | :P | 18:40 |
| *** eranra has joined #openstack-lbaas | 18:40 | |
| sbalukoff | mhayden: You have earned your first "-1" from me. :) Please note, this is how I show my love. | 18:43 |
| johnsom | sbalukoff Does putting the VIP on the public network under devstack now work? There was some ACL funniness that caused it to not work previously. | 18:46 |
| sbalukoff | johnsom: I'll give it a shot and let you know for sure. In theory it should work, but you're right that we might have a bug lurking in there which prevents it. | 18:47 |
| johnsom | It was something strange about the way that network is created, outside of our stuff. | 18:47 |
| sbalukoff | Aah! Ok, well, if it breaks in my test, I'll file and bug and get on it. | 18:47 |
| johnsom | We just always said, you need floats if you want an IP on devstack's "public" | 18:48 |
| sbalukoff | I know that we have some people internally here that want Octavia to work without floating IPs, so it's going to be important from my perspective for that to work. | 18:48 |
| johnsom | If you create your own external network it works fine | 18:48 |
| sbalukoff | johnsom: Did you see my note about the listener stats interface bug? | 18:49 |
| johnsom | Nope, can you give me a pointer? | 18:50 |
| sbalukoff | Sure... hold on... | 18:50 |
| *** localloo1 has joined #openstack-lbaas | 18:50 | |
| sbalukoff | https://bugs.launchpad.net/octavia/+bug/1543356 | 18:50 |
| openstack | Launchpad bug 1543356 in octavia "Errors found for octavia api----get listener stats" [Medium,Confirmed] - Assigned to Stephen Balukoff (sbalukoff) | 18:50 |
| *** eranra has quit IRC | 18:50 | |
| johnsom | Ok, sure, we can talk about changing it. I will pull the sharedpools tag from that bug. I bet the lbaasv2 sync puts those over in the neutron db and that is why the scenario tests/CLI work | 18:53 |
| sbalukoff | Aah, ok. | 18:53 |
| sbalukoff | I just figured that if we've never had working stats before, it's a good time to revisit that and see if we want to alter how it works, since we wouldn't be breaking any existing installations per se. | 18:55 |
| sbalukoff | But it sounds like Neutron LBaaS is a precedent that may have a pattern we want to follow there with Octavia. | 18:55 |
| sbalukoff | So... it may be a short discussion. :) | 18:55 |
| johnsom | Yeah, it has been working. I guess no one noticed that Octavia API part wasn't | 18:55 |
| mhayden | sbalukoff: ah, okay -- i think i addressed some of your concerns | 18:55 |
| *** doug-fis_ has joined #openstack-lbaas | 18:55 | |
| mhayden | some of that could be done in a follow-up patch by someone who knows octavia better than i do ;) | 18:56 |
| *** localloop127 has joined #openstack-lbaas | 18:56 | |
| *** localloo1 has quit IRC | 18:59 | |
| sbalukoff | mhayden: i'll have a look later today, eh! | 18:59 |
| *** doug-fish has quit IRC | 18:59 | |
| *** allan_h has quit IRC | 18:59 | |
| *** doug-fis_ has quit IRC | 19:00 | |
| *** allan_h has joined #openstack-lbaas | 19:03 | |
| *** doug-fish has joined #openstack-lbaas | 19:03 | |
| *** neelashah has joined #openstack-lbaas | 19:04 | |
| *** Bjoern has quit IRC | 19:06 | |
| *** rcernin has joined #openstack-lbaas | 19:06 | |
| *** Bjoern has joined #openstack-lbaas | 19:07 | |
| *** doug-fish has quit IRC | 19:07 | |
| *** eranra has joined #openstack-lbaas | 19:08 | |
| *** doug-fish has joined #openstack-lbaas | 19:13 | |
| johnsom | dougwig Can you kick this through? https://review.openstack.org/#/c/277556/ | 19:17 |
| dougwig | johnsom: aye | 19:18 |
| johnsom | Thank you! | 19:18 |
| TrevorV | Hey sbalukoff can I steal your attention for a minute | 19:22 |
| *** eranra has quit IRC | 19:24 | |
| sbalukoff | TrevorV attention thief! | 19:25 |
| TrevorV | YEs | 19:25 |
| TrevorV | That's me | 19:25 |
| sbalukoff | You have my attention. You bastard. | 19:26 |
| TrevorV | So, I'm still, STILL working on the get-me-a-loadbalancer change, but I'm working on it such that it includes L7 | 19:26 |
| sbalukoff | Ok. | 19:26 |
| TrevorV | Basically, I need to rebase my 2 commits on top of your stack of changes, or in this case, to make things simple, I want to base my API change on top of your L7 doc review, and then my CW change on top of my API review | 19:26 |
| sbalukoff | Ok. | 19:27 |
| TrevorV | However, to do this, I need your entire stack to be up-to-date to complete that. | 19:27 |
| TrevorV | IS that the case right now? | 19:27 |
| TrevorV | (I'm speaking in redundant fashion because my thoughts aren't 100% complete right now....) | 19:27 |
| sbalukoff | Yes-- that is to say, the first patch in the series is behind master by about 5 minor commits, but none of those conflict, so I was just going to leave it that way (unless something does conflict) so that people don't lose their place if they're actively reviewing and commenting. | 19:28 |
| sbalukoff | The nice part about L7 is it's almost entirely additive. So conflicts ought to be rare. | 19:28 |
| *** piet has joined #openstack-lbaas | 19:29 | |
| sbalukoff | Of course, the longer it goes without being merged as other stuff gets merged in the mean time, the more likely it is that something will conflict. :/ | 19:29 |
| TrevorV | Right. For example, the single-create for octavia is ready... except it needs L7 ha ha | 19:30 |
| TrevorV | So its not 100% ready | 19:30 |
| sbalukoff | Haha | 19:30 |
| TrevorV | Anway, I'm pulling down up-to-date l7-documentation review, and putting my reviews on top. Okay with you? | 19:30 |
| sbalukoff | Works for me! | 19:30 |
| TrevorV | Awesome. | 19:31 |
| TrevorV | See you in the dependency chain! | 19:31 |
| sbalukoff | Haha! | 19:31 |
| TrevorV | Just a note... seeing "Do you really want to submit the above commits?" as an option really really scares me, even though I literally just did "git review -d <l7 doc review number>" and then cherry-picked my review | 19:32 |
| TrevorV | sbalukoff ^^ | 19:32 |
| sbalukoff | Haha! Indeed-- there are lot of things in that chain, eh. | 19:33 |
| openstackgerrit | Trevor Vardeman proposed openstack/octavia: WIP - Get me a Load Balancer API https://review.openstack.org/256974 | 19:33 |
| *** neelashah1 has joined #openstack-lbaas | 19:41 | |
| *** neelashah has quit IRC | 19:41 | |
| *** armax has quit IRC | 19:42 | |
| *** ducttape_ has quit IRC | 19:42 | |
| *** piet has quit IRC | 19:46 | |
| *** allan_h has quit IRC | 19:47 | |
| *** evgenyf has joined #openstack-lbaas | 19:47 | |
| openstackgerrit | Trevor Vardeman proposed openstack/octavia: WIP - Get Me A Load Balancer Controller https://review.openstack.org/257013 | 19:48 |
| openstackgerrit | Trevor Vardeman proposed openstack/octavia: Update repos for L7 policy / methods https://review.openstack.org/265529 | 19:48 |
| openstackgerrit | Trevor Vardeman proposed openstack/octavia: Add L7 controller worker flows and tasks https://review.openstack.org/277768 | 19:48 |
| openstackgerrit | Trevor Vardeman proposed openstack/octavia: Add L7 api - rules https://review.openstack.org/277718 | 19:48 |
| openstackgerrit | Trevor Vardeman proposed openstack/octavia: Add L7 database structures https://review.openstack.org/265430 | 19:48 |
| openstackgerrit | Trevor Vardeman proposed openstack/octavia: WIP - Get me a Load Balancer API https://review.openstack.org/256974 | 19:48 |
| openstackgerrit | Trevor Vardeman proposed openstack/octavia: Add L7 api - policies https://review.openstack.org/265690 | 19:48 |
| openstackgerrit | Trevor Vardeman proposed openstack/octavia: Fix model update flows https://review.openstack.org/280478 | 19:48 |
| openstackgerrit | Trevor Vardeman proposed openstack/octavia: Add L7 jinja template updates https://review.openstack.org/278223 | 19:48 |
| openstackgerrit | Trevor Vardeman proposed openstack/octavia: Update data model conversion to be more complete https://review.openstack.org/280166 | 19:48 |
| openstackgerrit | Trevor Vardeman proposed openstack/octavia: Add L7 documentation https://review.openstack.org/278830 | 19:48 |
| openstackgerrit | Trevor Vardeman proposed openstack/octavia: Update repos for L7 rules / validations https://review.openstack.org/276643 | 19:48 |
| TrevorV | HOLY CRAP | 19:49 |
| TrevorV | I broke it. | 19:49 |
| TrevorV | See, I knew something was going to break. | 19:49 |
| sbalukoff | Uh-oh. | 19:49 |
| TrevorV | Idk what I broke, but I think something did. | 19:50 |
| rm_work | roooofl | 19:50 |
| TrevorV | Did it? Or did it just output all the changes in the chain for some reason? | 19:50 |
| rm_work | glad i already stacked | 19:50 |
| TrevorV | Now I'm confused. | 19:51 |
| rm_work | nope if it output them it made a new patchset | 19:51 |
| rm_work | which means it did something | 19:51 |
| rm_work | sbalukoff: gl;hf | 19:51 |
| blogan | if you didnt make any changes to the previous reviews, you just pushed up old PSes | 19:51 |
| blogan | guaranteed | 19:51 |
| TrevorV | Yeah... idk how though | 19:51 |
| blogan | sometimes its unavoidable with long dependency chains, especially if htey're not all rebased on each other's new PS'es | 19:51 |
| TrevorV | I did git review -d L7-docs and cherry-picked my API review | 19:52 |
| TrevorV | Then did a git review -d api-review and cherry picked the cw review | 19:52 |
| TrevorV | It shouldn't have broke anything.... | 19:52 |
| rm_work | yeah so if that CR wasn't based on the newest of EVERYTHING below it | 19:52 |
| rm_work | it will overwrite the CRs below it with older versions of themselves | 19:52 |
| rm_work | so like, if you have commits A/B/C chained, and you do like | 19:53 |
| *** allan_h has joined #openstack-lbaas | 19:53 | |
| rm_work | A1->B1->C1 | 19:53 |
| rm_work | then commit B2 | 19:53 |
| *** allan_h has quit IRC | 19:53 | |
| rm_work | you have A1->B2; B1->C1 | 19:53 |
| rm_work | then if you commit C2 it'll do: | 19:53 |
| sbalukoff | Er... I think I can just un-do this by committing my latest l7-doc patch set? | 19:54 |
| rm_work | A1->B3->C2 where B3==B1 | 19:54 |
| blogan | sbalukoff: yep | 19:54 |
| sbalukoff | (ie. the one from last night) | 19:54 |
| rm_work | sbalukoff: probably | 19:54 |
| sbalukoff | Ok: TrevorV: Mind if I do that? | 19:54 |
| TrevorV | Um, no? | 19:54 |
| TrevorV | But it has no changes | 19:54 |
| rm_work | yeah it should be fine | 19:54 |
| rm_work | it won't change his | 19:54 |
| rm_work | he'll just have to rebase again later | 19:54 |
| TrevorV | Sorry sbalukoff | 19:55 |
| TrevorV | If you can just push the change up and it fix the stack, then that'd be great. | 19:55 |
| TrevorV | I'm hoping your l7 changes get merged soon anyway | 19:55 |
| blogan | TrevorV: looks like you have the CW review directly on top of the L7 reviews | 19:56 |
| blogan | nope nvm | 19:56 |
| blogan | it just didn't push up a new API review | 19:56 |
| sbalukoff | So, this is weird: https://gist.github.com/sbalukoff/6606326f7ea63d82bab8 | 19:56 |
| blogan | just do a git review -d L7-docs, cherry-pick api, cherry-pick controller, git review | 19:57 |
| TrevorV | woah, what? | 19:57 |
| sbalukoff | Yeah... we're about to go into a meeting anyway. I'll have a closer look afterward. | 19:57 |
| blogan | TrevorV: sure you didn't do a bunch of cherry-picks or a rebase that would change those commit hashes? | 19:58 |
| TrevorV | I mentioned what I did already | 19:58 |
| TrevorV | I can gist my history | 19:58 |
| blogan | i know and if you ust cherry-picked on top of L7 that wouldn't change the L7 commit hashes, but they got changed | 19:58 |
| TrevorV | Okay, once again | 19:58 |
| *** blallau has joined #openstack-lbaas | 19:59 | |
| TrevorV | I did a review -d <l7-docs> | 19:59 |
| TrevorV | Cherry pick API review | 19:59 |
| TrevorV | review -d <api-review> | 19:59 |
| TrevorV | cherry pick cw review | 19:59 |
| TrevorV | and THAT broke it | 19:59 |
| blogan | and I am asking are you sure that is all you did? | 19:59 |
| blogan | oh wait | 19:59 |
| johnsom | Octavia meeting starting on #openstack-meeting-alt | 19:59 |
| blogan | oh wait after you did the cherry-pick of the API review youd didn't push it up? | 20:00 |
| blogan | bc review -d <api-review> would give you the old api review | 20:00 |
| TrevorV | blogan https://gist.github.com/TrevorV/a97abb094d93ec5757c7 | 20:01 |
| *** ducttape_ has joined #openstack-lbaas | 20:02 | |
| TrevorV | blogan I added a few more lines. The CW review needed a rebase, which made sense, but maybe was what caused the problem? The gist above is the history | 20:04 |
| blogan | and thats what changed the hashes | 20:04 |
| *** localloop127 has quit IRC | 20:05 | |
| TrevorV | That doesn't make sense to me. | 20:05 |
| TrevorV | Rebasing CW made the preview reviews change? | 20:05 |
| TrevorV | How? | 20:05 |
| rm_work | was gonna say, the first one was fine, i cheered (internally) for it just working | 20:05 |
| blogan | a rebase plays the commit back, but they're basically new commits | 20:05 |
| TrevorV | previous*** | 20:05 |
| blogan | when did it tell you it needed a rebase? | 20:06 |
| TrevorV | When I did git-review | 20:06 |
| *** localloop127 has joined #openstack-lbaas | 20:06 | |
| rm_work | AH | 20:06 |
| rm_work | that's what -R is for | 20:06 |
| rm_work | now I remember | 20:07 |
| TrevorV | in my gist, line 530 it said "rebase", line 531 I identified the conflict, and then made the change. | 20:07 |
| blogan | yeah -R or --no-rebase would have solved that but usually you dont need to use those | 20:07 |
| blogan | but it comes in handy with dependency chains sometimes | 20:07 |
| rm_work | i was trying to explain the use of -R earlier but it wasn't doing anything | 20:07 |
| rm_work | it's because it's for THIS problem, not the one i was trying to solve before | 20:07 |
| TrevorV | Once again, I don't follow. | 20:08 |
| TrevorV | I was rebasing my CW review, why does it update everything else? | 20:08 |
| TrevorV | I've never seen that before | 20:08 |
| blogan | i can't tell you why right now without seeing it happen honestly | 20:08 |
| *** Bjoern has quit IRC | 20:10 | |
| TrevorV | I see. Just incredibly confusing to me. API review went up, no problem. CW review goes up, changes all over? Just doesn't make sense | 20:10 |
| blogan | yeah sometimes you just dont know honestly, until you dive into and figure out why | 20:11 |
| *** Bjoern has joined #openstack-lbaas | 20:14 | |
| *** Bjoern is now known as Bjoern_zZzZzZzZ | 20:14 | |
| *** localloop127 has quit IRC | 20:15 | |
| *** localloop127 has joined #openstack-lbaas | 20:17 | |
| rm_work | there's some workflow guidelines you can follow to try to keep it from ever happening though | 20:20 |
| rm_work | my rules are: one commit at a time; git review -d <something>; git cherry-pick <something>; git review | 20:20 |
| rm_work | if you don't deviate AT ALL | 20:20 |
| rm_work | *usually* you will be ok | 20:20 |
| rm_work | but literally any little change to that and boom | 20:21 |
| rm_work | everything explodes and it's forensics time :P | 20:21 |
| *** allan_h has joined #openstack-lbaas | 20:25 | |
| *** allan_h has quit IRC | 20:27 | |
| *** allan_h has joined #openstack-lbaas | 20:27 | |
| *** Bjoern_zZzZzZzZ is now known as Bjoern | 20:40 | |
| *** allan_h has quit IRC | 20:45 | |
| blogan | my rule is if i think something might explode, check my local commit hashes with the ones in gerrit to be certain | 20:53 |
| TrevorV | rm_work that's EXACTLY what I did... look at the gist I sent blogan. | 20:54 |
| TrevorV | rm_work https://gist.github.com/TrevorV/a97abb094d93ec5757c7 | 20:55 |
| TrevorV | Just to save you the time | 20:55 |
| rm_work | what is 528 git commit --amend | 20:57 |
| rm_work | that remakes the commit hash for the commit above you | 20:57 |
| rm_work | though it should have been your OWN commit | 20:57 |
| rm_work | so not a huge issue | 20:57 |
| rm_work | just odd | 20:57 |
| TrevorV | Oh sorry, that commit --amend was me checking a message | 20:58 |
| rm_work | I am guessing the git review on 530 didn't work? | 20:58 |
| TrevorV | I was looking for the topic in the commit message, and instead of typing "git log | 20:58 |
| TrevorV | " I was prepared to update it | 20:58 |
| TrevorV | ha ha | 20:58 |
| TrevorV | Anyway, the 530 told me I had a rebase to do | 20:59 |
| *** neelashah has joined #openstack-lbaas | 20:59 | |
| *** neelashah1 has quit IRC | 20:59 | |
| TrevorV | hence me doing the status, and then rebasing | 20:59 |
| rm_work | yeah | 20:59 |
| rm_work | once it told you to do a rebase, something was f'd | 20:59 |
| rm_work | i'd have to see the full output of that git status | 20:59 |
| rm_work | and of the fetch+cherry-pick | 20:59 |
| rm_work | 529/531 | 21:00 |
| TrevorV | Of course I don't have it, because I ran tox | 21:00 |
| TrevorV | ugh | 21:00 |
| TrevorV | wait, I do have it | 21:00 |
| sbalukoff | Ok, I'mma try re-basing my commit chain to HEAD and re-upping. | 21:01 |
| blogan | ok i gottag et lunch before my next meeting | 21:04 |
| *** evgenyf has quit IRC | 21:06 | |
| TrevorV | rm_work https://gist.github.com/TrevorV/3a197bb0c984080d1c4d | 21:07 |
| TrevorV | sorry again sbalukoff | 21:07 |
| TrevorV | I'm still trying to wrap my head around what went wrong... | 21:08 |
| TrevorV | I still don't see any problems :( | 21:08 |
| rm_work | mean i see right where it redid all the commits, but not sure WHY | 21:10 |
| rm_work | though, you would have had some issues possibly because your --amend did change THAT commit hash | 21:10 |
| rm_work | which MAYBE could have made git-review look harder at rebasing? though it shouldn't have still | 21:11 |
| TrevorV | which again doesn't quite make sense rm_work | 21:11 |
| TrevorV | Since its only the top level commit hash that would have changed. | 21:11 |
| TrevorV | Idk, this was wonky. | 21:11 |
| rm_work | the git-review plugin may look deeper in the chain if you commit two new things instead of one? not sure | 21:11 |
| rm_work | basically, if it says anything to do with a rebase, ABORT MISSION EJECT EJECT | 21:12 |
| TrevorV | Yeah, next time I'll do that. | 21:12 |
| sbalukoff | TrevorV: Ok, so it's not letting me push out a new patchset (it's saying the existing patchset is the same). Glancing this over somewhat, I think that's the case. | 21:21 |
| sbalukoff | So... I'm guessing your code base actually was up to date with master. | 21:21 |
| neelashah | johnsonm: it looks like LBaaS v2 is not the default in M yet even though v1 was deprecated in L - is the plan to make v2 the default in M? | 21:21 |
| TrevorV | sbalukoff I'm not sure I follow. | 21:21 |
| sbalukoff | So... I *think it's OK. | 21:21 |
| sbalukoff | TrevorV: I'm not sure I do either. | 21:21 |
| TrevorV | Ha ha ha | 21:22 |
| neelashah | johnsom ^ | 21:22 |
| TrevorV | sbalukoff you're saying there are no changes to make, even though there was a complaint from gerrit originally? | 21:22 |
| sbalukoff | But, I'll look at each of these patches on gerrit to see if there's any difference between the last two patschsets. thankfully, the web interface makes this somewhat easy. | 21:22 |
| sbalukoff | TrevorV: Yes. | 21:22 |
| TrevorV | sbalukoff I was doing that, I did see one file before I waited to see you update. | 21:22 |
| sbalukoff | TrevorV: At least, I *think* that's what I'm saying. ;) | 21:22 |
| sbalukoff | Which file? | 21:22 |
| TrevorV | sbalukoff lemme verify | 21:23 |
| johnsom | neelashah Where do you see that LBaaSv2 is not the default? | 21:23 |
| TrevorV | sbalukoff | 21:23 |
| TrevorV | https://review.openstack.org/#/c/277768 | 21:23 |
| TrevorV | constants and controller_worker had changes | 21:23 |
| TrevorV | between the last two patch sets | 21:23 |
| sbalukoff | Ok, I'll have a look. | 21:23 |
| TrevorV | sbalukoff https://review.openstack.org/#/c/276643 | 21:24 |
| TrevorV | constants changed | 21:24 |
| TrevorV | sbalukoff https://review.openstack.org/#/c/265529 | 21:24 |
| TrevorV | constants changed | 21:24 |
| TrevorV | sbalukoff https://review.openstack.org/#/c/265430 | 21:25 |
| sbalukoff | I think that's the result of a rebase from HEAD. | 21:25 |
| TrevorV | constants changed. | 21:25 |
| TrevorV | Yeah. | 21:25 |
| sbalukoff | So... it's not disruptive. | 21:25 |
| sbalukoff | So, indvertently, you rebased all my patches to HEAD. XD | 21:25 |
| TrevorV | Yeah, but I think it pushed a new patch set to everything.... | 21:25 |
| sbalukoff | So, not a bad thing, per se. | 21:25 |
| TrevorV | Yeah, well, hopefully that doesn't break anything. | 21:25 |
| TrevorV | sorry! | 21:25 |
| TrevorV | Again! | 21:25 |
| sbalukoff | No worries. I'm not sure you did anything wrong. | 21:25 |
| *** rtheis has quit IRC | 21:26 | |
| sbalukoff | Yeah, everything I'm seeing here is just the result of a rebase to HEAD. So... I think we're completely fine. | 21:28 |
| TrevorV | Awesome! | 21:28 |
| TrevorV | I hope... | 21:28 |
| TrevorV | Heh | 21:28 |
| TrevorV | Now I'll re-focus and double my efforts for the single-create. | 21:28 |
| sbalukoff | Sounds good! | 21:29 |
| TrevorV | dougwig sorry for not having anything to report on the neutron lbaas side... :( | 21:29 |
| *** rtheis has joined #openstack-lbaas | 21:29 | |
| neelashah | johnsom: nm, looks like I need to explicitly enable lbaas (whether v1 or v2) since its a separate plugin | 21:31 |
| johnsom | neelashah Yep, ok, no problem | 21:31 |
| *** Guest45229 is now known as mariusv | 21:31 | |
| neelashah | johnsom but all the lbaas v2 tests are non voting at present, right? | 21:31 |
| *** mariusv has quit IRC | 21:32 | |
| *** mariusv has joined #openstack-lbaas | 21:32 | |
| johnsom | The API tests are voting. We are trying to track down a new issue that is causing some tests to fail, so we have made them non-voting | 21:32 |
| sbalukoff | It's a Heisenbug-- seems to go away whenever we look closely. Which obviously makes troubleshooting it really annoying. :P | 21:33 |
| neelashah | johnsom - oh I see, thanks | 21:34 |
| johnsom | Yes, I noticed they have cleared up. I'm wondering it it wasn't a bug in some other part of OpenStack that was impacting us | 21:34 |
| sbalukoff | It could still be a timing issue. :/ | 21:35 |
| sbalukoff | blogan: Can I get you to revisit your -1 on this: https://review.openstack.org/#/c/280166/ :) | 21:37 |
| johnsom | I was just looking at that one myself | 21:37 |
| sbalukoff | Yay! | 21:37 |
| *** armax has joined #openstack-lbaas | 21:41 | |
| xgerman | blogan, johnsom: https://bugs.launchpad.net/octavia/+bug/1496628 | 21:46 |
| openstack | Launchpad bug 1496628 in octavia "Amphora agent reload fails with socket in use" [High,Fix committed] - Assigned to German Eichberger (german-eichberger) | 21:46 |
| johnsom | Yeah, looking | 21:46 |
| xgerman | well, it works… so closed it | 21:48 |
| johnsom | xgerman Yeah, ok. I can repro that it works too | 21:48 |
| johnsom | Thanks for the followup | 21:48 |
| xgerman | y. w. | 21:49 |
| xgerman | I would have rather spend that hour bike shedding why I think the L7 flow design is wrong ;-) | 21:50 |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Delete SSH amphora driver https://review.openstack.org/281172 | 21:51 |
| doug-fish | Has anyone successfully created listeners using TERMINATED_HTTP protocol? | 21:52 |
| doug-fish | I'm getting this error: http://paste.openstack.org/show/487341/ and can't make sense of it | 21:52 |
| johnsom | xgerman sadly, the service amphora-agent reload triggers that issue still | 21:52 |
| doug-fish | TERMINATED_HTTPS of course | 21:52 |
| johnsom | Yeah, I demo'd that in Vancouver, so it worked at one time | 21:53 |
| doug-fish | johnsom: what should the certificate parameter look like? | 21:53 |
| rm_work | doug-fish: i just did one a second ago | 21:53 |
| doug-fish | I've tried the url and the id | 21:53 |
| xgerman | well, if curl works… then the flow is broken? | 21:53 |
| rm_work | just the ID | 21:53 |
| rm_work | doug-fish: | 21:54 |
| doug-fish | I'm going through neutron client ... rm_work that's good to know! | 21:54 |
| johnsom | Something like this: neutron lbaas-listener-create --loadbalancer lb1 --protocol-port 443 --protocol TERMINATED_HTTPS --name listener1 --default-tls-container=$(barbican secret container list | awk '/ tls_container / {print $2}') | 21:54 |
| rm_work | doug-fish: check out: https://gist.githubusercontent.com/rm-you/f7585ca4932b3ee1eed9/raw/f599251c5a554821ef748c32fd086d99f8010bb1/new_octavia_devstack.sh | 21:55 |
| rm_work | you can see how i get the IDs and such | 21:55 |
| rm_work | and at the bottom are two commented commands that I run verbatim | 21:55 |
| doug-fish | rm_work: excellent! I'll take a look | 21:55 |
| rm_work | that error though ... looks like maybe it can't communicate properly with Barbican? | 21:58 |
| rm_work | what user created the certificates in barbican, and what user are you trying to create the LB with? | 21:59 |
| rm_work | right now the ACLs don't work right in Barbican | 21:59 |
| rm_work | ... and even if they did, you'd need another step to do that | 21:59 |
| doug-fish | rm_work: I created the certificate in Barbican using their user guide example | 22:06 |
| doug-fish | http://docs.openstack.org/developer/barbican/api/userguide/certificates.html#simple-cmc-certificate-order | 22:06 |
| doug-fish | and I've verified that I get the same failure with a command line similar to yours. I think I must have the certificate wrong in Barbican | 22:07 |
| rm_work | hmm | 22:08 |
| doug-fish | I'm new to barbican -- do you have any suggestions for checking my container? | 22:09 |
| rm_work | well | 22:09 |
| rm_work | try just making a new one using my commands | 22:10 |
| rm_work | and see if IT works | 22:10 |
| rm_work | thats a trash but valid cert | 22:10 |
| doug-fish | yeah that's a good idea | 22:10 |
| *** blallau has quit IRC | 22:11 | |
| sbalukoff | By the way, for anyone smoke-testing the L7 stuff, this might come in handy: https://gist.github.com/sbalukoff/e6cd600b4a12ee582f5e | 22:17 |
| johnsom | Handy, thanks | 22:18 |
| *** neelashah has quit IRC | 22:19 | |
| minwang2 | thank you sbalukoff | 22:21 |
| sbalukoff | No problem. Obviously, please try out stuff I'm not doing in this as well... :) | 22:22 |
| TrevorV | sbalukoff no way, we only want test results from happy path | 22:23 |
| TrevorV | amirite? | 22:23 |
| sbalukoff | (For what it's worth, as I've been testing this, I've been closely watching the haproxy config that gets generated with each change.) | 22:23 |
| sbalukoff | TrevorV: Well, the happy path should work, too. ;) | 22:23 |
| johnsom | Yeah, me too, I have a watch going on the file | 22:24 |
| *** rtheis has quit IRC | 22:25 | |
| TrevorV | sbalukoff just found a nit. | 22:26 |
| TrevorV | oops | 22:26 |
| TrevorV | Wasn't even reviewing :P | 22:26 |
| sbalukoff | D'oh! | 22:26 |
| rm_work | yeah i've got happy-path for non-l7 pretty much all tested and looks good | 22:26 |
| rm_work | about to do actual L7 | 22:26 |
| TrevorV | l7policy types. L7PolicyResponse. "action" is just a string, not an enum like you've marked in POST and PUT classes. | 22:27 |
| TrevorV | sbalukoff ^^ | 22:27 |
| sbalukoff | TrevorV: That's actually just fine. | 22:27 |
| TrevorV | Oh? | 22:27 |
| sbalukoff | We don't really need to type-check our responses. | 22:27 |
| TrevorV | If you insist :D | 22:28 |
| sbalukoff | (And we don't, elsewhere) | 22:28 |
| TrevorV | Well if no precedent, then I shut up :P | 22:28 |
| *** diogogmt has joined #openstack-lbaas | 22:28 | |
| *** doug-fish has quit IRC | 22:33 | |
| *** doug-fish has joined #openstack-lbaas | 22:34 | |
| *** doug-fish has quit IRC | 22:34 | |
| *** doug-fish has joined #openstack-lbaas | 22:34 | |
| *** localloop127 has quit IRC | 22:40 | |
| *** alejandrito has quit IRC | 22:42 | |
| *** allan_h has joined #openstack-lbaas | 22:47 | |
| *** armax has quit IRC | 22:51 | |
| *** diogogmt has quit IRC | 22:54 | |
| doug-fish | rm_work: I issued commands based on your script and I was able to successfully create a listener based with TERMINATED_HTTPS -- thanks! | 22:54 |
| doug-fish | I'm wondering if we should be filtering the list of certificates a bit more in the UI | 22:54 |
| *** diogogmt has joined #openstack-lbaas | 22:54 | |
| openstackgerrit | Madhusudhan Kandadai proposed openstack/octavia: Implement tempest-plugin for octavia https://review.openstack.org/281569 | 22:55 |
| doug-fish | so that we only show certicates from containers where there is a private key and a certificate | 22:55 |
| madhu_ak | johnsom, fnaval : ^^ | 22:55 |
| *** doug-fish has quit IRC | 22:57 | |
| *** doug-fish has joined #openstack-lbaas | 22:57 | |
| fnaval | cool thanks madhu_ak - i'll take a look. | 22:57 |
| *** diogogmt has quit IRC | 22:59 | |
| openstackgerrit | Madhusudhan Kandadai proposed openstack/octavia: Implement tempest-plugin for octavia https://review.openstack.org/281569 | 22:59 |
| *** TrevorV has quit IRC | 22:59 | |
| *** allan_h has quit IRC | 22:59 | |
| madhu_ak | fnaval: if the above patch goes in, we just need to rebase https://review.openstack.org/#/c/172199/ | 23:00 |
| fnaval | does running the tests stay the same? | 23:01 |
| fnaval | line 28 on plugin.py might have been correct the previous way? | 23:02 |
| *** doug-fish has quit IRC | 23:02 | |
| *** dougw2 has quit IRC | 23:02 | |
| madhu_ak | yes. the same way we run the tests | 23:03 |
| *** allan_h has joined #openstack-lbaas | 23:03 | |
| madhu_ak | no confusion. The above plugin will be useful for setting up gate job in octavia | 23:03 |
| fnaval | ok ok, what benefit does it provide? | 23:03 |
| fnaval | ah, so it's for the gate job | 23:03 |
| johnsom | madhu_ak I see what fnaval is saying, should the return be in the for loop? | 23:03 |
| fnaval | making the tests themselves plug inable | 23:03 |
| madhu_ak | yeah johnsom if I understand correctly, will we be having api tests for octavia in future? | 23:04 |
| johnsom | Yes, we will have both tests, but return inside a for loop will only run once | 23:05 |
| madhu_ak | ah, I see. got you johnsom | 23:05 |
| madhu_ak | will resubmit and correct it | 23:06 |
| openstackgerrit | Madhusudhan Kandadai proposed openstack/octavia: Implement tempest-plugin for octavia https://review.openstack.org/281569 | 23:08 |
| *** allan_h has quit IRC | 23:11 | |
| *** Bjoern has quit IRC | 23:20 | |
| *** ducttape_ has quit IRC | 23:36 | |
| *** armax has joined #openstack-lbaas | 23:42 | |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Update data model conversion to be more complete https://review.openstack.org/280166 | 23:55 |
| sbalukoff | Ah-ha! | 23:55 |
| sbalukoff | Please note that the above commit (and the ones that follow) are literally just a comment update. | 23:56 |
| sbalukoff | TrevorV: I found that gerrit wouldn't let me create further dependent patches at the end of the chain because the first one failed to upload because it was exactly same as the one already up there. | 23:56 |
| sbalukoff | So, I made an inconsequential change to the first in the chain, and will bubble the rebases through that chain so that I can add a new dependent patch. | 23:57 |
| sbalukoff | (The new dependent patch adds listener stats. :P ) | 23:57 |
| openstackgerrit | Stephen Balukoff proposed openstack/octavia: Fix model update flows https://review.openstack.org/280478 | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!