Added conditional statement for demandPerInstanceType

Author: Bobbins228

controllers/appwrapper_controller.go

@@ -102,19 +102,21 @@ func (r *AppWrapperReconciler) Reconcile(ctx context.Context, req ctrl.Request)
 			if appwrapper.Status.State == arbv1.AppWrapperStateEnqueued || appwrapper.Status.State == "" {
 				demandPerInstanceType := discoverInstanceTypes(&appwrapper)
 				//TODO: simplify the looping
-				if useMachineSets {
-					if r.canScaleMachineset(ctx, demandPerInstanceType) {
-						r.scaleUp(ctx, &appwrapper, demandPerInstanceType)
-					} else {
-						klog.Infof("Cannot scale up replicas max replicas allowed is %v", maxScaleNodesAllowed)
-					}
-				} else {
-					if canScaleMachinepool(demandPerInstanceType) {
-						if err := r.scaleUp(ctx, &appwrapper, demandPerInstanceType); err != nil {
-							return ctrl.Result{}, err
+				if demandPerInstanceType != nil {
+					if useMachineSets {
+						if r.canScaleMachineset(ctx, demandPerInstanceType) {
+							r.scaleUp(ctx, &appwrapper, demandPerInstanceType)
+						} else {
+							klog.Infof("Cannot scale up replicas max replicas allowed is %v", maxScaleNodesAllowed)
 						}
 					} else {
-						klog.Infof("Cannot scale up replicas max replicas allowed is %v", maxScaleNodesAllowed)
+						if canScaleMachinepool(demandPerInstanceType) {
+							if err := r.scaleUp(ctx, &appwrapper, demandPerInstanceType); err != nil {
+								return ctrl.Result{}, err
+							}
+						} else {
+							klog.Infof("Cannot scale up replicas max replicas allowed is %v", maxScaleNodesAllowed)
+						}
 					}
 				}
 				return ctrl.Result{}, nil