gofmt100%

Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command

No problems detected. Good job!

go_vet100%

go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.

No problems detected. Good job!

gocyclo80%

Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.

• /kubedb.dev/elasticsearch/pkg/distribution/search_guard/config.go
  • Line 89: warning: cyclomatic complexity 25 of function (*Elasticsearch).EnsureDefaultConfig() is high (> 15) (gocyclo)

• /kubedb.dev/elasticsearch/pkg/admission/validator.go
  • Line 154: warning: cyclomatic complexity 48 of function ValidateElasticsearch() is high (> 15) (gocyclo)
  • Line 89: warning: cyclomatic complexity 20 of function (*ElasticsearchValidator).Admit() is high (> 15) (gocyclo)

• /kubedb.dev/elasticsearch/pkg/distribution/search_guard/certificate.go
  • Line 45: warning: cyclomatic complexity 16 of function (*Elasticsearch).EnsureCertSecrets() is high (> 15) (gocyclo)

• /kubedb.dev/elasticsearch/pkg/distribution/open_distro/certificate.go
  • Line 45: warning: cyclomatic complexity 16 of function (*Elasticsearch).EnsureCertSecrets() is high (> 15) (gocyclo)

• /kubedb.dev/elasticsearch/pkg/controller/reconciler.go
  • Line 45: warning: cyclomatic complexity 19 of function (*Reconciler).ReconcileNodes() is high (> 15) (gocyclo)

• /kubedb.dev/elasticsearch/pkg/controller/health.go
  • Line 58: warning: cyclomatic complexity 18 of function (*Controller).CheckElasticsearchHealthOnce() is high (> 15) (gocyclo)

• /kubedb.dev/elasticsearch/pkg/controller/workqueue.go
  • Line 46: warning: cyclomatic complexity 17 of function (*Controller).runElasticsearch() is high (> 15) (gocyclo)

• /kubedb.dev/elasticsearch/pkg/distribution/search_guard/statefulset.go
  • Line 197: warning: cyclomatic complexity 17 of function (*Elasticsearch).getVolumes() is high (> 15) (gocyclo)

• /kubedb.dev/elasticsearch/pkg/distribution/open_distro/config.go
  • Line 90: warning: cyclomatic complexity 25 of function (*Elasticsearch).EnsureDefaultConfig() is high (> 15) (gocyclo)

• /kubedb.dev/elasticsearch/pkg/distribution/open_distro/statefulset.go
  • Line 202: warning: cyclomatic complexity 17 of function (*Elasticsearch).getVolumes() is high (> 15) (gocyclo)

• /kubedb.dev/elasticsearch/pkg/controller/elasticsearch.go
  • Line 39: warning: cyclomatic complexity 27 of function (*Controller).create() is high (> 15) (gocyclo)

• /kubedb.dev/elasticsearch/pkg/distribution/elastic_stack/statefulset.go
  • Line 198: warning: cyclomatic complexity 16 of function (*Elasticsearch).getVolumes() is high (> 15) (gocyclo)

golint20%

Golint is a linter for Go source code.

