Thursday, 2019-09-19

openstackgerritMerged openstack/octavia master: Fix the tips job for octavia-lib  https://review.opendev.org/68214800:04
openstackgerritMerged openstack/octavia master: Fix the amphora no-op driver  https://review.opendev.org/68242000:17
*** gthiemonge has quit IRC00:57
*** gthiemonge has joined #openstack-lbaas00:57
*** yamamoto has joined #openstack-lbaas01:30
openstackgerritMerged openstack/octavia-dashboard master: Use lower-constraints template  https://review.opendev.org/68220101:45
*** ricolin has joined #openstack-lbaas02:22
*** yamamoto has quit IRC02:33
*** yamamoto has joined #openstack-lbaas03:05
johnsomFinally! Got the awk environment variable thing working... Now to actually fix the plugin for IPv6.03:05
johnsomhttps://usercontent.irccloud-cdn.com/file/GWw31mjL/image.png03:08
johnsomWorth doing the free certification if you want to learn more about IPv603:09
cjloaderjohnsom: hi03:24
johnsomcjloader Cam! how are you?03:25
cjloaderI'm holding up, you?03:25
johnsomcjloader I am well. Just about to go on vacation for a few weeks. So better every day!03:26
cjloaderNice. What are you up to these days?03:26
johnsomcjloader I am with Red Hat working on Octavia.03:27
johnsomSo, same story, new boss. lol03:27
cjloaderAwesome!! Glad you are still working on Octavia.03:27
*** psachin has joined #openstack-lbaas03:34
rm_workah yeah that might be worth doing, lol04:11
openstackgerritAdam Harwell proposed openstack/octavia master: Validate server_certs_key_passphrase is 32 chars  https://review.opendev.org/67073504:12
openstackgerritAdam Harwell proposed openstack/octavia master: Fix the diskimage-create tox "build" environment  https://review.opendev.org/67935804:12
rm_work^^ double rebase04:12
johnsomk04:12
rm_workand +A'd agaian04:12
rm_workthose two keep failing some random gate >_<04:13
rm_workif i had the ability to just +2 the merge, i would04:13
johnsomI raised that the DNS issue is back for infra04:13
rm_workbecause it's always some random other one04:13
rm_work*random other test that fails04:13
rm_workor times out for no good reason04:13
rm_workbut we're doing good on merges04:13
rm_workanything recently merge that needs backports?04:14
johnsomNo idea04:14
rm_workthese +1 backport on everything from the bots is making it difficult04:14
johnsomI have been a bit "focused" on stuff recently04:14
rm_worki'm serious about a new bucket04:14
rm_workI think i'm going to really propose it04:14
johnsomOk04:14
johnsomI won't fight it04:14
rm_workwhere's the code for that?04:14
rm_workis it in our project?04:14
rm_workor is it proposed to a different thing04:14
johnsomNo, it is in project-config04:15
rm_workk04:15
johnsomsearch for me in there, the last patch should be the backport patch04:16
*** gcheresh has joined #openstack-lbaas04:16
*** AustinR has quit IRC04:16
rm_work"A-Bot-Catcher"?04:17
rm_workyeah i found the spot04:17
johnsomFinally got the devstack plugin working, now we hit an odd oslo config issue. deep into netaddr package now... sigh04:19
*** gcheresh has quit IRC04:20
rm_workjohnsom: https://review.opendev.org/#/c/683028/04:29
*** AustinR has joined #openstack-lbaas04:30
johnsomrm_work +1 from me04:32
*** gcheresh has joined #openstack-lbaas04:38
*** pcaruana has joined #openstack-lbaas04:42
*** gcheresh has quit IRC04:42
*** ramishra has joined #openstack-lbaas04:43
eanderssonjohnsom, what is the issue?04:44
eanderssonI was fighting issues with netaddr not parsing [] properly04:45
johnsomeandersson https://zuul.opendev.org/t/openstack/build/ee5c6cc1ced24e4a9b88767791b87f52/log/controller/logs/devstacklog.txt.gz#2587204:45
johnsomNot fully convinced it is netaddr, but... in the neighborhood04:46
eanderssonYep same issue04:46
rm_workhmmm04:46
rm_workinteresting04:46
johnsomSomehow loopback is not valid....  <_<04:46
eandersson[] is not valid04:46
eanderssononly valid for uris04:46
eanderssoncheck for example MYSQL_IP in devstack04:47
eanderssonhttps://review.opendev.org/#/c/678321/04:47
rm_workah so it should just be `::1`?04:47
eanderssonYea04:47
eanderssonif it's an uri it can be wrapped with [], if not it can't04:47
johnsomWell, technically, [::1] is 100% valid.04:47
rm_workoh04:48
rm_workdoes ipv6_unquote work?04:48
eanderssonYea - but things like mysql, bind etc does not handle it04:48
eanderssonyep04:48
johnsomOtherwise ::1:8080 is ....04:48
eanderssonyea04:48
eanderssonlike pdns config can't handle it04:48
eanderssonbut pdns database needs it because of exactly that04:48
rm_workthat's kinda dumb04:48
rm_workbut ok, the fix is to ipv6_unquote <_<04:49
eanderssonit's real dumb04:49
johnsomYou designate PTLs and your pdns04:49
eanderssongotta love pdns :D04:49
johnsomI am a bit old school with bind. I have a patch or two in there....04:49
eanderssonI had a conversation about it with the oslo devs btw on irc04:50
eanderssonwai according to them (witht he oslo validation steps)04:50
eanderssonwas very confusing04:50
johnsomHmm, so maybe we have to hack $SERVICE_HOST from devstack....04:53
johnsomI think I am about done tonight. I will deal with this fun tomorrow04:53
eanderssonor just use ipv6_unquote ?04:54
johnsomYeah, so, hack it04:54
johnsomgrin04:54
eandersson:D04:54
johnsomSorry, my poor code BS meter is already a bit high...  lol  Probably a lack of sleep.04:55
*** rcernin_ has joined #openstack-lbaas04:56
openstackgerritMichael Johnson proposed openstack/octavia master: Allow IPv6 health network in devstack  https://review.opendev.org/66510304:57
*** rcernin has quit IRC04:59
*** AustinR has quit IRC05:03
*** ajay33 has joined #openstack-lbaas05:07
*** AustinR has joined #openstack-lbaas05:10
*** AustinR has quit IRC05:12
*** AustinR has joined #openstack-lbaas05:13
*** gcheresh has joined #openstack-lbaas05:19
*** AustinR has quit IRC05:32
*** AustinR has joined #openstack-lbaas05:33
rm_workerr can you do it that way?05:34
rm_work`ipv6_unquote($SERVICE_HOST)`?05:34
rm_workI thought it'd be:05:34
rm_work$(ipv6_unquote $SERVICE_HOST)05:34
rm_worki guess i'll wait to see if it passes and then fix it because i just tested that and it doesn't work for me :(05:36
rm_workerr well it passed but i think it has always been passing... need to find where to prove that worked or not05:45
johnsomWas there yet another zuul restart that hosed everyone's jobs?06:38
johnsomI rechecked, but no output, no job in zuul06:38
johnsomYep, sigh, so tired of jobs getting random killed06:40
rm_workT_T06:41
rm_workjohnsom: so how do you tell if this works or not06:41
rm_workjohnsom: i think you need to change it to what i put above?06:42
johnsomI think it *should* work, but I did just off the fly throw that in....06:42
johnsomYou might be right....06:43
johnsomI was watching the AGT finale when I pushed that06:43
johnsomBut we should have had an answer hours ago....06:44
*** rcernin_ has quit IRC06:48
*** takamatsu has quit IRC06:50
*** psachin has quit IRC06:53
rm_worki thought it ... finished?07:04
rm_workjohnsom: it seems like it finished07:04
*** rpittau|afk is now known as rpittau07:05
rm_worki see the +1 from zuul... but it's been +1s every time so...07:05
johnsomNon. voting07:05
johnsomI didn’t see any result on the last run07:05
rm_workah07:08
rm_workhmm07:08
rm_workwell, the tempest patch in the gate right now failed with:07:08
rm_work2019-09-19 06:53:20.370 | ERROR: Could not install packages due to an EnvironmentError: HTTPConnectionPool(host='mirror.ord.rax.opendev.org', port=80): Max retries exceeded with url: /pypifiles/packages/30/db/9e38760b32e3e7f40cce46dd5fb107b8c73840df38f0046d8e6514e675a1/pip-19.2.3-py2.py3-none-any.whl (Caused by ProtocolError('Connection aborted.', BadStatusLine('No status line received - the server has closed the07:08
rm_workconnection',)))07:08
rm_workalso what am i missing here07:09
rm_workare we looking at different patches?07:10
rm_workhttps://i.imgur.com/0heOKK6.png07:10
rm_workwhat other job am i supposed to see?07:11
cgoncalveshttps://review.opendev.org/#/c/682726/07:11
rm_workAH, rechecking that every time?07:11
rm_workok07:11
rm_workthat makes way more sense07:12
cgoncalveshmm, I co-authored that patch?! I can't take credits for something I didn't do07:12
rm_worklol07:13
rm_workhe added both of us in PS307:14
rm_workwonder why lol07:14
rm_workit was too long ago, maybe collab on IRC? <_<07:14
cgoncalvesis https://review.opendev.org/#/c/594078 still relevant?07:14
rm_workunclear07:21
rm_workmaybe that's why?07:21
rm_workpossible he just ported our work into that one07:21
* rm_work shrugs07:21
rm_worktho it doesn't have any of the test_base.py stuff07:22
rm_workso octavia will set up everything in the test with ipv4 <_<07:22
johnsom??? We test with ipv6 by default07:24
johnsomThat other tempest patch is ... questionable imo07:25
*** tesseract has joined #openstack-lbaas07:26
johnsomWell, I should say we teat mixed by default07:26
johnsomOk, really sleeping now. Super important meeting in 5 hours. Lol, probably not...07:27
johnsom80% chance it will be cancelled by the time I get up for it07:28
cgoncalveslikely yes07:29
*** luksky has joined #openstack-lbaas07:37
*** takamatsu has joined #openstack-lbaas07:44
*** mjozefcz has joined #openstack-lbaas07:45
*** ivve has joined #openstack-lbaas07:45
rm_workcgoncalves: i'm gonna crash, can you recheck https://review.opendev.org/682240 when it finishes failing? lol07:52
cgoncalvesyep07:55
rm_worksweet thx07:58
*** yamamoto has quit IRC08:13
*** tkajinam has quit IRC08:19
openstackgerritMerged openstack/octavia master: Fix the diskimage-create tox "build" environment  https://review.opendev.org/67935808:45
*** takamatsu_ has joined #openstack-lbaas08:46
*** takamatsu has quit IRC08:46
openstackgerritMerged openstack/octavia master: Validate server_certs_key_passphrase is 32 chars  https://review.opendev.org/67073509:03
*** yamamoto has joined #openstack-lbaas09:06
openstackgerritCarlos Goncalves proposed openstack/octavia stable/stein: Validate server_certs_key_passphrase is 32 chars  https://review.opendev.org/68307009:15
openstackgerritCarlos Goncalves proposed openstack/octavia stable/rocky: Validate server_certs_key_passphrase is 32 chars  https://review.opendev.org/68307109:15
openstackgerritCarlos Goncalves proposed openstack/octavia stable/queens: Validate server_certs_key_passphrase is 32 chars  https://review.opendev.org/68307209:15
*** takamatsu_ has quit IRC09:19
*** gcheresh has quit IRC09:26
*** gcheresh has joined #openstack-lbaas09:26
*** yamamoto has quit IRC09:32
*** takamatsu has joined #openstack-lbaas09:36
*** yamamoto has joined #openstack-lbaas09:38
openstackgerritCarlos Goncalves proposed openstack/octavia stable/stein: Fix health monitor API handling of None updates  https://review.opendev.org/68309009:55
openstackgerritCarlos Goncalves proposed openstack/octavia stable/stein: Fix member API handling of None/null updates  https://review.opendev.org/68309209:55
openstackgerritCarlos Goncalves proposed openstack/octavia stable/rocky: Fix member API handling of None/null updates  https://review.opendev.org/68309309:56
openstackgerritCarlos Goncalves proposed openstack/octavia stable/stein: Fix pool API handling of None/null updates  https://review.opendev.org/68309409:56
*** takamatsu has quit IRC10:00
openstackgerritMaciej Józefczyk proposed openstack/octavia master: Validate supported LB algorithm in Amphora provider drivers  https://review.opendev.org/67247710:06
*** takamatsu has joined #openstack-lbaas10:06
*** yamamoto has quit IRC10:10
mjozefczcgoncalves++ thanks for review10:11
*** yamamoto has joined #openstack-lbaas10:12
*** rcernin_ has joined #openstack-lbaas10:15
*** trident has quit IRC10:35
*** rcernin_ has quit IRC10:36
*** sapd1_x has joined #openstack-lbaas10:37
openstackgerritAnn Taraday proposed openstack/octavia master: Convert listener flows to use provider models  https://review.opendev.org/66023610:38
*** trident has joined #openstack-lbaas10:48
*** pcaruana has quit IRC10:54
*** rpittau is now known as rpittau|afk10:56
*** takamatsu has quit IRC11:00
*** takamatsu has joined #openstack-lbaas11:06
*** yamamoto has quit IRC11:13
*** pcaruana has joined #openstack-lbaas11:17
*** yamamoto has joined #openstack-lbaas11:17
*** takamatsu has quit IRC11:22
*** takamatsu has joined #openstack-lbaas11:28
*** sapd1_x has quit IRC11:38
*** boden has joined #openstack-lbaas11:41
*** yamamoto has quit IRC11:42
*** yamamoto has joined #openstack-lbaas11:46
*** yamamoto has quit IRC11:57
*** ajay33 has quit IRC11:58
*** yamamoto has joined #openstack-lbaas12:00
*** ricolin has quit IRC12:06
*** goldyfruit_ has quit IRC12:11
*** takamatsu has quit IRC12:21
*** takamatsu has joined #openstack-lbaas12:27
*** eandersson has quit IRC12:31
*** yamamoto has quit IRC12:33
*** takamatsu has quit IRC12:38
openstackgerritMerged openstack/octavia-tempest-plugin master: Associate floating IP after lb ACTIVE  https://review.opendev.org/68224012:46
openstackgerritMichael Johnson proposed openstack/octavia master: Allow IPv6 health network in devstack  https://review.opendev.org/66510313:07
*** takamatsu has joined #openstack-lbaas13:16
*** yamamoto has joined #openstack-lbaas13:16
*** goldyfruit_ has joined #openstack-lbaas13:16
*** yamamoto_ has joined #openstack-lbaas13:17
*** AustinR has quit IRC13:20
*** yamamoto has quit IRC13:21
*** yamamoto_ has quit IRC13:24
*** pcaruana has quit IRC13:28
*** AustinR has joined #openstack-lbaas13:36
*** yamamoto has joined #openstack-lbaas13:43
*** pcaruana has joined #openstack-lbaas13:54
*** tesseract has quit IRC14:00
*** reneluria has quit IRC14:02
*** Vorrtex has joined #openstack-lbaas14:04
cgoncalveshttps://github.com/openstack/octavia/blob/stable/queens/octavia/tests/functional/api/v2/test_l7rule.py#L596-L60514:05
cgoncalvesoops! :)14:06
johnsomopps?14:07
cgoncalvesL606 should assert for /images14:08
johnsomAh, yeah, lol14:08
cgoncalvessorry, L60514:08
*** ivve has quit IRC14:09
johnsomThat gremlin is three years old.... lol14:09
*** goldyfruit___ has joined #openstack-lbaas14:15
*** takamatsu has quit IRC14:16
*** goldyfruit_ has quit IRC14:18
openstackgerritGregory Thiemonge proposed openstack/octavia-tempest-plugin master: Fix spare_pool_enabled option type  https://review.opendev.org/68316814:23
*** pcaruana has quit IRC14:33
openstackgerritAustin Russell proposed openstack/octavia master: loadbalancer vip-network-id IP availability check  https://review.opendev.org/67335814:36
*** yamamoto has quit IRC14:59
*** yamamoto has joined #openstack-lbaas15:05
*** yamamoto has quit IRC15:10
*** gcheresh has quit IRC15:17
*** ricolin has joined #openstack-lbaas15:21
*** ajay33 has joined #openstack-lbaas15:21
openstackgerritMichael Johnson proposed openstack/octavia master: loadbalancer vip-network-id IP availability check  https://review.opendev.org/67335815:26
openstackgerritGregory Thiemonge proposed openstack/octavia-tempest-plugin master: Fix OpenStack clients region parameter  https://review.opendev.org/68318515:30
*** mjozefcz has quit IRC15:52
*** openstackgerrit has quit IRC16:06
*** takamatsu has joined #openstack-lbaas16:06
*** openstackgerrit has joined #openstack-lbaas16:22
openstackgerritMerged openstack/octavia stable/rocky: Work around strptime threading issue  https://review.opendev.org/68295016:22
openstackgerritMerged openstack/octavia stable/stein: Work around strptime threading issue  https://review.opendev.org/68294916:22
dulekcgoncalves, johnsom: Hey, can you outline me how health monitor works? Is it just setting HAProxy in the Amp to have some healthchecking?16:22
dulekOkay, maybe some further question - in case of HTTPS healthchecks - how do the certs work?16:29
johnsomThe heatlh monitors are, for the most part, driven by the HAProxy engine yes.16:32
johnsomAs of stein, if the member is TLS enabled, the health monitor will use the certificate configured for the member. There are two other TLS health checks that are legacy, HTTPS which is basically a TCP port handshake and TLS-HELLO which will do a TLS handshake on the port, but not use the certificate.16:34
openstackgerritMichael Johnson proposed openstack/neutron-lbaas stable/queens: Revert "Updated "create_pool" method in plugin"  https://review.opendev.org/64017516:37
*** nmagnezi has quit IRC16:41
*** pcaruana has joined #openstack-lbaas17:01
*** pcaruana has quit IRC17:14
*** gmann_afk is now known as gmann17:21
*** ramishra has quit IRC17:23
*** pcaruana has joined #openstack-lbaas17:25
johnsomNice, the ipv6 job just finished devstack. If tempest passes I will clean those up and we are good to go for the IPv6 goal and an IPv6 lb-mgmt-net test.17:26
cgoncalvessweet!17:28
*** ricolin has quit IRC17:31
johnsomI think the PDF patches are probably good enough for a first pass as well. The octavia-lib is now fixed with the "tips" patch.17:32
*** pcaruana has quit IRC17:55
*** trident has quit IRC18:45
cgoncalvesthe PDF doc is 836 pages :O18:53
*** trident has joined #openstack-lbaas18:55
*** nmagnezi has joined #openstack-lbaas19:01
*** trident has quit IRC19:01
openstackgerritMerged openstack/octavia stable/rocky: Fix base (VRRP) port abandoned on revert  https://review.opendev.org/68283619:02
*** trident has joined #openstack-lbaas19:10
johnsomYeah, and sections are missing still19:23
*** yamamoto has joined #openstack-lbaas19:49
*** boden has quit IRC19:49
openstackgerritMerged openstack/octavia master: Fix building configs for multiple listeners  https://review.opendev.org/68119519:51
*** pcaruana has joined #openstack-lbaas19:51
*** yamamoto has quit IRC19:53
*** gcheresh has joined #openstack-lbaas19:54
*** ajay33 has quit IRC19:54
*** gcheresh has quit IRC20:00
*** AustinR has quit IRC20:07
openstackgerritMichael Johnson proposed openstack/octavia master: Generate PDF documentation  https://review.opendev.org/66724920:12
*** AustinR has joined #openstack-lbaas20:19
*** yamamoto has joined #openstack-lbaas20:31
*** pcaruana has quit IRC20:31
*** yamamoto has quit IRC20:36
johnsomSo it sounds like the release team will create proposed RC1 patches for you next week:20:36
johnsomEarly in the week, the release team will be proposing RC1 patches for all cycle-with-rc projects, using the latest commit from master. If your team is ready to go for cutting RC1, please let us know by leaving a +1 on these patches.20:36
rm_workkk20:39
*** Vorrtex has quit IRC20:58
*** yamamoto has joined #openstack-lbaas21:20
*** yamamoto has quit IRC21:32
*** yamamoto has joined #openstack-lbaas21:35
*** luksky has quit IRC21:41
*** yamamoto has quit IRC21:44
*** yamamoto has joined #openstack-lbaas21:44
openstackgerritMerged openstack/octavia stable/rocky: Validate server_certs_key_passphrase is 32 chars  https://review.opendev.org/68307122:04
openstackgerritMerged openstack/octavia stable/queens: Validate server_certs_key_passphrase is 32 chars  https://review.opendev.org/68307222:04
openstackgerritMerged openstack/octavia stable/stein: Validate server_certs_key_passphrase is 32 chars  https://review.opendev.org/68307022:04
johnsomlol22:05
rm_work:D22:05
*** yamamoto has quit IRC22:57
*** goldyfruit___ has quit IRC23:00
*** tkajinam has joined #openstack-lbaas23:02
*** yamamoto has joined #openstack-lbaas23:03
*** yamamoto has quit IRC23:08
*** yamamoto has joined #openstack-lbaas23:08
openstackgerritAustin Russell proposed openstack/octavia master: loadbalancer vip-network-id IP availability check  https://review.opendev.org/67335823:10
*** yamamoto has quit IRC23:12
*** yamamoto has joined #openstack-lbaas23:12
openstackgerritMichael Johnson proposed openstack/octavia master: Improve the error message for bad pkcs12 bundles  https://review.opendev.org/68325423:19
johnsomrm_work ^^^ applying a bit of lipstick23:20
*** rcernin has joined #openstack-lbaas23:25
*** yamamoto has quit IRC23:26
*** yamamoto has joined #openstack-lbaas23:28
*** yamamoto has quit IRC23:31
*** yamamoto has joined #openstack-lbaas23:31
*** yamamoto has quit IRC23:39
johnsomWow, ok... found a bug in the version of mock we are using....23:45
johnsomnow that sucks for backport....23:45
*** yamamoto has joined #openstack-lbaas23:46
*** yamamoto has quit IRC23:48
*** yamamoto has joined #openstack-lbaas23:49
rm_workT_T23:50
rm_workWhat's the 5 second rundown on that bug?23:50
johnsomI think I can do something cheesy to work around it23:50
johnsomWell, when you use "wraps" in a mock it just doesn't work on version 2.0.023:51
johnsomLook at patch 683254 in zuul23:52
johnsomunit/functionals pass. lower doesn't. It returned a mock object instead of actually running the code23:52
johnsomAppears to have been issue #35330 fixed in 3.0.023:53
johnsomYeah, 3.0.0 works fine23:53
johnsomThere were no releases between them23:54
*** eandersson has joined #openstack-lbaas23:55
eanderssonrm_work, yea I think you need to do > $(ipv6_unquote $SERVICE_HOST)23:56
johnsomeandersson Back seat coding.... BTW, I fixed that this morning23:56
eanderssonIt's the best :p23:56
johnsomNow that patch has a few real issues. HM won't bind to the IPv6 (probably the inverse bracket  thing) and we can't seem to reach the amps. I'm guessing missing route/not picking up route advertisements.23:58
johnsomThose I can work on local, just haven't got back to it yet today23:58
eanderssonYea - I had to handle that in code for pdns23:58
eanderssonSuper stupid23:58
eanderssonbrackets -> unbrackets -> brackets23:58
johnsomyep23:58
eanderssonIt was an esay decision for me at least, as 90% of the components didn't support brackets23:59

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