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

batch.max_bytes incompatible with Prometheus remote write sink #18960

Closed
a26nine opened this issue Oct 27, 2023 · 3 comments
Closed

batch.max_bytes incompatible with Prometheus remote write sink #18960

a26nine opened this issue Oct 27, 2023 · 3 comments
Labels
sink: prometheus_remote_write Anything `prometheus_remote_write` sink related type: bug A code related bug.

Comments

@a26nine
Copy link

a26nine commented Oct 27, 2023

A note for the community

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Problem

The Prometheus remote write sink document says that batch.max_bytes works with the sink, but it seems incompatible:

2023-10-25T17:00:37.231123Z ERROR vector::topology: Configuration error. error=Sink "ne_sink_vm_prod": This sink does not allow setting `max_bytes`

Configuration

sinks:
  ne_sink_vm_prod:
    type: prometheus_remote_write
    healthcheck: false
    endpoint: xxx
    auth:
      strategy: basic
      user: xxx
      password: xxx
    batch:
      max_bytes: 5242880
      timeout_secs: 3

Version

0.33.0

Debug Output

No response

Example Data

No response

Additional Context

No response

References

No response

@a26nine a26nine added the type: bug A code related bug. label Oct 27, 2023
@neuronull
Copy link
Contributor

Hello and thanks for filing this report.

Would you mind including your configuration for the sink?

@neuronull neuronull added the sink: prometheus_remote_write Anything `prometheus_remote_write` sink related label Oct 27, 2023
@a26nine
Copy link
Author

a26nine commented Oct 27, 2023

Hello and thanks for filing this report.

Would you mind including your configuration for the sink?

Added.

sinks:
  ne_sink_vm_prod:
    type: prometheus_remote_write
    healthcheck: false
    endpoint: xxx
    auth:
      strategy: basic
      user: xxx
      password: xxx
    batch:
      max_bytes: 5242880
      timeout_secs: 3

@neuronull
Copy link
Contributor

👍 thanks for adding that.

Ok so I reproduced this on v0.33.0 , but on latest from master the issue does not reproduce.

It appears during a very recent change to this sink (#18676) , we fixed this issue.

In the interim you should be able to use the latest nightly build and get past this issue until v0.34.0 is released.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sink: prometheus_remote_write Anything `prometheus_remote_write` sink related type: bug A code related bug.
Projects
None yet
Development

No branches or pull requests

2 participants