akosiaris added a comment.
@Tarrow , @WMDE-leszek I 've noticed 3 things while working on the above
- The service seems to be configurable to reach directly out to the wikidata
endpoint. e.g. `WIKIBASE_REPO:
'{env(WIKIBASE_REPO,https://www.wikidata.org/w)}'`. Since, in the general case,
this goes via the edge caches (possibly polluting them with artificial
requests, and limits operational flexibility, e.g. in the case of a datacenter
switchover we can't really switch the app without switching the edge layer, we
've been defaulting to talking to internal endpoints and sending a `Host: `
HTTP for the identification of the exact project. Would it be possible to add
that functionality? (or guide me on how this is achieved if it is already
implemented).
- There was no `x-amples` stanza to `/termbox` endpoint. We need this for
monitoring checks so I went ahead and added one in
https://gerrit.wikimedia.org/r/#/c/wikibase/termbox/+/509391. Lemme know what
you think.
- The app does not emit statsd metrics for requests (it does however emit
nodejs heap and GC stats). This should be rather easy to add as service-runner
wraps it very nicely. See
https://github.com/wikimedia/service-runner#metric-reporting
What exactly you want to do for that is up to you, but up to now services
have been using
https://github.com/wikimedia/service-template-node/blob/master/lib/util.js#L107
to wrap the route handlers which provides them out of the box with requests
rates per endpoints, latencies per endpoint, errors and so on. I highly suggest
that.
For service specific stats, e.g. how many time language=de has been passed as
a request, `options.metrics.increment` is what has been used up to now as they
have invariably been of a counter nature.
Overall I think we are ready to deploy to staging, production will have to
wait a bit until we solve at least the first 2 items. The 3rd is not a hard
blocker but I am willing to bet you want stats :-)
I also think we should schedule a training session for how deployers deploy
code in the pipeline/kubernetes environment.
TASK DETAIL
https://phabricator.wikimedia.org/T220402
EMAIL PREFERENCES
https://phabricator.wikimedia.org/settings/panel/emailpreferences/
To: akosiaris
Cc: fsero, mobrovac, Matthias_Geisler_WMDE, Pablo-WMDE, Dzahn, Addshore,
Tarrow, Lea_WMDE, WMDE-leszek, Jakob_WMDE, Aklapper, akosiaris, darthmon_wmde,
alaa_wmde, joker88john, Legado_Shulgin, CucyNoiD, Nandana, NebulousIris,
thifranc, AndyTan, Gaboe420, Versusxo, Majesticalreaper22, Giuliamocci,
Davinaclare77, Adrian1985, Qtn1293, Cpaulf30, Lahi, Gq86, Baloch007,
Darkminds3113, Bsandipan, Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls,
Hfbn0, Ramalepe, Liugev6, QZanden, LawExplorer, WSH1906, Lewizho99, Zppix,
Maathavan, _jensen, rosalieper, Liudvikas, Wong128hk, Eevans, thcipriani,
Hardikj, Wikidata-bugs, aude, Lydia_Pintscher, dduvall, faidon,
Jdforrester-WMF, Mbch331, Jay8g, fgiunchedi
_______________________________________________
Wikidata-bugs mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs
