gofmt96%

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

• libstorage/api/server/server_names.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• libstorage/drivers/storage/mock/mock_driver.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• libstorage/drivers/storage/rbd/storage/rbd_storage.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• libstorage/drivers/storage/s3fs/storage/s3fs_storage.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• libstorage/drivers/storage/mock/tests/mock_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• libstorage/drivers/storage/scaleio/storage/scaleio_storage.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• libstorage/api/utils/schema/schema_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• libstorage/api/server/services/services_task.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• libstorage/api/types/types_model_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!

gocyclo94%

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.

• libstorage/api/client/client_http.go
  • Line 46: warning: cyclomatic complexity 33 of function (*client).httpDo() is high (> 15) (gocyclo)

• libstorage/api/server/router/volume/volume_routes.go
  • Line 131: warning: cyclomatic complexity 21 of function handleVolAttachments() is high (> 15) (gocyclo)
  • Line 281: warning: cyclomatic complexity 19 of function (*router).volumeInspect() is high (> 15) (gocyclo)
  • Line 669: warning: cyclomatic complexity 16 of function (*router).volumeDetachAll() is high (> 15) (gocyclo)

• libstorage/api/utils/filters/filters.go
  • Line 98: warning: cyclomatic complexity 34 of function compileFilter() is high (> 15) (gocyclo)

• libstorage/api/context/context_logger.go
  • Line 34: warning: cyclomatic complexity 16 of function ctxFieldsProcessValue() is high (> 15) (gocyclo)

• libstorage/drivers/storage/efs/storage/efs_storage.go
  • Line 511: warning: cyclomatic complexity 16 of function (*driver).VolumeRemove() is high (> 15) (gocyclo)

• libstorage/drivers/storage/s3fs/storage/s3fs_storage.go
  • Line 294: warning: cyclomatic complexity 26 of function (*driver).VolumeRemove() is high (> 15) (gocyclo)

• libstorage/client/client.go
  • Line 35: warning: cyclomatic complexity 20 of function New() is high (> 15) (gocyclo)

• libstorage/drivers/storage/gcepd/storage/gce_storage.go
  • Line 356: warning: cyclomatic complexity 17 of function (*driver).VolumeCreate() is high (> 15) (gocyclo)
  • Line 75: warning: cyclomatic complexity 16 of function (*driver).Init() is high (> 15) (gocyclo)

• libstorage/api/server/server.go
  • Line 65: warning: cyclomatic complexity 22 of function newServer() is high (> 15) (gocyclo)

• libstorage/cli/lss/lss.go
  • Line 57: warning: cyclomatic complexity 29 of function Run() is high (> 15) (gocyclo)

• libstorage/api/types/types_drivers_storage.go
  • Line 129: warning: cyclomatic complexity 17 of function ParseVolumeAttachmentTypes() is high (> 15) (gocyclo)

• libstorage/api/utils/utils_tls.go
  • Line 54: warning: cyclomatic complexity 37 of function ParseTLSConfig() is high (> 15) (gocyclo)

• libstorage/drivers/storage/fittedcloud/storage/fittedcloud_storage.go
  • Line 439: warning: cyclomatic complexity 18 of function (*driver).VolumeAttach() is high (> 15) (gocyclo)

• libstorage/drivers/integration/linux/linux.go
  • Line 173: warning: cyclomatic complexity 28 of function (*driver).Mount() is high (> 15) (gocyclo)
  • Line 328: warning: cyclomatic complexity 16 of function (*driver).Unmount() is high (> 15) (gocyclo)

• libstorage/drivers/storage/isilon/storage/isilon_storage.go
  • Line 334: warning: cyclomatic complexity 16 of function (*driver).VolumeAttach() is high (> 15) (gocyclo)
  • Line 528: warning: cyclomatic complexity 16 of function (*driver).getVolume() is high (> 15) (gocyclo)

• libstorage/drivers/storage/libstorage/libstorage_driver.go
  • Line 41: warning: cyclomatic complexity 18 of function (*driver).Init() is high (> 15) (gocyclo)

golint99%

Golint is a linter for Go source code.

• libstorage/c/libstor-c/libstor-c_exported.go
  • Line 26: warning: don't use underscores in Go names; var client_id should be clientID (golint)
  • Line 67: warning: don't use underscores in Go names; var service_names should be serviceNames (golint)
  • Line 68: warning: don't use underscores in Go names; var volume_maps should be volumeMaps (golint)
  • Line 75: warning: don't use underscores in Go names; var volume_ids should be volumeIds (golint)

• libstorage/api/context/context.go
  • Line 27: warning: context.Context should be the first parameter of a function (golint)
  • Line 262: warning: context.Context should be the first parameter of a function (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign99%

IneffAssign detects ineffectual assignments in Go code.

• libstorage/drivers/storage/isilon/storage/isilon_storage.go
  • Line 277: warning: ineffectual assignment to err (ineffassign)
  • Line 348: warning: ineffectual assignment to vol (ineffassign)

misspell95%

Misspell Finds commonly misspelled English words

• libstorage/drivers/storage/gcepd/storage/gce_storage.go
  • Line 981: warning: "occured" is a misspelling of "occurred" (misspell)

• libstorage/api/context/context_test.go
  • Line 28: warning: "unkown" is a misspelling of "unknown" (misspell)

• libstorage/drivers/storage/dobs/storage/dobs_storage.go
  • Line 515: warning: "occured" is a misspelling of "occurred" (misspell)

• libstorage/api/utils/utils_paths_test.go
  • Line 426: warning: "nto" is a misspelling of "not" (misspell)

• libstorage/api/types/types_context.go
  • Line 36: warning: "levle" is a misspelling of "level" (misspell)

• libstorage/api/types/types_errors.go
  • Line 30: warning: "avaialble" is a misspelling of "available" (misspell)

• libstorage/drivers/storage/ebs/storage/ebs_storage.go
  • Line 1275: warning: "occured" is a misspelling of "occurred" (misspell)

• libstorage/api/tests/tests_suite.go
  • Line 306: warning: "nto" is a misspelling of "not" (misspell)

• libstorage/drivers/storage/efs/storage/efs_storage.go
  • Line 475: warning: "retreive" is a misspelling of "retrieve" (misspell)
  • Line 499: warning: "occured" is a misspelling of "occurred" (misspell)
  • Line 571: warning: "occured" is a misspelling of "occurred" (misspell)
  • Line 618: warning: "occured" is a misspelling of "occurred" (misspell)

• libstorage/drivers/storage/fittedcloud/storage/fittedcloud_storage.go
  • Line 975: warning: "occured" is a misspelling of "occurred" (misspell)

• libstorage/drivers/storage/isilon/executor/isilon_executor.go
  • Line 62: warning: "addressess" is a misspelling of "addresses" (misspell)

• libstorage/drivers/storage/rbd/storage/rbd_storage.go
  • Line 124: warning: "detaild" is a misspelling of "detailed" (misspell)