From 567f68a2713edb6ed13eef07dc86b5519e5a3223 Mon Sep 17 00:00:00 2001 From: Ian Howell Date: Wed, 9 Dec 2020 13:21:55 -0600 Subject: [PATCH] Fix linting issues --- .gitignore | 1 - config/certmanager/certificate.yaml | 2 +- config/certmanager/kustomizeconfig.yaml | 2 +- config/default/kustomization.yaml | 6 +++--- config/default/manager_auth_proxy_patch.yaml | 2 +- config/samples/vino_cr.yaml | 2 +- main.go | 1 + 7 files changed, 8 insertions(+), 8 deletions(-) diff --git a/.gitignore b/.gitignore index c1cb092..853255d 100644 --- a/.gitignore +++ b/.gitignore @@ -1,4 +1,3 @@ - # Binaries for programs and plugins *.exe *.exe~ diff --git a/config/certmanager/certificate.yaml b/config/certmanager/certificate.yaml index 58db114..3d46cff 100644 --- a/config/certmanager/certificate.yaml +++ b/config/certmanager/certificate.yaml @@ -1,6 +1,6 @@ # The following manifests contain a self-signed issuer CR and a certificate CR. # More document can be found at https://docs.cert-manager.io -# WARNING: Targets CertManager 0.11 check https://docs.cert-manager.io/en/latest/tasks/upgrading/index.html for +# WARNING: Targets CertManager 0.11 check https://docs.cert-manager.io/en/latest/tasks/upgrading/index.html for # breaking changes apiVersion: cert-manager.io/v1alpha2 kind: Issuer diff --git a/config/certmanager/kustomizeconfig.yaml b/config/certmanager/kustomizeconfig.yaml index 90d7c31..e631f77 100644 --- a/config/certmanager/kustomizeconfig.yaml +++ b/config/certmanager/kustomizeconfig.yaml @@ -1,4 +1,4 @@ -# This configuration is for teaching kustomize how to update name ref and var substitution +# This configuration is for teaching kustomize how to update name ref and var substitution nameReference: - kind: Issuer group: cert-manager.io diff --git a/config/default/kustomization.yaml b/config/default/kustomization.yaml index 59ab95a..69475c4 100644 --- a/config/default/kustomization.yaml +++ b/config/default/kustomization.yaml @@ -16,12 +16,12 @@ bases: - ../crd - ../rbac - ../manager -# [WEBHOOK] To enable webhook, uncomment all the sections with [WEBHOOK] prefix including the one in +# [WEBHOOK] To enable webhook, uncomment all the sections with [WEBHOOK] prefix including the one in # crd/kustomization.yaml #- ../webhook # [CERTMANAGER] To enable cert-manager, uncomment all sections with 'CERTMANAGER'. 'WEBHOOK' components are required. #- ../certmanager -# [PROMETHEUS] To enable prometheus monitor, uncomment all sections with 'PROMETHEUS'. +# [PROMETHEUS] To enable prometheus monitor, uncomment all sections with 'PROMETHEUS'. #- ../prometheus patchesStrategicMerge: @@ -30,7 +30,7 @@ patchesStrategicMerge: # endpoint w/o any authn/z, please comment the following line. - manager_auth_proxy_patch.yaml -# [WEBHOOK] To enable webhook, uncomment all the sections with [WEBHOOK] prefix including the one in +# [WEBHOOK] To enable webhook, uncomment all the sections with [WEBHOOK] prefix including the one in # crd/kustomization.yaml #- manager_webhook_patch.yaml diff --git a/config/default/manager_auth_proxy_patch.yaml b/config/default/manager_auth_proxy_patch.yaml index 77e743d..a945ed1 100644 --- a/config/default/manager_auth_proxy_patch.yaml +++ b/config/default/manager_auth_proxy_patch.yaml @@ -1,4 +1,4 @@ -# This patch inject a sidecar container which is a HTTP proxy for the +# This patch inject a sidecar container which is a HTTP proxy for the # controller manager, it performs RBAC authorization against the Kubernetes API using SubjectAccessReviews. apiVersion: apps/v1 kind: Deployment diff --git a/config/samples/vino_cr.yaml b/config/samples/vino_cr.yaml index 9300b6b..546f0ee 100644 --- a/config/samples/vino_cr.yaml +++ b/config/samples/vino_cr.yaml @@ -95,7 +95,7 @@ status: sushy?: complete|failed domains: complete|failed bmh: complete|failed - lastUpdated: + lastUpdated: nodes: - node1: hypervisor: diff --git a/main.go b/main.go index 6fd00f5..dbc5517 100644 --- a/main.go +++ b/main.go @@ -36,6 +36,7 @@ var ( setupLog = ctrl.Log.WithName("setup") ) +//nolint:errcheck func init() { _ = clientgoscheme.AddToScheme(scheme)