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!

gocyclo88%

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.

• coherence-operator/pkg/runner/cmd_server.go
  • Line 37: warning: cyclomatic complexity 28 of function server() is high (> 15) (gocyclo)

• coherence-operator/controllers/statefulset/statefulset_controller.go
  • Line 316: warning: cyclomatic complexity 20 of function (*ReconcileStatefulSet).patchStatefulSet() is high (> 15) (gocyclo)
  • Line 107: warning: cyclomatic complexity 18 of function (*ReconcileStatefulSet).ReconcileAllResourceOfKind() is high (> 15) (gocyclo)

• coherence-operator/pkg/apis/coherence/legacy/zz_generated.deepcopy.go
  • Line 281: warning: cyclomatic complexity 36 of function (*CoherenceRoleSpec).DeepCopyInto() is high (> 15) (gocyclo)
  • Line 1218: warning: cyclomatic complexity 17 of function (*ServiceSpec).DeepCopyInto() is high (> 15) (gocyclo)

• coherence-operator/pkg/apis/coherence/legacy/coherence_types.go
  • Line 597: warning: cyclomatic complexity 18 of function (*SSLSpec).DeepCopyWithDefaults() is high (> 15) (gocyclo)
  • Line 1286: warning: cyclomatic complexity 17 of function (*ServiceSpec).DeepCopyWithDefaults() is high (> 15) (gocyclo)

• coherence-operator/api/v1/coherence_types.go
  • Line 821: warning: cyclomatic complexity 30 of function (*SSLSpec).CreateEnvVars() is high (> 15) (gocyclo)
  • Line 1498: warning: cyclomatic complexity 22 of function (*JvmMemorySpec).CreateEnvVars() is high (> 15) (gocyclo)

• coherence-operator/controllers/coherence_controller.go
  • Line 67: warning: cyclomatic complexity 30 of function (*CoherenceReconciler).Reconcile() is high (> 15) (gocyclo)

• coherence-operator/pkg/apis/coherence/legacy/coherencerolespec_types.go
  • Line 310: warning: cyclomatic complexity 17 of function (*CoherenceRoleSpec).DeepCopyWithDefaults() is high (> 15) (gocyclo)

• coherence-operator/pkg/runner/cmd_initialise.go
  • Line 30: warning: cyclomatic complexity 19 of function initialise() is high (> 15) (gocyclo)

• coherence-operator/pkg/runner/run_details.go
  • Line 110: warning: cyclomatic complexity 18 of function (*RunDetails).getShellName() is high (> 15) (gocyclo)

• coherence-operator/controllers/reconciler/base_controller.go
  • Line 553: warning: cyclomatic complexity 17 of function (*ReconcileSecondaryResource).ReconcileSingleResource() is high (> 15) (gocyclo)

• coherence-operator/controllers/servicemonitor/servicemonitor_controller.go
  • Line 106: warning: cyclomatic complexity 16 of function (*ReconcileServiceMonitor).ReconcileSingleResource() is high (> 15) (gocyclo)

• coherence-operator/pkg/runner/cmd_status_query.go
  • Line 75: warning: cyclomatic complexity 19 of function statusQuery() is high (> 15) (gocyclo)

• coherence-operator/pkg/runner/runner.go
  • Line 255: warning: cyclomatic complexity 51 of function createCommand() is high (> 15) (gocyclo)
  • Line 631: warning: cyclomatic complexity 19 of function configureSiteAndRack() is high (> 15) (gocyclo)
  • Line 844: warning: cyclomatic complexity 17 of function addSSL() is high (> 15) (gocyclo)

• coherence-operator/test/e2e/helper/e2e-helpers.go
  • Line 152: warning: cyclomatic complexity 16 of function NewContext() is high (> 15) (gocyclo)

• coherence-operator/api/v1/zz_generated.deepcopy.go
  • Line 150: warning: cyclomatic complexity 56 of function (*CoherenceResourceSpec).DeepCopyInto() is high (> 15) (gocyclo)
  • Line 1409: warning: cyclomatic complexity 24 of function (*ServiceSpec).DeepCopyInto() is high (> 15) (gocyclo)
  • Line 1314: warning: cyclomatic complexity 19 of function (*ServiceMonitorSpec).DeepCopyInto() is high (> 15) (gocyclo)

golint64%

Golint is a linter for Go source code.

• coherence-operator/pkg/runner/run_details.go
  • Line 19: warning: exported function NewRunDetails should have comment or be unexported (golint)

• coherence-operator/pkg/clients/clients.go
  • Line 18: warning: exported type ClientSet should have comment or be unexported (golint)
  • Line 25: warning: exported function New should have comment or be unexported (golint)
  • Line 33: warning: exported function NewForConfig should have comment or be unexported (golint)

• coherence-operator/controllers/servicemonitor/servicemonitor_controller.go
  • Line 51: warning: exported type ReconcileServiceMonitor should have comment or be unexported (golint)
  • Line 56: warning: exported method ReconcileServiceMonitor.GetReconciler should have comment or be unexported (golint)
  • Line 106: warning: exported method ReconcileServiceMonitor.ReconcileSingleResource should have comment or be unexported (golint)

• coherence-operator/pkg/management/pod_exec.go
  • Line 21: warning: exported type ExecRequest should have comment or be unexported (golint)
  • Line 30: warning: comment on exported function PodExec should be of the form "PodExec ..." (golint)

• coherence-operator/controllers/webhook/webhook_controller.go
  • Line 40: warning: exported type CertReconciler should have comment or be unexported (golint)
  • Line 47: warning: exported method CertReconciler.SetupWithManager should have comment or be unexported (golint)
  • Line 87: warning: exported method CertReconciler.Reconcile should have comment or be unexported (golint)
  • Line 267: warning: exported method CertReconciler.Cleanup should have comment or be unexported (golint)

• coherence-operator/test/e2e/remote/doc.go
  • Line 7: warning: package comment should be of the form "Package remote ..." (golint)

• coherence-operator/test/e2e/helper/e2e-helpers.go
  • Line 16: warning: should not use dot imports (golint)
  • Line 59: warning: exported var RetryInterval should have comment or be unexported (golint)
  • Line 76: warning: exported method TestContext.Logf should have comment or be unexported (golint)
  • Line 80: warning: exported method TestContext.CleanupAfterTest should have comment or be unexported (golint)
  • Line 90: warning: exported method TestContext.Cleanup should have comment or be unexported (golint)
  • Line 100: warning: exported method TestContext.CleanupNamespace should have comment or be unexported (golint)
  • Line 107: warning: exported method TestContext.CreateNamespace should have comment or be unexported (golint)
  • Line 122: warning: exported method TestContext.DeleteNamespace should have comment or be unexported (golint)
  • Line 141: warning: exported method TestContext.Close should have comment or be unexported (golint)
  • Line 404: warning: exported function ReplicaCountCondition should have comment or be unexported (golint)
  • Line 420: warning: exported function StatusPhaseCondition should have comment or be unexported (golint)
  • Line 669: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 684: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 709: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 894: warning: exported function DumpState should have comment or be unexported (golint)
  • Line 1301: warning: exported function DumpPodsForTest should have comment or be unexported (golint)
  • Line 1364: warning: exported function EnsureLogsDir should have comment or be unexported (golint)

• coherence-operator/test/e2e/helper/rest_helpers.go
  • Line 20: warning: comment on exported function StartCanary should be of the form "StartCanary ..." (golint)

• coherence-operator/controllers/webhook/webhook.go
  • Line 28: warning: exported type HookInstaller should have comment or be unexported (golint)
  • Line 81: warning: exported method HookInstaller.InstallWithCertManager should have comment or be unexported (golint)

• coherence-operator/controllers/statefulset/probe.go
  • Line 30: warning: exported type CoherenceProbe should have comment or be unexported (golint)
  • Line 37: warning: exported method CoherenceProbe.SetGetPodHostName should have comment or be unexported (golint)
  • Line 44: warning: exported method CoherenceProbe.GetPodHostName should have comment or be unexported (golint)
  • Line 51: warning: exported method CoherenceProbe.SetTranslatePort should have comment or be unexported (golint)
  • Line 58: warning: exported method CoherenceProbe.TranslatePort should have comment or be unexported (golint)
  • Line 76: warning: exported type ServiceSuspendStatus should have comment or be unexported (golint)
  • Line 79: warning: exported const ServiceSuspendSkipped should have comment (or a comment on this block) or be unexported (golint)
  • Line 119: warning: exported method CoherenceProbe.ExecuteProbe should have comment or be unexported (golint)
  • Line 189: warning: exported method CoherenceProbe.RunProbe should have comment or be unexported (golint)
  • Line 202: warning: exported method CoherenceProbe.ProbeUsingExec should have comment or be unexported (golint)
  • Line 223: warning: exported method CoherenceProbe.ProbeUsingHTTP should have comment or be unexported (golint)
  • Line 281: warning: exported method CoherenceProbe.ProbeUsingTCP should have comment or be unexported (golint)

