| rm_work | johnsom: so i am 90% sure i'm going to just take tomorrow off <_< | 00:02 |
|---|---|---|
| johnsom | Ok, have a good one! | 00:02 |
| rm_work | so you should force someone else to make quick changes if L7Policies is close! :P | 00:03 |
| johnsom | Haha, ok | 00:03 |
| rm_work | i did the rules changes... HM needs a little more work | 00:03 |
| rm_work | then I'll rebase sorting/paging again | 00:04 |
| rm_work | TBH I should probably test rules better locally though | 00:04 |
| rm_work | i've been in "churn through normalization as fast as possible" mode | 00:04 |
| rm_work | basically getting them behaving all the same | 00:04 |
| rm_work | might have missed some details :/ | 00:04 |
| *** reedip has joined #openstack-lbaas | 00:10 | |
| *** catintheroof has joined #openstack-lbaas | 00:25 | |
| openstackgerrit | German Eichberger proposed openstack/neutron-lbaas master: Octavia Proxy Plugin https://review.openstack.org/418530 | 00:37 |
| openstackgerrit | Alexander Amerine proposed openstack/octavia master: blueprint LBaaS Alternative Health Monitoring Port https://review.openstack.org/453001 | 00:52 |
| *** catintheroof has quit IRC | 01:02 | |
| openstackgerrit | Jason Niesz proposed openstack/octavia master: blueprint: l3-active-active https://review.openstack.org/453005 | 01:46 |
| *** reedip has quit IRC | 01:47 | |
| *** harlowja has quit IRC | 02:06 | |
| *** aojea has joined #openstack-lbaas | 02:21 | |
| *** aojea has quit IRC | 02:25 | |
| openstackgerrit | Alexander Amerine proposed openstack/octavia master: blueprint LBaaS Alternative Health Monitoring Port https://review.openstack.org/453001 | 02:36 |
| openstackgerrit | Alexander Amerine proposed openstack/octavia master: blueprint LBaaS Alternative Health Monitoring Port https://review.openstack.org/453001 | 02:37 |
| *** reedip has joined #openstack-lbaas | 02:47 | |
| *** harlowja has joined #openstack-lbaas | 02:50 | |
| *** sanfern has joined #openstack-lbaas | 02:53 | |
| *** reedip has quit IRC | 02:53 | |
| *** reedip has joined #openstack-lbaas | 03:02 | |
| *** fnaval has joined #openstack-lbaas | 03:06 | |
| *** links has joined #openstack-lbaas | 03:08 | |
| reedip | o\ /o | 03:32 |
| *** amotoki has joined #openstack-lbaas | 03:34 | |
| *** csomerville has quit IRC | 03:45 | |
| openstackgerrit | Alexander Amerine proposed openstack/octavia master: blueprint LBaaS Alternative Health Monitoring Port https://review.openstack.org/453001 | 03:46 |
| *** amotoki has quit IRC | 03:50 | |
| *** harlowja has quit IRC | 03:56 | |
| *** amotoki has joined #openstack-lbaas | 04:01 | |
| *** armax has joined #openstack-lbaas | 04:05 | |
| *** harlowja has joined #openstack-lbaas | 04:07 | |
| *** amotoki has quit IRC | 04:19 | |
| *** amotoki has joined #openstack-lbaas | 04:28 | |
| openstackgerrit | Ankur proposed openstack/octavia master: Add loadbalancer status tree call https://review.openstack.org/433982 | 04:29 |
| openstackgerrit | Ankur proposed openstack/octavia master: Add loadbalancer status tree call https://review.openstack.org/433982 | 04:29 |
| *** harlowja has quit IRC | 04:33 | |
| *** rcernin has joined #openstack-lbaas | 04:45 | |
| *** rcernin has quit IRC | 05:09 | |
| *** rcernin has joined #openstack-lbaas | 05:11 | |
| *** harlowja has joined #openstack-lbaas | 05:11 | |
| *** ankur-gupta-f4 has quit IRC | 05:14 | |
| *** fnaval has quit IRC | 05:15 | |
| *** amotoki has quit IRC | 05:26 | |
| *** fnaval has joined #openstack-lbaas | 05:29 | |
| *** amotoki has joined #openstack-lbaas | 05:30 | |
| *** aojea has joined #openstack-lbaas | 05:34 | |
| *** gcheresh_ has joined #openstack-lbaas | 05:37 | |
| *** aojea has quit IRC | 05:41 | |
| *** madgoat has joined #openstack-lbaas | 05:57 | |
| *** madgoat has left #openstack-lbaas | 05:57 | |
| *** harlowja has quit IRC | 06:02 | |
| *** voelzmo has joined #openstack-lbaas | 06:19 | |
| *** voelzmo has quit IRC | 06:24 | |
| *** tesseract has joined #openstack-lbaas | 06:26 | |
| *** voelzmo has joined #openstack-lbaas | 06:27 | |
| *** fnaval has quit IRC | 06:39 | |
| *** krypto has joined #openstack-lbaas | 06:41 | |
| *** sputnik13 has quit IRC | 06:50 | |
| *** sputnik13 has joined #openstack-lbaas | 06:54 | |
| *** sputnik13 has quit IRC | 06:54 | |
| *** sputnik13 has joined #openstack-lbaas | 06:56 | |
| *** amotoki has quit IRC | 07:00 | |
| *** amotoki has joined #openstack-lbaas | 07:03 | |
| *** ankur-gupta-f4 has joined #openstack-lbaas | 07:16 | |
| *** pcaruana has joined #openstack-lbaas | 07:19 | |
| *** aojea has joined #openstack-lbaas | 07:24 | |
| *** aojea_ has joined #openstack-lbaas | 07:27 | |
| *** aojea has quit IRC | 07:30 | |
| *** amotoki has quit IRC | 07:35 | |
| *** amotoki has joined #openstack-lbaas | 07:36 | |
| *** armax has quit IRC | 07:44 | |
| *** ianychoi has quit IRC | 07:54 | |
| *** ianychoi has joined #openstack-lbaas | 08:17 | |
| *** reedip has quit IRC | 08:20 | |
| *** reedip has joined #openstack-lbaas | 08:33 | |
| openstackgerrit | Nir Magnezi proposed openstack/octavia master: (WIP) Auto detect haproxy user_group https://review.openstack.org/429398 | 08:48 |
| openstackgerrit | Nir Magnezi proposed openstack/octavia master: (WIP) Auto detect haproxy user_group https://review.openstack.org/429398 | 08:49 |
| *** reedip has quit IRC | 08:50 | |
| openstackgerrit | Nir Magnezi proposed openstack/octavia master: (WIP) Auto detect haproxy user_group https://review.openstack.org/429398 | 09:29 |
| *** reedip has joined #openstack-lbaas | 09:34 | |
| *** belharar has joined #openstack-lbaas | 10:01 | |
| reedip | rm_work, johnsom, xgerman : hi | 10:02 |
| reedip | rm_work, xgerman, johnsom: Though LBaaS is not a Neutron Project, it still links up with networking( in turn neutron ). The following proposal is requested in Neutron, and I think Octavia can also be included in it. Let me know your thoughts #link https://review.openstack.org/#/c/333993/ | 10:04 |
| *** gcheresh_ has quit IRC | 10:15 | |
| *** gcheresh_ has joined #openstack-lbaas | 10:15 | |
| *** belharar has quit IRC | 10:36 | |
| *** krypto has quit IRC | 10:49 | |
| *** sanfern has quit IRC | 10:53 | |
| *** krypto has joined #openstack-lbaas | 10:59 | |
| *** belharar has joined #openstack-lbaas | 11:18 | |
| *** gcheresh_ has quit IRC | 11:22 | |
| *** gcheresh_ has joined #openstack-lbaas | 11:23 | |
| *** voelzmo has quit IRC | 11:58 | |
| *** voelzmo has joined #openstack-lbaas | 12:01 | |
| *** catintheroof has joined #openstack-lbaas | 12:05 | |
| *** chlong has joined #openstack-lbaas | 12:10 | |
| *** krypto has quit IRC | 12:19 | |
| *** voelzmo has quit IRC | 12:41 | |
| *** voelzmo has joined #openstack-lbaas | 12:43 | |
| *** voelzmo has quit IRC | 12:46 | |
| *** voelzmo has joined #openstack-lbaas | 12:49 | |
| *** krypto has joined #openstack-lbaas | 13:01 | |
| openstackgerrit | Nir Magnezi proposed openstack/octavia master: (WIP) Auto detect haproxy user_group https://review.openstack.org/429398 | 13:03 |
| *** links has quit IRC | 13:04 | |
| *** amotoki has quit IRC | 13:15 | |
| reedip | - - | 13:26 |
| nmagnezi | o/ | 13:39 |
| *** amotoki has joined #openstack-lbaas | 13:50 | |
| *** amotoki has quit IRC | 13:59 | |
| *** armax has joined #openstack-lbaas | 14:11 | |
| *** amotoki has joined #openstack-lbaas | 14:17 | |
| *** fnaval has joined #openstack-lbaas | 14:50 | |
| *** belharar_ has joined #openstack-lbaas | 14:50 | |
| diltram | nmagnezi: hey | 14:51 |
| *** belharar_ has quit IRC | 14:51 | |
| *** armax has quit IRC | 14:51 | |
| diltram | you made this patch too drastical | 14:51 |
| nmagnezi | diltram, hi | 14:51 |
| diltram | we need to deprecate those config options | 14:51 |
| diltram | and not remove them | 14:51 |
| *** belharar has quit IRC | 14:52 | |
| diltram | and just if image supports setting them it should be provided but if image is prebaked with this config it's gonna be omitted | 14:52 |
| diltram | we still need to provide backward compatibility | 14:53 |
| *** belharar has joined #openstack-lbaas | 14:53 | |
| *** armax has joined #openstack-lbaas | 14:56 | |
| nmagnezi | diltram, okay, so how will you know if the image supports it or not? | 14:59 |
| nmagnezi | diltram, from the controller side | 14:59 |
| *** belharar_ has joined #openstack-lbaas | 15:01 | |
| diltram | you can't test this from controller side | 15:01 |
| nmagnezi | i know | 15:01 |
| nmagnezi | i'm trying to understand your intention :) | 15:01 |
| diltram | :P | 15:01 |
| diltram | we need to made some changes in amphora | 15:02 |
| diltram | make* | 15:02 |
| *** belharar has quit IRC | 15:05 | |
| nmagnezi | this patch follows the following logic: the amp agent uses a user group which is pre-baked in the image. and the controller no longer render that param in amphora-agent.conf. for cases if new agent with old controllers the agent will ignore user_group from the config file by removing it somehow (this specific part is WIP) | 15:05 |
| nmagnezi | diltram, which part should be changed? | 15:06 |
| diltram | nmagnezi: trying to find :P | 15:06 |
| nmagnezi | O_o | 15:06 |
| diltram | but what about situation that there is new controller, old amphora? | 15:07 |
| diltram | it more possible than opposite | 15:07 |
| diltram | at first you're updating controll plane than you're working on data plane | 15:08 |
| nmagnezi | valid point | 15:08 |
| diltram | the same is with use_upstart | 15:08 |
| diltram | we're still supporting this | 15:08 |
| diltram | but amphora is automatically detecting this setting | 15:08 |
| diltram | problem is that we're genereting config on controller and we're not doing anything with it on amphora | 15:09 |
| nmagnezi | that's actually okay | 15:09 |
| nmagnezi | in case it is auto detected by the amp agent it should be okay to ignore what is coming from the agent | 15:09 |
| nmagnezi | there is a different problem here | 15:09 |
| nmagnezi | with new controller, old agent | 15:10 |
| nmagnezi | i need to make sure and new controllers still send that config option, just to make the amp agent to ignore it and auto detect | 15:10 |
| nmagnezi | so old agents will still work | 15:10 |
| nmagnezi | the assumption is that if the agent is "smart" enough to auto-detect, it does not need the controller to tell him the user_group | 15:11 |
| nmagnezi | we just need to make sure old agents won't break | 15:11 |
| diltram | nmagnezi: https://github.com/openstack/octavia/blob/master/octavia/amphorae/backends/agent/api_server/keepalived.py#L40 | 15:12 |
| *** gcheresh_ has quit IRC | 15:12 | |
| diltram | here you can strip this param | 15:13 |
| diltram | yeah, for the next two releases we need to support this param | 15:13 |
| diltram | even if new amphora will not use it | 15:13 |
| *** armax has quit IRC | 15:13 | |
| nmagnezi | why strip it in keepalived? | 15:13 |
| diltram | ok not even here | 15:14 |
| diltram | there is just keepalived config | 15:14 |
| diltram | but yeah | 15:14 |
| diltram | the whole controller code stays the same | 15:14 |
| nmagnezi | diltram, here: https://github.com/openstack/octavia/blob/master/octavia/amphorae/backends/agent/api_server/listener.py#L112-L114 | 15:14 |
| diltram | exactly :P | 15:15 |
| nmagnezi | diltram, aside from the deprecation note you are correct about the controller | 15:15 |
| nmagnezi | :) | 15:15 |
| diltram | great :) | 15:15 |
| nmagnezi | thanks for the feedback diltram :) | 15:15 |
| diltram | that agreed | 15:15 |
| diltram | sure, np :) | 15:15 |
| nmagnezi | diltram, heading home, catch you later | 15:16 |
| openstackgerrit | Shashank Kumar Shankar proposed openstack/octavia master: Run Octavia API in a WSGI server https://review.openstack.org/440934 | 15:16 |
| diltram | cu | 15:17 |
| *** bzhao has quit IRC | 15:23 | |
| *** bzhao has joined #openstack-lbaas | 15:24 | |
| *** belharar_ has quit IRC | 15:33 | |
| *** aojea_ has quit IRC | 15:49 | |
| diltram | johnsom: ping | 15:51 |
| openstackgerrit | Lubosz Kosnik (diltram) proposed openstack/octavia master: Update devstack to properly configure nova-lxd img https://review.openstack.org/453237 | 15:52 |
| *** pcaruana has quit IRC | 16:01 | |
| *** voelzmo has quit IRC | 16:03 | |
| johnsom | diltram Hi | 16:06 |
| diltram | hello | 16:07 |
| johnsom | You rang? | 16:08 |
| diltram | johnsom: can you take a look into this: https://review.openstack.org/#/c/453237/1/devstack/plugin.sh@67 | 16:08 |
| diltram | I need your advice | 16:08 |
| diltram | should we do this like I made it | 16:08 |
| diltram | or I should just move this into dib element? | 16:08 |
| diltram | I'm talking about this directories | 16:09 |
| diltram | the rest can be changed after removing those directories | 16:09 |
| johnsom | Yeah, I would lean towards doing it in an element | 16:10 |
| diltram | ok, had the same feeling | 16:10 |
| johnsom | Yeah, in the same patch you create the element we can create a lxd type in the script that gzips it\ | 16:11 |
| diltram | sure | 16:12 |
| diltram | gonna change it | 16:12 |
| diltram | any thoughts about the element name? | 16:13 |
| johnsom | lxd-config ? Something with LXD I guess | 16:14 |
| johnsom | Unless of course docker has the same issue, but I doubt it | 16:14 |
| *** cody-somerville has joined #openstack-lbaas | 16:17 | |
| *** cody-somerville has quit IRC | 16:17 | |
| *** cody-somerville has joined #openstack-lbaas | 16:17 | |
| diltram | ok | 16:17 |
| diltram | thx johnsom | 16:17 |
| *** ipsecguy_ has joined #openstack-lbaas | 16:24 | |
| *** ipsecguy has quit IRC | 16:26 | |
| *** krypto has quit IRC | 16:48 | |
| *** krypto has joined #openstack-lbaas | 16:49 | |
| *** tesseract has quit IRC | 17:04 | |
| *** armax has joined #openstack-lbaas | 17:08 | |
| *** armax has quit IRC | 17:18 | |
| *** krypto has quit IRC | 17:20 | |
| *** krypto has joined #openstack-lbaas | 17:21 | |
| *** krypto has quit IRC | 17:21 | |
| *** krypto has joined #openstack-lbaas | 17:21 | |
| openstackgerrit | Lubosz Kosnik (diltram) proposed openstack/octavia master: Update devstack to properly configure nova-lxd img https://review.openstack.org/453237 | 17:21 |
| *** armax has joined #openstack-lbaas | 17:27 | |
| *** voelzmo has joined #openstack-lbaas | 17:30 | |
| *** voelzmo has quit IRC | 17:36 | |
| *** voelzmo has joined #openstack-lbaas | 17:40 | |
| openstackgerrit | Merged openstack/octavia master: Fix pool validation exception typo https://review.openstack.org/452899 | 17:56 |
| *** gcheresh_ has joined #openstack-lbaas | 18:03 | |
| *** mdavidson has joined #openstack-lbaas | 18:11 | |
| *** armax has quit IRC | 18:14 | |
| openstackgerrit | Lubosz Kosnik (diltram) proposed openstack/octavia master: Update devstack to properly configure nova-lxd img https://review.openstack.org/453237 | 18:29 |
| *** gcheresh_ has quit IRC | 18:29 | |
| openstackgerrit | Sindhu Devale proposed openstack/octavia master: Octavia v2 API needs to return the pools object https://review.openstack.org/453288 | 18:31 |
| *** amotoki has quit IRC | 18:34 | |
| *** voelzmo has quit IRC | 18:35 | |
| openstackgerrit | German Eichberger proposed openstack/octavia master: Initial version of the operator guide https://review.openstack.org/448759 | 18:37 |
| *** krypto has quit IRC | 18:52 | |
| *** gcheresh_ has joined #openstack-lbaas | 19:01 | |
| openstackgerrit | Merged openstack/neutron-lbaas master: Explicitly list neutron api extensions for tests https://review.openstack.org/451612 | 19:14 |
| *** voelzmo has joined #openstack-lbaas | 19:17 | |
| *** voelzmo has quit IRC | 19:19 | |
| *** aojea has joined #openstack-lbaas | 19:24 | |
| *** harlowja has joined #openstack-lbaas | 19:24 | |
| *** voelzmo has joined #openstack-lbaas | 19:27 | |
| openstackgerrit | Sindhu Devale proposed openstack/octavia master: Introduce Octavia v2 API for Health Monitor https://review.openstack.org/406256 | 19:28 |
| *** voelzmo has quit IRC | 19:46 | |
| *** kbyrne has quit IRC | 19:46 | |
| *** ducttape_ has joined #openstack-lbaas | 19:48 | |
| *** kbyrne has joined #openstack-lbaas | 19:50 | |
| nmagnezi | o/ | 19:56 |
| xgerman | o/ | 20:03 |
| nmagnezi | xgerman, re: https://review.openstack.org/#/c/452213/ | 20:15 |
| xgerman | yep | 20:15 |
| nmagnezi | xgerman, if I want to test this, 2 compute nodes should be sufficient, right? | 20:15 |
| xgerman | yes | 20:15 |
| xgerman | I tested it with the new soft-affinity feature | 20:15 |
| xgerman | on one node | 20:15 |
| xgerman | ;-) | 20:16 |
| *** voelzmo has joined #openstack-lbaas | 20:18 | |
| nmagnezi | xgerman, how? you spawned two n-cpu? | 20:21 |
| xgerman | no, soft-anti-affinity let’s you schedule tow lb on one host if no other host can be found but retain the servergroup | 20:22 |
| *** voelzmo has quit IRC | 20:23 | |
| *** ducttape_ has quit IRC | 20:24 | |
| *** armax has joined #openstack-lbaas | 20:30 | |
| xgerman | neat: http://blog.spinhirne.com/2016/09/the-ovn-load-balancer.html | 20:33 |
| *** ducttape_ has joined #openstack-lbaas | 20:34 | |
| johnsom | Hi Nir | 20:38 |
| johnsom | diltram Image file /opt/stack/new/octavia/diskimage-create/amphora-x64-haproxy.tgz.qcow2 created... | 20:38 |
| diltram | johnsom: sigh | 20:38 |
| diltram | checking | 20:38 |
| johnsom | I got a chuckle.... | 20:38 |
| nmagnezi | Hi Michael | 20:40 |
| johnsom | Well, my API testing devstack is too scrambled with the DB migration changes. Restacking.... | 20:51 |
| *** csomerville has joined #openstack-lbaas | 20:54 | |
| *** rcernin has quit IRC | 20:55 | |
| *** cody-somerville has quit IRC | 20:57 | |
| *** gcheresh_ has quit IRC | 21:02 | |
| xgerman | so far so good in my tests | 21:06 |
| johnsom | I posted two comments to the l7policy patch, but needed to restack for the l7rules patch. The migrations changed (moved between patches) after I had already run them, so it was easier to just restack with the latest patches. | 21:08 |
| xgerman | that’s what I usually do | 21:09 |
| johnsom | I usually just checkout the latest and redeploy it. | 21:10 |
| *** aojea has quit IRC | 21:11 | |
| *** aojea has joined #openstack-lbaas | 21:15 | |
| *** aojea_ has joined #openstack-lbaas | 21:16 | |
| *** aojea has quit IRC | 21:16 | |
| xgerman | mmh, policies ook happy rules not sure what the json is to create them | 21:21 |
| xgerman | https://www.irccloud.com/pastebin/Brzsvd7C/ | 21:23 |
| xgerman | Huh? | 21:25 |
| xgerman | https://usercontent.irccloud-cdn.com/file/SYgZZQbF/Screen%20Shot%202017-04-04%20at%205.25.04%20PM.png | 21:25 |
| xgerman | johnsom you get that, too? | 21:25 |
| xgerman | mmh, then it shows up | 21:26 |
| xgerman | wonder if that is the right behavior - return what is set and not what we want to set? | 21:26 |
| johnsom | The issue where the response doesn't show the update content, but a followup get will? | 21:26 |
| xgerman | yep | 21:26 |
| johnsom | Yeah, I opened a bug for that. | 21:26 |
| xgerman | k | 21:27 |
| johnsom | It's something we need to decide. | 21:27 |
| johnsom | It's a side effect of the asynch API | 21:27 |
| xgerman | well, all the others return what we want to set it to | 21:27 |
| johnsom | Not on the octavia V2 API, all of them return the "old" info | 21:27 |
| xgerman | mmh, then I must have confused it with POST | 21:28 |
| xgerman | which means POST and PUT differ which is odd but not unheard of | 21:28 |
| xgerman | we can chat at tomorrow’s meeting and vote ;-) | 21:29 |
| johnsom | Well, it might just be inconsistent across the board.... | 21:29 |
| johnsom | We need to make a call on it. | 21:29 |
| xgerman | yep | 21:29 |
| *** aojea_ has quit IRC | 21:57 | |
| rm_work | here | 22:12 |
| rm_work | ah i see i got some sort of clipped line on the policies | 22:12 |
| rm_work | was that it though, besides reordering not working? :/ | 22:13 |
| johnsom | Those two so far. I had to restack due to the DB migration changes, so poking at rules now | 22:22 |
| openstackgerrit | Adam Harwell proposed openstack/octavia master: Align Octavia API to n-lbaasv2 for L7Rules https://review.openstack.org/406336 | 22:22 |
| openstackgerrit | Adam Harwell proposed openstack/octavia master: Align Octavia API to n-lbaasv2 for L7Policy https://review.openstack.org/406328 | 22:22 |
| rm_work | fixed the test in Rules (test change only) | 22:22 |
| rm_work | and the O -> OFFLINE typo in policies | 22:23 |
| johnsom | Ok | 22:23 |
| rm_work | i am really not understanding why the policies don't reorder | 22:23 |
| rm_work | gonna look again today with fresh eyes | 22:23 |
| johnsom | Yeah, haven't had a chance to poke at that today | 22:23 |
| rm_work | I might vote to open a bug for it | 22:24 |
| johnsom | Yeah, that is fair. I found a bug in LB create as well, I opened a bug for that. Specifying tenant_id doesn't work, throws a validation error | 22:24 |
| rm_work | uhh | 22:28 |
| johnsom | I hate that L7 does null and the older stuff does "" for empty fields. | 22:28 |
| rm_work | oh | 22:28 |
| rm_work | ll | 22:28 |
| rm_work | *lol | 22:28 |
| rm_work | that was the first one to go in | 22:28 |
| rm_work | I think we hadn't ironed out support for args yet | 22:28 |
| rm_work | johnsom: in neutron too? | 22:28 |
| johnsom | Yeah, I have been making sure it matches, bugs and all | 22:28 |
| johnsom | For all of these I'm running neutron-lbaas first, then Octavia v2 and comparing output: | 22:29 |
| johnsom | https://www.irccloud.com/pastebin/aD88S4i8/ | 22:29 |
| rm_work | k | 22:30 |
| rm_work | ... trying really hard not to throw the fix for the tenant_id thing into my next l7policy patch | 22:32 |
| johnsom | Well the bug is open | 22:32 |
| johnsom | I think sindhu is also working on fixing some of those bugs | 22:33 |
| johnsom | That said, don't let me stop you from fixing "stuff" | 22:35 |
| *** fnaval has quit IRC | 22:42 | |
| rm_work | the handling of this reorder stuff is so hacky... i have a theory but I think i need a devstack up to test it with debugging | 22:44 |
| johnsom | Ok, also admin-state-up: faluse on l7rule update doesn't do anything. | 22:45 |
| johnsom | It changes in the DB, but not the config in the amp | 22:45 |
| rm_work | hmmmmm | 22:46 |
| rm_work | yeah k | 22:46 |
| rm_work | I prolly know why | 22:46 |
| johnsom | Ok, a quick read over and I'm done testing out l7rules | 22:48 |
| rm_work | i just don't understand the difference here between v1 and v2 | 22:48 |
| rm_work | for policy order | 22:48 |
| johnsom | Functionally that was the only thing I found. | 22:48 |
| *** catintheroof has quit IRC | 22:57 | |
| johnsom | Yeah, ok, that is the only issue I have with L7rules, otherwise it is good to go in my book | 22:59 |
| johnsom | Is there something I can help investigate or should I do some other reviews? | 23:00 |
| rm_work | i'm about to test a theory with policies | 23:04 |
| rm_work | if you can test it... prolly post as soon as these tests finish | 23:13 |
| johnsom | Yep, NP | 23:13 |
| openstackgerrit | Adam Harwell proposed openstack/octavia master: Align Octavia API to n-lbaasv2 for L7Policy https://review.openstack.org/406328 | 23:14 |
| rm_work | about to post the fix for admin_state on rules | 23:22 |
| rm_work | trying to find the tests... | 23:22 |
| rm_work | so i can add one | 23:22 |
| rm_work | not sure what tests the jinja template | 23:22 |
| rm_work | to make sure it generated properly | 23:22 |
| johnsom | Hmmm, position still isn't changing for me | 23:26 |
| rm_work | >_< | 23:36 |
| rm_work | did i break v` | 23:36 |
| rm_work | *v1 | 23:36 |
| johnsom | Don't know. I'm looking at the code now to see what is going on | 23:36 |
| *** csomerville has quit IRC | 23:37 | |
| johnsom | Ok, that code is screwy | 23:37 |
| johnsom | There is no use of that inner session | 23:38 |
| johnsom | Yeah, so there is literally nothing actually setting the position value in the database. I'm still trying to figure out what the story is with this listener position code | 23:40 |
| johnsom | Ah, sqlalchemy "magic", hmmmm | 23:43 |
| johnsom | Still don't get the inner session | 23:43 |
| *** sticker has joined #openstack-lbaas | 23:46 | |
| *** ducttape_ has quit IRC | 23:46 | |
| *** ducttape_ has joined #openstack-lbaas | 23:46 | |
| *** ducttape_ has quit IRC | 23:48 | |
| *** ducttape_ has joined #openstack-lbaas | 23:48 | |
| rm_work | johnsom: i mean, keep in mind that this DOES work with v1, right??? >_> | 23:50 |
| *** _ducttape_ has joined #openstack-lbaas | 23:50 | |
| *** ducttape_ has quit IRC | 23:50 | |
| johnsom | I think you had me try that yes | 23:50 |
| rm_work | ugh this jinja stuff is a maze | 23:51 |
| rm_work | trying to do TDD but getting the tests to expect what I want is even confusing | 23:51 |
| *** yuanying_ has quit IRC | 23:53 | |
| johnsom | I am going to try v1 again. | 23:54 |
| rm_work | k | 23:56 |
| johnsom | v1 isn't working now | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
