Fix golint issues in pkg/registry/.../storage

hvaara · hvaara · commit cfc34712c203 · 2019-11-10T00:22:51.000+01:00
diff --git a/hack/.golint_failures b/hack/.golint_failures
@@ -158,27 +158,18 @@ pkg/registry/autoscaling/rest
 pkg/registry/batch/job
 pkg/registry/batch/rest
 pkg/registry/certificates/certificates
-pkg/registry/certificates/certificates/storage
 pkg/registry/certificates/rest
 pkg/registry/coordination/rest
 pkg/registry/core/componentstatus
-pkg/registry/core/endpoint/storage
-pkg/registry/core/event/storage
-pkg/registry/core/limitrange/storage
 pkg/registry/core/namespace
 pkg/registry/core/namespace/storage
 pkg/registry/core/node
-pkg/registry/core/node/storage
 pkg/registry/core/persistentvolume
-pkg/registry/core/persistentvolume/storage
 pkg/registry/core/persistentvolumeclaim
-pkg/registry/core/persistentvolumeclaim/storage
 pkg/registry/core/pod
 pkg/registry/core/pod/rest
-pkg/registry/core/podtemplate/storage
 pkg/registry/core/replicationcontroller
 pkg/registry/core/replicationcontroller/storage
-pkg/registry/core/resourcequota/storage
 pkg/registry/core/rest
 pkg/registry/core/secret
 pkg/registry/core/secret/storage
@@ -194,10 +185,8 @@ pkg/registry/events/rest
 pkg/registry/extensions/controller/storage
 pkg/registry/extensions/rest
 pkg/registry/flowcontrol/rest
-pkg/registry/networking/networkpolicy/storage
 pkg/registry/networking/rest
 pkg/registry/node/rest
-pkg/registry/policy/poddisruptionbudget/storage
 pkg/registry/policy/rest
 pkg/registry/rbac/clusterrole/policybased
 pkg/registry/rbac/clusterrolebinding
@@ -210,11 +199,9 @@ pkg/registry/rbac/rolebinding/policybased
 pkg/registry/rbac/validation
 pkg/registry/registrytest
 pkg/registry/scheduling/rest
-pkg/registry/settings/podpreset/storage
 pkg/registry/settings/rest
 pkg/registry/storage/rest
 pkg/registry/storage/storageclass
-pkg/registry/storage/storageclass/storage
 pkg/scheduler/apis/config/v1alpha1
 pkg/security/podsecuritypolicy
 pkg/security/podsecuritypolicy/group
diff --git a/pkg/registry/certificates/certificates/storage/storage.go b/pkg/registry/certificates/certificates/storage/storage.go
@@ -31,12 +31,12 @@ import (
 	csrregistry "k8s.io/kubernetes/pkg/registry/certificates/certificates"
 )
 
-// REST implements a RESTStorage for CertificateSigningRequest
+// REST implements a RESTStorage for CertificateSigningRequest.
 type REST struct {
 	*genericregistry.Store
 }
 
