Merge pull request #52685 from ixdy/kubectl-binary-public-visibility

Automatic merge from submit-queue. 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>. bazel: make //cmd/kubectl:kubectl binary publicly visible **What this PR does / why we need it**: making the `kubectl` binary visible to the release rules only seems unnecessarily restrictive (x-ref https://github.com/bazelbuild/bazel/issues/3744) - I think making this publicly visible should be fine. **Release note**: ```release-note NONE ``` /assign @monopole cc @achew22

Merge pull request #52685 from ixdy/kubectl-binary-public-visibility
fb51abb5 · Kubernetes Submit Queue · GitHub · 080db405 · 2bc88a81 · fb51abb5
Commit fb51abb5 authored Oct 02, 2017 by Kubernetes Submit Queue Committed by GitHub Oct 02, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 13 deletions

BUILD build/visible_to/BUILD +1 -10

BUILD cmd/kubectl/BUILD +1 -3

No files found.
--- a/build/visible_to/BUILD
+++ b/build/visible_to/BUILD
@@ -35,15 +35,6 @@ package_group(
 )
 package_group(
-    name = "COMMON_release",
-    packages = [
-        "//build",
-        "//build/debs",
-        "//build/rpms",
-    ],
-)
-package_group(
    name = "COMMON_testing",
    packages = [
        "//hack",
@@ -395,5 +386,5 @@ filegroup(
    name = "all-srcs",
    srcs = [":package-srcs"],
    tags = ["automanaged"],
-    visibility = ["//build/visible_to:COMMON_release"],
+    visibility = ["//visibility:public"],
 )
--- a/cmd/kubectl/BUILD
+++ b/cmd/kubectl/BUILD
@@ -19,9 +19,7 @@ go_binary(
        ],
    }),
    library = ":go_default_library",
-    visibility = [
+    visibility = ["//visibility:public"],
-        "//build/visible_to:COMMON_release",
-    ],
    x_defs = version_x_defs(),
 )