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

Support multi dimension values and configurable metricValues on AWS Cloudwatch Scaler #1517

Merged
merged 1 commit into from
Jan 21, 2021

Conversation

JonasMatos0
Copy link
Contributor

@JonasMatos0 JonasMatos0 commented Jan 20, 2021

…loudwatch Scaler

Co-authored-by: Jonas Matos js.matos@sidi.org.br
Co-authored-by: Thiago Teodoro Rodrigues t.teodoro@sidi.org.br

Provide a description of what has been changed

Checklist

Fixes #

@ThiagoTeodoro
Copy link

My name is wrong, pls when its possible make the correction =D , just to not forget.
Thanks for mentioned me as Co-authored.
Name : Thiago Teodoro Rodrigues
@JonasMatos0

@tomkerkhove tomkerkhove changed the title Support multi dimension values and configurable metricValues on AWS C… Support multi dimension values and configurable metricValues on AWS Cloudwatch Scaler Jan 21, 2021
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 open PR with this change in https://github.com/kedacore/keda-docs? And add this to the Changelog (Improvements section)

@JonasMatos0 JonasMatos0 force-pushed the cloudwatch-multidimension branch 6 times, most recently from fceb33e to 54772a9 Compare January 21, 2021 11:17
…loudwatch scaler

Signed-off-by: Jonas Matos <js.matos@sidi.org.br>
Signed-off-by: Thiago Teodoro Rodrigues <t.teodoro@sidi.org.br>
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, thanks!

@zroubalik
Copy link
Member

We can merge this one, once the docs PR si ready: kedacore/keda-docs#352

@zroubalik zroubalik merged commit 1b913f1 into kedacore:main Jan 21, 2021
ycabrer pushed a commit to ycabrer/keda that referenced this pull request Mar 1, 2021
…loudwatch scaler (kedacore#1517)

Signed-off-by: Jonas Matos <js.matos@sidi.org.br>

Co-authored-by: Jonas Matos js.matos@sidi.org.br
Co-authored-by: Thiago Teodoro Rodrigues t.teodoro@sidi.org.br
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