cmd/dagger/logger/plain.go

@@ -23,8 +23,6 @@ type PlainOutput struct {
 	Out io.Writer
 }
 
-const systemGroup = "system"
-
 func (c *PlainOutput) Write(p []byte) (int, error) {
 	event := map[string]interface{}{}
 	d := json.NewDecoder(bytes.NewReader(p))
@@ -119,7 +117,7 @@ func formatMessage(event map[string]interface{}) string {
 }
 
 func parseSource(event map[string]interface{}) string {
-	source := systemGroup
+	source := "system"
 	if task, ok := event["task"].(string); ok && task != "" {
 		source = task
 	}

cmd/dagger/logger/tty.go

@@ -43,14 +43,8 @@ func (l *Logs) Add(event Event) error {
 	l.l.Lock()
 	defer l.l.Unlock()
 
-	// same thing as in plain.go group all the non-identified tasks
-	// into a general group called system
-	source := systemGroup
 	taskPath, ok := event["task"].(string)
-
-	if ok && len(taskPath) > 0 {
-		source = taskPath
-	} else if !ok {
+	if !ok {
 		l.Messages = append(l.Messages, Message{
 			Event: event,
 		})
@@ -59,7 +53,7 @@ func (l *Logs) Add(event Event) error {
 	}
 
 	// Hide hidden fields (e.g. `._*`) from log group names
-	groupKey := strings.Split(source, "._")[0]
+	groupKey := strings.Split(taskPath, "._")[0]
 
 	group := l.groups[groupKey]
 
@@ -84,21 +78,13 @@ func (l *Logs) Add(event Event) error {
 	// For each task in a group, the status will transition from computing to complete, then back to computing and so on.
 	// The transition is fast enough not to cause a problem.
 	if st, ok := event["state"].(string); ok {
-		if t, err := task.ParseState(st); err != nil {
-			return err
-			// concurrent "system" tasks are the only exception to transition
-			// from another state to failed since we need to show the error to
-			// the user
-		} else if group.State.CanTransition(t) ||
-			(group.Name == systemGroup && t == task.StateFailed) {
-			group.State = t
+		group.State = task.State(st)
 		if group.State == task.StateComputing {
 			group.Completed = nil
 		} else {
 			now := time.Now()
 			group.Completed = &now
 		}
-		}
 
 		return nil
 	}
@@ -276,10 +262,6 @@ func (c *TTYOutput) printLine(w io.Writer, event Event, width int) int {
 func (c *TTYOutput) printGroup(group *Group, width, maxLines int) int {
 	lineCount := 0
 
-	var out string
-	// treat the "system" group as a special case as we don't
-	// want it to be displayed as an action in the output
-	if group.Name != systemGroup {
 	prefix := ""
 	switch group.State {
 	case task.StateComputing:
@@ -292,7 +274,7 @@ func (c *TTYOutput) printGroup(group *Group, width, maxLines int) int {
 		prefix = "[✔]"
 	}
 
-		out = prefix + " " + group.Name
+	out := prefix + " " + group.Name
 
 	endTime := time.Now()
 	if group.Completed != nil {
@@ -325,7 +307,6 @@ func (c *TTYOutput) printGroup(group *Group, width, maxLines int) int {
 	// Print
 	fmt.Fprint(c.cons, out)
 	lineCount++
-	}
 
 	printEvents := []Event{}
 	switch group.State {

docs/faq/1232-chain-actions.md

@@ -42,7 +42,7 @@ bar: bash.#Run & {
     script: contents: #"""
         echo "hello"
         """#
-    env: HACK: "\(foo.success)" // <== HACK: CHAINING of action happening here
+    env: HACK: "\(test.success)" // <== HACK: CHAINING of action happening here
 }
 ```
 

pkg/universe.dagger.io/x/vgjm456@qq.com/helm/image.cue

@@ -1,13 +0,0 @@
-package helm
-
-import (
-	"universe.dagger.io/docker"
-)
-
-#Image: {
-	version: string | *"latest"
-
-	docker.#Pull & {
-		source: "index.docker.io/alpine/helm:\(version)"
-	}
-}

pkg/universe.dagger.io/x/vgjm456@qq.com/helm/install.cue

@@ -1,64 +0,0 @@
-package helm
-
-import (
-	"dagger.io/dagger"
-	"universe.dagger.io/docker"
-)
-
-#Install: {
-	// Name of your release
-	name:       string | *""
-	kubeconfig: dagger.#Secret
-	source:     *"repository" | "URL"
-	{
-		source:     "repository"
-		chart:      string
-		repoName:   string
-		repository: string
-		run: {
-			env: {
-				CHART:      chart
-				REPO_NAME:  repoName
-				REPOSITORY: repository
-			}
-			_script: #"""
-				helm repo add $REPO_NAME $REPOSITORY
-				helm repo update
-				helm install $NAME $REPO_NAME/$CHART $GENERATE_NAME
-				"""#
-		}
-	} | {
-		source: "URL"
-		URL:    string
-		run: {
-			env: "URL": URL
-			_script: #"""
-				helm install $NAME $URL $GENERATE_NAME
-				"""#
-		}
-	}
-
-	_base: #Image
-	run:   docker.#Run & {
-		input: _base.output
-		env: {
-			NAME:          name
-			GENERATE_NAME: _generateName
-		}
-		mounts: "/root/.kube/config": {
-			dest:     "/root/.kube/config"
-			type:     "secret"
-			contents: kubeconfig
-		}
-		entrypoint: ["/bin/sh"]
-		command: {
-			name: "-c"
-			args: [run._script]
-		}
-	}
-
-	_generateName: string | *""
-	if name == "" {
-		_generateName: "--generate-name"
-	}
-}

pkg/universe.dagger.io/x/vgjm456@qq.com/helm/test/test.cue

@@ -1,33 +0,0 @@
-package helm
-
-import (
-	"dagger.io/dagger"
-	"universe.dagger.io/x/vgjm456@qq.com/helm"
-)
-
-dagger.#Plan & {
-	client: {
-		env: KUBECONFIG: string
-		commands: kubeconfig: {
-			name: "cat"
-			args: ["\(env.KUBECONFIG)"]
-			stdout: dagger.#Secret
-		}
-	}
-	actions: test: {
-		URL: helm.#Install & {
-			name:       "test-pgsql"
-			source:     "URL"
-			URL:        "https://charts.bitnami.com/bitnami/postgresql-11.1.12.tgz"
-			kubeconfig: client.commands.kubeconfig.stdout
-		}
-		repository: helm.#Install & {
-			name:       "test-redis"
-			source:     "repository"
-			chart:      "redis"
-			repoName:   "bitnami"
-			repository: "https://charts.bitnami.com/bitnami"
-			kubeconfig: client.commands.kubeconfig.stdout
-		}
-	}
-}

plan/runner.go

@@ -8,7 +8,6 @@ import (
 	"time"
 
 	"go.dagger.io/dagger/compiler"
-
 	"go.dagger.io/dagger/plan/task"
 	"go.dagger.io/dagger/plancontext"
 	"go.dagger.io/dagger/solver"
@@ -16,7 +15,6 @@ import (
 	"cuelang.org/go/cue"
 	cueflow "cuelang.org/go/tools/flow"
 
-	"github.com/rs/zerolog"
 	"github.com/rs/zerolog/log"
 	"go.opentelemetry.io/otel"
 )
@@ -126,15 +124,6 @@ func (r *Runner) shouldRun(p cue.Path) bool {
 	return ok
 }
 
-func taskLog(tp string, log *zerolog.Logger, t task.Task, fn func(lg zerolog.Logger)) {
-	fn(log.With().Str("task", tp).Logger())
-	// setup logger here
-	_, isDockerfileTask := t.(*task.DockerfileTask)
-	if isDockerfileTask {
-		fn(log.With().Str("task", "system").Logger())
-	}
-}
-
 func (r *Runner) taskFunc(flowVal cue.Value) (cueflow.Runner, error) {
 	v := compiler.Wrap(flowVal)
 	handler, err := task.Lookup(v)
@@ -153,45 +142,31 @@ func (r *Runner) taskFunc(flowVal cue.Value) (cueflow.Runner, error) {
 	// Wrapper around `task.Run` that handles logging, tracing, etc.
 	return cueflow.RunnerFunc(func(t *cueflow.Task) error {
 		ctx := t.Context()
-		taskPath := t.Path().String()
-		lg := log.Ctx(ctx).With().Logger()
+		lg := log.Ctx(ctx).With().Str("task", t.Path().String()).Logger()
 		ctx = lg.WithContext(ctx)
 		ctx, span := otel.Tracer("dagger").Start(ctx, fmt.Sprintf("up: %s", t.Path().String()))
 		defer span.End()
 
-		taskLog(taskPath, &lg, handler, func(lg zerolog.Logger) {
-			lg.Info().Str("state", task.StateComputing.String()).Msg(task.StateComputing.String())
-		})
+		lg.Info().Str("state", string(task.StateComputing)).Msg(string(task.StateComputing))
 
 		// Debug: dump dependencies
 		for _, dep := range t.Dependencies() {
-			taskLog(taskPath, &lg, handler, func(lg zerolog.Logger) {
 			lg.Debug().Str("dependency", dep.Path().String()).Msg("dependency detected")
-			})
 		}
 
 		start := time.Now()
 		result, err := handler.Run(ctx, r.pctx, r.s, compiler.Wrap(t.Value()))
 		if err != nil {
 			// FIXME: this should use errdefs.IsCanceled(err)
-
-			// we don't wrap taskLog here since in some cases, actions could still be
-			// running in goroutines which will scramble outputs.
 			if strings.Contains(err.Error(), "context canceled") {
-				taskLog(taskPath, &lg, handler, func(lg zerolog.Logger) {
-					lg.Error().Dur("duration", time.Since(start)).Str("state", task.StateCanceled.String()).Msg(task.StateCanceled.String())
-				})
+				lg.Error().Dur("duration", time.Since(start)).Str("state", string(task.StateCanceled)).Msg(string(task.StateCanceled))
 			} else {
-				taskLog(taskPath, &lg, handler, func(lg zerolog.Logger) {
-					lg.Error().Dur("duration", time.Since(start)).Err(compiler.Err(err)).Str("state", task.StateFailed.String()).Msg(task.StateFailed.String())
-				})
+				lg.Error().Dur("duration", time.Since(start)).Err(compiler.Err(err)).Str("state", string(task.StateFailed)).Msg(string(task.StateFailed))
 			}
 			return fmt.Errorf("%s: %w", t.Path().String(), compiler.Err(err))
 		}
 
-		taskLog(taskPath, &lg, handler, func(lg zerolog.Logger) {
-			lg.Info().Dur("duration", time.Since(start)).Str("state", task.StateCompleted.String()).Msg(task.StateCompleted.String())
-		})
+		lg.Info().Dur("duration", time.Since(start)).Str("state", string(task.StateCompleted)).Msg(string(task.StateCompleted))
 
 		// If the result is not concrete (e.g. empty value), there's nothing to merge.
 		if !result.IsConcrete() {
@@ -199,9 +174,7 @@ func (r *Runner) taskFunc(flowVal cue.Value) (cueflow.Runner, error) {
 		}
 
 		if src, err := result.Source(); err == nil {
-			taskLog(taskPath, &lg, handler, func(lg zerolog.Logger) {
 			lg.Debug().Str("result", string(src)).Msg("merging task result")
-			})
 		}
 
 		// Mirror task result and re-scan tasks that should run.
@@ -211,9 +184,7 @@ func (r *Runner) taskFunc(flowVal cue.Value) (cueflow.Runner, error) {
 		// }
 
 		if err := t.Fill(result.Cue()); err != nil {
-			taskLog(taskPath, &lg, handler, func(lg zerolog.Logger) {
 			lg.Error().Err(err).Msg("failed to fill task")
-			})
 			return err
 		}
 

plan/task/dockerfile.go

@@ -22,12 +22,13 @@ import (
 )
 
 func init() {
-	Register("Dockerfile", func() Task { return &DockerfileTask{} })
+	Register("Dockerfile", func() Task { return &dockerfileTask{} })
 }
 
-type DockerfileTask struct{}
+type dockerfileTask struct {
+}
 
-func (t *DockerfileTask) Run(ctx context.Context, pctx *plancontext.Context, s *solver.Solver, v *compiler.Value) (*compiler.Value, error) {
+func (t *dockerfileTask) Run(ctx context.Context, pctx *plancontext.Context, s *solver.Solver, v *compiler.Value) (*compiler.Value, error) {
 	lg := log.Ctx(ctx)
 	auths, err := v.Lookup("auth").Fields()
 	if err != nil {
@@ -75,7 +76,7 @@ func (t *DockerfileTask) Run(ctx context.Context, pctx *plancontext.Context, s *
 		}
 		dockerfileDef, err = s.Marshal(ctx,
 			llb.Scratch().File(
-				llb.Mkfile("/Dockerfile", 0o644, []byte(contents)),
+				llb.Mkfile("/Dockerfile", 0644, []byte(contents)),
 			),
 		)
 		if err != nil {
@@ -138,7 +139,7 @@ func (t *DockerfileTask) Run(ctx context.Context, pctx *plancontext.Context, s *
 	})
 }
 
-func (t *DockerfileTask) dockerBuildOpts(v *compiler.Value, pctx *plancontext.Context) (map[string]string, error) {
+func (t *dockerfileTask) dockerBuildOpts(v *compiler.Value, pctx *plancontext.Context) (map[string]string, error) {
 	opts := map[string]string{}
 
 	if dockerfilePath := v.Lookup("dockerfile.path"); dockerfilePath.Exists() {

plan/task/task.go

@@ -28,36 +28,13 @@ var (
 )
 
 // State is the state of the task.
-type State int8
-
-func (s State) String() string {
-	return [...]string{"computing", "cancelled", "failed", "completed"}[s]
-}
-
-func ParseState(s string) (State, error) {
-	switch s {
-	case "computing":
-		return StateComputing, nil
-	case "cancelled":
-		return StateCanceled, nil
-	case "failed":
-		return StateFailed, nil
-	case "completed":
-		return StateCompleted, nil
-	}
-
-	return -1, fmt.Errorf("invalid state [%s]", s)
-}
-
-func (s State) CanTransition(t State) bool {
-	return s == StateComputing && s <= t
-}
+type State string
 
 const (
-	StateComputing State = iota
-	StateCanceled
-	StateFailed
-	StateCompleted
+	StateComputing = State("computing")
+	StateCanceled  = State("canceled")
+	StateFailed    = State("failed")
+	StateCompleted = State("completed")
 )
 
 type NewFunc func() Task

plancontext/fs.go

@@ -12,11 +12,13 @@ import (
 	"go.dagger.io/dagger/pkg"
 )
 
-var fsIDPath = cue.MakePath(
+var (
+	fsIDPath = cue.MakePath(
 		cue.Str("$dagger"),
 		cue.Str("fs"),
 		cue.Hid("_id", pkg.DaggerPackage),
 	)
+)
 
 func IsFSValue(v *compiler.Value) bool {
 	return v.LookupPath(fsIDPath).Exists()

util/progressui/display.go

@@ -25,11 +25,9 @@ const (
 	defaultDisplayTimeout = 100 * time.Millisecond
 )
 
-type (
-	VertexPrintFunc func(v *client.Vertex, index int)
-	StatusPrintFunc func(v *client.Vertex, format string, a ...interface{})
-	LogPrintFunc    func(v *client.Vertex, stream int, partial bool, format string, a ...interface{})
-)
+type VertexPrintFunc func(v *client.Vertex, index int)
+type StatusPrintFunc func(v *client.Vertex, format string, a ...interface{})
+type LogPrintFunc func(v *client.Vertex, stream int, partial bool, format string, a ...interface{})
 
 func PrintSolveStatus(ctx context.Context, ch chan *client.SolveStatus, vertexPrintCb VertexPrintFunc, statusPrintCb StatusPrintFunc, logPrintCb LogPrintFunc) error {
 	printer := &textMux{
@@ -150,10 +148,8 @@ func DisplaySolveStatus(ctx context.Context, phase string, c console.Console, w
 	}
 }
 
-const (
-	termHeight = 6
-	termPad    = 10
-)
+const termHeight = 6
+const termPad = 10
 
 type displayInfo struct {
 	startTime      time.Time

util/progressui/printer.go

@@ -11,12 +11,10 @@ import (
 	"github.com/tonistiigi/units"
 )
 
-const (
-	antiFlicker      = 5 * time.Second
-	maxDelay         = 10 * time.Second
-	minTimeDelta     = 5 * time.Second
-	minProgressDelta = 0.05 // %
-)
+const antiFlicker = 5 * time.Second
+const maxDelay = 10 * time.Second
+const minTimeDelta = 5 * time.Second
+const minProgressDelta = 0.05 // %
 
 type lastStatus struct {
 	Current   int64

website/yarn.lock

@@ -2842,9 +2842,9 @@ astral-regex@^2.0.0:
   integrity sha512-Z7tMw1ytTXt5jqMcOP+OQteU1VuNK9Y02uuJtKQ1Sv69jXQKKg5cibLwGJow8yzZP+eAc18EmLGPal0bp36rvQ==
 
 async@^2.6.2:
-  version "2.6.4"
-  resolved "https://registry.yarnpkg.com/async/-/async-2.6.4.tgz#706b7ff6084664cd7eae713f6f965433b5504221"
-  integrity sha512-mzo5dfJYwAn29PeiJ0zvwTo04zj8HDJj0Mn8TD7sno7q12prdbnasKJHhkm2c1LgrhlJ0teaea8860oxi51mGA==
+  version "2.6.3"
+  resolved "https://registry.yarnpkg.com/async/-/async-2.6.3.tgz#d72625e2344a3656e3a3ad4fa749fa83299d82ff"
+  integrity sha512-zflvls11DCy+dQWzTW2dzuilv8Z5X/pjfmZOWba6TNIVDm+2UDaJmXSOXlasHKfNBs8oo3M0aT50fDEWfKZjXg==
   dependencies:
     lodash "^4.17.14"