-
Notifications
You must be signed in to change notification settings - Fork 433
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
Provide documentation for Kubernetes events #361
Conversation
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.
Thanks for these docs!
I've added a remark on kedacore/keda#1523 (comment) so I'll wait with reviewing until we decided on that.
Would you mind moving these docs from v2.0 to v2.2, please, since that's the version where this will be introduced.
19159b5
to
b0e593f
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.
This might be a stretch, but do you have raw payloads for these events as examples?
Signed-off-by: Ahmed ElSayed <ahmels@microsoft.com>
b0e593f
to
9d0c513
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, thanks a ton @ahmelsayed!
Signed-off-by: Ahmed ElSayed ahmels@microsoft.com
Closes kedacore/keda#530