[07:24:35] 10Lift-Wing, 10Machine-Learning-Team (Active Tasks), 10Patch-For-Review: API Gateway Integration - https://phabricator.wikimedia.org/T288789 (10elukey) >>! In T288789#7430212, @ACraze wrote: >> Since we are not production ready I'd loved to avoid this until the last moment, but we could push the trigger anyt... [15:43:39] o/ [15:56:32] elukey: nice work on those CRs for the api-gateway chart! [16:01:37] accraze: o/ thanks! Hope the they work, the envoy configs are not that straightforward to parse :( [16:01:50] in theory if the change is the right one we should be ok on that front [16:02:52] yeah i had trouble reading through the configs yesterday afternoon, currently reading through your proposed changes, it seems to make sense! [16:12:06] accraze: the last patch will likely need to be merged later on, we are not ready to expose the endpoint to the outside in my opinion [16:12:27] there are all the concerns on rate limiting etc.. [16:12:49] ahh yeah was just reading that patch [16:14:48] solid regex for host header mappings -- i think that should work for the other revscoring model classes too [16:23:03] holds the usual rule that nothing will work at first try for sure [16:23:03] :D [16:28:04] lol [16:38:10] going afk for the weekend, have a good rest of the day/weekend folks :) [16:42:47] have a good weekend elukey!