Skip to content
This repository has been archived by the owner on Apr 7, 2020. It is now read-only.

Change monitoring ConfigMaps' syntax of provider-specific components #344

Merged
merged 1 commit into from
Sep 30, 2019

Conversation

svetlinas
Copy link
Contributor

What this PR does / why we need it:
Change monitoring ConfigMaps' syntax of provider-specific components to be easily integrated with Gardener's Prometheus' configurations and Grafana's dashboards.

Which issue(s) this PR fixes:
Fixes #gardener/gardener#1351

Release note:

Change monitoring ConfigMaps' syntax of provider-specific components

rfranzke
rfranzke previously approved these changes Sep 27, 2019
@rfranzke rfranzke added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Sep 27, 2019
@gardener-robot-ci-1 gardener-robot-ci-1 added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Sep 27, 2019
Copy link

@wyb1 wyb1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing monitoring extensions! Just found one small thing.

@rfranzke rfranzke added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Sep 30, 2019
@rfranzke rfranzke requested a review from wyb1 September 30, 2019 06:46
@gardener-robot-ci-3 gardener-robot-ci-3 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Sep 30, 2019
Copy link

@wyb1 wyb1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants