Skip to content

test(promqle2etest): add vanilla Prometheus remote write 2.0 to GCM test case#334

Draft
bwplotka wants to merge 2 commits into
release-2.53.5-gmpfrom
rw
Draft

test(promqle2etest): add vanilla Prometheus remote write 2.0 to GCM test case#334
bwplotka wants to merge 2 commits into
release-2.53.5-gmpfrom
rw

Conversation

@bwplotka

Copy link
Copy Markdown
Collaborator

Add PrometheusRemoteWriteGCMBackend (prom-prw-gcm) to test vanilla Prometheus Remote Write 2.0 ingestion against GCM. Configure required feature flags (--enable-feature=st-synthesis, etc.) and remote_write settings (google_iam, write_relabel_configs, convert_classic_histograms_to_nhcb).

TAG=agy
CONV=e18940f4-6a0d-4ae6-b6cd-5bfcc9700aa5

…est case

Add PrometheusRemoteWriteGCMBackend (`prom-prw-gcm`) to test vanilla
Prometheus Remote Write 2.0 ingestion against GCM. Configure required
feature flags (`--enable-feature=st-synthesis`, etc.) and remote_write
settings (`google_iam`, `write_relabel_configs`, `convert_classic_histograms_to_nhcb`).

TAG=agy
CONV=e18940f4-6a0d-4ae6-b6cd-5bfcc9700aa5

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a new test backend, PrometheusRemoteWriteGCMBackend, which configures a vanilla Prometheus instance to push metrics to Google Cloud Monitoring (GCM) via the Prometheus Remote Write 2.0 protocol. This backend is integrated into the existing GCM acceptance tests to validate its behavior. A review comment identifies an issue in the Prometheus write relabel configuration where labeldrop is incorrectly configured with source_labels instead of regex, which could result in dropping all labels.

Comment thread google/internal/promqle2etest/backend_prw_gcm.go
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
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.

1 participant