Monday, 2019-09-16

« Sunday, 2019-09-15 Index Tuesday, 2019-09-17 »
*** rcernin has quit IRC02:59
*** psachin has joined #openstack-lbaas03:31
*** ramishra has joined #openstack-lbaas03:33
*** rcernin has joined #openstack-lbaas03:46
*** pcaruana has joined #openstack-lbaas05:11
*** logan- has quit IRC06:23
*** logan- has joined #openstack-lbaas06:26
*** trident has quit IRC06:48
*** gcheresh has joined #openstack-lbaas06:48
*** mjozefcz|away has joined #openstack-lbaas06:54
*** rpittau|afk is now known as rpittau06:55
*** trident has joined #openstack-lbaas06:57
*** trident has quit IRC07:03
*** trident has joined #openstack-lbaas07:12
openstackgerritCarlos Goncalves proposed openstack/octavia stable/rocky: Fix template that generates vrrp check script  https://review.opendev.org/68228207:14
openstackgerritCarlos Goncalves proposed openstack/octavia stable/queens: Fix template that generates vrrp check script  https://review.opendev.org/68228307:14
openstackgerritOpenStack Proposal Bot proposed openstack/octavia-dashboard master: Imported Translations from Zanata  https://review.opendev.org/68228507:22
*** mjozefcz|away is now known as mjozefcz07:26
*** ivve has joined #openstack-lbaas07:32
openstackgerritMerged openstack/octavia-dashboard master: Imported Translations from Zanata  https://review.opendev.org/68228507:51
openstackgerritCarlos Goncalves proposed openstack/octavia stable/stein: Fix l7rule API handling of None updates  https://review.opendev.org/67855007:57
openstackgerritCarlos Goncalves proposed openstack/octavia stable/rocky: Fix l7rule API handling of None updates  https://review.opendev.org/67855108:05
openstackgerritCarlos Goncalves proposed openstack/octavia stable/queens: Fix l7rule API handling of None updates  https://review.opendev.org/67855708:06
*** ricolin has joined #openstack-lbaas08:09
*** luksky has joined #openstack-lbaas08:30
openstackgerritMaciej Józefczyk proposed openstack/octavia master: Validate supported LB algorithm in Amphora provider drivers  https://review.opendev.org/67247709:21
lxkonghi, 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
gthiemongelxkong: yes, I'm working on a fix for it09:35
lxkonggthiemonge: thanks09:36
gthiemongenew python-openstackclient release breaks our devstack plugin09:36
openstackgerritGregory Thiemonge proposed openstack/octavia master: Fix openstack port show content handling  https://review.opendev.org/68232709:48
*** ricolin has quit IRC09:52
openstackgerritGregory Thiemonge proposed openstack/octavia-tempest-plugin master: Add tests for mixed IP networks UDP members  https://review.opendev.org/66861910:21
openstackgerritGregory Thiemonge proposed openstack/octavia-tempest-plugin master: Add UDP test scenario  https://review.opendev.org/65651510:21
openstackgerritGregory Thiemonge proposed openstack/octavia-tempest-plugin master: Added tests for creation of listeners on same port  https://review.opendev.org/66817610:21
*** tesseract has joined #openstack-lbaas10:51
*** tesseract has quit IRC11:09
*** mjozefcz has quit IRC11:20
*** mjozefcz has joined #openstack-lbaas11:20
openstackgerritMerged openstack/python-octaviaclient stable/train: Update .gitreview for stable/train  https://review.opendev.org/68216911:40
*** boden has joined #openstack-lbaas11:42
openstackgerritMerged openstack/octavia-lib stable/train: Update .gitreview for stable/train  https://review.opendev.org/68033311:44
openstackgerritMerged openstack/python-octaviaclient stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train  https://review.opendev.org/68217011:48
openstackgerritMerged openstack/octavia-lib stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train  https://review.opendev.org/68033411:52
openstackgerritMerged openstack/octavia-lib master: Update master for stable/train  https://review.opendev.org/68033511:52
*** luksky has quit IRC12:09
*** goldyfruit___ has quit IRC12:15
*** mjozefcz has quit IRC12:21
*** mjozefcz has joined #openstack-lbaas12:22
*** ccamposr__ has joined #openstack-lbaas12:32
*** ccamposr has quit IRC12:34
*** rcernin has quit IRC12:35
*** ccamposr has joined #openstack-lbaas12:46
*** ccamposr__ has quit IRC12:48
*** luksky has joined #openstack-lbaas12:57
*** rcernin has joined #openstack-lbaas12:59
*** KeithMnemonic has joined #openstack-lbaas13: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.html13:28
*** pcaruana has quit IRC13:45
*** rcernin has quit IRC13:49
*** goldyfruit___ has joined #openstack-lbaas13:50
openstackgerritGregory Thiemonge proposed openstack/octavia master: Fix healtmonitor message v2 for UDP listeners  https://review.opendev.org/68236513: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.html14: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-lbaas14:07
*** dulek has joined #openstack-lbaas14:16
*** mjozefcz has quit IRC14: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 applied14:36
*** goldyfruit_ has joined #openstack-lbaas14:36
*** goldyfruit___ has quit IRC14:39
*** gcheresh has quit IRC14:53
openstackgerritAustin Russell proposed openstack/octavia master: loadbalancer vip-network-id IP availability check  https://review.opendev.org/67335814:57
johnsomHmm, I thought we left the UDP listeners on the v1 format messages....14:58
*** goldyfruit___ has joined #openstack-lbaas15:03
*** goldyfruit_ has quit IRC15:05
*** openstackgerrit has quit IRC15:08
gthiemongejohnsom: https://opendev.org/openstack/octavia/src/branch/master/octavia/amphorae/backends/health_daemon/health_daemon.py#L17015:11
gthiemongejohnsom: it uses 'ver': 215:12
johnsomYeah, I saw that.15:12
*** ivve has quit IRC15:20
*** luksky has quit IRC15:44
*** mjozefcz has joined #openstack-lbaas16:05
*** nmagnezi has quit IRC16:18
*** openstackgerrit has joined #openstack-lbaas16:22
openstackgerritCarlos Goncalves proposed openstack/octavia master: WIP: Switch Fedora-based amphora to fedora-minimal  https://review.opendev.org/67317316:22
*** goldyfruit_ has joined #openstack-lbaas16:28
*** goldyfruit___ has quit IRC16:31
*** rpittau is now known as rpittau|afk16:59
openstackgerritMichael Johnson proposed openstack/octavia master: Fix the amphora no-op driver  https://review.opendev.org/68242017:07
*** ramishra has quit IRC17:17
*** psachin has quit IRC17:25
*** mjozefcz has quit IRC17:27
*** goldyfruit_ has quit IRC17:28
*** ccamposr__ has joined #openstack-lbaas17:30
*** ccamposr has quit IRC17:32
*** goldyfruit has joined #openstack-lbaas18:05
*** luksky has joined #openstack-lbaas18:24
openstackgerritCarlos Goncalves proposed openstack/octavia master: WIP: Switch Fedora-based amphora to fedora-minimal  https://review.opendev.org/67317318:27
*** openstackgerrit has quit IRC18:52
*** goldyfruit_ has joined #openstack-lbaas19:07
*** goldyfruit has quit IRC19:09
*** boden has quit IRC19:54
*** luksky has quit IRC20:16
*** luksky has joined #openstack-lbaas20:17
rm_workjohnsom: test__interface_by_mac_case_insensitive_ubuntu doesn't work on OSX? :(20:33
* rm_work is sad20:33
johnsomSome new pyroute issue?20:33
rm_workno i think it stopped working when you changed it last20:34
rm_workin "Lookup interfaces by MAC directly"20:34
johnsomThat has been a while though20:34
rm_worklike ... a month? :D20:34
johnsomIt is still using pyroute for that20:34
rm_workyeah but something changed20:34
rm_workI think20:35
rm_workhold on let me veirfy20:35
johnsommonth == years in Octavia time lol20:35
rm_worklol20:36
johnsomWell, I don’t have a mac anymore, so this one might be up to you.20:36
rm_workif I revert that change, all tests pass20:36
rm_workT_T20:36
johnsomIs MacOS on the OpenStack pti?20:37
rm_work<_<20:37
cgoncalveslol20:37
rm_workit makes things difficult for developers >_>20:38
johnsomSee what happens when you work on other things?  Grin20:38
rm_workyeah apparently lol20:38
rm_workI am single-handedly keeping our unit tests running on OSX T_T20:38
*** pcaruana has quit IRC20:38
johnsomI think that is a fair statement.20:39
*** trident has quit IRC20:47
rm_workfff20:50
rm_workhmm20:54
rm_work[Errno 47] Address family not supported by protocol family20:54
rm_worksocket.socket(family=AF_NETLINK,type=SOCK_DGRAM,proto=family)20:55
rm_workthat seems ... odd20:55
rm_workfamily=AF_NETLINK (16) ?20:55
rm_workand ... proto=family (0) ?20:55
rm_workdid socket.socket change arg order or something between OS?20:55
rm_workwhy is proto=family?20:56
rm_workah i guess the first is the address family and the second is the protocol *family*20:57
johnsomWhich line is throwing that?20:58
rm_workas you expect, it's deep in pyroute220:59
*** trident has joined #openstack-lbaas20:59
rm_worki think it's because it's trying to use the linux pyroute on osx20:59
rm_worki'm just trying to understand what EXACTLY the issue is so I can come up with the cleanest fix20:59
rm_workit's like 952 here: pyroute2/netlink/nlsocket.py21:00
*** openstackgerrit has joined #openstack-lbaas21:08
openstackgerritMerged openstack/octavia stable/stein: Add support for monitor_{address,port} in UDP members  https://review.opendev.org/67534921:08
JayFrm_work: you have a direct link to that?21:15
rm_workJayF: 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 functionality21:18
rm_workand it IS mocked -- but not correctly (the mock is too deep, it works for linux but not on OSX)21:19
JayFnice21:19
JayFthose sort of cross-OS compatability things with networking are a pet peeve21:20
JayFand by extension a minor point of expertise21:20
rm_workyeah it was a bad mock21:25
rm_workit was:21:25
rm_work@mock.patch('pyroute2.IPRoute.__enter__', create=True)21:25
JayFoh, yeah, that's probably never an awesome mock to get the enter to a context manager21:26
rm_workbut that actually doesn't work the way you'd expect, but in Linux it just technically worked without the mock apparently lol21:26
rm_workneeds to be:21:26
rm_work@mock.patch('pyroute2.IPRoute', create=True)21:26
rm_workand then actually do:21:26
rm_workmock_ipr().__enter__.return_value = blah21:26
JayFYep, that makes a lot of sense21:26
rm_workeasy fix fortunately21:27
*** rcernin has joined #openstack-lbaas21:47
johnsomI remember having troubles with that mock.21:58
rm_workyeah you must have thought you got it working on linux but really it was just *passing* :D22:01
*** spatel has joined #openstack-lbaas22:08
openstackgerritAdam Harwell proposed openstack/octavia master: Fix some plug.py unit tests that broke on OSX  https://review.opendev.org/68250122:11
*** spatel has quit IRC22:12
*** ccamposr__ has quit IRC22:21
rm_workffff forgot to run pep8 checks lol22:22
rm_workand of course, 100% of the time i run them it's ok, and 100% of the time i don't it isn't22:23
rm_workoh nm it's fine whew22:34
openstackgerritAdam Harwell proposed openstack/octavia master: Allow multiple VIPs per LB  https://review.opendev.org/66023922:46
*** goldyfruit_ has quit IRC22:50
*** tkajinam has joined #openstack-lbaas22:54
*** luksky has quit IRC23:58
« Sunday, 2019-09-15 Index Tuesday, 2019-09-17 »

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!