-
Notifications
You must be signed in to change notification settings - Fork 39
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
Arc: configuration settings for GA #1016
base: main
Are you sure you want to change the base?
Conversation
This reverts commit f9c4232.
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
@@ -1,4 +1,5 @@ | |||
{{- if and (ne .Values.AzureMonitorMetrics.ArcExtension true) (.Values.AzureMonitorMetrics.CollectorHPAEnabled)}} | |||
{{- $arcExtensionSettings := include "arc-extension-settings" . | fromYaml }} | |||
{{- if $arcExtensionSettings.hpaEnabled }} |
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.
grace, isn't the only way we are making the customer control replica count is thru deploymentReplicas
param for chart ? So this merge will not evaluate to true for arc ?
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.
Right, exactly. So the logic for this in the helper template is
hpaEnabled: true
{{- if or $isArcExtension (ne .Values.AzureMonitorMetrics.CollectorHPAEnabled true) }}
hpaEnabled: false
{{- end }}
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.
ok thanks. all these new params will be accessible thru arc cli, i am assuming (2 mount settings to enable/disable mounts and also deployment replicas). Also , the deployment replicas isn't synced to goal state by arc extension? is it also not validated? can they have it as 1 or say 21 ?
PR Description
New Feature Checklist