-
Notifications
You must be signed in to change notification settings - Fork 220
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
Allow customizing the metrics-server namespace #657
Conversation
Signed-off-by: Jesús Fernández <[email protected]>
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.
Looking good. Please revert the changes related with new version (we ship them manually) and update the chart readme (https://github.com/kedacore/charts/blob/main/CONTRIBUTING.md#documentation)
@@ -21,7 +21,7 @@ $ helm repo add kedacore https://kedacore.github.io/charts | |||
$ helm search repo kedacore | |||
NAME CHART VERSION APP VERSION DESCRIPTION | |||
kedacore/external-scaler-azure-cosmos-db 0.1.0 0.1.0 Event-based autoscaler for Azure Cosmos DB chan... | |||
kedacore/keda 2.14.2 2.14.0 Event-based autoscaler for workloads on Kubernetes | |||
kedacore/keda 2.14.3 2.14.0 Event-based autoscaler for workloads on Kubernetes |
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.
revert this change please
@@ -8,7 +8,7 @@ kubeVersion: ">=v1.23.0-0" | |||
|
|||
# This is the chart version. This version number should be incremented each time you make changes | |||
# to the chart and its templates, including the app version. | |||
version: 2.14.2 | |||
version: 2.14.3 |
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.
revert this change please
@@ -277,6 +277,8 @@ rbac: | |||
|
|||
# -- Whether RBAC for configured CRDs that can have a `scale` subresource should be created | |||
enabledCustomScaledRefKinds: true | |||
# -- Customize the namespace for metrics-server |
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.
We should highlight this is Kubernetes Metrics Server and not KEDA Metrics Server
Duplicate of #674 |
Add the ability to customize where
metrics-server
is installed, rather than defaultkube-system
, so the roleBinding is created accordingly.Checklist
Fixes #645