Skip to content

[prometheus-stackdriver-exporter] Clean up deprecated config and add descriptor cache TTL - #7222

Merged
rpahli merged 3 commits into
prometheus-community:mainfrom
eenchevlp:prometheus-stackdriver-exporter-config-cleanup
Sep 1, 2026
Merged

[prometheus-stackdriver-exporter] Clean up deprecated config and add descriptor cache TTL#7222
rpahli merged 3 commits into
prometheus-community:mainfrom
eenchevlp:prometheus-stackdriver-exporter-config-cleanup

Conversation

@eenchevlp

Copy link
Copy Markdown
Contributor

What this PR does / why we need it

This PR updates the prometheus-stackdriver-exporter chart to:

  • Add support for --monitoring.descriptor-cache-ttl via stackdriver.metrics.descriptorCacheTTL
  • Remove the deprecated stackdriver.projectId value in favor of stackdriver.projectIds
  • Remove the deprecated stackdriver.metrics.typePrefixes value in favor of stackdriver.metrics.prefixes
  • Preserve the existing default metric prefix behavior
  • Add ci/descriptor-cache-values.yaml to test a non-default descriptor cache TTL
  • Bump the chart major version due to removal of deprecated values

Which issue this PR fixes

N/A

Special notes for your reviewer

The new descriptorCacheTTL value defaults to 0s, matching the exporter default.

Checklist

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name
  • Add a helm unittest test case to cover this change

@eenchevlp
eenchevlp requested a review from rpahli as a code owner August 28, 2026 06:44
@eenchevlp
eenchevlp force-pushed the prometheus-stackdriver-exporter-config-cleanup branch from 4659511 to 9875818 Compare August 28, 2026 06:48
…escriptor cache TTL

Signed-off-by: Evgeni Enchev <eenchev@liveperson.com>
@eenchevlp
eenchevlp force-pushed the prometheus-stackdriver-exporter-config-cleanup branch from 9875818 to 2b5009f Compare August 28, 2026 06:49
@rpahli
rpahli merged commit 0648275 into prometheus-community:main Sep 1, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants