Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Latest main is not sending metrics compared to v0.5 #86

Closed
wiardvanrij opened this issue Sep 24, 2024 · 2 comments · Fixed by #88
Closed

Latest main is not sending metrics compared to v0.5 #86

wiardvanrij opened this issue Sep 24, 2024 · 2 comments · Fixed by #88

Comments

@wiardvanrij
Copy link

Hi folks, maybe I'm missing something but trying out the main branch, I cannot get it to send metrics to our remote write endpoint. Figured to create an issue for it.

Example usage:

docker run quay.io/prometheuscommunity/avalanche:main --remote-url=redacted --remote-tenant=redacted   --remote-requests-count=1000 --remote-tenant-header=redacted --gauge-metric-count=200
Emulate Docker CLI using podman. 
16:47:15 avalanche.go:84: initializing avalanche...
16:47:15 write.go:128: Sending:  0 timeseries, 1000 samples, 2000 timeseries per request, 100ms delay between requests

Maybe I'm missing a flag, but I'm unsure which.

Using the 0.5.0 version works fine:

 Sending:  5000 timeseries, 1000 samples, 2000 timeseries per request, 100ms delay between requests
@bwplotka
Copy link
Member

Yup, we are doing some development on main, potentially impacting things, thanks for reporting!

The 5000 timeseries, vs 0 timeseries is interesting... let me investigate a bit.

bwplotka added a commit that referenced this issue Sep 26, 2024
Fixes #86

Signed-off-by: bwplotka <[email protected]>
bwplotka added a commit that referenced this issue Sep 26, 2024
Fixes #86

Signed-off-by: bwplotka <[email protected]>
@bwplotka
Copy link
Member

Fix: #88 review welcome (:

bwplotka added a commit that referenced this issue Sep 26, 2024
Fixes #86

Signed-off-by: bwplotka <[email protected]>
prateeknayak pushed a commit to prateeknayak/avalanche that referenced this issue Jan 2, 2025
* fixed regressions around series leak due to races; refactor for clarity


* fixed remote write regresion.

Fixes prometheus-community#86


---------
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 a pull request may close this issue.

2 participants