Propagate resource requirements config to all init containers and upd…

…ate jobs (#628)

fixes: #613 

This MR:
- Propgates `OpenSearchCluster.Spec.InitHelper. Resources` config to all
init containers missing resource requirements.
-
[NewSTSForNodePool.keystoreInitContainer](https://github.com/Opster/opensearch-k8s-operator/blob/7bdbfc08fe92193ad792508f27ea5eff1eefdfc4/opensearch-operator/pkg/builders/cluster.go#L322)
-
[NewSTSForNodePool.SetVMMaxMapCount](https://github.com/Opster/opensearch-k8s-operator/blob/7bdbfc08fe92193ad792508f27ea5eff1eefdfc4/opensearch-operator/pkg/builders/cluster.go#L472)
-
[NewBootstrapPod.init](https://github.com/Opster/opensearch-k8s-operator/blob/7bdbfc08fe92193ad792508f27ea5eff1eefdfc4/opensearch-operator/pkg/builders/cluster.go#L751)
-
[NewBootstrapPod.SetVMMaxMapCount](https://github.com/Opster/opensearch-k8s-operator/blob/7bdbfc08fe92193ad792508f27ea5eff1eefdfc4/opensearch-operator/pkg/builders/cluster.go#L810C21-L810C37)
-
[NewSnapshotRepoconfigUpdateJob.snapshotrepoconfig](https://github.com/Opster/opensearch-k8s-operator/blob/7bdbfc08fe92193ad792508f27ea5eff1eefdfc4/opensearch-operator/pkg/builders/cluster.go#L927)
- Creates a new OS Cluster API field
`instance.Spec.Security.Config.UpdateJob.Resources` to allow specifying
resources for
[NewSecurityconfigUpdateJob.updater](https://github.com/Opster/opensearch-k8s-operator/blob/7bdbfc08fe92193ad792508f27ea5eff1eefdfc4/opensearch-operator/pkg/builders/cluster.go#L989).

---------

Signed-off-by: Ramon Heras <ramon98heras@gmail.com>

charts/opensearch-cluster/templates/opensearch-cluster-cr.yaml

@@ -134,8 +134,8 @@ spec:
         {{- if .Values.opensearchCluster.dashboards.resources.limits.cpu }}
         cpu: {{ .Values.opensearchCluster.dashboards.resources.limits.cpu }}
         {{- end }}
-        {{- end }}
-        {{- end }}
+      {{- end }}
+    {{- end }}
     {{- if .Values.opensearchCluster.dashboards.tls }}
     tls:
       {{- if .Values.opensearchCluster.dashboards.tls.enable }}
@@ -280,6 +280,30 @@ spec:
       securityConfigSecret:
         name: {{ .Values.opensearchCluster.security.config.securityConfigSecret.name }}
       {{- end }}
+      {{- if .Values.opensearchCluster.security.config.updateJob }}
+      updateJob:
+        {{- if .Values.opensearchCluster.security.config.updateJob.resources }}
+        resources:
+          {{- if .Values.opensearchCluster.security.config.updateJob.resources.requests }}
+          requests:
+            {{- if .Values.opensearchCluster.security.config.updateJob.resources.requests.memory }}
+            memory: {{ .Values.opensearchCluster.security.config.updateJob.resources.requests.memory }}
+            {{- end }}
+            {{- if .Values.opensearchCluster.security.config.updateJob.resources.requests.cpu }}
+            cpu: {{ .Values.opensearchCluster.security.config.updateJob.resources.requests.cpu }}
+            {{- end }}
+          {{- end }}
+          {{- if .Values.opensearchCluster.security.config.updateJob.resources.limits }}
+          limits:
+          {{- if .Values.opensearchCluster.security.config.updateJob.resources.limits.memory }}
+            memory: {{ .Values.opensearchCluster.security.config.updateJob.resources.limits.memory }}
+            {{- end }}
+            {{- if .Values.opensearchCluster.security.config.updateJob.resources.limits.cpu }}
+            cpu: {{ .Values.opensearchCluster.security.config.updateJob.resources.limits.cpu }}
+            {{- end }}
+          {{- end }}
+        {{- end }}
+      {{- end }}
     {{- end }}
     {{- if .Values.opensearchCluster.security.tls }}
     tls:

charts/opensearch-operator/files/opensearch.opster.io_opensearchactiongroups.yaml

@@ -3,8 +3,7 @@ apiVersion: apiextensions.k8s.io/v1
 kind: CustomResourceDefinition
 metadata:
   annotations:
-    controller-gen.kubebuilder.io/version: v0.10.0
-  creationTimestamp: null
+    controller-gen.kubebuilder.io/version: v0.14.0
   name: opensearchactiongroups.opensearch.opster.io
 spec:
   group: opensearch.opster.io
@@ -24,14 +23,19 @@ spec:
           API
         properties:
           apiVersion:
-            description: 'APIVersion defines the versioned schema of this representation
-              of an object. Servers should convert recognized schemas to the latest
-              internal value, and may reject unrecognized values. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#resources'
+            description: |-
+              APIVersion defines the versioned schema of this representation of an object.
+              Servers should convert recognized schemas to the latest internal value, and
+              may reject unrecognized values.
+              More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#resources
             type: string
           kind:
-            description: 'Kind is a string value representing the REST resource this
-              object represents. Servers may infer this from the endpoint the client
-              submits requests to. Cannot be updated. In CamelCase. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#types-kinds'
+            description: |-
+              Kind is a string value representing the REST resource this object represents.
+              Servers may infer this from the endpoint the client submits requests to.
+              Cannot be updated.
+              In CamelCase.
+              More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#types-kinds
             type: string
           metadata:
             type: object
@@ -45,12 +49,15 @@ spec:
               description:
                 type: string
               opensearchCluster:
-                description: LocalObjectReference contains enough information to let
-                  you locate the referenced object inside the same namespace.
+                description: |-
+                  LocalObjectReference contains enough information to let you locate the
+                  referenced object inside the same namespace.
                 properties:
                   name:
-                    description: 'Name of the referent. More info: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
-                      TODO: Add other useful fields. apiVersion, kind, uid?'
+                    description: |-
+                      Name of the referent.
+                      More info: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
+                      TODO: Add other useful fields. apiVersion, kind, uid?
                     type: string
                 type: object
                 x-kubernetes-map-type: atomic
@@ -67,10 +74,10 @@ spec:
               existingActionGroup:
                 type: boolean
               managedCluster:
-                description: UID is a type that holds unique ID values, including
-                  UUIDs.  Because we don't ONLY use UUIDs, this is an alias to string.  Being
-                  a type captures intent and helps make sure that UIDs and names do
-                  not get conflated.
+                description: |-
+                  UID is a type that holds unique ID values, including UUIDs.  Because we
+                  don't ONLY use UUIDs, this is an alias to string.  Being a type captures
+                  intent and helps make sure that UIDs and names do not get conflated.
                 type: string
               reason:
                 type: string