Remove missed runs check

This check caused TF applier to completely stop applying all modules
when it has been down for some time which is not the behaviour we want
in terraform applier.

controllers/module_controller.go

@@ -228,9 +228,6 @@ func NextSchedule(module *tfaplv1beta1.Module, now time.Time, minIntervalBetween
 		// of this controller. In that case, we want to not try to list
 		// all the missed start times.
 		numOfMissedRuns++
-		if numOfMissedRuns > 100 {
-			return 0, time.Time{}, fmt.Errorf("too many missed start times (%d) check clock skew", numOfMissedRuns)
-		}
 	}
 
 	return numOfMissedRuns, sched.Next(now), nil

controllers/module_controller_test.go

@@ -131,21 +131,21 @@ func Test_NextSchedule(t *testing.T) {
 			},
 			wantErr: true,
 		},
-		{
-			name: "no_job_run_for_days",
-			args: args{
-				module: &tfaplv1beta1.Module{
-					ObjectMeta: metav1.ObjectMeta{
-						CreationTimestamp: metav1.Time{Time: time.Date(2022, 02, 01, 01, 00, 00, 0000, time.UTC)},
-					},
-					Spec:   tfaplv1beta1.ModuleSpec{Schedule: "00 */1 * * *"},
-					Status: tfaplv1beta1.ModuleStatus{RunStartedAt: nil},
-				},
-				now:                    time.Date(2022, 03, 01, 01, 00, 00, 0000, time.UTC),
-				minIntervalBetweenRuns: time.Hour,
-			},
-			wantErr: true,
-		},
+		// {
+		// 	name: "no_job_run_for_days",
+		// 	args: args{
+		// 		module: &tfaplv1beta1.Module{
+		// 			ObjectMeta: metav1.ObjectMeta{
+		// 				CreationTimestamp: metav1.Time{Time: time.Date(2022, 02, 01, 01, 00, 00, 0000, time.UTC)},
+		// 			},
+		// 			Spec:   tfaplv1beta1.ModuleSpec{Schedule: "00 */1 * * *"},
+		// 			Status: tfaplv1beta1.ModuleStatus{RunStartedAt: nil},
+		// 		},
+		// 		now:                    time.Date(2022, 03, 01, 01, 00, 00, 0000, time.UTC),
+		// 		minIntervalBetweenRuns: time.Hour,
+		// 	},
+		// 	wantErr: true,
+		// },
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {