Skip to content

block panic when prom returns range vector#1637

Merged
aryan9600 merged 1 commit intofluxcd:mainfrom
ta924:matrixpanic
May 23, 2024
Merged

block panic when prom returns range vector#1637
aryan9600 merged 1 commit intofluxcd:mainfrom
ta924:matrixpanic

Conversation

@ta924
Copy link
Contributor

@ta924 ta924 commented Apr 30, 2024

Signed-off-by: Tanner Altares ta924@yahoo.com

This is a PR to address 1636, which keeps flagger from throwing a panic in the event the prom query returns a range vector.

@ta924 ta924 requested a review from stefanprodan as a code owner April 30, 2024 18:09
@codecov-commenter
Copy link

codecov-commenter commented Apr 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 34.10%. Comparing base (553184b) to head (e5dfbf4).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1637   +/-   ##
=======================================
  Coverage   34.09%   34.10%           
=======================================
  Files         282      282           
  Lines       20554    20556    +2     
=======================================
+ Hits         7008     7010    +2     
  Misses      12616    12616           
  Partials      930      930           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@aryan9600 aryan9600 left a comment

Choose a reason for hiding this comment

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

lgtm! thanks @ta924 🏅

block panic when prom returns range vector
@aryan9600 aryan9600 merged commit 8e86366 into fluxcd:main May 23, 2024
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.

4 participants