This Regex:
does not match RR models which have a dash in the second part. I.e. it matches revertrisk-multilingual, but not revertrisk-language-agnostic.
The proper RE would be using ([\w-]+), or in full: 'lw/inference/v1/models/revertrisk-([\w-]+):predict' to allow for arbitrary numbers of word-like (\w) and - characters.
See also: https://regex101.com/r/hjd93J/1
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| helmfile.d: Fix regex in api-gateway's config for revertrisk | operations/deployment-charts | master | +1 -1 |
Change 922805 had a related patch set uploaded (by Klausman; author: Tobias Klausmann):
[operations/deployment-charts@master] API GW: Fix RegEx in config for revertrisk models on Lift Wing
Change 922805 merged by Klausman:
[operations/deployment-charts@master] helmfile.d: Fix regex in api-gateway's config for revertrisk
Changes have been merged and deployed, Bot eqiad and codfw (and staging) sections of the API GW work fine (tested from within clusters), as well as remote (from my home machine).