Refactor main package

create internal package and move implementation into this package

tooling/image-sync/repository.go → tooling/image-sync/internal/repository.go

@@ -1,4 +1,4 @@
-package main
+package internal
 
 import (
 	"context"

tooling/image-sync/repository_test.go → tooling/image-sync/internal/repository_test.go

@@ -1,4 +1,4 @@
-package main
+package internal
 
 import (
 	"context"

tooling/image-sync/sync.go → tooling/image-sync/internal/sync.go

@@ -1,4 +1,4 @@
-package main
+package internal
 
 import (
 	"context"
@@ -12,8 +12,13 @@ import (
 	"github.com/containers/image/signature"
 	"github.com/containers/image/types"
 	"github.com/spf13/viper"
+	"go.uber.org/zap"
 )
 
+func Log() *zap.SugaredLogger {
+	return zap.L().Sugar()
+}
+
 // SyncConfig is the configuration for the image sync
 type SyncConfig struct {
 	Repositories   []string

tooling/image-sync/sync_test.go → tooling/image-sync/internal/sync_test.go

@@ -1,4 +1,4 @@
-package main
+package internal
 
 import (
 	"testing"

tooling/image-sync/main.go

@@ -5,6 +5,7 @@ import (
 	"os"
 	"time"
 
+	"github.com/Azure/ARO-HCP/tooling/image-sync/internal"
 	"github.com/spf13/cobra"
 	"github.com/spf13/viper"
 	"go.uber.org/zap"
@@ -17,17 +18,13 @@ var (
 		Short: "image-sync",
 		Long:  "image-sync",
 		RunE: func(cmd *cobra.Command, args []string) error {
-			return DoSync()
+			return internal.DoSync()
 		},
 	}
 	cfgFile  string
 	logLevel string
 )
 
-func Log() *zap.SugaredLogger {
-	return zap.L().Sugar()
-}
-
 func main() {
 	syncCmd.Flags().StringVarP(&cfgFile, "cfgFile", "c", "", "Configuration File")
 	syncCmd.Flags().StringVarP(&logLevel, "logLevel", "l", "", "Loglevel (info, debug, error, warn, fatal, panic)")
@@ -45,8 +42,8 @@ func initConfig() {
 	viper.SetConfigFile(cfgFile)
 	viper.AutomaticEnv()
 
-	if err := viper.ReadInConfig(); err == nil {
-		Log().Debugw("Using configuration", "config", cfgFile)
+	if err := viper.ReadInConfig(); err != nil {
+		defaultlog.Fatal(err)
 	}
 }