From 2fef845efbbff768935ba701d783126aa97070e2 Mon Sep 17 00:00:00 2001 From: Simon Gerber Date: Thu, 3 Apr 2025 14:57:41 +0200 Subject: [PATCH] Fix missing parameter name updates in documentation and comments The `operator.*_cluster_role_match_labels` parameter was adjusted to `_cluster_role_selectors` in #231, but we missed some references to the old names. --- component/rbac.jsonnet | 2 +- docs/modules/ROOT/pages/references/parameters.adoc | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/component/rbac.jsonnet b/component/rbac.jsonnet index 16a2394c..e03aa99c 100644 --- a/component/rbac.jsonnet +++ b/component/rbac.jsonnet @@ -59,7 +59,7 @@ local aggregated_rbac = [ // argocd-operator's `CONTROLLER_CLUSTER_ROLE` and `SERVER_CLUSTER_ROLE`, cf. // https://argocd-operator.readthedocs.io/en/latest/usage/custom_roles/ // NOTE(sg): we only deploy these cluster roles if the respective -// `cluster_role_match_labels` parameter in `operator` isn't empty. +// `cluster_role_selectors` parameter in `operator` isn't empty. local internalControllerAggregationLabel = { 'rbac.argocd.syn.tools/aggregate-to-controller': 'true', }; diff --git a/docs/modules/ROOT/pages/references/parameters.adoc b/docs/modules/ROOT/pages/references/parameters.adoc index e07fde8e..5a253cf3 100644 --- a/docs/modules/ROOT/pages/references/parameters.adoc +++ b/docs/modules/ROOT/pages/references/parameters.adoc @@ -410,12 +410,12 @@ This variable is configured through component parameter `operator.cluster_scope_ [NOTE] ==== -When parameter `operator.controller_cluster_role_match_labels` isn't empty, setting `CONTROLLER_CUSTOM_ROLE` through this parameter has no effect. +When parameter `operator.controller_cluster_role_selectors` isn't empty, setting `CONTROLLER_CUSTOM_ROLE` through this parameter has no effect. ==== [NOTE] ==== -When parameter `operator.server_cluster_role_match_labels` isn't empty, setting `SERVER_CUSTOM_ROLE` through this parameter has no effect. +When parameter `operator.server_cluster_role_selectors` isn't empty, setting `SERVER_CUSTOM_ROLE` through this parameter has no effect. ==== [IMPORTANT]