docker / app

Make your Docker Compose applications reusable, and share them on Docker Hub
Apache License 2.0
1.57k stars 177 forks source link

Add List command to list all the installations #510

Closed silvin-lubecki closed 5 years ago

silvin-lubecki commented 5 years ago

- What I did Add a new top level command list/ls which lists all the persisted installations from the local installation store, and display all the informations (result, creation and modifications dates, last know action).

- How to verify it

$ docker app install examples/hello-world/hello-world.dockerapp
Creating network hello-world_default
Creating service hello-world_hello
$ docker app ls
INSTALLATION ACTION  RESULT  CREATED   MODIFIED
hello-world  install success 6 seconds 2 seconds

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

image

codecov[bot] commented 5 years ago

Codecov Report

Merging #510 into master will increase coverage by 0.31%. The diff coverage is 83.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #510      +/-   ##
==========================================
+ Coverage   69.05%   69.37%   +0.31%     
==========================================
  Files          54       54              
  Lines        2870     2890      +20     
==========================================
+ Hits         1982     2005      +23     
+ Misses        618      616       -2     
+ Partials      270      269       -1
Impacted Files Coverage Δ
internal/commands/root.go 88.23% <100%> (+0.17%) :arrow_up:
internal/commands/list.go 82.97% <82.97%> (ø)
internal/store/app.go 73.33% <0%> (-0.87%) :arrow_down:
internal/commands/cnab.go 69.72% <0%> (-0.49%) :arrow_down:
internal/commands/install.go 63.07% <0%> (-0.31%) :arrow_down:
internal/store/installation.go
internal/commands/upgrade.go 61.11% <0%> (+3.48%) :arrow_up:
internal/commands/uninstall.go 62.5% <0%> (+5.35%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 7d43c90...3508b0e. Read the comment docs.

silvin-lubecki commented 5 years ago

@simonferquel PTAL

simonferquel commented 5 years ago

App bundle reference and templating will be done in a followup. Meging.

simonferquel commented 5 years ago

Ah, no please format the app name with something that does not look like a reference

chris-crone commented 5 years ago

Superseded by #517