gofmt96%

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

• arena/pkg/operators/tf-operator/apis/tensorflow/v1beta2/defaults_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• arena/pkg/operators/tf-operator/apis/tensorflow/v1/defaults_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• arena/pkg/operators/tf-operator/apis/tensorflow/validation/validation_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• arena/pkg/apis/types/serving.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• arena/pkg/argsbuilder/util.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• arena/pkg/operators/et-operator/api/common/zz_generated.deepcopy.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• arena/cmd/arena/main.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• arena/pkg/serving/serving_seldon.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• arena/pkg/topnode/gputopology.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• arena/pkg/apis/utils/utils.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• arena/pkg/operators/et-operator/client/clientset/versioned/typed/et/v1alpha1/generated_expansion.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• arena/pkg/serving/traffic_router_split.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• arena/pkg/apis/cron/cron_tfjob_builder.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• arena/pkg/apis/serving/seldon_builder.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• arena/pkg/argsbuilder/cron_tfjob.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• arena/pkg/operators/pytorch-operator/apis/pytorch/validation/validation_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

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!

gocyclo98%

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.

• arena/pkg/apis/utils/pods.go
  • Line 59: warning: cyclomatic complexity 28 of function DefinePodPhaseStatus() is high (> 15) (gocyclo)

• arena/pkg/operators/pytorch-operator/apis/pytorch/validation/validation.go
  • Line 23: warning: cyclomatic complexity 16 of function ValidateV1PyTorchJobSpec() is high (> 15) (gocyclo)

• arena/pkg/operators/spark-operator/apis/sparkoperator.k8s.io/v1beta2/zz_generated.deepcopy.go
  • Line 688: warning: cyclomatic complexity 37 of function (*SparkPodSpec).DeepCopyInto() is high (> 15) (gocyclo)
  • Line 533: warning: cyclomatic complexity 25 of function (*SparkApplicationSpec).DeepCopyInto() is high (> 15) (gocyclo)

• arena/pkg/operators/spark-operator/apis/sparkoperator.k8s.io/v1beta1/zz_generated.deepcopy.go
  • Line 667: warning: cyclomatic complexity 30 of function (*SparkPodSpec).DeepCopyInto() is high (> 15) (gocyclo)
  • Line 517: warning: cyclomatic complexity 24 of function (*SparkApplicationSpec).DeepCopyInto() is high (> 15) (gocyclo)

• arena/pkg/prometheus/prom.go
  • Line 95: warning: cyclomatic complexity 17 of function generateNodeGPUMetrics() is high (> 15) (gocyclo)

• arena/pkg/topnode/summary.go
  • Line 25: warning: cyclomatic complexity 17 of function DisplayNodeSummary() is high (> 15) (gocyclo)

golint41%

Golint is a linter for Go source code.

• arena/pkg/apis/training/scalein_etjob_builder.go
  • Line 11: warning: exported type ScaleInETJobBuilder should have comment or be unexported (golint)
  • Line 17: warning: exported function NewScaleInETJobBuilder should have comment or be unexported (golint)
  • Line 48: warning: comment on exported method ScaleInETJobBuilder.Count should be of the form "Count ..." (golint)

• arena/pkg/apis/serving/tensorflow_builder.go
  • Line 11: warning: exported type TFServingJobBuilder should have comment or be unexported (golint)
  • Line 17: warning: exported function NewTFServingJobBuilder should have comment or be unexported (golint)

• arena/pkg/argsbuilder/cron_tfjob.go
  • Line 11: warning: exported type CronTFJobArgsBuilder should have comment or be unexported (golint)
  • Line 17: warning: exported function NewCronTFJobArgsBuilder should have comment or be unexported (golint)
  • Line 30: warning: exported method CronTFJobArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 35: warning: exported method CronTFJobArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 42: warning: exported method CronTFJobArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 50: warning: exported method CronTFJobArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 62: warning: exported method CronTFJobArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 71: warning: exported method CronTFJobArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/commands/training/delete.go
  • Line 31: warning: comment on exported function NewDeleteCommand should be of the form "NewDeleteCommand ..." (golint)

• arena/pkg/apis/training/job.go
  • Line 38: warning: exported function NewJob should have comment or be unexported (golint)
  • Line 44: warning: exported method Job.Type should have comment or be unexported (golint)

• arena/pkg/workflow/workflow.go
  • Line 16: warning: exported function DeleteJob should have comment or be unexported (golint)
  • Line 45: warning: exported function SubmitOps should have comment or be unexported (golint)
  • Line 109: warning: exported function SubmitJob should have comment or be unexported (golint)

• arena/pkg/argsbuilder/submit_tfjob.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 31: warning: exported type SubmitTFJobArgsBuilder should have comment or be unexported (golint)
  • Line 37: warning: exported function NewSubmitTFJobArgsBuilder should have comment or be unexported (golint)
  • Line 52: warning: exported method SubmitTFJobArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 57: warning: exported method SubmitTFJobArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 64: warning: exported method SubmitTFJobArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 72: warning: exported method SubmitTFJobArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 158: warning: exported method SubmitTFJobArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 168: warning: exported method SubmitTFJobArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/apis/types/const.go
  • Line 4: warning: comment on exported const NvidiaGPUResourceName should be of the form "NvidiaGPUResourceName ..." (golint)
  • Line 9: warning: exported const GPUShareResourceName should have comment (or a comment on this block) or be unexported (golint)
  • Line 17: warning: exported const AliyunGPUResourceName should have comment (or a comment on this block) or be unexported (golint)

• arena/pkg/apis/types/submit_etjob.go
  • Line 3: warning: exported type SubmitETJobArgs should have comment or be unexported (golint)
  • Line 16: warning: exported type ScaleETJobArgs should have comment or be unexported (golint)
  • Line 35: warning: exported type ScaleInETJobArgs should have comment or be unexported (golint)
  • Line 40: warning: exported type ScaleOutETJobArgs should have comment or be unexported (golint)

• arena/pkg/apis/logger/logger.go
  • Line 12: warning: exported type LoggerBuilder should have comment or be unexported (golint)
  • Line 18: warning: exported function NewLoggerBuilder should have comment or be unexported (golint)
  • Line 30: warning: exported method LoggerBuilder.SinceSeconds should have comment or be unexported (golint)
  • Line 37: warning: exported method LoggerBuilder.SinceTime should have comment or be unexported (golint)
  • Line 44: warning: exported method LoggerBuilder.Instance should have comment or be unexported (golint)
  • Line 49: warning: exported method LoggerBuilder.Container should have comment or be unexported (golint)
  • Line 54: warning: exported method LoggerBuilder.Follow should have comment or be unexported (golint)
  • Line 59: warning: exported method LoggerBuilder.EnableTimestamp should have comment or be unexported (golint)
  • Line 64: warning: exported method LoggerBuilder.Tail should have comment or be unexported (golint)
  • Line 72: warning: exported method LoggerBuilder.WriterCloser should have comment or be unexported (golint)
  • Line 79: warning: exported method LoggerBuilder.Build should have comment or be unexported (golint)

• arena/pkg/apis/arenaclient/data_client.go
  • Line 8: warning: exported type DataClient should have comment or be unexported (golint)
  • Line 30: warning: exported method DataClient.ListAndPrintDataVolumes should have comment or be unexported (golint)

• arena/pkg/commands/root.go
  • Line 10: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)

• arena/pkg/training/dashboard_helper.go
  • Line 79: warning: exported function GetJobDashboards should have comment or be unexported (golint)

• arena/pkg/operators/et-operator/api/common/doc.go
  • Line 18: warning: package comment should be of the form "Package common ..." (golint)

• arena/pkg/commands/cron/cron.go
  • Line 34: warning: comment on exported function NewCronCommand should be of the form "NewCronCommand ..." (golint)

• arena/pkg/commands/cron/cron_tfjob.go
  • Line 13: warning: exported function NewCronTFJobCommand should have comment or be unexported (golint)

• arena/pkg/training/list.go
  • Line 22: warning: exported function ListTrainingJobs should have comment or be unexported (golint)
  • Line 74: warning: exported function DisplayTrainingJobList should have comment or be unexported (golint)
  • Line 141: warning: exported function PrintLine should have comment or be unexported (golint)
  • Line 147: warning: exported function CheckPrintFormat should have comment or be unexported (golint)

• arena/pkg/cron/cron_tfjob.go
  • Line 10: warning: exported function SubmitCronTFJob should have comment or be unexported (golint)

• arena/pkg/argsbuilder/scale_etjob.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 26: warning: exported type ScaleETJobArgsBuilder should have comment or be unexported (golint)
  • Line 32: warning: exported function NewScaleETJobArgsBuilder should have comment or be unexported (golint)
  • Line 41: warning: exported method ScaleETJobArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 46: warning: exported method ScaleETJobArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 53: warning: exported method ScaleETJobArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 61: warning: exported method ScaleETJobArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 83: warning: exported method ScaleETJobArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 98: warning: exported method ScaleETJobArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/operators/et-operator/api/v1alpha1/trainingjob_types.go
  • Line 46: warning: exported type ETReplicaSpecs should have comment or be unexported (golint)
  • Line 51: warning: exported type ETReplicaSpec should have comment or be unexported (golint)

• arena/pkg/commands/serving/serving_kubeflow.go
  • Line 14: warning: exported function NewSubmitKFServingJobCommand should have comment or be unexported (golint)

• arena/pkg/operators/pytorch-operator/apis/pytorch/v1/defaults.go
  • Line 88: warning: don't use underscores in Go names; func SetDefaults_PyTorchJob should be SetDefaultsPyTorchJob (golint)

• arena/pkg/commands/training/submit_etjob.go
  • Line 13: warning: exported function NewSubmitETJobCommand should have comment or be unexported (golint)

• arena/pkg/argsbuilder/submit_volcanojob.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 25: warning: exported type SubmitVolcanoJobArgsBuilder should have comment or be unexported (golint)
  • Line 31: warning: exported function NewSubmitVolcanoJobArgsBuilder should have comment or be unexported (golint)
  • Line 41: warning: exported method SubmitVolcanoJobArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 46: warning: exported method SubmitVolcanoJobArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 53: warning: exported method SubmitVolcanoJobArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 61: warning: exported method SubmitVolcanoJobArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 96: warning: exported method SubmitVolcanoJobArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 105: warning: exported method SubmitVolcanoJobArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/apis/types/serving.go
  • Line 24: warning: exported type ServingTypeInfo should have comment or be unexported (golint)
  • Line 89: warning: exported type Endpoint should have comment or be unexported (golint)
  • Line 98: warning: exported type ServingInstance should have comment or be unexported (golint)
  • Line 123: warning: exported type CommonServingArgs should have comment or be unexported (golint)
  • Line 148: warning: exported type CustomServingArgs should have comment or be unexported (golint)
  • Line 154: warning: exported type TensorFlowServingArgs should have comment or be unexported (golint)
  • Line 165: warning: exported type TensorRTServingArgs should have comment or be unexported (golint)
  • Line 174: warning: exported type KFServingArgs should have comment or be unexported (golint)
  • Line 182: warning: exported type SeldonServingArgs should have comment or be unexported (golint)

• arena/pkg/commands/training/scalein.go
  • Line 23: warning: exported function NewScaleInCommand should have comment or be unexported (golint)

• arena/pkg/operators/volcano-operator/client/clientset/versioned/typed/batch/v1alpha1/fake/fake_batch_client.go
  • Line 25: warning: exported type FakeBatchV1alpha1 should have comment or be unexported (golint)
  • Line 29: warning: exported method FakeBatchV1alpha1.Jobs should have comment or be unexported (golint)

• arena/pkg/training/tensorboard.go
  • Line 86: warning: exported function PrepareServicesAndNodesForTensorboard should have comment or be unexported (golint)

• arena/pkg/training/trainer.go
  • Line 31: warning: exported function GetAllTrainers should have comment or be unexported (golint)
  • Line 62: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
  • Line 66: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
  • Line 70: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
  • Line 76: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
  • Line 80: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
  • Line 90: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
  • Line 154: warning: exported function CheckOperatorIsInstalled should have comment or be unexported (golint)

• arena/pkg/commands/serving/serving.go
  • Line 16: warning: exported function NewServeCommand should have comment or be unexported (golint)

• arena/pkg/apis/types/gpu_metric.go
  • Line 7: warning: exported const PROMETHEUS_INSTALL_DOC_URL should have comment or be unexported (golint)
  • Line 8: warning: exported const KUBE_SYSTEM_NAMESPACE should have comment or be unexported (golint)
  • Line 9: warning: exported const PROMETHEUS_SCHEME should have comment or be unexported (golint)
  • Line 10: warning: exported const PROMETHEUS_SVC_LABEL should have comment or be unexported (golint)
  • Line 11: warning: exported const POD_METRIC_TMP should have comment or be unexported (golint)
  • Line 12: warning: exported const NODE_METRIC_TMP should have comment or be unexported (golint)
  • Line 13: warning: exported const KUBEFLOW_NAMESPACE should have comment or be unexported (golint)
  • Line 15: warning: exported var GPU_METRIC_LIST should have comment or be unexported (golint)
  • Line 28: warning: exported var SUPPORT_PROMETHEUS_SERVERS should have comment or be unexported (golint)
  • Line 69: warning: exported type PrometheusMetric should have comment or be unexported (golint)
  • Line 74: warning: exported type PrometheusMetricData should have comment or be unexported (golint)
  • Line 79: warning: exported type PrometheusMetricResult should have comment or be unexported (golint)
  • Line 84: warning: exported type PrometheusMetricValue should have comment or be unexported (golint)
  • Line 86: warning: exported type GpuMetricInfo should have comment or be unexported (golint)
  • Line 99: warning: exported type JobGpuMetric should have comment or be unexported (golint)
  • Line 101: warning: exported type PodGpuMetric should have comment or be unexported (golint)
  • Line 103: warning: comment on exported type NodeGpuMetric should be of the form "NodeGpuMetric ..." (with optional leading article) (golint)
  • Line 106: warning: exported type GpuMetric should have comment or be unexported (golint)
  • Line 112: warning: exported type AdvancedGpuMetric should have comment or be unexported (golint)

