Move other binaries out of openshift

They will live in oc temporarily until they get their own binary.

Author: smarterclayton

hack/import-restrictions.json

@@ -467,7 +467,9 @@
       "github.com/openshift/origin/pkg/cmd/flagtypes",
       "github.com/openshift/origin/pkg/cmd/infra/builder",
       "github.com/openshift/origin/pkg/cmd/infra/deployer",
+      "github.com/openshift/origin/pkg/cmd/infra/router",
       "github.com/openshift/origin/pkg/cmd/openshift-operators/generated/clientset/versioned",
+      "github.com/openshift/origin/pkg/cmd/recycle",
       "github.com/openshift/origin/pkg/cmd/server/admin",
       "github.com/openshift/origin/pkg/cmd/server/apis/config",
       "github.com/openshift/origin/pkg/cmd/server/apis/config/install",

hack/lib/build/binaries.sh

@@ -407,9 +407,11 @@ readonly -f os::build::release_sha
 function os::build::make_openshift_binary_symlinks() {
   platform=$(os::build::host_platform)
   if [[ -f "${OS_OUTPUT_BINPATH}/${platform}/openshift" ]]; then
-    for linkname in "${OPENSHIFT_BINARY_SYMLINKS[@]}"; do
-      ln -sf openshift "${OS_OUTPUT_BINPATH}/${platform}/${linkname}"
-    done
+    if [[ -n "${OPENSHIFT_BINARY_SYMLINKS-}" ]]; then
+      for linkname in "${OPENSHIFT_BINARY_SYMLINKS[@]}"; do
+        ln -sf openshift "${OS_OUTPUT_BINPATH}/${platform}/${linkname}"
+      done
+    fi
   fi
   if [[ -f "${OS_OUTPUT_BINPATH}/${platform}/oc" ]]; then
     for linkname in "${OC_BINARY_SYMLINKS[@]}"; do

hack/lib/constants.sh

@@ -65,8 +65,6 @@ readonly OS_GOVET_BLACKLIST=(
 
 #If you update this list, be sure to get the images/origin/Dockerfile
 readonly OPENSHIFT_BINARY_SYMLINKS=(
-  openshift-router
-  openshift-recycle
 )
 readonly OC_BINARY_SYMLINKS=(
   openshift-deploy
@@ -75,6 +73,8 @@ readonly OC_BINARY_SYMLINKS=(
   openshift-git-clone
   openshift-manage-dockerfile
   openshift-extract-image-content
+  openshift-router
+  openshift-recycle
 )
 readonly OC_BINARY_COPY=(
   kubectl
@@ -386,13 +386,13 @@ function os::build::images() {
   ( os::build::image "${tag_prefix}-control-plane"  images/origin ) &
   ( os::build::image "${tag_prefix}-deployer"       images/deployer ) &
   ( os::build::image "${tag_prefix}-docker-builder" images/builder/docker/docker-builder ) &
+  ( os::build::image "${tag_prefix}-haproxy-router" images/router/haproxy ) &
+  ( os::build::image "${tag_prefix}-recycler"       images/recycler ) &
+  ( os::build::image "${tag_prefix}-f5-router"      images/router/f5 ) &
 
   for i in `jobs -p`; do wait $i; done
 
   # images that depend on "${tag_prefix}-control-plane"
-  ( os::build::image "${tag_prefix}-haproxy-router" images/router/haproxy ) &
-  ( os::build::image "${tag_prefix}-recycler"       images/recycler ) &
-  ( os::build::image "${tag_prefix}-f5-router"      images/router/f5 ) &
   ( os::build::image "${tag_prefix}-node"           images/node ) &
 
   for i in `jobs -p`; do wait $i; done

images/recycler/Dockerfile

@@ -3,7 +3,7 @@
 #
 # The standard name for this image is openshift/origin-recycler
 #
-FROM openshift/origin-control-plane
+FROM openshift/origin-cli
 
 LABEL io.k8s.display-name="OpenShift Origin Volume Recycler" \
       io.k8s.description="This is a component of OpenShift Origin and is used to prepare persistent volumes for reuse after they are deleted." \

images/router/f5/Dockerfile

@@ -3,7 +3,7 @@
 #
 # The standard name for this image is openshift/origin-f5-router
 #
-FROM openshift/origin-control-plane
+FROM openshift/origin-cli
 
 LABEL io.k8s.display-name="OpenShift Origin F5 Router" \
       io.k8s.description="This is a component of OpenShift Origin and programs a BigIP F5 router to expose services within the cluster." \

images/router/haproxy/Dockerfile

@@ -3,7 +3,7 @@
 #
 # The standard name for this image is openshift/origin-haproxy-router
 #
-FROM openshift/origin-control-plane
+FROM openshift/origin-cli
 
 RUN INSTALL_PKGS="haproxy18" && \
     yum install -y $INSTALL_PKGS && \

images/router/nginx/Dockerfile

@@ -3,7 +3,7 @@
 #
 # The standard name for this image is openshift/origin-nginx-router
 #
-FROM openshift/origin-control-plane
+FROM openshift/origin-cli
 
 ENV NGINX_VERSION 1.13.12-1.el7_4.ngx
 

origin.spec

@@ -298,17 +298,12 @@ for cmd in \
     openshift-git-clone \
     openshift-manage-dockerfile \
     openshift-extract-image-content \
-    kubectl
-do
-    ln -s oc %{buildroot}%{_bindir}/$cmd
-done
-
-for cmd in \
     openshift-f5-router \
     openshift-recycle \
-    openshift-router
+    openshift-router \
+    kubectl
 do
-    ln -s openshift %{buildroot}%{_bindir}/$cmd
+    ln -s oc %{buildroot}%{_bindir}/$cmd
 done
 
 install -d -m 0755 %{buildroot}%{_sysconfdir}/origin/{master,node}
@@ -368,9 +363,6 @@ chmod 0744 $RPM_BUILD_ROOT/usr/sbin/%{name}-docker-excluder
 %doc README.md
 %license LICENSE
 %{_bindir}/openshift
-%{_bindir}/openshift-f5-router
-%{_bindir}/openshift-recycle
-%{_bindir}/openshift-router
 %{_sharedstatedir}/origin
 %{_sysconfdir}/bash_completion.d/openshift
 %defattr(-,root,root,0700)
@@ -424,6 +416,9 @@ chmod 0744 $RPM_BUILD_ROOT/usr/sbin/%{name}-docker-excluder
 %{_bindir}/openshift-git-clone
 %{_bindir}/openshift-extract-image-content
 %{_bindir}/openshift-manage-dockerfile
+%{_bindir}/openshift-f5-router
+%{_bindir}/openshift-recycle
+%{_bindir}/openshift-router
 %{_sysconfdir}/bash_completion.d/oc
 %{_mandir}/man1/oc*
 

pkg/cmd/openshift/openshift.go

@@ -16,8 +16,6 @@ import (
 	kcmdutil "k8s.io/kubernetes/pkg/kubectl/cmd/util"
 
 	"github.com/openshift/origin/pkg/cmd/flagtypes"
-	irouter "github.com/openshift/origin/pkg/cmd/infra/router"
-	"github.com/openshift/origin/pkg/cmd/recycle"
 	"github.com/openshift/origin/pkg/cmd/server/start"
 	"github.com/openshift/origin/pkg/cmd/templates"
 	cmdutil "github.com/openshift/origin/pkg/cmd/util"
@@ -40,23 +38,13 @@ var (
 func CommandFor(basename string, stopCh <-chan struct{}) *cobra.Command {
 	var cmd *cobra.Command
 
-	out := os.Stdout
-
 	// Make case-insensitive and strip executable suffix if present
 	if runtime.GOOS == "windows" {
 		basename = strings.ToLower(basename)
 		basename = strings.TrimSuffix(basename, ".exe")
 	}
 
 	switch basename {
-	case "openshift-router":
-		cmd = irouter.NewCommandTemplateRouter(basename)
-	case "openshift-f5-router":
-		cmd = irouter.NewCommandF5Router(basename)
-	case "openshift-recycle":
-		cmd = recycle.NewCommandRecycle(basename, out)
-	case "origin":
-		cmd = NewCommandOpenShift(basename, stopCh)
 	default:
 		cmd = NewCommandOpenShift("openshift", stopCh)
 	}

pkg/cmd/openshift/openshift_test.go

@@ -7,12 +7,7 @@ import (
 )
 
 func TestCommandFor(t *testing.T) {
-	cmd := CommandFor("openshift-router", wait.NeverStop)
-	if cmd.Use != "openshift-router" {
-		t.Errorf("expected command to start with prefix: %#v", cmd)
-	}
-
-	cmd = CommandFor("unknown", wait.NeverStop)
+	cmd := CommandFor("unknown", wait.NeverStop)
 	if cmd.Use != "openshift" {
 		t.Errorf("expected command to be openshift: %#v", cmd)
 	}