• coherence-operator/pkg/fakes/fake_logger.go
  • Line 21: warning: exported method TestLogger.Info should have comment or be unexported (golint)
  • Line 25: warning: exported method TestLogger.Enabled should have comment or be unexported (golint)
  • Line 25: warning: receiver name should not be an underscore, omit the name if it is unused (golint)
  • Line 33: warning: exported method TestLogger.V should have comment or be unexported (golint)
  • Line 37: warning: exported method TestLogger.WithName should have comment or be unexported (golint)
  • Line 41: warning: exported method TestLogger.WithValues should have comment or be unexported (golint)

• coherence-operator/pkg/apis/coherence/legacy/coherencerole_types.go
  • Line 83: warning: exported const RoleStatusCreated should have comment (or a comment on this block) or be unexported (golint)

• coherence-operator/pkg/apis/coherence/legacy/coherencerolespec_types.go
  • Line 172: warning: comment on exported method CoherenceRoleSpec.GetReplicas should be of the form "GetReplicas ..." (golint)
  • Line 186: warning: comment on exported method CoherenceRoleSpec.SetReplicas should be of the form "SetReplicas ..." (golint)
  • Line 193: warning: comment on exported method CoherenceRoleSpec.GetFullRoleName should be of the form "GetFullRoleName ..." (golint)
  • Line 202: warning: comment on exported method CoherenceRoleSpec.GetRoleName should be of the form "GetRoleName ..." (golint)
  • Line 214: warning: exported method CoherenceRoleSpec.GetCoherenceImage should have comment or be unexported (golint)
  • Line 221: warning: comment on exported method CoherenceRoleSpec.EnsureCoherenceImage should be of the form "EnsureCoherenceImage ..." (golint)
  • Line 232: warning: exported method CoherenceRoleSpec.GetCoherenceUtilsImage should have comment or be unexported (golint)
  • Line 239: warning: comment on exported method CoherenceRoleSpec.EnsureCoherenceUtilsImage should be of the form "EnsureCoherenceUtilsImage ..." (golint)
  • Line 250: warning: exported method CoherenceRoleSpec.GetEffectiveScalingPolicy should have comment or be unexported (golint)
  • Line 274: warning: comment on exported method CoherenceRoleSpec.GetHealthPort should be of the form "GetHealthPort ..." (golint)
  • Line 282: warning: comment on exported method CoherenceRoleSpec.GetScalingProbe should be of the form "GetScalingProbe ..." (golint)
  • Line 291: warning: comment on exported method CoherenceRoleSpec.GetDefaultScalingProbe should be of the form "GetDefaultScalingProbe ..." (golint)

• coherence-operator/test/e2e/helper/proj_helpers.go
  • Line 58: warning: exported function EnsureTestEnvVars should have comment or be unexported (golint)
  • Line 312: warning: exported function FindActualFile should have comment or be unexported (golint)

• coherence-operator/pkg/certs/certs.go
  • Line 41: warning: exported method CA.PopulateSecret should have comment or be unexported (golint)
  • Line 48: warning: exported method CA.ShouldRenew should have comment or be unexported (golint)
  • Line 81: warning: exported function CreateSelfSignedCA should have comment or be unexported (golint)
  • Line 252: warning: exported function CanReuseCA should have comment or be unexported (golint)
  • Line 256: warning: exported function CertIsValid should have comment or be unexported (golint)

• coherence-operator/pkg/utils/storage.go
  • Line 31: warning: exported type Storage should have comment or be unexported (golint)

• coherence-operator/test/e2e/helper/matchers/matchers.go
  • Line 7: warning: package comment should be of the form "Package matchers ..." (golint)
  • Line 20: warning: comment on exported function HaveEnvVarNamed should be of the form "HaveEnvVarNamed ..." (golint)
  • Line 50: warning: exported method HaveNamedEnvVarMatcher.FailureMessage should have comment or be unexported (golint)
  • Line 54: warning: exported method HaveNamedEnvVarMatcher.NegatedFailureMessage should have comment or be unexported (golint)
  • Line 87: warning: exported method HaveEnvVarMatcher.FailureMessage should have comment or be unexported (golint)
  • Line 91: warning: exported method HaveEnvVarMatcher.NegatedFailureMessage should have comment or be unexported (golint)

• coherence-operator/pkg/apis/coherence/legacy/converter.go
  • Line 24: warning: exported function Convert should have comment or be unexported (golint)

• coherence-operator/test/coherence_compatibility/suite_test.go
  • Line 7: warning: don't use an underscore in package name (golint)

