keikoproj/instance-manager

Validation: Add validation for Provisioner

eytan-avisror opened this issue · 0 comments

It seems we get a NPE when referenced provisioner is nil.
This causes the controller to crash.

e.g.

apiVersion: instancemgr.keikoproj.io/v1alpha1
kind: InstanceGroup
metadata:
  name: hello-world
  namespace: instance-manager
spec:
  provisioner: eks-managed
  eks:
    < ... >

We should add proper validation and error out instead of crashing the controller in this case.

CC: @uthark