• arena/pkg/apis/utils/training.go
  • Line 8: warning: exported function IsTensorFlowPod should have comment or be unexported (golint)
  • Line 38: warning: exported function IsPyTorchPod should have comment or be unexported (golint)
  • Line 58: warning: exported function IsMPIPod should have comment or be unexported (golint)
  • Line 77: warning: exported function IsHorovodPod should have comment or be unexported (golint)
  • Line 90: warning: exported function IsVolcanoPod should have comment or be unexported (golint)
  • Line 103: warning: exported function IsETPod should have comment or be unexported (golint)
  • Line 119: warning: exported function IsSparkPod should have comment or be unexported (golint)

• arena/pkg/argsbuilder/runtime/tfjob_runtime.go
  • Line 11: warning: exported function NewDefaultTFJobRuntime should have comment or be unexported (golint)

• arena/pkg/training/submit_volcanojob.go
  • Line 12: warning: exported function SubmitVolcanoJob should have comment or be unexported (golint)

• arena/pkg/operators/kubedl-operator/apis/apps/v1alpha1/common.go
  • Line 8: warning: exported var SchemeGroupVersion should have comment or be unexported (golint)

• arena/pkg/operators/spark-operator/apis/sparkoperator.k8s.io/v1beta1/types.go
  • Line 61: warning: exported type RestartPolicyType should have comment or be unexported (golint)
  • Line 64: warning: exported const Never should have comment (or a comment on this block) or be unexported (golint)
  • Line 74: warning: exported type ScheduledSparkApplication should have comment or be unexported (golint)
  • Line 81: warning: exported type ConcurrencyPolicy should have comment or be unexported (golint)
  • Line 93: warning: exported type ScheduledSparkApplicationSpec should have comment or be unexported (golint)
  • Line 114: warning: exported type ScheduleState should have comment or be unexported (golint)
  • Line 117: warning: exported const FailedValidationState should have comment (or a comment on this block) or be unexported (golint)
  • Line 121: warning: exported type ScheduledSparkApplicationStatus should have comment or be unexported (golint)
  • Line 521: warning: exported type GPUSpec should have comment or be unexported (golint)

• arena/pkg/operators/tf-operator/apis/tensorflow/v1alpha2/defaults.go
  • Line 90: warning: don't use underscores in Go names; func SetDefaults_TFJob should be SetDefaultsTFJob (golint)

• arena/pkg/topnode/define.go
  • Line 43: warning: exported type Node should have comment or be unexported (golint)
  • Line 133: warning: comment on exported function GetSupportedNodePorcessers should be of the form "GetSupportedNodePorcessers ..." (golint)
  • Line 232: warning: exported function BuildNodes should have comment or be unexported (golint)
  • Line 310: warning: exported function GetNodeGpuMetrics should have comment or be unexported (golint)

• arena/pkg/operators/tf-operator/apis/tensorflow/v1alpha1/register.go
  • Line 24: warning: comment on exported var SchemeBuilder should be of the form "SchemeBuilder ..." (golint)
  • Line 26: warning: exported var AddToScheme should have comment or be unexported (golint)

• arena/pkg/training/prune.go
  • Line 11: warning: exported type PruneArgs should have comment or be unexported (golint)
  • Line 15: warning: exported function PruneTrainingJobs should have comment or be unexported (golint)

• arena/pkg/argsbuilder/builder.go
  • Line 7: warning: comment on exported type ArgsBuilder should be of the form "ArgsBuilder ..." (with optional leading article) (golint)

• arena/pkg/commands/training/submit_volcanojob.go
  • Line 13: warning: exported function NewVolcanoJobCommand should have comment or be unexported (golint)

• arena/pkg/apis/serving/kubeflow_builder.go
  • Line 11: warning: exported type KFServingJobBuilder should have comment or be unexported (golint)
  • Line 17: warning: exported function NewKFServingJobBuilder should have comment or be unexported (golint)
  • Line 205: warning: comment on exported method KFServingJobBuilder.ModelType should be of the form "ModelType ..." (golint)

• arena/pkg/apis/serving/traffic_router_builder.go
  • Line 10: warning: exported type TrafficRouterBuilder should have comment or be unexported (golint)
  • Line 16: warning: exported function NewTrafficRouterBuilder should have comment or be unexported (golint)

• arena/pkg/operators/tf-operator/apis/tensorflow/v1alpha2/register.go
  • Line 24: warning: comment on exported var SchemeBuilder should be of the form "SchemeBuilder ..." (golint)
  • Line 28: warning: exported var AddToScheme should have comment or be unexported (golint)

• arena/pkg/argsbuilder/scalein_etjob.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 25: warning: exported type ScaleInETJobArgsBuilder should have comment or be unexported (golint)
  • Line 31: warning: exported function NewScaleInETJobArgsBuilder should have comment or be unexported (golint)
  • Line 44: warning: exported method ScaleInETJobArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 49: warning: exported method ScaleInETJobArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 56: warning: exported method ScaleInETJobArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 64: warning: exported method ScaleInETJobArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 70: warning: exported method ScaleInETJobArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 79: warning: exported method ScaleInETJobArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/argsbuilder/submit_etjob.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 25: warning: exported type SubmitETJobArgsBuilder should have comment or be unexported (golint)
  • Line 31: warning: exported function NewSubmitETJobArgsBuilder should have comment or be unexported (golint)
  • Line 46: warning: exported method SubmitETJobArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 51: warning: exported method SubmitETJobArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 58: warning: exported method SubmitETJobArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 66: warning: exported method SubmitETJobArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 76: warning: exported method SubmitETJobArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 86: warning: exported method SubmitETJobArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/commands/serving/traffic_router_split.go
  • Line 27: warning: exported function NewTrafficRouterSplitCommand should have comment or be unexported (golint)

• arena/pkg/commands/training/scaleout_etjob.go
  • Line 13: warning: exported function NewScaleOutETJobCommand should have comment or be unexported (golint)

• arena/pkg/datahouse/list.go
  • Line 37: warning: exported function DisplayDataVolumes should have comment or be unexported (golint)

• arena/pkg/apis/training/mpijob_builder.go
  • Line 11: warning: exported type MPIJobBuilder should have comment or be unexported (golint)
  • Line 17: warning: exported function NewMPIJobBuilder should have comment or be unexported (golint)

• arena/pkg/apis/arenaclient/training_client.go
  • Line 155: warning: exported method TrainingJobClient.Attach should have comment or be unexported (golint)
  • Line 205: warning: exported method TrainingJobClient.Top should have comment or be unexported (golint)

• arena/pkg/training/trainer_interface.go
  • Line 26: warning: comment on exported type TrainingJob should be of the form "TrainingJob ..." (with optional leading article) (golint)
  • Line 76: warning: exported type Trainer should have comment or be unexported (golint)

• arena/pkg/apis/types/submit_sparkjob.go
  • Line 3: warning: exported type SubmitSparkJobArgs should have comment or be unexported (golint)
  • Line 14: warning: exported type Driver should have comment or be unexported (golint)
  • Line 19: warning: exported type Executor should have comment or be unexported (golint)

• arena/pkg/operators/spark-operator/apis/sparkoperator.k8s.io/v1beta2/types.go
  • Line 76: warning: exported type RestartPolicyType should have comment or be unexported (golint)
  • Line 79: warning: exported const Never should have comment (or a comment on this block) or be unexported (golint)
  • Line 90: warning: exported type ScheduledSparkApplication should have comment or be unexported (golint)
  • Line 97: warning: exported type ConcurrencyPolicy should have comment or be unexported (golint)
  • Line 109: warning: exported type ScheduledSparkApplicationSpec should have comment or be unexported (golint)
  • Line 130: warning: exported type ScheduleState should have comment or be unexported (golint)
  • Line 133: warning: exported const FailedValidationState should have comment (or a comment on this block) or be unexported (golint)
  • Line 137: warning: exported type ScheduledSparkApplicationStatus should have comment or be unexported (golint)
  • Line 606: warning: exported type GPUSpec should have comment or be unexported (golint)

• arena/pkg/apis/cron/job.go
  • Line 36: warning: exported function NewJob should have comment or be unexported (golint)
  • Line 42: warning: exported method Job.Type should have comment or be unexported (golint)

• arena/pkg/apis/serving/custom_builder.go
  • Line 11: warning: exported type CustomServingJobBuilder should have comment or be unexported (golint)
  • Line 17: warning: exported function NewCustomServingJobBuilder should have comment or be unexported (golint)

• arena/pkg/serving/serving_tensorrt.go
  • Line 18: warning: exported function NewTensorrtServingProcesser should have comment or be unexported (golint)
  • Line 30: warning: exported function SubmitTensorRTServingJob should have comment or be unexported (golint)

• arena/pkg/commands/training/logs.go
  • Line 28: warning: exported function NewLogsCommand should have comment or be unexported (golint)

• arena/pkg/topnode/details.go
  • Line 15: warning: exported function ListNodeDetails should have comment or be unexported (golint)
  • Line 27: warning: exported function DisplayNodeDetails should have comment or be unexported (golint)
  • Line 56: warning: exported function PrintLine should have comment or be unexported (golint)

• arena/pkg/training/trainer_volcano.go
  • Line 35: warning: comment on exported type VolcanoJob should be of the form "VolcanoJob ..." (with optional leading article) (golint)
  • Line 46: warning: exported method VolcanoJob.Name should have comment or be unexported (golint)
  • Line 50: warning: exported method VolcanoJob.Uid should have comment or be unexported (golint)
  • Line 54: warning: comment on exported method VolcanoJob.ChiefPod should be of the form "ChiefPod ..." (golint)
  • Line 59: warning: comment on exported method VolcanoJob.Trainer should be of the form "Trainer ..." (golint)
  • Line 64: warning: comment on exported method VolcanoJob.AllPods should be of the form "AllPods ..." (golint)
  • Line 69: warning: exported method VolcanoJob.GetTrainJob should have comment or be unexported (golint)
  • Line 73: warning: exported method VolcanoJob.GetStatus should have comment or be unexported (golint)
  • Line 131: warning: exported method VolcanoJob.StartTime should have comment or be unexported (golint)
  • Line 136: warning: exported method VolcanoJob.Age should have comment or be unexported (golint)
  • Line 145: warning: comment on exported method VolcanoJob.Duration should be of the form "Duration ..." (golint)
  • Line 157: warning: exported method VolcanoJob.GetJobDashboards should have comment or be unexported (golint)
  • Line 200: warning: comment on exported method VolcanoJob.RequestedGPU should be of the form "RequestedGPU ..." (golint)
  • Line 218: warning: comment on exported method VolcanoJob.AllocatedGPU should be of the form "AllocatedGPU ..." (golint)
  • Line 230: warning: comment on exported method VolcanoJob.HostIPOfChief should be of the form "HostIPOfChief ..." (golint)
  • Line 239: warning: exported method VolcanoJob.Namespace should have comment or be unexported (golint)
  • Line 243: warning: comment on exported method VolcanoJob.GetPriorityClass should be of the form "GetPriorityClass ..." (golint)
  • Line 248: warning: comment on exported type VolcanoJobTrainer should be of the form "VolcanoJobTrainer ..." (with optional leading article) (golint)
  • Line 256: warning: exported function NewVolcanoJobTrainer should have comment or be unexported (golint)
  • Line 280: warning: exported method VolcanoJobTrainer.Type should have comment or be unexported (golint)
  • Line 284: warning: exported method VolcanoJobTrainer.IsSupported should have comment or be unexported (golint)
  • Line 296: warning: exported method VolcanoJobTrainer.GetTrainingJob should have comment or be unexported (golint)
  • Line 320: warning: exported method VolcanoJobTrainer.ListTrainingJobs should have comment or be unexported (golint)

• arena/pkg/topnode/gpuexclusive.go
  • Line 14: warning: exported var GPUExclusiveNodeDescription should have comment or be unexported (golint)
  • Line 47: warning: exported function NewGPUExclusiveNode should have comment or be unexported (golint)
  • Line 358: warning: exported function IsGPUExclusiveNode should have comment or be unexported (golint)
  • Line 496: warning: exported function NewGPUExclusiveNodeProcesser should have comment or be unexported (golint)

• arena/pkg/commands/data/data.go
  • Line 29: warning: comment on exported function NewDataCommand should be of the form "NewDataCommand ..." (golint)

• arena/pkg/apis/types/types.go
  • Line 3: warning: comment on exported type FormatStyle should be of the form "FormatStyle ..." (with optional leading article) (golint)
  • Line 8: warning: comment on exported const WideFormat should be of the form "WideFormat ..." (golint)
  • Line 10: warning: comment on exported const JsonFormat should be of the form "JsonFormat ..." (golint)
  • Line 12: warning: comment on exported const YamlFormat should be of the form "YamlFormat ..." (golint)
  • Line 14: warning: comment on exported const UnknownFormat should be of the form "UnknownFormat ..." (golint)
  • Line 18: warning: exported type ArenaClientArgs should have comment or be unexported (golint)

• arena/pkg/commands/training/list.go
  • Line 15: warning: exported function NewListCommand should have comment or be unexported (golint)

• arena/pkg/apis/training/volcano_builder.go
  • Line 10: warning: exported type VolcanoJobBuilder should have comment or be unexported (golint)
  • Line 16: warning: exported function NewVolcanoJobBuilder should have comment or be unexported (golint)
  • Line 49: warning: exported method VolcanoJobBuilder.MinAvailable should have comment or be unexported (golint)
  • Line 56: warning: exported method VolcanoJobBuilder.Queue should have comment or be unexported (golint)
  • Line 63: warning: exported method VolcanoJobBuilder.SchedulerName should have comment or be unexported (golint)
  • Line 70: warning: exported method VolcanoJobBuilder.TaskImages should have comment or be unexported (golint)
  • Line 77: warning: exported method VolcanoJobBuilder.TaskName should have comment or be unexported (golint)
  • Line 84: warning: exported method VolcanoJobBuilder.TaskReplicas should have comment or be unexported (golint)
  • Line 91: warning: exported method VolcanoJobBuilder.TaskCPU should have comment or be unexported (golint)
  • Line 98: warning: exported method VolcanoJobBuilder.TaskMemory should have comment or be unexported (golint)
  • Line 105: warning: exported method VolcanoJobBuilder.TaskPort should have comment or be unexported (golint)

