MLPAB-2408 tidy

Makefile

@@ -158,8 +158,8 @@ set-uploads-infected: ##@events calls emit-object-tags-added-with-virus for all
 		key=$$k $(MAKE) emit-object-tags-added-with-virus ; \
     done
 
-tail-logs: ##@app tails logs for app mock-notify, events-lambda, schedule-runner-lambda, mock-onelogin, mock-lpa-store and mock-uid and filters out noisy runner logs
-	docker compose --ansi=always -f docker/docker-compose.yml -f docker/docker-compose.dev.yml logs app mock-notify events-lambda schedule-runner-lambda mock-onelogin mock-lpa-store mock-uid -f | grep -v 'runner'
+tail-logs: ##@app tails logs for app mock-notify, events-lambda, schedule-runner-lambda, mock-onelogin, mock-lpa-store and mock-uid
+	docker compose --ansi=always -f docker/docker-compose.yml -f docker/docker-compose.dev.yml logs app mock-notify events-lambda schedule-runner-lambda mock-onelogin mock-lpa-store mock-uid -f
 
 terraform-update-docs: ##@terraform updates all terraform-docs managed documentation
 	terraform-docs --config terraform/environment/.terraform-docs.yml ./terraform/environment

docker/schedule-runner/Dockerfile

@@ -26,7 +26,7 @@ WORKDIR /app
 COPY --link  docker/install_lambda_insights.sh /app/
 
 RUN chmod +x "/app/install_lambda_insights.sh" \
-  && /app/install_lambda_insights.sh ${TARGETPLATFORM}
+  && /app/install_lambda_insights.sh "${TARGETPLATFORM}"
 
 COPY --from=build /app/schedule-runner ./schedule-runner
 COPY --link lang ./lang

internal/scheduled/runner.go

@@ -79,10 +79,8 @@ func (r *Runner) Run(ctx context.Context) error {
 		if errors.Is(err, dynamo.NotFoundError{}) {
 			r.logger.InfoContext(ctx, "no scheduled tasks to process")
 			return nil
-		} else if errors.Is(err, dynamo.MultipleResultsError{}) {
-			continue
 		} else if err != nil {
-			r.logger.ErrorContext(ctx, "error getting scheduled task:", slog.Any("err", err))
+			r.logger.ErrorContext(ctx, "error getting scheduled task", slog.Any("err", err))
 
 			if err := r.waiter.Wait(); err != nil {
 				return err

internal/scheduled/runner_test.go

@@ -106,7 +106,7 @@ func TestRunnerRun(t *testing.T) {
 func TestRunnerRunWhenStepErrors(t *testing.T) {
 	logger := newMockLogger(t)
 	logger.EXPECT().
-		ErrorContext(ctx, "error getting scheduled task:", slog.Any("err", expectedError))
+		ErrorContext(ctx, "error getting scheduled task", slog.Any("err", expectedError))
 
 	store := newMockScheduledStore(t)
 	store.EXPECT().
@@ -248,7 +248,7 @@ func TestRunnerRunWhenWaitingError(t *testing.T) {
 			logger.EXPECT().
 				InfoContext(ctx, "runner action", slog.String("action", "Action(99)"))
 			logger.EXPECT().
-				ErrorContext(ctx, "error getting scheduled task:", slog.Any("err", waitingError))
+				ErrorContext(ctx, "error getting scheduled task", slog.Any("err", waitingError))
 			logger.EXPECT().
 				InfoContext(ctx, "runner action success",
 					slog.String("action", "Action(99)"),
@@ -300,7 +300,7 @@ func TestRunnerRunWhenConditionalCheckFailsAndWaiterErrors(t *testing.T) {
 
 	logger := newMockLogger(t)
 	logger.EXPECT().
-		ErrorContext(ctx, "error getting scheduled task:", slog.Any("err", dynamo.ConditionalCheckFailedError{}))
+		ErrorContext(ctx, "error getting scheduled task", slog.Any("err", dynamo.ConditionalCheckFailedError{}))
 
 	runner := &Runner{
 		now:    testNowFn,