Merge pull request #30 from j-fuentes/aks-support

Add AKS data-gatherer

Author: jetstack-bot

cmd/check.go

@@ -11,6 +11,7 @@ import (
 	"time"
 
 	"github.com/jetstack/preflight/pkg/datagatherer"
+	"github.com/jetstack/preflight/pkg/datagatherer/aks"
 	"github.com/jetstack/preflight/pkg/datagatherer/eks"
 	"github.com/jetstack/preflight/pkg/datagatherer/gke"
 	"github.com/jetstack/preflight/pkg/datagatherer/k8s"
@@ -167,6 +168,27 @@ func check() {
 					Location: location,
 					Name:     cluster,
 				}, credentialsPath)
+			} else if name == "aks" {
+				aksConfig, ok := config.(map[string]interface{})
+				if !ok {
+					log.Fatal("Cannot parse 'data-gatherers.aks' in config.")
+				}
+				msg := "'data-gatherers.aks.%s' should be a non empty string."
+				var resourceGroup, clusterName, credentialsPath string
+				if resourceGroup, ok = aksConfig["resource-group"].(string); !ok {
+					log.Fatalf(msg, "resource-group")
+				}
+				if clusterName, ok = aksConfig["cluster"].(string); !ok {
+					log.Fatalf(msg, "cluster")
+				}
+				if credentialsPath, ok = aksConfig["credentials"].(string); !ok {
+					log.Fatalf(msg, "credentials")
+				}
+				var err error
+				dg, err = aks.NewAKSDataGatherer(ctx, resourceGroup, clusterName, credentialsPath)
+				if err != nil {
+					log.Fatalf("Cannot instantiate AKS datagatherer: %v", err)
+				}
 			} else if name == "k8s/pods" {
 				podsConfig, ok := config.(map[string]interface{})
 				if !ok {

examples/aks.preflight.yaml

@@ -0,0 +1,28 @@
+cluster-name: my-cluster
+
+data-gatherers:
+  aks:
+    cluster: preflight-test
+    resource-group: preflight
+    # Generated by running `az account get-access-token > /tmp/credentials.json`
+    credentials: /tmp/credentials.json
+
+package-sources:
+- type: local
+  dir: preflight-packages/
+
+enabled-packages:
+  - "examples.jetstack.io/aks_basic"
+
+# This configures how the results will be reported.
+outputs:
+# writes a human readable doc that can be opened with a browser.
+- type: local
+  path: ./output
+  format: json
+# writes a file with the data gathered.
+- type: local
+  path: ./output
+  format: intermediate
+# writes a the report to the CLI.
+- type: cli

go.mod

@@ -1,9 +1,10 @@
 module github.com/jetstack/preflight
 
-go 1.12
+go 1.13
 
 require (
-	cloud.google.com/go/storage v1.2.1
+	cloud.google.com/go/storage v1.4.0
+	github.com/Azure/aks-engine v0.43.1
 	github.com/aws/aws-sdk-go v1.25.30
 	github.com/gomarkdown/markdown v0.0.0-20191104174740-4d42851d4d5a
 	github.com/gookit/color v1.2.0
@@ -14,19 +15,19 @@ require (
 	github.com/mattn/go-colorable v0.1.4 // indirect
 	github.com/open-policy-agent/opa v0.16.0
 	github.com/pkg/errors v0.8.1
-	github.com/sergi/go-diff v1.0.0 // indirect
 	github.com/spf13/cobra v0.0.5
-	github.com/spf13/pflag v1.0.3
+	github.com/spf13/pflag v1.0.5
 	github.com/spf13/viper v1.5.0
 	github.com/yudai/gojsondiff v1.0.0
-	github.com/yudai/golcs v0.0.0-20170316035057-ecda9a501e82 // indirect
-	github.com/yudai/pp v2.0.1+incompatible // indirect
-	golang.org/x/arch v0.0.0-20191126211547-368ea8f32fff // indirect
-	golang.org/x/oauth2 v0.0.0-20190604053449-0f29369cfe45
-	google.golang.org/api v0.13.0
-	gopkg.in/mgo.v2 v2.0.0-20190816093944-a6b53ec6cb22 // indirect
-	gopkg.in/yaml.v2 v2.2.5
-	k8s.io/api v0.0.0-20190620084959-7cf5895f2711
-	k8s.io/apimachinery v0.0.0-20190612205821-1799e75a0719
-	k8s.io/client-go v0.0.0-20190620085101-78d2af792bab
+	golang.org/x/oauth2 v0.0.0-20191202225959-858c2ad4c8b6
+	google.golang.org/api v0.15.0
+	gopkg.in/yaml.v2 v2.2.7
+	k8s.io/api v0.17.0
+	k8s.io/apimachinery v0.17.0
+	k8s.io/client-go v10.0.0+incompatible
 )
+
+// This is needed because otherwise k8s.io/client-go is forced
+// to v10.0.0+incompatible because of aks-engine, and that
+// causes another set of problems with inconsistent dependencies.
+replace k8s.io/client-go => k8s.io/client-go v0.17.0