only use controller-tools CRD generator if in a Go project

Author: estroz

internal/util/projutil/project_util.go

@@ -21,16 +21,15 @@ import (
 	"path/filepath"
 	"strings"
 
-	"github.com/operator-framework/operator-sdk/pkg/scaffold/ansible"
-	"github.com/operator-framework/operator-sdk/pkg/scaffold/helm"
-
 	log "github.com/sirupsen/logrus"
 	"github.com/spf13/cobra"
 )
 
 const (
-	SrcDir          = "src"
+	srcDir          = "src"
 	mainFile        = "./cmd/manager/main.go"
+	rolesDir        = "./roles"
+	helmChartsDir   = "./helm-charts"
 	buildDockerfile = "./build/Dockerfile"
 )
 
@@ -87,7 +86,7 @@ func MustGetwd() string {
 // e.g: "github.com/example-inc/app-operator"
 func CheckAndGetProjectGoPkg() string {
 	gopath := SetGopath(GetGopath())
-	goSrc := filepath.Join(gopath, SrcDir)
+	goSrc := filepath.Join(gopath, srcDir)
 	wd := MustGetwd()
 	currPkg := strings.Replace(wd, goSrc+string(filepath.Separator), "", 1)
 	// strip any "/" prefix from the repo path.
@@ -102,15 +101,19 @@ func GetOperatorType() OperatorType {
 	if _, err := os.Stat(mainFile); err == nil {
 		return OperatorTypeGo
 	}
-	if stat, err := os.Stat(ansible.RolesDir); err == nil && stat.IsDir() {
+	if stat, err := os.Stat(rolesDir); err == nil && stat.IsDir() {
 		return OperatorTypeAnsible
 	}
-	if stat, err := os.Stat(helm.HelmChartsDir); err == nil && stat.IsDir() {
+	if stat, err := os.Stat(helmChartsDir); err == nil && stat.IsDir() {
 		return OperatorTypeHelm
 	}
 	return OperatorTypeUnknown
 }
 
+func IsOperatorGo() bool {
+	return GetOperatorType() == OperatorTypeGo
+}
+
 // GetGopath gets GOPATH and makes sure it is set and non-empty.
 func GetGopath() string {
 	gopath, ok := os.LookupEnv(GopathEnv)

pkg/scaffold/crd.go

@@ -22,6 +22,7 @@ import (
 	"strings"
 	"sync"
 
+	"github.com/operator-framework/operator-sdk/internal/util/projutil"
 	"github.com/operator-framework/operator-sdk/pkg/scaffold/input"
 
 	"github.com/ghodss/yaml"
@@ -82,8 +83,8 @@ func (s *Crd) CustomRender() ([]byte, error) {
 	// so generate crds in a cached, in-memory fs to extract the data we need.
 	// Note that controller-tools' generator makes different assumptions about
 	// how crd field values are structured, so we don't want to use the generated
-	// files directly.
-	if !cache.fileExists(path) {
+	// files directly. This generator will fail if not in a Go project.
+	if !cache.fileExists(path) && projutil.IsOperatorGo() {
 		g := &crdgenerator.Generator{
 			RootPath:          s.AbsProjectPath,
 			Domain:            "placeholder", // Our crds don't use this value.