| *** sapd1_x has joined #openstack-lbaas | 00:05 | |
| *** goldyfruit has quit IRC | 00:29 | |
| *** goldyfruit has joined #openstack-lbaas | 00:29 | |
| *** yamamoto has quit IRC | 00:34 | |
| *** mithilarun has quit IRC | 00:35 | |
| *** sapd1_x has quit IRC | 00:35 | |
| *** mithilarun has joined #openstack-lbaas | 00:35 | |
| *** mithilarun has quit IRC | 00:40 | |
| *** gthiemonge has quit IRC | 00:55 | |
| *** gthiemonge has joined #openstack-lbaas | 00:56 | |
| *** spatel has joined #openstack-lbaas | 01:01 | |
| *** mithilarun has joined #openstack-lbaas | 01:01 | |
| *** goldyfruit has quit IRC | 01:15 | |
| *** goldyfruit has joined #openstack-lbaas | 01:16 | |
| *** mithilarun has quit IRC | 01:24 | |
| johnsom | Yeah, looks like queens is working | 01:38 |
|---|---|---|
| *** hongbin has joined #openstack-lbaas | 01:47 | |
| *** happyhemant has quit IRC | 02:01 | |
| *** yamamoto has joined #openstack-lbaas | 02:23 | |
| *** Dinesh_Bhor has quit IRC | 02:37 | |
| *** goldyfruit has quit IRC | 03:11 | |
| *** spatel has quit IRC | 03:15 | |
| lxkong | cgoncalves: hey, in case you are here, may i ask what's the progress of python-octaviaclient release? | 03:18 |
| *** Dinesh_Bhor has joined #openstack-lbaas | 03:19 | |
| *** hongbin has quit IRC | 03:52 | |
| *** ricolin has joined #openstack-lbaas | 03:58 | |
| *** threestrands has joined #openstack-lbaas | 04:15 | |
| *** pcaruana has joined #openstack-lbaas | 04:30 | |
| openstackgerrit | Adit Sarfaty proposed openstack/neutron-lbaas stable/stein: Support URL query params in healthmonitor url_path https://review.opendev.org/660930 | 05:05 |
| johnsom | lxkong What are you looking for? I can help push for a release if there is something you need. | 05:08 |
| *** ivve has quit IRC | 05:46 | |
| *** luksky has joined #openstack-lbaas | 06:00 | |
| lxkong | we need this patch https://review.opendev.org/#/c/647628/ | 06:10 |
| lxkong | johnsom ^^ | 06:10 |
| *** takamatsu has joined #openstack-lbaas | 06:32 | |
| *** gcheresh has joined #openstack-lbaas | 06:45 | |
| openstackgerrit | Gregory Thiemonge proposed openstack/octavia stable/queens: DNM Testing CI https://review.opendev.org/662428 | 06:47 |
| *** rcernin has quit IRC | 06:59 | |
| *** ivve has joined #openstack-lbaas | 07:04 | |
| *** rpittau|afk is now known as rpittau | 07:14 | |
| *** ccamposr has joined #openstack-lbaas | 07:25 | |
| *** tesseract has joined #openstack-lbaas | 07:25 | |
| openstackgerrit | Sven Wegener proposed openstack/octavia master: neutron: Update security groups https://review.opendev.org/663258 | 07:26 |
| openstackgerrit | Sven Wegener proposed openstack/octavia master: housekeeping: Log exception with error level and exc_info https://review.opendev.org/663265 | 07:59 |
| *** ramishra has joined #openstack-lbaas | 08:09 | |
| *** happyhemant has joined #openstack-lbaas | 08:38 | |
| *** threestrands has quit IRC | 08:39 | |
| *** openstackgerrit has quit IRC | 08:47 | |
| *** openstackgerrit has joined #openstack-lbaas | 08:53 | |
| openstackgerrit | Sven Wegener proposed openstack/octavia master: worker: Re-add FailoverPreparationForAmphora https://review.opendev.org/663277 | 08:53 |
| *** rcernin has joined #openstack-lbaas | 08:57 | |
| *** Emine has joined #openstack-lbaas | 09:21 | |
| openstackgerrit | Merged openstack/octavia master: Remove references to OpenStack Anchor https://review.opendev.org/662866 | 09:32 |
| *** yamamoto has quit IRC | 09:50 | |
| *** yamamoto has joined #openstack-lbaas | 10:24 | |
| openstackgerrit | Adit Sarfaty proposed openstack/neutron-lbaas stable/stein: Support URL query params in healthmonitor url_path https://review.opendev.org/660930 | 10:27 |
| *** ivve has quit IRC | 10:31 | |
| *** ivve has joined #openstack-lbaas | 10:33 | |
| *** yamamoto has quit IRC | 10:33 | |
| *** yamamoto has joined #openstack-lbaas | 10:34 | |
| *** yamamoto has quit IRC | 10:35 | |
| *** ivve has quit IRC | 10:41 | |
| *** ricolin has quit IRC | 10:47 | |
| *** yamamoto has joined #openstack-lbaas | 10:48 | |
| *** yamamoto has quit IRC | 10:53 | |
| *** yamamoto has joined #openstack-lbaas | 11:06 | |
| *** goldyfruit has joined #openstack-lbaas | 11:11 | |
| *** yamamoto has quit IRC | 11:44 | |
| *** ricolin has joined #openstack-lbaas | 12:03 | |
| *** yamamoto has joined #openstack-lbaas | 12:09 | |
| *** boden has joined #openstack-lbaas | 12:15 | |
| *** goldyfruit has quit IRC | 12:24 | |
| *** rcernin has quit IRC | 12:37 | |
| openstackgerrit | Ann Taraday proposed openstack/octavia master: [WIP] Use retry for AmphoraComputeConnectivityWait https://review.opendev.org/662791 | 12:55 |
| openstackgerrit | Ann Taraday proposed openstack/octavia master: [WIP] Use retry for AmphoraComputeConnectivityWait https://review.opendev.org/662791 | 12:57 |
| *** lucashxu has joined #openstack-lbaas | 13:05 | |
| *** sapd1_x has joined #openstack-lbaas | 13:14 | |
| openstackgerrit | Ann Taraday proposed openstack/octavia master: [WIP] Use retry for AmphoraComputeConnectivityWait https://review.opendev.org/662791 | 13:22 |
| *** goldyfruit has joined #openstack-lbaas | 13:23 | |
| *** yamamoto has quit IRC | 13:28 | |
| *** spatel has joined #openstack-lbaas | 13:35 | |
| *** spatel has quit IRC | 13:39 | |
| *** rcernin has joined #openstack-lbaas | 13:41 | |
| *** spatel has joined #openstack-lbaas | 14:17 | |
| *** yamamoto has joined #openstack-lbaas | 14:27 | |
| *** yamamoto has quit IRC | 14:32 | |
| *** Vorrtex has joined #openstack-lbaas | 14:38 | |
| *** gcheresh has quit IRC | 14:42 | |
| *** spatel has quit IRC | 14:43 | |
| *** gthiemon1e has joined #openstack-lbaas | 14:46 | |
| *** goldyfruit has quit IRC | 14:47 | |
| *** gthiemonge has quit IRC | 14:47 | |
| *** goldyfruit has joined #openstack-lbaas | 14:53 | |
| *** sapd1_x has quit IRC | 14:59 | |
| *** gcheresh has joined #openstack-lbaas | 15:14 | |
| *** goldyfruit has quit IRC | 15:18 | |
| *** gcheresh has quit IRC | 15:25 | |
| *** rcernin has quit IRC | 15:26 | |
| *** spatel has joined #openstack-lbaas | 15:33 | |
| spatel | johnsom: morning! finally i figured out why my ssh key wasn't loading, issue was i need to upload key on octavia account ( earlier i was doing on admin account) | 15:35 |
| johnsom | That would be a problem | 15:36 |
| spatel | octavia account/username password was in /etc/octavia.conf file ( i load them in openrc and used cli command to load key in octavia account) | 15:36 |
| spatel | Document was little vague on that part. | 15:36 |
| *** goldyfruit has joined #openstack-lbaas | 15:49 | |
| *** ramishra has quit IRC | 15:52 | |
| cgoncalves | #startmeeting Octavia | 16:00 |
| openstack | Meeting started Wed Jun 5 16:00:12 2019 UTC and is due to finish in 60 minutes. The chair is cgoncalves. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| *** openstack changes topic to " (Meeting topic: Octavia)" | 16:00 | |
| openstack | The meeting name has been set to 'octavia' | 16:00 |
| johnsom | o/ | 16:00 |
| colin- | \o | 16:01 |
| *** ataraday_ has joined #openstack-lbaas | 16:01 | |
| cgoncalves | Hi folks! | 16:01 |
| ataraday_ | hi | 16:01 |
| *** nmagnezi_mobile has joined #openstack-lbaas | 16:01 | |
| johnsom | copy cat | 16:01 |
| cgoncalves | rm_work asked me if I could chair the meeting today. he was not sure he could attend today | 16:01 |
| cgoncalves | #topic Announcements | 16:02 |
| *** openstack changes topic to "Announcements (Meeting topic: Octavia)" | 16:02 | |
| nmagnezi_mobile | o/ | 16:02 |
| cgoncalves | I don't anything to announce. anyone? | 16:02 |
| johnsom | I don't have anything either | 16:03 |
| colin- | nope | 16:03 |
| nmagnezi_mobile | Nope | 16:03 |
| cgoncalves | just a reminder that the Open Infrastructure Summit Shanghai is still open | 16:03 |
| cgoncalves | deadline is July 2, 2019 at 11:59pm PT | 16:04 |
| johnsom | FYI, this week is milestone 1 for Train | 16:04 |
| *** goldyfruit has quit IRC | 16:04 | |
| johnsom | We don't cut milestone releases by default anymore, but if someone needs one, let us know. | 16:05 |
| johnsom | #link https://releases.openstack.org/train/schedule.html | 16:05 |
| openstackgerrit | Ann Taraday proposed openstack/octavia master: Use retry for AmphoraComputeConnectivityWait https://review.opendev.org/662791 | 16:05 |
| cgoncalves | thanks | 16:05 |
| cgoncalves | #topic Brief progress reports / bugs needing review | 16:05 |
| *** openstack changes topic to "Brief progress reports / bugs needing review (Meeting topic: Octavia)" | 16:05 | |
| *** nmagnezi_mobile has quit IRC | 16:06 | |
| johnsom | I have wrapped up all of the client unset patches and the octavia API patches to handle the unsets better. | 16:06 |
| johnsom | I have not done tags as I found that the octavia API implementation of tags is not yet complete. | 16:06 |
| *** goldyfruit has joined #openstack-lbaas | 16:07 | |
| cgoncalves | I was out of office for most of time since last meeting. I'm proposing a couple of patches in different OpenStack projects to remove neutron-lbaas support | 16:07 |
| ataraday_ | I would like to ask for review https://review.opendev.org/662791 - this change - it solves issue with redis jobboard expirity. If the concept is OK, etc... | 16:07 |
| johnsom | I also started moving some tags code up to osc-lib so we can use it when we get there: | 16:07 |
| johnsom | #link https://review.opendev.org/#/c/662859/ | 16:07 |
| cgoncalves | nice! | 16:08 |
| colin- | my time on octavia is focused on addressing the behavior described in this story https://storyboard.openstack.org/#!/story/2005512 as well as the failover behavior in the case of DB unavailability that i described at PTG. am hopeful that once addressed the production deployment i've prepared will be placed in service | 16:08 |
| *** rpittau is now known as rpittau|afk | 16:08 | |
| cgoncalves | I wonder how this aligns with tags in "openstack image" | 16:08 |
| johnsom | I am now starting work on the log offloading patches. I put together a plan yesterday. | 16:09 |
| ataraday_ | And johnsom change https://review.opendev.org/#/c/659689 - can we merge this one? As I have to rebase all my work on it and have some long queues of change.. | 16:09 |
| johnsom | Maybe in open discussion I can share the proposed log format and get feedback. | 16:09 |
| johnsom | cgoncalves probably not at all. lol | 16:09 |
| johnsom | ataraday_ From my perspective https://review.opendev.org/#/c/659689 is ready for review/merge | 16:10 |
| johnsom | #link https://review.opendev.org/#/c/659689 | 16:10 |
| johnsom | FYI, this is the "tags" spec we should be implementing: | 16:12 |
| cgoncalves | at a first glance, johnsom's patch looks large but it isn't that hard to review. mostly file renamings | 16:12 |
| johnsom | #link https://specs.openstack.org/openstack/api-wg/guidelines/tags.html | 16:12 |
| johnsom | neutron seems to have it, thus why I want to share OSC code with them | 16:12 |
| cgoncalves | makes sense | 16:12 |
| johnsom | cgoncalves It's mostly copies and renames. It creates the "v2" amphora driver for our parallel development with jobboard. | 16:13 |
| cgoncalves | right. the patch is passing on all jobs (except centos for unrelated reasons). should be relatively safe to merge it | 16:14 |
| johnsom | That is my perspective as well | 16:15 |
| cgoncalves | I'll review it this week and I encourage everyone to do the same, pretty please :) | 16:15 |
| cgoncalves | anything else on this topic? | 16:15 |
| *** CBR09 has joined #openstack-lbaas | 16:15 | |
| CBR09 | hi all, | 16:16 |
| cgoncalves | CBR09, hi | 16:16 |
| cgoncalves | ok, seems not | 16:16 |
| cgoncalves | #topic Open Discussion | 16:16 |
| *** openstack changes topic to "Open Discussion (Meeting topic: Octavia)" | 16:17 | |
| cgoncalves | johnsom, you wanted to talk about log offloading? | 16:17 |
| CBR09 | Can octavia support ssl cert with tcp protocol? | 16:17 |
| johnsom | So as I mentioned earlier in the meeting, I'm starting on log offloading | 16:17 |
| johnsom | For the user flow logs, I'm going to propose this format: | 16:17 |
| johnsom | project_id lb_id listener_id client_ip client_port date_time request_str status bytes_read %[ssl_c_verify] %{+Q}[ssl_c_s_dn] pool_id member_id processing_time termination_state | 16:17 |
| johnsom | Which is a hybrid of haproxy log format, apache, and OpenStack stuff. | 16:18 |
| johnsom | I also put "driver" specific fields at the end. | 16:18 |
| johnsom | So an example would look like: | 16:18 |
| johnsom | 2f7cf2a1-d521-400e-a7c8-5611304723e8 b2f76e55-adfb-4aae-93eb-152c82adabec bfe473c0-d9f5-4cf0-aff8-90da27265cb3 10.0.1.2 33317 [06/Feb/2009:12:14:14.655] "GET /index.html HTTP/1.1" 200 2751 0 "/C=FR/ST=Ile de France/L=Jouy en Josas/O=haproxy.com/CN=client1/emailAddress=ba@haproxy.com" 15c7f458-aeec-42b1-921c-2bf24f62cf8c a7cb861f-9254-4c73-9929-332b81b40abf 109 ---- | 16:19 |
| johnsom | CBR09 Not at this time. It is technically possible, but has not be implemented yet. | 16:19 |
| johnsom | Please let me know if you have feedback on that log format. | 16:20 |
| colin- | i think that looks reasonable | 16:20 |
| cgoncalves | would it be possible for operators to set a different format? | 16:20 |
| johnsom | Yes, that is on my list of things to fix in the current patch | 16:20 |
| cgoncalves | great | 16:21 |
| johnsom | I have 12 things I identified yesterday that need work | 16:21 |
| *** Vorrtex has quit IRC | 16:21 | |
| *** mithilarun has joined #openstack-lbaas | 16:21 | |
| johnsom | We have a good PoC, but I want to make it user friendly and allow easy customization | 16:21 |
| cgoncalves | this RFE isn't strictly specific to the amphora driver, right? other providers can also streamline logs | 16:22 |
| CBR09 | @johnsom: yea, thank you, I think we should have that | 16:22 |
| johnsom | Right, there are some things to figure out with the other drivers, but yes, this should be general enough that other drivers could implement it. | 16:22 |
| CBR09 | I can see haproxy support that | 16:22 |
| cgoncalves | +1 | 16:23 |
| johnsom | Yeah, all of the LBs do, we just need to do the work | 16:23 |
| johnsom | Ok, thanks for the feedback! | 16:23 |
| cgoncalves | one more question, sorry :) | 16:23 |
| johnsom | please | 16:24 |
| cgoncalves | I guess sink destination for admin logs will be configured in the configuration file | 16:24 |
| cgoncalves | how do you propose for tenant logs? | 16:25 |
| johnsom | Correct, I want to have to "destination" config settings. One for "user flow logs" and one for "admin logs". They could be the same, could be different, up to the operator. | 16:25 |
| cgoncalves | an additional API parameter at POST/PUT? | 16:25 |
| johnsom | At this point I am only targeting these for operator setting, not end user settings of a destination. That becomes a problem with networking, etc. | 16:26 |
| cgoncalves | hmm, maybe I asked an invalid question. I was thinking of the possibility for Octavia to send the logs to an external sink log system | 16:26 |
| cgoncalves | right | 16:26 |
| cgoncalves | ok, thank you | 16:26 |
| johnsom | That would be a different RFE IMO. This is really targeting operators that have ELK available for users, etc. | 16:27 |
| cgoncalves | agreed | 16:27 |
| cgoncalves | anything else anyone would like to discuss? we are on open discussion after all :) | 16:28 |
| colin- | is anyone operating a fleet larger than ~450 VIPs (Active/Standby) in a single region? | 16:29 |
| johnsom | CBR09 Out of curiosity, what protocol are you wanting the TCP TLS for? It might be nice to know a usecase. | 16:29 |
| colin- | s/VIPs/LoadBalancers/ | 16:29 |
| cgoncalves | colin-, not that I'm aware of | 16:30 |
| *** Emine has quit IRC | 16:30 | |
| johnsom | I know of some larger deployments, but I don't operate them. | 16:30 |
| cgoncalves | colin-, are you hitting some sort of limitations? | 16:30 |
| CBR09 | @johnsom: the use case is when I just want use LB as a SSL termination and app protocol is tcp | 16:31 |
| CBR09 | I can't use https with ssl like octavia currently support | 16:31 |
| johnsom | CBR09 Ok, so just generic. Not like SMTP, or something more specific | 16:32 |
| colin- | i think some of the symptoms i've described in the story above are exacerbated by the size of the fleet, and it makes me wonder if we are in uncharted territory with our scale. this is also a relatively "vanilla" deployment of stable/rocky with mostly default configuration settings save for some significant relaxation of heartbeat/failover timing | 16:32 |
| CBR09 | @johnsom: for example I need mqtt protocol, so I need tcp protocol and ssl to secure that | 16:34 |
| johnsom | I think the unique situation you have is how your network (VXLAN) behaves with the neutron ports (macs moving around, etc.) | 16:34 |
| cgoncalves | I can't be of much help to comment on such large scale deployments | 16:34 |
| johnsom | CBR09 Ok, thanks. | 16:35 |
| colin- | mostly just a straw poll, best case scenario would be getting feedback from someone operating at a larger scale but that is probably unrealistic | 16:35 |
| CBR09 | one example in https://github.com/lelylan/haproxy-mqtt/blob/master/haproxy.cfg | 16:36 |
| cgoncalves | colin-, have you tried asking to the OpenStack Ops team? | 16:36 |
| johnsom | CBR09 Never would have guessed that one, so it's good to know for testing when it's implemented | 16:36 |
| colin- | the behavior by the vxlan was extreme, indeed, but not that peculiar. in my opinion it is more peculiar to blithely proceed with bringing up a new resource when the opportunity for resource collision may exist by virtue of our employing allowed address pairs | 16:38 |
| CBR09 | I think mtqq is so popular in IOT world : D | 16:38 |
| johnsom | allowed address pairs is a strange concept on it's own.... | 16:39 |
| colin- | the bulk/cascading failover behavior, while being quite unrelated to duplicate mac addresses, is equally concerning but we've established that database unavailability can have unpredictable results | 16:39 |
| johnsom | CBR09 Agreed | 16:39 |
| CBR09 | I think octavia should have secure tcp (tcp+ssl) for support that usecase | 16:40 |
| colin- | cgoncalves: i'm not familiar with the distinction between that and this channel, sorry :) | 16:40 |
| CBR09 | what's about websocket? | 16:40 |
| johnsom | CBR09 I don't disagree, someone just needs to do the implementation work. It's been on the roadmap for a while. | 16:41 |
| CBR09 | do you load balancing websocket protocol with octavia? | 16:41 |
| cgoncalves | colin-, the Ops team mostly consists of operators running openstack deployments. I just thought that you might give it a try asking in their list or so if someone is running octavia at large scale like you and collect feedback | 16:42 |
| johnsom | CBR09 I think we do have support for it, but there may be a missing timeout setting for tuning. | 16:43 |
| cgoncalves | ok, anything else? otherwise we can end the meeting | 16:46 |
| cgoncalves | alright, thank you everyone for joining and participating! | 16:47 |
| cgoncalves | #endmeeting | 16:47 |
| *** openstack changes topic to "Discussions for OpenStack Octavia | Train PTG etherpad: https://etherpad.openstack.org/p/octavia-train-ptg" | 16:47 | |
| openstack | Meeting ended Wed Jun 5 16:47:01 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:47 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-06-05-16.00.html | 16:47 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-06-05-16.00.txt | 16:47 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-06-05-16.00.log.html | 16:47 |
| johnsom | o/ | 16:47 |
| CBR09 | @johnsom: we can use tcp protcol and tuning some timout setting to support websocket protocol? | 16:47 |
| CBR09 | that's your mean | 16:47 |
| CBR09 | and maybe we missing some settings | 16:48 |
| johnsom | CBR09 Websocket should work now, but we might be missing some timeout settings for it. They may not be configurable today. | 16:48 |
| CBR09 | I wonder do listener protocol is tcp or http? | 16:50 |
| johnsom | HTTP | 16:50 |
| johnsom | Websocket starts as HTTP and must "upgrade", so the listener would be configured as HTTP | 16:51 |
| CBR09 | what about pool protocol? | 16:53 |
| johnsom | Hmmm, not sure on that one. | 16:54 |
| CBR09 | yea, thank you | 16:55 |
| CBR09 | I think I need to figure out that | 16:55 |
| CBR09 | @johnsom: can you share me the roadmap?, thank you | 16:56 |
| johnsom | CBR09 https://wiki.openstack.org/wiki/Octavia/Roadmap | 16:57 |
| CBR09 | yea thank you :D | 16:58 |
| CBR09 | have a nice day | 17:01 |
| CBR09 | need to sleep : ) | 17:01 |
| *** CBR09 has quit IRC | 17:06 | |
| *** mithilarun has quit IRC | 17:17 | |
| *** mithilarun has joined #openstack-lbaas | 17:17 | |
| *** happyhemant has quit IRC | 17:22 | |
| boden | johnsom hi... if you get a min to chat on the vmware-nsx thing pls ping me... thx | 17:38 |
| johnsom | boden Available now | 17:38 |
| boden | johnsom so the reason the vmware-nsx code is using the db, is because the data model doesn't have everything we need.. for example we need the project_id for all resources (Pool, Listener, etc.) but it's only there for LBs | 17:40 |
| boden | same goes for the actual TLS cert -- we need to get it so that we can install it on the backend | 17:40 |
| boden | should we be using the octavia client for this, or? | 17:40 |
| johnsom | boden Yes, that is what I saw as well fir project_id. We can extend that data model pretty easy. | 17:41 |
| cgoncalves | +1, although not sure it could be backported | 17:41 |
| johnsom | The TLS cert is passed directly to the drivers, already expanded. Octavia is handling that for the drivers so we can support barbican and castellan based sources. | 17:41 |
| boden | johnsom ok... I see some data we need is already there but we are still trying to get it from the DB... I'm still working pick apart the exact details of what we're lakcing | 17:42 |
| boden | lacking | 17:42 |
| johnsom | The "default_tls_container_data" field in the data model is the full cert data | 17:42 |
| boden | johnsom ack, I see it | 17:43 |
| boden | I also see some VIP info is there, but we are trying to get from db still | 17:43 |
| johnsom | boden Tell you what, I will post the project_id patch today for you. | 17:43 |
| johnsom | I can hammer that out pretty quick | 17:43 |
| boden | johnsom if you get time that would be awesome... I can even help it along if you get something started | 17:43 |
| johnsom | I will add you as a reviewer on them, there should be two, one for octavia, one for the octavia-lib. | 17:44 |
| *** goldyfruit has quit IRC | 17:44 | |
| *** goldyfruit has joined #openstack-lbaas | 17:45 | |
| boden | johnsom another thing I see is that for a rule update we need the actual policy obj/dict.... is that something we should be fetching or is it realistic to populate + pass it with a rule? | 17:47 |
| johnsom | Hmm, for that I might propose adding a "get" method to the driver-agent. I'm a bit curious why you would need the policy though | 17:48 |
| boden | or with a HM, we need a pool obj | 17:48 |
| *** yamamoto has joined #openstack-lbaas | 17:48 | |
| johnsom | We provide the parent object ID in the existing data model. It seems odd that you would need the full policy definition | 17:49 |
| boden | johnsom appears to be needed by the backend... I would need to dig into why... either way shouldn't it be realistic for drivers to be able to get at the info? | 17:52 |
| boden | I will do some digging into why we really need those "related objects" | 17:52 |
| *** yamamoto has quit IRC | 17:53 | |
| johnsom | Well, technically you should already have that info since it is a parent object, but if you don't have a way to reference that on the backend it might be an issue | 17:53 |
| boden | true... I will dig... might only need the ID; which we already have... new code fore me | 17:55 |
| *** tesseract has quit IRC | 17:57 | |
| *** goldyfruit has quit IRC | 18:08 | |
| *** goldyfruit has joined #openstack-lbaas | 18:08 | |
| *** ataraday_ has quit IRC | 18:18 | |
| *** dulek has quit IRC | 18:37 | |
| *** ricolin has quit IRC | 18:41 | |
| *** dulek has joined #openstack-lbaas | 18:45 | |
| *** goldyfruit has quit IRC | 18:57 | |
| *** goldyfruit has joined #openstack-lbaas | 19:00 | |
| *** Emine has joined #openstack-lbaas | 19:45 | |
| *** mithilarun has quit IRC | 19:47 | |
| *** ccamposr has quit IRC | 19:54 | |
| *** goldyfruit has quit IRC | 20:07 | |
| *** mithilarun has joined #openstack-lbaas | 20:37 | |
| *** lucasxu has joined #openstack-lbaas | 20:41 | |
| *** lucashxu has quit IRC | 20:43 | |
| *** lucasxu has quit IRC | 20:45 | |
| *** lucashxu has joined #openstack-lbaas | 20:46 | |
| *** lucashxu has quit IRC | 20:50 | |
| *** goldyfruit has joined #openstack-lbaas | 20:50 | |
| *** pcaruana has quit IRC | 20:59 | |
| *** Emine has quit IRC | 21:00 | |
| *** boden has quit IRC | 21:17 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia-lib master: Add project_id to all of the data model objects https://review.opendev.org/663434 | 21:18 |
| *** mithilarun has quit IRC | 21:19 | |
| lxkong | johnsom, any chance for you to do a python-octaviaclient release? | 21:19 |
| rm_work | do you also have to *set* that project_id still in the conversion routines? | 21:20 |
| johnsom | lxkong You need rm_work or cgoncalves to do it. I am no longer PTL nor the release liaison. | 21:20 |
| rm_work | cgoncalves is the release liason :D | 21:21 |
| johnsom | rm_work yes, but that happens in the octavia repo, that was the octavia-lib side | 21:21 |
| rm_work | Actually I am not sure how to make a release, it would probably be valuable for me to figure out though | 21:21 |
| rm_work | johnsom: oh, yeah derp k | 21:21 |
| rm_work | so this should be fine then | 21:21 |
| johnsom | Yeah, they will just get Unsets until we set them | 21:21 |
| rm_work | similarly to https://review.opendev.org/#/c/660238/ | 21:22 |
| rm_work | actually, would we mind merging that? assuming we're ok with the direction that's going? | 21:22 |
| rm_work | because that patch is going to continue to fail really annoyingly in testing until it's merged | 21:22 |
| johnsom | rm_work https://releases.openstack.org/reference/using.html#using-new-release-command | 21:22 |
| lxkong | rm_work: if we have all the expected python-octaviaclient patches merged, i can propose a release patch | 21:22 |
| johnsom | Yeah, it would be nice to merge both of those and do a release so we can bump the minimum version in the octavia side | 21:23 |
| rm_work | if for some reason the multivip patch doesn't make it in to Train, we can revert it | 21:23 |
| rm_work | hmmm | 21:23 |
| rm_work | or not i guess if we do a release | 21:23 |
| rm_work | so we should be sure that's the way we want it named | 21:23 |
| rm_work | lxkong: yeah sounds good to me | 21:23 |
| rm_work | johnsom: humorously, they do not conflict because mine ONLY touches LB, and yours touches exactly the inverse :D | 21:24 |
| rm_work | convenient | 21:24 |
| rm_work | i marked that also as Do Not Backport, though that brings me to a couple of questions: | 21:25 |
| rm_work | 1) Do we want to bother setting that on every review? | 21:25 |
| rm_work | 2) I think actually my initial reaction was wrong and this is backportable, lol | 21:26 |
| *** rcernin has joined #openstack-lbaas | 21:26 | |
| lxkong | rm_work: https://review.opendev.org/#/c/663438/ | 21:29 |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Create Amphora V2 provider driver https://review.opendev.org/659689 | 21:30 |
| rm_work | lgtm | 21:32 |
| openstackgerrit | Michael Johnson proposed openstack/octavia-lib master: Replace git.openstack.org URLs with opendev.org URLs https://review.opendev.org/655043 | 21:37 |
| johnsom | Just rebasing that | 21:37 |
| johnsom | Wanted to see if any of those have already been changed. | 21:37 |
| johnsom | rm_work What goes in additional_vips? | 21:40 |
| johnsom | A list of the VIP objects? | 21:40 |
| johnsom | A list of dicts? | 21:40 |
| rm_work | a list of dicts representing vips (basically a nested net_info object) | 21:42 |
| rm_work | https://review.opendev.org/#/c/660239/22/octavia/amphorae/drivers/haproxy/rest_api_driver.py | 21:42 |
| openstackgerrit | Merged openstack/octavia-lib master: Cap sphinx for py2 to match global requirements https://review.opendev.org/659495 | 21:52 |
| rm_work | johnsom: do you think that makes sense? | 21:55 |
| rm_work | this might not be a bad time to review this patch and tell me if you think the direction is bad | 21:55 |
| johnsom | I think that works | 21:56 |
| rm_work | I wanted to kinda just combine all the vips into one thing, but | 21:57 |
| johnsom | I think it is better this way | 21:57 |
| rm_work | I can't change it on the server side and still keep old amp compat | 21:57 |
| rm_work | so i'm making do with just refactoring inside the amp | 21:57 |
| rm_work | diving in to the vrrp stuff now | 22:02 |
| *** rcernin has quit IRC | 22:04 | |
| cgoncalves | lxkong, https://review.opendev.org/#/c/663452/ | 22:20 |
| cgoncalves | AFKing for sleep time. back in 8h | 22:22 |
| *** spatel has quit IRC | 22:24 | |
| openstackgerrit | Merged openstack/octavia-lib master: Replace git.openstack.org URLs with opendev.org URLs https://review.opendev.org/655043 | 22:34 |
| *** goldyfruit has quit IRC | 22:40 | |
| *** lucashxu has joined #openstack-lbaas | 22:42 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia-tempest-plugin master: Adds provider flavor capabilities API tests https://review.opendev.org/631113 | 22:44 |
| *** yamamoto has joined #openstack-lbaas | 22:46 | |
| *** lucashxu has quit IRC | 22:48 | |
| *** Vorrtex has joined #openstack-lbaas | 23:12 | |
| *** goldyfruit has joined #openstack-lbaas | 23:16 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Add project_id to all of the provider objects https://review.opendev.org/663459 | 23:35 |
| *** goldyfruit has quit IRC | 23:41 | |
| *** luksky has quit IRC | 23:46 | |
| *** yamamoto has quit IRC | 23:48 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!