nutanix-cloud-native / cluster-api-runtime-extensions-nutanix

https://nutanix-cloud-native.github.io/cluster-api-runtime-extensions-nutanix/
Apache License 2.0
7 stars 4 forks source link

fix: reorder lifecycle handlers with serviceloadbalancer being last #848

Closed dkoshkin closed 1 month ago

dkoshkin commented 1 month ago

What problem does this PR solve?: The lifecycle handlers are called in the order defined (after https://github.com/nutanix-cloud-native/cluster-api-runtime-extensions-nutanix/pull/845). The MetalLB serviceloadbalancer has a dependency on CNI and CCM to run first and start its deployment process. This PR is a small optimization that moves CCM higher in the list and the serviceloadbalancer to be last with a comment on why its last.

Which issue(s) this PR fixes: Fixes #

How Has This Been Tested?:

Special notes for your reviewer: