Thursday, 2019-10-24

*** sapd1 has joined #openstack-lbaas00:08
*** sapd1 has quit IRC00:16
*** yamamoto has quit IRC00:19
rm_workdid we not already add a "--long" to the amphora list to give more info? i thought that happened? guess not?00:56
johnsom{0} octavia_tempest_plugin.tests.barbican_scenario.v2.test_tls_barbican.TLSWithBarbicanTest.test_basic_tls_SNI_traffic [12.092657s] ... ok01:24
johnsomFinally. One of those days where debugging dumb mistakes takes longer than the code.01:24
johnsomI want to add another test case or two before I push the final version up. Should be done tomorrow.01:25
*** abaindur has quit IRC01:47
openstackgerritMichael Johnson proposed openstack/octavia-tempest-plugin master: Add TLS SNI scenario tests  https://review.opendev.org/69077801:48
johnsomOk, that should fail as it doesn't have the depends-on for my SNI patch.01:50
johnsomPlus it's not done.01:50
johnsomBut calling it a night.01:50
*** ricolin has joined #openstack-lbaas02:12
rm_worko/02:29
*** ajay33 has joined #openstack-lbaas03:07
*** psachin has joined #openstack-lbaas03:39
*** yamamoto has joined #openstack-lbaas04:05
*** yamamoto has quit IRC04:10
*** yamamoto has joined #openstack-lbaas04:50
*** tkajinam_ has joined #openstack-lbaas05:03
*** tkajinam has quit IRC05:05
*** gcheresh_ has joined #openstack-lbaas05:15
*** ajay33 has quit IRC05:16
openstackgerritMichael Johnson proposed openstack/octavia-tempest-plugin master: Add TLS SNI scenario tests  https://review.opendev.org/69077805:43
*** tkajinam__ has joined #openstack-lbaas05:52
*** tkajinam_ has quit IRC05:54
*** psachin has quit IRC05:58
*** psachin has joined #openstack-lbaas06:03
*** maciejjozefczyk has joined #openstack-lbaas06:14
openstackgerritpengyuesheng proposed openstack/octavia-lib master: Switch to Ussuri jobs  https://review.opendev.org/69083606:17
*** ccamposr has joined #openstack-lbaas06:29
*** tkajinam__ is now known as tkajinam06:34
*** pcaruana has joined #openstack-lbaas06:41
*** tesseract has joined #openstack-lbaas07:18
*** vishalmanchanda has joined #openstack-lbaas07:23
*** trident has quit IRC07:46
*** abaindur has joined #openstack-lbaas07:50
openstackgerritCarlos Goncalves proposed openstack/octavia-tempest-plugin master: Add irrelevant-files to active-standby jobs  https://review.opendev.org/69087407:50
*** trident has joined #openstack-lbaas07:51
*** rpittau|afk is now known as rpittau07:52
openstackgerritAnn Taraday proposed openstack/octavia master: Convert listener flows to use provider models  https://review.opendev.org/66023608:07
*** abaindur has quit IRC08:07
*** tkajinam has quit IRC08:19
openstackgerritAnn Taraday proposed openstack/octavia master: Convert listener flows to use provider models  https://review.opendev.org/66023608:20
openstackgerritAjay Kumar proposed openstack/octavia master: Fix typo in doc agent.py->agent  https://review.opendev.org/69088408:57
*** ajay33 has joined #openstack-lbaas08:58
*** yamamoto has quit IRC09:16
*** yamamoto has joined #openstack-lbaas09:22
*** yamamoto has quit IRC09:22
*** yamamoto has joined #openstack-lbaas09:22
*** yamamoto has quit IRC09:26
openstackgerritMaciej Józefczyk proposed openstack/octavia-tempest-plugin master: Handle NotImplemented flavor exception  https://review.opendev.org/67613509:41
*** gcheresh_ has quit IRC10:15
*** gcheresh_ has joined #openstack-lbaas10:16
*** yamamoto has joined #openstack-lbaas10:26
*** yamamoto has quit IRC10:30
*** yamamoto has joined #openstack-lbaas10:42
*** yamamoto has quit IRC10:54
*** yamamoto has joined #openstack-lbaas10:57
*** yamamoto has quit IRC11:12
openstackgerritAnn Taraday proposed openstack/octavia master: Convert pool flows to use dicts  https://review.opendev.org/66538111:12
openstackgerritAnn Taraday proposed openstack/octavia master: Transition member flows to use dicts  https://review.opendev.org/65784211:14
openstackgerritAnn Taraday proposed openstack/octavia master: Transition amphora flows to dicts  https://review.opendev.org/66889811:15
CobHeadHi everyone! Anyone know if it is possible to enable HSTS on a Octavia Load Balancer? :)11:21
CobHeadAnd also, has anyone experienced intermediate certificates to be sorted alphabetically when passed to Octavia?11:22
*** gcheresh_ has quit IRC11:23
openstackgerritDat Le proposed openstack/octavia master: Support to create amphora with customized metadata  https://review.opendev.org/69092511:24
*** gcheresh_ has joined #openstack-lbaas11:28
openstackgerritMaciej Józefczyk proposed openstack/octavia-tempest-plugin master: Add an option to reuse connections  https://review.opendev.org/67297611:29
*** yamamoto has joined #openstack-lbaas11:46
*** yamamoto has quit IRC11:54
*** psachin has quit IRC12:18
openstackgerritGregory Thiemonge proposed openstack/octavia master: Fix controller worker graceful shutdown  https://review.opendev.org/68420112:28
*** yamamoto has joined #openstack-lbaas12:28
*** psachin has joined #openstack-lbaas12:33
*** yamamoto has quit IRC12:35
*** goldyfruit has joined #openstack-lbaas13:00
johnsomCobHead HSTS is not yet supported. Feel free to open a story for us: https://storyboard.openstack.org/#!/project/openstack/octavia13:09
johnsomAs for intermediates, I don't think they need to be sorted.13:10
*** yamamoto has joined #openstack-lbaas13:10
CobHeadjohnsom: Thanks for the clarification wrt HSTS. It's weird though. I supply Octavia with a private key, a certificate and a fullchain which has 1 root and 1 intermediate certificate. They are being sorted alphabetically, and SSL tests against the Load Balancer results in an invalid chain warning.13:18
johnsomCobHead How are you testing?13:20
CobHeadUsing SSL Labs.13:20
johnsomHmm13:21
CobHeadI'm not sure if it is Barbican or Octavia sorting them, though.13:21
johnsomIt is not likely to be Barbican13:21
johnsomYou are using the pkcs12 bundle right?13:22
CobHeadYes13:22
CobHead(Which is the only way I know works)13:22
johnsomYes, we had an old way that still works for backward compatibility, but pkcs12 is the way we are going forward.13:23
johnsomIt was too easy for users to make errors before, so we switched to pkcs12.13:23
CobHeadI see13:24
johnsomPlus the popular HSMs are pkcs12 based, so that also makes it easier for folks.13:24
johnsomIs your endpoint public? I.e. can I connect to it?13:25
*** psachin has quit IRC13:28
*** yamamoto has quit IRC13:30
*** yamamoto has joined #openstack-lbaas13:31
*** yamamoto has quit IRC13:31
*** ajay33 has quit IRC13:38
*** born2bake has joined #openstack-lbaas13:42
born2bakehi guys13:43
born2bakehttps://bugs.launchpad.net/kolla-ansible/+bug/1847905 there is a bug in octavia using kolla-ansible13:43
openstackLaunchpad bug 1847905 in kolla-ansible "Octavia loadbalancer pending create state" [Undecided,New]13:43
johnsomborn2bake Thanks for letting us know. There are a few people here that also work on the kolla ansible stuff.13:45
*** born2bake has quit IRC13:49
CobHeadPlease backport https://review.opendev.org/#/c/688221/, as it is destructive for users unless they are explicitly told not to delete the secret before deleting the LB. The only remedy for this is currently to access the DB and delete the rows associated with the LB in question.14:00
*** yamamoto has joined #openstack-lbaas14:03
CobHeadRephrase: The bug is destructive, not the patch.14:05
*** learnstack has joined #openstack-lbaas14:05
*** gcheresh_ has quit IRC14:06
*** gcheresh_ has joined #openstack-lbaas14:06
*** vishalmanchanda has quit IRC14:13
*** yamamoto has quit IRC14:14
*** pcaruana has quit IRC14:19
openstackgerritMichael Johnson proposed openstack/octavia stable/train: Fix issues with unavailable secrets  https://review.opendev.org/69098414:30
*** learnstack has quit IRC14:48
*** gcheresh_ has quit IRC14:57
*** learnstack has joined #openstack-lbaas14:59
*** learnstack has quit IRC15:06
*** born2bake has joined #openstack-lbaas15:09
*** bcafarel has quit IRC15:16
johnsomhaleyb FYI, those requirements changes you needed to make for the "stop py27 testing" patch, may have needed this: https://review.opendev.org/68958815:24
johnsomThe networkx issue was what led me to do that revert...15:24
haleybjohnsom: yes, possible, should I rebase to see it's still good?15:28
johnsomhaleyb Yeah, and maybe back out the requirements changes?  Basically we reverted that patch because it caused the upper-constraints to not  be applied.15:28
openstackgerritBrian Haley proposed openstack/octavia master: Stop testing python 2  https://review.opendev.org/68737015:29
*** maciejjozefczyk has quit IRC15:30
*** vishalmanchanda has joined #openstack-lbaas15:31
haleybjohnsom: they might be ok as i don't think we need some of those anymore looking again15:31
*** tesseract has quit IRC15:49
*** goldyfruit has quit IRC15:54
*** goldyfruit has joined #openstack-lbaas15:56
*** bcafarel has joined #openstack-lbaas16:00
*** rpittau is now known as rpittau|afk16:05
*** ccamposr has quit IRC16:13
*** ianychoi has joined #openstack-lbaas16:21
*** yamamoto has joined #openstack-lbaas16:45
*** yamamoto has quit IRC16:49
*** henriqueof has joined #openstack-lbaas17:25
*** TrevorV has joined #openstack-lbaas17:49
*** TrevorV has quit IRC17:50
*** TrevorV has joined #openstack-lbaas17:51
*** ricolin has quit IRC17:55
*** abaindur has joined #openstack-lbaas19:16
*** born2bake has quit IRC19:34
*** goldyfruit_ has joined #openstack-lbaas19:57
*** goldyfruit has quit IRC19:59
*** vishalmanchanda has quit IRC20:06
*** yamamoto has joined #openstack-lbaas20:11
*** yamamoto has quit IRC20:16
*** KeithMnemonic has quit IRC20:41
*** TrevorV has quit IRC20:44
rm_workjohnsom: should *delete* commands have a `--wait`? is that something people want to do?20:49
rm_workI guess so?20:49
johnsomHmmm, sure?20:50
johnsomThere is a clear state change when it completes, so I think it is fair20:50
*** KeithMnemonic has joined #openstack-lbaas20:52
rm_workyeah, and that way if you do `delete && add` for like, members or something, you can do --wait and be ok20:54
rm_workI don't know how to change the returncode for our client calls tho :/20:54
rm_workI'm not clear about whether we have control over that, besides raising an exception? :/20:54
johnsomYeah, why do you want to change the return code?20:55
*** maciejjozefczyk has joined #openstack-lbaas21:02
*** maciejjozefczyk has quit IRC21:10
rm_workso if the CUD operation fails (ERROR status) the client will have a failure return code, and not just "yeah ok" :D21:33
rm_workso people who do `o-s lb create && o-s listener create` will actually have things work sanely21:34
johnsomAh, yeah, it needs to raise an exception as it had an ERROR21:34
openstackgerritMerged openstack/neutron-lbaas stable/stein: "lbaas delete l7 rule" Parameter Passing Error  https://review.opendev.org/66531521:37
*** goldyfruit_ has quit IRC21:38
*** rcernin has quit IRC22:03
*** goldyfruit has joined #openstack-lbaas22:03
rm_workugh, it's always the testing22:33
johnsomyep22:34
johnsomI added pretty error messages when the SNI tests fail:22:35
johnsom    b': ERROR: Received certificate "default" with CN bd9838e0-d7a0-409a-91d1-bc0af7caef76.example.com is not the expected certificate "SNI2" with CN bb3090f5-9260-4249-8ae5-4a97a32dd774.example.com.'22:35
johnsomSadly, one of them is still failing... lol22:35
rm_workT_T22:36
johnsomAh, helps if you put listener 2's TCP port in instead of listener 1's22:36
*** gthiemonge has quit IRC22:37
*** gthiemonge has joined #openstack-lbaas22:37
johnsomBefore those it was a super handy:22:37
johnsomhttps://www.irccloud.com/pastebin/bp9hz92A/22:38
johnsomAnd you had to dig through tempest logs to decode22:38
rm_workwhat's that python lib that lets you easily create a fake class with a bunch of attributes for testing22:38
rm_worklike all it does is easily take a dict and make it into a fake class22:38
rm_workmaybe starts with 'p'?22:39
rm_workor 'f'...22:39
johnsomcollections.namedtuple ?22:39
rm_workno, isn't stdlib22:40
johnsomhttps://github.com/openstack/octavia/blob/master/octavia/tests/unit/common/sample_configs/sample_configs_combined.py#L3122:40
rm_workhmmm i guess that might work22:40
rm_workbut this other one was really slick22:40
johnsomYay, ok:22:41
johnsom{0} octavia_tempest_plugin.tests.barbican_scenario.v2.test_tls_barbican.TLSWithBarbicanTest.test_basic_tls_SNI_multi_listener_traffic [22.651614s] ... ok22:41
johnsomA little polish and the SNI stuff is ready to ship. lol22:41
rm_workfff namedtuples doesn't quite do what I want22:43
rm_workaha!22:44
rm_workpretend22:45
rm_workdamn, not in g-r22:45
rm_workhttps://pypi.org/project/pretend/22:46
rm_workthat would make my current task so much easier T_T22:47
rm_workbut instead I can use Munch i guess22:48
*** tkajinam has joined #openstack-lbaas23:04
*** rcernin has joined #openstack-lbaas23:13
*** yamamoto has joined #openstack-lbaas23:30
*** henriqueof has quit IRC23:33
*** yamamoto has quit IRC23:35

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