From 675c6ec19439053b2281db34f7cbfcad0d7238d0 Mon Sep 17 00:00:00 2001 From: Cor Zuurmond Date: Mon, 6 May 2024 16:19:11 +0200 Subject: [PATCH] Configuration parameter referencing secret should have double curly brackets --- src/databricks/labs/ucx/azure/access.py | 2 +- tests/unit/azure/test_access.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/databricks/labs/ucx/azure/access.py b/src/databricks/labs/ucx/azure/access.py index e11002df2c..94cb440b82 100644 --- a/src/databricks/labs/ucx/azure/access.py +++ b/src/databricks/labs/ucx/azure/access.py @@ -122,7 +122,7 @@ def _update_cluster_policy_definition( self._policy_config("OAuth") ) policy_dict[f"spark_conf.fs.azure.account.oauth2.client.secret.{storage.name}.dfs.core.windows.net"] = ( - self._policy_config(f"{{secrets/{inventory_database}/uber_principal_secret}}") + self._policy_config("{{secrets/" + inventory_database + "/uber_principal_secret}}") ) return json.dumps(policy_dict) diff --git a/tests/unit/azure/test_access.py b/tests/unit/azure/test_access.py index 639df7a878..8964b96e8e 100644 --- a/tests/unit/azure/test_access.py +++ b/tests/unit/azure/test_access.py @@ -343,7 +343,7 @@ def test_create_global_spn(): "spark_conf.fs.azure.account.auth.type.sto2.dfs.core.windows.net": {"type": "fixed", "value": "OAuth"}, "spark_conf.fs.azure.account.oauth2.client.secret.sto2.dfs.core.windows.net": { "type": "fixed", - "value": "{secrets/ucx/uber_principal_secret}", + "value": "{{secrets/ucx/uber_principal_secret}}", }, } w.cluster_policies.edit.assert_called_with(