Tuesday, 2021-03-23

johnsomno, maybe you do?00:00
johnsomDing, ding, ding, we have a winner: https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/50155900:01
johnsomIn the boneyard of great intentions...00:02
rm_worki tried >_>00:06
rm_workI have ... a lot of PRs like that00:06
rm_workstuff that was complicated and i got like 90% of the way there and ran out of steam / got pulled into something else00:07
rm_worki ran that one in our test quite at godaddy very successfully00:07
*** luksky has quit IRC00:07
rm_workit could probably be dusted off quickly and merged honestly00:07
rm_workbut i used it as like, a failover stress test00:08
johnsomcopy/paste from the active/standby would also be an option00:09
*** johnthetubaguy has quit IRC00:16
rm_workok well, here we go00:18
openstackgerritAdam Harwell proposed openstack/octavia master: Fix empty Batch Member Update to unlock objects  https://review.opendev.org/c/openstack/octavia/+/78234200:20
*** johnthetubaguy has joined #openstack-lbaas00:22
rm_workheh, looking at the AZ thing, a lot of the code I wrote in that actually ended up merged as part of the octavia-az stuff00:24
rm_workso this patch really gets trimmed down00:24
rm_workand we00:24
rm_work*and we've decided to deprecate Spares Pool right?00:24
rm_workso that cuts out a large chunk too if I don't write new code to handle it00:24
rm_worklike wow, a ton of it was verbatim00:25
rm_workI guess I did write both lol00:25
johnsomWe did, but that didn't happen in Wallaby00:26
rm_workhmm yeah but00:26
rm_worknew features might not need to support it?00:26
rm_workespecially if they wouldn't merge in Wallaby00:26
rm_worksince we're now past feature freeze?00:26
johnsomYeah, it would have been nice to actually deprecate spares in Wallaby though00:27
openstackgerritMerged openstack/octavia master: Fix LB failover for amphorav2: set security group  https://review.opendev.org/c/openstack/octavia/+/78225500:47
rm_workhmm do we not backref the availability_zone in our db model? weird00:51
rm_workhave to go look it up I guess00:51
rm_workthe FK relationship exists? this is where I get confused about whay SQLAlchemy does automagically00:52
rm_work*about what00:52
rm_workand no relationship between AZ and AZProfile?00:54
rm_workFK yes, but no easy to follow dot-relationship00:54
johnsomYeah, I don't see that you added back references for the AZ SA model00:55
rm_workT_T this is my fault too isn't it00:56
johnsomOr Sam I guess. Anyway, no backreferences in the code00:56
johnsomI didnt' have them in flavors, so .... grin00:56
rm_workyeah I think I copy/pasted XD00:57
rm_workI'm really starting to hate 2019-Adam00:57
rm_workupgrading from jerk to full-on asshole00:57
johnsomI didn't need them, so why add the overhead.00:57
rm_workto be nice to other people :D and follow the model we use for literally everything else00:57
johnsomI try to avoid SA magic as much as possible00:58
rm_workhmm doesn't oslo manage our json parsing?00:58
johnsomSpeaking of which there is a whole breakage bomb coming in that library.00:58
johnsomYes, oh, BTW, why did you add simplejson for the stats patch??????? It's like the only place that uses it?00:59
johnsomThode is coming for you00:59
johnsomPer his PTL nomination00:59
rm_workeither 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_workthere's a few edge cases I've run into01:00
rm_worki kinda doubt that tho01:00
rm_workprobably similar to right now, where i'm looking for oslo's json stuff and can't find it01:01
rm_workthere's no json* in oslo_utils now?01:01
johnsomfrom oslo_serialization import jsonutils01:01
johnsomI sometimes wonder about those... They are, just wrappers01:02
johnsomWidely ignored (see above simplejson reference)01:02
rm_workthis wrapper sucks too01:03
rm_workit doesn't provide any exceptions01:03
johnsomOk, I am out. Going to make dinner.01:03
rm_workso I just have to ... know that it's just a wrapper on the python stdlib json and import+use those anyway01:03
rm_workok, have a good one01:04
*** jamesdenton has quit IRC02:13
*** jamesden_ has joined #openstack-lbaas02:13
*** rcernin has quit IRC02:28
*** rcernin has joined #openstack-lbaas02:39
*** vishalmanchanda has joined #openstack-lbaas04:09
*** xgerman has quit IRC04:30
*** jamesden_ has quit IRC05:25
*** jamesdenton has joined #openstack-lbaas05:25
*** rcernin has quit IRC05:25
*** rcernin has joined #openstack-lbaas05:26
*** jamesdenton has quit IRC06:20
*** jamesden_ has joined #openstack-lbaas06:20
*** rcernin has quit IRC07:46
*** rpittau|afk is now known as rpittau08:13
*** rcernin has joined #openstack-lbaas08:23
*** vishalmanchanda has quit IRC09:28
*** luksky has joined #openstack-lbaas10:07
*** admin0 has quit IRC10:14
openstackgerritGregory Thiemonge proposed openstack/octavia master: Fix using subnets with host_routes in amphorav2 driver  https://review.opendev.org/c/openstack/octavia/+/78227910:37
*** rcernin has quit IRC10:46
*** rcernin has joined #openstack-lbaas11:09
*** vishalmanchanda has joined #openstack-lbaas11:11
*** rcernin has quit IRC11:17
*** rcernin has joined #openstack-lbaas11:18
*** rcernin has quit IRC11:25
*** admin0 has joined #openstack-lbaas12:02
openstackgerritGregory Thiemonge proposed openstack/octavia stable/victoria: Fix LB failover for amphorav2: set security group  https://review.opendev.org/c/openstack/octavia/+/78252313:39
openstackgerritGregory Thiemonge proposed openstack/octavia stable/ussuri: Fix LB failover for amphorav2: set security group  https://review.opendev.org/c/openstack/octavia/+/78252413:39
*** haleyb_ has joined #openstack-lbaas13:49
*** sapd1 has joined #openstack-lbaas13:51
openstackgerritGregory Thiemonge proposed openstack/octavia master: Fix using subnets with host_routes in amphorav2 driver  https://review.opendev.org/c/openstack/octavia/+/78227914:10
*** haleyb_ has quit IRC14:34
*** dosaboy has quit IRC15:35
*** oklhost has quit IRC15:36
*** dosaboy has joined #openstack-lbaas15:37
*** wolsen has quit IRC15:37
*** irclogbot_1 has quit IRC15:37
*** irclogbot_0 has joined #openstack-lbaas15:39
*** irclogbot_0 has quit IRC15:51
*** irclogbot_3 has joined #openstack-lbaas15:52
*** wolsen has joined #openstack-lbaas16:09
*** __ministry has joined #openstack-lbaas16:38
*** __ministry has quit IRC16:59
*** rpittau is now known as rpittau|afk17:26
*** sapd1 has quit IRC17:29
*** fnpanic has quit IRC17:43
*** vishalmanchanda has quit IRC18:17
*** xgerman has joined #openstack-lbaas18:32
*** servagem has quit IRC19:09
*** rcernin has joined #openstack-lbaas21:01
*** rcernin has quit IRC21:16
*** rcernin has joined #openstack-lbaas21:46
*** rcernin has quit IRC22:10
*** rcernin has joined #openstack-lbaas22:10
*** jamesden_ has quit IRC23:16
*** jamesdenton has joined #openstack-lbaas23:17
*** luksky has quit IRC23:22
*** johnthetubaguy has quit IRC23:36
*** johnthetubaguy has joined #openstack-lbaas23:38
*** johnthetubaguy has quit IRC23:49
*** johnthetubaguy has joined #openstack-lbaas23:57

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!