| rm_work | xgerman_: https://review.openstack.org/#/c/477045/ | 00:00 |
|---|---|---|
| *** sanfern has quit IRC | 00:03 | |
| *** sanfern has joined #openstack-lbaas | 00:04 | |
| rm_work | johnsom: did you fix that other doc bug i mentioned | 00:20 |
| johnsom | The deprecated issue? | 00:20 |
| johnsom | yes | 00:20 |
| rm_work | no | 00:20 |
| rm_work | the one where updated_at was in the POST | 00:20 |
| rm_work | err, was it the PUT | 00:21 |
| johnsom | Oh, yeah, that was fixed in one of the patches you just reviewed | 00:21 |
| rm_work | lol k | 00:22 |
| rm_work | must have glazed over it | 00:22 |
| rm_work | my eyes are starting to hurt | 00:22 |
| rm_work | launching in the morning | 00:22 |
| rm_work | first Octavia-only deployment? :P | 00:22 |
| johnsom | https://review.openstack.org/#/c/477174/3/api-ref/source/v2/listener.inc | 00:22 |
| rm_work | i feel like the amount of work we got done in this cycle is *insane* | 00:23 |
| johnsom | Agreed | 00:23 |
| johnsom | The change report for the final release should be entertaining. | 00:24 |
| rm_work | lol | 00:24 |
| rm_work | yeah i guess we really should have been making reno notes for bugfixes too | 00:24 |
| rm_work | that would have been entertaining | 00:24 |
| johnsom | We are #8 for Pike for lines of code too | 00:24 |
| rm_work | lol | 00:24 |
| johnsom | Yeah, well, release notes for bug fixes is umm, maybe a bit much in my mind. Unless it is major | 00:25 |
| johnsom | Oh, that reminds me. I need a release not for the RBAC | 00:25 |
| rm_work | ah | 00:29 |
| johnsom | I'm adding it to the open RBAC patch. Sorry, you will have to re-+2 | 00:29 |
| rm_work | lol np | 00:30 |
| rm_work | johnsom: so in http://docs-draft.openstack.org/73/478673/2/check/gate-octavia-api-ref/2788f7c//api-ref/build/html/ there is ... no Deprecated API version? | 00:31 |
| rm_work | you're leaving it there with nothing under it? | 00:31 |
| rm_work | could we put "None" or something under it | 00:32 |
| johnsom | We could | 00:33 |
| rm_work | also, your warning isn't rendering for me | 00:33 |
| johnsom | That's because the link is taking you to the "live" doc, not that draft one. | 00:34 |
| rm_work | ah | 00:34 |
| johnsom | It will when it's published | 00:34 |
| rm_work | yep ;) | 00:34 |
| rm_work | you are correct | 00:34 |
| rm_work | kk | 00:35 |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Correct RBAC Not Authorized status code https://review.openstack.org/477045 | 00:46 |
| johnsom | We will see how that looks | 00:46 |
| *** sanfern has quit IRC | 00:47 | |
| *** sanfern has joined #openstack-lbaas | 00:47 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Update our docs to point to the new api-ref https://review.openstack.org/478673 | 00:48 |
| johnsom | Ok, added "None" ^^^^ | 00:49 |
| openstackgerrit | Merged openstack/octavia master: Fix nova secgroup-add-rule which is unavailable https://review.openstack.org/478408 | 00:51 |
| openstackgerrit | Merged openstack/octavia master: Change default vagrant box to xenial64(16.04) https://review.openstack.org/478360 | 00:51 |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Correct RBAC Not Authorized status code https://review.openstack.org/477045 | 00:51 |
| johnsom | release notes link wasn't rendoring | 00:52 |
| johnsom | rendering ugh | 00:52 |
| rm_work | both look good to me now | 00:58 |
| rm_work | :thumbsup: | 00:58 |
| johnsom | Had to step away, I will be back in like 30 | 01:02 |
| *** leitan has joined #openstack-lbaas | 01:09 | |
| rm_work | xgerman_: if you are around, we've got a list of stuff for you :) | 01:09 |
| openstackgerrit | Adam Harwell proposed openstack/octavia master: Introduce nlbaas shim driver layer for APIv2 https://review.openstack.org/409398 | 01:13 |
| rm_work | rebase ^^ | 01:13 |
| *** leitan has quit IRC | 01:13 | |
| johnsom | Ok, back for a bit | 01:15 |
| johnsom | rm_work Are you ok with this too? https://review.openstack.org/#/c/478637/ | 01:15 |
| rm_work | johnsom: how do we specify the rabbit topic for the MAIN provisioning? | 01:15 |
| rm_work | the [oslo_messaging] section is for the neutron-lbaas queue | 01:15 |
| rm_work | AFAICT | 01:15 |
| rm_work | ah nm it's used for bot | 01:16 |
| rm_work | *both | 01:16 |
| rm_work | johnsom: if it's *working* working | 01:16 |
| rm_work | i need to check the run logs | 01:16 |
| johnsom | Yeah, I think there is room for improvement in the queue stuff | 01:16 |
| johnsom | Yeah, it's all there to see in the gates | 01:17 |
| rm_work | OCTAVIA_SERVICE_TYPE="load-balancer" | 01:17 |
| johnsom | Sigh, yep | 01:17 |
| rm_work | was really never set already? | 01:17 |
| rm_work | oh | 01:17 |
| rm_work | it wasn't a constant | 01:17 |
| rm_work | k | 01:18 |
| johnsom | Right, not a constant | 01:18 |
| johnsom | I thought you were going to comment on the "er" or "-" | 01:18 |
| johnsom | grin | 01:18 |
| rm_work | yep it seems to be uwsgi alright | 01:18 |
| rm_work | +2 | 01:18 |
| rm_work | lol | 01:19 |
| rm_work | yes i still hate the hyphen with a passion | 01:19 |
| rm_work | but whatever | 01:19 |
| johnsom | The uwsgi config is a bit *odd* IMO, but that is being generated by devstack for us, so.... | 01:19 |
| johnsom | I liked "ing" better than "er" myself, but eh, it's a token | 01:19 |
| rm_work | so, is there a reason we can't merge the shim and then tweak? | 01:20 |
| rm_work | i feel like we're at a sort of chicken/egg impasse at the moment | 01:21 |
| rm_work | vendors won't help us test because we "aren't ready", but we aren't ready because vendors won't help us test | 01:21 |
| johnsom | I want to look at it and decide if I think we need to do more up front design before committing to that. | 01:21 |
| rm_work | yeah k fair | 01:21 |
| johnsom | It's in my court I think, just keep nagging me | 01:21 |
| rm_work | and i guess that can wait a little bit | 01:21 |
| rm_work | compared to some of this other stuff | 01:21 |
| johnsom | Right, trying to prioritize | 01:22 |
| johnsom | Oye, I have eight patches in flight right now. Probably more before the night is out. | 01:24 |
| rm_work | yeah | 01:33 |
| rm_work | that's become kinda a normal feeling | 01:33 |
| rm_work | if xgerman_ would show up we could fix that | 01:33 |
| rm_work | :P | 01:33 |
| johnsom | Well, half of them are spread over other repos. Docs, governance, etc. | 01:34 |
| sanfern | johnsom, hi | 01:37 |
| johnsom | Hi | 01:37 |
| sanfern | i am emitting only LB provisioning_status | 01:37 |
| sanfern | do you want all other entities provisional_status to be emitted | 01:37 |
| johnsom | Yeah, we should probably sync all of them | 01:38 |
| johnsom | rm_work Freebee https://review.openstack.org/#/c/478084/ | 01:39 |
| rm_work | i should start checking our other projects >_> | 01:41 |
| johnsom | We still need to fix that bug in OSC and release again. | 01:41 |
| johnsom | I was looking to see if Jude had done that and I missed it | 01:41 |
| sanfern | ok thanks johnsom | 01:42 |
| openstackgerrit | Merged openstack/python-octaviaclient master: Updated from global requirements https://review.openstack.org/478084 | 01:46 |
| openstackgerrit | Adam Harwell proposed openstack/octavia master: WIP: Floating IP Network Driver (spans L3s) https://review.openstack.org/435612 | 01:49 |
| rm_work | ^^ rebased | 01:49 |
| rm_work | johnsom: ah yeah | 01:50 |
| rm_work | johnsom: i was gonna ask too | 01:50 |
| rm_work | which bug was it | 01:50 |
| rm_work | i can prolly do it really quick | 01:50 |
| johnsom | https://bugs.launchpad.net/octavia/+bug/1700778 | 01:50 |
| openstack | Launchpad bug 1700778 in octavia "Octavia OSC plugin is failing to create l7rules" [Critical,Triaged] - Assigned to Jude Cross (problem-v) | 01:50 |
| rm_work | oh just needed to remove the upper() calls? | 01:50 |
| johnsom | Yeah, and probably search to make sure there aren't any others | 01:51 |
| johnsom | I'm not 100% sure what the issue is there, I haven't dug into that one yet. Just know the client pukes but the API works | 01:52 |
| rm_work | i think it's just two | 01:52 |
| rm_work | let me push this really quick | 01:52 |
| openstackgerrit | Adam Harwell proposed openstack/python-octaviaclient master: Remove rogue calls to str.upper in l7rules https://review.openstack.org/478693 | 01:53 |
| rm_work | you have a stack up you can check really quick? | 01:53 |
| rm_work | mine is busted | 01:53 |
| rm_work | those were the only two upper calls left in the project | 01:54 |
| johnsom | Umm, I can revive it. I broke mine with uwsgi | 01:54 |
| rm_work | lol | 01:54 |
| johnsom | Just a minute, poking at OSC documentation stuffs. | 01:54 |
| rm_work | np | 01:54 |
| rm_work | i'm about to head to dinner, have a date-night thing kinda tonight | 01:55 |
| johnsom | Nice! | 01:55 |
| rm_work | so at some point I am going to have to stop working | 01:55 |
| rm_work | oh, what do you think are the chances of having the docs actually... up | 01:56 |
| rm_work | like | 01:56 |
| rm_work | tomorrow sometime :P | 01:56 |
| rm_work | that would be excellent as it would coincide with our go-live ^_^ | 01:56 |
| johnsom | The docs patches are all up for review except for OSC client, which I'm working on right now. | 01:57 |
| rm_work | oh actually | 01:58 |
| rm_work | err | 01:58 |
| rm_work | did it merge already? | 01:58 |
| rm_work | oh nm | 01:58 |
| rm_work | err | 01:58 |
| rm_work | is https://developer.openstack.org/api-ref/load-balancer/ the real place? | 01:58 |
| rm_work | it seems to be our current docs | 01:58 |
| rm_work | (includes quotas) | 01:58 |
| johnsom | Yes, that is the "official" location for the api-ref | 01:58 |
| rm_work | sooo | 01:58 |
| rm_work | the governance change is ... just to LIST it? | 01:59 |
| rm_work | ah | 01:59 |
| johnsom | Right | 01:59 |
| rm_work | yeah ok | 01:59 |
| rm_work | cool | 01:59 |
| rm_work | so we're fine then already | 01:59 |
| johnsom | We become a grown up and get listed here: https://developer.openstack.org/api-guide/quick-start/ | 01:59 |
| johnsom | Ugh, I guess we really should tag v1 as "supported" | 02:00 |
| johnsom | I knew there was something I was forgetting there | 02:02 |
| *** reedip_out_of_ar has joined #openstack-lbaas | 02:07 | |
| rm_work | ugh | 02:07 |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Update our docs to point to the new api-ref https://review.openstack.org/478673 | 02:08 |
| johnsom | There we go. | 02:08 |
| rm_work | AH there | 02:09 |
| rm_work | i like that | 02:09 |
| rm_work | I hated them both being "CURRENT" | 02:09 |
| johnsom | Yet | 02:09 |
| johnsom | Yep | 02:09 |
| johnsom | Totally forgot about the SUPPORTED state | 02:09 |
| rm_work | xgerman_: https://review.openstack.org/#/q/project:openstack/octavia+status:open+label:Code-Review%253D2+-label:Verified-1 | 02:09 |
| rm_work | plz to be reviewing | 02:10 |
| rm_work | i'm starting to work on my gerrit-search-foo | 02:10 |
| rm_work | k bbl | 02:12 |
| *** gtrxcb has joined #openstack-lbaas | 02:13 | |
| xgerman_ | Mmh, just got home... | 02:15 |
| johnsom | Someone is anxious for Christmas to arrive.... | 02:16 |
| *** reedip_out_of_ar has quit IRC | 02:27 | |
| *** sanfern has quit IRC | 02:51 | |
| openstackgerrit | Merged openstack/octavia master: Correct RBAC Not Authorized status code https://review.openstack.org/477045 | 03:08 |
| openstackgerrit | Merged openstack/octavia master: Enable uwsgi support for octavia devstack https://review.openstack.org/478637 | 03:19 |
| openstackgerrit | Merged openstack/octavia master: Event Streamer for StatsDb should respect config value https://review.openstack.org/478665 | 03:25 |
| *** gans has joined #openstack-lbaas | 03:32 | |
| *** sanfern has joined #openstack-lbaas | 03:47 | |
| *** yamamoto has joined #openstack-lbaas | 03:50 | |
| *** gans819 has joined #openstack-lbaas | 03:52 | |
| *** gans has quit IRC | 03:55 | |
| *** amotoki_away is now known as amotoki | 03:58 | |
| *** yuanying has joined #openstack-lbaas | 04:18 | |
| *** yuanying_ has quit IRC | 04:19 | |
| openstackgerrit | Merged openstack/octavia master: Update our docs to point to the new api-ref https://review.openstack.org/478673 | 04:20 |
| openstackgerrit | Kiran Totad proposed openstack/python-octaviaclient master: Replace six.iteritems() with .items() https://review.openstack.org/478724 | 04:31 |
| *** links has joined #openstack-lbaas | 04:53 | |
| *** gans819 has quit IRC | 05:02 | |
| *** links has quit IRC | 05:03 | |
| *** yuanying has quit IRC | 05:04 | |
| *** yuanying has joined #openstack-lbaas | 05:06 | |
| openstackgerrit | Joshua Harlow proposed openstack/octavia master: Gather fail/pass after executor is done https://review.openstack.org/477720 | 05:11 |
| *** pcaruana has joined #openstack-lbaas | 05:14 | |
| *** links has joined #openstack-lbaas | 05:15 | |
| openstackgerrit | tuninger proposed openstack/neutron-lbaas master: Update members data based on haproxy stats data. https://review.openstack.org/476848 | 05:18 |
| openstackgerrit | tuninger proposed openstack/neutron-lbaas master: Update members data based on haproxy stats data. https://review.openstack.org/476848 | 05:26 |
| openstackgerrit | tuninger proposed openstack/neutron-lbaas master: Update members data based on haproxy stats data. https://review.openstack.org/476848 | 05:28 |
| *** pcaruana has quit IRC | 05:30 | |
| openstackgerrit | tuninger proposed openstack/neutron-lbaas master: Update members data based on haproxy stats data. https://review.openstack.org/476848 | 05:31 |
| *** pcaruana has joined #openstack-lbaas | 05:33 | |
| *** pcaruana has quit IRC | 05:39 | |
| *** csomerville has quit IRC | 05:43 | |
| *** cody-somerville has joined #openstack-lbaas | 05:43 | |
| *** rcernin_ has joined #openstack-lbaas | 05:48 | |
| openstackgerrit | Santhosh Fernandes proposed openstack/octavia master: This patch will emit provisioning status to be sync with neutron db https://review.openstack.org/478385 | 05:59 |
| *** rcernin_ is now known as rcernin | 06:04 | |
| *** aojea has joined #openstack-lbaas | 07:21 | |
| *** slaweq has quit IRC | 07:24 | |
| *** tesseract has joined #openstack-lbaas | 07:31 | |
| *** pcaruana has joined #openstack-lbaas | 07:35 | |
| *** tesseract has quit IRC | 07:47 | |
| *** openstackgerrit has quit IRC | 07:47 | |
| *** tesseract has joined #openstack-lbaas | 07:48 | |
| *** junbo has quit IRC | 08:03 | |
| *** junbo has joined #openstack-lbaas | 08:06 | |
| *** tesseract has quit IRC | 08:30 | |
| *** tesseract has joined #openstack-lbaas | 08:32 | |
| *** openstackgerrit has joined #openstack-lbaas | 08:55 | |
| openstackgerrit | Andreas Jaeger proposed openstack/octavia master: Update api-ref for openstackdocstheme 1.11 https://review.openstack.org/478806 | 08:55 |
| openstackgerrit | Andreas Jaeger proposed openstack/octavia master: Update api-ref for openstackdocstheme 1.11 https://review.openstack.org/478806 | 08:56 |
| *** aojea has quit IRC | 09:19 | |
| *** aojea has joined #openstack-lbaas | 09:20 | |
| *** sanfern has quit IRC | 09:30 | |
| *** sanfern has joined #openstack-lbaas | 09:38 | |
| *** diltram has quit IRC | 09:38 | |
| *** tesseract has quit IRC | 09:38 | |
| *** tesseract has joined #openstack-lbaas | 09:40 | |
| *** yamamoto has quit IRC | 09:41 | |
| *** diltram has joined #openstack-lbaas | 09:42 | |
| *** yamamoto has joined #openstack-lbaas | 09:43 | |
| *** yamamoto has quit IRC | 09:43 | |
| *** sanfern has quit IRC | 09:45 | |
| *** sanfern has joined #openstack-lbaas | 09:46 | |
| *** aojea has quit IRC | 09:59 | |
| *** aojea has joined #openstack-lbaas | 10:00 | |
| *** kobis has joined #openstack-lbaas | 10:03 | |
| *** yamamoto has joined #openstack-lbaas | 10:06 | |
| *** kobis has quit IRC | 10:17 | |
| *** diltram has quit IRC | 10:37 | |
| *** diltram has joined #openstack-lbaas | 10:43 | |
| openstackgerrit | Santhosh Fernandes proposed openstack/octavia master: [WIP] This patch will emit provisioning status to be sync with neutron db https://review.openstack.org/478385 | 10:44 |
| *** atoth has joined #openstack-lbaas | 10:44 | |
| *** sanfern has quit IRC | 10:49 | |
| *** aojea has quit IRC | 10:55 | |
| *** aojea has joined #openstack-lbaas | 10:55 | |
| openstackgerrit | iswarya vakati proposed openstack/octavia master: Use get_rpc_transport instead of get_transport https://review.openstack.org/478859 | 10:57 |
| *** diltram has quit IRC | 11:04 | |
| *** diltram has joined #openstack-lbaas | 11:08 | |
| *** aojea has quit IRC | 11:15 | |
| *** aojea has joined #openstack-lbaas | 11:15 | |
| *** diltram has quit IRC | 11:15 | |
| *** diltram has joined #openstack-lbaas | 11:22 | |
| *** kobis has joined #openstack-lbaas | 11:36 | |
| *** yamamoto has quit IRC | 11:43 | |
| *** sanfern has joined #openstack-lbaas | 11:44 | |
| *** yamamoto has joined #openstack-lbaas | 11:55 | |
| *** yamamoto has quit IRC | 12:00 | |
| *** yamamoto has joined #openstack-lbaas | 12:12 | |
| *** leitan has joined #openstack-lbaas | 12:22 | |
| *** cpuga has joined #openstack-lbaas | 12:23 | |
| *** cpuga_ has joined #openstack-lbaas | 12:24 | |
| *** cpuga has quit IRC | 12:28 | |
| *** aojea has quit IRC | 12:28 | |
| *** aojea has joined #openstack-lbaas | 12:29 | |
| *** sanfern has quit IRC | 12:35 | |
| *** chlong_ has joined #openstack-lbaas | 12:35 | |
| *** sanfern has joined #openstack-lbaas | 12:35 | |
| *** leitan has quit IRC | 12:49 | |
| openstackgerrit | Merged openstack/neutron-lbaas-dashboard master: Exclude node_modules from pep8 checking https://review.openstack.org/475801 | 13:52 |
| openstackgerrit | Merged openstack/neutron-lbaas-dashboard master: Fix npm test https://review.openstack.org/475798 | 13:52 |
| *** kobis has quit IRC | 14:17 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Update Octavia docs for documentation migration https://review.openstack.org/478947 | 14:23 |
| *** aojea has quit IRC | 14:25 | |
| *** aojea has joined #openstack-lbaas | 14:26 | |
| *** csomerville has joined #openstack-lbaas | 14:34 | |
| *** cody-somerville has quit IRC | 14:37 | |
| *** reedip_out_of_ar has joined #openstack-lbaas | 14:44 | |
| *** fnaval has joined #openstack-lbaas | 14:48 | |
| *** reedip_out_of_ar is now known as reedip_ | 14:52 | |
| *** kobis has joined #openstack-lbaas | 14:59 | |
| *** kbyrne has quit IRC | 15:06 | |
| *** kbyrne has joined #openstack-lbaas | 15:07 | |
| *** rcernin has quit IRC | 15:28 | |
| *** blogan has joined #openstack-lbaas | 15:34 | |
| openstackgerrit | Santhosh Fernandes proposed openstack/octavia master: [WIP] This patch will emit provisioning status to be sync with neutron db https://review.openstack.org/478385 | 15:36 |
| *** yamamoto has quit IRC | 15:41 | |
| *** aojea has quit IRC | 15:51 | |
| *** diltram has quit IRC | 16:10 | |
| *** reedip_ has quit IRC | 16:11 | |
| *** diltram has joined #openstack-lbaas | 16:13 | |
| *** kobis has quit IRC | 16:16 | |
| *** reedip_ has joined #openstack-lbaas | 16:18 | |
| *** fnaval has quit IRC | 16:26 | |
| *** cpuga_ has quit IRC | 16:27 | |
| *** fnaval has joined #openstack-lbaas | 16:27 | |
| *** sshank has joined #openstack-lbaas | 16:27 | |
| *** diltram has quit IRC | 16:39 | |
| *** yamamoto has joined #openstack-lbaas | 16:41 | |
| *** diltram has joined #openstack-lbaas | 16:45 | |
| *** pcaruana has quit IRC | 16:48 | |
| *** diltram has quit IRC | 16:50 | |
| *** sshank has quit IRC | 16:50 | |
| *** sshank has joined #openstack-lbaas | 16:50 | |
| *** yamamoto has quit IRC | 16:53 | |
| *** diltram has joined #openstack-lbaas | 16:53 | |
| *** reedip_ has quit IRC | 16:56 | |
| *** tesseract has quit IRC | 16:58 | |
| *** aojea has joined #openstack-lbaas | 17:15 | |
| *** aojea has quit IRC | 17:20 | |
| *** aojea has joined #openstack-lbaas | 17:30 | |
| rm_work | johnsom: tried to do the rebase for https://review.openstack.org/#/c/478280/ last night and ran into some weirdness, so just went to bed | 17:30 |
| openstackgerrit | Adam Harwell proposed openstack/octavia master: Switch to use stable remote_client https://review.openstack.org/445003 | 17:31 |
| rm_work | ^^ rebase to see if that's passing yet | 17:32 |
| johnsom | Ok, want me to make an attempt or are you going to work on it? | 17:32 |
| rm_work | you make an attempt and see if you get the same thing I did? | 17:34 |
| rm_work | if so we can look at it | 17:34 |
| johnsom | ok | 17:34 |
| rm_work | pep8 failure that's just ODD | 17:34 |
| rm_work | oh actually maybe i see it, derp | 17:35 |
| rm_work | i was very tired last night, going to bed might have been the right call :P | 17:35 |
| johnsom | I am wasting time trying to figure out why one of our ancient graphviz diagrams is CRASHING graphviz dot which is hanging up our docs gate | 17:35 |
| rm_work | yep nm i'm dumb | 17:35 |
| rm_work | got it | 17:36 |
| johnsom | (requirements) michjohn@devstackpy27:/tmp$ dot v0.5-component-design.dot | 17:36 |
| johnsom | *** stack smashing detected ***: dot terminated | 17:36 |
| johnsom | Aborted (core dumped) | 17:36 |
| rm_work | i was just super tired when I did the rebase | 17:36 |
| johnsom | I kind of really hate graphviz, it's just broken in so many ways | 17:36 |
| rm_work | uhh let me look too | 17:36 |
| rm_work | that runs fine for me | 17:37 |
| johnsom | Yeah, pretty sure it's the ubuntu version | 17:38 |
| johnsom | It hasn't rendered in our gates for a while.... | 17:38 |
| rm_work | hmmmmmm | 17:38 |
| rm_work | oh johnsom i guess nevermind | 17:38 |
| rm_work | i do have weird problems again | 17:38 |
| rm_work | with the test_policy tests | 17:38 |
| rm_work | so yeah maybe try that rebase | 17:39 |
| rm_work | oh uhh | 17:42 |
| rm_work | i think i got it | 17:42 |
| rm_work | but i'm a little confused | 17:42 |
| rm_work | yeah nm i must have done something dumb in the rebase | 17:42 |
| rm_work | i got it | 17:42 |
| rm_work | again | 17:42 |
| openstackgerrit | Adam Harwell proposed openstack/octavia master: Don't reload the policy for every API request. https://review.openstack.org/478280 | 17:43 |
| rm_work | yeah you're right this is super hacky having to do this policy rules reset on every test | 17:44 |
| rm_work | hold on | 17:44 |
| rm_work | k | 17:47 |
| openstackgerrit | Adam Harwell proposed openstack/octavia master: Don't reload the policy for every API request. https://review.openstack.org/478280 | 17:47 |
| *** SumitNaiksatam has joined #openstack-lbaas | 17:51 | |
| *** aojea_ has joined #openstack-lbaas | 18:00 | |
| *** aojea has quit IRC | 18:02 | |
| *** aojea_ has quit IRC | 18:10 | |
| *** aojea has joined #openstack-lbaas | 18:11 | |
| rm_work | johnsom: umm, on the consumer side of the worker queue, in endpoint.py, all of the actions have "context" in their signature, unused | 18:11 |
| rm_work | do we ACTUALLY have the context on this side of the queue? | 18:11 |
| rm_work | or is that garbage? | 18:11 |
| johnsom | This is passing context over to the handler? | 18:13 |
| johnsom | There was this patch from Brandon: https://review.openstack.org/#/c/408315/ | 18:14 |
| johnsom | I don't know why we would do that. | 18:15 |
| johnsom | It seems like TMI IMO | 18:15 |
| *** aojea has quit IRC | 18:15 | |
| openstackgerrit | Merged openstack/octavia master: Update api-ref for openstackdocstheme 1.11 https://review.openstack.org/478806 | 18:41 |
| *** SumitNaiksatam has quit IRC | 18:56 | |
| *** SumitNaiksatam has joined #openstack-lbaas | 18:58 | |
| *** aojea has joined #openstack-lbaas | 19:05 | |
| *** SumitNaiksatam has quit IRC | 19:15 | |
| *** SumitNaiksatam has joined #openstack-lbaas | 19:19 | |
| *** sshank has quit IRC | 19:23 | |
| *** sshank has joined #openstack-lbaas | 19:49 | |
| *** aojea has quit IRC | 20:01 | |
| *** eandersson has quit IRC | 20:10 | |
| *** aojea has joined #openstack-lbaas | 20:11 | |
| *** SumitNaiksatam has quit IRC | 20:27 | |
| rm_work | well | 20:32 |
| johnsom | Yes? | 20:34 |
| *** sshank has quit IRC | 20:56 | |
| rm_work | right now i'm looking at logging the request-id on the worker side | 20:59 |
| rm_work | which is in the context | 20:59 |
| rm_work | context.global_id | 20:59 |
| rm_work | so we can trace back worker operations to the request that triggered them | 20:59 |
| rm_work | so we have a full end-to-end picture | 20:59 |
| rm_work | sorry got distracted by a really nasty failover bug | 20:59 |
| johnsom | Yeah, we need to figure out the request-id stuff. It should be passed to the other services too. Just not sure how to do it, need to research | 21:00 |
| rm_work | if we can't delete the VM everything seems to explode | 21:00 |
| rm_work | not sure if it's just a bug in my driver or not | 21:00 |
| johnsom | https://bugs.launchpad.net/octavia/+bug/1694861 | 21:00 |
| openstack | Launchpad bug 1694861 in octavia "Octavia should support global request IDs" [Wishlist,New] | 21:00 |
| johnsom | FYI.... | 21:00 |
| rm_work | k | 21:01 |
| rm_work | kk | 21:01 |
| johnsom | There is a link in there to a spec with some info | 21:01 |
| rm_work | yeah k | 21:01 |
| johnsom | As for the delete thing, it should go to "ERROR" right? | 21:02 |
| rm_work | err well | 21:03 |
| rm_work | yeah so | 21:03 |
| rm_work | it just STALLS the failover flow | 21:03 |
| rm_work | so if a hypervisor goes down | 21:03 |
| rm_work | (this is what happened) | 21:03 |
| rm_work | nova fails to delete the VM on it | 21:03 |
| rm_work | the whole failover flow stalls | 21:04 |
| rm_work | exactly not what we want to have happen | 21:04 |
| xgerman_ | sounds like a nova problem ? | 21:04 |
| johnsom | Oh, hmmm, like the nova client is blocking? | 21:04 |
| rm_work | like the nova client says "nope failed" | 21:04 |
| rm_work | I think | 21:04 |
| xgerman_ | so we error out instead of retrying? | 21:04 |
| xgerman_ | skipping | 21:05 |
| xgerman_ | but in failover we bring up the vm before deleting so confused | 21:05 |
| rm_work | do we? | 21:05 |
| rm_work | looking | 21:05 |
| rm_work | maybe this is my own flow's fault | 21:05 |
| johnsom | No, we don't bring up a new vm first, that is in a patch that isn't yet approved. | 21:07 |
| johnsom | We follow a standard amp build in master code, spare or boot one | 21:07 |
| johnsom | Yeah, so looking at the flow, the ComputeDelete task is basically a pass through to nova client. | 21:08 |
| johnsom | I mean it's like the fourth step in, with others just taking care of locking the amp from health monitor and other changes. | 21:09 |
| johnsom | Though the step right after nova delete is a wait for neutron to detach a port... Maybe CONF.networking.port_detach_timeout is too high? | 21:12 |
| johnsom | or CONF.networking.retry_interval | 21:13 |
| xgerman_ | mmh, it’s tough for us to simulate a hypervisor blow up in devstack | 21:14 |
| rm_work | yeah | 21:14 |
| rm_work | so i think what happened then | 21:14 |
| rm_work | is that the nova call completed, stuck the server in deleting | 21:14 |
| xgerman_ | yep, and we didn’t get the port detached | 21:14 |
| rm_work | moved on to detaching the port | 21:15 |
| rm_work | THAT relies on the HV actually being up | 21:15 |
| rm_work | and that died | 21:15 |
| rm_work | yeah | 21:15 |
| johnsom | Yeah, we need that port or we lose the VIP ip address | 21:15 |
| xgerman_ | +1 | 21:15 |
| johnsom | https://github.com/openstack/octavia/blob/master/octavia/network/drivers/neutron/allowed_address_pairs.py#L546 | 21:16 |
| johnsom | Just for reference, that is the wait_for_port_detach | 21:16 |
| rm_work | yeah i have my own | 21:17 |
| rm_work | hmm | 21:17 |
| johnsom | Maybe we should make that more aggressive and ask the port to be detached | 21:17 |
| rm_work | I think maybe in my driver | 21:20 |
| rm_work | i will just have it ... not wait for the port to detach | 21:20 |
| rm_work | because I don't need that port anyway | 21:20 |
| rm_work | I believe | 21:20 |
| rm_work | johnsom: err in the failover flow, do we even need to do MarkAmphoraHealthBusy ??? | 21:28 |
| rm_work | shouldn't that already be set? | 21:28 |
| rm_work | <_< | 21:29 |
| johnsom | We can't be sure how this flow is triggerd | 21:29 |
| *** diltram has quit IRC | 21:29 | |
| rm_work | i think that's set atomically when the healthmonitor process picks it up? | 21:29 |
| *** aojea_ has joined #openstack-lbaas | 21:29 | |
| rm_work | ah | 21:29 |
| rm_work | hmm k | 21:29 |
| johnsom | I.e. the failover aPI | 21:29 |
| openstackgerrit | Adam Harwell proposed openstack/octavia master: WIP: Floating IP Network Driver (spans L3s) https://review.openstack.org/435612 | 21:31 |
| *** aojea has quit IRC | 21:31 | |
| openstackgerrit | Adam Harwell proposed openstack/octavia master: WIP: Floating IP Network Driver (spans L3s) https://review.openstack.org/435612 | 21:31 |
| *** diltram has joined #openstack-lbaas | 21:35 | |
| *** fnaval has quit IRC | 21:38 | |
| *** aojea_ has quit IRC | 21:49 | |
| *** diltram has quit IRC | 21:50 | |
| *** diltram has joined #openstack-lbaas | 21:57 | |
| *** aojea has joined #openstack-lbaas | 22:01 | |
| *** diltram has quit IRC | 22:02 | |
| openstackgerrit | Merged openstack/octavia master: Don't reload the policy for every API request. https://review.openstack.org/478280 | 22:04 |
| *** aojea has quit IRC | 22:05 | |
| *** diltram has joined #openstack-lbaas | 22:06 | |
| *** sshank has joined #openstack-lbaas | 22:08 | |
| rm_work | is this not a real thing? https://review.openstack.org/#/c/445003/ | 22:12 |
| rm_work | i feel like this is just a troll commit | 22:13 |
| johnsom | Well, at a minimum it needs more work | 22:14 |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Update Octavia docs for documentation migration https://review.openstack.org/478947 | 22:36 |
| johnsom | Very WIP ^^^ | 22:37 |
| rm_work | what are they migrating? | 22:50 |
| johnsom | Basically everything | 22:50 |
| johnsom | There will be no real "manuals" project anymore, just index pages | 22:50 |
| johnsom | They are doing what we decided to do a long time ago, which is move everything into the repo | 22:51 |
| johnsom | There is a link to the spec in the patch if you are bored, otherwise I am working through it. | 22:52 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!