chore: add back traces when extracting the files from the box (#946)

* fix: use a more comprehensive initialisation method for configs

As go init() method is not deterministic, I found that the logger init was
not called at the right time. With change we ensure that the Init is:
1) called first
2) existing it the config was already populated

* chore: add back traces when extracting the files from the box

cli/cmd/deploy.go

@@ -17,7 +17,7 @@ import (
 var deployToProfile string
 
 func init() {
-	config.InitConfig()
+	config.Init()
 
 	for k := range config.AvailableServices() {
 		// deploy command

cli/cmd/run.go

@@ -19,7 +19,7 @@ var servicesToRun string
 var versionToRun string
 
 func init() {
-	config.InitConfig()
+	config.Init()
 
 	rootCmd.AddCommand(runCmd)
 

cli/cmd/stop.go

@@ -17,7 +17,7 @@ import (
 var versionToStop string
 
 func init() {
-	config.InitConfig()
+	config.Init()
 
 	rootCmd.AddCommand(stopCmd)
 

cli/cmd/sync.go

@@ -27,7 +27,7 @@ var excludedBlocks = []string{"build"}
 var remote = "elastic:master"
 
 func init() {
-	config.InitConfig()
+	config.Init()
 
 	syncIntegrationsCmd.Flags().BoolVarP(&deleteRepository, "delete", "d", false, "Will delete the existing Beats repository before cloning it again (default false)")
 	syncIntegrationsCmd.Flags().StringVarP(&remote, "remote", "r", "elastic:master", "Sets the remote for Beats, using 'user:branch' as format (i.e. elastic:master)")

cli/config/config.go

@@ -118,6 +118,10 @@ func GetServiceConfig(service string) (Service, bool) {
 
 // Init creates this tool workspace under user's home, in a hidden directory named ".op"
 func Init() {
+	if Op != nil {
+		return
+	}
+
 	configureLogger()
 
 	binaries := []string{
@@ -310,6 +314,12 @@ func extractProfileServiceConfig(op *OpConfig, box *packr.Box) error {
 				return err
 			}
 		}
+
+		log.WithFields(log.Fields{
+			"file": p,
+			"dir":  dir,
+		}).Trace("Extracting boxed file")
+
 		return ioutil.WriteFile(p, []byte(file.String()), 0644)
 	}