-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat: GCP Pub/Sub scaler add configurable fallback value when no metric value found #5897
feat: GCP Pub/Sub scaler add configurable fallback value when no metric value found #5897
Conversation
b7f6dcf
to
19237c2
Compare
19237c2
to
78765cc
Compare
/run-e2e gcp |
78765cc
to
6b43b92
Compare
@zroubalik anything else needed from my side to get this pr merged in ? |
Signed-off-by: Benjamin Woodley <bwoodley012@gmail.com>
6b43b92
to
aecfdfb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for the improvement 😄
/run-e2e gcp |
Thanks for the contribution! |
Added an optional valueIfNull configuration value to the GCP Pub/Sub as also found in the older GCP Stackdriver scaler. This allows for a default fallback value when GCP returns no value for a given metric. This prevents excessive error logging especially when using multiple pubsub scalers with low subscription traffic.
Checklist
Relates to #
#5896
Documentation update