FLPATH-1856 Enabling monitoring by default in Helm Based Orchestrator…

… operator
rhdhorchestrator · Dec 14, 2024 · 8684aa6 · 8684aa6
1 parent 9f27885
commit 8684aa6
Show file tree

Hide file tree

Showing 8 changed files with 73 additions and 5 deletions.
diff --git a/bundle/manifests/orchestrator-operator.clusterserviceversion.yaml b/bundle/manifests/orchestrator-operator.clusterserviceversion.yaml
@@ -15,9 +15,15 @@ metadata:
               "enabled": false,
               "namespace": ""
             },
+            "networkPolicy": {
+              "rhdhNamespace": "rhdh-operator"
+            },
             "orchestrator": {
               "namespace": "sonataflow-infra",
               "sonataflowPlatform": {
+                "monitoring": {
+                  "enabled": true
+                },
                 "resources": {
                   "limits": {
                     "cpu": "500m",
@@ -146,7 +152,7 @@ metadata:
     capabilities: Basic Install
     categories: Developer Tools
     console.openshift.io/disable-operand-delete: "true"
-    createdAt: "2024-09-12T17:54:38Z"
+    createdAt: "2024-12-14T05:55:46Z"
     features.operators.openshift.io/cnf: "false"
     features.operators.openshift.io/cni: "false"
     features.operators.openshift.io/csi: "false"
@@ -170,7 +176,7 @@ metadata:
     operatorframework.io/arch.ppc64le: unsupported
     operatorframework.io/arch.s390x: unsupported
     operatorframework.io/suggested-namespace: orchestrator
-  name: orchestrator-operator.v1.2.0-rc12
+  name: orchestrator-operator.v1.3.0-rc6
   namespace: placeholder
 spec:
   apiservicedefinitions: {}
@@ -422,6 +428,23 @@ spec:
           - delete
           - watch
           - patch
+        - apiGroups:
+          - operator.openshift.io
+          resources:
+          - imagecontentsourcepolicies
+          verbs:
+          - get
+          - create
+          - watch
+          - list
+          - update
+          - delete
+        - apiGroups:
+          - config.openshift.io
+          resources:
+          - clusterversions
+          verbs:
+          - get
         serviceAccountName: orchestrator-operator
       deployments:
       - label:
@@ -475,7 +498,7 @@ spec:
                 - --metrics-bind-address=127.0.0.1:8080
                 - --leader-elect
                 - --leader-election-id=orchestrator-operator
-                image: quay.io/orchestrator/orchestrator-operator:1.2.0-rc12
+                image: quay.io/orchestrator/orchestrator-operator:1.3.0-rc6
                 livenessProbe:
                   httpGet:
                     path: /healthz
@@ -564,4 +587,4 @@ spec:
   provider:
     name: Red Hat
     url: https://www.redhat.com
-  version: 1.2.0-rc12
+  version: 1.3.0-rc6
diff --git a/bundle/manifests/rhdh.redhat.com_orchestrators.yaml b/bundle/manifests/rhdh.redhat.com_orchestrators.yaml
@@ -53,6 +53,15 @@ spec:
                       operator is deployed and is watching for argoapp CR instances
                     type: string
                 type: object
+              networkPolicy:
+                description: NetworkPolicy contains the namespace of an existing RHDH
+                  instance
+                properties:
+                  rhdhNamespace:
+                    default: ""
+                    description: Namespace of the existing RHDH instance.
+                    type: string
+                type: object
               orchestrator:
                 description: Orchestrator contains the fields for the orchestrator
                   configuration
@@ -79,6 +88,15 @@ spec:
                           Job Service container image to be used instead of the provided
                           one by SonataFlow
                         type: string
+                      monitoring:
+                        description: Contains the monitoring configuration fields
+                        properties:
+                          enabled:
+                            default: true
+                            description: Enabled determines whether monitoring should
+                              be enabled. Defaults to true.
+                            type: boolean
+                        type: object
                       resources:
                         description: Resources contains the requests and limit of
                           CPU and memory resources for the pod instance
@@ -546,6 +564,16 @@ spec:
                         type: string
                     type: object
                 type: object
+              sonataflow:
+                description: SonatafFlow contains the configuration fields for the
+                  SonataFlow when using release candidate
+                properties:
+                  images:
+                    description: Indicates RC builds images that should be used by
+                      the chart to install Sonataflow
+                    type: object
+                    x-kubernetes-preserve-unknown-fields: true
+                type: object
               tekton:
                 description: Contains the tekton configuration fields
                 properties:

diff --git a/config/crd/bases/rhdh.redhat.com_orchestrators.yaml b/config/crd/bases/rhdh.redhat.com_orchestrators.yaml
@@ -416,6 +416,14 @@ spec:
                   sonataflowPlatform:
                     description: SonataflowPlatform contains the pod resource configuration to be used for the data index and job services
                     properties:
+                      monitoring:
+                        description: Contains the monitoring configuration fields
+                        properties:
+                          enabled:
+                            description: Enabled determines whether monitoring should be enabled. Defaults to true.
+                            default: true
+                            type: boolean
+                        type: object
                       resources:
                         description: Resources contains the requests and limit of CPU and memory resources for the pod instance
                         type: object

diff --git a/config/manager/kustomization.yaml b/config/manager/kustomization.yaml
@@ -5,4 +5,4 @@ kind: Kustomization
 images:
 - name: controller
   newName: quay.io/orchestrator/orchestrator-operator
-  newTag: 1.3.0
+  newTag: 1.3.0-rc6
diff --git a/config/samples/_v1alpha1_orchestrator.yaml b/config/samples/_v1alpha1_orchestrator.yaml
@@ -91,6 +91,8 @@ spec:
   orchestrator:
     namespace: "sonataflow-infra" # Namespace where sonataflow's workflows run. The value is captured when running the setup.sh script and stored as a label in the selected namespace. User can override the value by populating this field. Defaults to `sonataflow-infra`.
     sonataflowPlatform:
+      monitoring:
+        enabled: true
       resources:
         requests:
           memory: "64Mi"

diff --git a/helm-charts/orchestrator/templates/sonataflows.yaml b/helm-charts/orchestrator/templates/sonataflows.yaml
@@ -23,6 +23,8 @@ metadata:
   annotations:
     "meta.helm.sh/release-name": {{ .Release.Name}}
 spec:
+  monitoring:
+    enabled: {{ .Values.orchestrator.sonataflowPlatform.monitoring.enabled }}
   build:
     template:
       resources:

diff --git a/helm-charts/orchestrator/values.schema.json b/helm-charts/orchestrator/values.schema.json
@@ -1347,6 +1347,9 @@
         "orchestrator": {
             "namespace": "sonataflow-infra",
             "sonataflowPlatform": {
+                "monitoring": {
+                    "enabled": false
+                },
                 "resources": {
                     "requests": {
                         "memory": "64Mi",

diff --git a/helm-charts/orchestrator/values.yaml b/helm-charts/orchestrator/values.yaml
@@ -93,6 +93,8 @@ postgres:
 orchestrator:
   namespace: "sonataflow-infra" # Namespace where sonataflow's workflows run. The value is captured when running the setup.sh script and stored as a label in the selected namespace. User can override the value by populating this field. Defaults to `sonataflow-infra`.
   sonataflowPlatform:
+    monitoring:
+      enabled: true
     resources:
       requests:
         memory: "64Mi"