• arena/pkg/apis/serving/job.go
  • Line 38: warning: exported function NewJob should have comment or be unexported (golint)
  • Line 44: warning: exported method Job.Type should have comment or be unexported (golint)

• arena/pkg/serving/serving_tensorflow.go
  • Line 18: warning: exported function NewTensorflowServingProcesser should have comment or be unexported (golint)
  • Line 30: warning: exported function SubmitTensorflowServingJob should have comment or be unexported (golint)

• arena/pkg/commands/training/submit_horovodjob.go
  • Line 13: warning: comment on exported function NewSubmitHorovodJobCommand should be of the form "NewSubmitHorovodJobCommand ..." (golint)

• arena/pkg/commands/training/submit_sparkjob.go
  • Line 13: warning: exported function NewSubmitSparkJobCommand should have comment or be unexported (golint)

• arena/pkg/serving/list.go
  • Line 21: warning: exported function ListServingJobs should have comment or be unexported (golint)
  • Line 68: warning: exported function DisplayAllServingJobs should have comment or be unexported (golint)

• arena/pkg/commands/top/top.go
  • Line 31: warning: exported function NewTopCommand should have comment or be unexported (golint)

• arena/pkg/commands/training/scaleout.go
  • Line 27: warning: exported function NewScaleOutCommand should have comment or be unexported (golint)

• arena/pkg/cron/delete.go
  • Line 9: warning: exported function DeleteCron should have comment or be unexported (golint)

• arena/pkg/cron/suspend.go
  • Line 5: warning: exported function SuspendCron should have comment or be unexported (golint)

• arena/pkg/serving/delete.go
  • Line 12: warning: exported function DeleteServingJob should have comment or be unexported (golint)

• arena/pkg/podlogs/logger.go
  • Line 19: warning: exported var ErrPodNotFound should have comment or be unexported (golint)
  • Line 25: warning: exported type PodLogger should have comment or be unexported (golint)
  • Line 36: warning: exported function NewPodLogger should have comment or be unexported (golint)
  • Line 46: warning: exported method PodLogger.Print should have comment or be unexported (golint)
  • Line 50: warning: exported method PodLogger.AcceptLogs should have comment or be unexported (golint)

• arena/pkg/apis/training/sparkjob_builder.go
  • Line 8: warning: exported type SparkJobBuilder should have comment or be unexported (golint)
  • Line 14: warning: exported function NewSparkJobBuilder should have comment or be unexported (golint)
  • Line 44: warning: exported method SparkJobBuilder.Image should have comment or be unexported (golint)
  • Line 51: warning: exported method SparkJobBuilder.ExecutorReplicas should have comment or be unexported (golint)
  • Line 58: warning: exported method SparkJobBuilder.MainClass should have comment or be unexported (golint)
  • Line 65: warning: exported method SparkJobBuilder.Jar should have comment or be unexported (golint)
  • Line 72: warning: exported method SparkJobBuilder.DriverCPURequest should have comment or be unexported (golint)
  • Line 79: warning: exported method SparkJobBuilder.DriverMemoryRequest should have comment or be unexported (golint)
  • Line 86: warning: exported method SparkJobBuilder.ExecutorCPURequest should have comment or be unexported (golint)
  • Line 93: warning: exported method SparkJobBuilder.ExecutorMemoryRequest should have comment or be unexported (golint)

• arena/pkg/apis/arenaclient/cron_client.go
  • Line 13: warning: exported type CronClient should have comment or be unexported (golint)
  • Line 26: warning: comment on exported method CronClient.SubmitCronTrainingJob should be of the form "SubmitCronTrainingJob ..." (golint)
  • Line 64: warning: exported method CronClient.Get should have comment or be unexported (golint)
  • Line 68: warning: exported method CronClient.GetAndPrint should have comment or be unexported (golint)
  • Line 83: warning: exported method CronClient.Suspend should have comment or be unexported (golint)
  • Line 87: warning: exported method CronClient.Resume should have comment or be unexported (golint)
  • Line 91: warning: exported method CronClient.Delete should have comment or be unexported (golint)

• arena/pkg/serving/serving_custom.go
  • Line 18: warning: exported function NewCustomServingProcesser should have comment or be unexported (golint)
  • Line 30: warning: exported function SubmitCustomServingJob should have comment or be unexported (golint)

• arena/pkg/argsbuilder/submit_horovodjob.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 27: warning: exported type SubmitHorovodJobArgsBuilder should have comment or be unexported (golint)
  • Line 33: warning: exported function NewSubmitHorovodJobArgsBuilder should have comment or be unexported (golint)
  • Line 48: warning: exported method SubmitHorovodJobArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 53: warning: exported method SubmitHorovodJobArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 60: warning: exported method SubmitHorovodJobArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 68: warning: exported method SubmitHorovodJobArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 77: warning: exported method SubmitHorovodJobArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 87: warning: exported method SubmitHorovodJobArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/commands/cron/suspend.go
  • Line 26: warning: exported function NewCronSuspendCommand should have comment or be unexported (golint)

• arena/version.go
  • Line 1: warning: package comment should be of the form "Package arena ..." (golint)
  • Line 48: warning: exported type ChartsInfo should have comment or be unexported (golint)

• arena/pkg/training/const.go
  • Line 20: warning: exported const CHART_PKG_LOC should have comment (or a comment on this block) or be unexported (golint)
  • Line 21: warning: comment on exported const NVIDIAGPUResourceName should be of the form "NVIDIAGPUResourceName ..." (golint)

• arena/pkg/argsbuilder/submit_pytorchjob.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 26: warning: exported type SubmitPytorchJobArgsBuilder should have comment or be unexported (golint)
  • Line 32: warning: exported function NewSubmitPytorchJobArgsBuilder should have comment or be unexported (golint)
  • Line 47: warning: exported method SubmitPytorchJobArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 52: warning: exported method SubmitPytorchJobArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 59: warning: exported method SubmitPytorchJobArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 67: warning: exported method SubmitPytorchJobArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 76: warning: exported method SubmitPytorchJobArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 86: warning: exported method SubmitPytorchJobArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/argsbuilder/submit_sync_code.go
  • Line 13: warning: exported type SubmitSyncCodeArgsBuilder should have comment or be unexported (golint)
  • Line 19: warning: exported function NewSubmitSyncCodeArgsBuilder should have comment or be unexported (golint)
  • Line 27: warning: exported method SubmitSyncCodeArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 32: warning: exported method SubmitSyncCodeArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 39: warning: exported method SubmitSyncCodeArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 47: warning: exported method SubmitSyncCodeArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 56: warning: exported method SubmitSyncCodeArgsBuilder.Build should have comment or be unexported (golint)
  • Line 91: warning: exported method SubmitSyncCodeArgsBuilder.AddCommandFlags should have comment or be unexported (golint)

• arena/pkg/commands/serving/serving_tensorrt.go
  • Line 14: warning: exported function NewSubmitTRTServingJobCommand should have comment or be unexported (golint)

• arena/pkg/apis/types/submit.go
  • Line 1: warning: package comment should be of the form "Package types ..." (golint)
  • Line 130: warning: exported type SubmitSyncCodeArgs should have comment or be unexported (golint)

• arena/pkg/apis/attach/attach.go
  • Line 12: warning: exported type AttachBuilder should have comment or be unexported (golint)
  • Line 18: warning: exported function NewAttachArgsBuilder should have comment or be unexported (golint)
  • Line 37: warning: exported method AttachBuilder.PodName should have comment or be unexported (golint)
  • Line 42: warning: exported method AttachBuilder.ContainerName should have comment or be unexported (golint)
  • Line 47: warning: exported method AttachBuilder.IOStreams should have comment or be unexported (golint)
  • Line 52: warning: exported method AttachBuilder.Command should have comment or be unexported (golint)
  • Line 59: warning: exported method AttachBuilder.CmdArgsLenAtDash should have comment or be unexported (golint)
  • Line 64: warning: exported method AttachBuilder.Build should have comment or be unexported (golint)

• arena/pkg/serving/serving_kf.go
  • Line 13: warning: comment on exported type KFServingProcesser should be of the form "KFServingProcesser ..." (with optional leading article) (golint)
  • Line 18: warning: exported function NewKFServingProcesser should have comment or be unexported (golint)
  • Line 30: warning: exported function SubmitKFServingJob should have comment or be unexported (golint)

• arena/pkg/operators/et-operator/api/v1alpha1/scaleout_types.go
  • Line 26: warning: exported type EnvSpec should have comment or be unexported (golint)
  • Line 53: warning: exported type ToAddSpec should have comment or be unexported (golint)
  • Line 81: warning: exported type Selector should have comment or be unexported (golint)

• arena/pkg/operators/kubedl-operator/apis/apps/v1alpha1/groupversion_info.go
  • Line 28: warning: exported const KindCron should have comment (or a comment on this block) or be unexported (golint)

• arena/pkg/argsbuilder/serving_seldon.go
  • Line 11: warning: exported type SeldonServingArgsBuilder should have comment or be unexported (golint)
  • Line 17: warning: exported function NewSeldonServingArgsBuilder should have comment or be unexported (golint)
  • Line 30: warning: exported method SeldonServingArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 35: warning: exported method SeldonServingArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 42: warning: exported method SeldonServingArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 50: warning: exported method SeldonServingArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 58: warning: exported method SeldonServingArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 67: warning: exported method SeldonServingArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/apis/types/traffic_router.go
  • Line 8: warning: exported type TrafficRouterSplitArgs should have comment or be unexported (golint)
  • Line 16: warning: exported type ServingVersionWeight should have comment or be unexported (golint)
  • Line 21: warning: exported type PreprocesObject should have comment or be unexported (golint)
  • Line 28: warning: exported type DestinationRuleCRD should have comment or be unexported (golint)
  • Line 47: warning: exported type VirtualServiceCRD should have comment or be unexported (golint)
  • Line 66: warning: exported type VirtualService should have comment or be unexported (golint)
  • Line 71: warning: exported type HTTPRoute should have comment or be unexported (golint)
  • Line 77: warning: exported type HTTPMatchRequest should have comment or be unexported (golint)
  • Line 82: warning: exported type StringMatchPrefix should have comment or be unexported (golint)
  • Line 86: warning: exported type DestinationWeight should have comment or be unexported (golint)
  • Line 91: warning: exported type Destination should have comment or be unexported (golint)
  • Line 96: warning: exported type PortSelector should have comment or be unexported (golint)

• arena/pkg/apis/training/horovod_builder.go
  • Line 11: warning: exported type HorovodJobBuilder should have comment or be unexported (golint)
  • Line 17: warning: exported function NewHorovodJobBuilder should have comment or be unexported (golint)

• arena/pkg/operators/volcano-operator/client/clientset/versioned/typed/batch/v1alpha1/generated_expansion.go
  • Line 19: warning: exported type JobExpansion should have comment or be unexported (golint)

• arena/pkg/util/base.go
  • Line 13: warning: exported function GetClientSetForTest should have comment or be unexported (golint)
  • Line 35: warning: exported function Md5 should have comment or be unexported (golint)

• arena/pkg/operators/tf-operator/apis/tensorflow/v1alpha2/openapi_generated.go
  • Line 26: warning: exported function GetOpenAPIDefinitions should have comment or be unexported (golint)

• arena/pkg/operators/volcano-operator/client/clientset/versioned/clientset.go
  • Line 27: warning: exported type Interface should have comment or be unexported (golint)
  • Line 46: warning: comment on exported method Clientset.Batch should be of the form "Batch ..." (golint)

• arena/pkg/util/kubectl/configmap.go
  • Line 28: warning: exported const JOB_CONFIG_LABEL should have comment or be unexported (golint)
  • Line 30: warning: comment on exported function ListAppConfigMaps should be of the form "ListAppConfigMaps ..." (golint)

• arena/pkg/training/submit_horovod.go
  • Line 26: warning: exported function SubmitHorovodJob should have comment or be unexported (golint)

• arena/pkg/commands/serving/delete.go
  • Line 27: warning: comment on exported function NewDeleteCommand should be of the form "NewDeleteCommand ..." (golint)

• arena/pkg/apis/config/kubeclient.go
  • Line 1: warning: package comment should be of the form "Package config ..." (golint)

• arena/pkg/cron/get.go
  • Line 27: warning: exported function GetCronInfo should have comment or be unexported (golint)
  • Line 31: warning: exported function DisplayCron should have comment or be unexported (golint)

• arena/pkg/apis/arenaclient/serving_client.go
  • Line 117: warning: exported method ServingJobClient.Attach should have comment or be unexported (golint)
  • Line 156: warning: exported method ServingJobClient.TrafficRouterSplit should have comment or be unexported (golint)

• arena/pkg/operators/pytorch-operator/apis/pytorch/v1/types.go
  • Line 26: warning: comment on exported type PyTorchJob should be of the form "PyTorchJob ..." (with optional leading article) (golint)

• arena/pkg/operators/volcano-operator/client/clientset/versioned/scheme/register.go
  • Line 27: warning: exported var Scheme should have comment or be unexported (golint)
  • Line 28: warning: exported var Codecs should have comment or be unexported (golint)
  • Line 29: warning: exported var ParameterCodec should have comment or be unexported (golint)

• arena/pkg/util/helm/helm.go
  • Line 31: warning: comment on exported function InstallRelease should be of the form "InstallRelease ..." (golint)
  • Line 45: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 92: warning: comment on exported function CheckRelease should be of the form "CheckRelease ..." (golint)
  • Line 136: warning: exported function DeleteRelease should have comment or be unexported (golint)
  • Line 151: warning: exported function ListReleases should have comment or be unexported (golint)
  • Line 167: warning: exported function ListReleaseMap should have comment or be unexported (golint)
  • Line 197: warning: exported function ListAllReleasesWithDetail should have comment or be unexported (golint)

