Skip to content
This repository has been archived by the owner on Aug 25, 2021. It is now read-only.

Add component label to all services #480

Merged
merged 1 commit into from Jun 12, 2020

Conversation

chancez
Copy link
Contributor

@chancez chancez commented Jun 5, 2020

This enables querying specific services by labels.
Without this label, most services in the chart have no distinguishing
labels, making it difficult to target specific services with a label
selector.

This enables better integration with tools like prometheus-operator
which use label selectors to determine which endpoints to monitor, as
not all services need to be monitored, and some overlap and reference
the same pods.

This enables querying specific services by labels.
Without this label, most services in the chart have no distinguishing
labels, making it difficult to target specific services with a label
selector.

This enables better integration with tools like prometheus-operator
which use label selectors to determine which endpoints to monitor, as
not all services need to be monitored, and some overlap and reference
the same pods.
@hashicorp-cla
Copy link

hashicorp-cla commented Jun 5, 2020

CLA assistant check
All committers have signed the CLA.

@ishustava ishustava added area/chart-only Related to changes that simply require yaml chart changes, e.g. exposing a new field enhancement New feature or request labels Jun 12, 2020
Copy link
Member

@ishustava ishustava left a comment

Choose a reason for hiding this comment

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

Looks great, thanks for the PR!

@ishustava ishustava merged commit 982668d into hashicorp:master Jun 12, 2020
@chancez chancez deleted the component_service_labels branch June 12, 2020 23:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/chart-only Related to changes that simply require yaml chart changes, e.g. exposing a new field enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants