Upgrade operator SDK (#13)

* Migrate to v0.16.0

* Migrate to v0.17.2

* Migrate to v0.18.2

* Migrated to OperatorSDK 1.14.0

* Adding gotestsum

* Switch to Github Actions

* Autofixing golangci-lint

* Use UBI image

* Update CRD

* Update README

* Bundle build works well

* Adding release YAMl task

Author: cardil

.dockerignore

@@ -1,2 +1,7 @@
-build/Dockerfile
+# More info: https://docs.docker.com/engine/reference/builder/#dockerignore-file
+# Ignore build and test binaries.
+bundle.Dockerfile
 Dockerfile
+bin/
+build/
+testbin/

.github/workflows/go.yml

@@ -0,0 +1,31 @@
+name: Make
+
+on:
+  push:
+    branches: [ 'main', 'master', 'release-*' ]
+  pull_request:
+    types: [opened, synchronize, reopened]
+
+jobs:
+
+  build:
+    name: Build
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        go-version:
+          - '1.17'
+          - '1.16'
+    steps:
+
+    - name: Set up Go ${{ matrix.go-version }}
+      uses: actions/setup-go@v2
+      with:
+        go-version: ${{ matrix.go-version }}
+      id: go
+
+    - name: Check out code into the Go module directory
+      uses: actions/checkout@v2
+
+    - name: Build & Test
+      run: make test

.github/workflows/security.yml

@@ -0,0 +1,30 @@
+name: Security
+
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+    types: [opened, synchronize, reopened]
+
+jobs:
+  scan:
+    name: Snyk code scanning
+    runs-on: ubuntu-18.04
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v2
+
+      - name: Run Snyk to check for vulnerabilities
+        uses: snyk/actions/golang@master
+        # To make sure that SARIF upload gets called
+        continue-on-error: true
+        env:
+          SNYK_TOKEN: ${{ secrets.SNYK_TOKEN }}
+        with:
+          args: --sarif-file-output=snyk.sarif
+
+      - name: Upload result to GitHub Code Scanning
+        uses: github/codeql-action/upload-sarif@v1
+        with:
+          sarif_file: snyk.sarif

.gitignore

@@ -1,77 +1,31 @@
-# Temporary Build Files
-build/_output
-build/_test
-# Created by https://www.gitignore.io/api/go,vim,emacs,visualstudiocode
-### Emacs ###
-# -*- mode: gitignore; -*-
-*~
-\#*\#
-/.emacs.desktop
-/.emacs.desktop.lock
-*.elc
-auto-save-list
-tramp
-.\#*
-# Org-mode
-.org-id-locations
-*_archive
-# flymake-mode
-*_flymake.*
-# eshell files
-/eshell/history
-/eshell/lastdir
-# elpa packages
-/elpa/
-# reftex files
-*.rel
-# AUCTeX auto folder
-/auto/
-# cask packages
-.cask/
-dist/
-# Flycheck
-flycheck_*.el
-# server auth directory
-/server/
-# projectiles files
-.projectile
-projectile-bookmarks.eld
-# directory configuration
-.dir-locals.el
-# saveplace
-places
-# url cache
-url/cache/
-# cedet
-ede-projects.el
-# smex
-smex-items
-# company-statistics
-company-statistics-cache.el
-# anaconda-mode
-anaconda-mode/
-### Go ###
+
 # Binaries for programs and plugins
 *.exe
 *.exe~
 *.dll
 *.so
 *.dylib
-# Test binary, build with 'go test -c'
+bin
+testbin/*
+
+# Test binary, build with `go test -c`
 *.test
+
 # Output of the go coverage tool, specifically when used with LiteIDE
 *.out
-### Vim ###
-# swap
-.sw[a-p]
-.*.sw[a-p]
-# session
-Session.vim
-# temporary
-.netrwhist
-# auto-generated tag files
-tags
-### VisualStudioCode ###
-.vscode/*
-.history
-# End of https://www.gitignore.io/api/go,vim,emacs,visualstudiocode
+
+# Kubernetes Generated files - skip generated files, except for vendored files
+
+!vendor/**/zz_generated.*
+
+# editor and IDE paraphernalia
+.idea
+*.swp
+*.swo
+*~
+
+build/
+bundle/
+
+# kustomize manager file
+config/manager/kustomization.yaml

.golangci.yaml

@@ -0,0 +1,33 @@
+run:
+  timeout: 5m
+  build-tags:
+    - e2e
+
+linters:
+  disable-all: false
+  presets:
+    - bugs
+    - unused
+    - complexity
+    - format
+    - performance
+    - style
+  enable:
+    - gci
+  disable:
+    - paralleltest
+    - nlreturn
+    - exhaustivestruct
+    - wsl
+    - godox
+    - scopelint
+    - maligned
+    - interfacer
+    - golint
+
+issues:
+  exclude-rules:
+    - path: _test\.go
+      linters:
+        - wrapcheck
+

.revive.toml

@@ -0,0 +1,135 @@
+ignoreGeneratedHeader = true
+severity = "warning"
+
+# confidence <= 0.2 generate a lot of errors from package-comments rule. It marks files that do not contain
+# package-level comments as a warning irrespective of existing package-level coment in one file.
+confidence = 0.25
+errorCode = 1
+warningCode = 1
+
+# Rules block.
+# ⚠ Make sure to sort rules alpabetically for readability! ⚠
+
+# argument-limit rule is setting up a maximum number of parameters that can be passed to the functions/methods.
+[rule.argument-limit]
+arguments = [5]
+
+# atomic rule checks for commonly mistaken usages of the sync/atomic package.
+[rule.atomic]
+
+# blank-imports rule disallows blank imports.
+[rule.blank-imports]
+
+# bool-literal-in-expr suggests removing boolean literals from logic expressions like `bar == true`, `arg == false`,
+# `r != true`, `false && boolExpr` and `boolExpr || true`.
+[rule.bool-literal-in-expr]
+
+# constant-logical-expr rule warns on constant logical expressions, like `name == name`.
+[rule.constant-logical-expr]
+
+# context-as-argument rule makes sure that context.Context is the first argument of a function.
+[rule.context-as-argument]
+
+# context-keys-type rule disallows the usage of basic types in context.WithValue
+[rule.context-keys-type]
+
+# confusing-naming rule warns on methods with names that differ only by capitalization.
+[rule.confusing-naming]
+
+# confusing-results rule suggests to name potentially confusing function results.
+[rule.confusing-results]
+
+# cyclomatic rule sets restriction for maximum Cyclomatic complexity.
+[rule.cyclomatic]
+arguments = [15]
+
+# deep-exit rule looks for program exits in funcs other than `main()` or `init()`.
+[rule.deep-exit]
+
+# dot-imports rule forbids `.` imports.
+[rule.dot-imports]
+
+# empty-block warns on empty code blocks.
+[rule.empty-block]
+
+# error-return rule ensure that the error return parameter is the last.
+[rule.error-return]
+
+# error-strings rule ensure conventions around error strings.
+[rule.error-strings]
+
+# error-naming rule ensure naming of error variables (has `Err` or `err` prefix).
+[rule.error-naming]
+
+# errorf rule warns on usage errors.New(fmt.Sprintf()) instead of fmt.Errorf()
+[rule.errorf]
+
+# exported rule ensure naming and commenting conventions on exported symbols.
+[rule.exported]
+
+# flag-parameter rule warns on boolean parameters that create a control coupling.
+[rule.flag-parameter]
+
+# get-return rule warns on getters that do not yield any result.
+[rule.get-return]
+
+# if-return rule warns redundant if when returning an error.
+[rule.if-return]
+
+# increment-decrement rule forces to use `i++` and `i--` instead of `i += 1` and `i -= 1`.
+[rule.increment-decrement]
+
+# indent-error-flow rule prevents redundant else statements.
+[rule.indent-error-flow]
+
+# modifies-value-receiver warns on assignments to value-passed method receivers.
+[rule.modifies-value-receiver]
+
+# package-comments rule ensures package commenting conventions.
+[rule.package-comments]
+
+# range rule prevents redundant variables when iterating over a collection.
+[rule.range]
+
+# range-val-in-closure warns if range value is used in a closure dispatched as goroutine.
+[rule.range-val-in-closure]
+
+# receiver-naming ensures conventions around the naming of receivers.
+[rule.receiver-naming]
+
+# redefines-builtin-id warns on redefinitions of built-in (constants, variables, function and types) identifiers,
+# like `true := "false"` etc.
+[rule.redefines-builtin-id]
+
+# rule.superfluous-else prevents redundant else statements (extends indent-error-flow). Checks for `if-then-else`where
+# the then block ends with branching statement like `continue`, `break`, or `goto`.
+[rule.superfluous-else]
+
+# rule.struct-tag checks common struct tags like `json`, `xml`, `yaml`.
+[rule.struct-tag]
+
+# time-naming rule conventions around the naming of time variables. Like not to use unit suffixes (sec, min etc.) in
+# naming variables of type `time.Time` or `time.Duration`.
+[rule.time-naming]
+
+# unexported-return rule warns when a public return is from unexported type.
+[rule.unexported-return]
+
+# unnecessary-stmt  suggests removing or simplifying unnecessary statements like breaks at the end of cases or return at
+# the end of bodies of functions returning nothing.
+[rule.unnecessary-stmt]
+
+# unreachable-code rule warns on the unreachable code.
+[rule.unreachable-code]
+
+# unused-parameter rule suggests to rename or remove unused function parameters.
+[rule.unused-parameter]
+
+# var-declaration rule reduces redundancies around variable declaration.
+[rule.var-declaration]
+
+# var-naming checks naming rules.
+[rule.var-naming]
+
+# waitgroup-by-value rule warns on functions taking `sync.WaitGroup` as a by-value parameter.
+[rule.waitgroup-by-value]