• arena/pkg/commands/completion.go
  • Line 27: warning: exported function NewCompletionCommand should have comment or be unexported (golint)

• arena/pkg/argsbuilder/attach.go
  • Line 12: warning: exported type AttachArgsBuilder should have comment or be unexported (golint)
  • Line 18: warning: exported function NewAttachPodArgsBuilder should have comment or be unexported (golint)
  • Line 26: warning: exported method AttachArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 31: warning: exported method AttachArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 38: warning: exported method AttachArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 46: warning: exported method AttachArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 55: warning: exported method AttachArgsBuilder.Build should have comment or be unexported (golint)
  • Line 64: warning: exported method AttachArgsBuilder.AddCommandFlags should have comment or be unexported (golint)

• arena/pkg/commands/cron/resume.go
  • Line 26: warning: exported function NewCronResumeCommand should have comment or be unexported (golint)

• arena/pkg/commands/serving/list.go
  • Line 13: warning: exported function NewListCommand should have comment or be unexported (golint)

• arena/pkg/argsbuilder/logger.go
  • Line 20: warning: exported type LogArgsBuilder should have comment or be unexported (golint)
  • Line 26: warning: exported function NewLogArgsBuilder should have comment or be unexported (golint)
  • Line 34: warning: exported method LogArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 39: warning: exported method LogArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 46: warning: exported method LogArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 54: warning: exported method LogArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 63: warning: exported method LogArgsBuilder.Build should have comment or be unexported (golint)
  • Line 72: warning: exported method LogArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 122: warning: exported function ParseSinceTime should have comment or be unexported (golint)
  • Line 135: warning: exported function ParseSinceSeconds should have comment or be unexported (golint)

• arena/pkg/operators/et-operator/api/common/types.go
  • Line 1: warning: package comment should be of the form "Package common ..." (golint)
  • Line 21: warning: comment on exported type JobStatus should be of the form "JobStatus ..." (with optional leading article) (golint)
  • Line 69: warning: comment on exported type ReplicaSpec should be of the form "ReplicaSpec ..." (with optional leading article) (golint)
  • Line 87: warning: comment on exported type JobCondition should be of the form "JobCondition ..." (with optional leading article) (golint)
  • Line 134: warning: exported const Scaling should have comment (or a comment on this block) or be unexported (golint)
  • Line 147: warning: exported const CleanPodPolicyUndefined should have comment (or a comment on this block) or be unexported (golint)
  • Line 160: warning: exported const RestartPolicyAlways should have comment (or a comment on this block) or be unexported (golint)
  • Line 172: warning: comment on exported type RunPolicy should be of the form "RunPolicy ..." (with optional leading article) (golint)

• arena/pkg/commands/training/submit_pytorchjob.go
  • Line 13: warning: exported function NewSubmitPytorchJobCommand should have comment or be unexported (golint)

• arena/pkg/operators/tf-operator/apis/tensorflow/v1alpha1/defaults.go
  • Line 27: warning: don't use underscores in Go names; func SetDefaults_TFJob should be SetDefaultsTFJob (golint)

• arena/pkg/prometheus/prom.go
  • Line 14: warning: exported type JobGpuMetric should have comment or be unexported (golint)
  • Line 16: warning: exported method JobGpuMetric.SetPodMetric should have comment or be unexported (golint)
  • Line 42: warning: exported method JobGpuMetric.GetPodMetrics should have comment or be unexported (golint)
  • Line 50: warning: exported function GetPodsGpuInfo should have comment or be unexported (golint)
  • Line 82: warning: exported function GetNodeGPUMetrics should have comment or be unexported (golint)

• arena/pkg/topnode/gpushare.go
  • Line 15: warning: exported var GPUShareNodeDescription should have comment or be unexported (golint)
  • Line 48: warning: exported function NewGPUShareNode should have comment or be unexported (golint)
  • Line 575: warning: exported function IsGPUShareNode should have comment or be unexported (golint)
  • Line 587: warning: exported function NewGPUShareNodeProcesser should have comment or be unexported (golint)

• arena/pkg/training/pod_helper.go
  • Line 205: warning: comment on exported type SortPodConditionByLastTransitionTime should be of the form "SortPodConditionByLastTransitionTime ..." (with optional leading article) (golint)

• arena/pkg/apis/types/submit_horovodjob.go
  • Line 3: warning: exported type SubmitHorovodJobArgs should have comment or be unexported (golint)

• arena/pkg/k8saccesser/const.go
  • Line 4: warning: exported const TensorflowCRDName should have comment (or a comment on this block) or be unexported (golint)

• arena/pkg/argsbuilder/runtime/cron_tfjob_runtime.go
  • Line 11: warning: exported function NewDefaultCronTFJobRuntime should have comment or be unexported (golint)

• arena/pkg/serving/serving_seldon.go
  • Line 32: warning: exported function NewSeldonServingProcesser should have comment or be unexported (golint)
  • Line 44: warning: exported method SeldonServingProcesser.GetServingJobs should have comment or be unexported (golint)
  • Line 59: warning: exported method SeldonServingProcesser.ListServingJobs should have comment or be unexported (golint)
  • Line 171: warning: exported method SeldonServingProcesser.FilterServingJobs should have comment or be unexported (golint)
  • Line 227: warning: exported function SubmitSeldonServingJob should have comment or be unexported (golint)

• arena/pkg/argsbuilder/submit_mpijob.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 26: warning: exported type SubmitMPIJobArgsBuilder should have comment or be unexported (golint)
  • Line 32: warning: exported function NewSubmitMPIJobArgsBuilder should have comment or be unexported (golint)
  • Line 47: warning: exported method SubmitMPIJobArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 52: warning: exported method SubmitMPIJobArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 59: warning: exported method SubmitMPIJobArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 67: warning: exported method SubmitMPIJobArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 76: warning: exported method SubmitMPIJobArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 86: warning: exported method SubmitMPIJobArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/apis/utils/pods.go
  • Line 20: warning: exported function GPUCountInPod should have comment or be unexported (golint)
  • Line 28: warning: exported function AliyunGPUCountInPod should have comment or be unexported (golint)
  • Line 36: warning: exported function ResourceInContainers should have comment or be unexported (golint)
  • Line 133: warning: exported function GPUMemoryCountInPod should have comment or be unexported (golint)
  • Line 141: warning: exported function GetContainerAllocation should have comment or be unexported (golint)
  • Line 165: warning: exported function GetPodAllocation should have comment or be unexported (golint)
  • Line 185: warning: exported function AcquireAllActivePods should have comment or be unexported (golint)
  • Line 203: warning: exported function AcquireAllActivePodsOfNode should have comment or be unexported (golint)
  • Line 222: warning: exported function GetPodGPUTopologyAllocation should have comment or be unexported (golint)
  • Line 227: warning: exported function GetPodGPUTopologyVisibleGPUs should have comment or be unexported (golint)
  • Line 232: warning: exported function GetPendingTimeOfPod should have comment or be unexported (golint)
  • Line 245: warning: exported function GetRunningTimeOfPod should have comment or be unexported (golint)
  • Line 298: warning: exported function GetDurationOfPod should have comment or be unexported (golint)

• arena/pkg/apis/serving/seldon_builder.go
  • Line 10: warning: exported type SeldonJobBuilder should have comment or be unexported (golint)
  • Line 16: warning: exported function NewSeldonServingJobBuilder should have comment or be unexported (golint)

• arena/pkg/training/submit_etjob.go
  • Line 29: warning: exported const ETJOB_MAXWORKERS should have comment (or a comment on this block) or be unexported (golint)
  • Line 33: warning: exported function SubmitETJob should have comment or be unexported (golint)
  • Line 60: warning: exported function SubmitScaleInETJob should have comment or be unexported (golint)
  • Line 94: warning: exported function SubmitScaleOutETJob should have comment or be unexported (golint)

• arena/pkg/training/trainer_et.go
  • Line 47: warning: comment on exported type ETJob should be of the form "ETJob ..." (with optional leading article) (golint)
  • Line 58: warning: exported method ETJob.GetJobDashboards should have comment or be unexported (golint)
  • Line 63: warning: exported method ETJob.Name should have comment or be unexported (golint)
  • Line 67: warning: exported method ETJob.Uid should have comment or be unexported (golint)
  • Line 71: warning: comment on exported method ETJob.ChiefPod should be of the form "ChiefPod ..." (golint)
  • Line 76: warning: exported method ETJob.Trainer should have comment or be unexported (golint)
  • Line 80: warning: comment on exported method ETJob.AllPods should be of the form "AllPods ..." (golint)
  • Line 85: warning: comment on exported method ETJob.GetStatus should be of the form "GetStatus ..." (golint)
  • Line 107: warning: comment on exported method ETJob.StartTime should be of the form "StartTime ..." (golint)
  • Line 112: warning: comment on exported method ETJob.Age should be of the form "Age ..." (golint)
  • Line 123: warning: comment on exported method ETJob.Duration should be of the form "Duration ..." (golint)
  • Line 135: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 143: warning: comment on exported method ETJob.RequestedGPU should be of the form "RequestedGPU ..." (golint)
  • Line 158: warning: comment on exported method ETJob.AllocatedGPU should be of the form "AllocatedGPU ..." (golint)
  • Line 169: warning: comment on exported method ETJob.HostIPOfChief should be of the form "HostIPOfChief ..." (golint)
  • Line 179: warning: exported method ETJob.Namespace should have comment or be unexported (golint)
  • Line 183: warning: exported method ETJob.GetTrainJob should have comment or be unexported (golint)
  • Line 187: warning: exported method ETJob.GetWorkerMaxReplicas should have comment or be unexported (golint)
  • Line 199: warning: exported method ETJob.GetWorkerMinReplicas should have comment or be unexported (golint)
  • Line 234: warning: comment on exported type ETJobTrainer should be of the form "ETJobTrainer ..." (with optional leading article) (golint)
  • Line 243: warning: comment on exported function NewETJobTrainer should be of the form "NewETJobTrainer ..." (golint)
  • Line 263: warning: exported method ETJobTrainer.IsEnabled should have comment or be unexported (golint)
  • Line 267: warning: comment on exported method ETJobTrainer.Type should be of the form "Type ..." (golint)
  • Line 272: warning: comment on exported method ETJobTrainer.IsSupported should be of the form "IsSupported ..." (golint)
  • Line 281: warning: exported method ETJobTrainer.GetTrainingJob should have comment or be unexported (golint)
  • Line 377: warning: exported method ETJobTrainer.ListTrainingJobs should have comment or be unexported (golint)
  • Line 455: warning: comment on exported method ETJob.GetPriorityClass should be of the form "GetPriorityClass ..." (golint)

• arena/pkg/commands/serving/logs.go
  • Line 28: warning: exported function NewLogsCommand should have comment or be unexported (golint)

• arena/pkg/training/resource.go
  • Line 5: warning: exported type Resource should have comment or be unexported (golint)
  • Line 10: warning: exported type ResourceType should have comment or be unexported (golint)
  • Line 12: warning: exported const ResourceTypePod should have comment or be unexported (golint)
  • Line 13: warning: exported const ResourceTypeStatefulSet should have comment or be unexported (golint)
  • Line 14: warning: exported const ResourceTypeJob should have comment or be unexported (golint)
  • Line 28: warning: exported type BasicJobInfo should have comment or be unexported (golint)
  • Line 33: warning: exported method BasicJobInfo.Resources should have comment or be unexported (golint)

• arena/pkg/operators/tf-operator/apis/common/v1/types.go
  • Line 1: warning: package comment should be of the form "Package v1 ..." (golint)
  • Line 21: warning: comment on exported type JobStatus should be of the form "JobStatus ..." (with optional leading article) (golint)
  • Line 62: warning: comment on exported type ReplicaSpec should be of the form "ReplicaSpec ..." (with optional leading article) (golint)
  • Line 79: warning: comment on exported type JobCondition should be of the form "JobCondition ..." (with optional leading article) (golint)
  • Line 138: warning: exported const CleanPodPolicyUndefined should have comment (or a comment on this block) or be unexported (golint)
  • Line 151: warning: exported const RestartPolicyAlways should have comment (or a comment on this block) or be unexported (golint)
  • Line 155: warning: comment on exported const RestartPolicyExitCode should be of the form "RestartPolicyExitCode ..." (golint)

• arena/pkg/operators/spark-operator/apis/sparkoperator.k8s.io/v1beta2/register.go
  • Line 27: warning: exported const Version should have comment or be unexported (golint)
  • Line 30: warning: exported var SchemeBuilder should have comment or be unexported (golint)

• arena/pkg/operators/tf-operator/apis/tensorflow/v1/types.go
  • Line 26: warning: comment on exported type TFJob should be of the form "TFJob ..." (with optional leading article) (golint)

• arena/pkg/util/kubectl/kubectl.go
  • Line 34: warning: exported function SaveAppInfo should have comment or be unexported (golint)
  • Line 86: warning: comment on exported function UninstallApps should be of the form "UninstallApps ..." (golint)
  • Line 106: warning: comment on exported function UninstallAppsWithAppInfoFile should be of the form "UninstallAppsWithAppInfoFile ..." (golint)
  • Line 136: warning: comment on exported function InstallApps should be of the form "InstallApps ..." (golint)
  • Line 156: warning: comment on exported function CreateAppConfigmap should be of the form "CreateAppConfigmap ..." (golint)
  • Line 185: warning: exported function LabelAppConfigmap should have comment or be unexported (golint)
  • Line 200: warning: comment on exported function DeleteAppConfigMap should be of the form "DeleteAppConfigMap ..." (golint)
  • Line 218: warning: comment on exported function CheckAppConfigMap should be of the form "CheckAppConfigMap ..." (golint)
  • Line 240: warning: comment on exported function SaveAppConfigMapToFile should be of the form "SaveAppConfigMapToFile ..." (golint)
  • Line 292: warning: exported function GetCrdNames should have comment or be unexported (golint)

