Fixup adding support for security.keycloak.realm_display_name key (#1054

)
nebari-dev · Feb 11, 2022 · fa89caf · fa89caf
1 parent 81c6c34
commit fa89caf
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/qhub/deploy.py b/qhub/deploy.py
@@ -603,11 +603,13 @@ def _attempt_keycloak_connection(
 
 def provision_06_kubernetes_keycloak_configuration(stage_outputs, config, check=True):
     directory = "stages/06-kubernetes-keycloak-configuration"
+    realm_id = f"qhub-{config['project_name']}"
 
     stage_outputs[directory] = terraform.deploy(
         directory=directory,
         input_vars={
-            "realm": f"qhub-{config['project_name']}",
+            "realm": realm_id,
+            "realm_display_name": config['security']['keycloak'].get('realm_display_name', realm_id),
             "authentication": config["security"]["authentication"],
         },
     )

diff --git a/qhub/template/stages/06-kubernetes-keycloak-configuration/main.tf b/qhub/template/stages/06-kubernetes-keycloak-configuration/main.tf
@@ -1,7 +1,8 @@
 resource "keycloak_realm" "main" {
   provider = keycloak
 
-  realm = var.realm
+  realm        = var.realm
+  display_name = var.realm_display_name
 }
 
 

diff --git a/qhub/template/stages/06-kubernetes-keycloak-configuration/variables.tf b/qhub/template/stages/06-kubernetes-keycloak-configuration/variables.tf
@@ -3,6 +3,11 @@ variable "realm" {
   type        = string
 }
 
+variable "realm_display_name" {
+  description = "Keycloak realm display name for QHub"
+  type        = string
+}
+
 variable "keycloak_groups" {
   description = "Permission groups in keycloak used for granting access to services"
   type = set(string)