Switch to golangci-lint for the gocheck target

Signed-off-by: Arjun Naik <anaik@redhat.com>

boilerplate/_lib/ensure.sh

@@ -0,0 +1,15 @@
+#!/bin/bash
+set -euo pipefail
+
+GOLANGCI_LINT_VERSION="1.30.0"
+DEPENDENCY=${1:-}
+
+case "${DEPENDENCY}" in
+    golangci-lint)
+        GOPATH=$(go env GOPATH)
+        if [ ! -f "${GOPATH}/bin/golangci-lint" ]; then
+            DOWNLOAD_URL="https://github.com/golangci/golangci-lint/releases/download/v${GOLANGCI_LINT_VERSION}/golangci-lint-${GOLANGCI_LINT_VERSION}-linux-amd64.tar.gz"
+            curl -sfL "${DOWNLOAD_URL}" | tar -C "${GOPATH}/bin" -zx --strip-components=1 "golangci-lint-${GOLANGCI_LINT_VERSION}-linux-amd64/golangci-lint"
+        fi
+    ;;
+esac

boilerplate/openshift/golang_osd_cluster_operator/golangci.yml

@@ -0,0 +1,6 @@
+# options for analysis running
+run:
+  # default concurrency is a available CPU number
+  concurrency: 10
+  # timeout for analysis, e.g. 30s, 5m, default is 1m
+  timeout: 2m

boilerplate/openshift/golang_osd_cluster_operator/standard.mk

@@ -71,8 +71,8 @@ docker-push: push
 
 .PHONY: gocheck
 gocheck: ## Lint code
-	boilerplate/openshift/golang_osd_cluster_operator/golint.sh
-	${GOENV} go vet ./cmd/... ./pkg/...
+	boilerplate/_lib/ensure.sh golangci-lint
+	GOLANGCI_LINT_CACHE=/tmp/golangci-cache golangci-lint run -c boilerplate/openshift/golang_osd_cluster_operator/golangci.yml ./...
 
 .PHONY: gogenerate
 gogenerate:

boilerplate/openshift/golang_osd_cluster_operator/update

@@ -1,6 +1,6 @@
 #!/bin/bash -e
 
-source $CONVENTION_ROOT/_lib/common.sh
+source "$CONVENTION_ROOT/_lib/common.sh"
 
 HERE=${0%/*}
 
@@ -11,7 +11,10 @@ HERE=${0%/*}
 [[ "$1" == "POST" ]] || err "Got a parameter I don't understand: '$1'. Did the infrastructure change?"
 
 echo "Copying .codecov.yml to your repository root."
-cp ${HERE}/.codecov.yml $REPO_ROOT
+cp "${HERE}/.codecov.yml" "$REPO_ROOT"
+
+echo "Ensuring golangci-lint is available on the \$PATH"
+$CONVENTION_ROOT/_lib/ensure.sh golangci-lint
 
 echo <<EOF
 =====================