• coherence-operator/test/e2e/prometheus/prometheus_test.go
  • Line 118: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• coherence-operator/controllers/predicates/named_predicate.go
  • Line 21: warning: exported method NamedPredicate.Create should have comment or be unexported (golint)
  • Line 25: warning: exported method NamedPredicate.Delete should have comment or be unexported (golint)
  • Line 29: warning: exported method NamedPredicate.Update should have comment or be unexported (golint)
  • Line 33: warning: exported method NamedPredicate.Generic should have comment or be unexported (golint)

• coherence-operator/controllers/reconciler/simple_reconciler.go
  • Line 22: warning: exported function NewConfigMapReconciler should have comment or be unexported (golint)
  • Line 26: warning: exported function NewServiceReconciler should have comment or be unexported (golint)
  • Line 43: warning: exported type SimpleReconciler should have comment or be unexported (golint)
  • Line 47: warning: exported method SimpleReconciler.GetReconciler should have comment or be unexported (golint)

• coherence-operator/pkg/fakes/fake_client.go
  • Line 71: warning: comment on exported method ErrorIf.Matches should be of the form "Matches ..." (golint)
  • Line 306: warning: exported method ClientErrors.IsCreateError should have comment or be unexported (golint)
  • Line 310: warning: exported method ClientErrors.AddCreateError should have comment or be unexported (golint)
  • Line 317: warning: exported method ClientErrors.IsGetError should have comment or be unexported (golint)
  • Line 321: warning: exported method ClientErrors.AddGetError should have comment or be unexported (golint)
  • Line 328: warning: exported method ClientErrors.IsUpdateError should have comment or be unexported (golint)
  • Line 332: warning: exported method ClientErrors.AddUpdateError should have comment or be unexported (golint)
  • Line 339: warning: exported method ClientErrors.IsDeleteError should have comment or be unexported (golint)
  • Line 343: warning: exported method ClientErrors.AddDeleteError should have comment or be unexported (golint)
  • Line 350: warning: exported type ClientAction should have comment or be unexported (golint)
  • Line 353: warning: exported const ClientActionCreate should have comment (or a comment on this block) or be unexported (golint)
  • Line 359: warning: exported type ClientOperation should have comment or be unexported (golint)

• coherence-operator/test/e2e/compatibility/compatibility_helpers.go
  • Line 125: warning: exported type HelmInstallResult should have comment or be unexported (golint)
  • Line 131: warning: exported type HelmInstallResultFilter should have comment or be unexported (golint)
  • Line 133: warning: exported method HelmInstallResult.ToString should have comment or be unexported (golint)
  • Line 157: warning: exported method HelmInstallResult.Size should have comment or be unexported (golint)
  • Line 163: warning: exported method HelmInstallResult.Get should have comment or be unexported (golint)
  • Line 200: warning: exported method HelmInstallResult.List should have comment or be unexported (golint)

• coherence-operator/test/coherence_compatibility/certifiy_deployment_test.go
  • Line 7: warning: don't use an underscore in package name (golint)

• coherence-operator/pkg/apis/coherence/legacy/coherencecluster_types.go
  • Line 82: warning: comment on exported method CoherenceClusterStatus.SetRoleStatus should be of the form "SetRoleStatus ..." (golint)
  • Line 133: warning: exported method ClusterRoleStatus.Transition should have comment or be unexported (golint)
  • Line 157: warning: exported method ClusterRoleStatus.GetCondition should have comment or be unexported (golint)
  • Line 178: warning: exported method CoherenceCluster.GetWkaServiceName should have comment or be unexported (golint)
  • Line 185: warning: comment on exported method CoherenceCluster.GetRoles should be of the form "GetRoles ..." (golint)
  • Line 208: warning: comment on exported method CoherenceCluster.GetFullRoleName should be of the form "GetFullRoleName ..." (golint)
  • Line 216: warning: comment on exported method CoherenceCluster.GetFirstRole should be of the form "GetFirstRole ..." (golint)
  • Line 230: warning: comment on exported method CoherenceCluster.GetRole should be of the form "GetRole ..." (golint)
  • Line 244: warning: comment on exported method CoherenceCluster.SetRole should be of the form "SetRole ..." (golint)
  • Line 259: warning: comment on exported method CoherenceCluster.GetClusterSize should be of the form "GetClusterSize ..." (golint)
  • Line 268: warning: comment on exported method CoherenceCluster.GetRoleStatus should be of the form "GetRoleStatus ..." (golint)
  • Line 281: warning: comment on exported method CoherenceCluster.SetRoleStatus should be of the form "SetRoleStatus ..." (golint)

• coherence-operator/test/e2e/local/doc.go
  • Line 7: warning: package comment should be of the form "Package local ..." (golint)

