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

GCP Pub/Sub Scaler: add oldest unacked message age metric #2266

Merged

Conversation

fira42073
Copy link
Contributor

@fira42073 fira42073 commented Nov 9, 2021

add ability to scale using oldest unacked message age metric from gcp stackdriver and update tests

  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • Tests have been added
  • A PR is opened to update the documentation on (repo) (if applicable)
  • Changelog has been updated

Fixes #2226

… stackdriver and update tests

Signed-off-by: Friedrich Albert Kyuri <friedrichak42@gmail.com>
@fira42073 fira42073 force-pushed the feature/pubsub-oldestmessageage-scaler branch from e722f61 to cc89be6 Compare November 9, 2021 22:30
Signed-off-by: Friedrich Albert Kyuri <friedrichak42@gmail.com>
@zroubalik zroubalik changed the title Add oldest unacked message age metric GCP Pub/Sub Scaler: add oldest unacked message age metric Nov 12, 2021
@zroubalik
Copy link
Member

@hermanbanken @jmalvarezf-lmes PTAL

@fira42073
Copy link
Contributor Author

PR for documentation kedacore/keda-docs#579

Signed-off-by: Friedrich Albert Kyuri <friedrichak42@gmail.com>
Copy link
Contributor

@jmalvarezf-lmes jmalvarezf-lmes left a comment

Choose a reason for hiding this comment

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

Hope it helps!

pkg/scalers/gcp_pub_sub_scaler.go Show resolved Hide resolved
pkg/scalers/gcp_pub_sub_scaler.go Outdated Show resolved Hide resolved
Friedrich Albert Kyuri added 2 commits November 12, 2021 19:06
Signed-off-by: Friedrich Albert Kyuri <friedrichak42@gmail.com>
Signed-off-by: Friedrich Albert Kyuri <friedrichak42@gmail.com>
@fira42073
Copy link
Contributor Author

@jmalvarezf-lmes PTAL

@jmalvarezf-lmes
Copy link
Contributor

Hi @Friedrich42,

For me it is fine, thank you!

Regards,

Signed-off-by: Friedrich Albert Kyuri <friedrichak42@gmail.com>
Copy link
Member

@zroubalik zroubalik left a comment

Choose a reason for hiding this comment

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

Looking good, could you please update the Changelog as well?
https://github.com/kedacore/keda/blob/main/CHANGELOG.md#improvements

Unreleased->Improvements

@zroubalik zroubalik added this to the v2.5.0 milestone Nov 15, 2021
Friedrich Albert Kyuri added 4 commits November 18, 2021 13:10
Signed-off-by: Friedrich Albert Kyuri <friedrichak42@gmail.com>
…e field

Signed-off-by: Friedrich Albert Kyuri <friedrichak42@gmail.com>
Signed-off-by: Friedrich Albert Kyuri <friedrichak42@gmail.com>
Signed-off-by: Friedrich Albert Kyuri <friedrichak42@gmail.com>
CHANGELOG.md Outdated Show resolved Hide resolved
Signed-off-by: Friedrich Albert Kyuri <friedrichak42@gmail.com>
@fira42073 fira42073 force-pushed the feature/pubsub-oldestmessageage-scaler branch from 8a6454c to 95ecc7c Compare November 18, 2021 14:46
Signed-off-by: Friedrich Albert Kyuri <friedrichak42@gmail.com>
Signed-off-by: Friedrich Albert Kyuri <friedrichak42@gmail.com>
Copy link
Member

@zroubalik zroubalik left a comment

Choose a reason for hiding this comment

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

LGTM

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.

Google pubsub custom metrics support
4 participants