Move other binaries out of openshift

They will live in oc temporarily until they get their own binary.
openshift · Jun 14, 2018 · d5fa31f · d5fa31f
1 parent 10d8fc9
commit d5fa31f
Show file tree

Hide file tree

Showing 8 changed files with 23 additions and 32 deletions.
diff --git a/hack/lib/constants.sh b/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

diff --git a/images/recycler/Dockerfile b/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." \

diff --git a/images/router/f5/Dockerfile b/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." \

diff --git a/images/router/haproxy/Dockerfile b/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 && \

diff --git a/images/router/nginx/Dockerfile b/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
 

diff --git a/origin.spec b/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*
 

diff --git a/pkg/cmd/openshift/openshift.go b/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)
 	}

diff --git a/pkg/oc/cli/cli.go b/pkg/oc/cli/cli.go
@@ -18,6 +18,8 @@ import (
 	"github.com/openshift/origin/pkg/cmd/flagtypes"
 	"github.com/openshift/origin/pkg/cmd/infra/builder"
 	"github.com/openshift/origin/pkg/cmd/infra/deployer"
+	irouter "github.com/openshift/origin/pkg/cmd/infra/router"
+	"github.com/openshift/origin/pkg/cmd/recycle"
 	"github.com/openshift/origin/pkg/cmd/templates"
 	"github.com/openshift/origin/pkg/cmd/util/term"
 	"github.com/openshift/origin/pkg/oc/admin"
@@ -330,6 +332,12 @@ func CommandFor(basename string) *cobra.Command {
 		cmd = builder.NewCommandManageDockerfile(basename)
 	case "openshift-extract-image-content":
 		cmd = builder.NewCommandExtractImageContent(basename)
+	case "openshift-router":
+		cmd = irouter.NewCommandTemplateRouter(basename)
+	case "openshift-f5-router":
+		cmd = irouter.NewCommandF5Router(basename)
+	case "openshift-recycle":
+		cmd = recycle.NewCommandRecycle(basename, out)
 	default:
 		// we only need this change for `oc`.  `kubectl` should behave as close to `kubectl` as we can
 		resource.OAPIToGroupified = legacygroupification.OAPIToGroupified