Fixes Readme, adds dataplane spec to charts (#46)

* fix Readme and adds dataplane to helm charts

Author: prateekgogia

operator/README.md

@@ -41,7 +41,7 @@ KIT uses the [operator pattern](https://kubernetes.io/docs/concepts/extend-kuber
     --name kit-controller \
     --namespace kit \
     --cluster ${SUBSTRATE_CLUSTER_NAME} \
-    --attach-policy-arn arn:aws:iam::${AWS_ACCOUNT_ID}:policy/KitControllerPolicy \
+    --attach-policy-arn arn:aws:iam::${AWS_ACCOUNT_ID}:policy/KitControllerPolicy-${SUBSTRATE_CLUSTER_NAME}-cluster \
     --approve \
     --override-existing-serviceaccounts \
     --region=${AWS_REGION}
@@ -50,7 +50,8 @@ KIT uses the [operator pattern](https://kubernetes.io/docs/concepts/extend-kuber
 #### Install KIT operator to the cluster
 
 ```bash
-   helm upgrade --install kit-operator -n kit charts/kit-operator
+   helm repo add kit https://awslabs.github.io/kubernetes-iteration-toolkit/
+   helm upgrade --install kit-operator kit/kit-operator --namespace kit --create-namespace --version 0.0.1
 ```
 
 Once KIT operator is deployed in a Kubernetes cluster. You can create a new Kubernetes control plane and worker nodes by following these steps in any namespace in the substrate cluster
@@ -105,6 +106,6 @@ Once KIT operator is deployed in a Kubernetes cluster. You can create a new Kube
 ```bash
   eksctl delete iamserviceaccount --name kit-controller \
     --namespace kit \
-    --cluster ${CLUSTER_NAME} \
+    --cluster ${SUBSTRATE_CLUSTER_NAME} \
     --region=$AWS_REGION
 ```

operator/charts/kit-operator/templates/controller/rbac.yaml

@@ -80,6 +80,7 @@ rules:
   resources:
   - statefulsets
   - deployments
+  - daemonsets
   verbs:
   - get
   - create

operator/charts/kit-operator/templates/data-plane-crd.yaml

@@ -0,0 +1,93 @@
+
+---
+apiVersion: apiextensions.k8s.io/v1
+kind: CustomResourceDefinition
+metadata:
+  annotations:
+    controller-gen.kubebuilder.io/version: v0.4.1
+  creationTimestamp: null
+  name: dataplanes.kit.k8s.sh
+spec:
+  group: kit.k8s.sh
+  names:
+    kind: DataPlane
+    listKind: DataPlaneList
+    plural: dataplanes
+    singular: dataplane
+  scope: Namespaced
+  versions:
+  - name: v1alpha1
+    schema:
+      openAPIV3Schema:
+        description: DataPlane is the Schema for the DataPlanes API
+        properties:
+          apiVersion:
+            description: 'APIVersion defines the versioned schema of this representation
+              of an object. Servers should convert recognized schemas to the latest
+              internal value, and may reject unrecognized values. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#resources'
+            type: string
+          kind:
+            description: 'Kind is a string value representing the REST resource this
+              object represents. Servers may infer this from the endpoint the client
+              submits requests to. Cannot be updated. In CamelCase. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#types-kinds'
+            type: string
+          metadata:
+            type: object
+          spec:
+            properties:
+              clusterName:
+                type: string
+              nodeCount:
+                type: integer
+            type: object
+          status:
+            properties:
+              conditions:
+                description: Conditions is the set of conditions required for this
+                  DataPlane to create its objects, and indicates whether or not those
+                  conditions are met.
+                items:
+                  description: 'Condition defines a readiness condition for a Knative
+                    resource. See: https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/api-conventions.md#typical-status-properties'
+                  properties:
+                    lastTransitionTime:
+                      description: LastTransitionTime is the last time the condition
+                        transitioned from one status to another. We use VolatileTime
+                        in place of metav1.Time to exclude this from creating equality.Semantic
+                        differences (all other things held constant).
+                      format: date-time
+                      type: string
+                    message:
+                      description: A human readable message indicating details about
+                        the transition.
+                      type: string
+                    reason:
+                      description: The reason for the condition's last transition.
+                      type: string
+                    severity:
+                      description: Severity with which to treat failures of this type
+                        of condition. When this is not specified, it defaults to Error.
+                      type: string
+                    status:
+                      description: Status of the condition, one of True, False, Unknown.
+                      type: string
+                    type:
+                      description: Type of condition.
+                      type: string
+                  required:
+                  - status
+                  - type
+                  type: object
+                type: array
+            type: object
+        type: object
+    served: true
+    storage: true
+    subresources:
+      status: {}
+status:
+  acceptedNames:
+    kind: ""
+    plural: ""
+  conditions: []
+  storedVersions: []

operator/docs/kit.cloudformation.yaml

@@ -34,7 +34,7 @@ Resources:
   KitControllerPolicy:
     Type: "AWS::IAM::ManagedPolicy"
     Properties:
-      ManagedPolicyName: "KitControllerPolicy"
+      ManagedPolicyName: !Sub "KitControllerPolicy-${ClusterName}-cluster"
       PolicyDocument:
         Version: "2012-10-17"
         Statement: