Skip to content

Commit

Permalink
The presence of __tmp_port was breaking deduplication rules. (#740)
Browse files Browse the repository at this point in the history
Signed-off-by: Pete Wall <pete.wall@grafana.com>
  • Loading branch information
petewall authored Sep 18, 2024
1 parent 432f38f commit bacf05d
Show file tree
Hide file tree
Showing 61 changed files with 728 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,10 @@ discovery.relabel "annotation_autodiscovery_pods" {
action = "keepequal"
target_label = "__tmp_port"
}
rule {
action = "labeldrop"
regex = "__tmp_port"
}

// If the metrics port number annotation has a value, override the target address to use it, regardless whether it is
// one of the declared ports on that Pod.
Expand Down Expand Up @@ -128,6 +132,10 @@ discovery.relabel "annotation_autodiscovery_services" {
action = "keepequal"
target_label = "__tmp_port"
}
rule {
action = "labeldrop"
regex = "__tmp_port"
}

rule {
source_labels = ["__meta_kubernetes_service_annotation_{{ include "escape_label" .Values.metrics.autoDiscover.annotations.metricsScheme }}"]
Expand Down
8 changes: 8 additions & 0 deletions examples/alloy-autoscaling-and-storage/metrics.alloy

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

16 changes: 16 additions & 0 deletions examples/alloy-autoscaling-and-storage/output.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 8 additions & 0 deletions examples/application-observability/metrics.alloy

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

16 changes: 16 additions & 0 deletions examples/application-observability/output.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 8 additions & 0 deletions examples/azure-aks/metrics.alloy

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

16 changes: 16 additions & 0 deletions examples/azure-aks/output.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 8 additions & 0 deletions examples/bearer-token-auth/metrics.alloy

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

16 changes: 16 additions & 0 deletions examples/bearer-token-auth/output.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 8 additions & 0 deletions examples/beyla/metrics.alloy

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

16 changes: 16 additions & 0 deletions examples/beyla/output.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 8 additions & 0 deletions examples/control-plane-metrics/metrics.alloy

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

1 comment on commit bacf05d

@nightfall-for-github
Copy link

Choose a reason for hiding this comment

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

@petewall [CRITICAL_RISK] Hi, we noticed you made a commit in GitHub that may contain sensitive information (details below). Please double check if you intended to include or share this sensitive information in the issue.

Note that it may have already been addressed automatically by an administrator.

What types of sensitive information were detected?
API Key: Google API

What was the context?

10: value: "AI******************…" - name:

When did this happen?
Wed, 18 Sep 2024 at 19:59:37 UTC

Where did this happen?
Repository: k8s-monitoring-helm
File Path: examples/specific-namespace/output.yaml

What can I do now?
Please review the incident details carefully and help us remediate with these actions
Report as False Positive
Provide Business Justification

Please sign in to comment.