openstackgerrit | Ankur proposed openstack/octavia master: Fix v2 LB API return for null description and name https://review.openstack.org/448317 | 00:31 |
---|---|---|
*** reedip has quit IRC | 00:38 | |
*** armax has quit IRC | 00:41 | |
*** harlowja has joined #openstack-lbaas | 00:42 | |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Adds v2 load balancer API section https://review.openstack.org/438757 | 01:07 |
*** krypto has joined #openstack-lbaas | 01:09 | |
*** sanfern has quit IRC | 01:19 | |
*** reedip has joined #openstack-lbaas | 01:24 | |
*** gongysh has joined #openstack-lbaas | 01:33 | |
*** nrado has quit IRC | 01:34 | |
openstackgerrit | Ankur proposed openstack/python-octaviaclient master: Add skeleton to python-octaviaclient https://review.openstack.org/448331 | 01:43 |
openstackgerrit | Shashank Kumar Shankar proposed openstack/octavia master: Introduce Octavia v2 API for pools https://review.openstack.org/405922 | 02:10 |
openstackgerrit | Merged openstack/neutron-lbaas master: Use assertIsNone(...) instead of assertEqual(None, ...) https://review.openstack.org/446861 | 02:13 |
*** krypto has quit IRC | 02:15 | |
openstackgerrit | Ankur proposed openstack/python-octaviaclient master: Add skeleton to python-octaviaclient https://review.openstack.org/448331 | 02:20 |
*** Other-krypto has joined #openstack-lbaas | 02:26 | |
*** Other-krypto has quit IRC | 02:33 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-lbaas master: Updated from global requirements https://review.openstack.org/446970 | 02:34 |
*** reedip has quit IRC | 02:52 | |
*** sanfern has joined #openstack-lbaas | 02:55 | |
*** yuanying has quit IRC | 03:05 | |
*** links has joined #openstack-lbaas | 03:12 | |
*** faizy has joined #openstack-lbaas | 03:34 | |
*** harlowja has quit IRC | 03:35 | |
*** links has quit IRC | 03:38 | |
*** faizy has quit IRC | 03:39 | |
*** links has joined #openstack-lbaas | 03:52 | |
*** yuanying has joined #openstack-lbaas | 04:18 | |
*** links has quit IRC | 04:33 | |
*** armax has joined #openstack-lbaas | 04:44 | |
*** links has joined #openstack-lbaas | 04:50 | |
*** blogan has quit IRC | 04:58 | |
*** krypto has joined #openstack-lbaas | 04:59 | |
*** rcernin has joined #openstack-lbaas | 05:09 | |
*** rcernin has quit IRC | 05:09 | |
*** faizy has joined #openstack-lbaas | 05:19 | |
*** faizy has quit IRC | 05:22 | |
*** faizy has joined #openstack-lbaas | 05:22 | |
*** faizy_ has joined #openstack-lbaas | 05:23 | |
*** faizy_ has quit IRC | 05:27 | |
*** faizy_ has joined #openstack-lbaas | 05:27 | |
*** faizy has quit IRC | 05:27 | |
*** faizy_ has quit IRC | 05:30 | |
*** faizy_ has joined #openstack-lbaas | 05:31 | |
*** reedip has joined #openstack-lbaas | 05:55 | |
*** reedip has quit IRC | 06:06 | |
*** faizy_ has quit IRC | 06:07 | |
*** faizy has joined #openstack-lbaas | 06:08 | |
*** krypto has quit IRC | 06:09 | |
*** reedip has joined #openstack-lbaas | 06:10 | |
*** faizy has quit IRC | 06:11 | |
*** faizy has joined #openstack-lbaas | 06:11 | |
*** reedip has quit IRC | 06:12 | |
*** reedip has joined #openstack-lbaas | 06:13 | |
*** faizy has quit IRC | 06:15 | |
*** faizy has joined #openstack-lbaas | 06:15 | |
*** faizy has quit IRC | 06:18 | |
*** faizy has joined #openstack-lbaas | 06:19 | |
*** faizy has quit IRC | 06:21 | |
*** faizy has joined #openstack-lbaas | 06:22 | |
*** gcheresh_ has joined #openstack-lbaas | 06:23 | |
*** aojea has joined #openstack-lbaas | 06:42 | |
*** aojea has quit IRC | 06:56 | |
*** armax has quit IRC | 07:05 | |
*** armax has joined #openstack-lbaas | 07:05 | |
*** zioproto has joined #openstack-lbaas | 07:15 | |
*** reedip has quit IRC | 07:16 | |
*** reedip has joined #openstack-lbaas | 07:16 | |
*** reedip__ has joined #openstack-lbaas | 07:30 | |
*** reedip has quit IRC | 07:31 | |
*** reedip__ has quit IRC | 07:34 | |
*** reedip__ has joined #openstack-lbaas | 07:34 | |
*** reedip__ has quit IRC | 07:37 | |
*** reedip__ has joined #openstack-lbaas | 07:37 | |
*** pcaruana has joined #openstack-lbaas | 07:39 | |
*** reedip__ has quit IRC | 07:43 | |
*** reedip__ has joined #openstack-lbaas | 07:43 | |
*** eezhova has joined #openstack-lbaas | 07:43 | |
*** reedip__ has quit IRC | 07:44 | |
*** reedip__ has joined #openstack-lbaas | 07:44 | |
*** ducnc has quit IRC | 07:45 | |
*** reedip__ has quit IRC | 07:54 | |
*** faizy has quit IRC | 07:54 | |
*** tesseract has joined #openstack-lbaas | 07:55 | |
*** reedip__ has joined #openstack-lbaas | 07:56 | |
*** aojea has joined #openstack-lbaas | 08:16 | |
*** aojea has quit IRC | 08:16 | |
*** aojea has joined #openstack-lbaas | 08:17 | |
*** amotoki has joined #openstack-lbaas | 08:20 | |
*** sanfern has quit IRC | 08:25 | |
*** sanfern has joined #openstack-lbaas | 08:26 | |
*** reedip__ has quit IRC | 08:32 | |
*** eezhova has quit IRC | 08:41 | |
*** krypto has joined #openstack-lbaas | 09:17 | |
*** eezhova has joined #openstack-lbaas | 09:17 | |
*** reedip__ has joined #openstack-lbaas | 09:20 | |
*** yuanying has quit IRC | 09:31 | |
*** reedip__ has quit IRC | 09:33 | |
*** reedip__ has joined #openstack-lbaas | 09:33 | |
*** gongysh has quit IRC | 09:36 | |
*** reedip__ has quit IRC | 09:37 | |
*** reedip__ has joined #openstack-lbaas | 09:37 | |
*** reedip__ has quit IRC | 09:47 | |
*** reedip__ has joined #openstack-lbaas | 09:47 | |
*** aojea_ has joined #openstack-lbaas | 10:05 | |
*** aojea has quit IRC | 10:08 | |
*** yuanying has joined #openstack-lbaas | 10:11 | |
*** reedip__ has quit IRC | 10:19 | |
*** reedip__ has joined #openstack-lbaas | 10:46 | |
*** sanfern has quit IRC | 10:54 | |
*** krypto has quit IRC | 11:12 | |
*** krypto has joined #openstack-lbaas | 11:16 | |
*** krypto has quit IRC | 11:35 | |
*** krypto has joined #openstack-lbaas | 11:38 | |
*** krypto has joined #openstack-lbaas | 11:38 | |
*** gongysh has joined #openstack-lbaas | 11:49 | |
*** sanfern has joined #openstack-lbaas | 11:50 | |
reedip__ | o/ | 12:32 |
rm_work | o/ | 12:32 |
reedip__ | dont u sleep rm_work ??? | 12:32 |
reedip__ | :) | 12:32 |
rm_work | sometimes | 12:33 |
rm_work | not recently | 12:33 |
reedip__ | :) | 12:33 |
*** reedip__ has quit IRC | 12:36 | |
*** reedip has joined #openstack-lbaas | 12:38 | |
*** catintheroof has joined #openstack-lbaas | 12:42 | |
rm_work | should it be possible to PUT a listener with no body? >_> | 12:44 |
rm_work | that would not do anything... | 12:44 |
rm_work | I think it should be mandatory to have a PUT body? | 12:44 |
nmagnezi | reedip, I already checked that. rm_work never sleeps :) | 12:46 |
rm_work | ah, expect_errors is being used improperly in a lot of these tests, i think | 12:54 |
*** belharar has joined #openstack-lbaas | 12:58 | |
*** krypto has quit IRC | 13:05 | |
*** krypto has joined #openstack-lbaas | 13:05 | |
*** krypto has quit IRC | 13:05 | |
*** krypto has joined #openstack-lbaas | 13:05 | |
*** belharar has quit IRC | 13:17 | |
*** belharar has joined #openstack-lbaas | 13:22 | |
*** belharar has quit IRC | 13:29 | |
*** gongysh has quit IRC | 13:35 | |
*** krypto has quit IRC | 13:42 | |
*** krypto has joined #openstack-lbaas | 13:42 | |
reedip | anyone has an idea how to implement quotas for a resource ? | 13:50 |
reedip | I have defined the cfg options , used registry_helper in neutron.quota and registered plural mappings | 13:51 |
rm_work | err | 13:55 |
rm_work | neutron_lbaas or octavia? | 13:55 |
reedip | err ... a different module, but knowledge is common :D | 13:58 |
rm_work | well | 14:00 |
rm_work | it depends on how the project implements quotas, I think :P | 14:00 |
rm_work | I know how to do it in Octavia, but no idea in neutron-lbaas T_T | 14:00 |
*** zioproto has quit IRC | 14:01 | |
*** oomichi has quit IRC | 14:04 | |
*** chlong has joined #openstack-lbaas | 14:04 | |
reedip | so quotas can be implemented differently ? | 14:05 |
*** oomichi has joined #openstack-lbaas | 14:06 | |
*** krypto has quit IRC | 14:07 | |
*** krypto has joined #openstack-lbaas | 14:07 | |
rm_work | i mean, each project has some implementation I think | 14:09 |
rm_work | I don't think there's an oslo.quotas or anything | 14:09 |
reedip | hmm | 14:10 |
*** zioproto has joined #openstack-lbaas | 14:11 | |
*** chlong has quit IRC | 14:12 | |
*** krypto has quit IRC | 14:13 | |
*** chlong has joined #openstack-lbaas | 14:13 | |
*** krypto has joined #openstack-lbaas | 14:13 | |
*** krypto has joined #openstack-lbaas | 14:13 | |
johnsom | rm_work empty put is 400 in my book. Though I am not sure if you can just use ?name=foo as a replacement... | 14:17 |
rm_work | hmmm | 14:17 |
rm_work | well, i'll post a patchset in a sec | 14:18 |
rm_work | and see what you think | 14:18 |
rm_work | I don't want to make it MANDATORY in the type | 14:18 |
rm_work | because that interferes with other stuff i think | 14:18 |
rm_work | johnsom: so in v1, test_update_listeners_same_port has: self.skip('This test should pass with a validation layer.') | 14:19 |
rm_work | johnsom: in v2, should we be aiming to include this validation layer as part of these initial patches? or not? | 14:19 |
johnsom | reedip Every project does quota differently. I looked for a code to borrow, but ultimately kept is simple and did our own thing. Neutron uses reservations for example. We use DB transactions | 14:19 |
*** links has quit IRC | 14:21 | |
johnsom | rm_work I told them it can be a second pass. We aren't doing the project/policy checking yet either. The only validation now is the type / required stuff wsme brings | 14:22 |
johnsom | Though if we have the cycles I am not opposed to adding validation as we go | 14:23 |
reedip | johnsom : okay ... so there is no defined method ... | 14:23 |
johnsom | reedip nope. There was discussions of an oslo project, but it didn't go anywhere. They are talking about moving the limits into keystone now, but just limit storage | 14:24 |
reedip | johnsom okay .... thanks | 14:24 |
johnsom | rm_work I prioritized getting the bones of the API in place first, then building on those | 14:25 |
*** belharar has joined #openstack-lbaas | 14:25 | |
rm_work | kk | 14:25 |
rm_work | posting an update to the listeners patch | 14:29 |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Octavia v2 API for listeners https://review.openstack.org/424744 | 14:29 |
rm_work | johnsom: https://review.openstack.org/#/c/424744/26/octavia/api/v2/controllers/listener.py | 14:32 |
rm_work | johnsom: line 210-214 | 14:32 |
johnsom | Yeah, hmm, I wonder if we could do that in a validation hook and centralize it. | 14:35 |
johnsom | So that would catch a body with a root tag but no content | 14:35 |
rm_work | in this case, there's not even a root tag | 14:37 |
rm_work | the tests were passing in {} | 14:37 |
rm_work | so listener_.listener was an Unset type | 14:37 |
rm_work | where it'd try to do the to_dict | 14:37 |
rm_work | johnsom: so yeah we need to be careful about expect_errors=True in requests | 14:40 |
rm_work | in the testing | 14:40 |
rm_work | it is masking a lot of errors | 14:40 |
reedip | sindhu : working on the HM patch? | 14:40 |
sindhu | reedip: nope, u take it over | 14:41 |
rm_work | ummm, why does listener Create allow you to pass in an id at all? >_< | 14:41 |
reedip | sindhu : sure? | 14:41 |
rm_work | ok, so this is really ridiculous, our docs are so bad that even we can't tell what our API is supposed to do | 14:42 |
reedip | rm_work : lol | 14:42 |
rm_work | and our API has some really dumb stuff in it, that I honestly can't figure out why we did it | 14:43 |
rm_work | so it either wasn't in the spec and was an implementation mistake | 14:43 |
johnsom | rm_work it should not. Yep, that is why I am rewriting them.... | 14:43 |
rm_work | or else we should be ashamed T_T | 14:43 |
johnsom | The current docs still have v1 stuff intermixed | 14:44 |
johnsom | It is in bad shape | 14:44 |
sindhu | reedip: yeah! looks like everyone is in a hurry and I don't want to block anyone. Pls go ahead | 14:46 |
rm_work | grr i have a couple test_load_balancer fixes i want to throw into this listener patch so badly... | 14:47 |
rm_work | augh | 14:47 |
rm_work | i will forget... | 14:47 |
rm_work | bah i'll make another trivial patch really quick | 14:47 |
*** blogan has joined #openstack-lbaas | 14:47 | |
openstackgerrit | Sindhu Devale proposed openstack/octavia master: Introduce Octavia v2 API for Health Monitor https://review.openstack.org/406256 | 14:47 |
rm_work | oh i guess i could do them along with some enabling of stuff | 14:48 |
*** armax has quit IRC | 14:49 | |
*** belharar has quit IRC | 14:50 | |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Octavia v2 API for listeners https://review.openstack.org/424744 | 14:56 |
*** armax has joined #openstack-lbaas | 14:56 | |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Assorted trivial fixes for v2 loadbalancer tests https://review.openstack.org/448626 | 14:57 |
rm_work | ah hmm | 14:59 |
*** armax has quit IRC | 15:01 | |
*** fnaval has joined #openstack-lbaas | 15:09 | |
reedip | rm_work, johnsom : jfyi , the DVR meeting is on #openstack-meeting-alt , and they were discussing about https://bugs.launchpad.net/neutron/+bug/1672345 | 15:10 |
openstack | Launchpad bug 1672345 in neutron "Loadbalancer V2 ports are not serviced by DVR" [Undecided,New] | 15:10 |
johnsom | Yeah, I commented on that one that I think it is probably a duplicate to the on going DVR with AA ports issues | 15:15 |
rm_work | oops, messed up that last commit a little | 15:21 |
rm_work | pep8/py27 fails, embarassing | 15:21 |
rm_work | also embarrassing: misspelling embarrassing | 15:21 |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Octavia v2 API for listeners https://review.openstack.org/424744 | 15:22 |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Introduce Octavia v2 API for pools https://review.openstack.org/405922 | 15:22 |
*** armax has joined #openstack-lbaas | 15:26 | |
*** chlong has quit IRC | 15:30 | |
*** chlong has joined #openstack-lbaas | 15:33 | |
rm_work | johnsom: for member PUT: | 15:37 |
rm_work | Note: You cannot update the member ID, project_id, address, protocol_port, and subnet_id attributes. If you attempt to update any of these attributes, the service returns the HTTP Immutable (422) response code. | 15:37 |
rm_work | is that true? | 15:37 |
rm_work | obviously you can't update the ID or project-id | 15:38 |
rm_work | but not address or protocol_port?? | 15:38 |
rm_work | ugh and it says you can update the pool_id and member_id right below that | 15:39 |
rm_work | ffff | 15:39 |
johnsom | Yes, I think it was because you are basically doing the same as create/delete | 15:39 |
johnsom | You cannot update the IDs... | 15:39 |
johnsom | member you can only update weight, name, admin_state | 15:40 |
*** krypto has quit IRC | 15:41 | |
*** krypto has joined #openstack-lbaas | 15:41 | |
rm_work | k | 15:41 |
johnsom | For what it is worth, I am mostly looking at the lbaas code and not the docs as up create the new docs | 15:41 |
nmagnezi | ouch.. | 15:44 |
nmagnezi | sounds like A LOT of work | 15:44 |
rm_work | wait hold on | 15:49 |
rm_work | if you update a member's name... | 15:49 |
rm_work | the expectation is that the response for the PUT *doesn't* show the new value? | 15:50 |
johnsom | nmagnezi It is... Sigh. But it will be so useful when done.... | 15:50 |
johnsom | nmagnezi I am up to working on the load balancer section: http://docs-draft.openstack.org/57/438757/5/check/gate-octavia-api-ref/8a25d23//api-ref/build/html/v2/index.html | 15:50 |
johnsom | rm_work It probably should. This is the whole issue with async api and roll backs. I think in octavia I was doing some model tricks. It's hard because updating a name is just a DB call, updating a weight means the handler gets involved and it's aysnc | 15:53 |
openstackgerrit | Bernard Cafarelli proposed openstack/octavia master: DIB: drop some dependencies https://review.openstack.org/448657 | 15:53 |
rm_work | yeah, well... k that's fine then | 15:53 |
rm_work | just making sure | 15:53 |
rm_work | seems like a shitty test to try to write tho | 15:53 |
*** gcheresh_ has quit IRC | 15:57 | |
johnsom | We need to make sure we match neutron-lbaas behavior, it might be different than octavia v1 | 15:57 |
*** armax has quit IRC | 16:02 | |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Introduce Octavia v2 API for Members https://review.openstack.org/405932 | 16:03 |
*** blogan_ has joined #openstack-lbaas | 16:05 | |
rm_work | johnsom: so I know you have about 30 things, but, do you think you will have a chance to review the listeners patch today? | 16:05 |
*** harlowja has joined #openstack-lbaas | 16:06 | |
*** blogan has quit IRC | 16:06 | |
johnsom | Yes, I probably can give it a look today. I have LB status and stats left on the LB API ref, then I am onto listener anyway. | 16:07 |
johnsom | Trying to answer an ask.openstack.org question at the moment... | 16:13 |
*** blogan_ is now known as blogan | 16:17 | |
*** krypto has quit IRC | 16:30 | |
*** krypto has joined #openstack-lbaas | 16:35 | |
*** chlong has quit IRC | 16:45 | |
*** links has joined #openstack-lbaas | 16:49 | |
*** gcheresh_ has joined #openstack-lbaas | 16:51 | |
*** chlong has joined #openstack-lbaas | 16:52 | |
*** gcheresh_ has quit IRC | 16:56 | |
*** gcheresh_ has joined #openstack-lbaas | 16:58 | |
rm_work | johnsom: wait, is it even *allowed* to provide a project_id to things like member/pool? | 17:01 |
rm_work | they *must* be the same as the LB, no? | 17:02 |
rm_work | even if you're an admin | 17:02 |
johnsom | Not any more | 17:02 |
rm_work | wut | 17:02 |
johnsom | See https://review.openstack.org/442865 | 17:02 |
johnsom | It's the same as the LB now | 17:02 |
johnsom | We need to pull that over for the v2 api | 17:03 |
johnsom | Actually, not that link... | 17:03 |
johnsom | This link https://review.openstack.org/431207 | 17:03 |
johnsom | The first link was where I pulled it over for LB | 17:04 |
johnsom | But that is a special case as it is the top of the object tree | 17:04 |
rm_work | yeah k | 17:06 |
*** eezhova has quit IRC | 17:08 | |
rm_work | ok i took a stab at it on the members patch | 17:08 |
rm_work | gonna need to do the same on listeners/pools I think, since i believe it was done a little differently | 17:08 |
johnsom | Ok, I think sindhu added it and the base class method on listener | 17:09 |
johnsom | Maybe not | 17:09 |
rm_work | ah maybe | 17:10 |
rm_work | ah it's done a little differently | 17:11 |
rm_work | or rather | 17:11 |
rm_work | it's the same as the code in the listener.py controller for v1 | 17:11 |
rm_work | that you linked | 17:11 |
rm_work | but | 17:11 |
rm_work | that wouldn't allow it to be changed | 17:11 |
johnsom | You can't change it project ID for listeners under an LB, only the LB | 17:12 |
rm_work | so like, if you create a pool and in the POST body you send in a project_id, that doesn't allow it | 17:12 |
rm_work | ok that's what I was asking | 17:12 |
rm_work | [02:02:26] <rm_work>johnsom: wait, is it even *allowed* to provide a project_id to things like member/pool? | 17:12 |
johnsom | Yeah, I think we have to still take it, for backward compat, but it is ignored in favor of the owning LB project_id | 17:12 |
rm_work | so you *can't* provide project_id for listener/pool/member? | 17:12 |
rm_work | ahh ok | 17:12 |
johnsom | https://review.openstack.org/#/c/431207/3/octavia/api/v1/types/listener.py | 17:13 |
*** krypto has quit IRC | 17:13 | |
rm_work | >_< | 17:13 |
rm_work | ok | 17:13 |
*** krypto has joined #openstack-lbaas | 17:13 | |
*** krypto has quit IRC | 17:13 | |
*** krypto has joined #openstack-lbaas | 17:13 | |
rm_work | then i guess the others are fine | 17:18 |
*** krypto has quit IRC | 17:18 | |
*** krypto has joined #openstack-lbaas | 17:19 | |
*** gcheresh_ has quit IRC | 17:19 | |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Introduce Octavia v2 API for Members https://review.openstack.org/405932 | 17:22 |
johnsom | rm_work Do you know if we can put length restrictions on this? | 17:22 |
johnsom | insert_headers = wtypes.wsattr(wtypes.DictType(str, str)) | 17:22 |
rm_work | uhh | 17:25 |
rm_work | not sure, the wstypes stuff is kinda magic to me still | 17:25 |
johnsom | Yeah, me too | 17:25 |
rm_work | FYI though I just tested the patch chain all the way to members in my devstack, and it seems to work and generate the correct config on the amp | 17:25 |
rm_work | though i've only tested happy-path creates | 17:25 |
johnsom | Cool! | 17:25 |
johnsom | Getting close! | 17:26 |
rm_work | yeah | 17:26 |
rm_work | pssst | 17:26 |
rm_work | add me to your calendar event thing for the octavia meeting :P | 17:26 |
rm_work | pretty sure it is today | 17:26 |
johnsom | Ah, yeah, PM me the e-mail you want me to use.... | 17:27 |
*** links has quit IRC | 17:39 | |
*** aojea_ has quit IRC | 17:46 | |
*** eezhova has joined #openstack-lbaas | 17:48 | |
*** armax has joined #openstack-lbaas | 17:53 | |
*** armax has quit IRC | 17:54 | |
rm_work | johnsom: # TODO(johnsom) Remove after deprecation (R series) | 17:57 |
rm_work | for... tenant_id? | 17:57 |
johnsom | both tenant_id and project_id | 17:57 |
rm_work | and what is the max_length for input_headers stuff? | 17:57 |
rm_work | 255? | 17:57 |
johnsom | Since this is a POST we should validate the length of everything. Yeah, we should check the DB size | 17:58 |
rm_work | no reason to have max_length on the Response type, right? | 18:00 |
*** kobis has quit IRC | 18:00 | |
johnsom | Right, just the post/put | 18:00 |
johnsom | Yeah, just throw 255 in | 18:01 |
johnsom | I think that is large enough | 18:01 |
johnsom | I think we have some body issues with Listener | 18:02 |
johnsom | I'm seeing default_tls_container_ref in nlbaas | 18:03 |
rm_work | johnsom: I am thinking maybe we can't do lengths | 18:06 |
rm_work | ValueError: Dictionaries key can only be a pod type | 18:06 |
johnsom | rm_work those TLS questions are good ones for you to weigh in on.... Just posted some comments | 18:06 |
rm_work | yeah I think that stuff is disabled until later? | 18:06 |
rm_work | i saw stuff about TLS | 18:06 |
johnsom | rm_work I see this: http://git.openstack.org/cgit/openstack/solum/tree/solum/api/controllers/camp/v1_1/datamodel/plans.py#n36 I think the "pod" thing is for the key, maybe we can at least limit the value? | 18:08 |
rm_work | ah maybe | 18:08 |
johnsom | Seems, less than ideal, but... | 18:08 |
rm_work | yeah ok seems correct | 18:09 |
rm_work | ok got it | 18:13 |
rm_work | and added the listeners to it for ankur-gupta-f1 | 18:13 |
rm_work | proposing momentarily | 18:13 |
rm_work | gotta run tests | 18:13 |
ankur-gupta-f4 | Huzzah! | 18:14 |
*** gcheresh_ has joined #openstack-lbaas | 18:15 | |
johnsom | rm_work one more simple to fix comment on connection_limit minimum | 18:15 |
rm_work | k | 18:16 |
johnsom | Sorry, and another. Still comparing to nlbaas code | 18:16 |
johnsom | Ok, that is all the comments I have for POST | 18:17 |
rm_work | ummm not sure what's up with the TLSTermination object | 18:24 |
rm_work | because like | 18:24 |
rm_work | we can't take in that data | 18:24 |
rm_work | ... | 18:24 |
johnsom | I hoped you knew.... | 18:24 |
rm_work | I'm removing it | 18:25 |
rm_work | >_> | 18:25 |
johnsom | Yeah, looks like old cold from the initial API commit | 18:26 |
rm_work | kk testing | 18:27 |
*** armax has joined #openstack-lbaas | 18:41 | |
rm_work | ugh ok | 18:42 |
rm_work | i'll have soemthing in a bit | 18:43 |
johnsom | NP, I jumped back to API-REF | 18:43 |
*** sanfern has quit IRC | 18:49 | |
*** sanfern has joined #openstack-lbaas | 18:49 | |
*** sanfern has quit IRC | 18:49 | |
*** sanfern has joined #openstack-lbaas | 18:50 | |
*** sanfern has quit IRC | 18:50 | |
*** sanfern has joined #openstack-lbaas | 18:51 | |
*** sanfern has quit IRC | 18:51 | |
eandersson | We are looking into implementing support for an alternative monitoring port. Is this already on the roadmap? If not, should we do a blueprint for it? | 18:59 |
*** harlowja has quit IRC | 18:59 | |
johnsom | eandersson Yes, it is on the roadmap: https://wiki.openstack.org/wiki/Octavia/Roadmap "Health monitoring on alternate IPs and/or ports" | 19:00 |
*** harlowja has joined #openstack-lbaas | 19:00 | |
eandersson | Sweet | 19:01 |
*** cpuga has joined #openstack-lbaas | 19:01 | |
*** harlowja has quit IRC | 19:01 | |
eandersson | Is there a blueprint for it already? | 19:01 |
johnsom | No, not yet. | 19:02 |
*** armax has quit IRC | 19:04 | |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Adds v2 load balancer API section https://review.openstack.org/438757 | 19:08 |
*** gcheresh_ has quit IRC | 19:08 | |
johnsom | Almost done with load balancer. But it is lunch time.... | 19:08 |
openstackgerrit | German Eichberger proposed openstack/octavia master: Initial version of the operator guide https://review.openstack.org/448759 | 19:11 |
*** cpuga has quit IRC | 19:17 | |
*** eezhova has quit IRC | 19:22 | |
*** armax has joined #openstack-lbaas | 19:33 | |
*** gcheresh_ has joined #openstack-lbaas | 19:41 | |
rm_work | ugh | 19:43 |
rm_work | the tests | 19:43 |
*** harlowja has joined #openstack-lbaas | 19:45 | |
openstackgerrit | German Eichberger proposed openstack/neutron-lbaas master: Fix README with link to octavia launchpad page https://review.openstack.org/447029 | 19:46 |
openstackgerrit | German Eichberger proposed openstack/neutron-lbaas master: Updating import for linux/interface opts https://review.openstack.org/352471 | 19:47 |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Octavia v2 API for listeners https://review.openstack.org/424744 | 19:50 |
rm_work | johnsom / ankur-gupta-f1 ^^ | 19:50 |
ankur-gupta-f4 | yup. Quickly need to rebase something then will pull it down | 19:51 |
ankur-gupta-f4 | thanks rm_work | 19:51 |
ankur-gupta-f4 | and by quickly rebase I mean like its all messed. gimme 15 | 19:51 |
rm_work | rofl no worries | 19:51 |
rm_work | i am gonna need to hand off, have meetings and then i need to head out after | 19:52 |
ankur-gupta-f4 | rm_work: okay no worries. Off top of your head what did you complete, what is left. We will tackle that first on the handoff then do the rest. | 19:54 |
rm_work | got all the comments | 19:55 |
rm_work | AFAIK that should be pretty close to good, other than the testing is a little light | 19:55 |
rm_work | it could use a bit more on both the units and the functional, I think | 19:55 |
johnsom | Octavia meeting starting soon on #openstack-meeting-alt | 19:57 |
*** armax has quit IRC | 20:16 | |
*** tesseract has quit IRC | 20:17 | |
*** harlowja has quit IRC | 20:18 | |
openstackgerrit | Merged openstack/neutron-lbaas master: Change default auth_version to v3 https://review.openstack.org/446836 | 20:32 |
openstackgerrit | Merged openstack/neutron-lbaas master: Updated from global requirements https://review.openstack.org/446970 | 20:33 |
*** bzhao_ has joined #openstack-lbaas | 20:43 | |
*** eezhova has joined #openstack-lbaas | 20:56 | |
rm_work | doing some rebasing so I can easily grab the end of the chain for testing | 21:01 |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Introduce Octavia v2 API for pools https://review.openstack.org/405922 | 21:01 |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Introduce Octavia v2 API for Members https://review.openstack.org/405932 | 21:01 |
*** chlong has quit IRC | 21:08 | |
openstackgerrit | German Eichberger proposed openstack/octavia master: Initial version of the operator guide https://review.openstack.org/448759 | 21:25 |
*** gcheresh_ has quit IRC | 21:28 | |
*** eezhova has quit IRC | 21:32 | |
*** harlowja has joined #openstack-lbaas | 21:33 | |
*** sshank has quit IRC | 21:37 | |
*** ndahiwade has quit IRC | 21:37 | |
*** dasm has quit IRC | 21:39 | |
*** pcaruana has quit IRC | 21:47 | |
*** sshank has joined #openstack-lbaas | 21:57 | |
*** Guest3223 has joined #openstack-lbaas | 21:58 | |
openstackgerrit | Merged openstack/octavia master: Change v2 API LB create return status code https://review.openstack.org/448307 | 22:07 |
openstackgerrit | Merged openstack/octavia master: Fix v2 LB API return for null description and name https://review.openstack.org/448317 | 22:08 |
openstackgerrit | Merged openstack/octavia master: Assorted trivial fixes for v2 loadbalancer tests https://review.openstack.org/448626 | 22:08 |
openstackgerrit | Merged openstack/neutron-lbaas master: Fix README with link to octavia launchpad page https://review.openstack.org/447029 | 22:09 |
*** fnaval has quit IRC | 22:28 | |
*** eandersson has quit IRC | 22:51 | |
*** sticker has quit IRC | 22:55 | |
*** sticker has joined #openstack-lbaas | 22:55 | |
*** bzhao_ has quit IRC | 22:58 | |
*** catintheroof has quit IRC | 23:01 | |
*** catintheroof has joined #openstack-lbaas | 23:02 | |
*** catintheroof has quit IRC | 23:06 | |
*** eandersson has joined #openstack-lbaas | 23:15 | |
*** krypto has quit IRC | 23:39 | |
*** armax has joined #openstack-lbaas | 23:52 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!