• arena/pkg/training/top_job.go
  • Line 45: warning: exported function TopTrainingJobs should have comment or be unexported (golint)
  • Line 241: warning: exported function SortMapKeys should have comment or be unexported (golint)

• arena/pkg/operators/mpi-operator/apis/kubeflow/v1alpha1/types.go
  • Line 25: warning: exported type MPIJob should have comment or be unexported (golint)
  • Line 34: warning: exported type MPIJobList should have comment or be unexported (golint)
  • Line 40: warning: exported type MPIJobSpec should have comment or be unexported (golint)
  • Line 66: warning: exported type MPIJobLauncherStatusType should have comment or be unexported (golint)
  • Line 78: warning: exported type MPIJobStatus should have comment or be unexported (golint)

• arena/pkg/apis/types/gpunode.go
  • Line 3: warning: exported type NodeType should have comment or be unexported (golint)
  • Line 6: warning: exported const GPUShareNode should have comment (or a comment on this block) or be unexported (golint)
  • Line 14: warning: exported type NodeTypeInfo should have comment or be unexported (golint)
  • Line 20: warning: exported var NodeTypeSlice should have comment or be unexported (golint)
  • Line 43: warning: exported type CommonNodeInfo should have comment or be unexported (golint)
  • Line 67: warning: exported type CommonGPUNodeInfo should have comment or be unexported (golint)
  • Line 74: warning: exported type GPUDeviceInfo should have comment or be unexported (golint)
  • Line 82: warning: exported type GPUShareNodeInfo should have comment or be unexported (golint)
  • Line 91: warning: exported type GPUSharePodInfo should have comment or be unexported (golint)
  • Line 99: warning: exported type GPUShareNodeDevice should have comment or be unexported (golint)
  • Line 105: warning: exported type GPUExclusiveNodeInfo should have comment or be unexported (golint)
  • Line 111: warning: exported type GPUExclusivePodInfo should have comment or be unexported (golint)
  • Line 118: warning: exported type NormalNodeInfo should have comment or be unexported (golint)
  • Line 122: warning: exported type AllNodeInfo should have comment or be unexported (golint)
  • Line 124: warning: exported type GPUTopologyNodeInfo should have comment or be unexported (golint)
  • Line 132: warning: exported type GPUTopology should have comment or be unexported (golint)
  • Line 137: warning: exported type GPUTopologyPodInfo should have comment or be unexported (golint)
  • Line 146: warning: exported type GPUTopologyNodeDevice should have comment or be unexported (golint)

• arena/pkg/commands/training/attach.go
  • Line 28: warning: exported function NewAttachCommand should have comment or be unexported (golint)

• arena/pkg/commands/training/get.go
  • Line 33: warning: comment on exported function NewGetCommand should be of the form "NewGetCommand ..." (golint)

• arena/pkg/serving/traffic_router_split.go
  • Line 37: warning: exported function RunTrafficRouterSplit should have comment or be unexported (golint)

• arena/pkg/operators/volcano-operator/client/clientset/versioned/typed/batch/v1alpha1/batch_client.go
  • Line 25: warning: exported type BatchV1alpha1Interface should have comment or be unexported (golint)
  • Line 35: warning: exported method BatchV1alpha1Client.Jobs should have comment or be unexported (golint)

• arena/pkg/operators/volcano-operator/apis/batch/v1alpha1/register.go
  • Line 24: warning: exported var SchemeBuilder should have comment or be unexported (golint)

• arena/pkg/operators/kubedl-operator/apis/apis.go
  • Line 23: warning: exported var AddToSchemes should have comment or be unexported (golint)
  • Line 25: warning: exported function AddToScheme should have comment or be unexported (golint)

• arena/pkg/operators/pytorch-operator/apis/pytorch/v1/register.go
  • Line 24: warning: comment on exported var SchemeBuilder should be of the form "SchemeBuilder ..." (golint)
  • Line 28: warning: exported var AddToScheme should have comment or be unexported (golint)

• arena/pkg/serving/logs.go
  • Line 10: warning: exported function AcceptJobLog should have comment or be unexported (golint)

• arena/pkg/training/delete.go
  • Line 27: warning: exported function DeleteTrainingJob should have comment or be unexported (golint)

• arena/pkg/operators/kubedl-operator/apis/apps/v1alpha1/cron_types.go
  • Line 110: warning: exported type CronHistory should have comment or be unexported (golint)

• arena/pkg/commands/training/submit_mpijob.go
  • Line 27: warning: exported function NewSubmitMPIJobCommand should have comment or be unexported (golint)

• arena/pkg/apis/utils/utils.go
  • Line 30: warning: should omit 2nd value from range; this loop is equivalent to `for trainingType := range ...` (golint)
  • Line 36: warning: exported function GetSupportTrainingJobTypesInfo should have comment or be unexported (golint)
  • Line 64: warning: exported function GetSupportedNodeTypes should have comment or be unexported (golint)
  • Line 72: warning: exported function TransferNodeType should have comment or be unexported (golint)
  • Line 90: warning: exported function GetServingJobTypes should have comment or be unexported (golint)
  • Line 98: warning: exported function GetSupportServingJobTypesInfo should have comment or be unexported (golint)
  • Line 107: warning: exported function TransferServingJobType should have comment or be unexported (golint)
  • Line 125: warning: exported function GetLogLevel should have comment or be unexported (golint)
  • Line 133: warning: exported function TransferLogLevel should have comment or be unexported (golint)
  • Line 141: warning: exported function GetFormatStyle should have comment or be unexported (golint)
  • Line 149: warning: exported function TransferPrintFormat should have comment or be unexported (golint)
  • Line 158: warning: comment on exported function PrintErrorMessage should be of the form "PrintErrorMessage ..." (golint)
  • Line 175: warning: exported function DefineNodeStatus should have comment or be unexported (golint)
  • Line 201: warning: exported function CheckFileExist should have comment or be unexported (golint)
  • Line 210: warning: exported function DataUnitTransfer should have comment or be unexported (golint)

• arena/pkg/util/port_allocate.go
  • Line 15: warning: exported const AUTO_SELECT_PORT_MIN should have comment or be unexported (golint)
  • Line 16: warning: exported const AUTO_SELECT_PORT_MAX should have comment or be unexported (golint)
  • Line 18: warning: comment on exported function SelectAvailablePortWithDefault should be of the form "SelectAvailablePortWithDefault ..." (golint)
  • Line 28: warning: comment on exported function SelectAvailablePort should be of the form "SelectAvailablePort ..." (golint)

• arena/pkg/cron/cron_handler.go
  • Line 27: warning: exported function NewCronHandler should have comment or be unexported (golint)
  • Line 46: warning: exported function GetCronHandler should have comment or be unexported (golint)
  • Line 53: warning: exported method CronHandler.ListCrons should have comment or be unexported (golint)
  • Line 72: warning: exported method CronHandler.GetCron should have comment or be unexported (golint)
  • Line 82: warning: exported method CronHandler.DeleteCron should have comment or be unexported (golint)
  • Line 86: warning: exported method CronHandler.UpdateCron should have comment or be unexported (golint)

• arena/pkg/operators/volcano-operator/client/clientset/versioned/doc.go
  • Line 17: warning: package comment should be of the form "Package versioned ..." (golint)

• arena/pkg/commands/cron/list.go
  • Line 26: warning: exported function NewCronListCommand should have comment or be unexported (golint)

• arena/pkg/commands/serving/attach.go
  • Line 28: warning: exported function NewAttachCommand should have comment or be unexported (golint)

• arena/pkg/commands/serving/serving_seldon.go
  • Line 14: warning: exported function NewSubmitSeldonServingJobCommand should have comment or be unexported (golint)

• arena/pkg/apis/types/submit_mpijob.go
  • Line 3: warning: exported type SubmitMPIJobArgs should have comment or be unexported (golint)

• arena/pkg/apis/training/etjob_builder.go
  • Line 11: warning: exported type ETJobBuilder should have comment or be unexported (golint)
  • Line 17: warning: exported function NewETJobBuilder should have comment or be unexported (golint)

• arena/pkg/operators/spark-operator/apis/sparkoperator.k8s.io/v1beta1/register.go
  • Line 27: warning: exported const Version should have comment or be unexported (golint)
  • Line 30: warning: exported var SchemeBuilder should have comment or be unexported (golint)

• arena/cmd/job-monitor/main.go
  • Line 41: warning: exported var ErrNoNeedUpgrade should have comment or be unexported (golint)
  • Line 100: warning: exported function WaitJobComplete should have comment or be unexported (golint)
  • Line 133: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 141: warning: exported function DeleteStatefulSet should have comment or be unexported (golint)
  • Line 147: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 162: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 180: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• arena/pkg/argsbuilder/serving_tensorrt.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 26: warning: exported const DefaultTRTServingImage should have comment (or a comment on this block) or be unexported (golint)
  • Line 29: warning: exported type TensorRTServingArgsBuilder should have comment or be unexported (golint)
  • Line 35: warning: exported function NewTensorRTServingArgsBuilder should have comment or be unexported (golint)
  • Line 49: warning: exported method TensorRTServingArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 54: warning: exported method TensorRTServingArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 61: warning: exported method TensorRTServingArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 69: warning: exported method TensorRTServingArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 81: warning: exported method TensorRTServingArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 90: warning: exported method TensorRTServingArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/commands/serving/serving_custom.go
  • Line 14: warning: exported function NewSubmitCustomServingJobCommand should have comment or be unexported (golint)

• arena/pkg/operators/tf-operator/apis/common/v1beta2/types.go
  • Line 1: warning: package comment should be of the form "Package v1beta2 ..." (golint)
  • Line 21: warning: comment on exported type JobStatus should be of the form "JobStatus ..." (with optional leading article) (golint)
  • Line 63: warning: comment on exported type ReplicaSpec should be of the form "ReplicaSpec ..." (with optional leading article) (golint)
  • Line 81: warning: comment on exported type JobCondition should be of the form "JobCondition ..." (with optional leading article) (golint)
  • Line 133: warning: exported const CleanPodPolicyUndefined should have comment (or a comment on this block) or be unexported (golint)
  • Line 146: warning: exported const RestartPolicyAlways should have comment (or a comment on this block) or be unexported (golint)
  • Line 150: warning: comment on exported const RestartPolicyExitCode should be of the form "RestartPolicyExitCode ..." (golint)

• arena/pkg/apis/arenaclient/arenaclient.go
  • Line 68: warning: comment on exported method ArenaClient.Cron should be of the form "Cron ..." (golint)
  • Line 73: warning: exported method ArenaClient.Node should have comment or be unexported (golint)
  • Line 77: warning: exported method ArenaClient.Data should have comment or be unexported (golint)

• arena/pkg/serving/util.go
  • Line 11: warning: exported function PrintLine should have comment or be unexported (golint)

• arena/pkg/commands/serving/serving_tensorflow.go
  • Line 14: warning: exported function NewSubmitTFServingJobCommand should have comment or be unexported (golint)

• arena/pkg/k8saccesser/k8s_accesser.go
  • Line 56: warning: exported function InitK8sResourceAccesser should have comment or be unexported (golint)
  • Line 67: warning: exported function GetK8sResourceAccesser should have comment or be unexported (golint)
  • Line 77: warning: exported function NewK8sResourceAccesser should have comment or be unexported (golint)

• arena/pkg/operators/tf-operator/apis/tensorflow/v1alpha1/types.go
  • Line 23: warning: exported const CRDKind should have comment (or a comment on this block) or be unexported (golint)
  • Line 27: warning: comment on exported const AppLabel should be of the form "AppLabel ..." (golint)
  • Line 29: warning: comment on exported const TFPort should be of the form "TFPort ..." (golint)
  • Line 81: warning: exported const MASTER should have comment (or a comment on this block) or be unexported (golint)
  • Line 87: warning: exported const DefaultTFContainer should have comment (or a comment on this block) or be unexported (golint)
  • Line 91: warning: comment on exported type TFReplicaSpec should be of the form "TFReplicaSpec ..." (with optional leading article) (golint)
  • Line 110: warning: exported const TFJobPhaseNone should have comment (or a comment on this block) or be unexported (golint)
  • Line 122: warning: exported const StateUnknown should have comment (or a comment on this block) or be unexported (golint)
  • Line 145: warning: exported const ReplicaStateUnknown should have comment (or a comment on this block) or be unexported (golint)

• arena/pkg/topnode/gputopology.go
  • Line 15: warning: exported var GPUTopologyNodeDescription should have comment or be unexported (golint)
  • Line 49: warning: exported function NewGPUTopologyNode should have comment or be unexported (golint)
  • Line 291: warning: should omit 2nd value from range; this loop is equivalent to `for index := range ...` (golint)
  • Line 306: warning: should omit 2nd value from range; this loop is equivalent to `for index := range ...` (golint)
  • Line 438: warning: exported function IsGPUTopologyNode should have comment or be unexported (golint)
  • Line 605: warning: exported function NewGPUTopologyNodeProcesser should have comment or be unexported (golint)

• arena/pkg/operators/volcano-operator/apis/batch/v1alpha1/job.go
  • Line 22: warning: comment on exported type Job should be of the form "Job ..." (with optional leading article) (golint)
  • Line 99: warning: exported type JobEvent should have comment or be unexported (golint)
  • Line 102: warning: exported const CommandIssued should have comment (or a comment on this block) or be unexported (golint)
  • Line 112: warning: comment on exported const AnyEvent should be of the form "AnyEvent ..." (golint)
  • Line 118: warning: comment on exported const JobUnknownEvent should be of the form "JobUnknownEvent ..." (golint)
  • Line 199: warning: exported type JobPhase should have comment or be unexported (golint)
  • Line 285: warning: comment on exported type JobList should be of the form "JobList ..." (with optional leading article) (golint)

• arena/pkg/operators/tf-operator/apis/tensorflow/v1/defaults.go
  • Line 92: warning: don't use underscores in Go names; func SetDefaults_TFJob should be SetDefaultsTFJob (golint)