• coherence-operator/pkg/management/coherence_management.go
  • Line 7: warning: package comment should be of the form "Package management ..." (golint)
  • Line 29: warning: comment on exported type RestData should be of the form "RestData ..." (with optional leading article) (golint)
  • Line 35: warning: comment on exported type ClusterData should be of the form "ClusterData ..." (with optional leading article) (golint)
  • Line 47: warning: comment on exported type ServicesData should be of the form "ServicesData ..." (with optional leading article) (golint)
  • Line 54: warning: comment on exported type ServiceData should be of the form "ServiceData ..." (with optional leading article) (golint)
  • Line 62: warning: comment on exported type PartitionData should be of the form "PartitionData ..." (with optional leading article) (golint)
  • Line 75: warning: comment on exported type MembersData should be of the form "MembersData ..." (with optional leading article) (golint)
  • Line 82: warning: comment on exported type MemberData should be of the form "MemberData ..." (with optional leading article) (golint)
  • Line 99: warning: comment on exported function GetCluster should be of the form "GetCluster ..." (golint)
  • Line 108: warning: comment on exported function GetMembers should be of the form "GetMembers ..." (golint)
  • Line 117: warning: comment on exported function GetServices should be of the form "GetServices ..." (golint)
  • Line 126: warning: comment on exported function GetPartitionAssignment should be of the form "GetPartitionAssignment ..." (golint)

• coherence-operator/pkg/utils/files.go
  • Line 17: warning: exported type FileFilter should have comment or be unexported (golint)
  • Line 21: warning: exported function AlwaysFilter should have comment or be unexported (golint)
  • Line 25: warning: exported function CopyDir should have comment or be unexported (golint)
  • Line 61: warning: comment on exported function CopyFile should be of the form "CopyFile ..." (golint)

• coherence-operator/controllers/statefulset/statefulset_controller.go
  • Line 33: warning: exported const CreateMessage should have comment (or a comment on this block) or be unexported (golint)
  • Line 74: warning: exported type ReconcileStatefulSet should have comment or be unexported (golint)
  • Line 79: warning: exported method ReconcileStatefulSet.GetReconciler should have comment or be unexported (golint)

• coherence-operator/pkg/fakes/fake_events.go
  • Line 23: warning: exported type FakeEvent should have comment or be unexported (golint)
  • Line 32: warning: exported method FakeEventRecorder.Event should have comment or be unexported (golint)
  • Line 38: warning: exported method FakeEventRecorder.Eventf should have comment or be unexported (golint)
  • Line 44: warning: exported method FakeEventRecorder.PastEventf should have comment or be unexported (golint)
  • Line 48: warning: exported method FakeEventRecorder.AnnotatedEventf should have comment or be unexported (golint)

