johnsom | no, maybe you do? | 00:00 |
---|---|---|
johnsom | Ding, ding, ding, we have a winner: https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/501559 | 00:01 |
johnsom | In the boneyard of great intentions... | 00:02 |
rm_work | yep | 00:06 |
rm_work | i tried >_> | 00:06 |
rm_work | I have ... a lot of PRs like that | 00:06 |
rm_work | stuff that was complicated and i got like 90% of the way there and ran out of steam / got pulled into something else | 00:07 |
rm_work | i ran that one in our test quite at godaddy very successfully | 00:07 |
*** luksky has quit IRC | 00:07 | |
rm_work | it could probably be dusted off quickly and merged honestly | 00:07 |
rm_work | but i used it as like, a failover stress test | 00:08 |
johnsom | copy/paste from the active/standby would also be an option | 00:09 |
*** johnthetubaguy has quit IRC | 00:16 | |
rm_work | ok well, here we go | 00:18 |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Fix empty Batch Member Update to unlock objects https://review.opendev.org/c/openstack/octavia/+/782342 | 00:20 |
*** johnthetubaguy has joined #openstack-lbaas | 00:22 | |
rm_work | heh, looking at the AZ thing, a lot of the code I wrote in that actually ended up merged as part of the octavia-az stuff | 00:24 |
rm_work | so this patch really gets trimmed down | 00:24 |
rm_work | and we | 00:24 |
rm_work | *and we've decided to deprecate Spares Pool right? | 00:24 |
rm_work | so that cuts out a large chunk too if I don't write new code to handle it | 00:24 |
rm_work | like wow, a ton of it was verbatim | 00:25 |
rm_work | I guess I did write both lol | 00:25 |
johnsom | We did, but that didn't happen in Wallaby | 00:26 |
rm_work | hmm yeah but | 00:26 |
rm_work | new features might not need to support it? | 00:26 |
rm_work | especially if they wouldn't merge in Wallaby | 00:26 |
rm_work | since we're now past feature freeze? | 00:26 |
johnsom | Yeah, it would have been nice to actually deprecate spares in Wallaby though | 00:27 |
openstackgerrit | Merged openstack/octavia master: Fix LB failover for amphorav2: set security group https://review.opendev.org/c/openstack/octavia/+/782255 | 00:47 |
rm_work | hmm do we not backref the availability_zone in our db model? weird | 00:51 |
rm_work | have to go look it up I guess | 00:51 |
rm_work | the FK relationship exists? this is where I get confused about whay SQLAlchemy does automagically | 00:52 |
rm_work | *about what | 00:52 |
rm_work | and no relationship between AZ and AZProfile? | 00:54 |
rm_work | FK yes, but no easy to follow dot-relationship | 00:54 |
johnsom | Yeah, I don't see that you added back references for the AZ SA model | 00:55 |
rm_work | T_T this is my fault too isn't it | 00:56 |
johnsom | Or Sam I guess. Anyway, no backreferences in the code | 00:56 |
johnsom | I didnt' have them in flavors, so .... grin | 00:56 |
rm_work | yeah I think I copy/pasted XD | 00:57 |
rm_work | I'm really starting to hate 2019-Adam | 00:57 |
rm_work | upgrading from jerk to full-on asshole | 00:57 |
johnsom | I didn't need them, so why add the overhead. | 00:57 |
rm_work | to be nice to other people :D and follow the model we use for literally everything else | 00:57 |
johnsom | I try to avoid SA magic as much as possible | 00:58 |
rm_work | hmm doesn't oslo manage our json parsing? | 00:58 |
johnsom | Speaking of which there is a whole breakage bomb coming in that library. | 00:58 |
johnsom | Yes, oh, BTW, why did you add simplejson for the stats patch??????? It's like the only place that uses it? | 00:59 |
johnsom | Thode is coming for you | 00:59 |
johnsom | Per his PTL nomination | 00:59 |
rm_work | lol | 00:59 |
rm_work | either we couldn't find the normal one / I forgot that oslo had one, or we needed something specific that simplejson does better? | 01:00 |
rm_work | there's a few edge cases I've run into | 01:00 |
rm_work | i kinda doubt that tho | 01:00 |
rm_work | probably similar to right now, where i'm looking for oslo's json stuff and can't find it | 01:01 |
rm_work | there's no json* in oslo_utils now? | 01:01 |
johnsom | from oslo_serialization import jsonutils | 01:01 |
rm_work | ah | 01:01 |
johnsom | I sometimes wonder about those... They are, just wrappers | 01:02 |
johnsom | Widely ignored (see above simplejson reference) | 01:02 |
rm_work | this wrapper sucks too | 01:03 |
rm_work | it doesn't provide any exceptions | 01:03 |
johnsom | Ok, I am out. Going to make dinner. | 01:03 |
rm_work | so I just have to ... know that it's just a wrapper on the python stdlib json and import+use those anyway | 01:03 |
rm_work | rofl | 01:03 |
rm_work | ok, have a good one | 01:04 |
*** jamesdenton has quit IRC | 02:13 | |
*** jamesden_ has joined #openstack-lbaas | 02:13 | |
*** rcernin has quit IRC | 02:28 | |
*** rcernin has joined #openstack-lbaas | 02:39 | |
*** vishalmanchanda has joined #openstack-lbaas | 04:09 | |
*** xgerman has quit IRC | 04:30 | |
*** jamesden_ has quit IRC | 05:25 | |
*** jamesdenton has joined #openstack-lbaas | 05:25 | |
*** rcernin has quit IRC | 05:25 | |
*** rcernin has joined #openstack-lbaas | 05:26 | |
*** jamesdenton has quit IRC | 06:20 | |
*** jamesden_ has joined #openstack-lbaas | 06:20 | |
*** rcernin has quit IRC | 07:46 | |
*** rpittau|afk is now known as rpittau | 08:13 | |
*** rcernin has joined #openstack-lbaas | 08:23 | |
*** vishalmanchanda has quit IRC | 09:28 | |
*** luksky has joined #openstack-lbaas | 10:07 | |
*** admin0 has quit IRC | 10:14 | |
openstackgerrit | Gregory Thiemonge proposed openstack/octavia master: Fix using subnets with host_routes in amphorav2 driver https://review.opendev.org/c/openstack/octavia/+/782279 | 10:37 |
*** rcernin has quit IRC | 10:46 | |
*** rcernin has joined #openstack-lbaas | 11:09 | |
*** vishalmanchanda has joined #openstack-lbaas | 11:11 | |
*** rcernin has quit IRC | 11:17 | |
*** rcernin has joined #openstack-lbaas | 11:18 | |
*** rcernin has quit IRC | 11:25 | |
*** admin0 has joined #openstack-lbaas | 12:02 | |
openstackgerrit | Gregory Thiemonge proposed openstack/octavia stable/victoria: Fix LB failover for amphorav2: set security group https://review.opendev.org/c/openstack/octavia/+/782523 | 13:39 |
openstackgerrit | Gregory Thiemonge proposed openstack/octavia stable/ussuri: Fix LB failover for amphorav2: set security group https://review.opendev.org/c/openstack/octavia/+/782524 | 13:39 |
*** haleyb_ has joined #openstack-lbaas | 13:49 | |
*** sapd1 has joined #openstack-lbaas | 13:51 | |
openstackgerrit | Gregory Thiemonge proposed openstack/octavia master: Fix using subnets with host_routes in amphorav2 driver https://review.opendev.org/c/openstack/octavia/+/782279 | 14:10 |
*** haleyb_ has quit IRC | 14:34 | |
*** dosaboy has quit IRC | 15:35 | |
*** oklhost has quit IRC | 15:36 | |
*** dosaboy has joined #openstack-lbaas | 15:37 | |
*** wolsen has quit IRC | 15:37 | |
*** irclogbot_1 has quit IRC | 15:37 | |
*** irclogbot_0 has joined #openstack-lbaas | 15:39 | |
*** irclogbot_0 has quit IRC | 15:51 | |
*** irclogbot_3 has joined #openstack-lbaas | 15:52 | |
*** wolsen has joined #openstack-lbaas | 16:09 | |
*** __ministry has joined #openstack-lbaas | 16:38 | |
*** __ministry has quit IRC | 16:59 | |
*** rpittau is now known as rpittau|afk | 17:26 | |
*** sapd1 has quit IRC | 17:29 | |
*** fnpanic has quit IRC | 17:43 | |
*** vishalmanchanda has quit IRC | 18:17 | |
*** xgerman has joined #openstack-lbaas | 18:32 | |
*** servagem has quit IRC | 19:09 | |
*** rcernin has joined #openstack-lbaas | 21:01 | |
*** rcernin has quit IRC | 21:16 | |
*** rcernin has joined #openstack-lbaas | 21:46 | |
*** rcernin has quit IRC | 22:10 | |
*** rcernin has joined #openstack-lbaas | 22:10 | |
*** jamesden_ has quit IRC | 23:16 | |
*** jamesdenton has joined #openstack-lbaas | 23:17 | |
*** luksky has quit IRC | 23:22 | |
*** johnthetubaguy has quit IRC | 23:36 | |
*** johnthetubaguy has joined #openstack-lbaas | 23:38 | |
*** johnthetubaguy has quit IRC | 23:49 | |
*** johnthetubaguy has joined #openstack-lbaas | 23:57 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!