• arena/pkg/commands/training/scalein_etjob.go
  • Line 13: warning: exported function NewScaleInETJobCommand should have comment or be unexported (golint)

• arena/pkg/util/validate.go
  • Line 29: warning: exported const DNS1123SubdomainMaxLength should have comment or be unexported (golint)
  • Line 33: warning: exported const DNS1123LabelMaxLength should have comment or be unexported (golint)
  • Line 35: warning: comment on exported const JobMaxLength should be of the form "JobMaxLength ..." (golint)
  • Line 56: warning: comment on exported function ValidatePriorityClassName should be of the form "ValidatePriorityClassName ..." (golint)

• arena/pkg/commands/version.go
  • Line 24: warning: exported function NewVersionCmd should have comment or be unexported (golint)

• arena/pkg/training/get.go
  • Line 58: warning: comment on exported function SearchTrainingJob should be of the form "SearchTrainingJob ..." (golint)
  • Line 155: warning: exported function PrintTrainingJob should have comment or be unexported (golint)
  • Line 282: warning: comment on exported function GetResourcesEvents should be of the form "GetResourcesEvents ..." (golint)

• arena/pkg/apis/training/pytorchjob_builder.go
  • Line 11: warning: exported type PytorchJobBuilder should have comment or be unexported (golint)
  • Line 17: warning: exported function NewPytorchJobBuilder should have comment or be unexported (golint)

• arena/pkg/operators/tf-operator/apis/tensorflow/v1beta2/register.go
  • Line 24: warning: comment on exported var SchemeBuilder should be of the form "SchemeBuilder ..." (golint)
  • Line 28: warning: exported var AddToScheme should have comment or be unexported (golint)

• arena/pkg/util/resource.go
  • Line 18: warning: exported function AcquireAllPods should have comment or be unexported (golint)
  • Line 35: warning: exported function AcquireServingServices should have comment or be unexported (golint)

• arena/pkg/training/trainer_pytorch.go
  • Line 50: warning: comment on exported type PyTorchJob should be of the form "PyTorchJob ..." (with optional leading article) (golint)
  • Line 61: warning: exported method PyTorchJob.Name should have comment or be unexported (golint)
  • Line 65: warning: exported method PyTorchJob.Uid should have comment or be unexported (golint)
  • Line 69: warning: comment on exported method PyTorchJob.ChiefPod should be of the form "ChiefPod ..." (golint)
  • Line 74: warning: exported method PyTorchJob.Trainer should have comment or be unexported (golint)
  • Line 78: warning: comment on exported method PyTorchJob.AllPods should be of the form "AllPods ..." (golint)
  • Line 83: warning: exported method PyTorchJob.GetTrainJob should have comment or be unexported (golint)
  • Line 98: warning: comment on exported method PyTorchJob.GetStatus should be of the form "GetStatus ..." (golint)
  • Line 116: warning: comment on exported method PyTorchJob.StartTime should be of the form "StartTime ..." (golint)
  • Line 121: warning: comment on exported method PyTorchJob.Age should be of the form "Age ..." (golint)
  • Line 132: warning: comment on exported method PyTorchJob.Duration should be of the form "Duration ..." (golint)
  • Line 149: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 157: warning: comment on exported method PyTorchJob.GetJobDashboards should be of the form "GetJobDashboards ..." (golint)
  • Line 201: warning: comment on exported method PyTorchJob.RequestedGPU should be of the form "RequestedGPU ..." (golint)
  • Line 216: warning: comment on exported method PyTorchJob.AllocatedGPU should be of the form "AllocatedGPU ..." (golint)
  • Line 227: warning: comment on exported method PyTorchJob.HostIPOfChief should be of the form "HostIPOfChief ..." (golint)
  • Line 237: warning: exported method PyTorchJob.Namespace should have comment or be unexported (golint)
  • Line 241: warning: comment on exported type PyTorchJobTrainer should be of the form "PyTorchJobTrainer ..." (with optional leading article) (golint)
  • Line 250: warning: comment on exported function NewPyTorchJobTrainer should be of the form "NewPyTorchJobTrainer ..." (golint)
  • Line 276: warning: comment on exported method PyTorchJobTrainer.Type should be of the form "Type ..." (golint)
  • Line 281: warning: comment on exported method PyTorchJobTrainer.IsSupported should be of the form "IsSupported ..." (golint)
  • Line 296: warning: exported method PyTorchJobTrainer.GetTrainingJob should have comment or be unexported (golint)
  • Line 397: warning: exported method PyTorchJobTrainer.ListTrainingJobs should have comment or be unexported (golint)
  • Line 428: warning: comment on exported method PyTorchJob.GetPriorityClass should be of the form "GetPriorityClass ..." (golint)
  • Line 429: warning: receiver name p should be consistent with previous receiver name pj for PyTorchJob (golint)

• arena/pkg/apis/types/submit_tfjob.go
  • Line 1: warning: package comment should be of the form "Package types ..." (golint)
  • Line 16: warning: exported type SubmitTFJobArgs should have comment or be unexported (golint)
  • Line 88: warning: comment on exported type TFRuntime should be of the form "TFRuntime ..." (with optional leading article) (golint)
  • Line 97: warning: exported type Runtime should have comment or be unexported (golint)

• arena/pkg/operators/spark-operator/apis/sparkoperator.k8s.io/register.go
  • Line 20: warning: exported const GroupName should have comment (or a comment on this block) or be unexported (golint)

• arena/pkg/commands/training/submit.go
  • Line 18: warning: exported function NewSubmitCommand should have comment or be unexported (golint)

• arena/pkg/apis/training/scaleout_etjob_builder.go
  • Line 11: warning: exported type ScaleOutETJobBuilder should have comment or be unexported (golint)
  • Line 17: warning: exported function NewScaleOutETJobBuilder should have comment or be unexported (golint)
  • Line 48: warning: comment on exported method ScaleOutETJobBuilder.Count should be of the form "Count ..." (golint)

• arena/pkg/util/config/loader.go
  • Line 13: warning: comment on exported function ReadConfigFile should be of the form "ReadConfigFile ..." (golint)

• arena/pkg/serving/get.go
  • Line 35: warning: exported function SearchServingJob should have comment or be unexported (golint)
  • Line 94: warning: exported function PrintServingJob should have comment or be unexported (golint)

• arena/pkg/operators/et-operator/api/v1alpha1/register.go
  • Line 30: warning: exported var SchemeBuilder should have comment or be unexported (golint)

• arena/pkg/apis/types/cron.go
  • Line 18: warning: exported const ConcurrencyAllow should have comment (or a comment on this block) or be unexported (golint)
  • Line 54: warning: exported type CronInfo should have comment or be unexported (golint)
  • Line 96: warning: exported type CronHistoryInfo should have comment or be unexported (golint)
  • Line 106: warning: exported type CommonCronArgs should have comment or be unexported (golint)
  • Line 132: warning: exported type CronTFJobArgs should have comment or be unexported (golint)

• arena/pkg/commands/training/submit_tfjob.go
  • Line 13: warning: exported function NewSubmitTFJobCommand should have comment or be unexported (golint)

• arena/pkg/operators/volcano-operator/client/clientset/versioned/typed/batch/v1alpha1/doc.go
  • Line 17: warning: package comment should be of the form "Package v1alpha1 ..." (golint)

• arena/pkg/argsbuilder/submit_tensorboard.go
  • Line 15: warning: exported type SubmitTensorboardArgsBuilder should have comment or be unexported (golint)
  • Line 21: warning: exported function NewSubmitTensorboardArgsBuilder should have comment or be unexported (golint)
  • Line 29: warning: exported method SubmitTensorboardArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 34: warning: exported method SubmitTensorboardArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 41: warning: exported method SubmitTensorboardArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 48: warning: exported method SubmitTensorboardArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 57: warning: exported method SubmitTensorboardArgsBuilder.Build should have comment or be unexported (golint)
  • Line 69: warning: exported method SubmitTensorboardArgsBuilder.AddCommandFlags should have comment or be unexported (golint)

• arena/pkg/operators/pytorch-operator/apis/pytorch/validation/validation.go
  • Line 23: warning: exported function ValidateV1PyTorchJobSpec should have comment or be unexported (golint)

• arena/pkg/util/errors.go
  • Line 19: warning: exported function IsNeedWaitError should have comment or be unexported (golint)
  • Line 30: warning: exported function IsConnectionRefusedError should have comment or be unexported (golint)
  • Line 34: warning: exported function IsUnexpectedEOFError should have comment or be unexported (golint)

• arena/pkg/argsbuilder/submit.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 32: warning: exported type SubmitArgsBuilder should have comment or be unexported (golint)
  • Line 38: warning: exported function NewSubmitArgsBuilder should have comment or be unexported (golint)
  • Line 46: warning: exported method SubmitArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 51: warning: exported method SubmitArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 58: warning: exported method SubmitArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 66: warning: exported method SubmitArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 140: warning: exported method SubmitArgsBuilder.PreBuild should have comment or be unexported (golint)

• arena/pkg/commands/cron/delete.go
  • Line 11: warning: comment on exported function NewCronDeleteCommand should be of the form "NewCronDeleteCommand ..." (golint)

• arena/pkg/operators/volcano-operator/client/clientset/versioned/scheme/doc.go
  • Line 17: warning: package comment should be of the form "Package scheme ..." (golint)

• arena/pkg/podlogs/utils.go
  • Line 17: warning: exported type OuterRequestArgs should have comment or be unexported (golint)
  • Line 30: warning: exported function ParseSinceTime should have comment or be unexported (golint)
  • Line 43: warning: exported function ParseSinceSeconds should have comment or be unexported (golint)

• arena/pkg/util/volume.go
  • Line 30: warning: exported function ValidateDatasets should have comment or be unexported (golint)
  • Line 43: 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)

• arena/pkg/apis/arenaclient/node_client.go
  • Line 15: warning: exported type NodeClient should have comment or be unexported (golint)
  • Line 42: warning: comment on exported method NodeClient.ListAndPrintNodes should be of the form "ListAndPrintNodes ..." (golint)

• arena/pkg/training/trainer_tensorflow.go
  • Line 51: warning: comment on exported type TensorFlowJob should be of the form "TensorFlowJob ..." (with optional leading article) (golint)
  • Line 130: warning: comment on exported method TensorFlowJob.Duration should be of the form "Duration ..." (golint)
  • Line 151: warning: comment on exported method TensorFlowJob.GetJobDashboards should be of the form "GetJobDashboards ..." (golint)
  • Line 190: warning: comment on exported method TensorFlowJob.RequestedGPU should be of the form "RequestedGPU ..." (golint)
  • Line 205: warning: comment on exported method TensorFlowJob.AllocatedGPU should be of the form "AllocatedGPU ..." (golint)
  • Line 216: warning: comment on exported method TensorFlowJob.HostIPOfChief should be of the form "HostIPOfChief ..." (golint)
  • Line 226: warning: comment on exported method TensorFlowJob.GetPriorityClass should be of the form "GetPriorityClass ..." (golint)
  • Line 227: warning: receiver name t should be consistent with previous receiver name tj for TensorFlowJob (golint)
  • Line 241: warning: comment on exported type TensorFlowJobTrainer should be of the form "TensorFlowJobTrainer ..." (with optional leading article) (golint)
  • Line 254: warning: exported function NewTensorFlowJobTrainer should have comment or be unexported (golint)
  • Line 279: warning: exported method TensorFlowJobTrainer.Type should have comment or be unexported (golint)
  • Line 283: warning: comment on exported method TensorFlowJobTrainer.IsSupported should be of the form "IsSupported ..." (golint)
  • Line 296: warning: exported method TensorFlowJobTrainer.GetTrainingJob should have comment or be unexported (golint)
  • Line 337: 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 385: warning: exported method TensorFlowJobTrainer.ListTrainingJobs should have comment or be unexported (golint)

• arena/pkg/commands/data/list.go
  • Line 26: warning: exported function NewDataListCommand should have comment or be unexported (golint)

• arena/pkg/commands/cron/get.go
  • Line 26: warning: exported function NewCronGetCommand should have comment or be unexported (golint)

• arena/pkg/training/submit_tfjob.go
  • Line 26: warning: exported function SubmitTFJob should have comment or be unexported (golint)
  • Line 42: warning: don't use underscores in Go names; var tfjob_chart should be tfjobChart (golint)

• arena/pkg/argsbuilder/serving.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 33: warning: exported type ServingArgsBuilder should have comment or be unexported (golint)
  • Line 39: warning: exported function NewServingArgsBuilder should have comment or be unexported (golint)
  • Line 48: warning: exported method ServingArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 53: warning: exported method ServingArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 60: warning: exported method ServingArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 68: warning: exported method ServingArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 130: warning: exported method ServingArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 169: warning: exported method ServingArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/training/submit_mpijob.go
  • Line 26: warning: exported function SubmitMPIJob should have comment or be unexported (golint)

• arena/pkg/completion/completion_bash.go
  • Line 4: warning: exported var BashCompletionFlags should have comment or be unexported (golint)
  • Line 10: warning: exported const BashCompletionFunc should have comment (or a comment on this block) or be unexported (golint)

• arena/pkg/apis/config/arenaconfig.go
  • Line 22: warning: exported const RecommendedConfigPathEnvVar should have comment (or a comment on this block) or be unexported (golint)
  • Line 49: warning: exported type ArenaConfiger should have comment or be unexported (golint)
  • Line 110: warning: exported method ArenaConfiger.GetAPIExtensionClientSet should have comment or be unexported (golint)
  • Line 129: warning: exported method ArenaConfiger.IsDaemonMode should have comment or be unexported (golint)
  • Line 133: warning: exported method ArenaConfiger.GetClusterInstalledCRDs should have comment or be unexported (golint)

• arena/pkg/serving/serving.go
  • Line 22: warning: exported const GPU_RESOURCE_NAME should have comment (or a comment on this block) or be unexported (golint)
  • Line 43: warning: exported function GetAllProcesser should have comment or be unexported (golint)