-// NewREST returns a registry which will store CertificateSigningRequest in the given helper
+// NewREST returns a registry which will store CertificateSigningRequest in the given helper.
 func NewREST(optsGetter generic.RESTOptionsGetter) (*REST, *StatusREST, *ApprovalREST, error) {
 	store := &genericregistry.Store{
 		NewFunc:                  func() runtime.Object { return &certificates.CertificateSigningRequest{} },
@@ -80,6 +80,7 @@ type StatusREST struct {
 	store *genericregistry.Store
 }
 
+// New creates a new CertificateSigningRequest object.
 func (r *StatusREST) New() runtime.Object {
 	return &certificates.CertificateSigningRequest{}
 }
@@ -103,6 +104,7 @@ type ApprovalREST struct {
 	store *genericregistry.Store
 }
 
+// New creates a new CertificateSigningRequest object.
 func (r *ApprovalREST) New() runtime.Object {
 	return &certificates.CertificateSigningRequest{}
 }
diff --git a/pkg/registry/core/endpoint/storage/storage.go b/pkg/registry/core/endpoint/storage/storage.go
@@ -28,6 +28,7 @@ import (
 	"k8s.io/kubernetes/pkg/registry/core/endpoint"
 )
 
+// REST implements a RESTStorage for endpoints.
 type REST struct {
 	*genericregistry.Store
 }
diff --git a/pkg/registry/core/event/storage/storage.go b/pkg/registry/core/event/storage/storage.go
@@ -28,6 +28,7 @@ import (
 	"k8s.io/kubernetes/pkg/registry/core/event"
 )
 
+// REST implements a RESTStorage for events.
 type REST struct {
 	*genericregistry.Store
 }
diff --git a/pkg/registry/core/limitrange/storage/storage.go b/pkg/registry/core/limitrange/storage/storage.go
@@ -25,11 +25,12 @@ import (
 	"k8s.io/kubernetes/pkg/registry/core/limitrange"
 )
 
+// REST implements a RESTStorage for limit ranges.
 type REST struct {
 	*genericregistry.Store
 }
 
-// NewREST returns a RESTStorage object that will work against limitranges.
+// NewREST returns a RESTStorage object that will work against limit ranges.
 func NewREST(optsGetter generic.RESTOptionsGetter) (*REST, error) {
 	store := &genericregistry.Store{
 		NewFunc:                  func() runtime.Object { return &api.LimitRange{} },
diff --git a/pkg/registry/core/node/storage/storage.go b/pkg/registry/core/node/storage/storage.go
@@ -22,7 +22,7 @@ import (
 	"net/http"
 	"net/url"
 
-	"k8s.io/api/core/v1"
+	v1 "k8s.io/api/core/v1"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apiserver/pkg/registry/generic"
@@ -39,7 +39,7 @@ import (
 	noderest "k8s.io/kubernetes/pkg/registry/core/node/rest"
 )
 
-// NodeStorage includes storage for nodes and all sub resources
+// NodeStorage includes storage for nodes and all sub resources.
 type NodeStorage struct {
 	Node   *REST
 	Status *StatusREST
@@ -48,6 +48,7 @@ type NodeStorage struct {
 	KubeletConnectionInfo client.ConnectionInfoGetter
 }
 
+// REST implements a RESTStorage for nodes.
 type REST struct {
 	*genericregistry.Store
 	connection     client.ConnectionInfoGetter
@@ -59,6 +60,7 @@ type StatusREST struct {
 	store *genericregistry.Store
 }
 
+// New creates a new Node object.
 func (r *StatusREST) New() runtime.Object {
 	return &api.Node{}
 }
diff --git a/pkg/registry/core/persistentvolume/storage/storage.go b/pkg/registry/core/persistentvolume/storage/storage.go
@@ -31,6 +31,7 @@ import (
 	"k8s.io/kubernetes/pkg/registry/core/persistentvolume"
 )
 
+// REST implements a RESTStorage for persistent volumes.
 type REST struct {
 	*genericregistry.Store
 }
@@ -74,6 +75,7 @@ type StatusREST struct {
 	store *genericregistry.Store
 }
 
+// New creates a new PersistentVolume object.
 func (r *StatusREST) New() runtime.Object {
 	return &api.PersistentVolume{}
 }
diff --git a/pkg/registry/core/persistentvolumeclaim/storage/storage.go b/pkg/registry/core/persistentvolumeclaim/storage/storage.go
@@ -31,6 +31,7 @@ import (
 	"k8s.io/kubernetes/pkg/registry/core/persistentvolumeclaim"
 )
 
+// REST implements a RESTStorage for persistent volume claims.
 type REST struct {
 	*genericregistry.Store
 }
@@ -74,6 +75,7 @@ type StatusREST struct {
 	store *genericregistry.Store
 }
 
+// New creates a new PersistentVolumeClaim object.
 func (r *StatusREST) New() runtime.Object {
 	return &api.PersistentVolumeClaim{}
 }
diff --git a/pkg/registry/core/podtemplate/storage/storage.go b/pkg/registry/core/podtemplate/storage/storage.go
@@ -27,6 +27,7 @@ import (
 	"k8s.io/kubernetes/pkg/registry/core/podtemplate"
 )
 
+// REST implements a RESTStorage for pod templates.
 type REST struct {
 	*genericregistry.Store
 }
diff --git a/pkg/registry/core/resourcequota/storage/storage.go b/pkg/registry/core/resourcequota/storage/storage.go
@@ -28,6 +28,7 @@ import (
 	"k8s.io/kubernetes/pkg/registry/core/resourcequota"
 )
 
+// REST implements a RESTStorage for resource quotas.
 type REST struct {
 	*genericregistry.Store
 }
@@ -68,6 +69,7 @@ type StatusREST struct {
 	store *genericregistry.Store
 }
 
+// New creates a new ResourceQuota object.
 func (r *StatusREST) New() runtime.Object {
 	return &api.ResourceQuota{}
 }
diff --git a/pkg/registry/networking/networkpolicy/storage/storage.go b/pkg/registry/networking/networkpolicy/storage/storage.go
@@ -28,12 +28,12 @@ import (
 	"k8s.io/kubernetes/pkg/registry/networking/networkpolicy"
 )
 
-// rest implements a RESTStorage for NetworkPolicies against etcd
+// REST implements a RESTStorage for NetworkPolicies against etcd.
 type REST struct {
 	*genericregistry.Store
 }
 
-// NewREST returns a RESTStorage object that will work against NetworkPolicies
+// NewREST returns a RESTStorage object that will work against NetworkPolicies.
 func NewREST(optsGetter generic.RESTOptionsGetter) (*REST, error) {
 	store := &genericregistry.Store{
 		NewFunc:                  func() runtime.Object { return &networkingapi.NetworkPolicy{} },
diff --git a/pkg/registry/policy/poddisruptionbudget/storage/storage.go b/pkg/registry/policy/poddisruptionbudget/storage/storage.go
@@ -31,7 +31,7 @@ import (
 	"k8s.io/kubernetes/pkg/registry/policy/poddisruptionbudget"
 )
 
-// rest implements a RESTStorage for pod disruption budgets against etcd
+// REST implements a RESTStorage for pod disruption budgets against etcd.
 type REST struct {
 	*genericregistry.Store
 }
@@ -64,11 +64,12 @@ func (r *REST) ShortNames() []string {
 	return []string{"pdb"}
 }
 
-// StatusREST implements the REST endpoint for changing the status of an podDisruptionBudget
+// StatusREST implements the REST endpoint for changing the status of an podDisruptionBudget.
 type StatusREST struct {
 	store *genericregistry.Store
 }
 
+// New creates a new PodDisruptionBudget object.
 func (r *StatusREST) New() runtime.Object {
 	return &policyapi.PodDisruptionBudget{}
 }
diff --git a/pkg/registry/settings/podpreset/storage/storage.go b/pkg/registry/settings/podpreset/storage/storage.go
@@ -24,12 +24,12 @@ import (
 	"k8s.io/kubernetes/pkg/registry/settings/podpreset"
 )
 
-// rest implements a RESTStorage for replication controllers against etcd
+// REST implements a RESTStorage for pod presets against etcd.
 type REST struct {
 	*genericregistry.Store
 }
 
-// NewREST returns a RESTStorage object that will work against replication controllers.
+// NewREST returns a RESTStorage object that will work against pod presets.
 func NewREST(optsGetter generic.RESTOptionsGetter) (*REST, error) {
 	store := &genericregistry.Store{
 		NewFunc:                  func() runtime.Object { return &settingsapi.PodPreset{} },
diff --git a/pkg/registry/storage/storageclass/storage/storage.go b/pkg/registry/storage/storageclass/storage/storage.go
@@ -28,11 +28,12 @@ import (
 	"k8s.io/kubernetes/pkg/registry/storage/storageclass"
 )
 
+// REST implements a RESTStorage for storage classes.
 type REST struct {
 	*genericregistry.Store
 }
 
-// NewREST returns a RESTStorage object that will work against persistent volumes.
+// NewREST returns a RESTStorage object that will work against storage classes.
 func NewREST(optsGetter generic.RESTOptionsGetter) (*REST, error) {
 	store := &genericregistry.Store{
 		NewFunc:                  func() runtime.Object { return &storageapi.StorageClass{} },

Original file line number	Diff line number	Diff line change
`@@ -31,12 +31,12 @@ import (`
`31`	`31`	`csrregistry "k8s.io/kubernetes/pkg/registry/certificates/certificates"`
`32`	`32`	`)`
`33`	`33`
`34`		`-// REST implements a RESTStorage for CertificateSigningRequest`
	`34`	`+// REST implements a RESTStorage for CertificateSigningRequest.`
`35`	`35`	`type REST struct {`
`36`	`36`	`*genericregistry.Store`
`37`	`37`	`}`
`38`	`38`
`39`		`-// NewREST returns a registry which will store CertificateSigningRequest in the given helper`
	`39`	`+// NewREST returns a registry which will store CertificateSigningRequest in the given helper.`
`40`	`40`	`func NewREST(optsGetter generic.RESTOptionsGetter) (REST, StatusREST, *ApprovalREST, error) {`
`41`	`41`	`store := &genericregistry.Store{`
`42`	`42`	`NewFunc: func() runtime.Object { return &certificates.CertificateSigningRequest{} },`
`@@ -80,6 +80,7 @@ type StatusREST struct {`
`80`	`80`	`store *genericregistry.Store`
`81`	`81`	`}`
`82`	`82`
	`83`	`+// New creates a new CertificateSigningRequest object.`
`83`	`84`	`func (r *StatusREST) New() runtime.Object {`
`84`	`85`	`return &certificates.CertificateSigningRequest{}`
`85`	`86`	`}`
`@@ -103,6 +104,7 @@ type ApprovalREST struct {`
`103`	`104`	`store *genericregistry.Store`
`104`	`105`	`}`
`105`	`106`
	`107`	`+// New creates a new CertificateSigningRequest object.`
`106`	`108`	`func (r *ApprovalREST) New() runtime.Object {`
`107`	`109`	`return &certificates.CertificateSigningRequest{}`
`108`	`110`	`}`
Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,7 @@ import (`
`28`	`28`	`"k8s.io/kubernetes/pkg/registry/core/endpoint"`
`29`	`29`	`)`
`30`	`30`
	`31`	`+// REST implements a RESTStorage for endpoints.`
`31`	`32`	`type REST struct {`
`32`	`33`	`*genericregistry.Store`
`33`	`34`	`}`
Original file line number	Diff line number	Diff line change
`@@ -25,11 +25,12 @@ import (`
`25`	`25`	`"k8s.io/kubernetes/pkg/registry/core/limitrange"`
`26`	`26`	`)`
`27`	`27`
	`28`	`+// REST implements a RESTStorage for limit ranges.`
`28`	`29`	`type REST struct {`
`29`	`30`	`*genericregistry.Store`
`30`	`31`	`}`
`31`	`32`
`32`		`-// NewREST returns a RESTStorage object that will work against limitranges.`
	`33`	`+// NewREST returns a RESTStorage object that will work against limit ranges.`
`33`	`34`	`func NewREST(optsGetter generic.RESTOptionsGetter) (*REST, error) {`
`34`	`35`	`store := &genericregistry.Store{`
`35`	`36`	`NewFunc: func() runtime.Object { return &api.LimitRange{} },`
Original file line number	Diff line number	Diff line change
`@@ -31,6 +31,7 @@ import (`
`31`	`31`	`"k8s.io/kubernetes/pkg/registry/core/persistentvolume"`
`32`	`32`	`)`
`33`	`33`
	`34`	`+// REST implements a RESTStorage for persistent volumes.`
`34`	`35`	`type REST struct {`
`35`	`36`	`*genericregistry.Store`
`36`	`37`	`}`
`@@ -74,6 +75,7 @@ type StatusREST struct {`
`74`	`75`	`store *genericregistry.Store`
`75`	`76`	`}`
`76`	`77`
	`78`	`+// New creates a new PersistentVolume object.`
`77`	`79`	`func (r *StatusREST) New() runtime.Object {`
`78`	`80`	`return &api.PersistentVolume{}`
`79`	`81`	`}`
Original file line number	Diff line number	Diff line change
`@@ -27,6 +27,7 @@ import (`
`27`	`27`	`"k8s.io/kubernetes/pkg/registry/core/podtemplate"`
`28`	`28`	`)`
`29`	`29`
	`30`	`+// REST implements a RESTStorage for pod templates.`
`30`	`31`	`type REST struct {`
`31`	`32`	`*genericregistry.Store`
`32`	`33`	`}`