Merge pull request #512 from chris-crone/readme-20190417

Rework examples and top level README

Author: chris-crone

README.md

@@ -3,7 +3,6 @@ package e2e
 import (
 	"os"
 	"path/filepath"
-	"strings"
 	"testing"
 
 	"gotest.tools/assert"
@@ -15,22 +14,12 @@ func TestExamplesAreValid(t *testing.T) {
 	defer cleanup()
 
 	err := filepath.Walk("../examples", func(p string, info os.FileInfo, err error) error {
-		appPath := filepath.Join(p, filepath.Base(p)+".dockerapp")
-		_, statErr := os.Stat(appPath)
-		switch {
-		case strings.HasSuffix(p, "examples"):
-			return nil
-		case strings.HasSuffix(p, ".resources"):
-			return filepath.SkipDir
-		case !info.IsDir():
-			return nil
-		case os.IsNotExist(statErr):
-			return nil
-		default:
-			cmd.Command = dockerCli.Command("app", "validate", appPath)
+		if filepath.Ext(p) == ".dockerapp" {
+			t.Log("Validate example: " + p)
+			cmd.Command = dockerCli.Command("app", "validate", p)
 			icmd.RunCmd(cmd).Assert(t, icmd.Success)
-			return filepath.SkipDir
 		}
+		return nil
 	})
 	assert.NilError(t, err)
 }

e2e/example_test.go

@@ -1,29 +1,24 @@
 ## Examples
 
-This folder contains several examples of applications built using `docker-app`, from a simple `hello-world` application to more complex, multi-service applications.
+This folder contains several examples of applications built using Docker
+Application, from a simple `hello-world` application to more complex,
+multi-service applications.
 
-### Hello world application: [hello world](hello-world)
+### [Hello world!](hello-world)
 
-To learn how to create a simple app that displays a given text.
+Learn how the basics of how to write a simple Docker Application.
 
-![Hello world app screenshot](.resources/hello-world.png "Hello world app screenshot")
+### [Convert an existing Compose file](voting-app)
 
-### Voting application: [voting-app](voting-app)
+Learn how to create a Docker Application from an existing Compose file using the
+Docker voting app from [Docker samples](https://github.com/dockersamples).
 
-To learn how to create the Docker voting app from [Docker samples](https://github.com/dockersamples).
+### [Docker Application CNAB generation](cnab-simple)
 
-![Voting app screenshot](.resources/cats-or-dogs.png "Voting app screenshot")
+Create a [CNAB](https://cnab.io) `bundle.json` and invocation image from a
+Docker Application.
 
-### WordPress application: [WordPress](wordpress)
+### [Deploy a Helm Chart CNAB using Docker App](cnab-helm)
 
-To learn how to create a basic WordPress with a database.
-
-![WordPress app screenshot](.resources/wordpress.png "WordPress app screenshot")
-
-### [CNAB simple application](cnab-simple)
-
-Inspect and bundle a simple application as a [CNAB](https://cnab.io).
-
-### [Deploy a Helm Chart using docker-app](cnab-helm)
-
-As `docker-app` is a [CNAB](https://cnab.io) compliant client, you can deploy any CNAB.
+As Docker App is a [CNAB](https://cnab.io) compliant client, you can deploy any
+CNAB.

examples/.resources/cats-or-dogs.png

@@ -0,0 +1,21 @@
+    MIT License
+
+    Copyright (c) Microsoft Corporation. All rights reserved.
+
+    Permission is hereby granted, free of charge, to any person obtaining a copy
+    of this software and associated documentation files (the "Software"), to deal
+    in the Software without restriction, including without limitation the rights
+    to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+    copies of the Software, and to permit persons to whom the Software is
+    furnished to do so, subject to the following conditions:
+
+    The above copyright notice and this permission notice shall be included in all
+    copies or substantial portions of the Software.
+
+    THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+    IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+    FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+    AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+    LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+    OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+    SOFTWARE

examples/.resources/hello-world.png

@@ -1,21 +1,21 @@
-## Requirements
+## Using a CNAB with Docker Application
 
-* Any with Kubernetes enabled with [compose-on-kubernetes](https://github.com/docker/compose-on-kubernetes) installed
-* [`docker-app` with CNAB support](https://github.com/docker/app/releases/tag/cnab-dockercon-preview) installed
+### Requirements
+
+* [Docker Desktop](https://www.docker.com/products/docker-desktop) with Kubernetes enabled or any other Kubernetes cluster
 * Source code from this directory
-* Create a context with `docker-app context`
-* Set the `DOCKER_TARGET_CONTEXT` environment variable
-* Helm configured for your Kubernetes cluster
-* A `duffle` credential set created
+* [Helm](https://helm.sh) configured for your Kubernetes cluster
+* A `duffle` [credential set](https://github.com/deislabs/duffle/blob/1847f2f7127f13f62c1377f936cba522e8947dfb/docs/proposal/201-credentialset.md) created
 
-## Examples
+### Examples
 
-Install the Helm chart example using `docker-app`
+Install the Helm chart example using `docker app`
 
-**Note**: This example comes from [deislabs/bundles](https://github.com/deislabs/bundles/tree/master/hellohelm).
+**Note**: This example comes from
+[deislabs/example-bundles](https://github.com/deislabs/example-bundles/tree/d1d95e25a2092ac170d9accd749dffa8babb2e05/hellohelm). See the [license file](./LICENSE) in this directory.
 
 ```console
-$ docker-app install -c local bundle.json
+$ docker app install --credential-set mycreds.yml bundle.json
 Do install for hellohelm
 helm install --namespace hellohelm -n hellohelm /cnab/app/charts/alpine
 NAME:   hellohelm
@@ -32,7 +32,7 @@ hellohelm-alpine  0s
 Check the status of the Helm-based application:
 
 ```console
-$ docker-app status -c local hellohelm
+$ docker app status --credential-set mycreds.yml hellohelm
 Do Status
 helm status hellohelm
 LAST DEPLOYED: Wed Nov 28 13:58:22 2018
@@ -48,7 +48,7 @@ hellohelm-alpine  2m
 Uninstall the Helm-based application:
 
 ```console
-docker-app uninstall -c local hellohelm
+docker app uninstall --credential-set mycreds.yml hellohelm
 Do Uninstall
 helm delete --purge hellohelm
 release "hellohelm" deleted

examples/.resources/wordpress.png

@@ -1,14 +1,15 @@
 {
     "name": "hellohelm",
     "version": "0.1.0",
+    "schemaVersion": "v1.0.0-WD",
     "invocationImages": [
         {
         "imageType": "docker",
         "image": "cnab/hellohelm:latest"
         }
     ],
-    "images": [
-        {
+    "images": {
+        "alpine": {
             "description": "alpine",
             "image": "technosophos/demo2alpine:0.1.0",
             "imageType": "docker",
@@ -19,16 +20,24 @@
                 }
             ]
         }
-    ],
+    },
     "parameters": {
         "port": {
             "defaultValue": 8080,
-            "type": "int"
+            "type": "int",
+            "destination": {
+                "env": "PORT"
+            }
         }
     },
     "credentials": {
         "kubeconfig": {
             "path": "/root/.kube/config"
         }
+    },
+    "actions": {
+        "status": {
+            "modifies": false
+        }
     }
 }

examples/README.md

@@ -1,4 +1,4 @@
 FROM cnab/k8sbase:latest
 
 COPY app/Makefile /cnab/app/Makefile
-COPY app/charts /cnab/app/charts
+COPY app/charts /cnab/app/charts