Merge pull request #45157 from k82cn/rm_rc_comments

Automatic merge from submit-queue Removed TODO that renaming ReplicationManager. **What this PR does / why we need it**: Updated comments to avoid potential PRs on renaming. **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes # N/A **Release note**: ```release-note-none ```

Merge pull request #45157 from k82cn/rm_rc_comments
8c188b2b · Kubernetes Submit Queue · GitHub · a8e94004 · c55f30ba · 8c188b2b
Commit 8c188b2b authored Apr 30, 2017 by Kubernetes Submit Queue Committed by GitHub Apr 30, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

replication_controller.go pkg/controller/replication/replication_controller.go +2 -2

No files found.
--- a/pkg/controller/replication/replication_controller.go
+++ b/pkg/controller/replication/replication_controller.go
@@ -61,8 +61,8 @@ var controllerKind = v1.SchemeGroupVersion.WithKind("ReplicationController")

 // ReplicationManager is responsible for synchronizing ReplicationController objects stored
 // in the system with actual running pods.
-// TODO: this really should be called ReplicationController. The only reason why it's a Manager
-// is to distinguish this type from API object "ReplicationController". We should fix this.
+// NOTE: using this name to distinguish this type from API object "ReplicationController"; will
+//       not fix it right now. Refer to #41459 for more detail.
 type ReplicationManager struct {
 	kubeClient clientset.Interface
 	podControl controller.PodControlInterface