GoogleCloudPlatform / k8s-multicluster-ingress

kubemci: Command line tool to configure L7 load balancers using multiple kubernetes clusters
Apache License 2.0
376 stars 68 forks source link

kubemci remove-clusters assumes that ordering of target tags has not changed #147

Open nikhiljindal opened 6 years ago

nikhiljindal commented 6 years ago

From #146: // This assumes that the ordering of target tags has not changed on these instances. // A potential solution to that problem is to recompute the target tags for the existing clusters, // rather than removing the ones for old clusters. // But we do not want to change existing tags for clusters that are already working. // Ideally network tags should only be appended to, so this should not be a problem. // TODO(nikhiljindal): Fix this if it becomes a problem.

Filing this issue to track the problem and see if users run into this. cc @G-Harmon