Version validation and readme update

.gitignore

@@ -6,6 +6,8 @@
 *.so
 *.dylib
 
+
+nginx-operator/
 # Test binary, build with `go test -c`
 *.test
 

README.md

@@ -40,4 +40,7 @@ go run main.go convert <OperatorName> --helm-chart=/path/to/chart --kind=Kind --
 To create an operator from an *external* helm chart:
 ```
 go run main.go convert <OperatorName> --helm-chart-repo=https://charts.example.io/ --helm-chart=example-chart --kind=Kind --api-version=apps.example.com/v1alpha1
-```
+```
+
+## Common Problems
+If you are experiencing build errors: `go: error loading module requirements`, execute the following command `export GO111MODULES=off` within the operator folder.

cmd/helm2go-operator-sdk/convert/cmd.go

@@ -60,6 +60,11 @@ func convertFunc(cmd *cobra.Command, args []string) error {
 		return err
 	}
 
+	if err := verifyOperatorSDKVersion(); err != nil {
+		log.Error(err)
+		return err
+	}
+
 	log.Infof("🤠 Converting Existing Helm Chart %s to Go Operator %s!", helmChartRef, outputDir)
 
 	// load the spcecified helm chart

cmd/helm2go-operator-sdk/convert/cmd_util.go

@@ -2,6 +2,9 @@ package convert
 
 import (
 	"fmt"
+	"os/exec"
+	"regexp"
+	"strconv"
 	"strings"
 )
 
@@ -50,3 +53,84 @@ func checkKindString(kind string) error {
 	}
 	return nil
 }
+
+func verifyOperatorSDKVersion() error {
+	var (
+		cmdOut []byte
+		err    error
+	)
+	cmdName := "operator-sdk"
+	cmdArgs := []string{"version"}
+
+	// if operator-sdk is not installed, or not updated this will throw an error
+	if cmdOut, err = exec.Command(cmdName, cmdArgs...).Output(); err != nil {
+		return fmt.Errorf("unexpected error: %v when verifying operator-sdk version; please install operator-sdk or update to latest version", err)
+	}
+	// make sure operator-sdk is atleast version 0.8.0 or higher
+	if err = matchVersion(&cmdOut); err != nil {
+		return fmt.Errorf("unexpected error: %v when verifying operator-sdk version; please update to latest version", err)
+	}
+
+	return nil
+}
+
+func matchVersion(cmdOut *[]byte) error {
+	pattern := regexp.MustCompile(`.*version\: +v(\d.\d.\d).*commit\: +(.*)`)
+	matches := pattern.FindStringSubmatch(string(*cmdOut))
+
+	fmt.Println(matches)
+
+	if l := len(matches); l != 2+1 {
+		return fmt.Errorf("expected three matches, received %d instead", l)
+	}
+
+	version := matches[1]
+	if len(version) == 0 {
+		return fmt.Errorf("expected operator-sdk version number, got: %v", version)
+	}
+
+	outdated, err := outdatedVersion(version)
+	if err != nil {
+		return err
+	}
+	if outdated {
+		return fmt.Errorf("operator-sdk version is outdated")
+	}
+
+	return nil
+
+}
+
+func outdatedVersion(version string) (bool, error) {
+
+	pattern := regexp.MustCompile(`^(\d)\.(\d)\.(\d)$`)
+	matches := pattern.FindStringSubmatch(version)
+
+	if l := len(matches); l != 3+1 {
+		return true, fmt.Errorf("expected four matches, received %d instead", l)
+	}
+
+	first := matches[1]
+	second := matches[2]
+
+	if len(first) == 0 || len(second) == 0 {
+		return true, fmt.Errorf("error parsing version number: %v", version)
+	}
+
+	firstInt, err := strconv.Atoi(first)
+	if err != nil {
+		return true, err
+	}
+	secondInt, err := strconv.Atoi(second)
+	if err != nil {
+		return true, err
+	}
+
+	if firstInt == 0 {
+		if secondInt < 8 {
+			return true, nil
+		}
+	}
+
+	return false, nil
+}

pkg/render/files.go

@@ -7,6 +7,7 @@ import (
 
 // takes a map of injected files and writes them to a temp directory
 func writeToTemp(files map[string]string, outParentDir string) (string, error) {
+	os.RemoveAll(filepath.Join(outParentDir, "temp"))
 	err := os.Mkdir(filepath.Join(outParentDir, "temp"), 0700)
 	if err != nil {
 		return "", err