Wednesday, 2017-05-10

*** cody-somerville has joined #openstack-lbaas00:00
*** catintheroof has quit IRC00:36
*** cpuga has joined #openstack-lbaas00:55
*** cpuga has quit IRC01:05
*** cpuga has joined #openstack-lbaas01:05
*** cpuga has quit IRC01:09
openstackgerritMichael Johnson proposed openstack/octavia master: Add v2 pool API section  https://review.openstack.org/45827201:13
*** rm_mobile has joined #openstack-lbaas01:25
rm_mobilejohnsom: still submitting stuff? :P01:25
rm_mobileFenway party is... Kinda boring01:26
*** JudeC has quit IRC01:26
rm_mobileThinking about just doing reviews from here, lol01:26
*** cpuga has joined #openstack-lbaas01:27
*** sanfern has joined #openstack-lbaas01:27
*** fnaval has quit IRC01:36
*** fnaval has joined #openstack-lbaas01:38
johnsomHa, it was just a WIP01:46
johnsomI did remove all of the tenant_id references01:47
rm_mobileLol nice01:47
johnsomDid my prediction of Sam Adams come true?01:48
rm_mobileWait which was that? :P01:51
johnsomI was predicting the beer they would have at the Fenway event01:52
rm_mobileAh they have...01:52
rm_mobileI honestly didn't see if that had that01:53
rm_mobileI saw blue moon :P01:53
rm_mobileAnd Budweiser T_T01:53
*** rm_mobile has quit IRC01:55
*** rm_mobile has joined #openstack-lbaas01:55
rm_mobileWhoops01:56
rm_mobileAnyway yeah... Time to do something else01:56
johnsomHahaha02:00
johnsomJust to be clear, I mock the Boston lager02:01
rm_mobileLol I am not a huge fan of most beer :P don't like hops02:03
rm_mobileWhelp, I'm gonna head off. Catch you tomorrow02:03
*** rm_mobile has quit IRC02:03
*** amotoki has joined #openstack-lbaas02:15
*** gongysh has joined #openstack-lbaas02:38
*** cpuga has quit IRC02:44
*** cpuga has joined #openstack-lbaas02:45
*** gongysh has quit IRC02:55
*** amotoki has quit IRC02:56
*** fnaval has quit IRC03:05
*** fnaval_ has joined #openstack-lbaas03:05
*** csomerville has joined #openstack-lbaas03:06
*** cody-somerville has quit IRC03:09
*** fnaval_ has quit IRC03:12
*** cpuga_ has joined #openstack-lbaas03:50
*** cpuga has quit IRC03:53
*** cpuga has joined #openstack-lbaas03:54
*** cpuga__ has joined #openstack-lbaas03:56
*** cpuga_ has quit IRC03:58
*** cpuga has quit IRC03:59
*** csomerville has quit IRC04:25
openstackgerritMerged openstack/octavia master: Add monitor address and port to pool member  https://review.openstack.org/46164904:47
openstackgerritMerged openstack/octavia master: Add support for single-create for APIv2  https://review.openstack.org/45903204:51
rm_work\o/04:51
johnsomYay, hopefully xgerman will get the stable/newton patch too04:52
rm_workdidn't he?04:52
rm_workmost of them merged04:52
rm_workoh, right, they don't announce here04:52
rm_worki have just been staring at zuul like a hawk for 30m04:53
rm_workthere's a merge conflict on the stable data_utils04:53
rm_worki was gonna try to fix it in a sec04:53
rm_workbut was working on this04:53
rm_worknot sure why it was working intermittently?04:53
rm_workbut, mocking the context fixes it for good I think...04:54
openstackgerritAdam Harwell proposed openstack/octavia master: Introduce nlbaas shim driver layer for APIv2  https://review.openstack.org/40939804:54
rm_workand need to recheck cascade-delete but waiting for it to officially fail04:54
rm_workweird, rebase button worked04:55
rm_workwell, i guess we'll see04:55
rm_workapparently the VIP area at the Fenway party was awesome :P04:56
rm_workunfortunately I'm not VIP material04:56
rm_workdamn, going to need to recheck paging too05:00
*** csomerville has joined #openstack-lbaas05:01
openstackgerritMerged openstack/octavia master: Implement sorting and pagination for octavia  https://review.openstack.org/38214705:02
rm_workooor not05:02
rm_workweird05:03
rm_worki watched it fail in the console...05:03
rm_workoh nm thats a warning05:03
rm_work2017-05-10 05:01:22.132482 | 2017-05-10 05:01:22.131 | sys:1: ResourceWarning: unclosed file <_io.FileIO name=1 mode='wb' closefd=True>05:04
rm_workweird05:10
rm_worki wonder if we f'd something up somewhere or if that's internal and not our fault05:10
openstackgerritAdam Harwell proposed openstack/octavia master: WIP: Floating IP Network Driver (spans L3s)  https://review.openstack.org/43561205:10
*** fnaval has joined #openstack-lbaas05:13
*** aojea has joined #openstack-lbaas05:16
*** fnaval has quit IRC05:18
*** aojea has quit IRC05:20
*** gcheresh has joined #openstack-lbaas05:21
rm_workwtf something is pulling in eventlet05:26
rm_worki swear to #*U%#WJ05:26
rm_workhmm i'm going to decompose my FLIP driver into a few patches, a think a couple could actually merge, and at least one addresses an issue someone brought up in the lab that is an issue in their env too05:27
*** JudeC has joined #openstack-lbaas05:32
openstackgerritAdam Harwell proposed openstack/octavia master: Allow the vip_port_id to be slightly overloaded  https://review.openstack.org/46365705:32
johnsomUgh, we need to figure out this 404 issue....05:36
johnsomhttp://logs.openstack.org/82/461182/5/gate/gate-octavia-v1-dsvm-py3x-scenario-ubuntu-xenial/37dd60c/logs/screen-o-cw.txt.gz#_May_10_04_17_19_00531305:37
rm_workblegh yeah05:44
openstackgerritAdam Harwell proposed openstack/octavia master: Allow specifying empty amp_boot_network_list  https://review.openstack.org/46366005:45
openstackgerritJude Cross proposed openstack/python-octaviaclient master: Add l7rule commands to client  https://review.openstack.org/46280805:52
*** pcaruana has joined #openstack-lbaas05:57
rm_workugh I am going to need coffee in the morning06:01
JudeClol06:01
JudeC2am is up pretty late06:01
*** rcernin has joined #openstack-lbaas06:06
openstackgerritAdam Harwell proposed openstack/octavia master: WIP: Floating IP Network Driver (spans L3s)  https://review.openstack.org/43561206:08
rm_workmeeting xgerman to dry-run slides at about 8:15  <_<06:08
rm_workugh one of these days cascade-delete will pass both check+gate without a dumb bug06:11
*** pcaruana|afk| has joined #openstack-lbaas06:11
JudeClol06:11
rm_workagain, so glad we don't have EVEN MORE gates voting (neutron-lbaas)06:12
*** pcaruana has quit IRC06:12
*** pcaruana|afk| has quit IRC06:12
*** pcaruana has joined #openstack-lbaas06:13
openstackgerritAdam Harwell proposed openstack/octavia master: Additional logging in udp_heartbeat and post_plug_vip  https://review.openstack.org/46366506:15
*** cpuga__ has quit IRC06:16
*** cpuga has joined #openstack-lbaas06:16
*** cpuga has quit IRC06:16
*** cpuga has joined #openstack-lbaas06:17
*** cpuga has quit IRC06:17
openstackgerritAdam Harwell proposed openstack/octavia master: CalculateAmphoraDelta networks based on vrrp_port  https://review.openstack.org/46366706:23
*** chlong has quit IRC06:23
rm_workok stable data_utils newton patch merged06:24
rm_workyeah going to bed06:27
rm_workwaking up in... 4.5 hours06:27
rm_workI'm sure German could handle the talk by himself... <_<06:27
rm_workother than not having a laptop to show the slides...06:28
rm_workor the most up to date copy of them...06:28
* rm_work dies06:28
*** yamamoto has quit IRC06:29
rm_workkidding me, that one failed due to a reqs install fail06:45
rm_workugh06:45
rm_workJudeC: if you're up when this actually finishes failing, could you recheck it for me? :)06:46
rm_workhttps://review.openstack.org/#/c/463591/06:46
JudeCGo to bed!06:46
JudeCalso ok06:46
rm_workpretty sure it is failed but want it to complete06:46
rm_workk >_<06:47
JudeClol06:47
rm_worki'm addicted to merging code06:47
JudeCI am so close I cant help but finish what I am working on either06:47
rm_workgotta get my fix06:48
rm_worki'm a bad role model06:48
rm_workdon't do what i do06:48
rm_workAH there it went06:48
rm_worki got it06:48
*** csomerville has quit IRC06:50
*** tesseract has joined #openstack-lbaas06:56
*** yamamoto has joined #openstack-lbaas07:13
*** fnaval has joined #openstack-lbaas07:13
*** fnaval has quit IRC07:18
*** aojea has joined #openstack-lbaas07:20
openstackgerritMerged openstack/octavia master: Add cascade delete for APIv2  https://review.openstack.org/46118207:55
*** belharar has joined #openstack-lbaas08:06
nmagnezi_xgerman, best loadbalancer :D08:50
openstackgerritNir Magnezi proposed openstack/octavia master: Devstack plugin should mark the HM ovs port for cleanup skip  https://review.openstack.org/46052809:03
*** fnaval has joined #openstack-lbaas09:14
*** fnaval has quit IRC09:18
openstackgerritMerged openstack/octavia master: Two quick fixes (one log message, one cert create script thing)  https://review.openstack.org/46359109:21
*** JudeC has quit IRC09:33
*** aojea has quit IRC09:36
*** aojea has joined #openstack-lbaas09:38
openstackgerritNir Magnezi proposed openstack/octavia master: Devstack plugin should mark the HM ovs port for cleanup skip  https://review.openstack.org/46052809:44
*** jidar has quit IRC10:08
*** jidar has joined #openstack-lbaas10:09
*** sanfern has quit IRC10:39
*** gongysh has joined #openstack-lbaas10:49
*** gongysh has quit IRC10:51
*** kbyrne has quit IRC11:20
*** kbyrne has joined #openstack-lbaas11:21
*** amotoki has joined #openstack-lbaas11:34
*** chlong has joined #openstack-lbaas11:59
*** amotoki has quit IRC12:22
*** yamamoto has quit IRC12:28
*** yamamoto has joined #openstack-lbaas12:30
*** krypto has joined #openstack-lbaas12:32
*** amotoki has joined #openstack-lbaas12:35
*** yamamoto has quit IRC12:35
*** sanfern has joined #openstack-lbaas12:48
*** voelzmo has joined #openstack-lbaas12:49
*** rm_mobile has joined #openstack-lbaas12:51
rm_mobilejohnsom: we added a slide about the namespace driver12:51
rm_mobileHopefully we don't say anything to horribly wrong12:51
rm_mobileWe agreed it's moving out of tree, right?12:51
johnsomIt should, yes12:52
rm_mobileKk12:52
rm_mobileYou have any notes on the user survey?12:54
rm_mobileI'm doing those slides but all I know to say is "most requested feature then and now"12:54
rm_mobilejohnsom:12:54
johnsomUmm, that and over 50% of the respondents indicated that, more than any other networking feature12:55
rm_mobileIs the survey for networking stuff only?12:56
rm_mobileOr all of openstack12:56
johnsomWe weren't explicitly included in the survey so that is the only mention.12:56
johnsomAll of Openstack12:56
rm_mobileOk12:57
johnsomThe slide in the survey is only for networking12:57
*** amotoki has quit IRC12:57
rm_mobileThen it's... More than any other *feature* at all12:57
rm_mobileAh12:57
rm_mobileOk12:57
rm_mobileThat's what I meant12:57
johnsomNo, networking12:57
rm_mobileK12:57
johnsomI usually thank people for filling out the survey and supporting us12:58
rm_mobileKk12:58
rm_mobileIs that sent out to everyone?12:59
johnsomYes, not just devs12:59
rm_mobileKk12:59
*** yamamoto has joined #openstack-lbaas13:00
*** Andy_M has joined #openstack-lbaas13:03
*** amotoki has joined #openstack-lbaas13:03
*** KeithMnemonic has joined #openstack-lbaas13:05
*** chlong has quit IRC13:10
*** sanfern has quit IRC13:21
*** sanfern has joined #openstack-lbaas13:21
*** ttyKitty has joined #openstack-lbaas13:30
*** yamamoto has quit IRC13:32
*** ttyKitty has quit IRC13:36
*** cpuga has joined #openstack-lbaas13:44
*** sanfern has quit IRC13:45
*** KeithMnemonic has quit IRC13:46
*** cpuga_ has joined #openstack-lbaas13:49
rm_mobilejohnsom: hopefully if I said anything too horrifying I can walk it back at the project update13:50
*** cpuga has quit IRC13:50
rm_mobileOverall I think it went ok though13:50
*** rm_mobile has quit IRC13:51
johnsomI am sure you did great.  Out getting coffee, I will watch it when I get home13:52
rm_worklol13:53
rm_workdo you get coffee outside every morning? :P13:53
rm_workyou should get a keurig13:53
rm_workor one of those latte machines13:53
rm_workmmm13:53
openstackgerritAdam Harwell proposed openstack/octavia master: Additional logging in udp_heartbeat and post_plug_vip  https://review.openstack.org/46366513:54
openstackgerritAdam Harwell proposed openstack/octavia master: Allow specifying empty amp_boot_network_list  https://review.openstack.org/46366013:54
openstackgerritAdam Harwell proposed openstack/octavia master: CalculateAmphoraDelta networks based on vrrp_port  https://review.openstack.org/46366713:54
openstackgerritAdam Harwell proposed openstack/octavia master: WIP: Floating IP Network Driver (spans L3s)  https://review.openstack.org/43561213:55
rm_workshoving all of those into a nice little line13:55
johnsomIt is rare that I go put13:57
*** catintheroof has joined #openstack-lbaas13:59
*** yamamoto has joined #openstack-lbaas14:02
rm_workjohnsom: i have updated versions of the slides14:02
rm_workso if they ask for them, let me give you the most up to date version14:02
johnsomOk14:02
rm_workjohnsom: also, could use notes for the project update slides I guess, if you have any14:05
rm_workspecifically i'm not sure what to say about release themes14:05
rm_workjust ... read the slide I guess?14:05
*** yamamoto has quit IRC14:06
rm_work* Allow vendor drivers to plug into the Octavia V2 API endpoint14:07
rm_work   - Either directly as an Octavia “provider” or using the “shim driver”14:07
rm_workdoes that seem right?14:07
johnsomSounds right14:07
johnsomI planned to just talk about proposed work items on those slides.  I.e. Act/act, horizon, drivers, etc14:08
*** bzhao has quit IRC14:12
*** bzhao has joined #openstack-lbaas14:12
rm_workk14:19
rm_workjohnsom: my OCD on these "release themes" bubbles has had me adjusting them for like 10 minutes14:19
rm_workthey're all "slightly" off >_<14:19
rm_workfrom slide to slide14:19
*** chlong has joined #openstack-lbaas14:23
johnsomThey were auto-generated by the foundation...14:26
johnsomOk, I am back, is there a certain slide you would like notes on?14:26
johnsomThe project update was "new" so, not sure what all to include.  I think German said he would talk slowly...  grin14:27
*** Andy_M has quit IRC14:29
*** krypto has quit IRC14:32
rm_workjohnsom: for the project update, how specific is the format?14:33
rm_workare we not allowed to brand with our logo?14:33
rm_worklooking at the mistral video from this morning and they had a big logo plastered on the first few slides :P14:33
johnsomWell, they lost our "official" one, so, do whatever IMO14:33
rm_worklol14:34
rm_workhow long is the PU14:34
rm_workshort?14:34
*** krypto has joined #openstack-lbaas14:34
*** krypto has quit IRC14:34
*** krypto has joined #openstack-lbaas14:34
johnsomI think it is supposed to be short for a longer Q/A section.14:34
rm_workk14:34
rm_workuhh is there seriously nothing we ever had as "not a focus"? lol14:34
rm_workconstantly completely split focus on everything14:35
*** fnaval_ has joined #openstack-lbaas14:35
*** amotoki has quit IRC14:36
*** catintheroof has quit IRC14:36
rm_work"join the PTL of Octavia!" <_<14:36
johnsomIt's 40 minutes.14:37
rm_workk yeah14:38
johnsomWant to be PTL for a day???14:38
johnsomGrin14:38
rm_workthe mistral update video was 10m14:38
johnsomYeah, they are all over the board.  I don't think any of us really knew what to expect....14:39
rm_workk14:42
rm_workI shoved the octavia logo in the bottom right of the first few slides14:42
rm_workjohnsom: are you home and online yet? :P14:43
johnsomyes14:43
rm_workjohnsom: can you take a look at:14:44
rm_workhttps://review.openstack.org/#/c/463665/14:44
rm_workand the rest of that chain14:44
rm_workhttps://review.openstack.org/#/c/463660/14:44
rm_workhttps://review.openstack.org/#/c/463667/14:44
rm_workthose two are ... the ones that might actually need consideration14:44
rm_workthe FLIP driver is still just WIP obviously so whatever14:44
*** krypto has quit IRC14:45
*** krypto has joined #openstack-lbaas14:46
*** krypto has quit IRC14:46
*** krypto has joined #openstack-lbaas14:46
*** gcheresh has quit IRC14:48
rm_workI was busy last night decomposing the FLIP driver stuff so we can get some of the fixes/enhancements merged14:48
*** amotoki has joined #openstack-lbaas14:48
johnsomThe UDP logger, that is almost a trace level thing as it will be so noisy, but It's ok for now.  We may change our mind later14:49
rm_workhmm yeah14:50
rm_workit's DEBUG level14:50
rm_workit will be noisy14:50
rm_workif you don't like it I can pull it back out, but it's useful for debugging14:50
rm_workand you shouldn't be running debug in a prod cloud right? :P14:50
johnsomI know a shipping version of packaged OpenStack that always runs trace.  I thought they were crazy and hated looking at those logs....14:51
johnsomMy guess is they just wanted to sell more disk14:51
rm_worklol14:51
rm_workis there something higher that debug? lol14:51
*** atoth has joined #openstack-lbaas14:51
johnsomYeah, there really is TRACE14:51
rm_workok14:52
rm_workso I could change it to that14:52
johnsomNo, it's fine14:52
johnsomLike I said, we can always change it later14:52
*** aojea has quit IRC14:52
rm_workjohnsom: k, german is requesting it be trace :P14:52
*** amotoki has quit IRC14:52
rm_worki'll... leave it for now14:53
johnsomWell, I +2'd debug, so...14:53
rm_workyeah he says he doesn't feel THAT strongly :)14:53
johnsomDid like no one show up?14:54
rm_workTOTALLY PACKED14:54
rm_work#altfact14:54
rm_work(I made that joke)14:54
*** amotoki has joined #openstack-lbaas14:55
johnsomWell, at least the sessions weren't back to back....14:55
rm_workah here we are: https://www.openstack.org/videos/boston-2017/octavia-load-balancing-for-openstack14:55
*** amotoki has quit IRC14:55
rm_workyeah14:55
johnsomYeah, just posted14:55
rm_workhmm wonder who I contact to get my name on it instead of Lubosz14:56
johnsomYou did get a few positive reviews14:56
johnsomspeaker support e-mail14:56
rm_workreviews? :P14:56
johnsomYeah, they are popping up on the conference schedule details page for the talk14:56
*** sanfern has joined #openstack-lbaas14:57
xgermanHow do we find reviews?14:58
rm_workah14:58
xgermanAlso why don't we have an onboarding session like the cool kids?14:58
*** Andy_M has joined #openstack-lbaas14:58
johnsomBecause we already had four talks and I didn't feel like signing us up for more14:59
johnsomReviews: https://www.openstack.org/summit/boston-2017/summit-schedule/events/17859/octavia-load-balancing-for-openstack14:59
xgermanRed_robot did 6 talks- now I feel like a slacker14:59
*** amotoki has joined #openstack-lbaas15:00
rm_worklol15:01
*** catintheroof has joined #openstack-lbaas15:02
*** yamamoto has joined #openstack-lbaas15:04
*** belharar has quit IRC15:05
*** voelzmo has quit IRC15:06
*** fnaval has joined #openstack-lbaas15:08
*** rcernin has quit IRC15:11
*** fnaval_ has quit IRC15:11
*** yamamoto has quit IRC15:14
*** sanfern has quit IRC15:17
*** amotoki has quit IRC15:21
*** ianychoi has quit IRC15:26
*** amotoki has joined #openstack-lbaas15:35
*** cpuga_ has quit IRC15:37
*** cpuga has joined #openstack-lbaas15:41
*** Andy_M has quit IRC15:45
*** Andy_M has joined #openstack-lbaas15:48
*** Andy_M has quit IRC15:53
*** catintheroof has quit IRC15:54
*** tesseract has quit IRC15:57
openstackgerritMerged openstack/octavia master: Additional logging in udp_heartbeat and post_plug_vip  https://review.openstack.org/46366515:57
*** blogan has quit IRC16:00
*** blogan has joined #openstack-lbaas16:00
*** krypto has quit IRC16:12
*** krypto has joined #openstack-lbaas16:13
*** atoth has quit IRC16:18
*** krypto has quit IRC16:20
*** fnaval has quit IRC16:23
*** amotoki has quit IRC16:24
*** atoth has joined #openstack-lbaas16:31
*** JudeC has joined #openstack-lbaas16:37
*** fnaval has joined #openstack-lbaas16:38
*** ianychoi has joined #openstack-lbaas16:47
openstackgerritJude Cross proposed openstack/python-octaviaclient master: Add healthmonitor commands to client  https://review.openstack.org/46383016:56
*** cpuga has quit IRC17:03
*** ianychoi has quit IRC17:09
johnsomm-greene m-greene_ You guys overloaded l7policy???17:10
*** cody-somerville has joined #openstack-lbaas17:11
*** csomerville has joined #openstack-lbaas17:12
JudeCAll commands are in for the client. johnsom: is there somewhere I can look to get more defined help text for all of the commands. I know my help text kinda sucks.17:12
johnsomJudeC Umm, the source for that should probably come from the new API documentation (which I'm only up to pools so far).  As a baseline we can steal from the old neutron client, but otherwise I guess it will come in through reviews/comments.17:13
*** amotoki has joined #openstack-lbaas17:13
JudeCOK :)17:14
JudeCty17:14
johnsomhttp://docs-draft.openstack.org/72/458272/3/check/gate-octavia-api-ref/b23453d//api-ref/build/html/17:14
johnsomI hope to have pools done today17:14
*** cody-somerville has quit IRC17:15
openstackgerritJude Cross proposed openstack/python-octaviaclient master: Add healthmonitor commands to client  https://review.openstack.org/46383017:31
rm_workjohnsom: how do we get our slides uploaded somewhere by the summit folks? same email?17:41
*** cpuga has joined #openstack-lbaas17:41
johnsomrm_work watching the video...  UDP...17:42
rm_workjohnsom: lol17:42
rm_workyeah17:42
rm_worki'm coding it right now17:42
johnsomSo, you should be able to log into the summit schedule system, then go to the talk and there should be a "upload option"17:42
*** cpuga_ has joined #openstack-lbaas17:42
rm_workgoing to have the API query the driver for features17:42
rm_workdefault if unimplemented is our standard list17:43
johnsomYeah, I have been noodling over that17:43
rm_worktrying to imagine how to do it generically for all features, not just listener protocol17:43
rm_workI have a plan17:43
rm_workI'll submit it in code form and you can tell me what you think17:43
johnsomThinking a get_driver_features() then check against a well known list with IN?17:43
rm_workI figure it'll take me... about 45m17:43
rm_workyes17:43
rm_workconstants17:43
johnsomYeah, great minds....17:44
rm_workvendor drivers should also define/use constants17:44
johnsomrm_work Yeah, once they booted me from the sessions I can't upload anymore.17:46
rm_workT_T17:46
rm_workk17:46
*** cpuga has quit IRC17:46
rm_workjohnsom: it's the ... amphora driver?17:46
rm_workor17:47
rm_workis it the controller driver?17:47
rm_workthat will control the features17:47
*** amotoki has quit IRC17:47
johnsomhandler right?17:47
johnsomIt has to be at the API level so it can reject the request before the async hand off to the handler17:48
johnsomrm_work BTW, part of the UDP confusion is the old API docs for lbaas somehow got UDP added to the list.  Another example of why that stuff needs to be in-tree17:49
*** amotoki has joined #openstack-lbaas17:49
rm_workyeah it's the provider17:49
rm_workso, handler17:49
*** fnaval has quit IRC17:53
*** fnaval has joined #openstack-lbaas17:54
*** zz_amotoki_ has joined #openstack-lbaas17:55
*** fnaval has quit IRC17:55
*** fnaval has joined #openstack-lbaas17:56
johnsomYeah, so we would need a reference UDP implementation for testing....17:58
johnsomWe might be able to use LVS for that17:59
*** atoth has quit IRC18:02
*** atoth has joined #openstack-lbaas18:04
*** blogan_ has joined #openstack-lbaas18:06
*** blogan has quit IRC18:08
*** aojea has joined #openstack-lbaas18:20
*** amotoki has quit IRC18:24
*** aojea has quit IRC18:25
*** amotoki has joined #openstack-lbaas18:25
*** amotoki has quit IRC18:25
*** bread_ has quit IRC18:30
*** sanfern has joined #openstack-lbaas18:31
*** amotoki has joined #openstack-lbaas18:33
*** amotoki has quit IRC18:33
rm_workjohnsom: ok soooooo this is my first take:18:34
rm_worksorry that was premature, still running tests18:34
xgermanAll the K8 sessions Demo LBaaS18:35
*** voelzmo has joined #openstack-lbaas18:36
*** gcheresh has joined #openstack-lbaas18:40
xgermanWe are Huge!!!18:44
*** ianychoi has joined #openstack-lbaas18:44
*** JudeC has quit IRC18:44
rm_worklheh18:46
openstackgerritAdam Harwell proposed openstack/octavia master: Allow using custom things like Listener Protocols  https://review.openstack.org/46385118:46
rm_workOK there we go18:46
rm_workthat's my first take18:46
rm_workI might add pool protocol to that as well18:46
rm_workerr since I guess it needs to be? :P18:46
*** aojea has joined #openstack-lbaas18:46
rm_workbut that's the idea18:46
*** voelzmo has quit IRC18:53
*** atoth has quit IRC18:57
*** fnaval has quit IRC19:01
*** fnaval has joined #openstack-lbaas19:01
*** fnaval has quit IRC19:02
*** fnaval has joined #openstack-lbaas19:02
*** fnaval has quit IRC19:04
*** fnaval_ has joined #openstack-lbaas19:04
*** fnaval_ has quit IRC19:04
*** fnaval has joined #openstack-lbaas19:04
*** sanfern has quit IRC19:04
*** sanfern has joined #openstack-lbaas19:05
johnsomOk, I will have a look after lunch.  Just got done with a 1-1.  I think I will comment about the wording a bit, it's not like we will support arbitrary things...19:09
johnsomxgerman which session were you in?19:09
xgermanKuryr19:09
xgermanAnd some etherpad driven K8 on OS working group19:10
johnsomOk, cool, interested to take a look19:10
xgermanYeah. Now ML supported networking ...19:10
rm_workjohnsom: technically could support anything that comes in as an API field19:14
rm_workbut mostly what makes sense is stuff that's currently enum19:14
johnsomBut we have to test it, have it in the enum, and in the docs...19:16
rm_workwell19:16
rm_worksee the review19:17
rm_workit's validated19:17
rm_workper handler19:17
rm_workand we have it in constants19:17
*** voelzmo has joined #openstack-lbaas19:30
*** sanfern has quit IRC19:31
*** amotoki has joined #openstack-lbaas19:32
*** voelzmo has quit IRC19:35
rm_workjohnsom: couple of things, w/r/t TLS storage19:43
rm_workA) Cascading ACLs is coming (we knew this)19:44
rm_workB) Looking at allowing ACL creation for a service-type (so say "I want `load-balancer` to access my secrets") and have it map to the userid, instead of having to publish our service account name19:44
johnsomIs it?  Was wondering with Doug's change19:44
rm_workC) Instead of using Barbican certificate containers at all, we just require a PKCS12 formatted cert bundle, and it's stored as a single secret in Barbican, so we don't even use containers -- then we can use Castellan directly, and any backend will be able to more easily handle the formatting for our requirements in a well defined way19:46
johnsomB, if our API process is handling it, we don't have to publish the account anywhere right?19:46
rm_worksince PKCS12 *is* the defined format for how to store a cert and PK and ints together19:46
rm_workjohnsom: *we* don't add ACLs, the end-user has to create the ACLs19:46
johnsomI think we have had the pkcs12 discussion before19:46
rm_workI don't remember it internally to our project19:47
rm_workI've talked about it a ton with the barbican folks19:47
rm_workbut I think it's not a half-bad idea19:47
rm_workand we don't technically need to change the API signature19:47
johnsomrm_work: no, that is the change I want to make19:47
*** sanfern has joined #openstack-lbaas19:47
rm_work"container_id" is fine if it's a secret that's a single-container19:47
rm_workpkcs12 is a "container"19:47
rm_workso i think it's fine19:47
rm_workdon't need to change anything19:47
johnsomugh, ok, back at a real keyboard19:47
johnsomrm_work on ACLs, my plan is to have the API server, when it gets container IDs to use the user's token on their behalf to give ACL to what we need in barbican19:48
johnsomWe have the token now that we aren't behind neutron19:48
rm_workah true19:49
rm_workthat's a thought19:49
rm_workseems icky <_<19:49
rm_workalso you have weird edge-conditions with expiry times but probably not a HUGE problem19:49
johnsomIt's what the user intends to do...19:49
rm_workyeah19:49
rm_workfine19:49
rm_worki wouldn't block it19:50
johnsomWell, if it expires we just return to the user that their token expired and they have to make the request again19:50
*** zz_amotoki_ is now known as amotoki_19:50
johnsomJust like our API should do anyway19:50
johnsomOn C, I have no problem with that as long as we don't change the API.  Though it seems like changing the content is a bit of a stretch on if it is an API change or not.19:51
johnsomOur discussion on PKCS12 was me asking why we weren't doing it.19:52
johnsomNot sure if that was an issue with validation, or passphrases or....19:52
rm_worknah19:56
rm_workwe just weren't19:56
rm_workso, we'd basically stop using Containers in Barbican altogether19:56
rm_workPOOL protocol has to match LISTENER protocol, right?19:57
rm_workhow do we do that validation?19:57
johnsomNo19:57
rm_workhmm19:57
rm_workI am pretty sure we validate that19:57
johnsomThey do not have to match19:57
johnsomExample, listener TCP, pool PROXY19:57
rm_workI got an error about creating a pool with a protocol that wasn't valid for the listener protocol19:57
rm_worklistener HTTPS pool HTTP19:58
rm_workfailed19:58
*** blogan__ has joined #openstack-lbaas19:58
johnsomTerminated HTTPS to HTTP\19:58
rm_worksomewhere it's validating tha19:58
rm_workerr maybe it was that19:58
rm_workwhere does it do THAT validation19:58
rm_workbecause I think UDP needs to go to UDP >_>19:58
rm_workI don't think we can do UDP -> HTTP right? lol19:58
johnsomYeah, HTTPS (TCP) to HTTP, while in theory should work, I don't think it does with HAProxy19:58
rm_workbut we actually do the validation19:59
johnsomYeah, possibly.  Doesn't mean it is right....19:59
rm_work>_> k19:59
rm_workso you would rather do no validation re: protocols?19:59
johnsomNo, I am fine with validation, we just need to realize there are mis-match cases20:00
*** blogan_ has quit IRC20:00
johnsomListenerPoolHTTPHTTPHTTPSTCPTCPTCPTerminated HTTPSHTTPTCPPROXY https://usercontent.irccloud-cdn.com/file/vDLt18rM/image.png20:03
johnsomI think that is the grid20:03
johnsomIt gets a bit complicated in the api side though with shared pools and all.  There are multiple places it would need to be checked.  graph create, pool to listener, pool to l7policy to listener, etc.20:04
johnsomThere is some strange mapping code for pools that should probably go away and just have two separate constants lists.20:06
*** catintheroof has joined #openstack-lbaas20:08
*** amotoki has quit IRC20:09
*** amotoki_ is now known as zz_amotoki_20:09
*** sanfern has quit IRC20:10
*** sanfern has joined #openstack-lbaas20:15
rm_workjohnsom: right that was what i was curious about -- where is this validation done20:16
*** blogan has joined #openstack-lbaas20:17
johnsomThe mapping I am thinking of isn't done for validation, it's just done to convert the pool protocol to something sane as it's stored in the database20:17
rm_workhmm20:17
rm_workdo I need to add UDP to that?20:18
*** blogan__ has quit IRC20:18
johnsomWe don't have a UDP backend to test with unless you quickly did the lvs implementation....20:18
rm_worklol20:18
rm_workI did not20:18
rm_workbut i mean20:18
rm_worki can get the API pieces ready20:18
rm_workfor when radware can test20:18
rm_workwith their CI20:19
rm_workand their driver20:19
johnsomThey don't test right now20:19
rm_workbut they will soon20:19
rm_workSam is ready20:19
rm_workjust waiting on us supposedly20:19
rm_workso I'm making sure there's nothing he can be waiting on us for20:19
johnsomYeah, for providers support20:19
rm_workyeah20:19
rm_workthat's next20:19
rm_workI just did this now cause it's *easy*20:19
johnsomIt's a lot of work to add UDP support to octavia...20:20
johnsomWe have never had that, so why would that block radware?20:20
xgermanI want to see one more talk to then I can focus on adding value here :-)20:20
openstackgerritAdam Harwell proposed openstack/octavia master: Allow using custom enum values for API fields  https://review.openstack.org/46385120:21
rm_workjohnsom: well, that adds it ^^20:21
rm_workfor vendor drivers that support it20:21
xgermanBut there are people who like to add UDP for us20:21
rm_workyes20:21
rm_workyou talked to them, right?20:21
xgermanLet's see... Ubisoft needs UDP too20:21
xgermanYep20:21
johnsomIt needs a reference implementation, some kind of target, etc. to test20:21
rm_workI put LB_ALGORITHM in there as an example20:22
rm_workas well20:22
rm_worksince it's an enum20:22
xgermanBut first they like to add multithreaded TLS termination20:22
rm_workand there's a few more we could/should do to make this complete20:22
* johnsom scratches head given the other stuff we need to get working....20:22
rm_workyeah this took me ~1.5h and it's pretty close to done20:22
rm_workenough that they can do something with it20:22
xgermanjohnsom: it's open source people only work on what they like...20:22
rm_workI'm just in "remove roadblock mode"20:23
johnsomI know, just saying...20:23
rm_workprovider support is next on my list20:23
rm_workthinking about how to do that effectively now20:23
rm_workneed to add config for "available_providers" to start with20:24
johnsomIt needs a spec/driver writer guide20:24
xgermanrm_work: this is how LBaaS V1 came to be. Mark removed a roadblock...20:24
rm_workugh20:24
rm_worklol20:24
johnsomYeah, it should be done using named extensions stevedore20:24
rm_workyes20:24
rm_workso instead of loading "handler" we load "handlers_dict"20:24
rm_workand we pull the one they want20:24
rm_workbased on provider20:24
rm_workeasy enough20:24
rm_worki might do it now20:25
rm_workoh, johnsom: I asked on https://review.openstack.org/#/c/463660/ what part is confusing20:25
*** sanfern has quit IRC20:26
johnsomThe logic is a bit rough to follow at first glance that was all.  I still +2'd20:26
rm_workk20:26
rm_workyeah it was worse before I split it into multiple intermediate vars20:26
rm_workxgerman: so there's two patches that could use +A :P20:27
johnsomrm_work BTW, what you asked about: https://github.com/openstack/octavia/blob/master/octavia/common/jinja/haproxy/jinja_cfg.py#L2520:28
johnsomNot where I thought it was...20:28
rm_workok20:28
rm_workweird20:28
rm_workerm20:28
rm_workthat's not what would cause an API fail though obviously20:28
rm_workI still wonder where that is20:28
johnsomYeah, I don't see other validation on the octavia side of the house....20:29
johnsomI agree with you, that proto list should be split...20:29
xgermanTerminated-https being just http is odd in your patch20:29
xgermanOh wrong link20:30
rm_workjohnsom: yeah you saw I finally did it? :P20:30
rm_workI had meant to before20:30
rm_workbut didn't get around to it20:30
johnsomNo, I missed that patch20:31
*** zz_amotoki_ is now known as amotoki20:31
rm_workit's in https://review.openstack.org/#/c/463851/20:31
rm_workjohnsom: i'm thinking i might put that patch on top of the shim-driver patch20:31
rm_workthat way if 3rd-party drivers want to test including that as well as the shim, they can20:32
rm_workand it won't merge until one of them tests it20:32
*** gcheresh has quit IRC20:37
*** catintheroof has quit IRC20:41
*** pcaruana has quit IRC20:53
*** amotoki is now known as zz_amotoki21:01
*** zz_amotoki has quit IRC21:01
*** zz_amotoki has joined #openstack-lbaas21:02
*** zz_amotoki is now known as amotoki21:03
nmagnezi_rm_work, xgerman, currently watching the project update - good work! :-)21:03
*** amotoki_ has joined #openstack-lbaas21:04
rm_work:)21:04
*** amotoki is now known as zz_amotoki21:04
johnsomnmagnezi_ I had to put some humor in the slides/demos...  (best load balancer, etc.)21:06
*** zz_amotoki is now known as amotoki21:08
nmagnezi_johnsom, i saw it on the intro talk and liked it :-) still watching the project update one :)21:08
*** amotoki is now known as amotoki_away21:09
nmagnezi_nice to see the tripleo support was mentioned :-)21:11
*** amotoki_ has quit IRC21:16
*** amotoki has joined #openstack-lbaas21:19
*** JudeC has joined #openstack-lbaas21:20
*** amotoki has quit IRC21:25
openstackgerritAdam Harwell proposed openstack/octavia master: Allow using custom enum values for API fields  https://review.openstack.org/46385121:27
rm_workjohnsom: refactored that a bit working with xgerman21:27
rm_worki think this way is a little easier21:28
rm_workI hope people liked my #altfact joke <_<21:28
rm_workI stole it from German21:28
*** amotoki has joined #openstack-lbaas21:29
nmagnezi_rm_work, I actually LOLed when you danced there.. :)21:34
rm_worklol21:35
rm_workit was supposed to be punishment21:35
nmagnezi_hah21:35
johnsomIt seemed to work....21:35
nmagnezi_the crowed remained silent so I think johnsom is right :D21:35
nmagnezi_rm_work, btw, i noticed there was no mentioning of containers for future goals21:37
rm_workah did we not somehow?21:38
rm_workwe mentioned it in the earlier talk a lot21:38
rm_workblegh21:38
rm_workok21:38
johnsomYeah, I might have missed it on that last slide deck.  So much to do, so little slide space and time....21:38
rm_workjohnsom: I have been convinced that we do, in fact, need a spec for providers T_T21:38
rm_workthere are a number of questions21:38
rm_workthat need to be agreed upon21:39
johnsomSigh, yeah, I know...21:39
* johnsom thinks all slides and no code makes johnsom a dull boy21:39
openstackgerritMerged openstack/octavia master: Allow specifying empty amp_boot_network_list  https://review.openstack.org/46366021:40
openstackgerritMerged openstack/octavia master: CalculateAmphoraDelta networks based on vrrp_port  https://review.openstack.org/46366721:40
nmagnezi_rm_work, https://review.openstack.org/#/c/463615 ... mind = blown21:42
nmagnezi_rm_work, i never knew it got removed21:42
rm_worklol21:43
rm_workwe deemed it wasn't useful21:43
rm_workbecause there was no way to PUT THINGS IN IT21:43
rm_workbut i guess if you just ... put the file there by hand for testing21:43
rm_workit was removed SOOOO long ago21:43
rm_worklike21:43
rm_workmitaka maybe21:43
johnsomExcept from the example conf file... grin21:44
rm_work*originally* I had intended for the API to take in the certificate_container_id OR the individual cert data, and we'd store it on behalf of the user in the cert system21:44
rm_workso it would have been usefu21:44
rm_workbut then we ... didn't include that somehow21:44
*** aojea has quit IRC21:44
rm_workkinda forgot about it21:44
rm_workand so it wasn't useful21:45
rm_workthere was a suggestion at the key-manager forum today that we add that back in21:45
nmagnezi_rm_work, i don't feel worthy of reviewing this patch cuz i had no idea this is absent.. lol21:45
nmagnezi_:P21:45
*** JudeC has quit IRC21:47
rm_worklol21:49
rm_workas johnsom said, it has apparently never been removed from the config :P21:52
rm_workso I f'd up too21:52
rm_workjohnsom: what do you think about entirely skipping the whole "providers" thing and JUST having flavors21:57
rm_workcause, xgerman and I voted here, and it's 2-0 for just accomplishing providers as part of flavors21:57
rm_workand since summit votes are the only thing that matters...21:57
nmagnezi_xgerman, this one looks important https://review.openstack.org/#/c/444347 , I tried to locally resolve the merge conflicts but there was something I'm not so sure about :/21:58
johnsomEven though the implementation in LBaaS of flavor was == provider, we really should keep provider for backwards compatibility.  It was a part of LBaaSv2 that was actually used...(unlike flavors)21:58
johnsomrm_work ^^^^21:59
*** sanfern has joined #openstack-lbaas21:59
*** amotoki has quit IRC21:59
* johnsom thinks that now the presentations are over rm_work is creating LBaaSv3 API....21:59
johnsomThat would be popular...21:59
nmagnezi_lol22:00
xgermanProvider is a flavor without metadata22:00
xgermanSo can be API compatible22:00
johnsomand defined names22:00
xgermanYep22:00
rm_workbleh k so then i guess it WOULD technically be easier to do providers first22:00
xgermanMeh22:01
*** sanfern has quit IRC22:01
rm_workbbl drink time22:01
johnsomNice22:02
*** cpuga_ has quit IRC22:12
*** catintheroof has joined #openstack-lbaas22:38
*** csomerville has quit IRC22:44
*** catintheroof has quit IRC22:56
*** catintheroof has joined #openstack-lbaas22:57
*** catintheroof has quit IRC22:57
*** fnaval has quit IRC22:59
openstackgerritMichael Johnson proposed openstack/octavia master: Add v2 pool API section  https://review.openstack.org/45827223:41

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