rm_work | bleh | 00:01 |
---|---|---|
rm_work | also: my noop HMs seem to be going "ONLINE" | 00:02 |
rm_work | which ... | 00:03 |
rm_work | did HMs just not have an op-status before? | 00:03 |
rm_work | op_status = (constants.ONLINE if health_mon.enabled | 00:05 |
rm_work | else constants.OFFLINE) | 00:05 |
xgerman_ | Nope. dougwig Shot it down | 00:05 |
rm_work | so HMs always go online if they're enabled | 00:05 |
rm_work | different from neutron-lbaas | 00:05 |
rm_work | apparently | 00:05 |
dougwig | I did what now? | 00:07 |
xgerman_ | Shut down cascading delete in neutron lbsaas | 00:07 |
xgerman_ | Also why is mestery running away from you :-) | 00:08 |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Revert CentOS image size bump https://review.openstack.org/562882 | 00:29 |
rm_work | ^^ pending passing cent7 gate, but it should prove that's actually fine after all T_T | 00:30 |
dougwig | As I recall, I wanted it, but it wasn’t ready for a particular release. But I could be addled. | 00:31 |
rm_work | well, we're now in feature freeze, so >_> | 00:37 |
openstackgerrit | Jacky Hu proposed openstack/octavia-dashboard master: Allow cascade deletion of load balancer https://review.openstack.org/553381 | 00:41 |
rm_work | speaking of which | 00:41 |
rm_work | lolol | 00:41 |
rm_work | dayou: hilarious timing on that ;) | 00:42 |
dayou | rm_work: morning exercise :) | 00:43 |
rm_work | we were JUST talking about cascade delete | 00:43 |
*** harlowja has quit IRC | 00:46 | |
rm_work | johnsom: LOL this is an interesting one | 00:47 |
rm_work | johnsom: when you pass a non-int weight to a member in Octavia, we round it down >_> | 00:47 |
rm_work | so 0.1 is accepted and becomes 0 | 00:47 |
rm_work | ALSO, somehow the member update is not working for weight O_o | 00:55 |
*** slaweq has joined #openstack-lbaas | 00:56 | |
johnsom | rm_work You mean via the client? | 00:57 |
rm_work | i mean ... no | 00:57 |
rm_work | not via client | 00:57 |
johnsom | The API enforces int (or should) | 00:57 |
rm_work | i actually was able to add a *unit test* and verify | 00:57 |
rm_work | it accepts a float and does a floor() | 00:57 |
johnsom | https://github.com/openstack/octavia/blob/master/octavia/api/v2/types/member.py#L75 | 00:57 |
rm_work | so add to octavia/tests/unit/api/v2/types/test_members.py | 00:58 |
rm_work | http://paste.openstack.org/show/719600/ | 00:58 |
rm_work | via API i verified this | 00:58 |
rm_work | and also via unit test | 00:58 |
rm_work | it definitely is doing it | 00:58 |
johnsom | Blah python. Yeah, wsme doesn't seem to care | 00:58 |
rm_work | <_< | 00:59 |
rm_work | so yeah. awesome. | 00:59 |
rm_work | anyway, the next one | 00:59 |
rm_work | working on proving this out or figuring out what's up | 00:59 |
rm_work | i see it doing an update to a member to change the weight | 00:59 |
rm_work | and the weight doesn't change | 00:59 |
rm_work | :( | 00:59 |
johnsom | https://docs.python.org/2/library/functions.html#int | 01:00 |
johnsom | It doesn't change in the haproxy.cfg? | 01:01 |
rm_work | i'm in NOOP | 01:01 |
johnsom | oh | 01:01 |
*** slaweq has quit IRC | 01:01 | |
johnsom | less surprising | 01:01 |
rm_work | just doing API testing | 01:01 |
rm_work | <_< | 01:01 |
rm_work | our functional tests don't run flows | 01:02 |
rm_work | so we can't actually test this | 01:02 |
johnsom | A bit confused, but ok. I mean, yeah, the functional API tests are there to test the API, not the flows... | 01:03 |
johnsom | Ok, it's date night, so I'm out for the evening. | 01:04 |
rm_work | johnsom: API doesn't update the DB with the result? | 01:04 |
rm_work | or does it | 01:04 |
rm_work | oh it does nmn | 01:04 |
rm_work | ok have fun | 01:04 |
rm_work | oh no i was right, it doesn't | 01:05 |
rm_work | it sends the updated data to the queue, right? so with no flows, the updates never make it to the DB | 01:06 |
johnsom | Right, you are running no-op, so they get stored in the no-op memory store | 01:06 |
rm_work | err | 01:07 |
rm_work | i'm running noop for amp/compute/network | 01:07 |
rm_work | not for controller | 01:07 |
johnsom | It's not running that funky controller simulator thingy | 01:07 |
johnsom | ? | 01:07 |
rm_work | so they go through to the queue | 01:07 |
rm_work | and the worker picks up the job and does the flows | 01:07 |
rm_work | so in my env, it SHOULD be updating | 01:07 |
rm_work | no | 01:07 |
rm_work | i'm running the same as the API gate | 01:07 |
rm_work | it runs with noop drivers with the real controller | 01:08 |
rm_work | hmmm well it is updating when I do it from the UI | 01:12 |
johnsom | Hmm, then it should be running with the sqlite DB | 01:12 |
rm_work | but when tempest does it, it doesn't update | 01:12 |
rm_work | weird | 01:12 |
rm_work | johnsom: err, why would it be? | 01:12 |
rm_work | it's got mysql behind it | 01:12 |
johnsom | If it's a functional test | 01:13 |
rm_work | oh, functional | 01:13 |
rm_work | yeah | 01:13 |
rm_work | but the functional tests end at the producer | 01:13 |
rm_work | that's as far as they go -- no flows run | 01:13 |
rm_work | and the attribute updates don't happen in the DB until the flows | 01:13 |
rm_work | anyway, the problem must be in the way tempest does this update :/ | 01:13 |
johnsom | Ah, well, I'm out. | 01:14 |
rm_work | kk later | 01:14 |
rm_work | OH | 01:14 |
rm_work | yeah this test is wtfdumb | 01:15 |
rm_work | it's looking ... at the response of the update | 01:15 |
rm_work | which is lies | 01:15 |
openstackgerrit | Adam Harwell proposed openstack/neutron-lbaas master: WIP: Test l7 proxy to octavia https://review.openstack.org/561049 | 01:57 |
rm_work | still a few to fix | 01:57 |
rm_work | but figured out local testing so now my turnaround time is actually sane | 01:57 |
rm_work | lol this test catches a thing | 02:01 |
rm_work | neutron_lbaas.tests.tempest.v2.api.test_pools_non_admin.TestPools.test_update_pool_empty_session_persistence | 02:01 |
rm_work | if you pass a string as a session persistence, we 500 | 02:01 |
rm_work | will figure that out in a bit | 02:02 |
*** dlundquist has joined #openstack-lbaas | 02:04 | |
xgerman_ | if only that gate would work for me I could make some progress | 02:10 |
*** dlundquist has quit IRC | 02:15 | |
xgerman_ | well, I am the only job in there with tempest as a parent — guess once that fixes itself I should be in good shape | 02:16 |
xgerman_ | ok, rechecking daily it is | 02:21 |
johnsom | rm_work: I have a patch up for SP | 02:23 |
*** slaweq has joined #openstack-lbaas | 02:57 | |
*** threestrands has joined #openstack-lbaas | 02:59 | |
*** slaweq has quit IRC | 03:02 | |
*** KeithMnemonic2 has joined #openstack-lbaas | 03:14 | |
*** KeithMnemonic1 has quit IRC | 03:17 | |
*** annp has quit IRC | 03:54 | |
*** annp has joined #openstack-lbaas | 03:55 | |
*** sanfern has joined #openstack-lbaas | 04:29 | |
*** slaweq has joined #openstack-lbaas | 04:58 | |
*** slaweq has quit IRC | 05:03 | |
*** links has joined #openstack-lbaas | 05:26 | |
*** slaweq has joined #openstack-lbaas | 06:00 | |
*** slaweq has quit IRC | 06:04 | |
*** pcaruana has joined #openstack-lbaas | 06:43 | |
*** yamamoto has quit IRC | 06:50 | |
*** velizarx has joined #openstack-lbaas | 06:53 | |
*** threestrands has quit IRC | 07:00 | |
*** rcernin has quit IRC | 07:02 | |
*** velizarx has quit IRC | 07:10 | |
*** tesseract has joined #openstack-lbaas | 07:12 | |
*** velizarx has joined #openstack-lbaas | 07:26 | |
*** slaweq has joined #openstack-lbaas | 07:31 | |
*** AlexeyAbashkin has joined #openstack-lbaas | 07:37 | |
*** yamamoto has joined #openstack-lbaas | 07:51 | |
*** yamamoto has quit IRC | 08:01 | |
*** yamamoto has joined #openstack-lbaas | 08:13 | |
*** yamamoto has quit IRC | 08:44 | |
*** KeithMnemonic2 has quit IRC | 08:52 | |
*** KeithMnemonic2 has joined #openstack-lbaas | 08:53 | |
*** dayou has quit IRC | 09:09 | |
*** dayou has joined #openstack-lbaas | 09:11 | |
*** yamamoto has joined #openstack-lbaas | 09:44 | |
*** yamamoto has quit IRC | 09:54 | |
*** ndanl has joined #openstack-lbaas | 10:20 | |
*** pcaruana has quit IRC | 10:31 | |
*** dulek has quit IRC | 10:46 | |
*** dulek has joined #openstack-lbaas | 10:50 | |
*** sanfern has quit IRC | 10:55 | |
*** velizarx has quit IRC | 10:58 | |
*** velizarx has joined #openstack-lbaas | 10:59 | |
*** yamamoto has joined #openstack-lbaas | 11:31 | |
*** velizarx has quit IRC | 11:31 | |
*** velizarx has joined #openstack-lbaas | 11:34 | |
*** yamamoto_ has joined #openstack-lbaas | 11:44 | |
*** pcaruana has joined #openstack-lbaas | 11:46 | |
*** yamamoto has quit IRC | 11:47 | |
*** annp has quit IRC | 12:02 | |
*** sanfern has joined #openstack-lbaas | 12:03 | |
*** atoth has joined #openstack-lbaas | 12:07 | |
*** yamamoto_ has quit IRC | 12:46 | |
*** yamamoto has joined #openstack-lbaas | 12:47 | |
*** yamamoto has quit IRC | 12:52 | |
*** ndanl has quit IRC | 12:54 | |
*** yamamoto has joined #openstack-lbaas | 13:21 | |
*** fyx has quit IRC | 13:21 | |
*** fyx has joined #openstack-lbaas | 13:21 | |
*** samccann has joined #openstack-lbaas | 13:26 | |
*** atoth has quit IRC | 13:38 | |
*** dougwig has quit IRC | 13:40 | |
*** dougwig has joined #openstack-lbaas | 13:40 | |
*** atoth has joined #openstack-lbaas | 13:57 | |
*** dims has quit IRC | 13:58 | |
*** mnaser has quit IRC | 14:00 | |
*** mnaser has joined #openstack-lbaas | 14:01 | |
*** links has quit IRC | 14:03 | |
*** dims has joined #openstack-lbaas | 14:04 | |
*** velizarx has quit IRC | 14:08 | |
*** fnaval has quit IRC | 14:34 | |
*** fnaval has joined #openstack-lbaas | 14:35 | |
*** fnaval has quit IRC | 14:40 | |
*** fnaval has joined #openstack-lbaas | 14:45 | |
*** zioproto has quit IRC | 14:52 | |
*** zioproto has joined #openstack-lbaas | 14:52 | |
*** beisner has quit IRC | 14:58 | |
*** beisner has joined #openstack-lbaas | 14:58 | |
*** redondo-mk has quit IRC | 15:11 | |
*** redondo-mk has joined #openstack-lbaas | 15:12 | |
*** tesseract has quit IRC | 15:58 | |
*** dims has quit IRC | 16:06 | |
*** AlexeyAbashkin has quit IRC | 16:09 | |
*** dims has joined #openstack-lbaas | 16:11 | |
*** yamamoto has quit IRC | 16:31 | |
*** sanfern has quit IRC | 16:50 | |
rm_work | johnsom: oh, you already fixed the SP issue? nice | 16:52 |
johnsom | rm_work https://review.openstack.org/560579 | 16:53 |
rm_work | ummm | 16:53 |
rm_work | that already merged | 16:53 |
rm_work | this problem is in Master | 16:53 |
johnsom | So maybe something more needed? | 16:54 |
rm_work | octavia/tests/functional/api/v2/test_pool.py http://paste.openstack.org/show/719646/ | 16:55 |
rm_work | run that | 16:55 |
rm_work | i think that may be a wsme bug? | 16:59 |
johnsom | Yeah, definitely a wsme bug | 17:04 |
johnsom | I know a core over there if you want to fix | 17:04 |
johnsom | grin | 17:04 |
rm_work | i have no idea where the fix for that would be | 17:06 |
rm_work | but I will open a story | 17:06 |
johnsom | Somewhere here: https://github.com/openstack/wsme/blob/master/wsme/rest/json.py#L139 | 17:07 |
rm_work | errr if they are on storyboard | 17:07 |
johnsom | I don't think so | 17:07 |
johnsom | https://bugs.launchpad.net/wsme/+bugs | 17:07 |
rm_work | yeah i found it | 17:07 |
rm_work | looking to see if there's already one | 17:08 |
*** pcaruana has quit IRC | 17:22 | |
*** yamamoto has joined #openstack-lbaas | 17:32 | |
*** pcaruana has joined #openstack-lbaas | 17:33 | |
*** yamamoto has quit IRC | 17:38 | |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Pool PUT should validate name/description length https://review.openstack.org/563202 | 17:46 |
rm_work | johnsom: another bug ^^ | 17:47 |
rm_work | these tests are useful apparently | 17:47 |
rm_work | getting some legit failures | 17:47 |
johnsom | Yeah, darn, not sure how we missed that. Good catch | 17:48 |
johnsom | We should probably have functional test for that stuff | 17:49 |
rm_work | i mean, i do feel pretty personally responsible for most of these API bugs T_T | 17:49 |
rm_work | for the types? | 17:49 |
rm_work | I feel like unit for wsme validation is good enough | 17:49 |
johnsom | Oh, you had help. grin | 17:49 |
rm_work | what about https://review.openstack.org/#/c/562869/ | 17:51 |
rm_work | johnsom: and can you add the releasenote or whatever to the DELETED 404 patch so we can merge that | 17:53 |
johnsom | Yeah, in plan for today. I'm reviewing this OSA octavia certificate generation stuff at the moment. | 17:54 |
rm_work | k | 17:54 |
*** juggsmcb has joined #openstack-lbaas | 17:56 | |
*** juggsmcb has quit IRC | 17:58 | |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Pool PUT should validate name/description length https://review.openstack.org/563202 | 18:02 |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Add pecan middleware for neutron-lbaas compat https://review.openstack.org/562063 | 18:03 |
rm_work | rebase-juggling | 18:03 |
*** pcaruana has quit IRC | 18:29 | |
*** yamamoto has joined #openstack-lbaas | 18:34 | |
johnsom | rm_work The flavor_id think looks ok, we just need to update the api-ref at the same time. It has it in the loadbalancer.inc and the examples. | 18:35 |
rm_work | oh right | 18:36 |
rm_work | ok will fix | 18:36 |
johnsom | Geez, seems like I'm grumpy or something, handing out -1's left and right. | 18:37 |
*** yamamoto has quit IRC | 18:38 | |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Correct flavor to flavor_id in LB API https://review.openstack.org/562869 | 18:43 |
rm_work | fixed | 18:43 |
johnsom | PTG is back in Denver Sept 10-14 | 18:44 |
johnsom | Same hotel | 18:44 |
rm_work | TRAAAAAAAINS 2.0 | 18:51 |
rm_work | Also I hope they upgraded their internet stuff | 18:51 |
rm_work | the internet in that hotel was SHIIIIIIIITTY in the rooms | 18:51 |
rm_work | the openstack internet was fine | 18:51 |
rm_work | johnsom: https://github.com/openstack/neutron-lbaas/blob/master/neutron_lbaas/tests/tempest/v2/api/test_load_balancers_non_admin.py#L276-L278 | 18:53 |
rm_work | in neutron-lbaas that's a 404? | 18:53 |
rm_work | in octavia it's a 400 | 18:53 |
rm_work | not sure how/why the hell it's a 404 in neutron-lbaas | 18:53 |
rm_work | (it must be if ex.NotFound is passing?) | 18:53 |
rm_work | it's ex.BadRequest in Octavia | 18:53 |
johnsom | Flavors were grade D but implemented there, so... I think 400 is the right answer myself. | 18:54 |
xgerman_ | I spend some time comparing my fialing gate in proxy with the one we use in Octavia but couldn’t find a smoking gun… if somebody has an idea I am all ears | 18:59 |
rm_work | xgerman_: well, mine is almost done | 19:00 |
rm_work | there were a number of additional fixes i had to do | 19:00 |
rm_work | so you might get some inspiration there | 19:00 |
xgerman_ | yes, once that gate runs I shall be done in no-time as well — it’;s just zuul hates me | 19:01 |
rm_work | yeah but i mean, maybe check out how i'm running my gate | 19:01 |
rm_work | since it does run ;) | 19:01 |
*** harlowja has joined #openstack-lbaas | 19:02 | |
xgerman_ | lucky you :-) | 19:02 |
johnsom | xgerman_ is it the version /path thing with pip? | 19:02 |
xgerman_ | yep | 19:02 |
johnsom | xgerman_ add depends-on https://review.openstack.org/562356 | 19:03 |
xgerman_ | ok | 19:03 |
johnsom | Not sure if they are going to allow that to merge or not, but it fixes that change in devstack | 19:03 |
openstackgerrit | German Eichberger proposed openstack/neutron-lbaas master: Gate API test for the lbaasv2-proxy plugin https://review.openstack.org/539350 | 19:04 |
rm_work | johnsom: also interestingly, we do our validation at a different stage, so it's impossible to test a provider/flavor_id conflict when there is no valid flavor_id to pass in T_T | 19:05 |
xgerman_ | well, my proxy has been under a cloud — so will proabbly be another year until it merges | 19:05 |
rm_work | xgerman_: i thought you just hadn't been working on it for a long time :( | 19:05 |
xgerman_ | no, 1st I had to wait for the api to become available, then reviews, then… | 19:06 |
johnsom | rm_work That whole provider/flavor thing in octavia will need to change be updated. It's part of the provider WIP | 19:06 |
rm_work | k | 19:11 |
rm_work | I guess I will just skip those tests | 19:11 |
rm_work | and reference the octavia provider framework story | 19:11 |
rm_work | johnsom: i can't find the story for providers... | 19:25 |
rm_work | there isn't one linked on your patches, and nothing comes up on search | 19:26 |
rm_work | did we ... not make one? | 19:26 |
rm_work | ah i found it | 19:27 |
rm_work | https://storyboard.openstack.org/#!/story/1655768 | 19:27 |
rm_work | you should add that and the task 5165 to your provider support commits | 19:28 |
rm_work | man, the cleanup for these tests takes a long time and i have no idea why | 19:29 |
openstackgerrit | Adam Harwell proposed openstack/neutron-lbaas master: WIP: Test l7 proxy to octavia https://review.openstack.org/561049 | 19:34 |
*** yamamoto has joined #openstack-lbaas | 19:34 | |
*** imacdonn has quit IRC | 19:36 | |
*** imacdonn has joined #openstack-lbaas | 19:36 | |
*** yamamoto has quit IRC | 19:39 | |
*** atoth has quit IRC | 19:58 | |
*** yamamoto has joined #openstack-lbaas | 20:36 | |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Fix calls to "DELETED" items https://review.openstack.org/545493 | 20:38 |
johnsom | I feel dirty updating the API date to today's date.... | 20:38 |
*** yamamoto has quit IRC | 20:41 | |
rm_work | <_< | 20:41 |
rm_work | that's what we got | 20:41 |
johnsom | I think you have a good point on https://review.openstack.org/#/c/559460 I will add some logic for that | 20:45 |
rm_work | erg so we verify urls differently that neutron-lbaas I guess? | 20:47 |
johnsom | nlbaas verifies something? | 20:47 |
rm_work | SUPPORTED_URL_PATH = "^(/([a-zA-Z0-9-._~!$&\'()*+,;=:@]|(%[a-fA-F0-9]{2}))*)+$" | 20:47 |
rm_work | in neutron-lbaas | 20:47 |
rm_work | in octavia we use: | 20:47 |
rm_work | wsme URLPathType | 20:48 |
rm_work | which does: | 20:48 |
rm_work | rfc3986.urlparse(rfc3986.normalize_uri(url_path)) | 20:48 |
rm_work | and we verify the scheme/userinfo/host/port/path | 20:50 |
rm_work | or rather, we verify that it has none of those except a path, and that the path is not none | 20:50 |
rm_work | so what that boils down to is, we allow things without a leading `/` in octavia, but in neutron-lbaas we *require* a leading `/` | 20:51 |
rm_work | if we don't put a `/` does the HM work in haproxy? | 20:51 |
johnsom | I think that is a no, I vaguely remember something about that. | 20:51 |
johnsom | The API ref has The HTTP URL path of the request sent by the monitor to test the health of a backend member. Must be a string that begins with a forward slash (/). The default URL path is /. | 20:52 |
johnsom | I can check pretty quick if you want | 20:53 |
rm_work | ok so we should add that requirement somehow | 20:54 |
johnsom | Hmm, well, the config validated | 20:57 |
*** yamamoto has joined #openstack-lbaas | 20:57 | |
*** slaweq has quit IRC | 21:00 | |
*** slaweq has joined #openstack-lbaas | 21:01 | |
*** yamamoto has quit IRC | 21:04 | |
*** slaweq has quit IRC | 21:06 | |
*** KeithMnemonic2 has quit IRC | 21:07 | |
johnsom | rm_work https://github.com/openstack/octavia/blob/master/octavia/controller/healthmanager/health_drivers/update_db.py#L134 | 21:09 |
johnsom | Why did you put that so low in the method, shouldn't we throw it away if it's over time right at the start of the method? | 21:10 |
johnsom | I guess accounting for DB lookup time? | 21:11 |
rm_work | yes | 21:19 |
rm_work | that's where the time actually was | 21:19 |
rm_work | the threads "start" pretty quick | 21:19 |
rm_work | then it hangs on that could maybe have done it right after 114 | 21:19 |
*** yamamoto has joined #openstack-lbaas | 21:20 | |
rm_work | err that last one was supposed to be two sentences | 21:21 |
rm_work | that / could | 21:22 |
johnsom | Maybe it should be checked twice, one to quick fail before the first DB query and then later where it is now. If it was stuck on the queue too long we might as well fail early and save DB load | 21:24 |
*** yamamoto has quit IRC | 21:25 | |
* rm_work shrugs | 21:25 | |
johnsom | rm_work Along those lines, the backports are all in conflict. are you going to update those? | 21:26 |
rm_work | wait what | 21:26 |
rm_work | how, i just rebased yesterday T_T | 21:26 |
johnsom | a billion gate fixes most likely | 21:26 |
rm_work | T_T k yeah i'll fix | 21:27 |
rm_work | johnsom: weird, it doesn't show it being in merge conflict on the actual patch? | 21:29 |
rm_work | and it won't let me rebase because it looks like it's on HEAD | 21:29 |
rm_work | hmm, no it's not on HEAD | 21:30 |
rm_work | ok i'll do it manually | 21:30 |
rm_work | k, all rebased | 21:32 |
johnsom | Thanks! | 21:32 |
johnsom | I want to start working to get those in | 21:32 |
*** samccann has quit IRC | 21:37 | |
xgerman_ | +1 | 21:40 |
*** yamamoto has joined #openstack-lbaas | 21:41 | |
xgerman_ | rm_work: http://logs.openstack.org/50/539350/37/check/neutron-lbaasv2-dsvm-api-proxy/618b7cb/testr_results.html.gz | 21:41 |
rm_work | this is yours? | 21:43 |
rm_work | hmmm | 21:43 |
rm_work | yeah some of those look familiar | 21:44 |
xgerman_ | yep, after a week of gate failures… | 21:44 |
rm_work | yeah so it looks like you still don't have the policy set right | 21:44 |
xgerman_ | I do | 21:44 |
rm_work | it's not setting the legacy policy thing I think? | 21:45 |
rm_work | or somehow it's using the wrong account for deletes | 21:45 |
xgerman_ | the later | 21:45 |
rm_work | not sure how that could be happening | 21:45 |
xgerman_ | yeah, at least the gates are running again so I can test fixes | 21:46 |
rm_work | johnsom: so ... somehow the status of a NOOP loadbalancer gets set to ONLINE in neutron-lbaas? | 21:47 |
*** yamamoto has quit IRC | 21:47 | |
rm_work | see this: http://logs.openstack.org/49/561049/29/check/neutron-lbaasv2-dsvm-api/b4160c8/logs/stestr_results.html.gz | 21:47 |
johnsom | Yeah, I know. | 21:47 |
rm_work | ok so that's a known issue? | 21:48 |
rm_work | bug? in one of the two? | 21:48 |
openstackgerrit | Adam Harwell proposed openstack/neutron-lbaas master: WIP: Test l7 proxy to octavia https://review.openstack.org/561049 | 22:00 |
*** yamamoto has joined #openstack-lbaas | 22:04 | |
*** yamamoto has quit IRC | 22:08 | |
*** yamamoto has joined #openstack-lbaas | 22:24 | |
*** yamamoto has quit IRC | 22:29 | |
johnsom | rm_work are you working on that rebase or should I? | 22:33 |
rm_work | ? i did it? | 22:37 |
rm_work | oh | 22:37 |
rm_work | THAT rebase | 22:37 |
rm_work | i'll get it | 22:37 |
rm_work | debugging something with someone on zoom presently | 22:37 |
*** yamamoto has joined #openstack-lbaas | 22:45 | |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Fix calls to "DELETED" items https://review.openstack.org/545493 | 22:50 |
*** yamamoto has quit IRC | 22:51 | |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Fix version discovery for the Octavia API https://review.openstack.org/559460 | 22:55 |
*** fnaval has quit IRC | 22:57 | |
*** yamamoto has joined #openstack-lbaas | 23:08 | |
rm_work | johnsom: so i'm down to JUST these known issues: http://logs.openstack.org/49/561049/30/check/neutron-lbaasv2-dsvm-api-l7octavia/470ddb8/logs/stestr_results.html.gz | 23:08 |
rm_work | filed a bug for https://bugs.launchpad.net/wsme/+bug/1765796 | 23:10 |
openstack | Launchpad bug 1765796 in WSME "Passing a string to a nested type fails to json-decode" [Undecided,New] | 23:10 |
*** yamamoto has quit IRC | 23:12 | |
*** rcernin has joined #openstack-lbaas | 23:15 | |
*** joadavis-web has joined #openstack-lbaas | 23:18 | |
*** joadavis-web has quit IRC | 23:19 | |
rm_work | johnsom: so did the haproxy config validate? without / ? | 23:20 |
*** yamamoto has joined #openstack-lbaas | 23:20 | |
rm_work | did you actually test it? | 23:20 |
johnsom | Yeah, it did | 23:20 |
rm_work | but does it *work* work? | 23:20 |
johnsom | I don't have a way to test that right now. | 23:20 |
rm_work | if so... should we relax that requirement? | 23:21 |
rm_work | k | 23:21 |
johnsom | I think it sill makes sense to require the / | 23:21 |
rm_work | hmmm k | 23:24 |
johnsom | The way I read the RFC it should have the / | 23:25 |
johnsom | https://tools.ietf.org/html/rfc3986#section-3.3 | 23:25 |
*** yamamoto has quit IRC | 23:32 | |
rm_work | well, that doesn't seem to be the way the rfc was implemented in python T_T | 23:33 |
*** yamamoto has joined #openstack-lbaas | 23:33 | |
rm_work | unless there's some flag we missed, like ... "strict=True" or "actually_verify_correctly=True" | 23:33 |
johnsom | I thought the rfc python package looks for the /, it doesn't? | 23:33 |
rm_work | *in octavia*, we use the wsme validator for URLType | 23:34 |
rm_work | which uses: | 23:34 |
rm_work | p_url = rfc3986.urlparse(rfc3986.normalize_uri(url_path)) | 23:34 |
johnsom | Yeah, which should be good. Doesn't it require the /? | 23:34 |
rm_work | nope | 23:35 |
rm_work | verified in unit tests | 23:35 |
johnsom | Ah, ok, confused. | 23:35 |
johnsom | Well, then we probably could update the api-ref. It's still odd | 23:35 |
rm_work | I opened https://storyboard.openstack.org/#!/story/2001891 | 23:44 |
rm_work | I can fix it, the validator is ours | 23:45 |
rm_work | we can just add one more check | 23:45 |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Correct flavor to flavor_id in LB API https://review.openstack.org/562869 | 23:55 |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Pool PUT should validate name/description length https://review.openstack.org/563202 | 23:55 |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Health Monitor url_path requires leading slash https://review.openstack.org/563341 | 23:56 |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Add pecan middleware for neutron-lbaas compat https://review.openstack.org/562063 | 23:56 |
openstackgerrit | Adam Harwell proposed openstack/neutron-lbaas master: WIP: Test l7 proxy to octavia https://review.openstack.org/561049 | 23:56 |
rm_work | OK -- might actually *pass* | 23:57 |
rm_work | johnsom: omgWUT | 23:57 |
rm_work | johnsom: you 2.1'd the DELETE patch? | 23:57 |
rm_work | O_o | 23:58 |
johnsom | Which? | 23:58 |
rm_work | https://review.openstack.org/#/c/545493/10/octavia/api/root_controller.py | 23:58 |
johnsom | Oh! Ha, I was thinking like votes | 23:59 |
johnsom | Yeah, I think we need to indicate that change in the version # | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!