• /kubedb.dev/elasticsearch/pkg/lib/kernel/sysctl.go
  • Line 26: warning: exported function GetSysctlCommandString should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/util/go-es/api.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 39: warning: exported type ESClient should have comment or be unexported (golint)
  • Line 45: warning: exported function GetElasticClient should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/search_guard/pdb.go
  • Line 17: warning: don't use an underscore in package name (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/elastic_stack/certificate.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 461: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/open_distro/monitoring_sidecar.go
  • Line 17: warning: don't use an underscore in package name (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/open_distro/statefulset.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 46: warning: exported const ExporterCertDir should have comment (or a comment on this block) or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/api/api.go
  • Line 25: warning: exported type ElasticsearchInterface should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/search_guard/nodes.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 32: warning: exported method Elasticsearch.EnsureMasterNodes should have comment or be unexported (golint)
  • Line 113: warning: exported method Elasticsearch.EnsureDataNodes should have comment or be unexported (golint)
  • Line 178: warning: exported method Elasticsearch.EnsureIngestNodes should have comment or be unexported (golint)
  • Line 242: warning: exported method Elasticsearch.EnsureCombinedNode should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/open_distro/certificate.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 534: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• /kubedb.dev/elasticsearch/pkg/cmds/root.go
  • Line 34: warning: exported function NewRootCmd should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/lib/cert/pkcs12/pkcs12.go
  • Line 43: warning: exported function CreateCaCertificateJKS should have comment or be unexported (golint)
  • Line 79: warning: exported function CreateNodeCertificateJKS should have comment or be unexported (golint)
  • Line 131: warning: exported function CreateSGAdminCertificateJKS should have comment or be unexported (golint)
  • Line 188: warning: exported function CreateClientCertificateJKS should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/search_guard/credential.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 35: warning: exported method Elasticsearch.EnsureAuthSecret should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/elastic_stack/credential.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 33: warning: exported method Elasticsearch.EnsureAuthSecret should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/open_distro/open_distro.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 32: warning: exported type Elasticsearch should have comment or be unexported (golint)
  • Line 41: warning: exported function New should have comment or be unexported (golint)
  • Line 50: warning: exported method Elasticsearch.UpdatedElasticsearch should have comment or be unexported (golint)
  • Line 54: warning: exported method Elasticsearch.RequiredCertSecretNames should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/cmds/run.go
  • Line 30: warning: exported function NewCmdRun should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/controller/health.go
  • Line 38: warning: exported method Controller.RunHealthChecker should have comment or be unexported (golint)
  • Line 44: warning: exported method Controller.CheckElasticsearchHealth should have comment or be unexported (golint)
  • Line 58: warning: exported method Controller.CheckElasticsearchHealthOnce should have comment or be unexported (golint)
  • Line 213: warning: exported method Controller.GetElasticsearchClient should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/util/go-es/es_client_v7.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 26: warning: exported type ESClientV7 should have comment or be unexported (golint)
  • Line 30: warning: exported method ESClientV7.ClusterStatus should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/search_guard/config_test.go
  • Line 17: warning: don't use an underscore in package name (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/elastic_stack/monitoring_sidecar.go
  • Line 17: warning: don't use an underscore in package name (golint)

• /kubedb.dev/elasticsearch/pkg/controller/config.go
  • Line 43: warning: exported type OperatorConfig should have comment or be unexported (golint)
  • Line 57: warning: exported function NewOperatorConfig should have comment or be unexported (golint)
  • Line 63: warning: exported method OperatorConfig.New should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/controller/reconciler.go
  • Line 33: warning: exported type Reconciler should have comment or be unexported (golint)
  • Line 38: warning: exported function NewReconciler should have comment or be unexported (golint)
  • Line 45: warning: exported method Reconciler.ReconcileNodes should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/util/go-es/es_client_v6.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 26: warning: exported type ESClientV6 should have comment or be unexported (golint)
  • Line 30: warning: exported method ESClientV6.ClusterStatus should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/elastic_stack/nodes.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 32: warning: exported method Elasticsearch.EnsureMasterNodes should have comment or be unexported (golint)
  • Line 113: warning: exported method Elasticsearch.EnsureDataNodes should have comment or be unexported (golint)
  • Line 178: warning: exported method Elasticsearch.EnsureIngestNodes should have comment or be unexported (golint)
  • Line 242: warning: exported method Elasticsearch.EnsureCombinedNode should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/open_distro/config.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 36: warning: exported const ConfigFileName should have comment (or a comment on this block) or be unexported (golint)
  • Line 51: warning: don't use underscores in Go names; var opendistro_security_enabled should be opendistroSecurityEnabled (golint)
  • Line 69: warning: don't use underscores in Go names; var opendistro_security_disabled should be opendistroSecurityDisabled (golint)
  • Line 76: warning: don't use underscores in Go names; var https_enabled should be httpsEnabled (golint)
  • Line 86: warning: don't use underscores in Go names; var https_disabled should be httpsDisabled (golint)
  • Line 90: warning: exported method Elasticsearch.EnsureDefaultConfig should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/open_distro/credential.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 35: warning: exported method Elasticsearch.EnsureAuthSecret should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/distribution.go
  • Line 36: warning: exported function NewElasticsearch should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/lib/cert/certificate.go
  • Line 30: warning: exported const Duration365d should have comment (or a comment on this block) or be unexported (golint)
  • Line 70: warning: exported function ExtractSubjectFromCertificate should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/lib/user/es-user.go
  • Line 28: warning: comment on exported function HasUser should be of the form "HasUser ..." (golint)
  • Line 37: warning: comment on exported function SetMissingUser should be of the form "SetMissingUser ..." (golint)
  • Line 46: warning: exported function SetPasswordHashForUser should have comment or be unexported (golint)
  • Line 70: warning: comment on exported function InUserConfigCompareEqual should be of the form "InUserConfigCompareEqual ..." (golint)

