Browse Source

Remove TODO

Niko Kovacevic 5 years ago
parent
commit
bf1ad4aacb
1 changed files with 0 additions and 2 deletions
  1. 0 2
      pkg/costmodel/allocation.go

+ 0 - 2
pkg/costmodel/allocation.go

@@ -913,8 +913,6 @@ func labelsToPodControllerMap(podLabels map[podKey]map[string]string, controller
 
 
 			podLabelSet := labels.Set(pLabels)
 			podLabelSet := labels.Set(pLabels)
 			if selector.Matches(podLabelSet) {
 			if selector.Matches(podLabelSet) {
-				// TODO niko/computeallocation does this need to be one-to-many? In that case, we'd
-				// need a different Allocation schema
 				if _, ok := podControllerMap[pKey]; ok {
 				if _, ok := podControllerMap[pKey]; ok {
 					log.Warningf("CostModel.ComputeAllocation: PodControllerMap match already exists: %s matches %s and %s", pKey, podControllerMap[pKey], cKey)
 					log.Warningf("CostModel.ComputeAllocation: PodControllerMap match already exists: %s matches %s and %s", pKey, podControllerMap[pKey], cKey)
 				}
 				}