fix the fatal errors when reading config file

cmd/awscollector/main.go

@@ -47,12 +47,6 @@ func main() {
 
 	// init cfgFactory
 	cfgFactory := config.GetParserProvider()
-	// TODO(pingleig): a hack to keep consistent behaviour with previous code
-	// so we exit 0 when running on vm for config error.
-	// https://github.com/aws-observability/aws-otel-collector/issues/340
-	if err := config.TryFileConfig(); err != nil {
-		os.Exit(0)
-	}
 
 	// init lumberFunc for zap logger
 	lumberHook := logger.GetLumberHook()
@@ -74,7 +68,7 @@ func main() {
 		ApplicationStartInfo: info,
 		ParserProvider:       cfgFactory,
 		LoggingOptions:       []zap.Option{zap.Hooks(lumberHook)}}); err != nil {
-		log.Fatal(err)
+		logFatal(err)
 	}
 
 }

cmd/awscollector/main_others.go

@@ -39,3 +39,7 @@ func run(params service.Parameters) error {
 	}
 	return runInteractive(params)
 }
+
+func logFatal(err error) {
+	log.Fatal(err)
+}

cmd/awscollector/main_windows.go

@@ -18,6 +18,10 @@
 package main
 
 import (
+	"log"
+	"os"
+	"strings"
+
 	"github.com/pkg/errors"
 	"golang.org/x/sys/windows/svc"
 
@@ -40,8 +44,19 @@ func run(params service.Parameters) error {
 func runService(params service.Parameters) error {
 	// do not need to supply service name when startup is invoked through Service Control Manager directly
 	if err := svc.Run("", service.NewWindowsService(params)); err != nil {
+		if strings.Contains(err.Error(), "cannot load configuration's parser:") {
+			return errors.New(err.Error())
+		}
 		return errors.Wrap(err, "failed to start service")
 	}
 
 	return nil
 }
+
+// Has to set exit code 0 for the fatal errors when run the collector as service in Windows
+// to prevent infinite reboot by Windows service
+// https://github.com/aws-observability/aws-otel-collector/issues/340
+func logFatal(err error) {
+	log.Println(err.Error())
+	os.Exit(0)
+}