• arena/pkg/topnode/normal.go
  • Line 13: warning: exported var NormalNodeDescription should have comment or be unexported (golint)
  • Line 33: warning: exported function NewNormalNode should have comment or be unexported (golint)
  • Line 159: warning: exported function NewNormalNodeProcesser should have comment or be unexported (golint)

• arena/pkg/commands/serving/get.go
  • Line 29: warning: comment on exported function NewGetCommand should be of the form "NewGetCommand ..." (golint)

• arena/pkg/apis/serving/tensorrt_builder.go
  • Line 11: warning: exported type TRTServingJobBuilder should have comment or be unexported (golint)
  • Line 17: warning: exported function NewTRTServingJobBuilder should have comment or be unexported (golint)
  • Line 200: warning: comment on exported method TRTServingJobBuilder.HttpPort should be of the form "HttpPort ..." (golint)
  • Line 208: warning: comment on exported method TRTServingJobBuilder.GrpcPort should be of the form "GrpcPort ..." (golint)

• arena/pkg/topnode/summary.go
  • Line 13: warning: comment on exported function DisplayNodeSummary should be of the form "DisplayNodeSummary ..." (golint)

• arena/pkg/argsbuilder/serving_kf.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 25: warning: exported type KFServingArgsBuilder should have comment or be unexported (golint)
  • Line 31: warning: exported function NewKFServingArgsBuilder should have comment or be unexported (golint)
  • Line 44: warning: exported method KFServingArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 49: warning: exported method KFServingArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 56: warning: exported method KFServingArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 64: warning: exported method KFServingArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 74: warning: exported method KFServingArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 83: warning: exported method KFServingArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/cron/list.go
  • Line 13: warning: exported function ListCrons should have comment or be unexported (golint)
  • Line 17: warning: exported function DisplayAllCrons should have comment or be unexported (golint)

• arena/pkg/operators/volcano-operator/apis/batch/v1alpha1/doc.go
  • Line 15: warning: package comment should be of the form "Package v1alpha1 ..." (golint)

• arena/pkg/operators/tf-operator/apis/tensorflow/v1alpha2/types.go
  • Line 82: warning: exported const CleanPodPolicyUndefined should have comment (or a comment on this block) or be unexported (golint)
  • Line 95: warning: exported const RestartPolicyAlways should have comment (or a comment on this block) or be unexported (golint)
  • Line 99: warning: comment on exported const RestartPolicyExitCode should be of the form "RestartPolicyExitCode ..." (golint)

• arena/pkg/training/trainer_mpi.go
  • Line 38: warning: comment on exported type MPIJob should be of the form "MPIJob ..." (with optional leading article) (golint)
  • Line 50: warning: exported method MPIJob.Name should have comment or be unexported (golint)
  • Line 54: warning: exported method MPIJob.Uid should have comment or be unexported (golint)
  • Line 58: warning: comment on exported method MPIJob.ChiefPod should be of the form "ChiefPod ..." (golint)
  • Line 68: warning: exported method MPIJob.Trainer should have comment or be unexported (golint)
  • Line 72: warning: comment on exported method MPIJob.AllPods should be of the form "AllPods ..." (golint)
  • Line 77: warning: exported method MPIJob.GetTrainJob should have comment or be unexported (golint)
  • Line 81: warning: comment on exported method MPIJob.GetStatus should be of the form "GetStatus ..." (golint)
  • Line 101: warning: comment on exported method MPIJob.StartTime should be of the form "StartTime ..." (golint)
  • Line 106: warning: comment on exported method MPIJob.Age should be of the form "Age ..." (golint)
  • Line 117: warning: comment on exported method MPIJob.Duration should be of the form "Duration ..." (golint)
  • Line 133: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 141: warning: comment on exported method MPIJob.GetJobDashboards should be of the form "GetJobDashboards ..." (golint)
  • Line 185: warning: comment on exported method MPIJob.RequestedGPU should be of the form "RequestedGPU ..." (golint)
  • Line 200: warning: comment on exported method MPIJob.AllocatedGPU should be of the form "AllocatedGPU ..." (golint)
  • Line 211: warning: comment on exported method MPIJob.HostIPOfChief should be of the form "HostIPOfChief ..." (golint)
  • Line 221: warning: exported method MPIJob.Namespace should have comment or be unexported (golint)
  • Line 225: warning: comment on exported type MPIJobTrainer should be of the form "MPIJobTrainer ..." (with optional leading article) (golint)
  • Line 234: warning: comment on exported function NewMPIJobTrainer should be of the form "NewMPIJobTrainer ..." (golint)
  • Line 260: warning: comment on exported method MPIJobTrainer.Type should be of the form "Type ..." (golint)
  • Line 265: warning: comment on exported method MPIJobTrainer.IsSupported should be of the form "IsSupported ..." (golint)
  • Line 278: warning: exported method MPIJobTrainer.GetTrainingJob should have comment or be unexported (golint)
  • Line 416: warning: exported method MPIJobTrainer.ListTrainingJobs should have comment or be unexported (golint)
  • Line 476: warning: comment on exported method MPIJob.GetPriorityClass should be of the form "GetPriorityClass ..." (golint)
  • Line 477: warning: receiver name m should be consistent with previous receiver name mj for MPIJob (golint)

• arena/pkg/commands/top/node.go
  • Line 30: warning: exported function NewTopNodeCommand should have comment or be unexported (golint)

• arena/pkg/training/get_advanced.go
  • Line 53: warning: comment on exported function BuildJobInfo should be of the form "BuildJobInfo ..." (golint)
  • Line 146: warning: comment on exported function GetJobRealStatus should be of the form "GetJobRealStatus ..." (golint)
  • Line 167: warning: exported function GetJobGpuMetric should have comment or be unexported (golint)

• arena/pkg/apis/training/tfjob_builder.go
  • Line 11: warning: exported type TFJobBuilder should have comment or be unexported (golint)
  • Line 17: warning: exported function NewTFJobBuilder should have comment or be unexported (golint)
  • Line 33: warning: exported method TFJobBuilder.GetArgValues should have comment or be unexported (golint)
  • Line 37: warning: exported method TFJobBuilder.Name should have comment or be unexported (golint)
  • Line 44: warning: exported method TFJobBuilder.Command should have comment or be unexported (golint)
  • Line 49: warning: exported method TFJobBuilder.WorkingDir should have comment or be unexported (golint)
  • Line 56: warning: exported method TFJobBuilder.Envs should have comment or be unexported (golint)
  • Line 67: warning: exported method TFJobBuilder.GPUCount should have comment or be unexported (golint)
  • Line 74: warning: exported method TFJobBuilder.Image should have comment or be unexported (golint)
  • Line 81: warning: exported method TFJobBuilder.Tolerations should have comment or be unexported (golint)
  • Line 86: warning: exported method TFJobBuilder.ConfigFiles should have comment or be unexported (golint)
  • Line 97: warning: exported method TFJobBuilder.NodeSelectors should have comment or be unexported (golint)
  • Line 108: warning: exported method TFJobBuilder.Annotations should have comment or be unexported (golint)
  • Line 119: warning: exported method TFJobBuilder.EnableChief should have comment or be unexported (golint)
  • Line 124: warning: exported method TFJobBuilder.ChiefCPU should have comment or be unexported (golint)
  • Line 131: warning: exported method TFJobBuilder.ChiefMemory should have comment or be unexported (golint)
  • Line 138: warning: exported method TFJobBuilder.ChiefPort should have comment or be unexported (golint)
  • Line 145: warning: exported method TFJobBuilder.ChiefSelectors should have comment or be unexported (golint)
  • Line 156: warning: exported method TFJobBuilder.Datas should have comment or be unexported (golint)
  • Line 167: warning: exported method TFJobBuilder.DataDirs should have comment or be unexported (golint)
  • Line 178: warning: exported method TFJobBuilder.EnableEvaluator should have comment or be unexported (golint)
  • Line 183: warning: exported method TFJobBuilder.EvaluatorCPU should have comment or be unexported (golint)
  • Line 190: warning: exported method TFJobBuilder.EvaluatorMemory should have comment or be unexported (golint)
  • Line 197: warning: exported method TFJobBuilder.EvaluatorSelectors should have comment or be unexported (golint)
  • Line 208: warning: exported method TFJobBuilder.LogDir should have comment or be unexported (golint)
  • Line 215: warning: exported method TFJobBuilder.Priority should have comment or be unexported (golint)
  • Line 222: warning: exported method TFJobBuilder.PsCount should have comment or be unexported (golint)
  • Line 229: warning: exported method TFJobBuilder.PsCPU should have comment or be unexported (golint)
  • Line 236: warning: exported method TFJobBuilder.PsImage should have comment or be unexported (golint)
  • Line 243: warning: exported method TFJobBuilder.PsMemory should have comment or be unexported (golint)
  • Line 250: warning: exported method TFJobBuilder.PsPort should have comment or be unexported (golint)
  • Line 257: warning: exported method TFJobBuilder.PsSelectors should have comment or be unexported (golint)
  • Line 268: warning: exported method TFJobBuilder.EnableRDMA should have comment or be unexported (golint)
  • Line 273: warning: exported method TFJobBuilder.SyncImage should have comment or be unexported (golint)
  • Line 280: warning: exported method TFJobBuilder.SyncMode should have comment or be unexported (golint)
  • Line 287: warning: exported method TFJobBuilder.SyncSource should have comment or be unexported (golint)
  • Line 294: warning: exported method TFJobBuilder.EnableTensorboard should have comment or be unexported (golint)
  • Line 299: warning: exported method TFJobBuilder.TensorboardImage should have comment or be unexported (golint)
  • Line 306: warning: exported method TFJobBuilder.WorkerCPU should have comment or be unexported (golint)
  • Line 313: warning: exported method TFJobBuilder.WorkerImage should have comment or be unexported (golint)
  • Line 320: warning: exported method TFJobBuilder.WorkerMemory should have comment or be unexported (golint)
  • Line 327: warning: exported method TFJobBuilder.WorkerPort should have comment or be unexported (golint)
  • Line 334: warning: exported method TFJobBuilder.WorkerSelectors should have comment or be unexported (golint)
  • Line 345: warning: exported method TFJobBuilder.WorkerCount should have comment or be unexported (golint)
  • Line 351: warning: exported method TFJobBuilder.ImagePullSecrets should have comment or be unexported (golint)
  • Line 358: warning: exported method TFJobBuilder.CleanPodPolicy should have comment or be unexported (golint)
  • Line 365: warning: exported method TFJobBuilder.RoleSequence should have comment or be unexported (golint)
  • Line 372: warning: exported method TFJobBuilder.Build should have comment or be unexported (golint)

• arena/pkg/util/charts.go
  • Line 20: warning: exported function GetChartsFolder should have comment or be unexported (golint)
  • Line 34: warning: exported function StringInSlice should have comment or be unexported (golint)

• arena/pkg/util/retry.go
  • Line 25: warning: exported function Retry should have comment or be unexported (golint)
  • Line 49: warning: exported function RetryDuring should have comment or be unexported (golint)

• arena/pkg/operators/mpi-operator/apis/kubeflow/v1alpha1/register.go
  • Line 30: warning: exported var SchemeBuilder should have comment or be unexported (golint)

• arena/pkg/training/trainer_spark.go
  • Line 20: warning: comment on exported type SparkJob should be of the form "SparkJob ..." (with optional leading article) (golint)
  • Line 29: warning: exported method SparkJob.Name should have comment or be unexported (golint)
  • Line 33: warning: exported method SparkJob.Uid should have comment or be unexported (golint)
  • Line 37: warning: comment on exported method SparkJob.ChiefPod should be of the form "ChiefPod ..." (golint)
  • Line 42: warning: comment on exported method SparkJob.Trainer should be of the form "Trainer ..." (golint)
  • Line 47: warning: comment on exported method SparkJob.AllPods should be of the form "AllPods ..." (golint)
  • Line 52: warning: exported method SparkJob.GetTrainJob should have comment or be unexported (golint)
  • Line 56: warning: comment on exported method SparkJob.GetStatus should be of the form "GetStatus ..." (golint)
  • Line 133: warning: exported method SparkJob.StartTime should have comment or be unexported (golint)
  • Line 137: warning: exported method SparkJob.Age should have comment or be unexported (golint)
  • Line 146: warning: comment on exported method SparkJob.Duration should be of the form "Duration ..." (golint)
  • Line 162: warning: exported method SparkJob.GetJobDashboards should have comment or be unexported (golint)
  • Line 204: warning: comment on exported method SparkJob.RequestedGPU should be of the form "RequestedGPU ..." (golint)
  • Line 209: warning: comment on exported method SparkJob.AllocatedGPU should be of the form "AllocatedGPU ..." (golint)
  • Line 214: warning: comment on exported method SparkJob.HostIPOfChief should be of the form "HostIPOfChief ..." (golint)
  • Line 223: warning: exported method SparkJob.Namespace should have comment or be unexported (golint)
  • Line 227: warning: comment on exported method SparkJob.GetPriorityClass should be of the form "GetPriorityClass ..." (golint)
  • Line 232: warning: comment on exported type SparkJobTrainer should be of the form "SparkJobTrainer ..." (with optional leading article) (golint)
  • Line 240: warning: exported function NewSparkJobTrainer should have comment or be unexported (golint)
  • Line 260: warning: exported method SparkJobTrainer.IsEnabled should have comment or be unexported (golint)
  • Line 264: warning: exported method SparkJobTrainer.Type should have comment or be unexported (golint)
  • Line 268: warning: exported method SparkJobTrainer.IsSupported should have comment or be unexported (golint)
  • Line 295: warning: exported method SparkJobTrainer.GetTrainingJob should have comment or be unexported (golint)
  • Line 317: warning: exported method SparkJobTrainer.ListTrainingJobs should have comment or be unexported (golint)

