[Umbrella] capz MVP - Known Issues

[justaugustus]

As we move through a few refactoring efforts (#68, #88), it's a good time to capture some of the work required to bring us to a MVP capz implementation.

Here are some known issues that we'd want to resolve for the mvp:

• (master builds (i.e., make binaries-dev works), but doesn't result in a working cluster #100) master builds (i.e., make binaries-dev works), but doesn't result in a working cluster
• (Document versioning and release process #97) Repo versioning
• (Document developer workflow #98) Document development process
• (Investigate Azure Availability Zones #101) capz needs to be deployed in a region that supports Availability Zones
  • Resources (VM + NIC public IP) are hard-coded to Zone 3
• (VM cannot be attached to backend pool (NIC needs to either use a Standard SKU PIP or not have a PIP attached) & backend pool reconciliation code doesn't exist #102) VM cannot be attached to backend pool (NIC needs to either use a Standard SKU PIP or not have a PIP attached) & backend pool reconciliation code doesn't exist
• (Store public IP DNS name in Cluster object for use in machine scripts (includes updating associated types) #103) Store public IP DNS name in Cluster object for use in machine scripts (includes updating associated types)
• (Control plane NSG is open to the internet #104) Control plane VM is open to the internet
• (Subnets + NSGs: ReconcileNetwork should implement two subnets, control plane and node, with NSGs scoped to the role. #105) Subnets + NSGs: ReconcileNetwork should implement two subnets, control plane and node, with NSGs scoped to the role
• (Implement/update delete methods for Cluster/Machine objects #106) Implement/update delete methods for Cluster/Machine objects

Additional things to consider, but are not strictly required for mvp:

• (Investigating using distroless/static for image building #107) Investigating using distroless/static for image building (https://github.com/kubernetes-sigs/cluster-api-provider-azure/pull/88/files#r259902819)
• (Clean up Makefile #108) Update Makefile
• (Types need to be updated (+ regenerate deepcopy, CRDs, RBAC) #109) Types need to be updated (+ regenerate deepcopy, CRDs, RBAC)
• (Adopt a Reconcile/Delete pattern for exported methods #110) Minimize calls to Azure by storing more information in Cluster & Machine objects
• (Update helper packages (tokens, certificates, etc.) #111) Update helper packages (tokens, certificates, etc.)
• (Dedupe dependencies and move startup scripts to cloud-init #112) Dedupe dependencies and move startup scripts to cloud-init
• (Investigate VMSS (Virtual Machine Scale Sets) #113) Investigate VMSS (Virtual Machine Scale Sets)
• (Hard-coded parameters #114) Hard-coded parameters
• (Handle errors consistently #94) Improve logging
• (More intelligent reconcile logic for subresources #115) More intelligent reconcile logic for subresources

I'll get these into their own issues; just wanted to open this umbrella as a starting point.

/milestone mvp
/cc @soggiest @tariq1890

[justaugustus]

/assign

[justaugustus]

/kind feature
/priority critical-urgent

[justaugustus]

I think we've done a good job of taking care of most of the MVP items and I've re-prioritized the remaining issues on this umbrella.
/close

[k8s-ci-robot]

@justaugustus: Closing this issue.

In response to this:

I think we've done a good job of taking care of most of the MVP items and I've re-prioritized the remaining issues on this umbrella.
/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.