fix: New Compute and Network micro services convering machine, cluster to pure ARM calls (#139)

fix: New Compute and Network micro services convering machine, cluster to pure ARM calls

fix: New Network services which can be used by top level reconciler

Add Virtual Machine and Virtual Machine Extensions as well, unforunately this is a massive PR

Atleast create single node clusters

Add Azure Cloud provider config to all masters and client

Use Internal LB IP for nodes to communicate with Control plane

Move all defaults to a separate file under services

Minor fix to remove any accidental tabs present in yaml file used in startup script

Move AdminKubeconfig and DiscoveryHashes back to ClusterConfig, since kubeadm uses on disk certificates, we only update if the spec certs are empty, causing mismatch

Address review comments, convert spec to an defined interface and use ref rather than value

Author: awesomenix

Dockerfile

@@ -28,5 +28,4 @@ RUN CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -a -o manager sigs.k8s.io/clu
 FROM alpine:3.9
 WORKDIR /root/
 COPY --from=builder /go/src/sigs.k8s.io/cluster-api-provider-azure/manager .
-COPY --from=builder /go/src/sigs.k8s.io/cluster-api-provider-azure/pkg/cloud/azure/services/resources/template/deployment-template.json .
 ENTRYPOINT ["./manager"]

Gopkg.lock

@@ -47,7 +47,7 @@ CREDENTIALS_FILE=${OUTPUT_DIR}/credentials.sh
 CLUSTER_TEMPLATE_FILE=${DIR}/cluster.yaml.template
 CLUSTER_GENERATED_FILE=${OUTPUT_DIR}/cluster.yaml
 # TODO: Change the machine template once nodes are implemented
-MACHINES_TEMPLATE_FILE=${DIR}/machines_no_node.yaml.template
+MACHINES_TEMPLATE_FILE=${DIR}/machines.yaml.template
 MACHINES_GENERATED_FILE=${OUTPUT_DIR}/machines.yaml
 MANAGER_PATCH_TEMPLATE_FILE=${DIR}/azure_manager_image_patch.yaml.template
 MANAGER_PATCH_GENERATED_FILE=${OUTPUT_DIR}/azure_manager_image_patch.yaml

cmd/clusterctl/examples/azure/generate-yaml.sh

@@ -42,7 +42,6 @@ items:
     spec:
       versions:
         kubelet: 1.13.4
-        controlPlane: 1.13.4
       providerSpec:
         value:
           apiVersion: azureprovider/v1alpha1

cmd/clusterctl/examples/azure/machines.yaml.template

@@ -98,20 +98,16 @@ Here's an example of how to build controller images, if you're interested in tes
 
 ```bash
 # Build the image.
-PREFIX=quay.io/k8s \
-NAME=cluster-api-azure-controller \
-TAG=0.1.0-alpha.3 \
-make docker-build
+STABLE_DOCKER_REPO=quay.io/<name here> make docker-build
 
 # Push the image.
-PREFIX=quay.io/k8s \
-NAME=cluster-api-azure-controller \
-TAG=0.1.0-alpha.3 \
-make docker-push
+STABLE_DOCKER_REPO=quay.io/<name here> make docker-push
 ```
 
 **NOTE:** In order for the created images to be used for testing, you must push them to a public container registry.
 
+MANAGER_IMAGE must point to right repo, to make sure the image is picked up by bootstrap and target cluster
+
 ### Submitting PRs and testing
 
 Pull requests and issues are highly encouraged!

docs/getting-started.md

@@ -48,6 +48,15 @@ type AzureClusterProviderSpec struct {
 	// SAKeyPair is the service account key pair.
 	SAKeyPair KeyPair `json:"saKeyPair,omitempty"`
 
+	// AdminKubeconfig generated using the certificates part of the spec
+	// do not move to status, since it uses on disk ca certs, which causes issues during regeneration
+	AdminKubeconfig string `json:"adminKubeconfig,omitempty"`
+
+	// DiscoveryHashes generated using the certificates part of the spec, used by master and nodes bootstrapping
+	// this never changes until ca is rotated
+	// do not move to status, since it uses on disk ca certs, which causes issues during regeneration
+	DiscoveryHashes []string `json:"discoveryHashes,omitempty"`
+
 	// ClusterConfiguration holds the cluster-wide information used during a
 	// kubeadm init call.
 	ClusterConfiguration kubeadmv1beta1.ClusterConfiguration `json:"clusterConfiguration,omitempty"`

pkg/apis/azureprovider/v1alpha1/azureclusterproviderconfig_types.go

@@ -30,9 +30,8 @@ type AzureClusterProviderStatus struct {
 	metav1.TypeMeta   `json:",inline"`
 	metav1.ObjectMeta `json:"metadata,omitempty"`
 
-	CertificateStatus CertificateStatus `json:"certificateStatus,omitempty"`
-	Network           Network           `json:"network,omitempty"`
-	Bastion           VM                `json:"bastion,omitempty"`
+	Network Network `json:"network,omitempty"`
+	Bastion VM      `json:"bastion,omitempty"`
 }
 
 // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object

pkg/apis/azureprovider/v1alpha1/azureclusterproviderstatus_types.go

@@ -85,16 +85,6 @@ type Network struct {
 	APIServerIP PublicIP `json:"apiServerIp,omitempty"`
 }
 
-//CertificateStatus contains fields generated along with certificate generation
-type CertificateStatus struct {
-	// AdminKubeconfig generated using the certificates part of the spec
-	AdminKubeconfig string `json:"adminKubeconfig,omitempty"`
-
-	// DiscoveryHashes generated using the certificates part of the spec, used by master and nodes bootstrapping
-	// this never changes until ca is rotated
-	DiscoveryHashes []string `json:"discoveryHashes,omitempty"`
-}
-
 // TODO: Implement tagging
 /*
 // Tags defines resource tags.

pkg/apis/azureprovider/v1alpha1/types.go

@@ -0,0 +1,14 @@
+load("@io_bazel_rules_go//go:def.bzl", "go_library")
+
+go_library(
+    name = "go_default_library",
+    srcs = [
+        "defaults.go",
+        "errors.go",
+        "fake_interfaces.go",
+        "interfaces.go",
+    ],
+    importpath = "sigs.k8s.io/cluster-api-provider-azure/pkg/cloud/azure",
+    visibility = ["//visibility:public"],
+    deps = ["//vendor/github.com/Azure/go-autorest/autorest:go_default_library"],
+)