Monday, 2017-01-30

*** armax has quit IRC00:51
*** ducttape_ has quit IRC01:09
*** ducttape_ has joined #openstack-lbaas01:17
*** ducttape_ has quit IRC01:39
*** ducttape_ has joined #openstack-lbaas01:39
*** amotoki has joined #openstack-lbaas01:53
*** ducttape_ has quit IRC02:02
*** ducttape_ has joined #openstack-lbaas02:03
*** sferna1_ has quit IRC02:03
*** ducttape_ has quit IRC02:06
*** _ducttape_ has joined #openstack-lbaas02:06
*** amotoki has quit IRC02:18
*** _ducttape_ has quit IRC02:24
*** amotoki has joined #openstack-lbaas02:26
*** reedip_ has joined #openstack-lbaas02:36
*** amotoki has quit IRC02:45
*** rcernin has joined #openstack-lbaas03:11
*** rcernin has quit IRC03:13
*** reedip_ has quit IRC03:14
*** rcernin has joined #openstack-lbaas03:14
*** rcernin has quit IRC03:25
*** amotoki has joined #openstack-lbaas03:45
*** links has joined #openstack-lbaas03:56
*** links has quit IRC03:57
*** kevo has joined #openstack-lbaas04:15
*** sanfern has joined #openstack-lbaas04:18
*** ducttape_ has joined #openstack-lbaas04:25
*** ducttape_ has quit IRC04:30
*** yuanying has joined #openstack-lbaas04:48
*** sanfern has quit IRC04:49
*** sanfern has joined #openstack-lbaas04:50
*** amotoki has quit IRC04:56
*** xilch has joined #openstack-lbaas05:08
*** amotoki has joined #openstack-lbaas05:25
*** sanfern has quit IRC05:26
*** sanfern has joined #openstack-lbaas05:27
*** sferna1_ has joined #openstack-lbaas05:32
openstackgerritSindhu Devale proposed openstack/octavia: Introduce Octavia v2 API for Health Monitor  https://review.openstack.org/40625605:33
*** sanfern has quit IRC05:34
*** anilvenkata has joined #openstack-lbaas06:08
*** ducttape_ has joined #openstack-lbaas06:26
*** ducttape_ has quit IRC06:31
*** xilch has quit IRC07:00
*** Alex_Staf has joined #openstack-lbaas07:01
*** rcernin has joined #openstack-lbaas07:11
*** rcernin has quit IRC07:12
*** rcernin has joined #openstack-lbaas07:12
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lbaas-dashboard: Imported Translations from Zanata  https://review.openstack.org/42665607:20
*** eezhova has joined #openstack-lbaas07:21
*** Alex_Staf has quit IRC07:22
*** yamamoto has quit IRC07:27
*** tesseract has joined #openstack-lbaas07:50
*** sferna1__ has joined #openstack-lbaas08:06
*** zw_ is now known as zw08:08
*** sferna1_ has quit IRC08:08
*** gcheresh_ has joined #openstack-lbaas08:11
*** pcaruana has joined #openstack-lbaas08:15
*** Alex_Staf has joined #openstack-lbaas08:27
*** ducttape_ has joined #openstack-lbaas08:27
*** ducttape_ has quit IRC08:32
openstackgerritNir Magnezi proposed openstack/octavia: Fix the amphora image support for RH Linux flavors  https://review.openstack.org/33184108:41
*** amotoki has quit IRC08:59
*** amotoki has joined #openstack-lbaas09:00
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - added disable arp  https://review.openstack.org/42517509:01
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - LB create cluster impl  https://review.openstack.org/42656009:01
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - added disable arp  https://review.openstack.org/42517509:19
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - LB create cluster impl  https://review.openstack.org/42656009:19
*** sferna1__ has left #openstack-lbaas09:26
*** kevo has quit IRC09:26
*** jschwarz is now known as jschwarz|lunch09:27
*** rcernin has quit IRC09:31
*** rcernin has joined #openstack-lbaas09:33
*** amotoki has quit IRC10:05
*** amotoki has joined #openstack-lbaas10:15
openstackgerritReedip proposed openstack/neutron-lbaas: TrivialFix: Added link for modindex  https://review.openstack.org/42669610:20
*** ducttape_ has joined #openstack-lbaas10:28
*** ducttape_ has quit IRC10:33
*** amotoki has quit IRC10:34
*** yuanying_ has joined #openstack-lbaas10:48
*** yuanying has quit IRC10:51
*** eezhova has quit IRC11:20
*** eezhova has joined #openstack-lbaas11:22
*** yuanying_ has quit IRC11:25
*** ducttape_ has joined #openstack-lbaas11:35
*** ducttape_ has quit IRC11:52
*** catintheroof has joined #openstack-lbaas12:25
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - added disable arp  https://review.openstack.org/42517512:44
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - LB create cluster impl  https://review.openstack.org/42656012:44
*** Alex_Staf has quit IRC12:48
*** Alex_Staf has joined #openstack-lbaas13:01
*** ducttape_ has joined #openstack-lbaas13:04
*** sanfern has joined #openstack-lbaas13:18
*** ducttape_ has quit IRC13:41
*** jschwarz|lunch is now known as jschwarz14:01
xgermano/14:13
*** ducttape_ has joined #openstack-lbaas14:19
*** matt-borland has joined #openstack-lbaas14:22
*** ducttape_ has quit IRC14:26
*** rcernin has quit IRC14:48
*** ducttape_ has joined #openstack-lbaas14:52
nmagnezixgerman, o/14:53
xgerman:-)14:54
*** reedip_ has joined #openstack-lbaas14:54
*** ajo has left #openstack-lbaas14:56
*** ajo has joined #openstack-lbaas14:57
ajomultinode vs singlenode graphite: http://bit.ly/2jv1Tmg14:57
ajowe should really stabilise scenarios14:57
ajoO:)14:57
ajobut at least multinode is close to single node14:57
*** rcernin has joined #openstack-lbaas15:00
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - added disable arp  https://review.openstack.org/42517515:13
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - LB create cluster impl  https://review.openstack.org/42656015:13
*** Alex_Staf has quit IRC15:17
*** armax has joined #openstack-lbaas15:17
openstackgerritBernard Cafarelli proposed openstack/octavia: Use cryptsetup/LUKS for encrypted ramfs  https://review.openstack.org/40381715:20
*** Alex_Staf has joined #openstack-lbaas15:30
*** fnaval has joined #openstack-lbaas15:41
*** gcheresh_ has quit IRC16:00
*** _ducttape_ has joined #openstack-lbaas16:00
*** ducttape_ has quit IRC16:01
*** cody-somerville has joined #openstack-lbaas16:10
*** Alex_Staf has quit IRC16:15
*** armax_ has joined #openstack-lbaas16:16
*** armax has quit IRC16:17
*** armax_ is now known as armax16:17
*** rcernin has quit IRC16:24
*** csomerville has joined #openstack-lbaas16:31
*** cody-somerville has quit IRC16:35
*** armax has quit IRC16:36
*** _ducttape_ has quit IRC16:40
*** ducttape_ has joined #openstack-lbaas16:40
johnsomo/16:44
xgermano/16:48
openstackgerritNakul Dahiwade proposed openstack/octavia: Align Octavia API to n-lbaasv2 for L7Policy  https://review.openstack.org/40632816:51
*** gcheresh_ has joined #openstack-lbaas16:53
*** tesseract has quit IRC16:58
*** anilvenkata has quit IRC17:01
*** gcheresh_ has quit IRC17:02
nmagnezio/17:18
johnsomnmagnezi How is it going?17:23
johnsomReady to hand off the patch back to me?17:23
nmagnezijohnsom, i'm done :) happy to help if you need me to17:28
johnsomOk.  I can look through the coverage report and pull out the current gaps....  I basically compare the change list to the coverage report: http://sonar.octavia.cloud/archive/octavia/6ea237591ab81cce6d75b1ade9a1adec7d431ad5/3523/cover/17:29
johnsomnmagnezi It looks like there are still some gaps in osutils: http://sonar.octavia.cloud/archive/octavia/6ea237591ab81cce6d75b1ade9a1adec7d431ad5/3523/cover/octavia_amphorae_backends_agent_api_server_osutils_py.html17:31
nmagnezijohnsom, ack. I'll be back online in 2hours. I'll ping you when I'm here17:31
johnsomOk17:31
johnsomI'm going to finish up server api tests and amphora_info17:31
johnsomIf we can get those done, I with we are ready to go17:36
*** kevo has joined #openstack-lbaas17:42
*** reedip_ has left #openstack-lbaas17:46
*** armax has joined #openstack-lbaas18:02
*** harlowja has joined #openstack-lbaas18:02
*** xilch has joined #openstack-lbaas19:11
*** gcheresh_ has joined #openstack-lbaas19:12
*** anilvenkata has joined #openstack-lbaas19:20
openstackgerritShashank Kumar Shankar proposed openstack/octavia: Align Octavia API to n-lbaasv2 for L7Rules  https://review.openstack.org/40633619:46
rm_workare we going to FTE it in?19:50
johnsom???19:52
*** Alex_Staf has joined #openstack-lbaas19:55
rm_workthe rpm amphora patch19:56
rm_worksince it seems it didn't make the cut last week?19:56
openstackgerritNakul Dahiwade proposed openstack/octavia: Align Octavia API to n-lbaasv2 for L7Policy  https://review.openstack.org/40632819:58
*** _ducttape_ has joined #openstack-lbaas20:31
*** anilvenkata has quit IRC20:32
*** ducttape_ has quit IRC20:34
rm_workjohnsom: did you see my question/notes before about the pagination returns?20:37
rm_workjohnsom: I finally actually read through the neutron sorting/paging api spec on that page you linked before20:37
rm_workand it looks like we're not doing a bunch of the stuff they expect in a return20:38
rm_worknext/prev links20:38
rm_workfor example20:38
johnsomNo, I must have missed that20:38
rm_workthe actual data is like, a sub-resource20:38
johnsomCan I ping you in line 20 mins?20:39
rm_workyeah no problem20:39
rm_workor if this can wait until after the ocata final release, i can just review or focus on doing bugfix stuff20:40
johnsomI'm in a meeting, but can chat after20:41
rm_workyeah np stop listening to me (unless the meeting is SUPER boring, then do whatever you want :P)20:41
johnsomI kind of need to pay attention to this one20:42
rm_worklolol stop responding and do that then, hahadidn't mean to interrupt20:42
johnsomGrin20:42
*** Alex_Staf has quit IRC20:59
openstackgerritNakul Dahiwade proposed openstack/octavia: Align Octavia API to n-lbaasv2 for L7Policy  https://review.openstack.org/40632821:00
*** _ducttape_ has quit IRC21:02
*** ducttape_ has joined #openstack-lbaas21:02
johnsomrm_work Ok.  So, with a lack of guidance from the API working group I'm not sure I would spend much time to implement anything that neutron doesn't have.   It's just too fluid.  MVP is match neutron, so...21:05
rm_workok, I just need to double-check neutron doesn't already do that21:06
johnsomrm_work Also on the RH thing, yeah, still planning to get that in.  It looks like stable cut is now RC1 timeline (this week), so I felt like giving it a pass.  Especially since it is working but just missing unit/functional test updates21:06
rm_workyeah ok21:07
johnsomhttps://developer.openstack.org/api-ref/networking/v2/#pagination21:07
rm_workwasn't sure if you could just decide or if we needed to submit something21:07
johnsomWe are our own project now, so it's up to us (me I guess)21:08
johnsomHmm, it looks like they do have the next/previous links21:09
rm_workthat's what I was getting it from, needed to double-check that they actually DO that21:09
rm_workand not just say they do it <_<21:09
johnsomHahaha, true...21:09
*** kevo has quit IRC21:25
*** xilch has quit IRC21:39
*** catintheroof has quit IRC21:40
*** gcheresh_ has quit IRC21:41
*** catintheroof has joined #openstack-lbaas21:42
rm_workjohnsom: yep they definitely do that21:43
rm_workbuuuut21:43
rm_workmaybe I can leave that to the API alignment? :/21:43
rm_worksince it would require a rework of the way the LBaaS API returns21:43
rm_workbecause right now, the neutron API returns everything with a root resource, like { "loadbalancers": [] }21:44
rm_workand then when you do any sorting/paging it adds the "loadbalancers_links" resource at the same level21:44
johnsomRight, nlbaas has root tags21:44
rm_workyeah21:44
rm_workand doing this without root tags wouldn't work21:44
rm_workthey're necessary21:45
rm_workwhich, actually, makes using root tags WAY more sensical21:45
rm_workI get it now :P21:45
rm_workso I'm thinking, sorting/paging maybe should go in as-is21:45
rm_workand then once LBs have root tags21:45
rm_workI'll do the followup21:45
rm_workwhich would make this a "partially-implements"21:45
johnsomOk, so let's open a bug to track what will be left to complete when the api stuff is done21:45
rm_workso yeah when does the final release happen? IE, when are we allowed to start merging stuff besides bugfixes again?21:46
*** kevo has joined #openstack-lbaas21:47
johnsomSometime between now and Feb 13th.  Basically when everything is in/finished the way we want for ocata and stable21:48
rm_workok21:48
*** catinthe_ has joined #openstack-lbaas21:52
*** catintheroof has quit IRC21:53
*** armax has quit IRC21:57
*** armax has joined #openstack-lbaas21:59
nmagnezijohnsom, o/22:00
johnsomnmagnezi Hi22:01
nmagnezijohnsom, i was not aware of that test coverage tool, very nice!22:01
johnsomnmagnezi It is the link for the "Rackspace-Octavia-Sonar-CI" check gate22:02
*** yamamoto has joined #openstack-lbaas22:03
nmagnezijohnsom, yup, I get it.. I just never looked22:03
nmagnezinot bad..22:03
nmagneziwhere do we stand now? how can I help?22:03
*** catintheroof has joined #openstack-lbaas22:04
johnsomnmagnezi I will finish up everything except osutils.  You can work on osutils in the morning.  I might have some time to poke at those tests this afternoon, but I want to finish up amphora_info22:05
johnsomThen we should be good to go22:05
nmagnezijohnsom, will core members have time to review? this patch became a monster22:06
johnsomYeah, we will do it.  It's mostly tests that ballooned it up (The work I did Friday)22:06
openstackgerritAnkur proposed openstack/octavia: Add common base type for v1 and v2  https://review.openstack.org/42354222:06
openstackgerritAnkur proposed openstack/octavia: Add v2 load balancer type and controllers  https://review.openstack.org/42354322:06
*** catinthe_ has quit IRC22:07
nmagnezijohnsom, yup, but also the production code portion changes things in many places22:08
johnsomAre you getting some sleep now?22:08
nmagnezijohnsom, soon, yes22:09
nmagnezijohnsom, why?22:09
johnsomI can post a status update to the patch when I finish for the day22:09
johnsomJust trying to keep in sync so we don't push patches on ourselves22:09
nmagnezijohnsom, I didn't plan to push any changes without pinging you first.. unless you are asleep you I just notify you :)22:10
nmagnezijohnsom, I have a question about the tests coverage tool22:10
nmagnezijohnsom, http://sonar.octavia.cloud/archive/octavia/6ea237591ab81cce6d75b1ade9a1adec7d431ad5/3523/cover/octavia_amphorae_backends_agent_api_server_osutils_py.html#n5522:10
johnsomOk.  Yep.  Have a good evening.  I will keep working on the tests and post a status on the patch when I finish today.22:10
johnsomSure22:11
nmagnezijohnsom, red = no converge, right?22:11
johnsomRed means the line isn't being tested.  Yellow means that only one of the code branches (conditional) is being tested22:11
johnsomAlso, only worry about lines you changed in the patch.  Don't spend time on lines that were already being missed if they weren't impacted in your patch.  We will get to those over time.22:12
nmagnezijohnsom, what i don't get in the example I sent you is that I actually added a pretty extensive test for get_network_interface_file22:12
nmagnezijohnsom, https://review.openstack.org/#/c/331841/77/octavia/tests/unit/amphorae/backends/agent/api_server/test_osutils.py here at line 6222:12
nmagnezijohnsom, so what am I missing here?22:12
johnsomYour test runs the ubuntu and centos class versions, but not the base class version22:13
johnsomWhich is #5522:13
nmagnezijohnsom, oh22:13
nmagnezijohnsom, hmm... okay I'll need to find an elegant way to invoke the base class as well22:14
*** catinthe_ has joined #openstack-lbaas22:17
*** catinth__ has joined #openstack-lbaas22:18
*** catintheroof has quit IRC22:18
nmagnezijohnsom, okay last question before I go, not at all related to patch if that is ok22:21
johnsomSure, NP22:21
nmagnezihttp://paste.openstack.org/show/596942/ lines 134-13822:21
*** catinthe_ has quit IRC22:22
nmagneziare those options related to the way the agent communicates securely with Octavia services? or is this related to end user certificates?22:22
johnsomYes, the config lines that get passed into the /etc/octavia/amphora-agent.conf in the inmage22:22
nmagnezimeaning certs that are used in the loadbalancing itself22:23
johnsomThey are for controller<->amphora only, not the user listener certificates22:23
johnsomThey are used by the amphora-agent process22:23
nmagnezijohnsom, yup that is exactly what I wanted to know. Thanks! :)22:24
johnsomOk22:24
nmagnezijohnsom, okay we'll sync again tomorrow. good night!22:27
johnsomGood night!22:28
nmagnezijohnsom, if there is anything other than additional coverage to osutils that you want me to handle, please comment in the patch22:28
nmagnezijohnsom, also, thanks a lot for helping with the tests..22:29
johnsomYep, will do, but I think we have it otherwise.  I will do a review at the end of the day22:29
*** catinth__ has quit IRC22:29
johnsomNP22:29
openstackgerritShashank Kumar Shankar proposed openstack/octavia: Introduce Octavia v2 API for pools  https://review.openstack.org/40592222:33
*** eezhova has quit IRC22:37
*** armax has quit IRC22:49
*** armax has joined #openstack-lbaas22:51
*** matt-borland has quit IRC22:59
*** ducttape_ has quit IRC23:21
*** armax has quit IRC23:29
*** armax has joined #openstack-lbaas23:30
*** armax has quit IRC23:31
openstackgerritAnkur proposed openstack/octavia: Add common base type for v1 and v2  https://review.openstack.org/42354223:33
*** xilch has joined #openstack-lbaas23:38
*** armax has joined #openstack-lbaas23:39
*** kevo has quit IRC23:41
*** kevo has joined #openstack-lbaas23:47
*** catintheroof has joined #openstack-lbaas23:50

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