-
Notifications
You must be signed in to change notification settings - Fork 447
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 support for serverAPIPath #1197
base: main
Are you sure you want to change the base?
Add support for serverAPIPath #1197
Conversation
✅ Deploy Preview for keda ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Thank you for your contribution! 🙏 We will review your PR as soon as possible.
While you are waiting, make sure to:
Learn more about:
|
@@ -32,6 +33,7 @@ triggers: | |||
**Parameter list:** | |||
|
|||
- `serverAddress` - Address of Prometheus server. If using VictoriaMetrics cluster version, set full URL to Prometheus querying API, e.g. `http://<vmselect>:8481/select/0/prometheus` | |||
- `serverAPIPath` - API path to query the Prometheus Server. On Grafana Cloud, this would be `/api/prom`. (Default: `/api/v1/query`) |
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.
- `serverAPIPath` - API path to query the Prometheus Server. On Grafana Cloud, this would be `/api/prom`. (Default: `/api/v1/query`) | |
- `serverAPIPath` - API path to query the Prometheus Server. (Default: "/api/v1/query", Optional, ex: On Grafana Cloud, this would be `/api/prom`) |
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.
I have updated a bit the style to follow the style guide 😄
I have added a new config value for the Prometheus Scaler,
serverAPIPath
. This is to support hosted Prometheus vendors who do not expose Prometheus on/api/v1/query
by defaultChecklist