remove prometheus metrics

[syphar]

needs rust-lang/simpleinfra#832 to be merged & applied before deploying, so the collector doesn't run wild. thanks @marcoieni !

This also removes the "crates / releases / targets visited" metrics. I didn't look at them for quite some time. There was an initial learning we had (= most requests are crawlers), but nothing else after that, so we can safe the processing time & memory.

If we feel like adding it back, we easily can.

[GuillaumeGomez]

Seems like you forgot to update a docker file.

[syphar]

Seems like you forgot to update a docker file.

can you see the actual error? I seem to miss it in the logs

[GuillaumeGomez]

No, just:

container docsrs-web-1 is unhealthy
error: Recipe `compose-up` failed on line 4 with exit code 1
error: Recipe `compose-up-web` failed on line 9 with exit code 1

So I just assumed (maybe incorrectly) that a docker image needed to be updated.

[syphar]

odd. I'll try locally

[syphar]

ah, I think I know why

[syphar]

I used the metrics endpoints as health-checks for most containers :)

[GuillaumeGomez]

So I was right, yeay me. \o/

[GuillaumeGomez]

Looks good to me, thanks!

[syphar]

Looks good to me, thanks!

I'll still wait until CI passes :)

[syphar]

just confirmed again by @marcoieni : rust-lang/simpleinfra#832 is applied, the collectors don't collect any more.