Merge pull request #37208 from gmarek/controllers

Automatic merge from submit-queue (batch tested with PRs 37208, 37446, 37420) Change 'controller.go' filenames to more meaningfull ones Controller logic should sit in uniquely named file, not `controller.go`, as the latter makes it very hard to understand controller-manager logs. @deads2k @bgrant0607 @soltysh @gtank

Merge pull request #37208 from gmarek/controllers
3888ea75 · Kubernetes Submit Queue · GitHub · 7d1a7eae · 770e1c28 · 3888ea75
Commit 3888ea75 authored Dec 05, 2016 by Kubernetes Submit Queue Committed by GitHub Dec 05, 2016
6 changed files
--- a/pkg/controller/certificates/BUILD
+++ b/pkg/controller/certificates/BUILD
@@ -13,8 +13,8 @@ load(
 go_library(
    name = "go_default_library",
    srcs = [
-        "controller.go",
+        "certificate_controller.go",
-        "controller_utils.go",
+        "certificate_controller_utils.go",
        "doc.go",
        "groupapprove.go",
    ],

--- a/pkg/controller/certificates/controller.go
+++ b/pkg/controller/certificates/controller.go
--- a/pkg/controller/certificates/controller_utils.go
+++ b/pkg/controller/certificates/controller_utils.go
--- a/pkg/controller/cronjob/BUILD
+++ b/pkg/controller/cronjob/BUILD
@@ -13,7 +13,7 @@ load(
 go_library(
    name = "go_default_library",
    srcs = [
-        "controller.go",
+        "cronjob_controller.go",
        "doc.go",
        "injection.go",
        "utils.go",
@@ -44,7 +44,7 @@ go_library(
 go_test(
    name = "go_default_test",
    srcs = [
-        "controller_test.go",
+        "cronjob_controller_test.go",
        "utils_test.go",
    ],
    library = "go_default_library",

--- a/pkg/controller/cronjob/controller.go
+++ b/pkg/controller/cronjob/controller.go
--- a/pkg/controller/cronjob/controller_test.go
+++ b/pkg/controller/cronjob/controller_test.go