-
Notifications
You must be signed in to change notification settings - Fork 449
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 keda_scaled_object_paused to the docs #1111
add keda_scaled_object_paused to the docs #1111
Conversation
Signed-off-by: Elad Motola <eladmotola95@gmail.com>
✅ Deploy Preview for keda ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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 nit
Signed-off-by: Elad Motola <eladmotola95@gmail.com>
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.
@geoffrey1330 Please update this pr:
- add opentelemetry docs
- move to version 2.13
@@ -16,6 +16,7 @@ The KEDA Operator exposes Prometheus metrics which can be scraped on port `8080` | |||
- `keda_scaler_errors` - The number of errors that have occurred for each scaler. | |||
- `keda_scaler_errors_total` - The total number of errors encountered for all scalers. | |||
- `keda_scaled_object_errors` - The number of errors that have occurred for each ScaledObejct. | |||
- `keda_scaled_object_paused` - The number of paused scaledObject (using the annotation detailed [here](https://keda.sh/docs/latest/concepts/scaling-deployments/#pause-autoscaling)) |
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.
it is not number of paused scaledobjects, but it marks whether the SO is paused or not.
Move this to line 14, so it is next to activity
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.
Alright I will do that.
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.
@geoffrey1330 please fix this, so we can go ahead and merge the feature
replaced by #1249 |
related to kedacore/keda#4446
Checklist
Fixes #