Node selector fix

General MR

Summary

Added additional value to allow defender to have its own value for node selector instead of using the current node selector for twistlock console. Also updated the helm template to translate it properly as it was previously getting dumped into the secret as map[key:value] which breaks the deployment of defender.

Relevant logs/screenshots

Validated it functions when specifying a node selector for twistlock and a different one for defender:

Relates issue: #87 (closed)

added kindbug statusdoing labels

added 1 commit

• e199c4b5 - Forgot to bump version in chart.yaml

Compare with previous version

added 1 commit

• f26cdc3a - Updated readme after chart bump

Compare with previous version

marked this merge request as ready

added statusreview label and removed statusdoing label

To test use the following values for twistlock values in a local development environment (will have to use a valid license):

   nodeSelector: 
      node-type: "twistlock"
    console:
      persistence:
        size: 256Mi
      localVolumeUpgrade: true
      license: <Will Have to use internal license to validate>
    defender:
      nodeSelector:
        node-type: "defender"

Prior to deployment make sure to label one or more nodes so they can get scheduled. The labels can be different or the same, it shouldn't matter either way so long as at least one node has it.

I'm not sure if this should be seen as a breaking change as defender was previously just using the same nodeSelector as twistlock console. However, that portion was never set up properly so it would have never worked.

changed milestone to %2.18.0

changed the description

approved this merge request

resolved all threads

Changes all look good, just since Twistlock isn't based on an upstream chart let's bump the minor version for this tag

unapproved this merge request

added priority4 statusdoing labels and removed statusreview label

assigned to @jimmy.bourque

resolved all threads

added 1 commit

• 21bc6377 - Bumped Chart to major

Compare with previous version

added statusreview label and removed statusdoing label

added 1 commit

• 48affa6f - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version