Add praefect prometheus_exclude_database_from_default_metrics config value
What does this MR do?
Praefect has a new configuration value to control whether or not it will
hit the database for metrics. This change adds the
prometheus_exclude_database_from_default_metrics
config value and writes
it into the right place in the praefect config toml.
corresponding gitaly MR is gitaly!4085 (merged)
Related issues
Checklist
See Definition of done.
For anything in this list which will not be completed, please provide a reason in the MR discussion
Required
-
Merge Request Title, and Description are up to date, accurate, and descriptive -
MR targeting the appropriate branch -
MR has a green pipeline on GitLab.com -
Pipeline is green on dev.gitlab.org if the change is touching anything besides documentation or internal cookbooks -
trigger-package
has a green pipeline running against latest commit
Expected (please provide an explanation if not completing)
-
Test plan indicating conditions for success has been posted and passes -
Documentation created/updated -
Tests added -
Integration tests added to GitLab QA -
Equivalent MR/issue for the GitLab Chart opened
Merge request reports
Activity
added devopssystems groupdistribution sectioncore platform labels
gitaly!4085 (merged) is still in progress and needs to be reviewed/merged. But, this is high priority so would be great to have it reviewed in parallel.
assigned to @jcaigitlab
added 9 commits
-
cb9550bb...d313af57 - 8 commits from branch
master
- 35d37fd9 - Add praefect prometheus_exclude_database_from_default_metrics config value
-
cb9550bb...d313af57 - 8 commits from branch
mentioned in merge request gitaly!4085 (merged)
added workflowready for review label
/cc @timzallmann
changed milestone to %14.5
The
Trigger:ee-package
job from pipeline https://gitlab.com/gitlab-org/omnibus-gitlab/-/pipelines/410909082 triggered https://gitlab.com/gitlab-org/build/omnibus-gitlab-mirror/-/pipelines/411026180 downstream.The
qa-test
job from pipeline https://gitlab.com/gitlab-org/build/omnibus-gitlab-mirror/-/pipelines/411026180 triggered https://gitlab.com/gitlab-org/gitlab-qa-mirror/-/pipelines/411048706 downstream.The
gitlab-qa
downstream pipeline failed! .The qa failures appear unrelated to this MR, as they are also happening in master but the main failures are for gitaly-cluster tests: https://gitlab.com/gitlab-org/gitlab-qa-mirror/-/jobs/1793870306 so not all the qa validation is working here
Thanks a lot @twk3 + team helping us on moving it forward.
added featureaddition label
removed featureaddition label
added featureenhancement label
added typefeature label
requested review from @twk3
added workflowin review label and removed workflowready for review label
- Resolved by DJ Mountney
Visually this looks good, I'm just doing some manual validation of the change now