All CLA requirements met.

Closes #392
I also upped the version of kustomize as it made things a lot faster (and while I was trying to trouble shoot the double quote issue). After looking over #392 this PR should close it as I pushed up the new installfiles

@nottagg thanks for the PR and sorry we haven't yet taken a look at it. Promise that I'll do it soon, thanks for understanding!

@nottagg apologies again for the delay. I really am not a fan of injecting both the integer and the string form of the port, have spent all day trying to find an alternative.
FWIW, the problem is not with kustomize env variables, we are not using them. What we use is envsubst to replace the env values in the YAML files. envsubst (or the shell? not 100% sure about that) unfortunately replaces the entire "${ENV_NAME}" removing the double quotes. Will continue looking for an alternative. At the same, we can always replace the value 5000 with a port that does not collide with any known apps. Thanks again, appreciate the hard work in this!

Now that I think about it though, an alternative solution would be NOT to define the port in the Makefile. We can do something like that on the pkg/operator/config/manager/manager.yaml file:

# rest of YAML file...
 - name: ALLOC_API_SVC_PORT
    value: "5000"
  - name: TLS_SECRET_NAMESPACE
    valueFrom:
      fieldRef:
        fieldPath: metadata.namespace
name: manager
securityContext:
  allowPrivilegeEscalation: false
livenessProbe:
  httpGet:
    path: /healthz
    port: 8081
  initialDelaySeconds: 15
  periodSeconds: 20
readinessProbe:
  httpGet:
    path: /readyz
    port: 8081
  initialDelaySeconds: 5
  periodSeconds: 10
resources:
  requests:
    cpu: 100m
    memory: 500Mi
  limits:
    cpu: 1000m
    memory: 500Mi
ports:
  - containerPort: 5000
    hostPort: 5000
# rest of YAML file...

People that use the port 5000 can manually change it on this file. Agreed, it's more cumbersome than customizing directly but I think it partially solves the problem. Can we do it this way? Left some more comments as well. Thanks again!

Hi Dimitris. No problem with the delay! I'm just happy to be a part of a OSS project I thoroughly enjoy. Thanks for all of the feedback!

thank you!

• the unit tests fail on this step, can you check? https://github.com/PlayFab/thundernetes/blob/main/.github/workflows/unit-tests.yml#L29
• there was a panic on the end to end tests, let me know if you want any help there!

ping @nottagg, we'd like to push 0.6 next week and we'd love to include your changes!

hey @nottagg, thanks again for working on this! Let me know when it's ready to review again

thanks @nottagg!