• arena/pkg/argsbuilder/serving_custom.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 25: warning: exported type CustomServingArgsBuilder should have comment or be unexported (golint)
  • Line 31: warning: exported function NewCustomServingArgsBuilder should have comment or be unexported (golint)
  • Line 44: warning: exported method CustomServingArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 49: warning: exported method CustomServingArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 56: warning: exported method CustomServingArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 64: warning: exported method CustomServingArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 72: warning: exported method CustomServingArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 81: warning: exported method CustomServingArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/argsbuilder/submit_sparkjob.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 26: warning: exported type SubmitSparkJobArgsBuilder should have comment or be unexported (golint)
  • Line 32: warning: exported function NewSubmitSparkJobArgsBuilder should have comment or be unexported (golint)
  • Line 42: warning: exported method SubmitSparkJobArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 47: warning: exported method SubmitSparkJobArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 54: warning: exported method SubmitSparkJobArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 62: warning: exported method SubmitSparkJobArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 81: warning: exported method SubmitSparkJobArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 90: warning: exported method SubmitSparkJobArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/operators/et-operator/api/v1alpha1/scalein_types.go
  • Line 48: warning: exported type ToDeleteSpec should have comment or be unexported (golint)

• arena/pkg/commands/top/job.go
  • Line 13: warning: exported function NewTopJobCommand should have comment or be unexported (golint)

• arena/pkg/apis/types/loglevel.go
  • Line 10: warning: exported type LogLevel should have comment or be unexported (golint)
  • Line 13: warning: exported const LogDebug should have comment (or a comment on this block) or be unexported (golint)
  • Line 20: warning: exported type LogArgs should have comment or be unexported (golint)

• arena/pkg/apis/types/training.go
  • Line 43: warning: exported type TrainingJobTypeInfo should have comment or be unexported (golint)
  • Line 49: warning: exported var ErrTrainingJobNotFound should have comment or be unexported (golint)
  • Line 51: warning: comment on exported var TrainingTypeMap should be of the form "TrainingTypeMap ..." (golint)
  • Line 162: warning: exported const RequestGPUsOfJobAnnoKey should have comment (or a comment on this block) or be unexported (golint)

• arena/pkg/operators/tf-operator/apis/tensorflow/v1beta2/defaults.go
  • Line 92: warning: don't use underscores in Go names; func SetDefaults_TFJob should be SetDefaultsTFJob (golint)

• arena/pkg/training/submit_sparkjob.go
  • Line 26: warning: exported function SubmitSparkJob should have comment or be unexported (golint)

• arena/pkg/argsbuilder/serving_tensorflow.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 34: warning: exported const DefaultTfServingImage should have comment (or a comment on this block) or be unexported (golint)
  • Line 39: warning: exported type TensorflowServingArgsBuilder should have comment or be unexported (golint)
  • Line 45: warning: exported function NewTensorflowServingArgsBuilder should have comment or be unexported (golint)
  • Line 59: warning: exported method TensorflowServingArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 64: warning: exported method TensorflowServingArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 71: warning: exported method TensorflowServingArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 79: warning: exported method TensorflowServingArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 101: warning: exported method TensorflowServingArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 110: warning: exported method TensorflowServingArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/commands/training/logviewer.go
  • Line 27: warning: exported function NewLogViewerCommand should have comment or be unexported (golint)

• arena/pkg/commands/training/prune.go
  • Line 1: warning: package comment should be of the form "Package training ..." (golint)
  • Line 26: warning: exported function NewPruneCommand should have comment or be unexported (golint)

• arena/pkg/operators/volcano-operator/apis/batch/v1alpha1/labels.go
  • Line 18: warning: exported const TaskSpecKey should have comment (or a comment on this block) or be unexported (golint)

• arena/pkg/argsbuilder/runtime/runtime.go
  • Line 13: warning: exported function GetTFRuntime should have comment or be unexported (golint)

• arena/pkg/util/helm/helm_advanced.go
  • Line 27: warning: comment on exported function GenerateValueFile should be of the form "GenerateValueFile ..." (golint)
  • Line 46: warning: comment on exported function GenerateHelmTemplate should be of the form "GenerateHelmTemplate ..." (golint)
  • Line 111: warning: exported function GetChartVersion should have comment or be unexported (golint)
  • Line 148: warning: exported function GetChartName should have comment or be unexported (golint)

• arena/pkg/training/submit_pytorchjob.go
  • Line 26: warning: exported function SubmitPytorchJob should have comment or be unexported (golint)

• arena/pkg/argsbuilder/traffic_router_builder.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 27: warning: exported type TrafficRouterArgsBuilder should have comment or be unexported (golint)
  • Line 33: warning: exported function NewTrafficRouterArgsBuilder should have comment or be unexported (golint)
  • Line 42: warning: exported method TrafficRouterArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 47: warning: exported method TrafficRouterArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 54: warning: exported method TrafficRouterArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 62: warning: exported method TrafficRouterArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 78: warning: exported method TrafficRouterArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 87: warning: exported method TrafficRouterArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/operators/mpi-operator/apis/kubeflow/v1alpha1/doc.go
  • Line 15: warning: package comment should be of the form "Package v1alpha1 ..." (golint)

• arena/pkg/apis/types/submit_volcanojob.go
  • Line 3: warning: exported type SubmitVolcanoJobArgs should have comment or be unexported (golint)

• arena/pkg/podexec/exec.go
  • Line 49: warning: exported type AttachPodArgs should have comment or be unexported (golint)
  • Line 63: warning: exported method DefaultRemoteExecutor.Execute should have comment or be unexported (golint)
  • Line 78: warning: exported type StreamOptions should have comment or be unexported (golint)
  • Line 166: warning: exported method StreamOptions.SetupTTY should have comment or be unexported (golint)

• arena/pkg/apis/cron/cron_tfjob_builder.go
  • Line 10: warning: exported type CronTFJobBuilder should have comment or be unexported (golint)
  • Line 17: warning: exported function NewCronTFJobBuilder should have comment or be unexported (golint)
  • Line 33: warning: exported method CronTFJobBuilder.Name should have comment or be unexported (golint)
  • Line 38: warning: exported method CronTFJobBuilder.Schedule should have comment or be unexported (golint)
  • Line 45: warning: exported method CronTFJobBuilder.ConcurrencyPolicy should have comment or be unexported (golint)
  • Line 52: warning: exported method CronTFJobBuilder.Deadline should have comment or be unexported (golint)
  • Line 59: warning: exported method CronTFJobBuilder.HistoryLimit should have comment or be unexported (golint)
  • Line 66: warning: exported method CronTFJobBuilder.WorkingDir should have comment or be unexported (golint)
  • Line 71: warning: exported method CronTFJobBuilder.Envs should have comment or be unexported (golint)
  • Line 76: warning: exported method CronTFJobBuilder.GPUCount should have comment or be unexported (golint)
  • Line 81: warning: exported method CronTFJobBuilder.Image should have comment or be unexported (golint)
  • Line 86: warning: exported method CronTFJobBuilder.Tolerations should have comment or be unexported (golint)
  • Line 91: warning: exported method CronTFJobBuilder.ConfigFiles should have comment or be unexported (golint)
  • Line 96: warning: exported method CronTFJobBuilder.NodeSelectors should have comment or be unexported (golint)
  • Line 101: warning: exported method CronTFJobBuilder.Annotations should have comment or be unexported (golint)
  • Line 106: warning: exported method CronTFJobBuilder.EnableChief should have comment or be unexported (golint)
  • Line 111: warning: exported method CronTFJobBuilder.ChiefCPU should have comment or be unexported (golint)
  • Line 116: warning: exported method CronTFJobBuilder.ChiefMemory should have comment or be unexported (golint)
  • Line 121: warning: exported method CronTFJobBuilder.ChiefPort should have comment or be unexported (golint)
  • Line 126: warning: exported method CronTFJobBuilder.ChiefSelectors should have comment or be unexported (golint)
  • Line 131: warning: exported method CronTFJobBuilder.Datas should have comment or be unexported (golint)
  • Line 136: warning: exported method CronTFJobBuilder.DataDirs should have comment or be unexported (golint)
  • Line 141: warning: exported method CronTFJobBuilder.EnableEvaluator should have comment or be unexported (golint)
  • Line 146: warning: exported method CronTFJobBuilder.EvaluatorCPU should have comment or be unexported (golint)
  • Line 151: warning: exported method CronTFJobBuilder.EvaluatorMemory should have comment or be unexported (golint)
  • Line 156: warning: exported method CronTFJobBuilder.EvaluatorSelectors should have comment or be unexported (golint)
  • Line 161: warning: exported method CronTFJobBuilder.LogDir should have comment or be unexported (golint)
  • Line 166: warning: exported method CronTFJobBuilder.Priority should have comment or be unexported (golint)
  • Line 171: warning: exported method CronTFJobBuilder.PsCount should have comment or be unexported (golint)
  • Line 176: warning: exported method CronTFJobBuilder.PsCPU should have comment or be unexported (golint)
  • Line 181: warning: exported method CronTFJobBuilder.PsImage should have comment or be unexported (golint)
  • Line 186: warning: exported method CronTFJobBuilder.PsMemory should have comment or be unexported (golint)
  • Line 191: warning: exported method CronTFJobBuilder.PsPort should have comment or be unexported (golint)
  • Line 196: warning: exported method CronTFJobBuilder.PsSelectors should have comment or be unexported (golint)
  • Line 201: warning: exported method CronTFJobBuilder.EnableRDMA should have comment or be unexported (golint)
  • Line 206: warning: exported method CronTFJobBuilder.SyncImage should have comment or be unexported (golint)
  • Line 211: warning: exported method CronTFJobBuilder.SyncMode should have comment or be unexported (golint)
  • Line 216: warning: exported method CronTFJobBuilder.SyncSource should have comment or be unexported (golint)
  • Line 221: warning: exported method CronTFJobBuilder.EnableTensorboard should have comment or be unexported (golint)
  • Line 226: warning: exported method CronTFJobBuilder.TensorboardImage should have comment or be unexported (golint)
  • Line 231: warning: exported method CronTFJobBuilder.WorkerCPU should have comment or be unexported (golint)
  • Line 236: warning: exported method CronTFJobBuilder.WorkerImage should have comment or be unexported (golint)
  • Line 241: warning: exported method CronTFJobBuilder.WorkerMemory should have comment or be unexported (golint)
  • Line 246: warning: exported method CronTFJobBuilder.WorkerPort should have comment or be unexported (golint)
  • Line 251: warning: exported method CronTFJobBuilder.WorkerSelectors should have comment or be unexported (golint)
  • Line 256: warning: exported method CronTFJobBuilder.WorkerCount should have comment or be unexported (golint)
  • Line 260: warning: exported method CronTFJobBuilder.ImagePullSecrets should have comment or be unexported (golint)
  • Line 265: warning: exported method CronTFJobBuilder.CleanPodPolicy should have comment or be unexported (golint)
  • Line 270: warning: exported method CronTFJobBuilder.RoleSequence should have comment or be unexported (golint)
  • Line 275: warning: exported method CronTFJobBuilder.Command should have comment or be unexported (golint)
  • Line 280: warning: exported method CronTFJobBuilder.Build should have comment or be unexported (golint)

• arena/pkg/prometheus/query.go
  • Line 30: warning: exported function GetPrometheusClient should have comment or be unexported (golint)
  • Line 67: warning: exported function QueryPrometheusMetrics should have comment or be unexported (golint)
  • Line 194: warning: exported function GetPrometheusServer should have comment or be unexported (golint)

• arena/pkg/argsbuilder/scaleout_etjob.go
  • Line 1: warning: package comment should be of the form "Package argsbuilder ..." (golint)
  • Line 25: warning: exported type ScaleOutETJobArgsBuilder should have comment or be unexported (golint)
  • Line 31: warning: exported function NewScaleOutETJobArgsBuilder should have comment or be unexported (golint)
  • Line 45: warning: exported method ScaleOutETJobArgsBuilder.GetName should have comment or be unexported (golint)
  • Line 50: warning: exported method ScaleOutETJobArgsBuilder.AddSubBuilder should have comment or be unexported (golint)
  • Line 57: warning: exported method ScaleOutETJobArgsBuilder.AddArgValue should have comment or be unexported (golint)
  • Line 65: warning: exported method ScaleOutETJobArgsBuilder.AddCommandFlags should have comment or be unexported (golint)
  • Line 71: warning: exported method ScaleOutETJobArgsBuilder.PreBuild should have comment or be unexported (golint)
  • Line 80: warning: exported method ScaleOutETJobArgsBuilder.Build should have comment or be unexported (golint)

• arena/pkg/apis/types/submit_pytorchjob.go
  • Line 3: warning: exported type SubmitPyTorchJobArgs should have comment or be unexported (golint)

• arena/pkg/apis/training/common.go
  • Line 7: warning: exported var DefaultCommonSubmitArgs should have comment or be unexported (golint)
  • Line 12: warning: exported var DefaultSubmitTensorboardArgs should have comment or be unexported (golint)

• arena/pkg/operators/tf-operator/apis/tensorflow/v1/register.go
  • Line 24: warning: comment on exported var SchemeBuilder should be of the form "SchemeBuilder ..." (golint)
  • Line 28: warning: exported var AddToScheme should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign98%

IneffAssign detects ineffectual assignments in Go code.

• arena/samples/sdk/etjob/etjob.go
  • Line 17: warning: ineffectual assignment to err (ineffassign)

• arena/samples/sdk/custom-serving/main.go
  • Line 18: warning: ineffectual assignment to err (ineffassign)

• arena/pkg/util/port_allocate_test.go
  • Line 36: warning: ineffectual assignment to err (ineffassign)

• arena/samples/sdk/mpijob/mpijob.go
  • Line 17: warning: ineffectual assignment to err (ineffassign)

• arena/pkg/topnode/gputopology.go
  • Line 426: warning: ineffectual assignment to deviceLines (ineffassign)

• arena/pkg/training/get_advanced.go
  • Line 70: warning: ineffectual assignment to err (ineffassign)

misspell0%

Misspell Finds commonly misspelled English words

An error occurred while running this test (exit status 3)