Auto-register Prometheus metrics

• Update to latest upstream Prometheus library.
• Update to latest upstream Prometheus gRPC library.
• Switch to promauto package to avoid missing metrics.

Closes: #326 (closed)

Signed-off-by: Ben Kochie bjk@gitlab.com

added 2 commits

• 4f22712e - Remove duplicate metric
• 4c299b87 - Add changelog entry

Compare with previous version

FYI: I've split this into a couple of commits. I'd like to remove the duplicate metrics that are provided by the requests_total counter an the request_duration_seconds histogram counter.

Thanks @bjk-gitlab !

@patrickbajao WDYT?

assigned to @patrickbajao and unassigned @mkaeppler

added 3 commits

• cb708c3a - Auto-register Prometheus metrics
• c990cb65 - Remove duplicate metric
• 9744421c - Add changelog entry

Compare with previous version

changed title from Update http metrics to Auto-register Prometheus metrics

added 1 commit

• 33ed0166 - Add changelog entry

Compare with previous version

resolved all threads

assigned to @bjk-gitlab and unassigned @patrickbajao

added 8 commits

• 33ed0166...e1254d74 - 7 commits from branch master
• 35403c6c - Auto-register Prometheus metrics

Compare with previous version

resolved all threads

assigned to @patrickbajao and unassigned @bjk-gitlab

I dropped the metric removal and added all instances of converting MustRegister to promauto.

added 1 commit

• a2bc2f4b - Auto-register Prometheus metrics

Compare with previous version

added 1 commit

• be3c45d1 - Auto-register Prometheus metrics

Compare with previous version

Note, I have removed gitlab_workhorse_object_storage_upload_bytes. This (incorrectly named) counter is registered, but never used, so the values are always zero.

I noticed there are a number of issues with Prometheus naming conventions. We should probably open a separate issue to fix them.

For example: gitlab_workhorse_object_storage_upload_time should be gitlab_workhorse_object_storage_upload_duration_seconds.

mentioned in issue gitlab#249637 (closed)

assigned to @bjk-gitlab and unassigned @patrickbajao

mentioned in issue #327 (moved)

added 1 commit

• 2caaafca - Auto-register Prometheus metrics

Compare with previous version

added 1 commit

• d7e07e91 - Auto-register Prometheus metrics

Compare with previous version

resolved all threads

Ok, I think I fixed the objectstorage upload bytes metric.

assigned to @patrickbajao and unassigned @bjk-gitlab

resolved all threads

@bjk-gitlab thanks for making the changes and they LGTM. There are conflicts with master though. Can you please rebase? Please assign it back to me so I can merge.

assigned to @bjk-gitlab and unassigned @patrickbajao

added 13 commits

• d7e07e91...35047496 - 12 commits from branch master
• fa659fc3 - Auto-register Prometheus metrics

Compare with previous version

assigned to @patrickbajao

approved this merge request

resolved all threads

enabled automatic add to merge train when the pipeline for 7ebd03dd succeeds

started a merge train

mentioned in commit 5b34783e

merged

mentioned in commit gitlab@99fba74a

mentioned in merge request gitlab!49534 (merged)

mentioned in issue gitlab#324828