• coherence-operator/pkg/apis/coherence/legacy/coherence_types.go
  • Line 22: warning: comment on exported const DefaultReplicas should be of the form "DefaultReplicas ..." (golint)
  • Line 25: warning: comment on exported const DefaultHealthPort should be of the form "DefaultHealthPort ..." (golint)
  • Line 28: warning: comment on exported const DefaultRoleName should be of the form "DefaultRoleName ..." (golint)
  • Line 31: warning: comment on exported const WKAServiceNameSuffix should be of the form "WKAServiceNameSuffix ..." (golint)
  • Line 34: warning: comment on exported const CoherenceClusterLabel should be of the form "CoherenceClusterLabel ..." (golint)
  • Line 37: warning: comment on exported const CoherenceRoleLabel should be of the form "CoherenceRoleLabel ..." (golint)
  • Line 40: warning: comment on exported const CoherenceComponentLabel should be of the form "CoherenceComponentLabel ..." (golint)
  • Line 43: warning: comment on exported const CoherenceOperatorVersionLabel should be of the form "CoherenceOperatorVersionLabel ..." (golint)
  • Line 49: warning: comment on exported function MergeMap should be of the form "MergeMap ..." (golint)
  • Line 79: warning: comment on exported type ApplicationSpec should be of the form "ApplicationSpec ..." (with optional leading article) (golint)
  • Line 176: warning: comment on exported type CoherenceSpec should be of the form "CoherenceSpec ..." (with optional leading article) (golint)
  • Line 281: warning: comment on exported type JVMSpec should be of the form "JVMSpec ..." (with optional leading article) (golint)
  • Line 366: warning: comment on exported type ImageSpec should be of the form "ImageSpec ..." (with optional leading article) (golint)
  • Line 379: warning: comment on exported method ImageSpec.EnsureImage should be of the form "EnsureImage ..." (golint)
  • Line 477: warning: comment on exported type PersistentStorageSpec should be of the form "PersistentStorageSpec ..." (with optional leading article) (golint)
  • Line 698: warning: comment on exported type PortSpec should be of the form "PortSpec ..." (with optional leading article) (golint)
  • Line 749: warning: comment on exported type NamedPortSpec should be of the form "NamedPortSpec ..." (with optional leading article) (golint)
  • Line 801: warning: comment on exported function MergeNamedPortSpecs should be of the form "MergeNamedPortSpecs ..." (golint)
  • Line 843: warning: comment on exported type JvmDebugSpec should be of the form "JvmDebugSpec ..." (with optional leading article) (golint)
  • Line 907: warning: comment on exported type JvmGarbageCollectorSpec should be of the form "JvmGarbageCollectorSpec ..." (with optional leading article) (golint)
  • Line 970: warning: comment on exported type JvmMemorySpec should be of the form "JvmMemorySpec ..." (with optional leading article) (golint)
  • Line 1054: warning: comment on exported type JvmOutOfMemorySpec should be of the form "JvmOutOfMemorySpec ..." (with optional leading article) (golint)
  • Line 1100: warning: comment on exported type JvmJmxmpSpec should be of the form "JvmJmxmpSpec ..." (with optional leading article) (golint)
  • Line 1203: warning: comment on exported type ServiceSpec should be of the form "ServiceSpec ..." (with optional leading article) (golint)
  • Line 1277: warning: comment on exported method ServiceSpec.SetServiceType should be of the form "SetServiceType ..." (golint)
  • Line 1387: warning: comment on exported type ScalingSpec should be of the form "ScalingSpec ..." (with optional leading article) (golint)
  • Line 1445: warning: comment on exported method ScalingProbe.GetTimeout should be of the form "GetTimeout ..." (golint)
  • Line 1522: warning: exported type ProbeHandler should have comment or be unexported (golint)

• coherence-operator/api/v1/coherence_webhook.go
  • Line 25: warning: exported method Coherence.SetupWebhookWithManager should have comment or be unexported (golint)

• coherence-operator/api/v1/constants.go
  • Line 126: warning: comment on exported const PortNameManagement should be of the form "PortNameManagement ..." (golint)
  • Line 155: warning: exported const EnvVarAppType should have comment (or a comment on this block) or be unexported (golint)

• coherence-operator/controllers/reconciler/base_controller.go
  • Line 59: warning: exported type BaseReconciler should have comment or be unexported (golint)
  • Line 79: warning: exported method CommonReconciler.GetControllerName should have comment or be unexported (golint)
  • Line 80: warning: exported method CommonReconciler.GetManager should have comment or be unexported (golint)
  • Line 81: warning: exported method CommonReconciler.GetClient should have comment or be unexported (golint)
  • Line 82: warning: exported method CommonReconciler.GetMutex should have comment or be unexported (golint)
  • Line 83: warning: exported method CommonReconciler.GetPatchType should have comment or be unexported (golint)
  • Line 84: warning: exported method CommonReconciler.SetPatchType should have comment or be unexported (golint)
  • Line 85: warning: exported method CommonReconciler.GetEventRecorder should have comment or be unexported (golint)
  • Line 88: warning: exported method CommonReconciler.GetLog should have comment or be unexported (golint)
  • Line 95: warning: exported method CommonReconciler.SetCommonReconciler should have comment or be unexported (golint)
  • Line 518: warning: exported method ReconcileSecondaryResource.GetTemplate should have comment or be unexported (golint)
  • Line 519: warning: exported method ReconcileSecondaryResource.CanWatch should have comment or be unexported (golint)
  • Line 673: warning: exported method ReconcileSecondaryResource.FindResource should have comment or be unexported (golint)

• coherence-operator/test/e2e/elastic/elasticsearch_helpers.go
  • Line 15: warning: should not use dot imports (golint)
  • Line 91: warning: exported function NewESSearchFunction should have comment or be unexported (golint)
  • Line 211: warning: exported type ESSearchTotal should have comment or be unexported (golint)

• coherence-operator/pkg/fakes/doc.go
  • Line 7: warning: package comment should be of the form "Package fakes ..." (golint)

