-
Notifications
You must be signed in to change notification settings - Fork 147
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: need support cron hpa #574
Conversation
d022bc7
to
1ac4e87
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.
Cool works!
fc71217
to
28b99e1
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
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.
Please add applyonce policy in example
Signed-off-by: StevenLeiZhang <zhangleiic@163.com>
28b99e1
to
40e5fce
Compare
Done |
Signed-off-by: StevenLeiZhang zhangleiic@163.com
fix:#573
Description of your changes
build one addon based on https://github.com/AliyunContainerService/kubernetes-cronhpa-controller
After addon installed, one new Trait is exposed, which is named "cronhpa". User could use it to set cronhpa for Component.
How has this code been tested?
test it locally.
3.run test Application
It will scale up and down every 30s.
Checklist
I have:
[Addon]
,[example]
or[Doc]
).version
inmetadata.yaml
to generate a new version.Verified Addon promotion rules
If this pr wants to promote an experimental addon to verified, you must check whether meet these conditions too:
metadata.yaml
.README.md
.