updated comments and readme

kyaml/openapi/Makefile

@@ -17,6 +17,8 @@ clean:
 
 # This will remove all currently built-in schema,
 # so think twice before deleting.
+# To replace what this will delete typically requires the ability
+# to contact a live kubernetes API server.
 .PHONY: nuke
 nuke: clean
 	rm -r kubernetesapi/*

kyaml/openapi/README.md

@@ -1,11 +1,10 @@
 # Sampling New OpenAPI Data
 
 [OpenAPI schema]: ./kubernetesapi/
-[kind clusters]: https://hub.docker.com/r/kindest/node/tags
+[kind]: https://hub.docker.com/r/kindest/node/tags
 
-This document describes how to fetch OpenAPI data from
-a particular kubernetes version number. Note that you can
-only fetch data from versions supported by [kind clusters].
+This document describes how to fetch OpenAPI data from a
+live kubernetes API server, e.g. an instance of [kind].
 
 ### Delete all currently built-in schema
 ```

kyaml/openapi/openapi.go

@@ -7,6 +7,7 @@ import (
 	"encoding/json"
 	"fmt"
 	"io/ioutil"
+	"path/filepath"
 	"reflect"
 	"strings"
 	"sync"
@@ -436,7 +437,10 @@ func initSchema() {
 		}
 
 		// parse the swagger, this should never fail
-		assetName := "kubernetesapi/" + kubernetesAPIDefaultVersion + "/swagger.json"
+		assetName := filepath.Join(
+			"kubernetesapi",
+			kubernetesAPIDefaultVersion,
+			"swagger.json")
 		if err := parse(kubernetesapi.OpenApiMustAsset[kubernetesAPIDefaultVersion](assetName)); err != nil {
 			// this should never happen
 			panic(err)

kyaml/openapi/scripts/fetchSchemaFromCluster.sh

@@ -11,4 +11,4 @@ $MYGOBIN/kpt live  fetch-k8s-schema  --pretty-print > /tmp/new_swagger.json
 $MYGOBIN/kind delete cluster --name=getopenapidata
 cp /tmp/kubeconfig.txt $HOME/.kube/config | true
 mkdir -p kubernetesapi/"${VERSION//.}"
-cp /tmp/new_swagger.json kubernetesapi/"${VERSION//.}"/swagger.json
+cp /tmp/new_swagger.json kubernetesapi/"${VERSION//.}"/swagger.json

kyaml/openapi/scripts/generateSwaggerDotGo.sh

@@ -8,4 +8,4 @@ VERSION=$1
 $MYGOBIN/go-bindata \
   --pkg "${VERSION//.}" \
   -o kubernetesapi/"${VERSION//.}"/swagger.go \
-  kubernetesapi/"${VERSION//.}"/swagger.json
+  kubernetesapi/"${VERSION//.}"/swagger.json