| *** yamamoto has joined #openstack-lbaas | 00:00 | |
| *** armax has quit IRC | 00:17 | |
| *** armax has joined #openstack-lbaas | 00:34 | |
| *** njohnston has quit IRC | 00:47 | |
| rm_work | still need reviews on https://review.opendev.org/#/c/737111/ | 01:01 |
|---|---|---|
| rm_work | cgoncalves? :D | 01:02 |
| rm_work | err did not mean to put that twice | 01:09 |
| *** armax has quit IRC | 01:15 | |
| johnsom | I have the window open for that, but got distracted, by someone. | 01:45 |
| *** ZhuJoseph has joined #openstack-lbaas | 01:49 | |
| *** ZhuXiaoYu has quit IRC | 01:52 | |
| rm_work | hmmm | 01:54 |
| ZhuJoseph | Just a reminder, the spec https://review.opendev.org/#/c/723864 still need reviews | 01:54 |
| rm_work | i wonder who that could be | 01:54 |
| rm_work | surely not me! | 01:54 |
| *** spatel has joined #openstack-lbaas | 02:13 | |
| *** zhuxiaoyu_inspur has joined #openstack-lbaas | 02:23 | |
| *** ZhuJoseph has quit IRC | 02:26 | |
| rm_work | so did svinota just release pyroute2 and then go on vacation for a week? lol | 02:27 |
| *** spatel has quit IRC | 03:02 | |
| *** psachin has joined #openstack-lbaas | 03:31 | |
| *** rcernin has quit IRC | 04:03 | |
| *** rcernin has joined #openstack-lbaas | 04:24 | |
| *** gcheresh_ has joined #openstack-lbaas | 05:00 | |
| *** yamamoto has quit IRC | 05:29 | |
| *** yamamoto has joined #openstack-lbaas | 05:36 | |
| *** ZhuJoseph has joined #openstack-lbaas | 05:42 | |
| *** zhuxiaoyu_inspur has quit IRC | 05:45 | |
| *** vishalmanchanda has joined #openstack-lbaas | 06:09 | |
| *** ZhuJoseph has quit IRC | 06:11 | |
| *** ccamposr has quit IRC | 06:50 | |
| *** rcernin has quit IRC | 06:54 | |
| *** born2bake has joined #openstack-lbaas | 06:59 | |
| *** ataraday_ has joined #openstack-lbaas | 07:15 | |
| *** rcernin has joined #openstack-lbaas | 07:20 | |
| *** maciejjozefczyk has joined #openstack-lbaas | 07:20 | |
| *** mvorwerk has joined #openstack-lbaas | 07:23 | |
| *** rcernin has quit IRC | 07:25 | |
| openstackgerrit | Ann Taraday proposed openstack/octavia master: [AmphoraV2] Fix l7policy/l7rule quota decrement https://review.opendev.org/744701 | 07:34 |
| *** takamatsu has quit IRC | 08:02 | |
| *** sapd__x has joined #openstack-lbaas | 08:04 | |
| *** osmanlicilegi has quit IRC | 08:32 | |
| *** oklhost has quit IRC | 08:44 | |
| *** oklhost has joined #openstack-lbaas | 08:50 | |
| *** takamatsu has joined #openstack-lbaas | 08:58 | |
| *** takamatsu has quit IRC | 09:07 | |
| *** takamatsu has joined #openstack-lbaas | 09:07 | |
| *** osmanlicilegi has joined #openstack-lbaas | 09:10 | |
| *** takamatsu has quit IRC | 09:53 | |
| openstackgerrit | Merged openstack/octavia-tempest-plugin master: Add skip_if_not_implemented to the service client https://review.opendev.org/714003 | 10:12 |
| *** yamamoto has quit IRC | 10:20 | |
| *** tkajinam has quit IRC | 10:33 | |
| *** yamamoto has joined #openstack-lbaas | 10:36 | |
| openstackgerrit | Carlos Goncalves proposed openstack/octavia master: Fix listener tls_versions and tls_ciphers clearing https://review.opendev.org/745114 | 10:55 |
| *** yamamoto has quit IRC | 11:05 | |
| *** sapd__x has quit IRC | 11:06 | |
| *** takamatsu has joined #openstack-lbaas | 11:06 | |
| *** yamamoto has joined #openstack-lbaas | 11:12 | |
| openstackgerrit | Carlos Goncalves proposed openstack/python-octaviaclient master: Add TLS versions and ciphers unsets https://review.opendev.org/745118 | 11:34 |
| *** ccamposr has joined #openstack-lbaas | 11:48 | |
| *** ramishra has quit IRC | 12:04 | |
| *** rcernin has joined #openstack-lbaas | 12:08 | |
| *** yamamoto has quit IRC | 12:12 | |
| *** ramishra has joined #openstack-lbaas | 12:26 | |
| *** ccamposr is now known as ccamposr|lunch | 12:39 | |
| *** njohnston has joined #openstack-lbaas | 12:42 | |
| openstackgerrit | Merged openstack/octavia master: Add a periodic fast forward upgrade job https://review.opendev.org/732986 | 12:42 |
| *** yamamoto has joined #openstack-lbaas | 12:55 | |
| *** yamamoto has quit IRC | 13:07 | |
| *** TrevorV has joined #openstack-lbaas | 13:36 | |
| *** yamamoto has joined #openstack-lbaas | 13:39 | |
| *** rcernin has quit IRC | 13:40 | |
| *** ramishra has quit IRC | 13:40 | |
| *** yamamoto has quit IRC | 13:45 | |
| openstackgerrit | Carlos Goncalves proposed openstack/octavia master: Add ALPN support for TLS-terminated HTTPS LBs https://review.opendev.org/744520 | 13:59 |
| *** ataraday_ has quit IRC | 14:07 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia-tempest-plugin master: Adjust scenario tests for NotImplemented skip https://review.opendev.org/714004 | 14:12 |
| *** ramishra has joined #openstack-lbaas | 14:13 | |
| haleyb | johnsom: look at the failure in this patch, https://review.opendev.org/#/c/744730/ - 'insert-headers is not a valid option for a TCP protocol listener.' and i know i found a recent fix with that string | 14:19 |
| *** gcheresh_ has quit IRC | 14:21 | |
| johnsom | haleyb Ok, yeah, it was fixed here: https://review.opendev.org/744047 and this patch will update the scenario: https://review.opendev.org/714004 | 14:21 |
| johnsom | Not really sure why your scenario would fail on that but the runs on the patch itself didn't fail. | 14:23 |
| *** mvorwerk has quit IRC | 14:23 | |
| johnsom | Oh! I know, it's because the OVN hacks I am removing in tempest magically change the listener CRUD test to TCP from HTTP when it is the OVN provider | 14:23 |
| johnsom | This is basically the whole point of removing those | 14:25 |
| *** mvorwerk has joined #openstack-lbaas | 14:25 | |
| haleyb | so is there a bug where the OVN driver isn't setting something correctly? i guess i can always depend-on the tempest change and recheck to see | 14:26 |
| johnsom | It's the OVN hacks in the octavia tempest test that is switching the protocol in a test from HTTP, where header insertion is valid, to TCP where header insertion is not valid. The hack didn't take that into account. Thus why I am removing all of that. | 14:28 |
| *** mvorwerk has quit IRC | 14:32 | |
| *** armax has joined #openstack-lbaas | 14:35 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia-tempest-plugin master: Adjust scenario tests for NotImplemented skip https://review.opendev.org/714004 | 14:37 |
| *** psachin has quit IRC | 14:43 | |
| *** tow has joined #openstack-lbaas | 15:44 | |
| johnsom | haleyb Looks like you need to set this parameter in the OVN jobs: https://review.opendev.org/#/c/714004/29/zuul.d/jobs.yaml | 15:48 |
| johnsom | not_implemented_is_error: False | 15:48 |
| johnsom | It came in here: https://review.opendev.org/#/c/714003/11/octavia_tempest_plugin/config.py | 15:49 |
| *** ramishra has quit IRC | 15:53 | |
| openstackgerrit | Carlos Goncalves proposed openstack/octavia master: Fix listener tls_versions and tls_ciphers clearing https://review.opendev.org/745114 | 15:56 |
| haleyb | johnsom: yes, that would probably fix the 86 errors :) it looks like it really didn't run any tests, only 10 passed | 15:59 |
| johnsom | Yeah, I think 10 is all the ovn driver supports | 15:59 |
| johnsom | This was my latest run: https://121a82104cc98f0bf47f-8e50817bcbd575f2341914da48a13cbe.ssl.cf5.rackcdn.com/714004/29/check/neutron-ovn-provider-v2-scenario/ec5cbf5/testr_results.html | 15:59 |
| johnsom | for the OVN driver | 16:00 |
| johnsom | Though you might be able to pick up a few more if you enable ipv6 | 16:01 |
| haleyb | i'll look into it, is it just a yaml setting? | 16:02 |
| johnsom | test_with_ipv6 True | 16:04 |
| johnsom | Hmm, I guess true is the default, so maybe that is disabled on the OVN side? | 16:08 |
| johnsom | Yeah, it's disabled here: https://opendev.org/openstack/ovn-octavia-provider/src/branch/master/zuul.d/base.yaml#L93 | 16:09 |
| johnsom | So maybe ipv6 was broken in OVN at some point | 16:09 |
| haleyb | maybe, can easily test it | 16:10 |
| haleyb | johnsom: one thing i noticed is that in your testing, octavia_tempest_plugin.tests.scenario.v2.test_load_balancer.LoadBalancerScenarioTest ran two tests, but in the run i just did i see none, don't know why that is yet | 16:13 |
| johnsom | Hmm, Yeah, not sure why your run did "This provider does not support validating flavors" I thought I fixed that in this patch. | 16:16 |
| johnsom | Ah, it's the same, the "skip" from ovn triggered it to be an error that the ovn driver doesn't support flavors. That setting change will change let the lb tests run. | 16:17 |
| *** yamamoto has joined #openstack-lbaas | 16:18 | |
| *** yamamoto has quit IRC | 16:23 | |
| openstackgerrit | Carlos Goncalves proposed openstack/octavia-tempest-plugin master: Add tests for allowed CIDRs in listeners https://review.opendev.org/702629 | 16:35 |
| openstackgerrit | Carlos Goncalves proposed openstack/octavia master: Add noop certificate manager https://review.opendev.org/717619 | 17:19 |
| openstackgerrit | Carlos Goncalves proposed openstack/octavia-tempest-plugin master: Add experimental CentOS 8 Stream amphora job https://review.opendev.org/698450 | 17:22 |
| *** ccamposr|lunch has quit IRC | 18:02 | |
| *** gmann is now known as gmann_afk | 18:35 | |
| johnsom | Wee, looks like we have a bug in the API sort order code. | 18:38 |
| johnsom | We aren't translating the API parameter name to the data model name. We do for filtering, but missed sort. | 18:40 |
| *** vishalmanchanda has quit IRC | 18:57 | |
| *** gcheresh_ has joined #openstack-lbaas | 19:07 | |
| *** gcheresh_ has quit IRC | 19:46 | |
| openstackgerrit | Merged openstack/octavia master: Fix haproxy user flow log format substitution https://review.opendev.org/743059 | 20:09 |
| *** yamamoto has joined #openstack-lbaas | 20:22 | |
| *** yamamoto has quit IRC | 20:27 | |
| *** TrevorV has quit IRC | 20:47 | |
| *** rcernin has joined #openstack-lbaas | 21:21 | |
| *** rcernin has quit IRC | 21:27 | |
| *** born2bake has quit IRC | 21:33 | |
| *** rcernin has joined #openstack-lbaas | 21:34 | |
| *** rcernin has quit IRC | 21:42 | |
| *** yamamoto has joined #openstack-lbaas | 21:54 | |
| *** yamamoto has quit IRC | 21:59 | |
| *** rcernin has joined #openstack-lbaas | 22:11 | |
| *** ccamposr has joined #openstack-lbaas | 22:15 | |
| *** rcernin has quit IRC | 22:17 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Fix API sort key for complex columns https://review.opendev.org/745239 | 22:18 |
| johnsom | Blah, forgot a release note | 22:18 |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Fix API sort key for complex columns https://review.opendev.org/745239 | 22:24 |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Fix API sort key for complex columns https://review.opendev.org/745239 | 22:25 |
| *** ccamposr has quit IRC | 22:26 | |
| *** gmann_afk is now known as gmann | 22:35 | |
| *** rcernin has joined #openstack-lbaas | 22:47 | |
| *** rcernin has quit IRC | 22:52 | |
| *** tkajinam has joined #openstack-lbaas | 22:55 | |
| *** yamamoto has joined #openstack-lbaas | 22:58 | |
| *** rcernin has joined #openstack-lbaas | 23:20 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!