• coherence-operator/pkg/fakes/fake_manager.go
  • Line 12: warning: should not use dot imports (golint)
  • Line 75: warning: exported type FakeManager should have comment or be unexported (golint)
  • Line 83: warning: exported method FakeManager.Start should have comment or be unexported (golint)
  • Line 87: warning: exported method FakeManager.GetLogger should have comment or be unexported (golint)
  • Line 91: warning: exported method FakeManager.Elected should have comment or be unexported (golint)
  • Line 95: warning: exported method FakeManager.AddMetricsExtraHandler should have comment or be unexported (golint)
  • Line 99: warning: exported method FakeManager.GetEventRecorderFor should have comment or be unexported (golint)
  • Line 103: warning: exported method FakeManager.GetAPIReader should have comment or be unexported (golint)
  • Line 107: warning: exported method FakeManager.GetWebhookServer should have comment or be unexported (golint)
  • Line 111: warning: exported type ReconcileResult should have comment or be unexported (golint)
  • Line 122: warning: exported method FakeManager.AddHealthzCheck should have comment or be unexported (golint)
  • Line 126: warning: exported method FakeManager.AddReadyzCheck should have comment or be unexported (golint)
  • Line 130: warning: exported method FakeManager.Add should have comment or be unexported (golint)
  • Line 134: warning: exported method FakeManager.SetFields should have comment or be unexported (golint)
  • Line 138: warning: exported method FakeManager.GetConfig should have comment or be unexported (golint)
  • Line 142: warning: exported method FakeManager.GetScheme should have comment or be unexported (golint)
  • Line 146: warning: exported method FakeManager.GetClient should have comment or be unexported (golint)
  • Line 150: warning: exported method FakeManager.GetFieldIndexer should have comment or be unexported (golint)
  • Line 154: warning: exported method FakeManager.GetCache should have comment or be unexported (golint)
  • Line 158: warning: exported method FakeManager.GetRESTMapper should have comment or be unexported (golint)
  • Line 169: warning: comment on exported method FakeManager.AssertNoRemainingEvents should be of the form "AssertNoRemainingEvents ..." (golint)
  • Line 225: warning: exported method FakeManager.GetCoherences should have comment or be unexported (golint)

• coherence-operator/api/v1/coherence_types.go
  • Line 1057: warning: exported method NamedPortSpec.IsEnabled should have comment or be unexported (golint)
  • Line 1061: warning: exported method NamedPortSpec.GetProtocol should have comment or be unexported (golint)
  • Line 1068: warning: exported method NamedPortSpec.GetPort should have comment or be unexported (golint)
  • Line 1083: warning: exported method NamedPortSpec.GetServicePort should have comment or be unexported (golint)
  • Line 1100: warning: exported method NamedPortSpec.GetNodePort should have comment or be unexported (golint)
  • Line 1107: warning: exported method NamedPortSpec.CreatePort should have comment or be unexported (golint)
  • Line 1213: warning: exported method ServiceMonitorSpec.CreateServiceMonitor should have comment or be unexported (golint)
  • Line 1226: warning: exported method ServiceMonitorSpec.CreateEndpoint should have comment or be unexported (golint)
  • Line 2142: warning: exported method PodDNSConfig.UpdateStatefulSet should have comment or be unexported (golint)
  • Line 2294: warning: exported method ConfigMapVolumeSpec.AddVolumeMounts should have comment or be unexported (golint)
  • Line 2414: warning: exported method SecretVolumeSpec.AddVolumeMounts should have comment or be unexported (golint)
  • Line 2437: warning: exported type ResourceType should have comment or be unexported (golint)
  • Line 2439: warning: exported method ResourceType.Name should have comment or be unexported (golint)
  • Line 2444: warning: exported const ResourceTypeCoherence should have comment (or a comment on this block) or be unexported (golint)
  • Line 2452: warning: exported function ToResourceType should have comment or be unexported (golint)
  • Line 2508: warning: exported method Resource.GetFullName should have comment or be unexported (golint)
  • Line 2515: warning: exported method Resource.IsDelete should have comment or be unexported (golint)
  • Line 2523: warning: exported method Resource.IsPresent should have comment or be unexported (golint)
  • Line 2580: warning: exported method Resources.MarshalJSON should have comment or be unexported (golint)
  • Line 2602: warning: exported method Resources.UnmarshalJSON should have comment or be unexported (golint)
  • Line 2636: warning: exported method Resources.EnsureGVK should have comment or be unexported (golint)
  • Line 2656: warning: exported method Resources.GetResource should have comment or be unexported (golint)
  • Line 2665: warning: exported method Resources.GetResourcesOfKind should have comment or be unexported (golint)

• coherence-operator/api/v1/hasher.go
  • Line 17: warning: exported function EnsureHashLabel should have comment or be unexported (golint)

