Wednesday, 2019-02-20

*** abaindur_ has quit IRC00:00
*** abaindur has joined #openstack-lbaas00:07
*** abaindur has quit IRC00:41
*** abaindur has joined #openstack-lbaas00:44
*** abaindur has quit IRC00:44
*** abaindur has joined #openstack-lbaas00:45
openstackgerritMichael Johnson proposed openstack/octavia master: DNM: Gate test for yappi g-r blacklist  https://review.openstack.org/63799500:59
*** hongbin has joined #openstack-lbaas01:07
*** abaindur_ has joined #openstack-lbaas01:37
*** abaindur has quit IRC01:40
*** Dinesh_Bhor has joined #openstack-lbaas01:41
*** abaindur has joined #openstack-lbaas01:44
*** yamamoto has joined #openstack-lbaas01:45
*** yamamoto has quit IRC01:45
*** yamamoto has joined #openstack-lbaas01:46
*** abaindur_ has quit IRC01:46
*** abaindur_ has joined #openstack-lbaas01:52
*** abaindur has quit IRC01:52
*** abaindur has joined #openstack-lbaas01:56
openstackgerritMichael Johnson proposed openstack/octavia master: Add crl-file option for certification  https://review.openstack.org/61226901:57
*** abaindur_ has quit IRC01:58
*** abaindur_ has joined #openstack-lbaas02:03
*** abaindur has quit IRC02:03
*** abaindur has joined #openstack-lbaas02:05
*** abaindu__ has joined #openstack-lbaas02:08
*** abaindur_ has quit IRC02:08
*** abaindu__ is now known as abaindur__02:08
*** abaindur has quit IRC02:10
openstackgerritMichael Johnson proposed openstack/octavia master: DNM: Gate test for yappi g-r blacklist  https://review.openstack.org/63799502:10
openstackgerritMichael Johnson proposed openstack/octavia master: Add crl-file option for certification  https://review.openstack.org/61226902:15
*** abaindur__ has quit IRC02:15
openstackgerritMichael Johnson proposed openstack/octavia master: Add client_ca_tls_container_ref to listener API  https://review.openstack.org/61226702:19
*** abaindur has joined #openstack-lbaas02:20
openstackgerritMichael Johnson proposed openstack/octavia master: Add an option to the Octavia V2 listener API for client cert  https://review.openstack.org/61226802:23
openstackgerritMichael Johnson proposed openstack/octavia master: Add crl-file option for certification  https://review.openstack.org/61226902:23
openstackgerritMichael Johnson proposed openstack/octavia master: Add crl-file option for certification  https://review.openstack.org/61226902:37
*** psachin has joined #openstack-lbaas03:05
*** abaindur has quit IRC03:14
*** ramishra has joined #openstack-lbaas04:23
openstackgerritMichael Johnson proposed openstack/octavia master: DNM: Gate test for yappi g-r blacklist  https://review.openstack.org/63799504:42
*** hongbin has quit IRC05:04
openstackgerritMichael Johnson proposed openstack/octavia master: DNM: Gate test for yappi g-r blacklist  https://review.openstack.org/63799506:00
*** AlexStaf has joined #openstack-lbaas06:09
*** yamamoto has quit IRC07:19
*** gcheresh has joined #openstack-lbaas07:35
*** ccamposr has joined #openstack-lbaas07:38
*** pcaruana has joined #openstack-lbaas07:59
*** yamamoto has joined #openstack-lbaas08:10
*** yamamoto has quit IRC08:10
*** yamamoto has joined #openstack-lbaas08:11
*** rpittau has joined #openstack-lbaas08:11
*** yamamoto has quit IRC08:13
*** yamamoto has joined #openstack-lbaas08:13
*** yamamoto has quit IRC08:13
openstackgerritCarlos Goncalves proposed openstack/octavia master: DNM: Gate test for yappi g-r blacklist  https://review.openstack.org/63799508:16
*** rcernin has quit IRC08:29
*** yamamoto has joined #openstack-lbaas08:38
*** yamamoto has quit IRC08:38
*** yamamoto has joined #openstack-lbaas08:38
openstackgerritCarlos Goncalves proposed openstack/python-octaviaclient master: Add load balancer flavor support to the OSC client  https://review.openstack.org/62994508:39
*** celebdor has joined #openstack-lbaas08:43
*** yamamoto has quit IRC08:49
*** yetiszaf has quit IRC09:09
*** yamamoto has joined #openstack-lbaas09:13
*** glebz has joined #openstack-lbaas09:17
*** yetiszaf has joined #openstack-lbaas09:27
openstackgerritCarlos Goncalves proposed openstack/octavia master: DNM: Gate test for yappi g-r blacklist  https://review.openstack.org/63799509:27
openstackgerritMerged openstack/python-octaviaclient master: Add load balancer flavor support to the OSC client  https://review.openstack.org/62994509:27
*** Dinesh_Bhor has quit IRC09:41
*** yamamoto has quit IRC10:11
*** yamamoto has joined #openstack-lbaas10:14
*** yamamoto has quit IRC10:19
*** ianychoi has quit IRC10:59
*** yamamoto has joined #openstack-lbaas11:02
*** yamamoto has quit IRC11:07
*** yamamoto has joined #openstack-lbaas11:20
*** yamamoto has quit IRC11:34
*** yamamoto has joined #openstack-lbaas11:36
*** yamamoto has quit IRC11:37
*** yamamoto has joined #openstack-lbaas11:37
*** sapd1 has quit IRC11:38
*** yamamoto has quit IRC11:42
*** yamamoto has joined #openstack-lbaas11:42
*** yamamoto has quit IRC11:47
*** sapd1 has joined #openstack-lbaas11:55
*** sapd1 has quit IRC12:10
*** sapd1 has joined #openstack-lbaas12:22
*** ccamposr__ has joined #openstack-lbaas12:44
*** ccamposr__ has quit IRC12:45
*** ccamposr__ has joined #openstack-lbaas12:45
*** ccamposr has quit IRC12:47
*** rpittau has quit IRC12:49
openstackgerritMerged openstack/python-octaviaclient master: add python 3.7 unit test job  https://review.openstack.org/63777712:57
*** psachin has quit IRC13:01
*** rpittau has joined #openstack-lbaas13:02
*** psachin has joined #openstack-lbaas13:04
*** glebz has quit IRC13:08
*** ivve has joined #openstack-lbaas13:10
*** ianychoi has joined #openstack-lbaas13:22
*** trown is now known as trown|outtypewww13:31
*** ccamposr has joined #openstack-lbaas13:39
*** ccamposr__ has quit IRC13:43
*** ccamposr has quit IRC13:44
*** ccamposr has joined #openstack-lbaas13:47
*** yamamoto has joined #openstack-lbaas13:50
*** ccamposr has quit IRC13:53
*** ccamposr has joined #openstack-lbaas13:53
*** ccamposr has quit IRC13:58
*** ccamposr has joined #openstack-lbaas14:02
*** ccamposr has quit IRC14:21
*** ccamposr has joined #openstack-lbaas14:21
*** yamamoto has quit IRC14:29
*** yamamoto has joined #openstack-lbaas14:30
*** ccamposr has quit IRC14:39
*** ccamposr has joined #openstack-lbaas14:39
*** fnaval has joined #openstack-lbaas15:03
*** sapd1_x has joined #openstack-lbaas15:23
*** sapd1 has quit IRC15:23
*** ltomasbo has joined #openstack-lbaas15:32
openstackgerritMichael Johnson proposed openstack/octavia master: DNM: Gate test for yappi g-r blacklist  https://review.openstack.org/63799515:46
*** rpittau_ has joined #openstack-lbaas15:50
*** rpittau_ has left #openstack-lbaas15:50
*** gcheresh has quit IRC15:52
*** ivve has quit IRC15:52
openstackgerritNir Magnezi proposed openstack/octavia master: Encrypt certs and keys  https://review.openstack.org/62706415:55
*** rpittau has quit IRC15:55
*** rpittau_ has joined #openstack-lbaas15:55
*** rpittau_ is now known as rpittau15:57
*** yamamoto has quit IRC16:13
*** yamamoto has joined #openstack-lbaas16:16
*** yamamoto has quit IRC16:21
*** rpittau is now known as rpittau|sardegna16:28
*** psachin has quit IRC17:04
*** ramishra has quit IRC17:12
*** pcaruana has quit IRC17:13
*** yamamoto has joined #openstack-lbaas17:30
*** henriqueof has joined #openstack-lbaas17:54
johnsomThe summit schedule is up. "OpenStack load balancing new feautres" got accepted17:54
colin-nice guys18:13
colin-that's great18:13
jitekagood news19:08
*** ccamposr has quit IRC19:16
cgoncalveshey folks! I'll not be able to attend today's meeting. I arrived today to Portugal and will have family dinner :)19:23
cgoncalvesI'll catch up later with the minutes19:23
johnsomEnjoy!19:28
rm_workHave fun and good food ;)19:59
johnsom#startmeeting Octavia20:00
openstackMeeting started Wed Feb 20 20:00:08 2019 UTC and is due to finish in 60 minutes.  The chair is johnsom. Information about MeetBot at http://wiki.debian.org/MeetBot.20:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.20:00
*** openstack changes topic to " (Meeting topic: Octavia)"20:00
openstackThe meeting name has been set to 'octavia'20:00
johnsomHi folks20:00
nmagnezio/20:00
colin-hello20:00
johnsom#topic Announcements20:00
*** openstack changes topic to "Announcements (Meeting topic: Octavia)"20:00
johnsomThe TC election question and answer period is now open.20:01
johnsomYou can ask questions of the candidates on the discuss mailing list.20:01
johnsomThe OpenStack-> Open Infrastructure Summit schedule has been posted.20:02
johnsomYou should have received an e-mail if one of your talks was accepted.20:02
johnsomI know we had one accepted: OpenStack Load Balancing New Features Deep Dive20:02
johnsomI haven't had a minute to look for others.20:02
johnsomNext up in announcements is the end-of-Stein planning.20:03
johnsomI plan to start making a cut line next week for features to make Stein.20:03
johnsomPlease contact me if you have must-have features for Stein that have not yet merged and are ready for review20:03
johnsomAlong those lines, I am updating the priority review list to break out items by deadlines.20:04
johnsom#link https://etherpad.openstack.org/p/octavia-priority-reviews20:04
johnsomI should have that updated in an hour or two.20:04
johnsomThe first deadline is for octavia-lib next week.20:04
johnsomFollowed by feature freeze the week after.20:05
johnsomThat is all I have for announcements. Any other announcements today?20:05
nmagneziI picked up 'Encrypt certs and keys' again today. Will finish it early next week20:05
johnsomnmagnezi "early" is key there20:05
nmagneziNoted :)20:06
nmagnezijohnsom, will let you know when it is ready for more feedback20:06
johnsomIt seems like feature freeze deadline is early, but I guess it always feels that way20:06
rm_workYep20:06
johnsom#topic Brief progress reports / bugs needing review20:07
*** openstack changes topic to "Brief progress reports / bugs needing review (Meeting topic: Octavia)"20:07
johnsomI have been heads down trying to help get the TLS patches ready for the feature freeze.20:07
johnsomI am pretty close to having client authentication done. The first three patches are ready for review.20:08
johnsomYesterday I tracked down the gate breakage to an oslo.service release that pulled in a broken package.20:08
johnsomHopefully we can get a requirements update merged today and get our gates going again. Sadly, even though we don't use it, oslo.messaging pulls it in.20:09
johnsomI poked the SDK folks today so the reset of our SDK patches should merge today(if their gate issues don't block it).20:09
johnsomOtherwise, it's going to be time trying to prioritize the patch reviews and make sure we don't miss something.20:10
johnsomAlso, don't forget, after feature freeze, we can still work on bug fixes for another two weeks20:10
nmagnezijohnsom, thank you for maintaining that etherpad. It really helps!20:11
johnsomAny other progress updates?20:11
johnsomAlso the tempest stuff can happen at any time, so we don't need to prioritize those20:13
johnsomOk, no other updates?20:13
johnsom#topic Open Discussion20:14
*** openstack changes topic to "Open Discussion (Meeting topic: Octavia)"20:14
johnsomOther topics today?20:14
colin-I'd like to share some feedback, for the benefit of the project, about our experience discovering the bug detailed in this story last week https://storyboard.openstack.org/#!/story/2004993 and attempting to work through it in this change https://review.openstack.org/#/c/636428/20:14
johnsomOk, go for it20:16
colin-I was hoping for a much simpler experience for everybody involved in the linked change, and while I recognize the benefits that extreme scrutiny in code reviews brings to a project, I feel that in this case the project would have been better served by moving forward with the identified bug fix and tabling ancillary concerns for follow-up (read: non bug fix) changes related to other objectives.20:16
colin-Our efforts to adopt Octavia are going to be most successful if we feel confident in our opportunity to contribute specific, atomic changes that directly affect our own deployment. Deeply appreciate all of the effort so far to encourage us, hope we can emphasize that in the future if we are able to get some of our developers onto an Octavia task again. If I could offer one suggestion it would be to differentiate opportunities for feature enrichment20:18
johnsomFair feedback. I will say, less than a week turn around for an OpenStack patch is pretty good. grin.20:18
johnsomI think one of the strengths of open development is the diversity of perspectives and experiences that the teams can bring.20:19
johnsomSo I do want to champion that as well.20:19
johnsomAny other feedback on how that patch went or our processes?20:20
colin-that's all from me, thanks for reading :)20:20
nmagneziI was not involved in the linked patch specifically, but fully agree with johnsom here. We are very happy to have you guys her :)20:20
nmagneziHere*20:20
jitekajust ack what colin said as I followed a bit this story20:20
johnsomcolin- Thank you for bringing up the topic. We should continue to evolve as a team and identify things we can do better.20:21
jitekaI don't think it's really about how long it take but more how to get situation unstuck when some different dev philosophy collide20:21
johnsomYeah, I think dialog is the best way to keep things moving forward. Both on the patch and in the channel.20:23
johnsomThose of us that have worked in OpenStack for a while all know of some of the discussions I have had about my patches...  I think it is just part of the "open" process and helps us all produce better code.  I know I have had significant discussions myself.20:24
* johnsom coughs "Stephen"20:24
*** celebdor has quit IRC20:24
johnsomOk, any other comments or topics today?20:25
jitekaall good for me20:26
johnsomOk, then I am going to hop over to the requirements channel and see if we can get this oslo.service thing fixed. Their meeting starts in a few minutes.20:27
johnsomThanks folks. Have a good week!20:27
jitekatime for yappy to take a nap(py) :D20:27
johnsom#endmeeting20:27
*** openstack changes topic to "Discussions for Octavia | Stein priority review list: https://etherpad.openstack.org/p/octavia-priority-reviews"20:27
openstackMeeting ended Wed Feb 20 20:27:52 2019 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)20:27
openstackMinutes:        http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-02-20-20.00.html20:27
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-02-20-20.00.txt20:27
openstackLog:            http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-02-20-20.00.log.html20:27
johnsomlol20:27
rm_workYeah I hope you don't feel I was being too combative there -- I tried to be very clear I wouldn't block that patch even if I disagreed, and other cores should go ahead and vote for it if they liked it ;)20:27
rm_workBut in talking to folks it seemed like they did, and I was given go-ahead by the patch author directly to make the changes I was asking for, so I did20:28
rm_workAnd then we discussed it for a bit and merged :)20:28
rm_workSeems to me like a pretty good example of the system succeeding20:29
rm_workActually if you think that was painful, I'm worried about expectations <_<20:29
*** celebdor has joined #openstack-lbaas20:31
*** celebdor has quit IRC20:35
*** celebdor has joined #openstack-lbaas20:37
rm_workAs johnsom said, one week is great 👍20:44
johnsomLooks like in a few hours we can start rechecking our stuff. The yappi UC update is in the merge queue20:47
rm_workSweet20:51
cgoncalvesyay!21:13
xgerman+121:14
*** takamatsu_ has joined #openstack-lbaas21:31
*** takamatsu has quit IRC21:31
*** takamatsu_ has quit IRC21:45
*** takamatsu has joined #openstack-lbaas21:46
openstackgerritMichael Johnson proposed openstack/neutron-lbaas-dashboard master: Display error when entering a non-integer  https://review.openstack.org/62114121:53
johnsomcgoncalves Do you have some sort of trigger on my patch votes?21:59
cgoncalvesjohnsom, no, don't feel so special :)22:00
johnsomIt was like 1 minute after my vote on lbaas-dashboard. lol22:00
cgoncalvesI get emails and notification of those on desktop and phone22:01
cgoncalvessorry I took 1 minute. I'll try to be quicker22:01
johnsomI like it!22:03
*** ccamposr has joined #openstack-lbaas22:04
openstackgerritMerged openstack/neutron-lbaas-dashboard master: Update the URL in HACKING.rst  https://review.openstack.org/60476922:10
openstackgerritMerged openstack/neutron-lbaas-dashboard master: Update home-page  https://review.openstack.org/63043222:13
johnsomcgoncalves Are you done with https://review.openstack.org/628007?22:22
cgoncalvesjohnsom, not done in the sense the grenade job is still broken but just abandoned the patch22:23
johnsomYeah, I was just going though the patches. If you didn't need it I was going to abandon. Just to clean up the list a bit.22:23
johnsomTrying to make sure we aren't missing patches for review, so going through them all....  Some of that PTL fun!22:24
openstackgerritMichael Johnson proposed openstack/octavia master: Update requirements for ubuntu  https://review.openstack.org/62440522:24
cgoncalvesspeaking of that, should we update our motto to "5 more years"?22:25
johnsomI think 4 is still valid... sigh22:25
cgoncalvesheh22:25
johnsomrm_work I assume we are not targeting "Add usage admin resource" for Stein? Or do you have cycles to work on that?22:26
johnsomIt's zuul -1 at the moment22:26
rm_worki could22:27
rm_workdo we want it?22:27
johnsomwant != need22:27
rm_workI don't care so much anymore but some others showed interest22:27
rm_workalso, members as a base resource? :P22:27
*** rcernin has joined #openstack-lbaas22:27
johnsomYeah, probably haven't hit that one yet in the list I'm going down22:27
johnsomMy question is to you. Do you *need* it in Stein and have cycles for it? or are these early train items?22:28
johnsomAh, members is marked WIP so I skipped it22:28
rm_worki don't care personally22:28
rm_workah did i22:29
rm_workyeah prolly i did22:29
rm_workyeah, Train is fine22:29
johnsomOk, given the list we have, I vote for bumping it to train22:29
johnsomrm_work Does this patch make sense? https://review.openstack.org/60691822:31
openstackgerritCarlos Goncalves proposed openstack/octavia master: Update requirements for ubuntu  https://review.openstack.org/62440522:32
cgoncalves^ sorry johnsom and xgerman, but I had to22:32
xgermanok22:32
johnsomNo worries, sorry I didn't catch your earlier comment.22:32
johnsomThough you didn't have a -1 on it....22:33
cgoncalvesyour review triggered my revisit to this patch ;)22:34
openstackgerritMerged openstack/neutron-lbaas-dashboard master: Display error when entering a non-integer  https://review.openstack.org/62114122:35
*** fnaval has quit IRC22:36
*** fnaval has joined #openstack-lbaas22:43
*** fnaval has quit IRC22:44
johnsomOk, I think that is our list for Stein. Everything else is either failing gates, in conflict, or -1'd23:00
johnsomThat said, there is still more TLS patches coming23:00
johnsomThey are posted, but need some polish23:00
*** ccamposr has quit IRC23:01
cgoncalvesoh, wow! nice work, Michael (https://etherpad.openstack.org/p/octavia-priority-reviews)23:04
xgermanok, let the +2 begin23:04
johnsomIt is a "living" document, that isn't the set in stone Stein yet.23:06
johnsomJust FYI. But a helpful prioritization  list23:06
cgoncalves"mom! German is spammimg again"23:06
*** rcernin has quit IRC23:06
* johnsom Hopes there is "review" in that....23:06
*** rcernin has joined #openstack-lbaas23:08
xgermanguess, need to give cgoncalves more -123:09
cgoncalves:D23:09
cgoncalvesmy comment was because I received two notifications of your reviews23:10
xgermanah, yeah I did 2 and then stopped since one of my compiles finished :-)23:11
rm_workumm, johnsom i remember being confused by that CR before23:14
rm_workbut i am looking again23:14
johnsomOk, yeah, maybe just vote -1 with concerns?23:14
rm_worki just didn't think it was necessary, but i will try to parse it again23:15
johnsomWell, you have the -2 power too....23:16
johnsomsapd1_x Are you around?23:17
johnsomI am trying to create a gate test for the cinder backed amps. I'm not sure what a good setting for devstack would be for volume type.23:18
johnsomAh, it's optional23:19
rm_workjohnsom: did we go the "don't ever delete ACLs route?"23:21
johnsomrm_work Yes: https://review.openstack.org/63764623:21
rm_workkk23:21
openstackgerritMichael Johnson proposed openstack/octavia-tempest-plugin master: Add a test job for cinder volume backed amps  https://review.openstack.org/63829323:23
johnsomLet's see how that does...23:23
johnsomWell, darn, the requirements fix still hasn't merged23:24
rm_workjohnsom: nm, really re-reviewed and I think this is right23:32
johnsomThank you23:32
rm_worktook me a bit to see what they were getting at23:34
rm_workbut actually a very quality patchset :)23:34
rm_workeven added a missing test ^Z_^23:35
rm_work^_^23:35

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