openshift-diagnostics => diagnostics subcommands

Instead of building a separate `openshift-diagnostics` just turn its
subcommands into hidden subcommands on `oc adm diagnostics`.
Then use these within diagnostics pods.

fixes openshift#18141
fixes https://bugzilla.redhat.com/show_bug.cgi?id=1534513

contrib/vagrant/provision-util.sh

@@ -57,7 +57,7 @@ os::provision::install-cmds() {
   local deployed_root=$1
 
   local output_path="$(os::build::get-bin-output-path "${deployed_root}")"
-  cp ${output_path}/{openshift,openshift-diagnostics,oc} /usr/bin
+  cp ${output_path}/{openshift,oc} /usr/bin
 }
 
 os::provision::add-to-hosts-file() {

docs/man/man1/.files_generated_oc

@@ -47,6 +47,8 @@ oc-adm-diagnostics-clusterrouter.1
 oc-adm-diagnostics-configcontexts.1
 oc-adm-diagnostics-diagnosticpod.1
 oc-adm-diagnostics-etcdwritevolume.1
+oc-adm-diagnostics-inpod-networkcheck.1
+oc-adm-diagnostics-inpod-poddiagnostic.1
 oc-adm-diagnostics-masterconfigcheck.1
 oc-adm-diagnostics-masternode.1
 oc-adm-diagnostics-metricsapiproxy.1
@@ -200,6 +202,8 @@ oc-ex-diagnostics-clusterrouter.1
 oc-ex-diagnostics-configcontexts.1
 oc-ex-diagnostics-diagnosticpod.1
 oc-ex-diagnostics-etcdwritevolume.1
+oc-ex-diagnostics-inpod-networkcheck.1
+oc-ex-diagnostics-inpod-poddiagnostic.1
 oc-ex-diagnostics-masterconfigcheck.1
 oc-ex-diagnostics-masternode.1
 oc-ex-diagnostics-metricsapiproxy.1

docs/man/man1/oc-adm-diagnostics-inpod-networkcheck.1

@@ -0,0 +1,3 @@
+This file is autogenerated, but we've stopped checking such files into the
+repository to reduce the need for rebases. Please run hack/generate-docs.sh to
+populate this file.

docs/man/man1/oc-adm-diagnostics-inpod-poddiagnostic.1

@@ -0,0 +1,3 @@
+This file is autogenerated, but we've stopped checking such files into the
+repository to reduce the need for rebases. Please run hack/generate-docs.sh to
+populate this file.

docs/man/man1/oc-ex-diagnostics-inpod-networkcheck.1

@@ -0,0 +1,3 @@
+This file is autogenerated, but we've stopped checking such files into the
+repository to reduce the need for rebases. Please run hack/generate-docs.sh to
+populate this file.

docs/man/man1/oc-ex-diagnostics-inpod-poddiagnostic.1

@@ -0,0 +1,3 @@
+This file is autogenerated, but we've stopped checking such files into the
+repository to reduce the need for rebases. Please run hack/generate-docs.sh to
+populate this file.

hack/dind-cluster.sh

@@ -537,7 +537,6 @@ function copy-runtime() {
   local target=$2
 
   cp "$(os::util::find::built_binary openshift)" "${target}"
-  cp "$(os::util::find::built_binary openshift-diagnostics)" "${target}"
   cp "$(os::util::find::built_binary oc)" "${target}"
   cp "$(os::util::find::built_binary host-local)" "${target}"
   cp "$(os::util::find::built_binary loopback)" "${target}"

hack/lib/constants.sh

@@ -43,7 +43,6 @@ readonly OS_CROSS_COMPILE_TARGETS=(
   cmd/openshift
   cmd/oc
   cmd/oadm
-  cmd/openshift-diagnostics
   cmd/template-service-broker
   vendor/k8s.io/kubernetes/cmd/hyperkube
 )

images/dind/node/Dockerfile

@@ -70,7 +70,6 @@ RUN mkdir -p /opt/cni/bin
 # make reloading easy.  Revisit if/when dind becomes useful for more
 # than dev/test.
 RUN ln -sf /data/openshift /usr/local/bin/ && \
-    ln -sf /data/openshift-diagnostics /usr/local/bin/ && \
     ln -sf /data/oc        /usr/local/bin/ && \
     ln -sf /data/openshift /usr/local/bin/openshift-deploy && \
     ln -sf /data/openshift /usr/local/bin/openshift-docker-build && \

pkg/oc/admin/diagnostics/diagnostics.go

@@ -17,7 +17,9 @@ import (
 
 	"github.com/openshift/origin/pkg/client/config"
 	"github.com/openshift/origin/pkg/cmd/flagtypes"
+	clientdiag "github.com/openshift/origin/pkg/oc/admin/diagnostics/diagnostics/client"
 	"github.com/openshift/origin/pkg/oc/admin/diagnostics/diagnostics/log"
+	networkdiag "github.com/openshift/origin/pkg/oc/admin/diagnostics/diagnostics/network"
 	"github.com/openshift/origin/pkg/oc/admin/diagnostics/diagnostics/types"
 	"github.com/openshift/origin/pkg/oc/admin/diagnostics/options"
 	osclientcmd "github.com/openshift/origin/pkg/oc/cli/util/clientcmd"
@@ -141,6 +143,11 @@ func NewCmdDiagnostics(name string, fullName string, out io.Writer) *cobra.Comma
 	for _, diag := range available {
 		cmd.AddCommand(NewCmdDiagnosticsIndividual(strings.ToLower(diag.Name()), fullName+" "+strings.ToLower(diag.Name()), out, diag))
 	}
+	// add hidden in-pod subcommands
+	cmd.AddCommand(
+		NewCommandPodDiagnostics(clientdiag.InPodDiagnosticRecommendedName, out),
+		NewCommandNetworkPodDiagnostics(networkdiag.InPodNetworkCheckRecommendedName, out),
+	)
 
 	return cmd
 }

pkg/oc/admin/diagnostics/diagnostics/client/run_diagnostics_pod.go

@@ -20,9 +20,10 @@ import (
 )
 
 const (
-	DiagnosticPodName  = "DiagnosticPod"
-	ImageTemplateParam = "images"
-	LatestImageParam   = "latest-images"
+	InPodDiagnosticRecommendedName = "inpod-poddiagnostic"
+	DiagnosticPodName              = "DiagnosticPod"
+	ImageTemplateParam             = "images"
+	LatestImageParam               = "latest-images"
 )
 
 // DiagnosticPod is a diagnostic that runs a diagnostic pod and relays the results.
@@ -85,7 +86,7 @@ func (d *DiagnosticPod) runDiagnosticPod(r types.DiagnosticResult) {
 				{
 					Name:    "pod-diagnostics",
 					Image:   imageName,
-					Command: []string{"openshift-diagnostics", "diagnostic-pod", "-l", strconv.Itoa(loglevel)},
+					Command: []string{"oc", "adm", "diagnostics", InPodDiagnosticRecommendedName, "-l", strconv.Itoa(loglevel)},
 				},
 			},
 		},

pkg/oc/admin/diagnostics/diagnostics/network/run_pod.go

@@ -27,12 +27,13 @@ import (
 )
 
 const (
-	NetworkDiagnosticName          = "NetworkCheck"
-	FlagNetworkDiagLogDir          = "logdir"
-	FlagNetworkDiagPodImage        = "pod-image"
-	FlagNetworkDiagTestPodImage    = "test-pod-image"
-	FlagNetworkDiagTestPodProtocol = "test-pod-protocol"
-	FlagNetworkDiagTestPodPort     = "test-pod-port"
+	InPodNetworkCheckRecommendedName = "inpod-networkcheck"
+	NetworkDiagnosticName            = "NetworkCheck"
+	FlagNetworkDiagLogDir            = "logdir"
+	FlagNetworkDiagPodImage          = "pod-image"
+	FlagNetworkDiagTestPodImage      = "test-pod-image"
+	FlagNetworkDiagTestPodProtocol   = "test-pod-protocol"
+	FlagNetworkDiagTestPodPort       = "test-pod-port"
 )
 
 // NetworkDiagnostic is a diagnostic that runs a network diagnostic pod and relays the results.
@@ -181,7 +182,7 @@ func (d *NetworkDiagnostic) runNetworkDiagnostic() {
 	// In Collection phase, results from each node are moved to the user machine where the CLI cmd is executed.
 
 	// TEST Phase: Run network diagnostic pod on all valid nodes in parallel
-	command := fmt.Sprintf("openshift-diagnostics network-diagnostic-pod -l %d", loglevel)
+	command := fmt.Sprintf("oc adm diagnostics %s -l %d", InPodNetworkCheckRecommendedName, loglevel)
 	if err := d.runNetworkPod(command); err != nil {
 		d.res.Error("DNet2006", err, err.Error())
 		return

pkg/oc/admin/diagnostics/network_pod.go

@@ -52,9 +52,10 @@ func NewCommandNetworkPodDiagnostics(name string, out io.Writer) *cobra.Command
 	}
 
 	cmd := &cobra.Command{
-		Use:   name,
-		Short: "Within a privileged pod, run network diagnostics",
-		Long:  fmt.Sprintf(longNetworkPodDiagDescription),
+		Use:    name,
+		Short:  "Within a privileged pod, run network diagnostics",
+		Long:   fmt.Sprintf(longNetworkPodDiagDescription),
+		Hidden: true,
 		Run: func(c *cobra.Command, args []string) {
 			kcmdutil.CheckErr(o.Complete(args))
 

pkg/oc/admin/diagnostics/pod.go

@@ -50,9 +50,10 @@ func NewCommandPodDiagnostics(name string, out io.Writer) *cobra.Command {
 	}
 
 	cmd := &cobra.Command{
-		Use:   name,
-		Short: "Within a pod, run pod diagnostics",
-		Long:  fmt.Sprintf(longPodDiagDescription),
+		Use:    name,
+		Short:  "Within a pod, run pod diagnostics",
+		Long:   fmt.Sprintf(longPodDiagDescription),
+		Hidden: true,
 		Run: func(c *cobra.Command, args []string) {
 			kcmdutil.CheckErr(o.Complete(args))