20:00:04 <johnsom> #startmeeting Octavia
20:00:05 <openstack> Meeting started Wed Nov 29 20:00:04 2017 UTC and is due to finish in 60 minutes.  The chair is johnsom. Information about MeetBot at http://wiki.debian.org/MeetBot.
20:00:06 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
20:00:08 <openstack> The meeting name has been set to 'octavia'
20:00:11 <johnsom> Hi folks
20:00:14 <nmagnezi> o/
20:00:18 <longstaff> hi
20:00:45 <johnsom> I am back from a short holiday.
20:00:56 <johnsom> #topic Announcements
20:01:06 <johnsom> Not much on the announcement front.
20:01:21 <johnsom> The PTG registration is open for Dublin in February.
20:01:37 <johnsom> #link https://www.openstack.org/ptg
20:01:50 <xgerman_> o/
20:02:16 <johnsom> Let me know if you are planning to attend as the foundation is working on room sizing.
20:02:25 <johnsom> I know that German and I are attending.
20:02:53 <xgerman_> I think we still need to sign up…
20:03:02 <johnsom> Any other announcements today?
20:03:32 <xgerman_> Q-2
20:03:41 <johnsom> Oh, yeah, good one.
20:03:48 <johnsom> Queens milestone 2 is next week.
20:03:58 <xgerman_> ^^ !!!!!
20:04:07 <johnsom> #link https://releases.openstack.org/queens/schedule.html#q-release
20:04:46 <johnsom> I would really like to get these OSC parts merged so I can cut a client release
20:04:55 <johnsom> #link https://review.openstack.org/#/q/project:openstack/python-octaviaclient+status:open
20:05:42 <johnsom> #topic Brief progress reports / bugs needing review
20:06:06 <johnsom> I was out for three days this week on holiday, so mostly I am trying to catch up on e-mails and reviews.
20:06:21 <johnsom> There was a lot of good work done over the week, so lots of stuff to review.
20:06:54 <johnsom> Any other updates?
20:07:12 <johnsom> We should keep plugging away with reviews on the provider driver spec
20:07:13 <bar_> can we prioritize failover?
20:07:47 <bar_> I think it should into pike, is it not?
20:07:55 <bar_> *get backported into
20:07:56 <johnsom> #link https://review.openstack.org/509957
20:08:10 <johnsom> bar_ Prioritize failover how?  The API issue?
20:08:18 <xgerman_> ?
20:08:20 <bar_> the flow issue
20:08:34 <bar_> both my recent patch and johnsom's
20:08:39 <johnsom> #link https://review.openstack.org/#/c/523242/
20:08:43 <johnsom> I think you mean this one?
20:08:51 <bar_> yup
20:09:40 <johnsom> I think that only applies to Queens.  I'm pretty sure the patch that broke it was a queens only patch.
20:09:48 <xgerman_> +1
20:10:03 <bar_> I checked - it is in stable/pike too
20:10:51 <bar_> this is the patch that introduced the bug: 8baf0ca1d7a9a64c2ae80b5d8731b36b0d16a901
20:10:51 <openstack> bug 8 in Launchpad itself "Translator forums/means of communication" [Low,Won't fix] https://launchpad.net/bugs/8
20:11:35 <johnsom> Ha, the bot got too excited
20:12:02 <nmagnezi> lol
20:12:20 <xgerman_> https://github.com/openstack/octavia/blob/stable/pike/octavia/controller/worker/flows/amphora_flows.py#L297
20:12:25 <johnsom> bar_ I don't see the failover API in stable/pike
20:12:39 <johnsom> https://github.com/openstack/octavia/blob/stable/pike/octavia/api/v2/controllers/load_balancer.py#L468
20:12:44 <nmagnezi> johnsom, I do, at least in gerrit https://review.openstack.org/#/c/479109/
20:12:58 <nmagnezi> in the 'Include In' button
20:13:39 <johnsom> nmagnezi Yes, that patch is there, but the failover API isn't, so this patch is fine in Pike.
20:13:41 <bar_> nmagnezi, the API call itself is in a later patch, but I think it is in pike too
20:13:51 <bar_> perhaps i'm mistaken?
20:14:30 <xgerman_> API. call is not in Pike
20:14:33 <nmagnezi> johnsom, oh, okay I was under the impression we are talking about the patch bar mentioned
20:14:57 <johnsom> This one:
20:15:00 <johnsom> #link https://review.openstack.org/#/c/444347/
20:15:11 <xgerman_> so the path to the flow only causes problems in combination with the API patch
20:15:28 <johnsom> It's the combination of the two patches that runs into trouble
20:15:39 <xgerman_> +1
20:16:50 <johnsom> So, it would be good to get the fix into Q2, but I don't think we need to backport it.
20:17:13 <bar_> however, this sec_group patch i proposed could be relevant: https://review.openstack.org/#/c/523931/
20:17:49 <bar_> The is less severe indeed, but I can easily reproduce it.
20:18:02 <johnsom> That one needs review.  I'm not sure I'm confortable with a blanket exception clause like that.
20:18:15 <bar_> makes sense.
20:18:46 <johnsom> Ok, any other progress reports or updates?
20:18:52 <xgerman_> speaking of VIPs: https://review.openstack.org/#/c/523946/2
20:19:18 <xgerman_> pls. kindly review
20:19:30 <johnsom> #topic Open Discussion
20:19:36 <johnsom> Yep, we should mention that.
20:19:47 <jniesz> #link https://review.openstack.org/#/c/519509/
20:20:03 <jniesz> looks like we are close on getting that merged, just need additional core review
20:20:05 <johnsom> So, with neutron-lbaas the VIP port is owned by the tenant but the base port is owned by the neutron service account.
20:20:23 <johnsom> In octavia they are currently both owned by the octavia service account.
20:20:52 <johnsom> This blocks users from being able to apply a floating IP to the VIP port
20:21:02 <johnsom> I think we need to change it as German proposed.
20:21:18 <johnsom> The bummer is that users can now remove the security group from the VIP port.
20:21:24 <xgerman_> :-(
20:21:50 <jniesz> even though it was bad that was prior behavior
20:22:17 <johnsom> Yeah, I think we really need to let them use floating IPs...
20:22:22 <xgerman_> +1
20:22:56 <xgerman_> the grand of FWaaS V2 might solve our impediement in the future
20:23:07 <jniesz> +1
20:23:36 <johnsom> Yep.  So, something to review and discuss
20:24:38 <johnsom> Other topics today?
20:25:15 <xgerman_> review, review, review
20:25:19 <bar_> +1
20:25:42 <johnsom> Yes.  I just want to reiterate, all review votes count and are important for the project.
20:25:59 <johnsom> We all appreciate any review help you can provide
20:26:08 <xgerman_> +1000
20:26:18 <nmagnezi> +2000
20:26:37 <johnsom> Ok, thanks folks!
20:26:44 <xgerman_> o/
20:26:47 <nmagnezi> o/
20:26:54 <johnsom> #endmeeting