Tuesday, 2019-07-23

openstackgerritMerged openstack/octavia master: Add cloud-init-datasources DIB element  https://review.opendev.org/67101600:01
*** mithilarun has quit IRC00:01
*** mithilarun has joined #openstack-lbaas00:02
*** mithilarun has quit IRC00:04
*** mithilarun has joined #openstack-lbaas00:04
*** tkajinam has quit IRC00:10
*** mithilarun has quit IRC00:17
*** Vorrtex has quit IRC00:20
*** mithilarun has joined #openstack-lbaas00:30
*** mithilarun has quit IRC00:34
*** mithilarun has joined #openstack-lbaas00:38
*** hongbin has joined #openstack-lbaas00:55
*** ccamposr has quit IRC00:55
*** ccamposr has joined #openstack-lbaas00:55
*** ccamposr has quit IRC01:04
*** ccamposr has joined #openstack-lbaas01:04
*** ricolin has joined #openstack-lbaas01:19
*** mithilarun has quit IRC01:29
*** mithilarun has joined #openstack-lbaas01:32
*** yamamoto has quit IRC01:34
*** mithilarun has quit IRC01:42
*** yamamoto has joined #openstack-lbaas01:58
*** yamamoto has quit IRC02:17
*** yamamoto has joined #openstack-lbaas02:21
*** gcheresh_ has joined #openstack-lbaas02:43
*** gcheresh_ has quit IRC03:20
*** psachin has joined #openstack-lbaas03:28
*** hongbin has quit IRC03:45
*** hongbin has joined #openstack-lbaas03:50
*** hongbin has quit IRC03:50
*** rcernin has quit IRC04:13
*** rcernin has joined #openstack-lbaas04:14
*** rcernin has quit IRC04:20
*** gcheresh_ has joined #openstack-lbaas04:22
*** gcheresh_ has quit IRC04:27
*** pcaruana has joined #openstack-lbaas04:43
*** ramishra has quit IRC04:54
*** maciejjozefczyk has joined #openstack-lbaas05:27
*** ramishra has joined #openstack-lbaas05:29
*** ccamposr has quit IRC06:03
*** ccamposr has joined #openstack-lbaas06:05
*** ccamposr has quit IRC06:11
*** ccamposr has joined #openstack-lbaas06:12
openstackgerritAdam Harwell proposed openstack/octavia-dashboard stable/queens: Update tox.ini for new upper constraints strategy  https://review.opendev.org/66826106:37
rm_workcgoncalves: good catch on that one, the rest of them were already fine, and all have my +2 now, so...06:39
rm_workcgoncalves: if you tag those we could knock like 7 patches of the review list :D06:40
cgoncalves+W'd it06:45
openstackgerritMerged openstack/python-octaviaclient master: Update tox.ini for new upper constraints strategy  https://review.opendev.org/66826406:47
*** gcheresh_ has joined #openstack-lbaas06:49
rm_workok so there's 6 more :D06:58
rm_workare you working now or did you just pop in?06:58
*** yamamoto has quit IRC07:05
openstackgerritMerged openstack/octavia-dashboard stable/queens: Update tox.ini for new upper constraints strategy  https://review.opendev.org/66826107:05
*** yamamoto has joined #openstack-lbaas07:06
*** tesseract has joined #openstack-lbaas07:09
*** rpittau|afk is now known as rpittau07:09
*** maciejjozefczyk_ has joined #openstack-lbaas07:20
*** irclogbot_3 has quit IRC07:20
*** openstackstatus has quit IRC07:20
*** nmagnezi has quit IRC07:21
*** irclogbot_0 has joined #openstack-lbaas07:21
*** maciejjozefczyk has quit IRC07:23
*** maciejjozefczyk_ is now known as maciejjozefczyk07:23
openstackgerritGregory Thiemonge proposed openstack/octavia master: Fix provisioning_state after a failover of a listener in ERROR  https://review.opendev.org/67173907:49
cgoncalvesrm_work, I am starting my day07:51
rm_workah, k07:52
cgoncalvesrm_work, what are the other 6 you're referring to?07:52
rm_workhttps://review.opendev.org/#/q/topic:constraints-updates+status:open07:53
rm_work5 i guess07:53
rm_workah soon 407:53
cgoncalvesah, I missed those. I was doing some bedside reviewing yesterday night xD07:54
rm_workyeah, and you happened to catch the ONE that had that issue, lol07:54
rm_workI went through the others again a minute ago but they are all good07:54
rm_workI missed that since I started skimming after the 3rd one or so, and apparently that was the only instance where he missed stuff07:55
cgoncalvesall approved07:55
*** nmagnezi has joined #openstack-lbaas07:56
rm_worksweet, will cull those from the list07:59
rm_workugh, need to start looking at those "client unset" patches08:01
openstackgerritMerged openstack/octavia-dashboard stable/rocky: Update tox.ini for new upper constraints strategy  https://review.opendev.org/66826008:16
openstackgerritMerged openstack/neutron-lbaas-dashboard stable/queens: Update tox.ini for new upper constraints strategy  https://review.opendev.org/66827108:17
openstackgerritMerged openstack/neutron-lbaas-dashboard stable/rocky: Update tox.ini for new upper constraints strategy  https://review.opendev.org/66827008:19
*** yamamoto has quit IRC08:20
*** ivve has joined #openstack-lbaas08:22
openstackgerritMerged openstack/octavia master: Update api-ref location  https://review.opendev.org/67210208:25
*** yamamoto has joined #openstack-lbaas08:50
*** ajay33 has joined #openstack-lbaas08:51
*** sapd1_x has joined #openstack-lbaas08:55
*** ccamposr__ has joined #openstack-lbaas08:56
*** ccamposr has quit IRC08:59
*** yamamoto has quit IRC09:02
*** psachin has quit IRC09:20
*** psachin has joined #openstack-lbaas09:35
openstackgerritMerged openstack/octavia stable/rocky: Update amphora-agent to report UDP listener health  https://review.opendev.org/66568309:46
*** yamamoto has joined #openstack-lbaas10:27
*** yamamoto has quit IRC10:28
*** yamamoto has joined #openstack-lbaas10:28
*** ricolin_ has joined #openstack-lbaas10:34
*** ricolin has quit IRC10:37
*** luksky11 has joined #openstack-lbaas10:47
*** yamamoto has quit IRC10:48
*** luksky123 has joined #openstack-lbaas10:48
*** luksky11 has quit IRC10:52
*** yamamoto has joined #openstack-lbaas10:57
*** yamamoto has quit IRC11:02
*** yamamoto has joined #openstack-lbaas11:05
openstackgerritMaciej Józefczyk proposed openstack/octavia-tempest-plugin master: Add option to reuse connections and skip spread validation for OVN  https://review.opendev.org/67226411:08
*** spatel has joined #openstack-lbaas11:20
*** spatel has quit IRC11:25
*** ccamposr__ has quit IRC11:26
maciejjozefczykcgoncalves, hey! Thanks for the review :) I'm going to recover this patch: https://review.opendev.org/#/c/660369 to address the LB algorithm name change, and then I'll add its support to the API11:27
maciejjozefczykcgoncalves, it will make things clear in tempest then, why we modify the tests11:28
maciejjozefczykcgoncalves, is this ok to do it this way?11:28
cgoncalvessure11:42
*** boden has joined #openstack-lbaas11:51
*** yamamoto has quit IRC12:03
*** ricolin_ is now known as ricolin12:05
*** yamamoto has joined #openstack-lbaas12:08
maciejjozefczykcgoncalves, thx12:24
*** goldyfruit has quit IRC12:26
openstackgerritMaciej Józefczyk proposed openstack/octavia-lib master: Add new LB Algorithm - SOURCE_IP_PORT  https://review.opendev.org/67227512:27
*** henriqueof has joined #openstack-lbaas12:33
*** luksky123 has quit IRC12:39
*** openstackstatus has joined #openstack-lbaas13:21
*** ChanServ sets mode: +v openstackstatus13:21
openstackgerritAnn Taraday proposed openstack/octavia master: [WIP] Lb flows to dicts  https://review.opendev.org/67172513:24
*** goldyfruit has joined #openstack-lbaas13:35
*** lemko has joined #openstack-lbaas13:38
*** yamamoto has quit IRC13:53
*** yamamoto has joined #openstack-lbaas13:54
*** yamamoto has quit IRC13:58
*** rpittau is now known as rpittau|afk14:03
*** Vorrtex has joined #openstack-lbaas14:09
*** ccamposr has joined #openstack-lbaas14:09
*** yamamoto has joined #openstack-lbaas14:37
*** dosaboy has joined #openstack-lbaas14:41
*** yamamoto has quit IRC14:43
*** gcheresh_ has quit IRC15:09
*** trident has quit IRC15:18
*** trident has joined #openstack-lbaas15:20
*** boden has quit IRC15:22
*** boden has joined #openstack-lbaas15:24
openstackgerritGregory Thiemonge proposed openstack/octavia-dashboard master: Added UDP support for listener and health-monitor  https://review.opendev.org/67231915:37
*** sapd1_x has quit IRC15:44
*** trident has quit IRC15:52
dulekcgoncalves, johnsom: Regarding our yesterday's discussions of Kuryr using single Octavia LB to serve multiple K8s Services…15:55
*** trident has joined #openstack-lbaas15:55
dulekHow can it work with K8s arbitrarily assigning IP's to Services?15:56
dulekCan I have multiple IP's attached to one Amp?15:56
cgoncalvesdulek, https://review.opendev.org/#/c/660239/15:56
dulekcgoncalves: Whoa, recently things are changing pretty fast in OpenStack loadbalancing world. :D15:57
johnsomdulek Do you need multiple IPs? You can't ask k8s to just assign ports?15:57
dulekjohnsom: What do you mean by "assign ports"? (seems you're right about terminology differences)15:58
johnsomYep, grin. I meant TCP ports.16:04
*** tesseract has quit IRC16:07
*** lemko has quit IRC16:08
johnsomI guess I'm still not clear on the inner workings as I thought that IP came from our code and the IP on the LB.16:09
*** ajay33 has quit IRC16:26
*** ricolin has quit IRC16:34
*** yamamoto has joined #openstack-lbaas16:43
*** yamamoto has quit IRC16:48
*** yamamoto has joined #openstack-lbaas16:49
*** trident has quit IRC17:07
*** trident has joined #openstack-lbaas17:10
openstackgerritCarlos Goncalves proposed openstack/octavia master: DNM: test centos-minimal++  https://review.opendev.org/67233217:11
*** yamamoto has quit IRC17:11
*** yamamoto has joined #openstack-lbaas17:13
*** yamamoto has quit IRC17:18
*** ramishra has quit IRC17:24
openstackgerritCarlos Goncalves proposed openstack/octavia master: DNM: test centos-minimal++  https://review.opendev.org/67233217:33
dulekjohnsom: Sorry, got dragged to another stuff and forgot to answer. Basically for each Service K8s will reserve one IP from the service network that this Service should be accessible on.17:38
dulekAnd that's why creating one LB per such IP is so convenient.17:38
dulekAh, and Kuryr does not have any saying about that IP - this is arbitrarily assigned by K8s on per-Service basis, so we cannot really do anything to modify it.17:40
johnsomSo you take that and use it as the fixed IP when provisioning the LB?17:40
dulekjohnsom: Yup! And that's why this trick with allocation ranges works - for VRRP that comes from the allocation range that is outside from range used by K8s.17:41
dulekEven though in OpenStack both are on the same Neutron subnet.17:41
johnsomIt seems that services can be multi-port and you can specify your own IP address: https://kubernetes.io/docs/concepts/services-networking/service/#multi-port-services17:48
*** psachin has quit IRC18:19
colin-cgoncalves: i like how "minimal" ~918MB heh18:24
colin-got excited for a moment there thinking it was a truly small image18:24
colin-last i checked the ubuntu equivalent was still ~150M18:28
johnsomThe current state: http://tarballs.openstack.org/octavia/test-images/18:29
colin-how much does the complied amphora agent take up?18:31
colin-sorry for lazy question lol18:31
colin-s/complied/built/18:32
johnsomNot that much really, but the python environment is pretty large18:32
colin-ah, duh, that makes sense18:32
colin-what state are those test images in? base or with https://github.com/openstack/octavia/tree/master/diskimage-create already run?18:54
johnsomThose are the output of diskimage-create18:55
*** goldyfruit has quit IRC19:02
*** goldyfruit has joined #openstack-lbaas19:06
johnsomWell, VScode fails miserably at renaming objects.19:31
johnsomGoing to give pycharm a try19:31
johnsomMuch better19:38
*** henriqueof has quit IRC19:40
colin-so complicated, what's wrong with a good sed expression in vim :)19:41
johnsomYeah, I almost went that way, but this is a *slightly* large rename, so thought I would try the magic tools19:41
johnsomLooks like pycharm got everything except the mock.patch lines19:45
*** maciejjozefczyk has quit IRC19:52
openstackgerritMichael Johnson proposed openstack/octavia master: Add missing test requirement "fixtures"  https://review.opendev.org/67234919:53
*** Vorrtex has quit IRC20:28
cgoncalvescolin-, 918MB? not sure how you got to that. I have a 405MB amp with the centos-minimal++ patch20:37
cgoncalveshttp://logs.openstack.org/32/672332/2/check/octavia-v2-dsvm-py2-scenario-centos-7/43fb9e6/controller/logs/devstacklog.txt.gz#_2019-07-23_18_26_42_11720:39
cgoncalvescolin-, I built a centos-minimal image (non-amphora one) with my recent DIB patches taking only 234M20:41
rm_workdulek: yeah I was working on multivip specifically to help with senlin issues around dual-stack (IPv4/v6) LBs, though if it helps with the k8s usecase I should make sure I have it totally right for that too20:45
*** pcaruana has quit IRC20:50
cgoncalves-rw-r--r--. 1 cgoncalves cgoncalves 348M Jul 23 23:24 amphora-x64-haproxy.qcow221:26
cgoncalvescentos-minimal + https://review.opendev.org/#/c/639155/ + rm -rf /opt/amphora-agent21:26
*** boden has quit IRC21:27
*** mithilarun has joined #openstack-lbaas21:27
openstackgerritMichael Johnson proposed openstack/octavia master: Fix multi-listener load balancers  https://review.opendev.org/66806821:28
*** altlogbot_0 has quit IRC21:33
*** irclogbot_0 has quit IRC21:33
*** altlogbot_0 has joined #openstack-lbaas21:34
*** irclogbot_0 has joined #openstack-lbaas21:34
*** mithilarun has quit IRC21:45
*** mithilarun has joined #openstack-lbaas21:45
*** yamamoto has joined #openstack-lbaas21:50
openstackgerritMichael Johnson proposed openstack/octavia master: Make Ubuntu bionic the default for amphora images  https://review.opendev.org/65267921:52
*** yamamoto has quit IRC21:55
*** irclogbot_0 has quit IRC21:59
*** altlogbot_0 has quit IRC22:01
openstackgerritMichael Johnson proposed openstack/octavia master: Build amphora image using centos-minimal element  https://review.opendev.org/61230922:05
johnsomRebased to have the amp console logs22:06
*** altlogbot_0 has joined #openstack-lbaas22:22
*** altlogbot_0 has quit IRC22:27
*** goldyfruit has quit IRC22:30
*** goldyfruit has joined #openstack-lbaas22:30
*** goldyfruit has quit IRC22:38
*** mithilarun has quit IRC22:46
*** mithilarun has joined #openstack-lbaas22:46
*** mithilarun has quit IRC22:51
*** tkajinam has joined #openstack-lbaas22:51
rm_workcgoncalves: is that literally just an extra directory of code? >_>23:09
rm_workduplicated python env?23:09
*** altlogbot_2 has joined #openstack-lbaas23:14
*** rcernin has joined #openstack-lbaas23:16
*** altlogbot_2 has quit IRC23:19
*** mithilarun has joined #openstack-lbaas23:27
*** altlogbot_2 has joined #openstack-lbaas23:28
*** irclogbot_1 has joined #openstack-lbaas23:32
*** mithilarun has quit IRC23:42
johnsomYeah, it's where the venv installs from I think.  It's kind of handy to have as it has the git log / commit message, but otherwise not completely necessary.23:58

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