-
Notifications
You must be signed in to change notification settings - Fork 441
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
Add OpenStack Swift scaler documentation #342
Add OpenStack Swift scaler documentation #342
Conversation
c235cc4
to
3fda91f
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, just one request for the authentication
|
||
```yaml | ||
triggers: | ||
- type: swift |
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.
Not sure about the name of the scaler, maybeopenstack-swift
is more appropriate?
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.
Makes sense since other scalers use the same standard (azure-queue, azure-blob, redis-streams).
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.
Yeah, thanks. Could you please do the same change in the code? (where relevant of course)
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.
Sure! I'm on it 😄
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.
Scaler name in documentation updated by 2f61e83.
Signed-off-by: Pedro Felipe Dominguite <p.dominguite@sidi.org.br> * Update example section Signed-off-by: Pedro Felipe Dominguite <p.dominguite@sidi.org.br> * Change scaler name to openstack-swift Signed-off-by: Pedro Felipe Dominguite <p.dominguite@sidi.org.br> * Fix typos Signed-off-by: Pedro Felipe Dominguite <p.dominguite@sidi.org.br> * Add more timeout details Signed-off-by: Pedro Felipe Dominguite <p.dominguite@sidi.org.br>
751f063
to
204acc1
Compare
Signed-off-by: Pedro Dominguite p.dominguite@sidi.org.br
This PR adds documentation for OpenStack Swift scaler.
PR for the scaler can be found at kedacore/keda#1462.
Relates to kedacore/keda#1342.