gofmt100%

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

No problems detected. Good job!

go_vet100%

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

No problems detected. Good job!

gocyclo89%

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.

• postgres-operator/internal/controller/postgrescluster/pgmonitor.go
  • Line 73: warning: cyclomatic complexity 16 of function (*Reconciler).reconcilePGMonitorExporter() is high (> 15) (gocyclo)

• postgres-operator/internal/controller/postgrescluster/pgbackrest.go
  • Line 1775: warning: cyclomatic complexity 40 of function (*Reconciler).reconcileManualBackup() is high (> 15) (gocyclo)
  • Line 1996: warning: cyclomatic complexity 31 of function (*Reconciler).reconcileReplicaCreateBackup() is high (> 15) (gocyclo)
  • Line 1338: warning: cyclomatic complexity 25 of function (*Reconciler).reconcilePostgresClusterDataSource() is high (> 15) (gocyclo)
  • Line 734: warning: cyclomatic complexity 24 of function (*Reconciler).observeRestoreEnv() is high (> 15) (gocyclo)
  • Line 356: warning: cyclomatic complexity 24 of function unstructuredToRepoResources() is high (> 15) (gocyclo)
  • Line 980: warning: cyclomatic complexity 23 of function (*Reconciler).reconcileRestoreJob() is high (> 15) (gocyclo)
  • Line 2398: warning: cyclomatic complexity 21 of function getRepoVolumeStatus() is high (> 15) (gocyclo)
  • Line 250: warning: cyclomatic complexity 20 of function (*Reconciler).cleanupRepoResources() is high (> 15) (gocyclo)
  • Line 859: warning: cyclomatic complexity 18 of function (*Reconciler).prepareForRestore() is high (> 15) (gocyclo)
  • Line 2542: warning: cyclomatic complexity 16 of function (*Reconciler).reconcilePGBackRestCronJob() is high (> 15) (gocyclo)
  • Line 2225: warning: cyclomatic complexity 16 of function (*Reconciler).reconcileStanzaCreate() is high (> 15) (gocyclo)
  • Line 1194: warning: cyclomatic complexity 16 of function (*Reconciler).reconcilePGBackRest() is high (> 15) (gocyclo)

• postgres-operator/internal/controller/postgrescluster/instance_test.go
  • Line 414: warning: cyclomatic complexity 37 of function TestAddPGBackRestToInstancePodSpec() is high (> 15) (gocyclo)

• postgres-operator/internal/controller/postgrescluster/cluster.go
  • Line 267: warning: cyclomatic complexity 24 of function (*Reconciler).reconcileDataSource() is high (> 15) (gocyclo)

• postgres-operator/internal/controller/postgrescluster/delete_test.go
  • Line 52: warning: cyclomatic complexity 17 of function TestReconcilerHandleDelete() is high (> 15) (gocyclo)

• postgres-operator/internal/controller/postgrescluster/pgbouncer.go
  • Line 326: warning: cyclomatic complexity 16 of function (*Reconciler).reconcilePGBouncerDeployment() is high (> 15) (gocyclo)

• postgres-operator/internal/pki/leaf_test.go
  • Line 35: warning: cyclomatic complexity 32 of function TestLeafCertificate() is high (> 15) (gocyclo)

• postgres-operator/pkg/apis/postgres-operator.crunchydata.com/v1beta1/zz_generated.deepcopy.go
  • Line 971: warning: cyclomatic complexity 22 of function (*PostgresClusterSpec).DeepCopyInto() is high (> 15) (gocyclo)

• postgres-operator/internal/controller/postgrescluster/controller_ref_manager_test.go
  • Line 36: warning: cyclomatic complexity 24 of function TestManageControllerRefs() is high (> 15) (gocyclo)

• postgres-operator/internal/controller/postgrescluster/postgres.go
  • Line 143: warning: cyclomatic complexity 17 of function (*Reconciler).reconcilePostgresDatabases() is high (> 15) (gocyclo)
  • Line 272: warning: cyclomatic complexity 17 of function (*Reconciler).reconcilePostgresUserSecrets() is high (> 15) (gocyclo)
  • Line 379: warning: cyclomatic complexity 17 of function (*Reconciler).reconcilePostgresUsersInPostgreSQL() is high (> 15) (gocyclo)

