Preparing report...

Report for github.com/weaveworks/eksctl

A+    Excellent!    Found 130 issues across 562 files

Tweet

gofmt99%

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


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!


golint79%

Golint is a linter for Go source code.

    • eksctl/pkg/flux/flux.go
    • Line 20: warning: exported type Client should have comment or be unexported (golint)
    • Line 25: warning: exported function NewClient should have comment or be unexported (golint)
    • Line 32: warning: exported method Client.PreFlight should have comment or be unexported (golint)
    • Line 52: warning: exported method Client.Bootstrap should have comment or be unexported (golint)
    • eksctl/pkg/cfn/manager/delete_tasks.go
    • Line 127: warning: exported type DeleteWaitCondition should have comment or be unexported (golint)
    • Line 133: warning: exported type DeleteUnownedNodegroupTask should have comment or be unexported (golint)
    • Line 141: warning: exported method DeleteUnownedNodegroupTask.Describe should have comment or be unexported (golint)
    • Line 145: warning: exported method DeleteUnownedNodegroupTask.Do should have comment or be unexported (golint)
    • Line 176: warning: exported method StackCollection.NewTaskToDeleteUnownedNodeGroup should have comment or be unexported (golint)
    • eksctl/pkg/gitops/gitops.go
    • Line 24: warning: exported type FluxInstaller should have comment or be unexported (golint)
    • Line 78: warning: comment on exported function DeleteKey should be of the form "DeleteKey ..." (golint)
    • eksctl/pkg/actions/flux/enable.go
    • Line 16: warning: comment on exported type InstallerClient should be of the form "InstallerClient ..." (with optional leading article) (golint)
    • Line 23: warning: exported type Installer should have comment or be unexported (golint)
    • Line 29: warning: exported function New should have comment or be unexported (golint)
    • Line 48: warning: exported method Installer.Run should have comment or be unexported (golint)
    • eksctl/pkg/addons/default/kube_proxy.go
    • Line 27: warning: exported const ArchBetaLabel should have comment (or a comment on this block) or be unexported (golint)
    • Line 31: warning: exported function IsKubeProxyUpToDate should have comment or be unexported (golint)
    • eksctl/pkg/utils/tasks/tasks.go
    • Line 17: warning: exported type GenericTask should have comment or be unexported (golint)
    • Line 22: warning: exported method GenericTask.Describe should have comment or be unexported (golint)
    • Line 25: warning: exported method GenericTask.Do should have comment or be unexported (golint)
    • Line 30: warning: exported type SynchronousTaskIface should have comment or be unexported (golint)
    • Line 35: warning: exported type SynchronousTask should have comment or be unexported (golint)
    • Line 39: warning: exported method SynchronousTask.Do should have comment or be unexported (golint)
    • Line 192: warning: exported type TaskWithoutParams should have comment or be unexported (golint)
    • Line 197: warning: exported method TaskWithoutParams.Describe should have comment or be unexported (golint)
    • Line 198: warning: exported method TaskWithoutParams.Do should have comment or be unexported (golint)
    • Line 200: warning: exported type TaskWithNameParam should have comment or be unexported (golint)
    • Line 206: warning: exported method TaskWithNameParam.Describe should have comment or be unexported (golint)
    • Line 207: warning: exported method TaskWithNameParam.Do should have comment or be unexported (golint)
    • eksctl/pkg/actions/cluster/unowned.go
    • Line 31: warning: exported type UnownedCluster should have comment or be unexported (golint)
    • Line 38: warning: exported function NewUnownedCluster should have comment or be unexported (golint)
    • Line 49: warning: exported method UnownedCluster.Upgrade should have comment or be unexported (golint)
    • Line 60: warning: exported method UnownedCluster.Delete should have comment or be unexported (golint)
    • eksctl/pkg/nodebootstrap/legacy/ubuntu.go
    • Line 16: warning: exported type UbuntuBootstrapper should have comment or be unexported (golint)
    • Line 21: warning: exported function NewUbuntuBootstrapper should have comment or be unexported (golint)
    • Line 28: warning: exported method UbuntuBootstrapper.UserData should have comment or be unexported (golint)
    • eksctl/pkg/git/git.go
    • Line 1: warning: package comment should be of the form "Package git ..." (golint)
    • Line 193: warning: exported method Client.DeleteRemoteBranch should have comment or be unexported (golint)
    • eksctl/pkg/actions/identityproviders/disassociate.go
    • Line 15: warning: exported type DisassociateIdentityProvidersOptions should have comment or be unexported (golint)
    • Line 20: warning: exported type DisassociateIdentityProvider should have comment or be unexported (golint)
    • Line 25: warning: exported method Manager.Disassociate should have comment or be unexported (golint)
    • eksctl/pkg/actions/identityproviders/tasks.go
    • Line 9: warning: exported type AssociateProvidersTask should have comment or be unexported (golint)
    • Line 15: warning: exported function NewAssociateProvidersTask should have comment or be unexported (golint)
    • Line 25: warning: exported method AssociateProvidersTask.Describe should have comment or be unexported (golint)
    • Line 29: warning: exported method AssociateProvidersTask.Do should have comment or be unexported (golint)
    • eksctl/pkg/nodebootstrap/al2.go
    • Line 13: warning: exported type AmazonLinux2 should have comment or be unexported (golint)
    • Line 18: warning: exported function NewAL2Bootstrapper should have comment or be unexported (golint)
    • Line 25: warning: exported method AmazonLinux2.UserData should have comment or be unexported (golint)
    • eksctl/pkg/apis/eksctl.io/v1alpha5/identity_provider.go
    • Line 9: warning: exported type IdentityProviderType should have comment or be unexported (golint)
    • Line 12: warning: exported const OIDCIdentityProviderType should have comment (or a comment on this block) or be unexported (golint)
    • Line 41: warning: don't use underscores in Go names; struct field type_ should be type (golint)
    • Line 45: warning: exported function FromIdentityProvider should have comment or be unexported (golint)
    • Line 52: warning: exported method IdentityProvider.UnmarshalJSON should have comment or be unexported (golint)
    • Line 91: warning: exported method OIDCIdentityProvider.DeepCopyIdentityProviderInterface should have comment or be unexported (golint)
    • Line 95: warning: exported method OIDCIdentityProvider.Type should have comment or be unexported (golint)
    • eksctl/pkg/actions/nodegroup/create.go
    • Line 25: warning: comment on exported type CreateOpts should be of the form "CreateOpts ..." (with optional leading article) (golint)
    • Line 35: warning: exported method Manager.Create should have comment or be unexported (golint)
    • eksctl/pkg/actions/label/label.go
    • Line 9: warning: comment on exported type Service should be of the form "Service ..." (with optional leading article) (golint)
    • Line 16: warning: exported type Manager should have comment or be unexported (golint)
    • Line 22: warning: exported function New should have comment or be unexported (golint)
    • eksctl/pkg/ctl/ctltest/utils.go
    • Line 15: warning: exported type CommandFunction should have comment or be unexported (golint)
    • Line 35: warning: exported type MockCmd should have comment or be unexported (golint)
    • Line 40: warning: exported method MockCmd.Execute should have comment or be unexported (golint)
    • eksctl/pkg/cfn/builder/fakes/fake_cfn_template.go
    • Line 7: warning: exported type FakeTemplate should have comment or be unexported (golint)
    • Line 19: warning: exported type Tag should have comment or be unexported (golint)
    • Line 26: warning: exported type Properties should have comment or be unexported (golint)
    • Line 115: warning: exported type SGIngress should have comment or be unexported (golint)
    • Line 123: warning: exported type LaunchTemplateData should have comment or be unexported (golint)
    • Line 145: warning: exported type Placement should have comment or be unexported (golint)
    • Line 149: warning: exported type BlockDeviceMappings should have comment or be unexported (golint)
    • Line 154: warning: exported type MetadataOptions should have comment or be unexported (golint)
    • Line 159: warning: exported type TagSpecification should have comment or be unexported (golint)
    • Line 164: warning: exported type NetworkInterface should have comment or be unexported (golint)
    • eksctl/pkg/cfn/manager/api.go
    • Line 355: warning: comment on exported method StackCollection.ListClusterStackNames should be of the form "ListClusterStackNames ..." (golint)
    • Line 558: warning: exported method StackCollection.HasClusterStack should have comment or be unexported (golint)
    • Line 566: warning: exported method StackCollection.HasClusterStackUsingCachedList should have comment or be unexported (golint)
    • eksctl/pkg/nodebootstrap/bottlerocket.go
    • Line 13: warning: exported type Bottlerocket should have comment or be unexported (golint)
    • Line 18: warning: exported function NewBottlerocketBootstrapper should have comment or be unexported (golint)
    • Line 25: warning: comment on exported method Bottlerocket.UserData should be of the form "UserData ..." (golint)
    • eksctl/pkg/nodebootstrap/ubuntu.go
    • Line 13: warning: exported type Ubuntu should have comment or be unexported (golint)
    • Line 18: warning: exported function NewUbuntuBootstrapper should have comment or be unexported (golint)
    • Line 25: warning: exported method Ubuntu.UserData should have comment or be unexported (golint)
    • eksctl/pkg/cfn/manager/iam.go
    • Line 125: warning: exported method StackCollection.GetIAMAddonsStacks should have comment or be unexported (golint)
    • Line 144: warning: exported method StackCollection.GetIAMAddonName should have comment or be unexported (golint)
    • eksctl/pkg/actions/repo/enable.go
    • Line 1: warning: package comment should be of the form "Package repo ..." (golint)
    • Line 40: warning: exported type Installer should have comment or be unexported (golint)
    • Line 49: warning: exported function New should have comment or be unexported (golint)
    • Line 73: warning: exported method Installer.Run should have comment or be unexported (golint)
    • Line 299: warning: exported method Installer.GetManifests should have comment or be unexported (golint)
    • eksctl/pkg/drain/nodegroup.go
    • Line 28: warning: comment on exported type Evictor should be of the form "Evictor ..." (with optional leading article) (golint)
    • Line 36: warning: exported type NodeGroupDrainer should have comment or be unexported (golint)
    • Line 44: warning: exported function NewNodeGroupDrainer should have comment or be unexported (golint)
    • eksctl/pkg/cfn/template/matchers/matchers.go
    • Line 11: warning: should not use dot imports (golint)
    • Line 26: warning: exported type Loader should have comment or be unexported (golint)
    • Line 33: warning: exported function LoadBytesWithoutErrors should have comment or be unexported (golint)
    • Line 41: warning: exported function LoadStringWithoutErrors should have comment or be unexported (golint)
    • Line 49: warning: exported function LoadFileWithoutErrors should have comment or be unexported (golint)
    • Line 57: warning: exported method Loader.Match should have comment or be unexported (golint)
    • Line 78: warning: exported method Loader.FailureMessage should have comment or be unexported (golint)
    • Line 82: warning: exported method Loader.NegatedFailureMessage should have comment or be unexported (golint)
    • Line 86: warning: exported type ResourceNameAndTypeMatcher should have comment or be unexported (golint)
    • Line 92: warning: exported function HaveResource should have comment or be unexported (golint)
    • Line 101: warning: exported method ResourceNameAndTypeMatcher.Match should have comment or be unexported (golint)
    • Line 120: warning: exported method ResourceNameAndTypeMatcher.FailureMessage should have comment or be unexported (golint)
    • Line 124: warning: exported method ResourceNameAndTypeMatcher.NegatedFailureMessage should have comment or be unexported (golint)
    • Line 128: warning: exported type ResourcePropertiesMatcher should have comment or be unexported (golint)
    • Line 135: warning: exported function HaveResourceWithProperties should have comment or be unexported (golint)
    • Line 144: warning: exported method ResourcePropertiesMatcher.Match should have comment or be unexported (golint)
    • Line 166: warning: exported method ResourcePropertiesMatcher.FailureMessage should have comment or be unexported (golint)
    • Line 170: warning: exported method ResourcePropertiesMatcher.NegatedFailureMessage should have comment or be unexported (golint)
    • Line 174: warning: exported type ResourceWithPropertyValueMatcher should have comment or be unexported (golint)
    • Line 181: warning: exported function HaveResourceWithPropertyValue should have comment or be unexported (golint)
    • Line 191: warning: exported method ResourceWithPropertyValueMatcher.Match should have comment or be unexported (golint)
    • Line 223: warning: exported method ResourceWithPropertyValueMatcher.FailureMessage should have comment or be unexported (golint)
    • Line 227: warning: exported method ResourceWithPropertyValueMatcher.NegatedFailureMessage should have comment or be unexported (golint)
    • Line 231: warning: exported type OutputsMatcher should have comment or be unexported (golint)
    • Line 235: warning: exported function HaveOutputs should have comment or be unexported (golint)
    • Line 241: warning: exported method OutputsMatcher.Match should have comment or be unexported (golint)
    • Line 258: warning: exported method OutputsMatcher.FailureMessage should have comment or be unexported (golint)
    • Line 262: warning: exported method OutputsMatcher.NegatedFailureMessage should have comment or be unexported (golint)
    • Line 266: warning: exported type OutputValueMatcher should have comment or be unexported (golint)
    • Line 272: warning: exported function HaveOutputWithValue should have comment or be unexported (golint)
    • Line 281: warning: exported method OutputValueMatcher.Match should have comment or be unexported (golint)
    • Line 301: warning: exported method OutputValueMatcher.FailureMessage should have comment or be unexported (golint)
    • Line 305: warning: exported method OutputValueMatcher.NegatedFailureMessage should have comment or be unexported (golint)
    • Line 309: warning: exported type OutputExportNameMatcher should have comment or be unexported (golint)
    • Line 315: warning: exported function HaveOutputExportedAs should have comment or be unexported (golint)
    • Line 324: warning: exported method OutputExportNameMatcher.Match should have comment or be unexported (golint)
    • Line 348: warning: exported method OutputExportNameMatcher.FailureMessage should have comment or be unexported (golint)
    • Line 352: warning: exported method OutputExportNameMatcher.NegatedFailureMessage should have comment or be unexported (golint)
    • eksctl/pkg/apis/eksctl.io/v1alpha5/vpc.go
    • Line 34: warning: exported function NewAZSubnetMapping should have comment or be unexported (golint)
    • Line 38: warning: exported function AZSubnetMappingFromMap should have comment or be unexported (golint)
    • Line 49: warning: exported method AZSubnetMapping.Set should have comment or be unexported (golint)
    • Line 56: warning: exported method AZSubnetMapping.SetAZ should have comment or be unexported (golint)
    • Line 170: warning: exported type AZSubnetSpec should have comment or be unexported (golint)
    • Line 405: warning: exported method ClusterConfig.HasPrivateEndpointAccess should have comment or be unexported (golint)
    • eksctl/pkg/eks/api.go
    • Line 60: warning: comment on exported type KubeProvider should be of the form "KubeProvider ..." (with optional leading article) (golint)
    • Line 137: warning: exported method ProviderServices.ConfigProvider should have comment or be unexported (golint)
    • Line 141: warning: exported method ProviderServices.Session should have comment or be unexported (golint)
    • eksctl/pkg/eks/fargate.go
    • Line 19: warning: comment on exported type FargateClient should be of the form "FargateClient ..." (with optional leading article) (golint)
    • Line 88: warning: exported function ScheduleCoreDNSOnFargateIfRelevant should have comment or be unexported (golint)
    • eksctl/pkg/apis/eksctl.io/v1alpha5/types.go
    • Line 229: warning: exported const EKSNodeGroupNameLabel should have comment (or a comment on this block) or be unexported (golint)
    • Line 524: warning: exported type EKSCTLCreated should have comment or be unexported (golint)
    • Line 883: warning: exported method NodeGroup.InstanceTypeList should have comment or be unexported (golint)
    • Line 924: warning: comment on exported function NewGit should be of the form "NewGit ..." (golint)
    • Line 1030: warning: comment on exported method ClusterConfig.HasBootstrapProfile should be of the form "HasBootstrapProfile ..." (golint)
    • Line 1036: warning: comment on exported method ClusterConfig.HasGitopsRepoConfigured should be of the form "HasGitopsRepoConfigured ..." (golint)
    • Line 1378: warning: exported type LaunchTemplate should have comment or be unexported (golint)
    • Line 1426: warning: exported method ManagedNodeGroup.InstanceTypeList should have comment or be unexported (golint)
    • Line 1433: warning: exported method ManagedNodeGroup.ListOptions should have comment or be unexported (golint)
    • eksctl/pkg/actions/addon/get.go
    • Line 15: warning: exported type Summary should have comment or be unexported (golint)
    • Line 24: warning: exported method Manager.Get should have comment or be unexported (golint)
    • Line 66: warning: exported method Manager.GetAll should have comment or be unexported (golint)
    • eksctl/pkg/addons/device_plugin.go
    • Line 64: warning: exported type MkDevicePlugin should have comment or be unexported (golint)
    • Line 66: warning: exported type DevicePlugin should have comment or be unexported (golint)
    • Line 131: warning: exported method NeuronDevicePlugin.RawClient should have comment or be unexported (golint)
    • Line 135: warning: exported method NeuronDevicePlugin.PlanMode should have comment or be unexported (golint)
    • Line 139: warning: exported method NeuronDevicePlugin.Manifest should have comment or be unexported (golint)
    • Line 143: warning: exported method NeuronDevicePlugin.SetImage should have comment or be unexported (golint)
    • Line 168: warning: exported method NvidiaDevicePlugin.RawClient should have comment or be unexported (golint)
    • Line 172: warning: exported method NvidiaDevicePlugin.PlanMode should have comment or be unexported (golint)
    • Line 176: warning: exported method NvidiaDevicePlugin.SetImage should have comment or be unexported (golint)
    • Line 180: warning: exported method NvidiaDevicePlugin.Manifest should have comment or be unexported (golint)
    • Line 196: warning: exported method EFADevicePlugin.RawClient should have comment or be unexported (golint)
    • Line 200: warning: exported method EFADevicePlugin.PlanMode should have comment or be unexported (golint)
    • Line 204: warning: exported method EFADevicePlugin.Manifest should have comment or be unexported (golint)
    • Line 208: warning: exported method EFADevicePlugin.SetImage should have comment or be unexported (golint)
    • eksctl/pkg/nodebootstrap/windows.go
    • Line 15: warning: exported type Windows should have comment or be unexported (golint)
    • Line 20: warning: exported function NewWindowsBootstrapper should have comment or be unexported (golint)
    • Line 27: warning: exported method Windows.UserData should have comment or be unexported (golint)
    • eksctl/pkg/ctl/cmdutils/gitops.go
    • Line 38: warning: comment on exported function AddCommonFlagsForFlux should be of the form "AddCommonFlagsForFlux ..." (golint)
    • Line 63: warning: comment on exported function AddCommonFlagsForGitRepo should be of the form "AddCommonFlagsForGitRepo ..." (golint)
    • Line 87: warning: comment on exported type GitConfigLoader should be of the form "GitConfigLoader ..." (with optional leading article) (golint)
    • Line 99: warning: comment on exported function NewGitConfigLoader should be of the form "NewGitConfigLoader ..." (golint)
    • Line 169: warning: comment on exported method GitConfigLoader.WithRepoValidation should be of the form "WithRepoValidation ..." (golint)
    • Line 208: warning: comment on exported method GitConfigLoader.WithProfileValidation should be of the form "WithProfileValidation ..." (golint)
    • Line 231: warning: comment on exported method GitConfigLoader.Load should be of the form "Load ..." (golint)
    • eksctl/pkg/actions/cluster/owned.go
    • Line 18: warning: exported type OwnedCluster should have comment or be unexported (golint)
    • Line 25: warning: exported function NewOwnedCluster should have comment or be unexported (golint)
    • Line 36: warning: exported method OwnedCluster.Upgrade should have comment or be unexported (golint)
    • Line 69: warning: exported method OwnedCluster.Delete should have comment or be unexported (golint)
    • eksctl/pkg/vpc/vpc.go
    • Line 79: warning: exported function SplitInto16 should have comment or be unexported (golint)
    • Line 104: warning: exported function SplitInto8 should have comment or be unexported (golint)
    • Line 380: warning: exported function ValidateLegacySubnetsForNodeGroups should have comment or be unexported (golint)
    • Line 548: warning: exported function SelectNodeGroupSubnets should have comment or be unexported (golint)
    • eksctl/pkg/executor/executor.go
    • Line 11: warning: comment on exported type Executor should be of the form "Executor ..." (with optional leading article) (golint)
    • Line 19: warning: exported type EnvVars should have comment or be unexported (golint)
    • eksctl/pkg/cfn/builder/fargate.go
    • Line 37: warning: exported method FargateResourceSet.AddAllResources should have comment or be unexported (golint)
    • Line 48: warning: exported method FargateResourceSet.WithIAM should have comment or be unexported (golint)
    • Line 52: warning: exported method FargateResourceSet.WithNamedIAM should have comment or be unexported (golint)
    • Line 56: warning: exported method FargateResourceSet.RenderJSON should have comment or be unexported (golint)
    • Line 60: warning: exported method FargateResourceSet.GetAllOutputs should have comment or be unexported (golint)
    • eksctl/pkg/ctl/cmdutils/builder.go
    • Line 5: warning: exported type ValidateCmdFunc should have comment or be unexported (golint)
    • Line 7: warning: exported type ConfigLoaderBuilder should have comment or be unexported (golint)
    • Line 15: warning: exported method ConfigLoaderBuilder.ValidateWithoutConfigFile should have comment or be unexported (golint)
    • Line 19: warning: exported method ConfigLoaderBuilder.ValidateWithConfigFile should have comment or be unexported (golint)
    • Line 22: warning: exported method ConfigLoaderBuilder.Validate should have comment or be unexported (golint)
    • Line 26: warning: exported method ConfigLoaderBuilder.Build should have comment or be unexported (golint)
    • Line 50: warning: exported function NewConfigLoaderBuilder should have comment or be unexported (golint)
    • eksctl/pkg/testutils/mockprovider/mock_provider.go
    • Line 37: warning: exported type MockAWSClient should have comment or be unexported (golint)
    • Line 41: warning: exported type MockInput should have comment or be unexported (golint)
    • Line 42: warning: exported type MockOutput should have comment or be unexported (golint)
    • Line 45: warning: exported type MockState should have comment or be unexported (golint)
    • Line 182: warning: exported method MockProvider.Session should have comment or be unexported (golint)
    • Line 186: warning: exported function NewMockAWSClient should have comment or be unexported (golint)
    • Line 201: warning: exported method MockAWSClient.MockRequestForMockOutput should have comment or be unexported (golint)
    • Line 218: warning: exported method MockAWSClient.MockRequestForGivenOutput should have comment or be unexported (golint)
    • eksctl/pkg/schema/test/test.go
    • Line 31: warning: exported type SumTypeA should have comment or be unexported (golint)
    • Line 35: warning: exported type SumTypeB should have comment or be unexported (golint)
    • Line 39: warning: comment on exported type SumType should be of the form "SumType ..." (with optional leading article) (golint)
    • eksctl/pkg/ctl/cmdutils/addon.go
    • Line 15: warning: exported function NewCreateOrUpgradeAddonLoader should have comment or be unexported (golint)
    • Line 42: warning: exported function NewDeleteAddonLoader should have comment or be unexported (golint)
    • eksctl/pkg/eks/nodegroup_service.go
    • Line 25: warning: comment on exported type InstanceSelector should be of the form "InstanceSelector ..." (with optional leading article) (golint)
    • Line 32: warning: comment on exported type NodeGroupInitialiser should be of the form "NodeGroupInitialiser ..." (with optional leading article) (golint)
    • Line 218: warning: exported method NodeGroupService.ValidateLegacySubnetsForNodeGroups should have comment or be unexported (golint)
    • eksctl/pkg/testutils/client.go
    • Line 9: warning: should not use dot imports (golint)
    • Line 25: warning: exported function LoadSamples should have comment or be unexported (golint)
    • Line 47: warning: exported function NewFakeClientSetWithSamples should have comment or be unexported (golint)
    • Line 54: warning: exported type CollectionTracker should have comment or be unexported (golint)
    • Line 61: warning: exported function NewCollectionTracker should have comment or be unexported (golint)
    • Line 70: warning: exported type RequestTracker should have comment or be unexported (golint)
    • Line 93: warning: exported method RequestTracker.Append should have comment or be unexported (golint)
    • Line 95: warning: exported method RequestTracker.Methods should have comment or be unexported (golint)
    • Line 102: warning: exported method RequestTracker.IsMissing should have comment or be unexported (golint)
    • Line 111: warning: exported method RequestTracker.Create should have comment or be unexported (golint)
    • Line 126: warning: exported method CollectionTracker.Created should have comment or be unexported (golint)
    • Line 128: warning: exported method CollectionTracker.CreatedItems should have comment or be unexported (golint)
    • Line 135: warning: exported method RequestTracker.Update should have comment or be unexported (golint)
    • Line 145: warning: exported method CollectionTracker.Updated should have comment or be unexported (golint)
    • Line 147: warning: exported method CollectionTracker.UpdatedItems should have comment or be unexported (golint)
    • Line 154: warning: exported method RequestTracker.Delete should have comment or be unexported (golint)
    • Line 170: warning: exported method CollectionTracker.Deleted should have comment or be unexported (golint)
    • Line 172: warning: exported method CollectionTracker.DeletedItems should have comment or be unexported (golint)
    • Line 179: warning: exported method CollectionTracker.AllTracked should have comment or be unexported (golint)
    • Line 181: warning: exported method CollectionTracker.AllTrackedItems should have comment or be unexported (golint)
    • Line 188: warning: exported function NewFakeRawResource should have comment or be unexported (golint)
    • Line 266: warning: exported type FakeRawClient should have comment or be unexported (golint)
    • Line 273: warning: exported function NewFakeRawClient should have comment or be unexported (golint)
    • Line 279: warning: exported method FakeRawClient.ClientSet should have comment or be unexported (golint)
    • Line 292: warning: exported method FakeRawClient.NewRawResource should have comment or be unexported (golint)
    • Line 297: warning: exported method FakeRawClient.ClearUpdated should have comment or be unexported (golint)
    • eksctl/pkg/nodebootstrap/legacy/al2.go
    • Line 14: warning: exported type AL2Bootstrapper should have comment or be unexported (golint)
    • Line 19: warning: exported function NewAL2Bootstrapper should have comment or be unexported (golint)
    • Line 26: warning: exported method AL2Bootstrapper.UserData should have comment or be unexported (golint)
    • eksctl/pkg/drain/evictor/pod.go
    • Line 44: warning: exported type PodDelete should have comment or be unexported (golint)
    • Line 49: warning: exported type PodDeleteList should have comment or be unexported (golint)
    • Line 53: warning: exported method PodDeleteList.Pods should have comment or be unexported (golint)
    • Line 63: warning: exported method PodDeleteList.Warnings should have comment or be unexported (golint)
    • Line 96: warning: exported type PodDeleteStatus should have comment or be unexported (golint)

gocyclo96%

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.

    • eksctl/pkg/apis/eksctl.io/v1alpha5/validation.go
    • Line 532: warning: cyclomatic complexity 61 of function ValidateManagedNodeGroup() is high (> 15) (gocyclo)
    • Line 59: warning: cyclomatic complexity 33 of function ValidateClusterConfig() is high (> 15) (gocyclo)
    • Line 694: warning: cyclomatic complexity 30 of function validateInstancesDistribution() is high (> 15) (gocyclo)
    • Line 330: warning: cyclomatic complexity 27 of function ValidateNodeGroup() is high (> 15) (gocyclo)
    • Line 203: warning: cyclomatic complexity 22 of function validateNodeGroupBase() is high (> 15) (gocyclo)
    • Line 264: warning: cyclomatic complexity 19 of function validateVolumeOpts() is high (> 15) (gocyclo)

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!


misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!