reduce redundancy on context to cluster flags in solo deployment create
#977
Labels
Internal Requirement
Multiple Cluster Support
Issues which are only required to enable Solo support for multi-cluster deployments.
P1
High priority issue. Required to be completed in the assigned milestone.
It seems redundant that we would need to supply both the list of clusters for the deployment (
--deployment-clusters
) as well as the context to cluster mappings (--context-cluster
). Originally, we were going to included this in the LocalConfig design, but we realized that it can be retrieved directly from the kubernetes client and that since it could change it would be better to pull it fresh each time.I propose that we just remove the
--context-cluster
, and instead use the--deployment-clusters
, and then do a look up for the context using the Kubernetes client API. I think this would also be more re-usable across multiple users machines since the context can be renamed locally and may not match.The following:
whereas if I run the following:
The text was updated successfully, but these errors were encountered: