Remove manager and collector commands

Author: marccampbell

Makefile

@@ -34,7 +34,7 @@ define LDFLAGS
 "
 endef
 
-all: test manager
+all: test
 
 .PHONY: ffi
 ffi: fmt vet
@@ -44,10 +44,6 @@ ffi: fmt vet
 test: generate fmt vet manifests
 	go test ./pkg/... ./cmd/... -coverprofile cover.out
 
-.PHONY: manager
-manager: generate fmt vet
-	go build ${LDFLAGS} -o bin/manager github.com/replicatedhq/troubleshoot/cmd/manager
-
 .PHONY: support-bundle
 support-bundle: generate fmt vet
 	go build ${LDFLAGS} -o bin/support-bundle github.com/replicatedhq/troubleshoot/cmd/troubleshoot
@@ -60,10 +56,6 @@ preflight: generate fmt vet
 analyze: generate fmt vet
 	go build ${LDFLAGS} -o bin/analyze github.com/replicatedhq/troubleshoot/cmd/analyze
 
-.PHONY: run
-run: generate fmt vet
-	TROUBLESHOOT_EXTERNAL_MANAGER=1 go run ./cmd/manager/main.go
-
 .PHONY: install
 install: manifests
 	kubectl apply -f config/crds
@@ -109,13 +101,6 @@ else
 CLIENT_GEN=$(shell which client-gen)
 endif
 
-.PHONY: snapshot-release
-snapshot-release:
-	curl -sL https://git.io/goreleaser | bash -s -- --rm-dist --snapshot --config deploy/.goreleaser.snapshot.yml
-	docker push replicated/troubleshoot:alpha
-	docker push replicated/preflight:alpha
-	docker push replicated/troubleshoot-manager:alpha
-
 .PHONY: release
 release: export GITHUB_TOKEN = $(shell echo ${GITHUB_TOKEN_TROUBLESHOOT})
 release:
@@ -126,10 +111,8 @@ local-release:
 	curl -sL https://git.io/goreleaser | bash -s -- --rm-dist --snapshot --config deploy/.goreleaser.local.yml
 	docker tag replicated/troubleshoot:alpha localhost:32000/troubleshoot:alpha
 	docker tag replicated/preflight:alpha localhost:32000/preflight:alpha
-	docker tag replicated/troubleshoot-manager:alpha localhost:32000/troubleshoot-manager:alpha
 	docker push localhost:32000/troubleshoot:alpha
 	docker push localhost:32000/preflight:alpha
-	docker push localhost:32000/troubleshoot-manager:alpha
 
 .PHONY: run-preflight
 run-preflight: preflight

cmd/collector/cli/root.go

@@ -444,6 +444,17 @@ spec:
                       - namespace
                       - containerPath
                       type: object
+                    data:
+                      properties:
+                        collectorName:
+                          type: string
+                        data:
+                          type: string
+                        name:
+                          type: string
+                      required:
+                      - data
+                      type: object
                     exec:
                       properties:
                         args:

cmd/collector/cli/run.go

@@ -849,6 +849,17 @@ spec:
                       - namespace
                       - containerPath
                       type: object
+                    data:
+                      properties:
+                        collectorName:
+                          type: string
+                        data:
+                          type: string
+                        name:
+                          type: string
+                      required:
+                      - data
+                      type: object
                     exec:
                       properties:
                         args:

cmd/collector/cli/server.go

@@ -419,6 +419,11 @@ func (in *Collect) DeepCopyInto(out *Collect) {
 		*out = new(Exec)
 		(*in).DeepCopyInto(*out)
 	}
+	if in.Data != nil {
+		in, out := &in.Data, &out.Data
+		*out = new(Data)
+		**out = **in
+	}
 	if in.Copy != nil {
 		in, out := &in.Copy, &out.Copy
 		*out = new(Copy)
@@ -762,6 +767,22 @@ func (in *CustomResourceDefinition) DeepCopy() *CustomResourceDefinition {
 	return out
 }
 
+// DeepCopyInto is an autogenerated deepcopy function, copying the receiver, writing into out. in must be non-nil.
+func (in *Data) DeepCopyInto(out *Data) {
+	*out = *in
+	out.CollectorMeta = in.CollectorMeta
+}
+
+// DeepCopy is an autogenerated deepcopy function, copying the receiver, creating a new Data.
+func (in *Data) DeepCopy() *Data {
+	if in == nil {
+		return nil
+	}
+	out := new(Data)
+	in.DeepCopyInto(out)
+	return out
+}
+
 // DeepCopyInto is an autogenerated deepcopy function, copying the receiver, writing into out. in must be non-nil.
 func (in *DeploymentStatus) DeepCopyInto(out *DeploymentStatus) {
 	*out = *in