Merge pull request #199 from dagger/fix-log-flags

fix --log-* flags
This commit is contained in:
Andrea Luzzardi 2021-03-18 16:02:24 -07:00 committed by GitHub
commit fee4431d48
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 6 deletions

View File

@ -3,9 +3,9 @@ package cmd
import (
"fmt"
"dagger.io/go/cmd/dagger/logger"
"dagger.io/go/dagger"
"github.com/rs/zerolog/log"
"github.com/spf13/cobra"
"github.com/spf13/viper"
)
@ -27,8 +27,8 @@ var computeCmd = &cobra.Command{
}
},
Run: func(cmd *cobra.Command, args []string) {
ctx := cmd.Context()
lg := log.Ctx(ctx)
lg := logger.New()
ctx := lg.WithContext(cmd.Context())
env, err := dagger.NewEnv()
if err != nil {

View File

@ -44,9 +44,10 @@ func init() {
func Execute() {
var (
lg = logger.New()
ctx = lg.WithContext(appcontext.Context())
ctx = appcontext.Context()
// `--log-*` flags have not been parsed yet at this point so we get a
// default logger. Therefore, we can't store the logger into the context.
lg = logger.New()
closer = logger.InitTracing()
span opentracing.Span
)