Skip to content

Conversation

@mikouaj
Copy link
Contributor

@mikouaj mikouaj commented Jun 18, 2021

Using resource labels is mandatory for most Stackdriver resources other then Global, i.e. k8s_pod.
While setting resource type is supported in StackdriverProperties, it is not possible to set labels in the same, configurable way.
Configuring resource type only lead to issues as described in micrometer-metrics/micrometer#1907.

Using valid resource type along with required labels makes it possible to use given metric in a wider set of
GCP solutions, i.e. custom metric based GKE horizontal pod autoscaler.

Using resource labels is mandatory for most Stackdriver resources other
than 'Global', i.e. k8s_pod. Configuring valid resource type along with
related labels makes it possible to use given metric in a wider set of
GCP solutions, i.e. custom metric based GKE pod horizontal autoscaler.
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jun 18, 2021
@wilkinsona wilkinsona added this to the 2.6.x milestone Jul 13, 2021
@wilkinsona wilkinsona added type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Jul 13, 2021
@wilkinsona wilkinsona self-assigned this Jul 13, 2021
@wilkinsona wilkinsona modified the milestones: 2.6.x, 2.6.0-M1 Jul 13, 2021
wilkinsona pushed a commit that referenced this pull request Jul 13, 2021
Using resource labels is mandatory for most Stackdriver resources other
than 'Global', i.e. k8s_pod. Configuring valid resource type along with
related labels makes it possible to use given metric in a wider set of
GCP solutions, i.e. custom metric based GKE pod horizontal autoscaler.

See gh-26961
@wilkinsona
Copy link
Member

@mikouaj Thank you very much for making your first contribution to Spring Boot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants