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.

Sign up for GitHub

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

Jump to bottom

index >= CMT_SUMMARY_QUANTILE_ELEMENT_LIMIT #167

Closed
nokute78 opened this issue Nov 3, 2022 · 1 comment
Closed

index >= CMT_SUMMARY_QUANTILE_ELEMENT_LIMIT #167

nokute78 opened this issue Nov 3, 2022 · 1 comment
Assignees
@leonardo-albertovich @pwhelan

Comments

@nokute78
Copy link
Contributor

nokute78 commented Nov 3, 2022

fluent/fluent-bit#6322

Fluent-bit master reaches following limit and returned error.
https://github.com/fluent/fluent-bit/blob/v2.0.3/lib/cmetrics/src/cmt_decode_msgpack.c#L446

[2022/11/03 09:28:07] [error] [output:stdout:stdout.0] could not process metrics payload

How to reproduce

  1. Run prometheus
  2. Run Fluent-bit

Fluent-bit configuration

[INPUT]
    name prometheus_scrape
    host 0.0.0.0
    port 9090
    tag prometheus
    metrics_path /metrics?format=prometheus
    scrape_interval 5s

[OUTPUT]
    Name stdout
@nokute78 nokute78 mentioned this issue Nov 3, 2022
Closed
pwhelan added a commit that referenced this issue Nov 3, 2022
@pwhelan
decode: remove quantile limit, re #167.
f061751 
Signed-off-by: Phillip Whelan <[email protected]>
@pwhelan pwhelan mentioned this issue Nov 3, 2022
Merged
pwhelan added a commit that referenced this issue Nov 6, 2022
@pwhelan
decode: remove quantile limit, re #167.
c74ddc7 
Signed-off-by: Phillip Whelan <[email protected]>
edsiper pushed a commit that referenced this issue Nov 7, 2022
@pwhelan @edsiper
decode: remove quantile limit, re #167.
1d32567 
Signed-off-by: Phillip Whelan <[email protected]>
@edsiper
Copy link
Member

edsiper commented Nov 9, 2022

fixed in #168

@edsiper edsiper closed this as completed Nov 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@leonardo-albertovich leonardo-albertovich

@pwhelan pwhelan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@leonardo-albertovich @edsiper @pwhelan @nokute78