Add "--service-images" to pull command #791
Conversation
doanac
force-pushed
the
789-docker-pull-containers
branch
from
9b596e9 to
445fe77
Compare
internal/commands/pull.go
Outdated
| var opts pullOptions | ||
|
|
||
| cmd := &cobra.Command{ | ||
| Use: "pull APP_IMAGE", |
There was a problem hiding this comment.
We should also add [OPTIONS] here:
Use: "pull [OPTIONS] APP_IMAGE",There was a problem hiding this comment.
I've force pushed a fix for this.
| } | ||
| defer responseBody.Close() | ||
|
|
||
| return jsonmessage.DisplayJSONMessagesStream(responseBody, cli.Out(), cli.Out().FD(), false, nil) |
There was a problem hiding this comment.
As those multiple pulls may be really verbose, should we also add a --quiet flag to this command? WDYT @thaJeztah ?
|
|
||
| if opts.serviceImages { | ||
| ctx := context.Background() | ||
| for name, image := range bndl.Images { |
There was a problem hiding this comment.
I wonder if it makes sense to parallelize pulls here 🤔
There was a problem hiding this comment.
Let me know what you all think. I will warn you that I know just enough about Golang that I might be dangerous doing this bit :)
|
Thank you @doanac for this PR! |
This allows users in a single node configuration to optionally pull down the service images defined in an app bundle with: docker app pull --service-images [your app] Fixes: fixes docker-archive-public#789 Signed-off-by: Andy Doan <andy@foundries.io>
doanac
force-pushed
the
789-docker-pull-containers
branch
from
445fe77 to
42f1daa
Compare
Codecov Report
@@ Coverage Diff @@
## master #791 +/- ##
==========================================
- Coverage 70.03% 69.97% -0.06%
==========================================
Files 67 67
Lines 3684 3710 +26
==========================================
+ Hits 2580 2596 +16
- Misses 759 764 +5
- Partials 345 350 +5
Continue to review full report at Codecov.
|
3 participants
This allows users in a single node configuration to optionally pull
down the service images defined in an app bundle with:
docker app pull --service-images [your app]
Fixes: fixes #789
Signed-off-by: Andy Doan andy@foundries.io