| *** armax has quit IRC | 00:29 | |
| openstackgerrit | wu.shiming proposed openstack/python-octaviaclient master: Remove Babel https://review.opendev.org/751627 | 01:45 |
|---|---|---|
| openstackgerrit | wu.shiming proposed openstack/python-octaviaclient master: Remove install unnecessary packages https://review.opendev.org/751629 | 01:57 |
| *** ramishra has joined #openstack-lbaas | 02:04 | |
| *** zzzeek has quit IRC | 02:20 | |
| *** zzzeek has joined #openstack-lbaas | 02:22 | |
| *** TMM has quit IRC | 03:35 | |
| *** TMM has joined #openstack-lbaas | 03:35 | |
| *** psachin has joined #openstack-lbaas | 03:37 | |
| *** zzzeek has quit IRC | 03:50 | |
| *** zzzeek has joined #openstack-lbaas | 03:51 | |
| *** gcheresh has joined #openstack-lbaas | 04:34 | |
| *** ramishra has quit IRC | 04:41 | |
| *** gcheresh has quit IRC | 04:44 | |
| *** vishalmanchanda has joined #openstack-lbaas | 04:45 | |
| *** gcheresh has joined #openstack-lbaas | 04:59 | |
| *** numans has joined #openstack-lbaas | 05:47 | |
| *** dasp has quit IRC | 05:48 | |
| *** cgoncalves has joined #openstack-lbaas | 06:13 | |
| openstackgerrit | wu.shiming proposed openstack/octavia master: Remove install unnecessary packages https://review.opendev.org/751650 | 06:19 |
| *** ramishra has joined #openstack-lbaas | 06:23 | |
| *** ccamposr has quit IRC | 07:01 | |
| openstackgerrit | Merged openstack/octavia-lib master: Update master for stable/victoria https://review.opendev.org/750502 | 07:01 |
| *** ccamposr has joined #openstack-lbaas | 07:01 | |
| openstackgerrit | Merged openstack/octavia-lib master: Add Python3 wallaby unit tests https://review.opendev.org/750503 | 07:05 |
| openstackgerrit | Carlos Goncalves proposed openstack/octavia-tempest-plugin master: Switch default system under test to CentOS 8 https://review.opendev.org/746996 | 07:11 |
| openstackgerrit | Arkady Shtempler proposed openstack/octavia-tempest-plugin master: Adding failover test. Send HTTP traffic while MASTER Amphore is rebooted. BACKUP Amphore should serve the traffic. https://review.opendev.org/751617 | 08:04 |
| openstackgerrit | Carlos Goncalves proposed openstack/octavia master: Install HAProxy 2.x in CentOS amphora images https://review.opendev.org/744102 | 09:05 |
| *** spatel has joined #openstack-lbaas | 09:38 | |
| *** spatel has quit IRC | 09:44 | |
| openstackgerrit | wu.shiming proposed openstack/octavia master: Remove install unnecessary packages https://review.opendev.org/751650 | 10:07 |
| *** servagem has joined #openstack-lbaas | 11:45 | |
| *** ramishra has quit IRC | 11:48 | |
| *** ramishra has joined #openstack-lbaas | 11:52 | |
| *** ramishra_ has joined #openstack-lbaas | 12:05 | |
| *** ramishra has quit IRC | 12:08 | |
| *** njohnston has joined #openstack-lbaas | 12:26 | |
| *** vishalmanchanda has quit IRC | 12:28 | |
| *** spatel has joined #openstack-lbaas | 12:56 | |
| gthiemonge | johnsom: looks like octavia-grenade job is still broken on stable/ussuri: https://zuul.opendev.org/t/openstack/builds?job_name=octavia-grenade&branch=stable%2Fussuri | 12:58 |
| cgoncalves | ouch! the scenario test refactor patch more than doubled the gate time (45 minutes -> 1h54m) | 13:09 |
| *** spatel has quit IRC | 13:12 | |
| openstackgerrit | Merged openstack/python-octaviaclient master: Update master for stable/victoria https://review.opendev.org/751081 | 13:16 |
| openstackgerrit | Merged openstack/python-octaviaclient master: Add Python3 wallaby unit tests https://review.opendev.org/751082 | 13:16 |
| *** irclogbot_0 has quit IRC | 13:19 | |
| *** irclogbot_3 has joined #openstack-lbaas | 13:25 | |
| *** mnaser has quit IRC | 13:32 | |
| *** mnaser has joined #openstack-lbaas | 13:32 | |
| *** mnaser has quit IRC | 13:32 | |
| *** mnaser has joined #openstack-lbaas | 13:32 | |
| *** TrevorV has joined #openstack-lbaas | 13:33 | |
| *** tkajinam has quit IRC | 13:37 | |
| openstackgerrit | Carlos Goncalves proposed openstack/octavia master: Install HAProxy 2.x in CentOS amphora images https://review.opendev.org/744102 | 14:01 |
| *** ccamposr__ has joined #openstack-lbaas | 14:20 | |
| *** ccamposr has quit IRC | 14:22 | |
| *** laerling has joined #openstack-lbaas | 14:50 | |
| openstackgerrit | Brian Haley proposed openstack/octavia master: Remove idempotent id check code https://review.opendev.org/751835 | 14:57 |
| johnsom | cgoncalves In fairness, we more than doubled the test permutations we are now running.... | 15:01 |
| cgoncalves | sure, and that is great! | 15:11 |
| *** armax has joined #openstack-lbaas | 15:12 | |
| johnsom | There is probably some good work to do in looking at our concurrency and the longest running classes. | 15:13 |
| *** ianychoi has joined #openstack-lbaas | 15:24 | |
| *** gcheresh has quit IRC | 15:44 | |
| johnsom | The docs job timed out? lol, happy Monday! | 15:50 |
| *** gcheresh has joined #openstack-lbaas | 15:51 | |
| *** zzzeek has quit IRC | 15:54 | |
| *** zzzeek has joined #openstack-lbaas | 15:55 | |
| *** zzzeek has quit IRC | 16:02 | |
| *** zzzeek has joined #openstack-lbaas | 16:04 | |
| *** zzzeek has quit IRC | 16:09 | |
| *** psachin has quit IRC | 16:09 | |
| *** zzzeek has joined #openstack-lbaas | 16:10 | |
| *** zzzeek has quit IRC | 16:15 | |
| *** zzzeek has joined #openstack-lbaas | 16:19 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia stable/ussuri: Update grenade job to run one smoke test https://review.opendev.org/751854 | 16:25 |
| *** dasp has joined #openstack-lbaas | 16:46 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Switch grenade to run the octavia smoke tests https://review.opendev.org/751860 | 16:47 |
| johnsom | Let's see if that works.... | 16:48 |
| *** gcheresh has quit IRC | 16:56 | |
| openstackgerrit | Carlos Goncalves proposed openstack/octavia master: Fix image build when installing agent from package https://review.opendev.org/751863 | 16:57 |
| *** dasp has quit IRC | 16:58 | |
| johnsom | Hmm, interesting. I thought DIB package install mode skipped all of those scripts. | 16:58 |
| *** dasp has joined #openstack-lbaas | 17:00 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Update protocol combination api-ref for PROXYV2 https://review.opendev.org/750468 | 17:00 |
| openstackgerrit | Carlos Goncalves proposed openstack/octavia master: Install HAProxy 2.x in CentOS amphora images https://review.opendev.org/744102 | 17:02 |
| *** vishalmanchanda has joined #openstack-lbaas | 17:04 | |
| *** sapd1_x has quit IRC | 17:11 | |
| *** gthiemonge has quit IRC | 17:54 | |
| *** gthiemonge has joined #openstack-lbaas | 17:54 | |
| *** njohnston has quit IRC | 18:21 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia stable/ussuri: Update grenade job to run one smoke test https://review.opendev.org/751854 | 18:45 |
| rm_work | cgoncalves: mmm liking the look of that haproxy2 patch :D | 19:12 |
| *** zzzeek has quit IRC | 19:14 | |
| cgoncalves | rm_work, cool. let me know if you test it. as soon as I get it passing CI, I'll provide feedback to CentOS NFV SIG | 19:14 |
| cgoncalves | I also wanted to update haproxy to 2.2.3 released last week. they want it first in fedora rawhide so I proposed a PR | 19:15 |
| *** zzzeek has joined #openstack-lbaas | 19:16 | |
| *** zzzeek has quit IRC | 19:20 | |
| rm_work | cool | 19:20 |
| rm_work | not sure how easy testing will be for me... | 19:20 |
| rm_work | we'll see | 19:20 |
| rm_work | need to work out a less manual pipeline to build/deploy test images without actually merging the patch into our official CI process | 19:21 |
| cgoncalves | PR failed CI on armv7hl. this is a known haproxy bug, fixed already IIRC but pending release | 19:23 |
| *** zzzeek has joined #openstack-lbaas | 19:23 | |
| cgoncalves | yep, here http://git.haproxy.org/?p=haproxy-2.2.git;a=commitdiff_plain;h=10c627ab | 19:24 |
| *** zzzeek has quit IRC | 19:28 | |
| cgoncalves | ah, Ryan skipped my PR and updated to 2.2.3 just now | 19:28 |
| *** zzzeek has joined #openstack-lbaas | 19:29 | |
| rm_work | ... are you talking to someone i can't see? | 19:29 |
| rm_work | i see only you and me for the last like 20m in this channel | 19:29 |
| cgoncalves | speaking to myself :P | 19:30 |
| rm_work | k lol | 19:30 |
| rm_work | good info tho :D | 19:30 |
| *** zzzeek has quit IRC | 19:33 | |
| *** zzzeek has joined #openstack-lbaas | 19:35 | |
| johnsom | I was following along | 19:36 |
| cgoncalves | <3 | 19:44 |
| *** vishalmanchanda has quit IRC | 19:44 | |
| *** zzzeek has quit IRC | 19:54 | |
| *** gcheresh has joined #openstack-lbaas | 19:56 | |
| *** zzzeek has joined #openstack-lbaas | 19:57 | |
| haleyb | johnsom: so a question on this failure we're seeing with allowed_cidrs and the ovn driver... i know the driver doesn't support it, but since the address is 0.0.0.0/0 is should "just work", right? | 20:44 |
| haleyb | i'm trying to backtrack to when/if it worked | 20:44 |
| johnsom | haleyb I don't think it was ever tested before this. Maybe cgoncalves would know if the cidrs stuff was tested on OVN provider. | 20:46 |
| openstackgerrit | Merged openstack/octavia master: Remove haproxy_check_script for UDP-only LBs https://review.opendev.org/751379 | 20:46 |
| haleyb | the support matrix shows "missing" so i'm assuming wasn't | 20:47 |
| johnsom | Well, that just means the driver can't do it | 20:47 |
| haleyb | johnsom: i guess since the failure is traffic was unbalanced it seems things were not blocked with the all-zeros address, i can wait the cgoncalves on the "if it was tested" and if not just raise notimplmentedskip | 20:54 |
| * haleyb can't type | 20:54 | |
| *** gcheresh has quit IRC | 21:04 | |
| openstackgerrit | Adam Harwell proposed openstack/octavia master: Use routed network filter if it exists https://review.opendev.org/706153 | 21:10 |
| haleyb | johnsom: ok, one last question... i'm trying to follow the test, _test_listener_with_allowed_cidrs() - and it catches a failure on create_pool() but not create_listener() - should it be doing it for both? e.g. if i want to change the ovn driver to raise on create_listener() with allowed_cidrs present | 21:22 |
| johnsom | I didn't write the test, so let me open it and look at what is there. I'm not sure I follow your question | 21:23 |
| johnsom | Are you looking at scenario or api? | 21:23 |
| haleyb | scenario test | 21:24 |
| haleyb | i'm just wanting to skip the test for now as our gate is broken, and if i raise on listener create i'm not sure it will skip or fail | 21:25 |
| haleyb | short of just creating a patch to test my theory :) | 21:25 |
| johnsom | Oh, well, if you raise a "not implemented" anywhere for that test it will skip | 21:25 |
| johnsom | So, yeah, if you just want to skip the test, raise a not implemented or not implemented option on the listener create when the allowed_cidrs field is set. | 21:26 |
| haleyb | ack, i can then just add a bug to our backlog | 21:28 |
| johnsom | +1 | 21:28 |
| openstackgerrit | Michael Johnson proposed openstack/octavia stable/ussuri: Remove haproxy_check_script for UDP-only LBs https://review.opendev.org/751907 | 21:30 |
| openstackgerrit | Michael Johnson proposed openstack/octavia stable/train: Remove haproxy_check_script for UDP-only LBs https://review.opendev.org/751908 | 21:30 |
| *** ccamposr has joined #openstack-lbaas | 21:31 | |
| *** TrevorV has quit IRC | 21:33 | |
| *** ccamposr__ has quit IRC | 21:34 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Fix requirements.txt for Victoria https://review.opendev.org/751918 | 22:44 |
| *** tkajinam has joined #openstack-lbaas | 22:52 | |
| *** dtruong has quit IRC | 22:54 | |
| *** dtruong has joined #openstack-lbaas | 22:54 | |
| rm_work | ugh, beating my head against a wall on a mock not working | 23:03 |
| johnsom | Oh, I hate that. | 23:06 |
| johnsom | BTW, gates are broken on some nodepool instances can't find oslo.log 4.4.0 | 23:06 |
| rm_work | also wtf, was going to point to the error in this patch, but almost every job is failing with a different req issu.... ah | 23:06 |
| rm_work | yeah ok | 23:06 |
| johnsom | We have seen this before, it seems like a CDN issue | 23:06 |
| rm_work | some it's oslo.log, some oslo,serialization, one was some other lib | 23:06 |
| johnsom | Oh, I have only seen oslo.log | 23:07 |
| rm_work | https://7e1855dd4f4cae214e00-4bc1af96845a4b2efb646674c5f1b41c.ssl.cf2.rackcdn.com/706153/10/check/openstack-tox-py38/e86cd4b/job-output.txt | 23:13 |
| rm_work | ERROR: Could not find a version that satisfies the requirement oslo.serialization===4.0.1 | 23:13 |
| rm_work | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_724/706153/10/check/octavia-tox-functional-py37-tips/724104a/job-output.txt | 23:14 |
| rm_work | ERROR: Could not find a version that satisfies the requirement taskflow===4.5.0 | 23:14 |
| johnsom | Yeah, it's all borked | 23:14 |
| rm_work | looks like the thing i need to mock is already loaded.... | 23:18 |
| rm_work | `stevedore.driver.DriverManager.driver` | 23:18 |
| rm_work | so ... i thought i could mock it directly where it was used? like | 23:18 |
| rm_work | `octavia.common.utils.stevedore_driver.DriverManager.driver` | 23:19 |
| rm_work | but that isn't working either | 23:19 |
| rm_work | I think the issue is that the utils file is loaded before the test code even runs (it's imported in the test file itself) | 23:19 |
| johnsom | We mock our stevedore in a ton of places | 23:22 |
| rm_work | yeah | 23:22 |
| rm_work | and it's mocked here | 23:22 |
| rm_work | and USED to work, but now doesn't | 23:22 |
| johnsom | Where are you trying to mock it? | 23:22 |
| rm_work | test_compute_tasks.py | 23:23 |
| rm_work | either v1 or v2 | 23:23 |
| rm_work | I figured out how I can fix the mock tho I think | 23:23 |
| rm_work | changed to mocking `stevedore.driver.DriverManager` | 23:23 |
| rm_work | and changed the references to like | 23:23 |
| rm_work | `mock_driver().driver.build.return_value = COMPUTE_ID` | 23:23 |
| rm_work | instead of `mock_driver.build.return_value = COMPUTE_ID` | 23:24 |
| rm_work | normally it's not a problem there because only my patch adds usage of utils.get_network_driver into that task | 23:25 |
| rm_work | you'd only see the issue on https://review.opendev.org/#/c/706153/ | 23:25 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!