*** yamamoto has joined #openstack-lbaas | 00:13 | |
*** yamamoto has quit IRC | 00:17 | |
*** longkb has joined #openstack-lbaas | 00:21 | |
*** linhnm has joined #openstack-lbaas | 00:33 | |
*** yamamoto has joined #openstack-lbaas | 00:33 | |
*** yamamoto has quit IRC | 00:40 | |
*** yamamoto has joined #openstack-lbaas | 00:56 | |
*** yamamoto has quit IRC | 01:00 | |
*** yamamoto has joined #openstack-lbaas | 01:17 | |
openstackgerrit | Merged openstack/octavia-dashboard master: Fix sphinx-docs job for sphinx >1.7 https://review.openstack.org/568708 | 01:22 |
---|---|---|
*** yamamoto has quit IRC | 01:23 | |
*** yamamoto has joined #openstack-lbaas | 01:39 | |
*** yamamoto has quit IRC | 01:44 | |
*** atoth has quit IRC | 01:59 | |
*** yamamoto has joined #openstack-lbaas | 02:00 | |
*** annp has quit IRC | 02:03 | |
*** longkb has quit IRC | 02:03 | |
*** longkb has joined #openstack-lbaas | 02:03 | |
*** annp has joined #openstack-lbaas | 02:03 | |
*** linhnm has quit IRC | 02:03 | |
*** yamamoto has quit IRC | 02:05 | |
dayou | git review stops working for me, I got: ssh: connect to host review.openstack.org port 29418: Connection timed out, is it down or I am blocked? | 02:09 |
johnsom | Hmm, don’t know. You could ask in #openstack-infra | 02:19 |
*** yamamoto has joined #openstack-lbaas | 02:21 | |
*** yamamoto has quit IRC | 02:28 | |
dayou | Thanks, I asked there | 02:30 |
*** yamamoto has joined #openstack-lbaas | 02:44 | |
*** yamamoto has quit IRC | 02:48 | |
*** KeithMnemonic has quit IRC | 02:58 | |
*** yamamoto has joined #openstack-lbaas | 03:04 | |
*** yamamoto has quit IRC | 03:09 | |
johnsom | Hmm, seems like a lot of infra folks are not around due to early travel for the summit. | 03:12 |
johnsom | I'm about to work on a patch and will test git review in a bit | 03:12 |
*** annp has quit IRC | 03:20 | |
*** yamamoto has joined #openstack-lbaas | 03:27 | |
*** yamamoto has quit IRC | 03:32 | |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Implement provider drivers - Pool https://review.openstack.org/566698 | 03:41 |
johnsom | git review seems to work for me | 03:41 |
*** links has joined #openstack-lbaas | 03:41 | |
*** yamamoto has joined #openstack-lbaas | 03:48 | |
*** yamamoto has quit IRC | 03:54 | |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Implement provider drivers - Members https://review.openstack.org/566939 | 04:07 |
*** yamamoto has joined #openstack-lbaas | 04:11 | |
*** yamamoto has quit IRC | 04:15 | |
*** yamamoto has joined #openstack-lbaas | 04:16 | |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Implement provider drivers - L7 Policy https://review.openstack.org/567059 | 04:18 |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Implement provider drivers - Health Monitor https://review.openstack.org/567037 | 04:30 |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Implement provider drivers - Pool https://review.openstack.org/566698 | 04:31 |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Implement provider drivers - Members https://review.openstack.org/566939 | 04:31 |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Implement provider drivers - Health Monitor https://review.openstack.org/567037 | 04:31 |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Implement provider drivers - L7 Policy https://review.openstack.org/567059 | 04:31 |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Implement provider drivers - L7 Rules https://review.openstack.org/567073 | 04:44 |
johnsom | rm_work Ok, the chain has been updated | 04:45 |
*** sapd has quit IRC | 05:17 | |
*** annp has joined #openstack-lbaas | 05:34 | |
*** sapd has joined #openstack-lbaas | 05:56 | |
*** AlexStaf has quit IRC | 06:02 | |
*** xuhaiwei has joined #openstack-lbaas | 06:05 | |
*** kobis has joined #openstack-lbaas | 06:12 | |
*** ltomasbo has joined #openstack-lbaas | 06:34 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/octavia-dashboard master: Imported Translations from Zanata https://review.openstack.org/567789 | 06:42 |
*** longkb has quit IRC | 07:04 | |
*** longkb has joined #openstack-lbaas | 07:05 | |
*** rcernin has quit IRC | 07:10 | |
*** tesseract has joined #openstack-lbaas | 07:17 | |
*** yamamoto has quit IRC | 07:25 | |
rm_work | ok, gave you a number of +2'd | 07:30 |
rm_work | *+2's | 07:30 |
rm_work | we can merge a ways into that chain | 07:30 |
rm_work | but for a couple you'll have to merge the test side too | 07:30 |
rm_work | so ;) | 07:30 |
rm_work | reviews appreciated ;) | 07:31 |
openstackgerrit | Adam Harwell proposed openstack/octavia-tempest-plugin master: Fix sphinx-docs job for sphinx >1.7 https://review.openstack.org/568771 | 07:36 |
rm_work | ^^ just copied the fix that we did from the main repo, hopefully did it right | 07:37 |
xuhaiwei | Hi, I am installing octavia of Ocata version, but meets some issues, can anyone help? | 07:38 |
rm_work | Ocata is ... very old, in Octavia terms ;P | 07:39 |
rm_work | it doesn't even have the v2 api exposed yet | 07:39 |
xuhaiwei | when I create a lb from "neutron lbaas-loadbalancer-create" command, I saw this error in octavia api log, "MissingAuthPlugin: An auth plugin is required to fetch a token" | 07:39 |
rm_work | xuhaiwei: Octavia of any version can run on any cloud (it is not locked to the same versions as the other services) so I recommend you run something newer, ideally Queens | 07:40 |
xuhaiwei | rm_work: yes, it seems that, but my openstack is using ocata version | 07:40 |
rm_work | yeah, but you can install the Queens version of Octavia on an Ocata cloud | 07:40 |
rm_work | anyway, I think you just are missing some config | 07:40 |
rm_work | auth_strategy | 07:41 |
xuhaiwei | hmm, though octavia ocata version is very old, it should be running, right? | 07:41 |
rm_work | probably `auth_strategy = keystone` | 07:41 |
rm_work | in the default section | 07:41 |
*** AlexeyAbashkin has joined #openstack-lbaas | 07:42 | |
xuhaiwei | Yes, from the error log, it seems octavia is not configured with authentication | 07:42 |
xuhaiwei | oh, I will try it | 07:42 |
rm_work | ohh right | 07:42 |
rm_work | because you're doing neutron-lbaas | 07:42 |
rm_work | err | 07:42 |
rm_work | hold on | 07:42 |
rm_work | so probably actually it's missing auth data from one of the other sections | 07:42 |
xuhaiwei | I have these configs in [keystone_authtoken] | 07:43 |
xuhaiwei | auth_version=3 admin_project_domain = Default admin_tenant_name = service admin_user_domain = Default admin_password = Support admin_user = octavia auth_type = password | 07:43 |
xuhaiwei | and aslo auth_url | 07:43 |
rm_work | [service_auth] or [keystone_authtoken] section | 07:43 |
rm_work | do you also have service_auth section set up? | 07:44 |
xuhaiwei | yes, I even configured both of them, though I think I dont need to do | 07:44 |
xuhaiwei | I am not sure the exact parameter name, for exp, it is "tenant_name" or "admin_tenant_name"? | 07:45 |
*** yamamoto has joined #openstack-lbaas | 07:45 | |
xuhaiwei | Can you give me a correct example? | 07:45 |
*** yamamoto has quit IRC | 07:47 | |
*** yamamoto has joined #openstack-lbaas | 07:47 | |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Let healthmanager process shutdown cleanly (again) https://review.openstack.org/568711 | 07:47 |
rm_work | https://github.com/openstack/octavia/blob/master/etc/octavia.conf#L308 | 07:48 |
rm_work | and https://github.com/openstack/octavia/blob/master/etc/octavia.conf#L85 | 07:49 |
xuhaiwei | Ok, I will try again, thank you rm_work . | 07:49 |
*** dims has quit IRC | 07:59 | |
rm_work | unfortunately i'm not ... 100% awake right now. wish I could help more. good luck! | 08:01 |
*** dims has joined #openstack-lbaas | 08:02 | |
*** dims has quit IRC | 08:07 | |
*** dims has joined #openstack-lbaas | 08:07 | |
rm_work | yep https://review.openstack.org/#/c/568771/ passed, mergeplz | 08:21 |
rm_work | nmagnezi + dayou :P | 08:21 |
rm_work | assuming it looks good to you guys | 08:22 |
rm_work | I think a similar one merged in dashboard too... | 08:22 |
rm_work | ah, though maybe we don't need to exclude db/migration in tempest... but i don't think it hurts either. | 08:22 |
* rm_work shrugs | 08:22 | |
*** yboaron has joined #openstack-lbaas | 08:26 | |
openstackgerrit | Adam Harwell proposed openstack/octavia-tempest-plugin master: Fix sphinx-docs job for sphinx >1.7 https://review.openstack.org/568771 | 08:26 |
rm_work | ah i guess i should fix it | 08:26 |
rm_work | fixed | 08:26 |
rm_work | (i hope) | 08:26 |
rm_work | slightly less blind this time | 08:26 |
*** kobis has quit IRC | 08:41 | |
*** salmankhan has joined #openstack-lbaas | 08:54 | |
*** links has quit IRC | 09:07 | |
*** links has joined #openstack-lbaas | 09:23 | |
openstackgerrit | Jacky Hu proposed openstack/octavia-dashboard master: Replace noop tests with registration test https://review.openstack.org/550721 | 09:24 |
dayou | Sigh, ssh was blocked by our firewall | 09:25 |
dayou | switched to http works again | 09:25 |
openstackgerrit | Merged openstack/octavia-tempest-plugin master: Fix sphinx-docs job for sphinx >1.7 https://review.openstack.org/568771 | 09:31 |
openstackgerrit | Merged openstack/python-octaviaclient master: Fix for docs build in tox.ini https://review.openstack.org/568212 | 09:41 |
*** ispp has joined #openstack-lbaas | 09:49 | |
*** ispp has left #openstack-lbaas | 09:50 | |
*** issp has joined #openstack-lbaas | 09:50 | |
*** issp has quit IRC | 09:50 | |
*** ispp has joined #openstack-lbaas | 09:51 | |
*** ispp has quit IRC | 09:51 | |
*** ispp has joined #openstack-lbaas | 09:52 | |
*** yboaron_ has joined #openstack-lbaas | 09:59 | |
openstackgerrit | melissaml proposed openstack/octavia master: Trivial: Update pypi url to new url https://review.openstack.org/563386 | 10:01 |
*** yboaron has quit IRC | 10:01 | |
*** sapd has quit IRC | 10:10 | |
*** sapd has joined #openstack-lbaas | 10:11 | |
*** salmankhan has quit IRC | 10:14 | |
*** kobis has joined #openstack-lbaas | 10:14 | |
*** xuhaiwei has quit IRC | 10:17 | |
*** salmankhan has joined #openstack-lbaas | 10:18 | |
*** PagliaccisCloud has quit IRC | 10:21 | |
*** PagliaccisCloud has joined #openstack-lbaas | 10:23 | |
*** links has quit IRC | 10:44 | |
*** AlexStaf has joined #openstack-lbaas | 10:49 | |
*** kobis has quit IRC | 10:55 | |
*** links has joined #openstack-lbaas | 10:58 | |
*** AlexStaf has quit IRC | 10:58 | |
*** yboaron has joined #openstack-lbaas | 10:59 | |
*** yboaron_ has quit IRC | 10:59 | |
*** ispp has quit IRC | 11:04 | |
*** ispp has joined #openstack-lbaas | 11:06 | |
*** longkb has quit IRC | 11:22 | |
*** atoth has joined #openstack-lbaas | 11:51 | |
*** ispp has quit IRC | 12:03 | |
*** ispp has joined #openstack-lbaas | 12:14 | |
*** salmankhan has quit IRC | 12:20 | |
*** salmankhan has joined #openstack-lbaas | 12:23 | |
*** samccann has joined #openstack-lbaas | 12:55 | |
openstackgerrit | Jacky Hu proposed openstack/octavia-dashboard master: Allow member name to be set and updated https://review.openstack.org/563438 | 12:59 |
openstackgerrit | Jacky Hu proposed openstack/octavia-dashboard master: Allow members to be set as "backup" https://review.openstack.org/561387 | 12:59 |
openstackgerrit | Jacky Hu proposed openstack/octavia-dashboard master: Allow detail pages to auto refresh upon action https://review.openstack.org/561458 | 13:36 |
*** ivve has quit IRC | 13:47 | |
*** Alexey_Abashkin has joined #openstack-lbaas | 14:44 | |
*** AlexeyAbashkin has quit IRC | 14:46 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 14:46 | |
*** ispp has quit IRC | 14:57 | |
*** links has quit IRC | 14:58 | |
*** ispp has joined #openstack-lbaas | 15:05 | |
*** ispp has quit IRC | 15:27 | |
*** yboaron has quit IRC | 15:45 | |
*** salmankhan has quit IRC | 16:54 | |
*** links has joined #openstack-lbaas | 16:54 | |
johnsom | rm_work Ping me when you have a second, I have a question about one of the tempest patches | 16:59 |
*** yamamoto has quit IRC | 17:07 | |
*** yamamoto has joined #openstack-lbaas | 17:08 | |
*** links has quit IRC | 17:16 | |
*** AlexeyAbashkin has quit IRC | 17:16 | |
*** kobis has joined #openstack-lbaas | 17:22 | |
*** tesseract has quit IRC | 17:27 | |
*** AlexStaf has joined #openstack-lbaas | 17:41 | |
*** sshank has joined #openstack-lbaas | 17:43 | |
*** kobis has quit IRC | 17:45 | |
rm_work | johnsom: sure | 17:50 |
johnsom | rm_work https://review.openstack.org/#/c/565640/14/octavia_tempest_plugin/tests/api/v2/test_load_balancer.py Line 415, Why was this removed | 17:51 |
johnsom | ? | 17:51 |
rm_work | because it's not needed | 17:52 |
rm_work | switched to addCleanup | 17:52 |
rm_work | from addClassResourceCleanup | 17:52 |
rm_work | so tempest does the removing after the test instead of after the class | 17:52 |
johnsom | Ok, that was what I was wondering, so that runs cleanup after each test? | 17:52 |
rm_work | yes | 17:52 |
johnsom | Got it | 17:52 |
rm_work | so anywhere we were doing that "pre-cleanup" stuff... really we need to just tell tempest to do it correctly | 17:53 |
johnsom | Ok also on pool, there are TODOs for session persistence, are you planning to add those before we merge it? | 17:53 |
rm_work | hmmmm let me look | 17:53 |
rm_work | i thought i did get that | 17:53 |
johnsom | https://review.openstack.org/#/c/565640/14/octavia_tempest_plugin/tests/scenario/v2/test_pool.py | 17:53 |
johnsom | Like line 100 | 17:53 |
rm_work | ohhhh i fixed it in the other test | 17:54 |
rm_work | the API ones | 17:54 |
rm_work | but i had copied over the stuff already before that i guess | 17:54 |
rm_work | to the scenario | 17:54 |
rm_work | hmmm | 17:54 |
rm_work | i can prolly fix that if you'd like | 17:54 |
rm_work | or I can do a TODO cleanup run after we get the bases in | 17:55 |
rm_work | there's a number of them that needed to be done after-the-fact anyway | 17:55 |
johnsom | Yeah, ok | 17:55 |
johnsom | I was finishing testing the dashboard updates, then going to switch to testing those, but I ran into something strange, so I'm investigating. | 17:56 |
johnsom | The amp came up, but is failing network setup inside the amp. | 17:56 |
johnsom | Trying to enable serial console access in my devstack | 17:56 |
rm_work | :/ | 17:58 |
johnsom | blah, this isn't going to work. I don't have passwords enabled | 18:13 |
johnsom | Well, it's a back image I guess. Booting additional LBs doesn't work either. I'm going to build an image with root enabled and see what happens | 18:31 |
*** salmankhan has joined #openstack-lbaas | 18:36 | |
*** salmankhan has quit IRC | 18:41 | |
*** atoth has quit IRC | 18:42 | |
johnsom | Pretty sure the ubuntu image is broken at the moment | 18:55 |
johnsom | Hmm, maybe not. Maybe this is something strange on my devstack | 18:58 |
*** kobis has joined #openstack-lbaas | 19:09 | |
*** kobis has quit IRC | 19:13 | |
*** sshank has quit IRC | 19:15 | |
*** LutzB has quit IRC | 19:39 | |
*** yamamoto has quit IRC | 19:46 | |
*** LutzB has joined #openstack-lbaas | 19:49 | |
*** yamamoto has joined #openstack-lbaas | 19:49 | |
*** yamamoto has quit IRC | 19:51 | |
*** yamamoto has joined #openstack-lbaas | 19:51 | |
*** yamamoto has quit IRC | 19:51 | |
johnsom | #startmeeting Octavia | 20:00 |
openstack | Meeting started Wed May 16 20:00:18 2018 UTC and is due to finish in 60 minutes. The chair is johnsom. Information about MeetBot at http://wiki.debian.org/MeetBot. | 20:00 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 20:00 |
*** openstack changes topic to " (Meeting topic: Octavia)" | 20:00 | |
openstack | The meeting name has been set to 'octavia' | 20:00 |
johnsom | Hi folks! | 20:00 |
*** sapd_ has joined #openstack-lbaas | 20:00 | |
nmagnezi | o/ | 20:00 |
cgoncalves | hi | 20:00 |
AlexStaf | o/ | 20:00 |
xgerman_ | o/ | 20:00 |
johnsom | #topic Announcements | 20:01 |
*** openstack changes topic to "Announcements (Meeting topic: Octavia)" | 20:01 | |
*** rm_mobile has joined #openstack-lbaas | 20:01 | |
johnsom | Just a heads up, I am going to cancel next weeks meeting as many of us will be at the OpenStack summit in Vancouver. | 20:01 |
rm_mobile | o/ | 20:01 |
johnsom | BTW, who all is going to the summit? I know that rm_work/mobile, xgerman, and I will be there. Anyone else? | 20:02 |
nmagnezi | not me, sorry | 20:02 |
* johnsom looks sad | 20:02 | |
johnsom | Ok, well, there will at least be three of us representing Octavia. | 20:03 |
cgoncalves | I'm not | 20:03 |
AlexStaf | nope | 20:03 |
nmagnezi | how about you dayou? | 20:03 |
johnsom | I have two sessions, a project update and onboarding. | 20:03 |
*** sapd has quit IRC | 20:03 | |
nmagnezi | johnsom, nice! I'll watch those :) | 20:04 |
johnsom | Any other announcements I missed? | 20:04 |
johnsom | #topic Brief progress reports / bugs needing review | 20:05 |
*** openstack changes topic to "Brief progress reports / bugs needing review (Meeting topic: Octavia)" | 20:05 | |
johnsom | I have been focused on getting the provider driver code done. Everything is now posted for review except the driver library for the call backs. | 20:05 |
johnsom | I have a few documentation things to update as well. | 20:05 |
johnsom | And of course working on slides for the summti | 20:06 |
johnsom | Any other progress updates to share today? | 20:06 |
johnsom | There is some good stuff on dashboard. One patch posted makes the panels update status automatically. | 20:06 |
johnsom | Very cool to see actually | 20:06 |
johnsom | Adam has also been very busy getting the tempest plugin going. Great progress there | 20:07 |
nmagnezi | dayou promised this, and delivered :-) | 20:07 |
johnsom | yes, he has been doing great work on dashboard | 20:07 |
xgerman_ | +1 | 20:08 |
nmagnezi | +1 | 20:08 |
johnsom | I also hope to get some reviews in on some client patches as I would like to do a release for that soon too. | 20:08 |
rm_mobile | Ah yeah I have some debt in the client | 20:08 |
johnsom | Yeah, I need to add the provider API | 20:08 |
johnsom | The release folks are reminding me that we should have a client release soon-ish | 20:09 |
rm_mobile | Well the debt I have are for features only in master, so | 20:09 |
rm_mobile | Less urgent | 20:09 |
johnsom | Any other updates? Rally or Grenade? | 20:10 |
cgoncalves | I think I'm waiting for reviews on grenade | 20:10 |
johnsom | MS2 is the week of June 4th BTW | 20:10 |
xgerman_ | yeah, painfully aware | 20:11 |
nmagnezi | nothing new on Rally, we are at the final stretch with tripleO so it got priority | 20:11 |
johnsom | Ok, cool, good to know! Happen to have the link handy? | 20:11 |
nmagnezi | right afterwards I'll complete the Rally scenario | 20:11 |
nmagnezi | but it's in a good shape | 20:11 |
johnsom | Ok, cool | 20:11 |
*** yamamoto has joined #openstack-lbaas | 20:11 | |
nmagnezi | johnsom, yes, both cgoncalves and I can share some URLs to patches | 20:11 |
nmagnezi | #link https://review.openstack.org/#/c/568089/ | 20:12 |
nmagnezi | #link https://review.openstack.org/#/c/568138/ | 20:12 |
johnsom | Thanks! | 20:12 |
cgoncalves | https://review.openstack.org/#/q/(owner:%22Carlos+Goncalves+%253Ccgoncalves%2540redhat.com%253E%22)+OR+(owner:%22Nir+Magnezi+%253Cnmagnezi%2540redhat.com%253E%22+) | 20:12 |
nmagnezi | i guess that work too :) | 20:13 |
rm_mobile | I'm stuck in summit prep and travel prep for other stuff too, so I'm basically just trying to get through the reviews I have up already for the rest of this week | 20:13 |
nmagnezi | #link https://review.openstack.org/#/c/568395/ | 20:13 |
nmagnezi | #link https://review.openstack.org/#/c/568801/ | 20:14 |
rm_mobile | We've got a few large chunks of stuff just waiting because everyone is working hard on getting stuff done, but now there is so much done that it's a giant pile of reviews | 20:14 |
nmagnezi | johnsom, we have bunch more.. | 20:14 |
johnsom | Yep, ok, thanks! | 20:14 |
rm_mobile | So we may want to do a hardcore review spike during summit off-hours if possible <_< | 20:14 |
rm_mobile | I'll be prodding people about that | 20:15 |
xgerman_ | I hope there are off hours… | 20:15 |
rm_mobile | Lol yeah... | 20:15 |
johnsom | Yeah, I have been trying to mix some in. | 20:15 |
rm_mobile | I feel like usually we have a bit of time | 20:15 |
johnsom | #topic We need to decide on HTTP status codes for driver issues | 20:15 |
*** openstack changes topic to "We need to decide on HTTP status codes for driver issues (Meeting topic: Octavia)" | 20:15 | |
johnsom | So I would like some input on the status codes we want to return for provider driver issues. | 20:16 |
cgoncalves | rm_mobile, off-hours as in at 2 am? :) | 20:16 |
johnsom | #link https://review.openstack.org/#/c/563795/12/octavia/common/exceptions.py | 20:16 |
rm_mobile | Lol if need be | 20:16 |
johnsom | Ha, his off ours is like 8am | 20:16 |
cgoncalves | lol | 20:16 |
*** yamamoto has quit IRC | 20:16 | |
johnsom | So I initially picked 5xx for some of those | 20:16 |
rm_mobile | I think 501 for basically all of them | 20:17 |
johnsom | The bummer is that it would be nice if we never return a 500 on purpose | 20:17 |
johnsom | Well, ProviderNotEnabled is clearly user error and valid with 400 | 20:17 |
rm_mobile | 501 != 500 | 20:17 |
rm_mobile | Right | 20:17 |
rm_mobile | But the rest, 501 | 20:17 |
johnsom | Basically they didn't enter a valid driver name | 20:17 |
*** rm_mobile has quit IRC | 20:17 | |
*** rm_mobile has joined #openstack-lbaas | 20:18 | |
openstackgerrit | Merged openstack/octavia-dashboard master: Imported Translations from Zanata https://review.openstack.org/567789 | 20:18 |
johnsom | 501 is "Not Implemented" | 20:19 |
cgoncalves | 406 Not Acceptable for ProviderUnsupportedOptionError? | 20:19 |
johnsom | Which seems wrong for "ProviderDriverError" which is basically the driver raised some unknown exception | 20:19 |
*** rm_mobile| has joined #openstack-lbaas | 20:19 | |
cgoncalves | I think we should use more of 4xx at a quick glance | 20:19 |
johnsom | 406 is typically header stuff I think. | 20:20 |
cgoncalves | 405 Method Not Allowed for ProviderUnsupportedOptionError | 20:20 |
rm_mobile| | 4xx is all classified as client errrors | 20:20 |
johnsom | Both not implemented and unsupported option could be 400 as well | 20:20 |
rm_mobile| | Which is why I don't like it | 20:20 |
xgerman_ | https://http.cat/418 | 20:20 |
cgoncalves | 4xx "is intended for situations in which the error seems to have been caused by the client" | 20:21 |
rm_mobile| | Which in most cases here is not at all true | 20:22 |
rm_mobile| | We need to stick with 5xx for server config fails | 20:22 |
johnsom | Depends on if you expect a user asking for a feature knows if the driver supports it or not. lol | 20:22 |
rm_mobile| | Lol | 20:22 |
xgerman_ | johnsom: +1 | 20:22 |
*** rm_mobile has quit IRC | 20:23 | |
rm_mobile| | Ok time to figure out user intent programmatically | 20:23 |
johnsom | So, maybe switch ProviderNotImplementedError and ProviderUnsupportedOptionError to 400? | 20:23 |
rm_mobile| | No thanks IMO | 20:23 |
xgerman_ | yep, if it’s a RTFM type situation or the user is left intentionally in the dark about whihc providers are installed and support waht | 20:23 |
johnsom | Well there is an API that tells them which drivers are enabled for use. That is why ProviderNotEnabled is clearly a 400 | 20:24 |
cgoncalves | +1 on 400 | 20:24 |
johnsom | Yeah, looking again, 501 not implemented is probably right for those. | 20:25 |
rm_mobile| | Yes that one only | 20:25 |
johnsom | So it's really just ProviderDriverError the catch all | 20:25 |
rm_mobile| | The rest need to be 5xx | 20:25 |
johnsom | Do we leave it 500 or give it something else. | 20:25 |
rm_mobile| | 501? | 20:26 |
*** rm_mobile| has quit IRC | 20:26 | |
*** rm_mobile has joined #openstack-lbaas | 20:26 | |
johnsom | If people don't read the error, 500 gets interpreted as "octavia is broken". But.... in a way that driver is broken | 20:26 |
cgoncalves | ProviderDriverError 500. driver was expected to fulfill request but failed | 20:27 |
johnsom | I guess my vote is to leave it 500 | 20:27 |
johnsom | It's the most accurate technically | 20:27 |
johnsom | Any other votes? I think rm_mobile is 501 on that | 20:28 |
xgerman_ | I like to blame the user… | 20:30 |
rm_mobile | Yep | 20:30 |
rm_mobile | 500 I never even read, I just assume the server screwed up | 20:31 |
xgerman_ | so lean towards 4xx but not strong enought for a vote | 20:31 |
rm_mobile | Usually it's uncaught explosions | 20:31 |
johnsom | Yeah, that is what this is, uncaught driver explosions | 20:31 |
rm_mobile | Well, we catch it | 20:32 |
rm_mobile | And we know kinda what's up | 20:32 |
xgerman_ | how do other driver based projects do it? neutron? | 20:32 |
*** rm_mobile| has joined #openstack-lbaas | 20:32 | |
johnsom | Ok, well, it doesn't sound like we have strong opinions so I'll leave it to the reviewers votes... | 20:33 |
johnsom | I just thought it would be good to discuss as a team | 20:33 |
rm_mobile| | Yeah | 20:33 |
xgerman_ | +1 | 20:33 |
johnsom | #topic Open Discussion | 20:33 |
*** openstack changes topic to "Open Discussion (Meeting topic: Octavia)" | 20:33 | |
johnsom | Any other topics for this week? | 20:33 |
*** yamamoto has joined #openstack-lbaas | 20:34 | |
xgerman_ | oh, I need some OSP tech support | 20:34 |
xgerman_ | especially why Octavia doesn’t work in OSP 12… | 20:35 |
johnsom | Ha | 20:35 |
cgoncalves | xgerman_, it's not supported (tm) | 20:35 |
cgoncalves | what's the status code for that? 4xx or 5xx? :P | 20:35 |
nmagnezi | that's because.. we don't actually ship it in OSP12 as a supported component :) | 20:35 |
johnsom | Very happy to see the OSP 13-beta release notes with Octavia fully supported | 20:35 |
xgerman_ | +1 | 20:35 |
nmagnezi | johnsom, indeed :) | 20:35 |
xgerman_ | anywho, the Tech Preview in 12 doesn’t install the SSL certs — hope that’s fixed in 13 | 20:36 |
cgoncalves | yep! | 20:36 |
cgoncalves | either use can provide certs or let OSP generate | 20:36 |
*** rm_mobile has quit IRC | 20:36 | |
cgoncalves | https://bugzilla.redhat.com/show_bug.cgi?id=1462670 | 20:37 |
openstack | bugzilla.redhat.com bug 1462670 in openstack-tripleo-common "Octavia TripleO support: allow auto-generated or user-provided certificates when configuring octavia" [High,Verified] - Assigned to beagles | 20:37 |
nmagnezi | yup. and anyhow for OSP13 we'll be happy to get any feedback and fix what's needed | 20:37 |
cgoncalves | OSP Octavia tracker: https://bugzilla.redhat.com/show_bug.cgi?id=1433523 | 20:37 |
openstack | bugzilla.redhat.com bug 1433523 in openstack-octavia "[Tracker] [Octavia] Full support" [High,Assigned] - Assigned to nmagnezi | 20:37 |
xgerman_ | somehow my org wants to use OSP12 | 20:37 |
nmagnezi | "Assigned to nmagnezi" | 20:37 |
* nmagnezi runs | 20:37 | |
cgoncalves | nmagnezi, you're the squad lead :P | 20:37 |
nmagnezi | cgoncalves, that was a secret.. | 20:37 |
cgoncalves | xgerman_, bad! OSP13 is LTS | 20:37 |
nmagnezi | :) | 20:38 |
xgerman_ | I am juts an engineer (TM) | 20:38 |
johnsom | Anything else today? | 20:39 |
*** yamamoto has quit IRC | 20:40 | |
johnsom | Ok, thanks folks! Enjoy the quiet while we are at the Summit. Great time for reviewing..... grin | 20:40 |
johnsom | #endmeeting | 20:41 |
nmagnezi | o/ | 20:41 |
*** openstack changes topic to "Discussion of OpenStack Load Balancing (Octavia) | https://etherpad.openstack.org/p/octavia-priority-reviews" | 20:41 | |
openstack | Meeting ended Wed May 16 20:41:04 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 20:41 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/octavia/2018/octavia.2018-05-16-20.00.html | 20:41 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/octavia/2018/octavia.2018-05-16-20.00.txt | 20:41 |
openstack | Log: http://eavesdrop.openstack.org/meetings/octavia/2018/octavia.2018-05-16-20.00.log.html | 20:41 |
xgerman_ | o/ | 20:41 |
*** yamamoto has joined #openstack-lbaas | 20:55 | |
*** salmankhan has joined #openstack-lbaas | 20:56 | |
*** sshank has joined #openstack-lbaas | 20:58 | |
*** salmankhan has quit IRC | 21:01 | |
*** yamamoto has quit IRC | 21:02 | |
*** samccann has quit IRC | 21:04 | |
*** yamamoto has joined #openstack-lbaas | 21:18 | |
*** yamamoto has quit IRC | 21:23 | |
*** yamamoto has joined #openstack-lbaas | 21:39 | |
*** yamamoto has quit IRC | 21:46 | |
*** rm_mobile| has quit IRC | 21:47 | |
*** rm_mobile has joined #openstack-lbaas | 21:48 | |
*** sshank has quit IRC | 21:49 | |
*** rm_mobile| has joined #openstack-lbaas | 21:50 | |
*** rm_mobile has quit IRC | 21:55 | |
*** leitan has joined #openstack-lbaas | 21:55 | |
*** leitan has quit IRC | 22:01 | |
*** leitan has joined #openstack-lbaas | 22:01 | |
*** yamamoto has joined #openstack-lbaas | 22:22 | |
*** rcernin has joined #openstack-lbaas | 22:25 | |
*** yamamoto has quit IRC | 22:29 | |
*** rm_mobile| has quit IRC | 22:31 | |
rm_work | johnsom: had to fix the test on https://review.openstack.org/#/c/568711/ but it's good now | 22:42 |
johnsom | Ok | 22:43 |
*** yamamoto has joined #openstack-lbaas | 22:45 | |
*** yamamoto has quit IRC | 22:51 | |
rm_work | is https://review.openstack.org/#/c/541039/ going to just be stuck forever? are they working on the dependency still? | 22:53 |
rm_work | is that dependency strictly necessary? | 22:53 |
johnsom | yeah, they don't like my solution. I haven't gone back to find a different approach. That change they make is a breaking change for any tempest plugin at a local path | 22:54 |
johnsom | make/made | 22:54 |
rm_work | err | 22:55 |
rm_work | we don't have one of those? | 22:55 |
rm_work | or... do we? | 22:55 |
rm_work | oh, neutron-lbaas? | 22:55 |
johnsom | Both neutron-lbaas and octavia v1 tempest tests are local | 22:56 |
rm_work | oh right i forgot about v1 tempests even | 22:56 |
rm_work | should we just ... move those out to the plugin? | 22:56 |
rm_work | I could copy them out there | 22:56 |
rm_work | we have the structure for it and such | 22:56 |
johnsom | Their flippant answer was "you were supposed to make separate repo for the plugins" | 22:56 |
johnsom | No, I don't want to contaminate our new shiny plugin | 22:57 |
rm_work | lol | 22:57 |
rm_work | i mean, it's split up | 22:57 |
johnsom | I have considered creating repos for those | 22:57 |
rm_work | v1 / v2 | 22:57 |
rm_work | such that you could just wipe out any v1 folders and it'd be fine again | 22:57 |
rm_work | so, i'd say decently well quarantined | 22:57 |
johnsom | I think the path forward might be hacking their code to re-remove the project from the upper constraints file. See the problem is they edit the upper constraints file and add in the project itself, so when it later comes along to install the plugin I blows up because the project is in the constraints file and it can't check a local file path for version | 22:59 |
johnsom | It's a bit messier than that I think | 22:59 |
johnsom | So it's like, undo their hack with another hack | 22:59 |
rm_work | have we decided we're going to be bullheaded and not just ... get the stuff out of the repo? :P | 23:00 |
johnsom | bullheaded -> lazy | 23:01 |
rm_work | lol | 23:01 |
rm_work | i mean yeah | 23:01 |
johnsom | Both of those tempest plugins are going away | 23:01 |
rm_work | well | 23:01 |
rm_work | how long do we need to keep OUR v1? | 23:02 |
johnsom | It should die when neutron-lbaas dies IMO | 23:02 |
rm_work | as long as the API right? | 23:02 |
rm_work | which ... is that on the same deprecation cycle? | 23:02 |
rm_work | k | 23:02 |
rm_work | i think i agree | 23:02 |
rm_work | but we never officially stated it, did we? | 23:02 |
johnsom | No, not yet | 23:03 |
rm_work | hmmm did we ever switch it to DEPRECATED in version discovery/docs even? | 23:03 |
johnsom | I planned to do it when we start the clock for nlbaas | 23:03 |
rm_work | k | 23:03 |
johnsom | we switched it to "supported" | 23:03 |
*** yamamoto has joined #openstack-lbaas | 23:08 | |
rm_work | johnsom: updating the pools patch to do SP now | 23:08 |
rm_work | did you want me to do all of the ipv4/6 stuff on members now? | 23:08 |
johnsom | Ok | 23:08 |
openstackgerrit | Adam Harwell proposed openstack/octavia-tempest-plugin master: Create api+scenario tests for pools https://review.openstack.org/565640 | 23:09 |
johnsom | Yeah, I think so, it should be quick, just a wrapper like LB create is | 23:09 |
rm_work | k | 23:09 |
johnsom | Now live traffic, that is going to be harder | 23:09 |
rm_work | learning about `git review -x` is a huge timesaver :P | 23:09 |
rm_work | well did you look at my HM test | 23:09 |
rm_work | that added live traffic testing for HM stuff? | 23:09 |
johnsom | No | 23:10 |
rm_work | i guess it doesn't use ipv6 member addresses | 23:10 |
rm_work | so that'll be a little ... yeah | 23:10 |
rm_work | but traffic testing for HMs and members is actually not too bad | 23:10 |
johnsom | I looked at the HM patch but not the live traffic one yet | 23:10 |
rm_work | I believe I was pretty thorough | 23:10 |
rm_work | i mean it's part of the HM patch | 23:10 |
johnsom | Oh, yeah, nevermind, I didn't get to the HM one yet | 23:11 |
johnsom | I stopped at members | 23:11 |
*** yamamoto has quit IRC | 23:12 | |
*** leitan has quit IRC | 23:19 | |
openstackgerrit | Adam Harwell proposed openstack/octavia-tempest-plugin master: Create api+scenario tests for members https://review.openstack.org/566199 | 23:29 |
*** yamamoto has joined #openstack-lbaas | 23:29 | |
*** yamamoto has quit IRC | 23:33 | |
*** yamamoto has joined #openstack-lbaas | 23:49 | |
*** yamamoto has quit IRC | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!