• coherence-operator/pkg/operator/operator.go
  • Line 7: warning: package comment should be of the form "Package operator ..." (golint)
  • Line 26: warning: exported const DefaultRestHost should have comment (or a comment on this block) or be unexported (golint)
  • Line 73: warning: exported var DefaultSiteLabels should have comment or be unexported (golint)
  • Line 78: warning: exported function SetupFlags should have comment or be unexported (golint)
  • Line 212: warning: exported function ValidateFlags should have comment or be unexported (golint)
  • Line 227: warning: exported function IsDevMode should have comment or be unexported (golint)
  • Line 231: warning: exported function GetDefaultCoherenceImage should have comment or be unexported (golint)
  • Line 235: warning: exported function GetDefaultUtilsImage should have comment or be unexported (golint)
  • Line 239: warning: exported function GetRestHost should have comment or be unexported (golint)
  • Line 243: warning: exported function GetRestPort should have comment or be unexported (golint)
  • Line 247: warning: exported function GetRestServiceName should have comment or be unexported (golint)
  • Line 256: warning: exported function GetRestServicePort should have comment or be unexported (golint)
  • Line 259: warning: exported function GetSiteLabel should have comment or be unexported (golint)
  • Line 263: warning: exported function GetRackLabel should have comment or be unexported (golint)
  • Line 267: warning: exported function ShouldInstallCRDs should have comment or be unexported (golint)
  • Line 271: warning: exported function ShouldEnableWebhooks should have comment or be unexported (golint)
  • Line 275: warning: exported function ShouldUseSelfSignedCerts should have comment or be unexported (golint)
  • Line 279: warning: exported function ShouldUseCertManager should have comment or be unexported (golint)
  • Line 283: warning: exported function GetNamespace should have comment or be unexported (golint)
  • Line 287: warning: exported function GetWebhookCertDir should have comment or be unexported (golint)
  • Line 291: warning: exported function GetCACertRotateBefore should have comment or be unexported (golint)
  • Line 295: warning: exported function GetWebhookServiceDNSNames should have comment or be unexported (golint)
  • Line 311: warning: exported function DetectKubernetesVersion should have comment or be unexported (golint)

• coherence-operator/controllers/coherence_controller.go
  • Line 67: warning: exported method CoherenceReconciler.Reconcile should have comment or be unexported (golint)
  • Line 238: warning: exported method CoherenceReconciler.SetupWithManager should have comment or be unexported (golint)
  • Line 296: warning: exported method CoherenceReconciler.GetReconciler should have comment or be unexported (golint)

• coherence-operator/docgen/main.go
  • Line 153: warning: exported function ProcessFields should have comment or be unexported (golint)

• coherence-operator/test/e2e/helm/helm_helpers.go
  • Line 125: warning: exported type HelmInstallResult should have comment or be unexported (golint)
  • Line 131: warning: exported type HelmInstallResultFilter should have comment or be unexported (golint)
  • Line 133: warning: exported method HelmInstallResult.ToString should have comment or be unexported (golint)
  • Line 157: warning: exported method HelmInstallResult.Size should have comment or be unexported (golint)
  • Line 163: warning: exported method HelmInstallResult.Get should have comment or be unexported (golint)
  • Line 200: warning: exported method HelmInstallResult.List should have comment or be unexported (golint)

• coherence-operator/test/coherence_compatibility/certify_persistence_test.go
  • Line 8: warning: don't use an underscore in package name (golint)

• coherence-operator/pkg/utils/arrays.go
  • Line 10: warning: exported function StringArrayContains should have comment or be unexported (golint)
  • Line 19: warning: exported function StringArrayDoesNotContain should have comment or be unexported (golint)

• coherence-operator/api/v1/coherenceresourcespec_types.go
  • Line 683: warning: exported method CoherenceResourceSpec.GetImagePullSecrets should have comment or be unexported (golint)
  • Line 963: warning: exported method CoherenceResourceSpec.GetMetricsPort should have comment or be unexported (golint)
  • Line 970: warning: exported method CoherenceResourceSpec.GetManagementPort should have comment or be unexported (golint)

• coherence-operator/test/e2e/helper/helm_types.go
  • Line 18: warning: exported type OperatorValues should have comment or be unexported (golint)
  • Line 65: warning: comment on exported method OperatorValues.SetEnableClusterRole should be of the form "SetEnableClusterRole ..." (golint)
  • Line 99: warning: comment on exported method OperatorValues.ToMap should be of the form "ToMap ..." (golint)

ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

misspell99%

Misspell Finds commonly misspelled English words

• coherence-operator/test/e2e/helm/helm_test.go
  • Line 195: warning: "shoudl" is a misspelling of "should" (misspell)
  • Line 208: warning: "shoudl" is a misspelling of "should" (misspell)