• /kubedb.dev/elasticsearch/pkg/controller/controller.go
  • Line 52: warning: exported type Controller should have comment or be unexported (golint)
  • Line 68: warning: exported function New should have comment or be unexported (golint)
  • Line 103: warning: comment on exported method Controller.EnsureCustomResourceDefinitions should be of the form "EnsureCustomResourceDefinitions ..." (golint)
  • Line 114: warning: comment on exported method Controller.Init should be of the form "Init ..." (golint)
  • Line 130: warning: comment on exported method Controller.Run should be of the form "Run ..." (golint)

• /kubedb.dev/elasticsearch/pkg/admission/mutator.go
  • Line 40: warning: exported type ElasticsearchMutator should have comment or be unexported (golint)
  • Line 51: warning: exported method ElasticsearchMutator.Resource should have comment or be unexported (golint)
  • Line 60: warning: exported method ElasticsearchMutator.Initialize should have comment or be unexported (golint)
  • Line 76: warning: exported method ElasticsearchMutator.Admit should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/search_guard/config.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 36: warning: exported const ConfigFileName should have comment (or a comment on this block) or be unexported (golint)
  • Line 51: warning: don't use underscores in Go names; var searchguard_security_enabled should be searchguardSecurityEnabled (golint)
  • Line 71: warning: don't use underscores in Go names; var searchguard_security_disabled should be searchguardSecurityDisabled (golint)
  • Line 75: warning: don't use underscores in Go names; var https_enabled should be httpsEnabled (golint)
  • Line 85: warning: don't use underscores in Go names; var https_disabled should be httpsDisabled (golint)
  • Line 89: warning: exported method Elasticsearch.EnsureDefaultConfig should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/search_guard/search_guard.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 32: warning: exported type Elasticsearch should have comment or be unexported (golint)
  • Line 41: warning: exported function New should have comment or be unexported (golint)
  • Line 50: warning: exported method Elasticsearch.UpdatedElasticsearch should have comment or be unexported (golint)
  • Line 54: warning: exported method Elasticsearch.RequiredCertSecretNames should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/search_guard/statefulset.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 46: warning: exported const ExporterCertDir should have comment (or a comment on this block) or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/open_distro/config_test.go
  • Line 17: warning: don't use an underscore in package name (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/open_distro/nodes.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 32: warning: exported method Elasticsearch.EnsureMasterNodes should have comment or be unexported (golint)
  • Line 113: warning: exported method Elasticsearch.EnsureDataNodes should have comment or be unexported (golint)
  • Line 178: warning: exported method Elasticsearch.EnsureIngestNodes should have comment or be unexported (golint)
  • Line 242: warning: exported method Elasticsearch.EnsureCombinedNode should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/server/server.go
  • Line 54: warning: exported var Scheme should have comment or be unexported (golint)
  • Line 76: warning: exported type ElasticsearchServerConfig should have comment or be unexported (golint)
  • Line 82: warning: exported type ExtraConfig should have comment or be unexported (golint)
  • Line 92: warning: exported method ElasticsearchServer.Run should have comment or be unexported (golint)
  • Line 103: warning: exported type CompletedConfig should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/lib/keytool/jks.go
  • Line 39: warning: exported function PKCS12ToJKS should have comment or be unexported (golint)
  • Line 88: warning: exported function PEMToJKS should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/controller/appbinding.go
  • Line 75: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
  • Line 136: warning: exported method Controller.GetPrimaryServicePort should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/search_guard/certificate.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 534: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/elastic_stack/config.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 31: warning: exported const ConfigFileName should have comment (or a comment on this block) or be unexported (golint)
  • Line 34: warning: don't use underscores in Go names; var xpack_security_enabled should be xpackSecurityEnabled (golint)
  • Line 44: warning: don't use underscores in Go names; var xpack_security_disabled should be xpackSecurityDisabled (golint)
  • Line 48: warning: don't use underscores in Go names; var https_enabled should be httpsEnabled (golint)
  • Line 55: warning: don't use underscores in Go names; var https_disabled should be httpsDisabled (golint)
  • Line 59: warning: exported method Elasticsearch.EnsureDefaultConfig should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/elastic_stack/elastic_stack.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 28: warning: exported type Elasticsearch should have comment or be unexported (golint)
  • Line 37: warning: exported function New should have comment or be unexported (golint)
  • Line 46: warning: exported method Elasticsearch.UpdatedElasticsearch should have comment or be unexported (golint)
  • Line 50: warning: exported method Elasticsearch.RequiredCertSecretNames should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/open_distro/pdb.go
  • Line 17: warning: don't use an underscore in package name (golint)

