Fix agentNamespace in spoke

Signed-off-by: Tamal Saha <[email protected]>
kluster-manager · Nov 13, 2023 · 1a47b4d · 1a47b4d
1 parent 0ca0904
commit 1a47b4d
Show file tree

Hide file tree

Showing 8 changed files with 20 additions and 10 deletions.
diff --git a/apis/installer/v1alpha1/cluster_manager_spoke_types.go b/apis/installer/v1alpha1/cluster_manager_spoke_types.go
@@ -46,6 +46,9 @@ type ClusterManagerSpokeSpec struct {
 	// ClusterName: the name of the joined cluster on the hub
 	ClusterName string `json:"clusterName"`
 
+	// AgentNamespace: the namespace to deploy the agent
+	AgentNamespace string `json:"agentNamespace"`
+
 	// SpokeHub: Hub information
 	// +optional
 	Hub SpokeHub `json:"hub"`

diff --git a/charts/cluster-manager-spoke/README.md b/charts/cluster-manager-spoke/README.md
@@ -48,6 +48,7 @@ The following table lists the configurable parameters of the `cluster-manager-sp
 |               Parameter                | Description |                   Default                    |
 |----------------------------------------|-------------|----------------------------------------------|
 | clusterName                            |             | <code>""</code>                              |
+| agentNamespace                         |             | <code>""</code>                              |
 | hub.apiServer                          |             | <code>""</code>                              |
 | hub.kubeConfig                         |             | <code>""</code>                              |
 | klusterlet.name                        |             | <code>klusterlet</code>                      |

diff --git a/charts/cluster-manager-spoke/templates/bootstrap_hub_kubeconfig.yaml b/charts/cluster-manager-spoke/templates/bootstrap_hub_kubeconfig.yaml
@@ -3,7 +3,7 @@ apiVersion: v1
 kind: Secret
 metadata:
   name: bootstrap-hub-kubeconfig
-  namespace: {{ .Release.Namespace }}
+  namespace: {{ required "A valid .agentNamespace entry required!" .Values.agentNamespace }}
 type: Opaque
 data:
   kubeconfig: {{ required "A valid .hub.kubeConfig entry required!" .Values.hub.kubeConfig }}
diff --git a/charts/cluster-manager-spoke/templates/hosted/external_managed_kubeconfig.yaml b/charts/cluster-manager-spoke/templates/hosted/external_managed_kubeconfig.yaml
@@ -5,7 +5,7 @@ apiVersion: v1
 kind: Secret
 metadata:
   name: external-managed-kubeconfig
-  namespace: {{ .Release.Namespace }}
+  namespace: {{ required "A valid .agentNamespace entry required!" .Values.agentNamespace }}
 type: Opaque
 data:
   kubeconfig: {{ .Values.managedKubeconfig }}

diff --git a/charts/cluster-manager-spoke/templates/namespace.yaml b/charts/cluster-manager-spoke/templates/namespace.yaml
@@ -5,3 +5,12 @@ metadata:
   annotations:
     workload.openshift.io/allowed: "management"
   name: open-cluster-management
+
+---
+
+apiVersion: v1
+kind: Namespace
+metadata:
+  annotations:
+    workload.openshift.io/allowed: "management"
+  name: {{ required "A valid .agentNamespace entry required!" .Values.agentNamespace }}
diff --git a/charts/cluster-manager-spoke/values.openapiv3_schema.yaml b/charts/cluster-manager-spoke/values.openapiv3_schema.yaml
@@ -1,4 +1,6 @@
 properties:
+  agentNamespace:
+    type: string
   bundleVersion:
     properties:
       operatorImageVersion:
@@ -65,5 +67,6 @@ properties:
       type: object
     type: array
 required:
+- agentNamespace
 - clusterName
 type: object
diff --git a/charts/cluster-manager-spoke/values.yaml b/charts/cluster-manager-spoke/values.yaml
@@ -1,5 +1,7 @@
 clusterName: ""
 
+agentNamespace: ""
+
 hub:
   apiServer: ""
   kubeConfig: ""

diff --git a/s.yaml b/s.yaml