• postgres-operator/internal/controller/postgrescluster/pgbackrest_test.go
  • Line 174: warning: cyclomatic complexity 59 of function TestReconcilePGBackRest() is high (> 15) (gocyclo)
  • Line 3041: warning: cyclomatic complexity 20 of function TestReconcileScheduledBackups() is high (> 15) (gocyclo)
  • Line 923: warning: cyclomatic complexity 20 of function TestReconcileReplicaCreateBackup() is high (> 15) (gocyclo)
  • Line 1095: warning: cyclomatic complexity 19 of function TestReconcileManualBackup() is high (> 15) (gocyclo)
  • Line 1998: warning: cyclomatic complexity 17 of function TestReconcilePostgresClusterDataSource() is high (> 15) (gocyclo)

• postgres-operator/internal/controller/postgrescluster/controller.go
  • Line 78: warning: cyclomatic complexity 42 of function (*Reconciler).Reconcile() is high (> 15) (gocyclo)

• postgres-operator/internal/controller/postgrescluster/instance.go
  • Line 298: warning: cyclomatic complexity 18 of function (*Reconciler).observeInstances() is high (> 15) (gocyclo)
  • Line 758: warning: cyclomatic complexity 16 of function (*Reconciler).rolloutInstances() is high (> 15) (gocyclo)

• postgres-operator/internal/controller/postgrescluster/volumes.go
  • Line 338: warning: cyclomatic complexity 17 of function (*Reconciler).reconcileDirMoveJobs() is high (> 15) (gocyclo)
  • Line 46: warning: cyclomatic complexity 16 of function (*Reconciler).observePersistentVolumeClaims() is high (> 15) (gocyclo)

• postgres-operator/internal/pki/root_test.go
  • Line 133: warning: cyclomatic complexity 17 of function TestRootCertificateAuthority() is high (> 15) (gocyclo)

• postgres-operator/internal/controller/postgrescluster/cluster_test.go
  • Line 96: warning: cyclomatic complexity 19 of function TestCustomLabels() is high (> 15) (gocyclo)
  • Line 358: warning: cyclomatic complexity 19 of function TestCustomAnnotations() is high (> 15) (gocyclo)

• postgres-operator/internal/controller/postgrescluster/volumes_test.go
  • Line 51: warning: cyclomatic complexity 16 of function TestPersistentVolumeClaimLimitations() is high (> 15) (gocyclo)

golint94%

Golint is a linter for Go source code.

• postgres-operator/pkg/apis/postgres-operator.crunchydata.com/v1beta1/pgbouncer_types.go
  • Line 157: warning: exported type PGBouncerPodStatus should have comment or be unexported (golint)

• postgres-operator/pkg/apis/postgres-operator.crunchydata.com/v1beta1/postgrescluster_types.go
  • Line 369: warning: exported type PostgresInstanceSetSpec should have comment or be unexported (golint)
  • Line 443: warning: exported type PostgresInstanceSetStatus should have comment or be unexported (golint)
  • Line 487: warning: exported type PostgresProxyStatus should have comment or be unexported (golint)
  • Line 597: warning: exported type ExporterSpec should have comment or be unexported (golint)

• postgres-operator/internal/patroni/config.go
  • Line 559: warning: don't use underscores in Go names; var data_dir should be dataDir (golint)

• postgres-operator/internal/pgmonitor/api.go
  • Line 28: warning: exported type Executor should have comment or be unexported (golint)
  • Line 32: warning: comment on exported method Executor.GetExporterSetupSQL should be of the form "GetExporterSetupSQL ..." (golint)

• postgres-operator/pkg/apis/postgres-operator.crunchydata.com/v1beta1/patroni_types.go
  • Line 22: warning: exported type PatroniSpec should have comment or be unexported (golint)
  • Line 83: warning: exported type PatroniStatus should have comment or be unexported (golint)

• postgres-operator/pkg/apis/postgres-operator.crunchydata.com/v1beta1/pgbackrest_types.go
  • Line 60: warning: exported type PGBackRestScheduledBackupStatus should have comment or be unexported (golint)
  • Line 159: warning: exported type BackupJobs should have comment or be unexported (golint)

• postgres-operator/pkg/apis/postgres-operator.crunchydata.com/v1beta1/postgres_types.go
  • Line 18: warning: comment on exported type PostgresIdentifier should be of the form "PostgresIdentifier ..." (with optional leading article) (golint)
  • Line 25: warning: exported type PostgresUserSpec should have comment or be unexported (golint)

• postgres-operator/pkg/apis/postgres-operator.crunchydata.com/v1beta1/shared_types.go
  • Line 20: warning: exported type ServiceSpec should have comment or be unexported (golint)

• postgres-operator/internal/naming/labels.go
  • Line 27: warning: exported const LabelInstance should have comment (or a comment on this block) or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!

misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!