• /kubedb.dev/elasticsearch/cmd/es-operator/version.go
  • Line 24: warning: exported var Version should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/cmds/server/options.go
  • Line 44: warning: exported type ExtraOptions should have comment or be unexported (golint)
  • Line 56: warning: exported function NewExtraOptions should have comment or be unexported (golint)
  • Line 70: warning: exported method ExtraOptions.AddGoFlags should have comment or be unexported (golint)
  • Line 82: warning: exported method ExtraOptions.AddFlags should have comment or be unexported (golint)
  • Line 88: warning: exported method ExtraOptions.ApplyTo should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/cmds/server/start.go
  • Line 39: warning: exported type ElasticsearchServerOptions should have comment or be unexported (golint)
  • Line 47: warning: exported function NewElasticsearchServerOptions should have comment or be unexported (golint)
  • Line 66: warning: exported method ElasticsearchServerOptions.AddFlags should have comment or be unexported (golint)
  • Line 71: warning: exported method ElasticsearchServerOptions.Validate should have comment or be unexported (golint)
  • Line 75: warning: exported method ElasticsearchServerOptions.Complete should have comment or be unexported (golint)
  • Line 79: warning: exported method ElasticsearchServerOptions.Config should have comment or be unexported (golint)
  • Line 104: warning: exported method ElasticsearchServerOptions.Run should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/lib/cert/pkcs8/pkcs8.go
  • Line 41: warning: comment on exported function CreateCaCertificate should be of the form "CreateCaCertificate ..." (golint)
  • Line 78: warning: comment on exported function CreateTransportCertificate should be of the form "CreateTransportCertificate ..." (golint)
  • Line 120: warning: comment on exported function CreateHTTPCertificate should be of the form "CreateHTTPCertificate ..." (golint)
  • Line 167: warning: comment on exported function CreateClientCertificate should be of the form "CreateClientCertificate ..." (golint)

• /kubedb.dev/elasticsearch/pkg/admission/validator.go
  • Line 46: warning: exported type ElasticsearchValidator should have comment or be unexported (golint)
  • Line 64: warning: exported method ElasticsearchValidator.Resource should have comment or be unexported (golint)
  • Line 73: warning: exported method ElasticsearchValidator.Initialize should have comment or be unexported (golint)
  • Line 89: warning: exported method ElasticsearchValidator.Admit should have comment or be unexported (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/search_guard/monitoring_sidecar.go
  • Line 17: warning: don't use an underscore in package name (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/elastic_stack/pdb.go
  • Line 17: warning: don't use an underscore in package name (golint)

• /kubedb.dev/elasticsearch/pkg/distribution/elastic_stack/statefulset.go
  • Line 17: warning: don't use an underscore in package name (golint)
  • Line 46: warning: exported const ExporterCertDir should have comment (or a comment on this block) or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!

misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!