test: add nodeSelector Helm test

Author: shaimendel

test/integration/kubernetes.test.ts

@@ -349,6 +349,20 @@ tap.test(`snyk-monitor has resource limits`, async (t) => {
   t.ok(monitorResources.requests.memory !== undefined, 'snyk-monitor has memory resource request');
 });
 
+tap.test('snyk-monitor has nodeSelector', async (t) => {
+  t.plan(1);
+
+  if (process.env['DEPLOYMENT_TYPE'] !== 'Helm') {
+    t.pass('Not testing nodeSelector because we\'re not installing with Helm');
+    return;
+  }
+
+  const snykMonitorDeployment = await kubectl.getDeploymentJson('snyk-monitor', 'snyk-monitor');
+  const spec = snykMonitorDeployment.spec.template.spec;
+
+  t.ok('nodeSelector' in spec, 'snyk-monitor has nodeSelector');
+});
+
 tap.test('snyk-monitor secure configuration is as expected', async (t) => {
   const kubeConfig = new KubeConfig();
   kubeConfig.loadFromDefault();

test/setup/deployers/helm-with-proxy.ts

@@ -35,7 +35,7 @@ async function deployKubernetesMonitor(
       `--set image.tag=${imageTag} ` +
       `--set image.pullPolicy=${imagePullPolicy} ` +
       '--set integrationApi=https://kubernetes-upstream.dev.snyk.io ' +
-      '--set https_proxy=http://forwarding-proxy:8080',
+      '--set https_proxy=http://forwarding-proxy:8080'
   );
   console.log(`Deployed ${imageOptions.nameAndTag} with pull policy ${imageOptions.pullPolicy}`);
 }

test/setup/deployers/helm.ts

@@ -29,7 +29,8 @@ async function deployKubernetesMonitor(
       `--set image.repository=${imageName} ` +
       `--set image.tag=${imageTag} ` +
       `--set image.pullPolicy=${imagePullPolicy} ` +
-      '--set integrationApi=https://kubernetes-upstream.dev.snyk.io',
+      '--set integrationApi=https://kubernetes-upstream.dev.snyk.io ' +
+      '--set nodeSelector."kubernetes\\.io/os"=linux'
   );
   console.log(`Deployed ${imageOptions.nameAndTag} with pull policy ${imageOptions.pullPolicy}`);
 }