feat: add linux node selector

CHANGELOG.md

@@ -6,6 +6,8 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## Unreleased
+### enhancement
+- Add linux node selector @dbudziwojskiNR [#418](https://github.com/newrelic/newrelic-infra-operator/pull/418)
 
 ## v0.17.0 - 2024-02-05
 

charts/newrelic-infra-operator/templates/admission-webhooks/job-patch/job-createSecret.yaml

@@ -47,10 +47,9 @@ spec:
         runAsGroup: 2000
         runAsNonRoot: true
         runAsUser: 2000
-      {{- with include "newrelic.common.nodeSelector" . }}
       nodeSelector:
-        {{- . | nindent 8 -}}
-      {{- end }}
+        kubernetes.io/os: linux
+        {{ include "newrelic.common.nodeSelector" . | nindent 8 }}
       {{- with include "newrelic.common.tolerations" . }}
       tolerations:
         {{- . | nindent 8 -}}

charts/newrelic-infra-operator/templates/admission-webhooks/job-patch/job-patchWebhook.yaml

@@ -47,10 +47,9 @@ spec:
         runAsGroup: 2000
         runAsNonRoot: true
         runAsUser: 2000
-      {{- with include "newrelic.common.nodeSelector" . }}
       nodeSelector:
-        {{- . | nindent 8 -}}
-      {{- end }}
+        kubernetes.io/os: linux
+        {{ include "newrelic.common.nodeSelector" . | nindent 8 }}
       {{- with include "newrelic.common.tolerations" . }}
       tolerations:
         {{- . | nindent 8 -}}

charts/newrelic-infra-operator/templates/deployment.yaml

@@ -71,10 +71,9 @@ spec:
       {{- with include "newrelic.common.priorityClassName" . }}
       priorityClassName: {{ . }}
       {{- end }}
-      {{- with include "newrelic.common.nodeSelector" . }}
       nodeSelector:
-        {{- . | nindent 8 -}}
-      {{- end }}
+        kubernetes.io/os: linux
+        {{ include "newrelic.common.nodeSelector" . | nindent 8 }}
       {{- with include "newrelic.common.tolerations" . }}
       tolerations:
         {{- . | nindent 8 -}}

charts/newrelic-infra-operator/tests/deployment_test.yaml

@@ -0,0 +1,32 @@
+suite: test cluster environment variable setup
+templates:
+  - templates/deployment.yaml
+  - templates/configmap.yaml
+  - templates/secret.yaml
+release:
+  name: my-release
+  namespace: my-namespac
+tests:
+  - it: has a linux node selector by default
+    set:
+      cluster: my-cluster
+      licenseKey: use-whatever
+    asserts:
+      - equal:
+          path: spec.template.spec.nodeSelector
+          value:
+            kubernetes.io/os: linux
+        template: templates/deployment.yaml
+  - it: has a linux node selector and additional selectors
+    set:
+      cluster: my-cluster
+      licenseKey: use-whatever
+      nodeSelector:
+        aCoolTestLabel: aCoolTestValue
+    asserts:
+      - equal:
+          path: spec.template.spec.nodeSelector
+          value:
+            kubernetes.io/os: linux
+            aCoolTestLabel: aCoolTestValue
+        template: templates/deployment.yaml

charts/newrelic-infra-operator/tests/job_serviceaccount_test.yaml

@@ -39,3 +39,26 @@ tests:
       - equal:
           path: spec.template.spec.serviceAccountName
           value: default
+
+  - it: has a linux node selector by default
+    set:
+      cluster: my-cluster
+      licenseKey: use-whatever
+    asserts:
+      - equal:
+          path: spec.template.spec.nodeSelector
+          value:
+            kubernetes.io/os: linux
+
+  - it: has a linux node selector and additional selectors
+    set:
+      cluster: my-cluster
+      licenseKey: use-whatever
+      nodeSelector:
+        aCoolTestLabel: aCoolTestValue
+    asserts:
+      - equal:
+          path: spec.template.spec.nodeSelector
+          value:
+            kubernetes.io/os: linux
+            aCoolTestLabel: aCoolTestValue