Merge pull request #59174 from tianshapjq/todo-already-done

Automatic merge from submit-queue (batch tested with PRs 65230, 57355, 59174, 63698, 63659). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. TODO has already been implemented **What this PR does / why we need it**: TODO has already been implemented, remove the TODO tag. **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*: Fixes # **Special notes for your reviewer**: **Release note**: ```release-note ```NONE

Merge pull request #59174 from tianshapjq/todo-already-done
c399c306 · Kubernetes Submit Queue · GitHub · 5e16fd55 · 21702e3c · c399c306
Unverified Commit c399c306 authored Jun 19, 2018 by Kubernetes Submit Queue Committed by GitHub Jun 19, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 0 additions and 1 deletion

manager.go pkg/kubelet/cm/devicemanager/manager.go +0 -1

No files found.
--- a/pkg/kubelet/cm/devicemanager/manager.go
+++ b/pkg/kubelet/cm/devicemanager/manager.go
@@ -259,7 +259,6 @@ func (m *ManagerImpl) Devices() map[string][]pluginapi.Device {
 func (m *ManagerImpl) Allocate(node *schedulercache.NodeInfo, attrs *lifecycle.PodAdmitAttributes) error {
 	pod := attrs.Pod
 	devicesToReuse := make(map[string]sets.String)
-	// TODO: Reuse devices between init containers and regular containers.
 	for _, container := range pod.Spec.InitContainers {
 		if err := m.allocateContainerResources(pod, &container, devicesToReuse); err != nil {
 			return err