Support for partially running a DAG#1302
Support for partially running a DAG#1302aluzzardi merged 1 commit intodagger:mainfrom aluzzardi:up-target
Conversation
|
+1 on merging this. It solves the number one blocker of the CI spec: #1230 |
|
I like the simplicity of this. Do we need some tests to verify the behavior? |
|
See #1396 . This is only part of the solution. It’s very important that before merging, we get on the same page on when to use this feature and when not to use it. |
shykes
left a comment
There was a problem hiding this comment.
See #1396 . I agree with this feature BUT only for troubleshooting purposes (same as terraform apply —target). It’s very important that we get on the same page before merging, on when to use this feature and when not to use it.
cmd/dagger/cmd/up.go
Outdated
There was a problem hiding this comment.
Could you add “(for debugging only)” to the end of that description please? So that we have at least one small trace of the usage pattern we recommend.
There was a problem hiding this comment.
Otherwise I’ll do it as a follow-up PR, that’s fine too. I don’t want to hold up merging.
Signed-off-by: Andrea Luzzardi <[email protected]>
Based of @samalba's proposal for CI #1230
-t/--target(placeholder name) flag todagger upExample:
dagger up -t actions.imagewill only runactions.imagedagger up -t actions.execwill runactions.imageand thenactions.exec(becauseactions.imageis necessary to runactions.exec)