Skip to content

Commit b880ad1

Browse files
authored
Merge pull request #1493 from devtron-labs/prometh-cluster
fix: not able to change Authentication Type for Prometheus.
2 parents 73602ca + 147a82c commit b880ad1

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

src/components/cluster/Cluster.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -827,7 +827,7 @@ function Cluster({
827827
config={{}}
828828
reload={reload}
829829
prometheus_url={prometheus_url}
830-
prometheusAuth={state.prometheus}
830+
prometheusAuth={prometheusAuth}
831831
defaultClusterComponent={state.defaultClusterComponent}
832832
isTlsConnection={isTlsConnection}
833833
isClusterDetails={state.isClusterDetails}

src/components/cluster/ClusterForm.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -428,10 +428,11 @@ export default function ClusterForm({
428428
},
429429
prometheus_url: prometheusToggleEnabled ? state.endpoint.value : '',
430430
prometheusAuth: {
431-
userName: prometheusToggleEnabled ? state.userName.value : '',
432-
password: prometheusToggleEnabled ? state.password.value : '',
431+
userName: prometheusToggleEnabled && state.authType.value === AuthenticationType.BASIC ? state.userName.value : '',
432+
password: prometheusToggleEnabled && state.authType.value === AuthenticationType.BASIC ? state.password.value : '',
433433
tlsClientKey: prometheusToggleEnabled ? state.prometheusTlsClientKey.value : '',
434434
tlsClientCert: prometheusToggleEnabled ? state.prometheusTlsClientCert.value : '',
435+
isAnonymous: state.authType.value === AuthenticationType.ANONYMOUS,
435436
},
436437
}
437438
}

0 commit comments

Comments
 (0)