fix bazel build issues

Author: Mengqi Yu

pkg/admission/cert/writer/secret.go

@@ -28,7 +28,6 @@ import (
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apimachinery/pkg/types"
-	apitypes "k8s.io/apimachinery/pkg/types"
 	"k8s.io/client-go/kubernetes/scheme"
 	"sigs.k8s.io/controller-runtime/pkg/admission/cert/generator"
 	"sigs.k8s.io/controller-runtime/pkg/client"
@@ -129,7 +128,7 @@ type secretReadWriter struct {
 
 type webhookAndSecret struct {
 	webhook *admissionregistrationv1beta1.Webhook
-	secret  apitypes.NamespacedName
+	secret  types.NamespacedName
 }
 
 var _ certReadWriter = &secretReadWriter{}
@@ -194,7 +193,7 @@ func secretToCerts(secret *corev1.Secret) *generator.Artifacts {
 	}
 }
 
-func certsToSecret(certs *generator.Artifacts, sec apitypes.NamespacedName) *corev1.Secret {
+func certsToSecret(certs *generator.Artifacts, sec types.NamespacedName) *corev1.Secret {
 	return &corev1.Secret{
 		ObjectMeta: metav1.ObjectMeta{
 			Namespace: sec.Namespace,

pkg/controller/controllerutil/controllerutil_test.go

@@ -1,4 +1,4 @@
-package controllerutil
+package controllerutil_test
 
 import (
 	. "github.com/onsi/ginkgo"
@@ -8,6 +8,7 @@ import (
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/client-go/kubernetes/scheme"
+	"sigs.k8s.io/controller-runtime/pkg/controller/controllerutil"
 )
 
 var _ = Describe("Controllerutil", func() {
@@ -18,7 +19,7 @@ var _ = Describe("Controllerutil", func() {
 				ObjectMeta: metav1.ObjectMeta{Name: "foo", UID: "foo-uid"},
 			}
 
-			Expect(SetControllerReference(dep, rs, scheme.Scheme)).NotTo(HaveOccurred())
+			Expect(controllerutil.SetControllerReference(dep, rs, scheme.Scheme)).NotTo(HaveOccurred())
 			t := true
 			Expect(rs.OwnerReferences).To(ConsistOf(metav1.OwnerReference{
 				Name:               "foo",
@@ -35,12 +36,12 @@ var _ = Describe("Controllerutil", func() {
 			dep := &extensionsv1beta1.Deployment{
 				ObjectMeta: metav1.ObjectMeta{Name: "foo"},
 			}
-			Expect(SetControllerReference(dep, rs, runtime.NewScheme())).To(HaveOccurred())
+			Expect(controllerutil.SetControllerReference(dep, rs, runtime.NewScheme())).To(HaveOccurred())
 		})
 
 		It("should return an error if the owner isn't a runtime.Object", func() {
 			rs := &appsv1.ReplicaSet{}
-			Expect(SetControllerReference(&errMetaObj{}, rs, scheme.Scheme)).To(HaveOccurred())
+			Expect(controllerutil.SetControllerReference(&errMetaObj{}, rs, scheme.Scheme)).To(HaveOccurred())
 		})
 	})
 })

pkg/source/internal/internal_suite_test.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package internal
+package internal_test
 
 import (
 	"testing"