Skip to content

Draft: [BREAKING CHANGE] ReferenceResolver.dereference: Use new RestAPI instead of ActionAPI

Jforrester requested to merge T274269 into main

This means going from:

https://www.wikifunctions.org/w/api.php?action=wikilambda_fetch&format=json&uselang=content&zids=Z1|Z2

… to:

https://www.wikifunctions.org/w/rest.php/wikifunctions/v0/fetch/Z1%7CZ2

… and the fetched results not being wrapped in an extra layer of indirection with a 'wikilambda_fetch' sub-object. The requested object will now emit (and our proxy layer will respect) a 24 hour cache on these, which in practice will mean the slightly faster request (~30ms vs. ~35ms) should be much faster for common dereferences (~15ms), reducing slowness in the orchestrator. I hope.

There's no nice way to support both, so this cut-over will need a Beta Cluster config change to work, and when deployed, the same commit that updates our used production image tag will need to adjust config at the same time, hence marking as a 'breaking change'. Joy.

Bug: T274269

Merge request reports