*** rcernin has quit IRC | 02:59 | |
*** psachin has joined #openstack-lbaas | 03:31 | |
*** ramishra has joined #openstack-lbaas | 03:33 | |
*** rcernin has joined #openstack-lbaas | 03:46 | |
*** pcaruana has joined #openstack-lbaas | 05:11 | |
*** logan- has quit IRC | 06:23 | |
*** logan- has joined #openstack-lbaas | 06:26 | |
*** trident has quit IRC | 06:48 | |
*** gcheresh has joined #openstack-lbaas | 06:48 | |
*** mjozefcz|away has joined #openstack-lbaas | 06:54 | |
*** rpittau|afk is now known as rpittau | 06:55 | |
*** trident has joined #openstack-lbaas | 06:57 | |
*** trident has quit IRC | 07:03 | |
*** trident has joined #openstack-lbaas | 07:12 | |
openstackgerrit | Carlos Goncalves proposed openstack/octavia stable/rocky: Fix template that generates vrrp check script https://review.opendev.org/682282 | 07:14 |
---|---|---|
openstackgerrit | Carlos Goncalves proposed openstack/octavia stable/queens: Fix template that generates vrrp check script https://review.opendev.org/682283 | 07:14 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/octavia-dashboard master: Imported Translations from Zanata https://review.opendev.org/682285 | 07:22 |
*** mjozefcz|away is now known as mjozefcz | 07:26 | |
*** ivve has joined #openstack-lbaas | 07:32 | |
openstackgerrit | Merged openstack/octavia-dashboard master: Imported Translations from Zanata https://review.opendev.org/682285 | 07:51 |
openstackgerrit | Carlos Goncalves proposed openstack/octavia stable/stein: Fix l7rule API handling of None updates https://review.opendev.org/678550 | 07:57 |
openstackgerrit | Carlos Goncalves proposed openstack/octavia stable/rocky: Fix l7rule API handling of None updates https://review.opendev.org/678551 | 08:05 |
openstackgerrit | Carlos Goncalves proposed openstack/octavia stable/queens: Fix l7rule API handling of None updates https://review.opendev.org/678557 | 08:06 |
*** ricolin has joined #openstack-lbaas | 08:09 | |
*** luksky has joined #openstack-lbaas | 08:30 | |
openstackgerrit | Maciej Józefczyk proposed openstack/octavia master: Validate supported LB algorithm in Amphora provider drivers https://review.opendev.org/672477 | 09:21 |
lxkong | hi, is there known issue for octavia CI? My patch https://review.opendev.org/#/c/682240/ keeps failing because of http://paste.openstack.org/show/776707/ | 09:33 |
gthiemonge | lxkong: yes, I'm working on a fix for it | 09:35 |
lxkong | gthiemonge: thanks | 09:36 |
gthiemonge | new python-openstackclient release breaks our devstack plugin | 09:36 |
openstackgerrit | Gregory Thiemonge proposed openstack/octavia master: Fix openstack port show content handling https://review.opendev.org/682327 | 09:48 |
*** ricolin has quit IRC | 09:52 | |
openstackgerrit | Gregory Thiemonge proposed openstack/octavia-tempest-plugin master: Add tests for mixed IP networks UDP members https://review.opendev.org/668619 | 10:21 |
openstackgerrit | Gregory Thiemonge proposed openstack/octavia-tempest-plugin master: Add UDP test scenario https://review.opendev.org/656515 | 10:21 |
openstackgerrit | Gregory Thiemonge proposed openstack/octavia-tempest-plugin master: Added tests for creation of listeners on same port https://review.opendev.org/668176 | 10:21 |
*** tesseract has joined #openstack-lbaas | 10:51 | |
*** tesseract has quit IRC | 11:09 | |
*** mjozefcz has quit IRC | 11:20 | |
*** mjozefcz has joined #openstack-lbaas | 11:20 | |
openstackgerrit | Merged openstack/python-octaviaclient stable/train: Update .gitreview for stable/train https://review.opendev.org/682169 | 11:40 |
*** boden has joined #openstack-lbaas | 11:42 | |
openstackgerrit | Merged openstack/octavia-lib stable/train: Update .gitreview for stable/train https://review.opendev.org/680333 | 11:44 |
openstackgerrit | Merged openstack/python-octaviaclient stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train https://review.opendev.org/682170 | 11:48 |
openstackgerrit | Merged openstack/octavia-lib stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train https://review.opendev.org/680334 | 11:52 |
openstackgerrit | Merged openstack/octavia-lib master: Update master for stable/train https://review.opendev.org/680335 | 11:52 |
*** luksky has quit IRC | 12:09 | |
*** goldyfruit___ has quit IRC | 12:15 | |
*** mjozefcz has quit IRC | 12:21 | |
*** mjozefcz has joined #openstack-lbaas | 12:22 | |
*** ccamposr__ has joined #openstack-lbaas | 12:32 | |
*** ccamposr has quit IRC | 12:34 | |
*** rcernin has quit IRC | 12:35 | |
*** ccamposr has joined #openstack-lbaas | 12:46 | |
*** ccamposr__ has quit IRC | 12:48 | |
*** luksky has joined #openstack-lbaas | 12:57 | |
*** rcernin has joined #openstack-lbaas | 12:59 | |
*** KeithMnemonic has joined #openstack-lbaas | 13:18 | |
-openstackstatus- NOTICE: The Gerrit service on review.opendev.org will be offline briefly starting at 14:00 UTC (that's roughly 30 minutes from now) for maintenance: http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009064.html | 13:28 | |
*** pcaruana has quit IRC | 13:45 | |
*** rcernin has quit IRC | 13:49 | |
*** goldyfruit___ has joined #openstack-lbaas | 13:50 | |
openstackgerrit | Gregory Thiemonge proposed openstack/octavia master: Fix healtmonitor message v2 for UDP listeners https://review.opendev.org/682365 | 13:53 |
-openstackstatus- NOTICE: The Gerrit service on review.opendev.org is offline briefly for maintenance: http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009064.html | 14:04 | |
*** ChanServ changes topic to "The Gerrit service on review.opendev.org is offline briefly for maintenance: http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009064.html" | 14:04 | |
*** pcaruana has joined #openstack-lbaas | 14:07 | |
*** dulek has joined #openstack-lbaas | 14:16 | |
*** mjozefcz has quit IRC | 14:22 | |
*** ChanServ changes topic to "Discussions for OpenStack Octavia | Priority bug review list: https://etherpad.openstack.org/p/octavia-priority-reviews" | 14:36 | |
-openstackstatus- NOTICE: The Gerrit outage portion of the current maintenance is complete and the service is back on line, however reindexing for renamed repositories is still underway and some Zuul job fixes are in the process of being applied | 14:36 | |
*** goldyfruit_ has joined #openstack-lbaas | 14:36 | |
*** goldyfruit___ has quit IRC | 14:39 | |
*** gcheresh has quit IRC | 14:53 | |
openstackgerrit | Austin Russell proposed openstack/octavia master: loadbalancer vip-network-id IP availability check https://review.opendev.org/673358 | 14:57 |
johnsom | Hmm, I thought we left the UDP listeners on the v1 format messages.... | 14:58 |
*** goldyfruit___ has joined #openstack-lbaas | 15:03 | |
*** goldyfruit_ has quit IRC | 15:05 | |
*** openstackgerrit has quit IRC | 15:08 | |
gthiemonge | johnsom: https://opendev.org/openstack/octavia/src/branch/master/octavia/amphorae/backends/health_daemon/health_daemon.py#L170 | 15:11 |
gthiemonge | johnsom: it uses 'ver': 2 | 15:12 |
johnsom | Yeah, I saw that. | 15:12 |
*** ivve has quit IRC | 15:20 | |
*** luksky has quit IRC | 15:44 | |
*** mjozefcz has joined #openstack-lbaas | 16:05 | |
*** nmagnezi has quit IRC | 16:18 | |
*** openstackgerrit has joined #openstack-lbaas | 16:22 | |
openstackgerrit | Carlos Goncalves proposed openstack/octavia master: WIP: Switch Fedora-based amphora to fedora-minimal https://review.opendev.org/673173 | 16:22 |
*** goldyfruit_ has joined #openstack-lbaas | 16:28 | |
*** goldyfruit___ has quit IRC | 16:31 | |
*** rpittau is now known as rpittau|afk | 16:59 | |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Fix the amphora no-op driver https://review.opendev.org/682420 | 17:07 |
*** ramishra has quit IRC | 17:17 | |
*** psachin has quit IRC | 17:25 | |
*** mjozefcz has quit IRC | 17:27 | |
*** goldyfruit_ has quit IRC | 17:28 | |
*** ccamposr__ has joined #openstack-lbaas | 17:30 | |
*** ccamposr has quit IRC | 17:32 | |
*** goldyfruit has joined #openstack-lbaas | 18:05 | |
*** luksky has joined #openstack-lbaas | 18:24 | |
openstackgerrit | Carlos Goncalves proposed openstack/octavia master: WIP: Switch Fedora-based amphora to fedora-minimal https://review.opendev.org/673173 | 18:27 |
*** openstackgerrit has quit IRC | 18:52 | |
*** goldyfruit_ has joined #openstack-lbaas | 19:07 | |
*** goldyfruit has quit IRC | 19:09 | |
*** boden has quit IRC | 19:54 | |
*** luksky has quit IRC | 20:16 | |
*** luksky has joined #openstack-lbaas | 20:17 | |
rm_work | johnsom: test__interface_by_mac_case_insensitive_ubuntu doesn't work on OSX? :( | 20:33 |
* rm_work is sad | 20:33 | |
johnsom | Some new pyroute issue? | 20:33 |
rm_work | no i think it stopped working when you changed it last | 20:34 |
rm_work | in "Lookup interfaces by MAC directly" | 20:34 |
johnsom | That has been a while though | 20:34 |
rm_work | like ... a month? :D | 20:34 |
johnsom | It is still using pyroute for that | 20:34 |
rm_work | yeah but something changed | 20:34 |
rm_work | I think | 20:35 |
rm_work | hold on let me veirfy | 20:35 |
johnsom | month == years in Octavia time lol | 20:35 |
rm_work | lol | 20:36 |
johnsom | Well, I don’t have a mac anymore, so this one might be up to you. | 20:36 |
rm_work | if I revert that change, all tests pass | 20:36 |
rm_work | T_T | 20:36 |
johnsom | Is MacOS on the OpenStack pti? | 20:37 |
rm_work | <_< | 20:37 |
cgoncalves | lol | 20:37 |
rm_work | it makes things difficult for developers >_> | 20:38 |
johnsom | See what happens when you work on other things? Grin | 20:38 |
rm_work | yeah apparently lol | 20:38 |
rm_work | I am single-handedly keeping our unit tests running on OSX T_T | 20:38 |
*** pcaruana has quit IRC | 20:38 | |
johnsom | I think that is a fair statement. | 20:39 |
*** trident has quit IRC | 20:47 | |
rm_work | fff | 20:50 |
rm_work | hmm | 20:54 |
rm_work | [Errno 47] Address family not supported by protocol family | 20:54 |
rm_work | socket.socket(family=AF_NETLINK,type=SOCK_DGRAM,proto=family) | 20:55 |
rm_work | that seems ... odd | 20:55 |
rm_work | family=AF_NETLINK (16) ? | 20:55 |
rm_work | and ... proto=family (0) ? | 20:55 |
rm_work | did socket.socket change arg order or something between OS? | 20:55 |
rm_work | why is proto=family? | 20:56 |
rm_work | ah i guess the first is the address family and the second is the protocol *family* | 20:57 |
johnsom | Which line is throwing that? | 20:58 |
rm_work | as you expect, it's deep in pyroute2 | 20:59 |
*** trident has joined #openstack-lbaas | 20:59 | |
rm_work | i think it's because it's trying to use the linux pyroute on osx | 20:59 |
rm_work | i'm just trying to understand what EXACTLY the issue is so I can come up with the cleanest fix | 20:59 |
rm_work | it's like 952 here: pyroute2/netlink/nlsocket.py | 21:00 |
*** openstackgerrit has joined #openstack-lbaas | 21:08 | |
openstackgerrit | Merged openstack/octavia stable/stein: Add support for monitor_{address,port} in UDP members https://review.opendev.org/675349 | 21:08 |
JayF | rm_work: you have a direct link to that? | 21:15 |
rm_work | JayF: eh i think i figured out the real issue anyway, which is that this whole lib should be mocked out -- we don't need to be unit-testing internal pyroute functionality | 21:18 |
rm_work | and it IS mocked -- but not correctly (the mock is too deep, it works for linux but not on OSX) | 21:19 |
JayF | nice | 21:19 |
JayF | those sort of cross-OS compatability things with networking are a pet peeve | 21:20 |
JayF | and by extension a minor point of expertise | 21:20 |
rm_work | yeah it was a bad mock | 21:25 |
rm_work | it was: | 21:25 |
rm_work | @mock.patch('pyroute2.IPRoute.__enter__', create=True) | 21:25 |
JayF | oh, yeah, that's probably never an awesome mock to get the enter to a context manager | 21:26 |
rm_work | but that actually doesn't work the way you'd expect, but in Linux it just technically worked without the mock apparently lol | 21:26 |
rm_work | needs to be: | 21:26 |
rm_work | @mock.patch('pyroute2.IPRoute', create=True) | 21:26 |
rm_work | and then actually do: | 21:26 |
rm_work | mock_ipr().__enter__.return_value = blah | 21:26 |
JayF | Yep, that makes a lot of sense | 21:26 |
rm_work | easy fix fortunately | 21:27 |
*** rcernin has joined #openstack-lbaas | 21:47 | |
johnsom | I remember having troubles with that mock. | 21:58 |
rm_work | yeah you must have thought you got it working on linux but really it was just *passing* :D | 22:01 |
*** spatel has joined #openstack-lbaas | 22:08 | |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Fix some plug.py unit tests that broke on OSX https://review.opendev.org/682501 | 22:11 |
*** spatel has quit IRC | 22:12 | |
*** ccamposr__ has quit IRC | 22:21 | |
rm_work | ffff forgot to run pep8 checks lol | 22:22 |
rm_work | and of course, 100% of the time i run them it's ok, and 100% of the time i don't it isn't | 22:23 |
rm_work | oh nm it's fine whew | 22:34 |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Allow multiple VIPs per LB https://review.opendev.org/660239 | 22:46 |
*** goldyfruit_ has quit IRC | 22:50 | |
*** tkajinam has joined #openstack-lbaas | 22:54 | |
*** luksky has quit IRC | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!