Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Provide support for Huawei Cloud CloudEye scaler #478

Merged
merged 4 commits into from
Dec 4, 2019
Merged

Provide support for Huawei Cloud CloudEye scaler #478

merged 4 commits into from
Dec 4, 2019

Conversation

iyacontrol
Copy link
Contributor

add support for huaweicloud cloudeye scaler.

@msftclas
Copy link

msftclas commented Nov 22, 2019

CLA assistant check
All CLA requirements met.

@iyacontrol
Copy link
Contributor Author

samples and docs

@tomkerkhove tomkerkhove changed the title Ces Provide support for Huawei Cloud CloudEye scaler Nov 22, 2019
@tomkerkhove
Copy link
Member

Awesome, thanks @iyacontrol!

@jeffhollan jeffhollan mentioned this pull request Nov 22, 2019
@iyacontrol
Copy link
Contributor Author

hi @tomkerkhove , can you review my code ? thanks.

@tomkerkhove
Copy link
Member

I'll leave this up to @ahmelsayed

@ahmelsayed
Copy link
Contributor

sorry for the delay, I was out last week. I'll take a look at the PR this week

@iyacontrol
Copy link
Contributor Author

hi @ahmelsayed . If you can review my code this week, I would really appreciate it.

Copy link
Contributor

@ahmelsayed ahmelsayed left a comment

Choose a reason for hiding this comment

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

Thanks @iyacontrol The change looks good. I need to resolve #489 first then will merge this PR

@ahmelsayed ahmelsayed merged commit 1cb5be1 into kedacore:master Dec 4, 2019
@iyacontrol iyacontrol deleted the ces branch December 4, 2019 07:50
preflightsiren pushed a commit to preflightsiren/keda that referenced this pull request Nov 7, 2021
Co-authored-by: Taylor Ryan <taylor@orbweaver.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants