cleanup: address godot warnings

Top level comments should end in a period

Signed-off-by: Yug <yuggupta27@gmail.com>

e2e/utils.go

@@ -418,7 +418,7 @@ func createRBDSecret(c kubernetes.Interface, f *framework.Framework) {
 
 // updateSecretForEncryption is an hack to update the secrets created by rook to
 // include the encyption key
-// TODO in cephcsi we need to create own users in ceph cluster and use it for E2E
+// TODO in cephcsi we need to create own users in ceph cluster and use it for E2E.
 func updateSecretForEncryption(c kubernetes.Interface) error {
 	secrets, err := c.CoreV1().Secrets(rookNamespace).Get(context.TODO(), rbdProvisionerSecretName, metav1.GetOptions{})
 	if err != nil {
@@ -633,7 +633,7 @@ func unmarshal(fileName string, obj interface{}) error {
 }
 
 // createPVCAndApp creates pvc and pod
-// if name is not empty same will be set as pvc and app name
+// if name is not empty same will be set as pvc and app name.
 func createPVCAndApp(name string, f *framework.Framework, pvc *v1.PersistentVolumeClaim, app *v1.Pod, pvcTimeout int) error {
 	if name != "" {
 		pvc.Name = name
@@ -649,7 +649,7 @@ func createPVCAndApp(name string, f *framework.Framework, pvc *v1.PersistentVolu
 }
 
 // deletePVCAndApp delete pvc and pod
-// if name is not empty same will be set as pvc and app name
+// if name is not empty same will be set as pvc and app name.
 func deletePVCAndApp(name string, f *framework.Framework, pvc *v1.PersistentVolumeClaim, app *v1.Pod) error {
 	if name != "" {
 		pvc.Name = name
@@ -703,7 +703,7 @@ type imageInfoFromPVC struct {
 }
 
 // getImageInfoFromPVC reads volume handle of the bound PV to the passed in PVC,
-// and returns imageInfoFromPVC or error
+// and returns imageInfoFromPVC or error.
 func getImageInfoFromPVC(pvcNamespace, pvcName string, f *framework.Framework) (imageInfoFromPVC, error) {
 	var imageData imageInfoFromPVC
 
@@ -756,7 +756,7 @@ func getMountType(appName, appNamespace, mountPath string, f *framework.Framewor
 //  * authenticate with vault and ignore any stdout (we do not need output)
 //  * issue get request for particular key
 // resulting in stdOut (first entry in tuple) - output that contains the key
-// or stdErr (second entry in tuple) - error getting the key
+// or stdErr (second entry in tuple) - error getting the key.
 func readVaultSecret(key string, f *framework.Framework) (string, string) {
 	loginCmd := fmt.Sprintf("vault login -address=%s sample_root_token_id > /dev/null", vaultAddr)
 	readSecret := fmt.Sprintf("vault kv get -address=%s %s%s", vaultAddr, vaultSecretNs, key)

internal/cephfs/cephfs_util.go

@@ -23,12 +23,12 @@ import (
 	"github.com/ceph/ceph-csi/internal/util"
 )
 
-// MDSMap is a representation of the mds map sub-structure returned by 'ceph fs get'
+// MDSMap is a representation of the mds map sub-structure returned by 'ceph fs get'.
 type MDSMap struct {
 	FilesystemName string `json:"fs_name"`
 }
 
-// CephFilesystemDetails is a representation of the main json structure returned by 'ceph fs get'
+// CephFilesystemDetails is a representation of the main json structure returned by 'ceph fs get'.
 type CephFilesystemDetails struct {
 	ID     int64  `json:"id"`
 	MDSMap MDSMap `json:"mdsmap"`
@@ -53,7 +53,7 @@ func getFscID(ctx context.Context, monitors string, cr *util.Credentials, fsName
 	return fsDetails.ID, nil
 }
 
-// CephFilesystem is a representation of the json structure returned by 'ceph fs ls'
+// CephFilesystem is a representation of the json structure returned by 'ceph fs ls'.
 type CephFilesystem struct {
 	Name           string   `json:"name"`
 	MetadataPool   string   `json:"metadata_pool"`
@@ -87,7 +87,7 @@ func getMetadataPool(ctx context.Context, monitors string, cr *util.Credentials,
 	return "", util.ErrPoolNotFound{Pool: fsName, Err: fmt.Errorf("fsName (%s) not found in Ceph cluster", fsName)}
 }
 
-// CephFilesystemDump is a representation of the main json structure returned by 'ceph fs dump'
+// CephFilesystemDump is a representation of the main json structure returned by 'ceph fs dump'.
 type CephFilesystemDump struct {
 	Filesystems []CephFilesystemDetails `json:"filesystems"`
 }

internal/cephfs/controllerserver.go

@@ -38,7 +38,7 @@ type ControllerServer struct {
 	VolumeLocks *util.VolumeLocks
 }
 
-// createBackingVolume creates the backing subvolume and on any error cleans up any created entities
+// createBackingVolume creates the backing subvolume and on any error cleans up any created entities.
 func (cs *ControllerServer) createBackingVolume(ctx context.Context, volOptions *volumeOptions, vID *volumeIdentifier, secret map[string]string) error {
 	cr, err := util.NewAdminCredentials(secret)
 	if err != nil {
@@ -54,7 +54,7 @@ func (cs *ControllerServer) createBackingVolume(ctx context.Context, volOptions
 	return nil
 }
 
-// CreateVolume creates a reservation and the volume in backend, if it is not already present
+// CreateVolume creates a reservation and the volume in backend, if it is not already present.
 func (cs *ControllerServer) CreateVolume(ctx context.Context, req *csi.CreateVolumeRequest) (*csi.CreateVolumeResponse, error) {
 	if err := cs.validateCreateVolumeRequest(req); err != nil {
 		klog.Errorf(util.Log(ctx, "CreateVolumeRequest validation failed: %v"), err)
@@ -150,7 +150,7 @@ func (cs *ControllerServer) CreateVolume(ctx context.Context, req *csi.CreateVol
 	return &csi.CreateVolumeResponse{Volume: volume}, nil
 }
 
-// DeleteVolume deletes the volume in backend and its reservation
+// DeleteVolume deletes the volume in backend and its reservation.
 func (cs *ControllerServer) DeleteVolume(ctx context.Context, req *csi.DeleteVolumeRequest) (*csi.DeleteVolumeResponse, error) {
 	if err := cs.validateDeleteVolumeRequest(); err != nil {
 		klog.Errorf(util.Log(ctx, "DeleteVolumeRequest validation failed: %v"), err)
@@ -256,7 +256,7 @@ func (cs *ControllerServer) ValidateVolumeCapabilities(
 	}, nil
 }
 
-// ControllerExpandVolume expands CephFS Volumes on demand based on resizer request
+// ControllerExpandVolume expands CephFS Volumes on demand based on resizer request.
 func (cs *ControllerServer) ControllerExpandVolume(ctx context.Context, req *csi.ControllerExpandVolumeRequest) (*csi.ControllerExpandVolumeResponse, error) {
 	if err := cs.validateExpandVolumeRequest(req); err != nil {
 		klog.Errorf(util.Log(ctx, "ControllerExpandVolumeRequest validation failed: %v"), err)

internal/cephfs/driver.go

@@ -37,7 +37,7 @@ const (
 	radosNamespace = "csi"
 )
 
-// Driver contains the default identity,node and controller struct
+// Driver contains the default identity,node and controller struct.
 type Driver struct {
 	cd *csicommon.CSIDriver
 
@@ -56,19 +56,19 @@ var (
 	volJournal *journal.Config
 )
 
-// NewDriver returns new ceph driver
+// NewDriver returns new ceph driver.
 func NewDriver() *Driver {
 	return &Driver{}
 }
 
-// NewIdentityServer initialize a identity server for ceph CSI driver
+// NewIdentityServer initialize a identity server for ceph CSI driver.
 func NewIdentityServer(d *csicommon.CSIDriver) *IdentityServer {
 	return &IdentityServer{
 		DefaultIdentityServer: csicommon.NewDefaultIdentityServer(d),
 	}
 }
 
-// NewControllerServer initialize a controller server for ceph CSI driver
+// NewControllerServer initialize a controller server for ceph CSI driver.
 func NewControllerServer(d *csicommon.CSIDriver) *ControllerServer {
 	return &ControllerServer{
 		DefaultControllerServer: csicommon.NewDefaultControllerServer(d),
@@ -85,7 +85,7 @@ func NewNodeServer(d *csicommon.CSIDriver, t string, topology map[string]string)
 }
 
 // Run start a non-blocking grpc controller,node and identityserver for
-// ceph CSI driver which can serve multiple parallel requests
+// ceph CSI driver which can serve multiple parallel requests.
 func (fs *Driver) Run(conf *util.Config) {
 	var err error
 	var topology map[string]string

internal/cephfs/errors.go

@@ -17,7 +17,7 @@ limitations under the License.
 package cephfs
 
 // ErrInvalidVolID is returned when a CSI passed VolumeID is not conformant to any known volume ID
-// formats
+// formats.
 type ErrInvalidVolID struct {
 	err error
 }
@@ -33,7 +33,7 @@ func (e ErrInvalidVolID) Unwrap() error {
 }
 
 // ErrNonStaticVolume is returned when a volume is detected as not being
-// statically provisioned
+// statically provisioned.
 type ErrNonStaticVolume struct {
 	err error
 }
@@ -48,7 +48,7 @@ func (e ErrNonStaticVolume) Unwrap() error {
 	return e.err
 }
 
-// ErrVolumeNotFound is returned when a subvolume is not found in CephFS
+// ErrVolumeNotFound is returned when a subvolume is not found in CephFS.
 type ErrVolumeNotFound struct {
 	err error
 }

internal/cephfs/fsjournal.go

@@ -24,7 +24,7 @@ import (
 )
 
 // volumeIdentifier structure contains an association between the CSI VolumeID to its subvolume
-// name on the backing CephFS instance
+// name on the backing CephFS instance.
 type volumeIdentifier struct {
 	FsSubvolName string
 	VolumeID     string
@@ -100,7 +100,7 @@ func checkVolExists(ctx context.Context, volOptions *volumeOptions, secret map[s
 	return &vid, nil
 }
 
-// undoVolReservation is a helper routine to undo a name reservation for a CSI VolumeName
+// undoVolReservation is a helper routine to undo a name reservation for a CSI VolumeName.
 func undoVolReservation(ctx context.Context, volOptions *volumeOptions, vid volumeIdentifier, secret map[string]string) error {
 	cr, err := util.NewAdminCredentials(secret)
 	if err != nil {
@@ -135,7 +135,7 @@ func updateTopologyConstraints(volOpts *volumeOptions) error {
 }
 
 // reserveVol is a helper routine to request a UUID reservation for the CSI VolumeName and,
-// to generate the volume identifier for the reserved UUID
+// to generate the volume identifier for the reserved UUID.
 func reserveVol(ctx context.Context, volOptions *volumeOptions, secret map[string]string) (*volumeIdentifier, error) {
 	var (
 		vid       volumeIdentifier

internal/cephfs/identityserver.go

@@ -30,7 +30,7 @@ type IdentityServer struct {
 	*csicommon.DefaultIdentityServer
 }
 
-// GetPluginCapabilities returns available capabilities of the ceph driver
+// GetPluginCapabilities returns available capabilities of the ceph driver.
 func (is *IdentityServer) GetPluginCapabilities(ctx context.Context, req *csi.GetPluginCapabilitiesRequest) (*csi.GetPluginCapabilitiesResponse, error) {
 	return &csi.GetPluginCapabilitiesResponse{
 		Capabilities: []*csi.PluginCapability{

internal/cephfs/nodeserver.go

@@ -197,7 +197,7 @@ func (*NodeServer) mount(ctx context.Context, volOptions *volumeOptions, req *cs
 }
 
 // NodePublishVolume mounts the volume mounted to the staging path to the target
-// path
+// path.
 func (ns *NodeServer) NodePublishVolume(ctx context.Context, req *csi.NodePublishVolumeRequest) (*csi.NodePublishVolumeResponse, error) {
 	mountOptions := []string{"bind", "_netdev"}
 	if err := util.ValidateNodePublishVolumeRequest(req); err != nil {
@@ -250,7 +250,7 @@ func (ns *NodeServer) NodePublishVolume(ctx context.Context, req *csi.NodePublis
 	return &csi.NodePublishVolumeResponse{}, nil
 }
 
-// NodeUnpublishVolume unmounts the volume from the target path
+// NodeUnpublishVolume unmounts the volume from the target path.
 func (ns *NodeServer) NodeUnpublishVolume(ctx context.Context, req *csi.NodeUnpublishVolumeRequest) (*csi.NodeUnpublishVolumeResponse, error) {
 	var err error
 	if err = util.ValidateNodeUnpublishVolumeRequest(req); err != nil {
@@ -281,7 +281,7 @@ func (ns *NodeServer) NodeUnpublishVolume(ctx context.Context, req *csi.NodeUnpu
 	return &csi.NodeUnpublishVolumeResponse{}, nil
 }
 
-// NodeUnstageVolume unstages the volume from the staging path
+// NodeUnstageVolume unstages the volume from the staging path.
 func (ns *NodeServer) NodeUnstageVolume(ctx context.Context, req *csi.NodeUnstageVolumeRequest) (*csi.NodeUnstageVolumeResponse, error) {
 	var err error
 	if err = util.ValidateNodeUnstageVolumeRequest(req); err != nil {
@@ -306,7 +306,7 @@ func (ns *NodeServer) NodeUnstageVolume(ctx context.Context, req *csi.NodeUnstag
 	return &csi.NodeUnstageVolumeResponse{}, nil
 }
 
-// NodeGetCapabilities returns the supported capabilities of the node server
+// NodeGetCapabilities returns the supported capabilities of the node server.
 func (ns *NodeServer) NodeGetCapabilities(ctx context.Context, req *csi.NodeGetCapabilitiesRequest) (*csi.NodeGetCapabilitiesResponse, error) {
 	return &csi.NodeGetCapabilitiesResponse{
 		Capabilities: []*csi.NodeServiceCapability{

internal/cephfs/util.go

@@ -76,7 +76,7 @@ func execCommandJSON(ctx context.Context, v interface{}, program string, args ..
 	return nil
 }
 
-// Controller service request validation
+// Controller service request validation.
 func (cs *ControllerServer) validateCreateVolumeRequest(req *csi.CreateVolumeRequest) error {
 	if err := cs.Driver.ValidateControllerServiceRequest(csi.ControllerServiceCapability_RPC_CREATE_DELETE_VOLUME); err != nil {
 		return fmt.Errorf("invalid CreateVolumeRequest: %v", err)
@@ -108,7 +108,7 @@ func (cs *ControllerServer) validateDeleteVolumeRequest() error {
 	return nil
 }
 
-// Controller expand volume request validation
+// Controller expand volume request validation.
 func (cs *ControllerServer) validateExpandVolumeRequest(req *csi.ControllerExpandVolumeRequest) error {
 	if err := cs.Driver.ValidateControllerServiceRequest(csi.ControllerServiceCapability_RPC_EXPAND_VOLUME); err != nil {
 		return fmt.Errorf("invalid ExpandVolumeRequest: %v", err)

internal/cephfs/volumemounter.go

@@ -67,7 +67,7 @@ var (
 )
 
 // Load available ceph mounters installed on system into availableMounters
-// Called from driver.go's Run()
+// Called from driver.go's Run().
 func loadAvailableMounters(conf *util.Config) error {
 	// #nosec
 	fuseMounterProbe := exec.Command("ceph-fuse", "--version")

internal/cephfs/volumeoptions.go

@@ -142,7 +142,7 @@ func getClusterInformation(options map[string]string) (*util.ClusterInfo, error)
 }
 
 // newVolumeOptions generates a new instance of volumeOptions from the provided
-// CSI request parameters
+// CSI request parameters.
 func newVolumeOptions(ctx context.Context, requestName string, req *csi.CreateVolumeRequest,
 	secret map[string]string) (*volumeOptions, error) {
 	var (
@@ -218,7 +218,7 @@ func newVolumeOptions(ctx context.Context, requestName string, req *csi.CreateVo
 }
 
 // newVolumeOptionsFromVolID generates a new instance of volumeOptions and volumeIdentifier
-// from the provided CSI VolumeID
+// from the provided CSI VolumeID.
 func newVolumeOptionsFromVolID(ctx context.Context, volID string, volOpt, secrets map[string]string) (*volumeOptions, *volumeIdentifier, error) {
 	var (
 		vi         util.CSIIdentifier
@@ -308,7 +308,7 @@ func newVolumeOptionsFromVolID(ctx context.Context, volID string, volOpt, secret
 }
 
 // newVolumeOptionsFromMonitorList generates a new instance of volumeOptions and
-// volumeIdentifier from the provided CSI volume context
+// volumeIdentifier from the provided CSI volume context.
 func newVolumeOptionsFromMonitorList(volID string, options, secrets map[string]string) (*volumeOptions, *volumeIdentifier, error) {
 	var (
 		opts                volumeOptions
@@ -361,7 +361,7 @@ func newVolumeOptionsFromMonitorList(volID string, options, secrets map[string]s
 
 // newVolumeOptionsFromStaticVolume generates a new instance of volumeOptions and
 // volumeIdentifier from the provided CSI volume context, if the provided context is
-// detected to be a statically provisioned volume
+// detected to be a statically provisioned volume.
 func newVolumeOptionsFromStaticVolume(volID string, options map[string]string) (*volumeOptions, *volumeIdentifier, error) {
 	var (
 		opts      volumeOptions

internal/csi-common/controllerserver-default.go

@@ -26,38 +26,38 @@ import (
 	"google.golang.org/grpc/status"
 )
 
-// DefaultControllerServer points to default driver
+// DefaultControllerServer points to default driver.
 type DefaultControllerServer struct {
 	Driver *CSIDriver
 }
 
-// ControllerPublishVolume publish volume on node
+// ControllerPublishVolume publish volume on node.
 func (cs *DefaultControllerServer) ControllerPublishVolume(ctx context.Context, req *csi.ControllerPublishVolumeRequest) (*csi.ControllerPublishVolumeResponse, error) {
 	return nil, status.Error(codes.Unimplemented, "")
 }
 
-// ControllerUnpublishVolume unpublish on node
+// ControllerUnpublishVolume unpublish on node.
 func (cs *DefaultControllerServer) ControllerUnpublishVolume(ctx context.Context, req *csi.ControllerUnpublishVolumeRequest) (*csi.ControllerUnpublishVolumeResponse, error) {
 	return nil, status.Error(codes.Unimplemented, "")
 }
 
-// ControllerExpandVolume expand volume
+// ControllerExpandVolume expand volume.
 func (cs *DefaultControllerServer) ControllerExpandVolume(ctx context.Context, req *csi.ControllerExpandVolumeRequest) (*csi.ControllerExpandVolumeResponse, error) {
 	return nil, status.Error(codes.Unimplemented, "")
 }
 
-// ListVolumes lists volumes
+// ListVolumes lists volumes.
 func (cs *DefaultControllerServer) ListVolumes(ctx context.Context, req *csi.ListVolumesRequest) (*csi.ListVolumesResponse, error) {
 	return nil, status.Error(codes.Unimplemented, "")
 }
 
-// GetCapacity get volume capacity
+// GetCapacity get volume capacity.
 func (cs *DefaultControllerServer) GetCapacity(ctx context.Context, req *csi.GetCapacityRequest) (*csi.GetCapacityResponse, error) {
 	return nil, status.Error(codes.Unimplemented, "")
 }
 
 // ControllerGetCapabilities implements the default GRPC callout.
-// Default supports all capabilities
+// Default supports all capabilities.
 func (cs *DefaultControllerServer) ControllerGetCapabilities(ctx context.Context, req *csi.ControllerGetCapabilitiesRequest) (*csi.ControllerGetCapabilitiesResponse, error) {
 	util.TraceLog(ctx, "Using default ControllerGetCapabilities")
 
@@ -66,17 +66,17 @@ func (cs *DefaultControllerServer) ControllerGetCapabilities(ctx context.Context
 	}, nil
 }
 
-// CreateSnapshot creates snapshot
+// CreateSnapshot creates snapshot.
 func (cs *DefaultControllerServer) CreateSnapshot(ctx context.Context, req *csi.CreateSnapshotRequest) (*csi.CreateSnapshotResponse, error) {
 	return nil, status.Error(codes.Unimplemented, "")
 }
 
-// DeleteSnapshot deletes snapshot
+// DeleteSnapshot deletes snapshot.
 func (cs *DefaultControllerServer) DeleteSnapshot(ctx context.Context, req *csi.DeleteSnapshotRequest) (*csi.DeleteSnapshotResponse, error) {
 	return nil, status.Error(codes.Unimplemented, "")
 }
 
-// ListSnapshots lists snapshots
+// ListSnapshots lists snapshots.
 func (cs *DefaultControllerServer) ListSnapshots(ctx context.Context, req *csi.ListSnapshotsRequest) (*csi.ListSnapshotsResponse, error) {
 	return nil, status.Error(codes.Unimplemented, "")
 }

internal/csi-common/driver.go

@@ -27,7 +27,7 @@ import (
 	"github.com/ceph/ceph-csi/internal/util"
 )
 
-// CSIDriver stores driver information
+// CSIDriver stores driver information.
 type CSIDriver struct {
 	name    string
 	nodeID  string
@@ -67,7 +67,7 @@ func NewCSIDriver(name, v, nodeID string) *CSIDriver {
 }
 
 // ValidateControllerServiceRequest validates the controller
-// plugin capabilities
+// plugin capabilities.
 func (d *CSIDriver) ValidateControllerServiceRequest(c csi.ControllerServiceCapability_RPC_Type) error {
 	if c == csi.ControllerServiceCapability_RPC_UNKNOWN {
 		return nil
@@ -82,7 +82,7 @@ func (d *CSIDriver) ValidateControllerServiceRequest(c csi.ControllerServiceCapa
 }
 
 // AddControllerServiceCapabilities stores the controller capabilities
-// in driver object
+// in driver object.
 func (d *CSIDriver) AddControllerServiceCapabilities(cl []csi.ControllerServiceCapability_RPC_Type) {
 	var csc []*csi.ControllerServiceCapability
 
@@ -94,7 +94,7 @@ func (d *CSIDriver) AddControllerServiceCapabilities(cl []csi.ControllerServiceC
 	d.cap = csc
 }
 
-// AddVolumeCapabilityAccessModes stores volume access modes
+// AddVolumeCapabilityAccessModes stores volume access modes.
 func (d *CSIDriver) AddVolumeCapabilityAccessModes(vc []csi.VolumeCapability_AccessMode_Mode) []*csi.VolumeCapability_AccessMode {
 	var vca []*csi.VolumeCapability_AccessMode
 	for _, c := range vc {
@@ -105,7 +105,7 @@ func (d *CSIDriver) AddVolumeCapabilityAccessModes(vc []csi.VolumeCapability_Acc
 	return vca
 }
 
-// GetVolumeCapabilityAccessModes returns access modes
+// GetVolumeCapabilityAccessModes returns access modes.
 func (d *CSIDriver) GetVolumeCapabilityAccessModes() []*csi.VolumeCapability_AccessMode {
 	return d.vc
 }

internal/csi-common/identityserver-default.go

@@ -26,12 +26,12 @@ import (
 	"google.golang.org/grpc/status"
 )
 
-// DefaultIdentityServer stores driver object
+// DefaultIdentityServer stores driver object.
 type DefaultIdentityServer struct {
 	Driver *CSIDriver
 }
 
-// GetPluginInfo returns plugin information
+// GetPluginInfo returns plugin information.
 func (ids *DefaultIdentityServer) GetPluginInfo(ctx context.Context, req *csi.GetPluginInfoRequest) (*csi.GetPluginInfoResponse, error) {
 	util.TraceLog(ctx, "Using default GetPluginInfo")
 
@@ -49,12 +49,12 @@ func (ids *DefaultIdentityServer) GetPluginInfo(ctx context.Context, req *csi.Ge
 	}, nil
 }
 
-// Probe returns empty response
+// Probe returns empty response.
 func (ids *DefaultIdentityServer) Probe(ctx context.Context, req *csi.ProbeRequest) (*csi.ProbeResponse, error) {
 	return &csi.ProbeResponse{}, nil
 }
 
-// GetPluginCapabilities returns plugin capabilities
+// GetPluginCapabilities returns plugin capabilities.
 func (ids *DefaultIdentityServer) GetPluginCapabilities(ctx context.Context, req *csi.GetPluginCapabilitiesRequest) (*csi.GetPluginCapabilitiesResponse, error) {
 	util.TraceLog(ctx, "Using default capabilities")
 	return &csi.GetPluginCapabilitiesResponse{

internal/csi-common/nodeserver-default.go

@@ -31,28 +31,28 @@ import (
 	"k8s.io/kubernetes/pkg/volume"
 )
 
-// DefaultNodeServer stores driver object
+// DefaultNodeServer stores driver object.
 type DefaultNodeServer struct {
 	Driver *CSIDriver
 	Type   string
 }
 
-// NodeStageVolume returns unimplemented response
+// NodeStageVolume returns unimplemented response.
 func (ns *DefaultNodeServer) NodeStageVolume(ctx context.Context, req *csi.NodeStageVolumeRequest) (*csi.NodeStageVolumeResponse, error) {
 	return nil, status.Error(codes.Unimplemented, "")
 }
 
-// NodeUnstageVolume returns unimplemented response
+// NodeUnstageVolume returns unimplemented response.
 func (ns *DefaultNodeServer) NodeUnstageVolume(ctx context.Context, req *csi.NodeUnstageVolumeRequest) (*csi.NodeUnstageVolumeResponse, error) {
 	return nil, status.Error(codes.Unimplemented, "")
 }
 
-// NodeExpandVolume returns unimplemented response
+// NodeExpandVolume returns unimplemented response.
 func (ns *DefaultNodeServer) NodeExpandVolume(ctx context.Context, req *csi.NodeExpandVolumeRequest) (*csi.NodeExpandVolumeResponse, error) {
 	return nil, status.Error(codes.Unimplemented, "")
 }
 
-// NodeGetInfo returns node ID
+// NodeGetInfo returns node ID.
 func (ns *DefaultNodeServer) NodeGetInfo(ctx context.Context, req *csi.NodeGetInfoRequest) (*csi.NodeGetInfoResponse, error) {
 	util.TraceLog(ctx, "Using default NodeGetInfo")
 
@@ -66,7 +66,7 @@ func (ns *DefaultNodeServer) NodeGetInfo(ctx context.Context, req *csi.NodeGetIn
 	}, nil
 }
 
-// NodeGetCapabilities returns RPC unknow capability
+// NodeGetCapabilities returns RPC unknow capability.
 func (ns *DefaultNodeServer) NodeGetCapabilities(ctx context.Context, req *csi.NodeGetCapabilitiesRequest) (*csi.NodeGetCapabilitiesResponse, error) {
 	util.TraceLog(ctx, "Using default NodeGetCapabilities")
 
@@ -83,7 +83,7 @@ func (ns *DefaultNodeServer) NodeGetCapabilities(ctx context.Context, req *csi.N
 	}, nil
 }
 
-// NodeGetVolumeStats returns volume stats
+// NodeGetVolumeStats returns volume stats.
 func (ns *DefaultNodeServer) NodeGetVolumeStats(ctx context.Context, req *csi.NodeGetVolumeStatsRequest) (*csi.NodeGetVolumeStatsResponse, error) {
 	var err error
 	targetPath := req.GetVolumePath()
@@ -175,7 +175,7 @@ func (ns *DefaultNodeServer) NodeGetVolumeStats(ctx context.Context, req *csi.No
 	}, nil
 }
 
-// ConstructMountOptions returns only unique mount options in slice
+// ConstructMountOptions returns only unique mount options in slice.
 func ConstructMountOptions(mountOptions []string, volCap *csi.VolumeCapability) []string {
 	if m := volCap.GetMount(); m != nil {
 		hasOption := func(options []string, opt string) bool {
@@ -195,7 +195,7 @@ func ConstructMountOptions(mountOptions []string, volCap *csi.VolumeCapability)
 	return mountOptions
 }
 
-// MountOptionContains checks the opt is present in mountOptions
+// MountOptionContains checks the opt is present in mountOptions.
 func MountOptionContains(mountOptions []string, opt string) bool {
 	for _, mnt := range mountOptions {
 		if mnt == opt {

internal/csi-common/server.go

@@ -33,7 +33,7 @@ import (
 	"github.com/ceph/ceph-csi/internal/util"
 )
 
-// NonBlockingGRPCServer defines Non blocking GRPC server interfaces
+// NonBlockingGRPCServer defines Non blocking GRPC server interfaces.
 type NonBlockingGRPCServer interface {
 	// Start services at the endpoint
 	Start(endpoint, hstOptions string, ids csi.IdentityServer, cs csi.ControllerServer, ns csi.NodeServer, metrics bool)
@@ -45,24 +45,24 @@ type NonBlockingGRPCServer interface {
 	ForceStop()
 }
 
-// NewNonBlockingGRPCServer return non-blocking GRPC
+// NewNonBlockingGRPCServer return non-blocking GRPC.
 func NewNonBlockingGRPCServer() NonBlockingGRPCServer {
 	return &nonBlockingGRPCServer{}
 }
 
-// NonBlocking server
+// NonBlocking server.
 type nonBlockingGRPCServer struct {
 	wg     sync.WaitGroup
 	server *grpc.Server
 }
 
-// Start start service on endpoint
+// Start start service on endpoint.
 func (s *nonBlockingGRPCServer) Start(endpoint, hstOptions string, ids csi.IdentityServer, cs csi.ControllerServer, ns csi.NodeServer, metrics bool) {
 	s.wg.Add(1)
 	go s.serve(endpoint, hstOptions, ids, cs, ns, metrics)
 }
 
-// Wait blocks until the WaitGroup counter
+// Wait blocks until the WaitGroup counter.
 func (s *nonBlockingGRPCServer) Wait() {
 	s.wg.Wait()
 }

internal/csi-common/utils.go

@@ -43,12 +43,12 @@ func parseEndpoint(ep string) (string, string, error) {
 	return "", "", fmt.Errorf("invalid endpoint: %v", ep)
 }
 
-// NewVolumeCapabilityAccessMode returns volume access mode
+// NewVolumeCapabilityAccessMode returns volume access mode.
 func NewVolumeCapabilityAccessMode(mode csi.VolumeCapability_AccessMode_Mode) *csi.VolumeCapability_AccessMode {
 	return &csi.VolumeCapability_AccessMode{Mode: mode}
 }
 
-// NewDefaultNodeServer initializes default node server
+// NewDefaultNodeServer initializes default node server.
 func NewDefaultNodeServer(d *CSIDriver, t string, topology map[string]string) *DefaultNodeServer {
 	d.topology = topology
 	return &DefaultNodeServer{
@@ -57,21 +57,21 @@ func NewDefaultNodeServer(d *CSIDriver, t string, topology map[string]string) *D
 	}
 }
 
-// NewDefaultIdentityServer initializes default identity servier
+// NewDefaultIdentityServer initializes default identity servier.
 func NewDefaultIdentityServer(d *CSIDriver) *DefaultIdentityServer {
 	return &DefaultIdentityServer{
 		Driver: d,
 	}
 }
 
-// NewDefaultControllerServer initializes default controller server
+// NewDefaultControllerServer initializes default controller server.
 func NewDefaultControllerServer(d *CSIDriver) *DefaultControllerServer {
 	return &DefaultControllerServer{
 		Driver: d,
 	}
 }
 
-// NewControllerServiceCapability returns controller capabilities
+// NewControllerServiceCapability returns controller capabilities.
 func NewControllerServiceCapability(ctrlCap csi.ControllerServiceCapability_RPC_Type) *csi.ControllerServiceCapability {
 	return &csi.ControllerServiceCapability{
 		Type: &csi.ControllerServiceCapability_Rpc{
@@ -82,7 +82,7 @@ func NewControllerServiceCapability(ctrlCap csi.ControllerServiceCapability_RPC_
 	}
 }
 
-// RunNodePublishServer starts node server
+// RunNodePublishServer starts node server.
 func RunNodePublishServer(endpoint, hstOption string, d *CSIDriver, ns csi.NodeServer, m bool) {
 	ids := NewDefaultIdentityServer(d)
 
@@ -91,7 +91,7 @@ func RunNodePublishServer(endpoint, hstOption string, d *CSIDriver, ns csi.NodeS
 	s.Wait()
 }
 
-// RunControllerPublishServer starts controller server
+// RunControllerPublishServer starts controller server.
 func RunControllerPublishServer(endpoint, hstOption string, d *CSIDriver, cs csi.ControllerServer, m bool) {
 	ids := NewDefaultIdentityServer(d)
 
@@ -100,7 +100,7 @@ func RunControllerPublishServer(endpoint, hstOption string, d *CSIDriver, cs csi
 	s.Wait()
 }
 
-// RunControllerandNodePublishServer starts both controller and node server
+// RunControllerandNodePublishServer starts both controller and node server.
 func RunControllerandNodePublishServer(endpoint, hstOption string, d *CSIDriver, cs csi.ControllerServer, ns csi.NodeServer, m bool) {
 	ids := NewDefaultIdentityServer(d)
 

internal/journal/voljournal.go

@@ -30,7 +30,7 @@ import (
 	klog "k8s.io/klog/v2"
 )
 
-// Length of string representation of uuid, xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx is 36 bytes
+// Length of string representation of uuid, xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx is 36 bytes.
 const uuidEncodedLength = 36
 
 /*
@@ -111,7 +111,7 @@ const (
 	defaultSnapshotNamingPrefix string = "csi-snap-"
 )
 
-// CSIJournal defines the interface and the required key names for the above RADOS based OMaps
+// CSIJournal defines the interface and the required key names for the above RADOS based OMaps.
 type Config struct {
 	// csiDirectory is the name of the CSI volumes object map that contains CSI volume-name (or
 	// snapshot name) based keys
@@ -153,7 +153,7 @@ type Config struct {
 	commonPrefix string
 }
 
-// NewCSIVolumeJournal returns an instance of CSIJournal for volumes
+// NewCSIVolumeJournal returns an instance of CSIJournal for volumes.
 func NewCSIVolumeJournal(suffix string) *Config {
 	return &Config{
 		csiDirectory:            "csi.volumes." + suffix,
@@ -170,7 +170,7 @@ func NewCSIVolumeJournal(suffix string) *Config {
 	}
 }
 
-// NewCSISnapshotJournal returns an instance of CSIJournal for snapshots
+// NewCSISnapshotJournal returns an instance of CSIJournal for snapshots.
 func NewCSISnapshotJournal(suffix string) *Config {
 	return &Config{
 		csiDirectory:            "csi.snaps." + suffix,
@@ -195,7 +195,7 @@ func NewCSIVolumeJournalWithNamespace(suffix, ns string) *Config {
 	return j
 }
 
-// GetNameForUUID returns volume name
+// GetNameForUUID returns volume name.
 func (cj *Config) GetNameForUUID(prefix, uid string, isSnapshot bool) string {
 	if prefix == "" {
 		if isSnapshot {
@@ -207,7 +207,7 @@ func (cj *Config) GetNameForUUID(prefix, uid string, isSnapshot bool) string {
 	return prefix + uid
 }
 
-// ImageData contains image name and stored CSI properties
+// ImageData contains image name and stored CSI properties.
 type ImageData struct {
 	ImageUUID       string
 	ImagePool       string
@@ -433,7 +433,7 @@ func (conn *Connection) UndoReservation(ctx context.Context,
 
 // reserveOMapName creates an omap with passed in oMapNamePrefix and a generated <uuid>.
 // It ensures generated omap name does not already exist and if conflicts are detected, a set
-// number of retires with newer uuids are attempted before returning an error
+// number of retires with newer uuids are attempted before returning an error.
 func reserveOMapName(ctx context.Context, monitors string, cr *util.Credentials, pool, namespace, oMapNamePrefix string) (string, error) {
 	var iterUUID string
 
@@ -581,7 +581,7 @@ func (conn *Connection) ReserveName(ctx context.Context,
 	return volUUID, imageName, nil
 }
 
-// ImageAttributes contains all CSI stored image attributes, typically as OMap keys
+// ImageAttributes contains all CSI stored image attributes, typically as OMap keys.
 type ImageAttributes struct {
 	RequestName   string // Contains the request name for the passed in UUID
 	SourceName    string // Contains the parent image name for the passed in UUID, if it is a snapshot
@@ -591,7 +591,7 @@ type ImageAttributes struct {
 	JournalPoolID int64  // Pool ID of the CSI journal pool, stored in big endian format (on-disk data)
 }
 
-// GetImageAttributes fetches all keys and their values, from a UUID directory, returning ImageAttributes structure
+// GetImageAttributes fetches all keys and their values, from a UUID directory, returning ImageAttributes structure.
 func (conn *Connection) GetImageAttributes(ctx context.Context, pool, objectUUID string, snapSource bool) (*ImageAttributes, error) {
 	var (
 		err             error
@@ -665,7 +665,7 @@ func (conn *Connection) GetImageAttributes(ctx context.Context, pool, objectUUID
 	return imageAttributes, nil
 }
 
-// StoreImageID stores the image ID in omap
+// StoreImageID stores the image ID in omap.
 func (conn *Connection) StoreImageID(ctx context.Context, pool, reservedUUID, imageID string, cr *util.Credentials) error {
 	err := setOMapKeys(ctx, conn, pool, conn.config.namespace, conn.config.cephUUIDDirectoryPrefix+reservedUUID,
 		map[string]string{conn.config.csiImageIDKey: imageID})

internal/liveness/liveness.go

@@ -82,7 +82,7 @@ func recordLiveness(endpoint, drivername string, pollTime, timeout time.Duration
 	}
 }
 
-// Run starts liveness collection and prometheus endpoint
+// Run starts liveness collection and prometheus endpoint.
 func Run(conf *util.Config) {
 	util.ExtendedLogMsg("Liveness Running")
 

internal/rbd/controllerserver.go

@@ -202,7 +202,7 @@ func validateRequestedVolumeSize(rbdVol, parentVol *rbdVolume, rbdSnap *rbdSnaps
 	return nil
 }
 
-// CreateVolume creates the volume in backend
+// CreateVolume creates the volume in backend.
 func (cs *ControllerServer) CreateVolume(ctx context.Context, req *csi.CreateVolumeRequest) (*csi.CreateVolumeResponse, error) {
 	if err := cs.validateVolumeReq(ctx, req); err != nil {
 		return nil, err
@@ -357,7 +357,7 @@ func flattenTemporaryClonedImages(ctx context.Context, rbdVol *rbdVolume, cr *ut
 // checkFlatten ensures that that the image chain depth is not reached
 // hardlimit or softlimit. if the softlimit is reached it adds a task and
 // return success,the hardlimit is reached it starts a task to flatten the
-// image and return Aborted
+// image and return Aborted.
 func checkFlatten(ctx context.Context, rbdVol *rbdVolume, cr *util.Credentials) error {
 	err := rbdVol.flattenRbdImage(ctx, cr, false, rbdHardMaxCloneDepth, rbdSoftMaxCloneDepth)
 	if err != nil {
@@ -931,7 +931,7 @@ func (cs *ControllerServer) doSnapshotClone(ctx context.Context, parentVol *rbdV
 }
 
 // DeleteSnapshot deletes the snapshot in backend and removes the
-// snapshot metadata from store
+// snapshot metadata from store.
 func (cs *ControllerServer) DeleteSnapshot(ctx context.Context, req *csi.DeleteSnapshotRequest) (*csi.DeleteSnapshotResponse, error) {
 	if err := cs.Driver.ValidateControllerServiceRequest(csi.ControllerServiceCapability_RPC_CREATE_DELETE_SNAPSHOT); err != nil {
 		klog.Errorf(util.Log(ctx, "invalid delete snapshot req: %v"), protosanitizer.StripSecrets(req))
@@ -1022,7 +1022,7 @@ func (cs *ControllerServer) DeleteSnapshot(ctx context.Context, req *csi.DeleteS
 	return &csi.DeleteSnapshotResponse{}, nil
 }
 
-// ControllerExpandVolume expand RBD Volumes on demand based on resizer request
+// ControllerExpandVolume expand RBD Volumes on demand based on resizer request.
 func (cs *ControllerServer) ControllerExpandVolume(ctx context.Context, req *csi.ControllerExpandVolumeRequest) (*csi.ControllerExpandVolumeResponse, error) {
 	if err := cs.Driver.ValidateControllerServiceRequest(csi.ControllerServiceCapability_RPC_EXPAND_VOLUME); err != nil {
 		klog.Errorf(util.Log(ctx, "invalid expand volume req: %v"), protosanitizer.StripSecrets(req))

internal/rbd/driver.go

@@ -34,7 +34,7 @@ const (
 	csiConfigFile = "/etc/ceph-csi-config/config.json"
 )
 
-// Driver contains the default identity,node and controller struct
+// Driver contains the default identity,node and controller struct.
 type Driver struct {
 	cd *csicommon.CSIDriver
 
@@ -62,19 +62,19 @@ var (
 	skipForceFlatten     bool
 )
 
-// NewDriver returns new rbd driver
+// NewDriver returns new rbd driver.
 func NewDriver() *Driver {
 	return &Driver{}
 }
 
-// NewIdentityServer initialize a identity server for rbd CSI driver
+// NewIdentityServer initialize a identity server for rbd CSI driver.
 func NewIdentityServer(d *csicommon.CSIDriver) *IdentityServer {
 	return &IdentityServer{
 		DefaultIdentityServer: csicommon.NewDefaultIdentityServer(d),
 	}
 }
 
-// NewControllerServer initialize a controller server for rbd CSI driver
+// NewControllerServer initialize a controller server for rbd CSI driver.
 func NewControllerServer(d *csicommon.CSIDriver) *ControllerServer {
 	return &ControllerServer{
 		DefaultControllerServer: csicommon.NewDefaultControllerServer(d),
@@ -94,7 +94,7 @@ func NewNodeServer(d *csicommon.CSIDriver, t string, topology map[string]string)
 }
 
 // Run start a non-blocking grpc controller,node and identityserver for
-// rbd CSI driver which can serve multiple parallel requests
+// rbd CSI driver which can serve multiple parallel requests.
 func (r *Driver) Run(conf *util.Config) {
 	var err error
 	var topology map[string]string

internal/rbd/errors.go

@@ -16,7 +16,7 @@ limitations under the License.
 
 package rbd
 
-// ErrImageNotFound is returned when image name is not found in the cluster on the given pool
+// ErrImageNotFound is returned when image name is not found in the cluster on the given pool.
 type ErrImageNotFound struct {
 	imageName string
 	err       error
@@ -33,7 +33,7 @@ func (e ErrImageNotFound) Unwrap() error {
 }
 
 // ErrSnapNotFound is returned when snap name passed is not found in the list of snapshots for the
-// given image
+// given image.
 type ErrSnapNotFound struct {
 	snapName string
 	err      error
@@ -50,7 +50,7 @@ func (e ErrSnapNotFound) Unwrap() error {
 }
 
 // ErrVolNameConflict is generated when a requested CSI volume name already exists on RBD but with
-// different properties, and hence is in conflict with the passed in CSI volume name
+// different properties, and hence is in conflict with the passed in CSI volume name.
 type ErrVolNameConflict struct {
 	requestName string
 	err         error
@@ -67,7 +67,7 @@ func (e ErrVolNameConflict) Unwrap() error {
 }
 
 // ErrInvalidVolID is returned when a CSI passed VolumeID does not conform to any known volume ID
-// formats
+// formats.
 type ErrInvalidVolID struct {
 	err error
 }
@@ -82,7 +82,7 @@ func (e ErrInvalidVolID) Unwrap() error {
 	return e.err
 }
 
-// ErrMissingStash is returned when the image metadata stash file is not found
+// ErrMissingStash is returned when the image metadata stash file is not found.
 type ErrMissingStash struct {
 	err error
 }
@@ -97,7 +97,7 @@ func (e ErrMissingStash) Unwrap() error {
 	return e.err
 }
 
-// ErrFlattenInProgress is returned when flatten is inprogess for an image
+// ErrFlattenInProgress is returned when flatten is inprogess for an image.
 type ErrFlattenInProgress struct {
 	err error
 }

internal/rbd/identityserver.go

@@ -30,7 +30,7 @@ type IdentityServer struct {
 	*csicommon.DefaultIdentityServer
 }
 
-// GetPluginCapabilities returns available capabilities of the rbd driver
+// GetPluginCapabilities returns available capabilities of the rbd driver.
 func (is *IdentityServer) GetPluginCapabilities(ctx context.Context, req *csi.GetPluginCapabilitiesRequest) (*csi.GetPluginCapabilitiesResponse, error) {
 	return &csi.GetPluginCapabilitiesResponse{
 		Capabilities: []*csi.PluginCapability{

internal/rbd/nodeserver.go

@@ -38,7 +38,7 @@ import (
 )
 
 // NodeServer struct of ceph rbd driver with supported methods of CSI
-// node server spec
+// node server spec.
 type NodeServer struct {
 	*csicommon.DefaultNodeServer
 	mounter mount.Interface
@@ -49,7 +49,7 @@ type NodeServer struct {
 
 // stageTransaction struct represents the state a transaction was when it either completed
 // or failed
-// this transaction state can be used to rollback the transaction
+// this transaction state can be used to rollback the transaction.
 type stageTransaction struct {
 	// isStagePathCreated represents whether the mount path to stage the volume on was created or not
 	isStagePathCreated bool
@@ -375,7 +375,7 @@ func (ns *NodeServer) createStageMountPoint(ctx context.Context, mountPath strin
 }
 
 // NodePublishVolume mounts the volume mounted to the device path to the target
-// path
+// path.
 func (ns *NodeServer) NodePublishVolume(ctx context.Context, req *csi.NodePublishVolumeRequest) (*csi.NodePublishVolumeResponse, error) {
 	err := util.ValidateNodePublishVolumeRequest(req)
 	if err != nil {
@@ -539,7 +539,7 @@ func (ns *NodeServer) createTargetMountPath(ctx context.Context, mountPath strin
 	return notMnt, err
 }
 
-// NodeUnpublishVolume unmounts the volume from the target path
+// NodeUnpublishVolume unmounts the volume from the target path.
 func (ns *NodeServer) NodeUnpublishVolume(ctx context.Context, req *csi.NodeUnpublishVolumeRequest) (*csi.NodeUnpublishVolumeResponse, error) {
 	err := util.ValidateNodeUnpublishVolumeRequest(req)
 	if err != nil {
@@ -585,7 +585,7 @@ func (ns *NodeServer) NodeUnpublishVolume(ctx context.Context, req *csi.NodeUnpu
 }
 
 // getStagingTargetPath concats either NodeStageVolumeRequest's or
-// NodeUnstageVolumeRequest's target path with the volumeID
+// NodeUnstageVolumeRequest's target path with the volumeID.
 func getStagingTargetPath(req interface{}) string {
 	switch vr := req.(type) {
 	case *csi.NodeStageVolumeRequest:
@@ -597,7 +597,7 @@ func getStagingTargetPath(req interface{}) string {
 	return ""
 }
 
-// NodeUnstageVolume unstages the volume from the staging path
+// NodeUnstageVolume unstages the volume from the staging path.
 func (ns *NodeServer) NodeUnstageVolume(ctx context.Context, req *csi.NodeUnstageVolumeRequest) (*csi.NodeUnstageVolumeResponse, error) {
 	var err error
 	if err = util.ValidateNodeUnstageVolumeRequest(req); err != nil {
@@ -680,7 +680,7 @@ func (ns *NodeServer) NodeUnstageVolume(ctx context.Context, req *csi.NodeUnstag
 	return &csi.NodeUnstageVolumeResponse{}, nil
 }
 
-// NodeExpandVolume resizes rbd volumes
+// NodeExpandVolume resizes rbd volumes.
 func (ns *NodeServer) NodeExpandVolume(ctx context.Context, req *csi.NodeExpandVolumeRequest) (*csi.NodeExpandVolumeResponse, error) {
 	volumeID := req.GetVolumeId()
 	if volumeID == "" {
@@ -740,7 +740,7 @@ func getDevicePath(ctx context.Context, volumePath string) (string, error) {
 	return "", fmt.Errorf("failed to get device for stagingtarget path %v", volumePath)
 }
 
-// NodeGetCapabilities returns the supported capabilities of the node server
+// NodeGetCapabilities returns the supported capabilities of the node server.
 func (ns *NodeServer) NodeGetCapabilities(ctx context.Context, req *csi.NodeGetCapabilitiesRequest) (*csi.NodeGetCapabilitiesResponse, error) {
 	return &csi.NodeGetCapabilitiesResponse{
 		Capabilities: []*csi.NodeServiceCapability{

internal/rbd/rbd_attach.go

@@ -56,7 +56,7 @@ func init() {
 	hasNBD = checkRbdNbdTools()
 }
 
-// rbdDeviceInfo strongly typed JSON spec for rbd device list output (of type krbd)
+// rbdDeviceInfo strongly typed JSON spec for rbd device list output (of type krbd).
 type rbdDeviceInfo struct {
 	ID     string `json:"id"`
 	Pool   string `json:"pool"`
@@ -67,7 +67,7 @@ type rbdDeviceInfo struct {
 // nbdDeviceInfo strongly typed JSON spec for rbd-nbd device list output (of type nbd)
 // NOTE: There is a bug in rbd output that returns id as number for nbd, and string for krbd, thus
 // requiring 2 different JSON structures to unmarshal the output.
-// NOTE: image key is "name" in krbd output and "image" in nbd output, which is another difference
+// NOTE: image key is "name" in krbd output and "image" in nbd output, which is another difference.
 type nbdDeviceInfo struct {
 	ID     int64  `json:"id"`
 	Pool   string `json:"pool"`
@@ -76,7 +76,7 @@ type nbdDeviceInfo struct {
 }
 
 // rbdGetDeviceList queries rbd about mapped devices and returns a list of rbdDeviceInfo
-// It will selectively list devices mapped using krbd or nbd as specified by accessType
+// It will selectively list devices mapped using krbd or nbd as specified by accessType.
 func rbdGetDeviceList(accessType string) ([]rbdDeviceInfo, error) {
 	// rbd device list --format json --device-type [krbd|nbd]
 	var (
@@ -278,7 +278,7 @@ func detachRBDDevice(ctx context.Context, devicePath, volumeID string, encrypted
 }
 
 // detachRBDImageOrDeviceSpec detaches an rbd imageSpec or devicePath, with additional checking
-// when imageSpec is used to decide if image is already unmapped
+// when imageSpec is used to decide if image is already unmapped.
 func detachRBDImageOrDeviceSpec(ctx context.Context, imageOrDeviceSpec string, isImageSpec, ndbType, encrypted bool, volumeID string) error {
 	var output []byte
 

internal/rbd/rbd_journal.go

@@ -339,7 +339,7 @@ func (rv *rbdVolume) Exists(ctx context.Context, parentVol *rbdVolume) (bool, er
 }
 
 // reserveSnap is a helper routine to request a rbdSnapshot name reservation and generate the
-// volume ID for the generated name
+// volume ID for the generated name.
 func reserveSnap(ctx context.Context, rbdSnap *rbdSnapshot, rbdVol *rbdVolume, cr *util.Credentials) error {
 	var (
 		err error
@@ -407,7 +407,7 @@ func updateTopologyConstraints(rbdVol *rbdVolume, rbdSnap *rbdSnapshot) error {
 }
 
 // reserveVol is a helper routine to request a rbdVolume name reservation and generate the
-// volume ID for the generated name
+// volume ID for the generated name.
 func reserveVol(ctx context.Context, rbdVol *rbdVolume, rbdSnap *rbdSnapshot, cr *util.Credentials) error {
 	var (
 		err error
@@ -453,7 +453,7 @@ func reserveVol(ctx context.Context, rbdVol *rbdVolume, rbdSnap *rbdSnapshot, cr
 	return nil
 }
 
-// undoSnapReservation is a helper routine to undo a name reservation for rbdSnapshot
+// undoSnapReservation is a helper routine to undo a name reservation for rbdSnapshot.
 func undoSnapReservation(ctx context.Context, rbdSnap *rbdSnapshot, cr *util.Credentials) error {
 	j, err := snapJournal.Connect(rbdSnap.Monitors, cr)
 	if err != nil {
@@ -468,7 +468,7 @@ func undoSnapReservation(ctx context.Context, rbdSnap *rbdSnapshot, cr *util.Cre
 	return err
 }
 
-// undoVolReservation is a helper routine to undo a name reservation for rbdVolume
+// undoVolReservation is a helper routine to undo a name reservation for rbdVolume.
 func undoVolReservation(ctx context.Context, rbdVol *rbdVolume, cr *util.Credentials) error {
 	j, err := volJournal.Connect(rbdVol.Monitors, cr)
 	if err != nil {

internal/rbd/rbd_util.go

@@ -67,7 +67,7 @@ const (
 	imageOptionCloneFormat = librbd.RbdImageOption(12)
 )
 
-// rbdVolume represents a CSI volume and its RBD image specifics
+// rbdVolume represents a CSI volume and its RBD image specifics.
 type rbdVolume struct {
 	// RbdImageName is the name of the RBD image backing this rbdVolume. This does not have a
 	//   JSON tag as it is not stashed in JSON encoded config maps in v1.0.0
@@ -114,7 +114,7 @@ type rbdVolume struct {
 	ioctx *rados.IOContext
 }
 
-// rbdSnapshot represents a CSI snapshot and its RBD snapshot specifics
+// rbdSnapshot represents a CSI snapshot and its RBD snapshot specifics.
 type rbdSnapshot struct {
 	// SourceVolumeID is the volume ID of RbdImageName, that is exchanged with CSI drivers
 	// RbdImageName is the name of the RBD image, that is this rbdSnapshot's source image
@@ -144,7 +144,7 @@ var (
 	supportedFeatures = sets.NewString(librbd.FeatureNameLayering)
 )
 
-// Connect an rbdVolume to the Ceph cluster
+// Connect an rbdVolume to the Ceph cluster.
 func (rv *rbdVolume) Connect(cr *util.Credentials) error {
 	if rv.conn != nil {
 		return nil
@@ -170,12 +170,12 @@ func (rv *rbdVolume) Destroy() {
 	}
 }
 
-// String returns the image-spec (pool/image) format of the image
+// String returns the image-spec (pool/image) format of the image.
 func (rv *rbdVolume) String() string {
 	return fmt.Sprintf("%s/%s", rv.Pool, rv.RbdImageName)
 }
 
-// String returns the snap-spec (pool/image@snap) format of the snapshot
+// String returns the snap-spec (pool/image@snap) format of the snapshot.
 func (rs *rbdSnapshot) String() string {
 	return fmt.Sprintf("%s/%s@%s", rs.Pool, rs.RbdImageName, rs.RbdSnapName)
 }
@@ -239,7 +239,7 @@ func (rv *rbdVolume) openIoctx() error {
 }
 
 // getImageID queries rbd about the given image and stores its id, returns
-// ErrImageNotFound if provided image is not found
+// ErrImageNotFound if provided image is not found.
 func (rv *rbdVolume) getImageID() error {
 	if rv.ImageID != "" {
 		return nil
@@ -313,7 +313,7 @@ func rbdStatus(ctx context.Context, pOpts *rbdVolume, cr *util.Credentials) (boo
 
 // addRbdManagerTask adds a ceph manager task to execute command
 // asynchronously. If command is not found returns a bool set to false
-// example arg ["trash", "remove","pool/image"]
+// example arg ["trash", "remove","pool/image"].
 func addRbdManagerTask(ctx context.Context, pOpts *rbdVolume, arg []string) (bool, error) {
 	var output []byte
 	args := []string{"rbd", "task", "add"}
@@ -530,7 +530,7 @@ func (rv *rbdVolume) checkImageChainHasFeature(ctx context.Context, feature uint
 }
 
 // genSnapFromSnapID generates a rbdSnapshot structure from the provided identifier, updating
-// the structure with elements from on-disk snapshot metadata as well
+// the structure with elements from on-disk snapshot metadata as well.
 func genSnapFromSnapID(ctx context.Context, rbdSnap *rbdSnapshot, snapshotID string, cr *util.Credentials) error {
 	var (
 		options map[string]string
@@ -589,7 +589,7 @@ func genSnapFromSnapID(ctx context.Context, rbdSnap *rbdSnapshot, snapshotID str
 }
 
 // genVolFromVolID generates a rbdVolume structure from the provided identifier, updating
-// the structure with elements from on-disk image metadata as well
+// the structure with elements from on-disk image metadata as well.
 func genVolFromVolID(ctx context.Context, volumeID string, cr *util.Credentials, secrets map[string]string) (*rbdVolume, error) {
 	var (
 		options map[string]string
@@ -794,7 +794,7 @@ func genSnapFromOptions(ctx context.Context, rbdVol *rbdVolume, snapOptions map[
 	return rbdSnap
 }
 
-// hasSnapshotFeature checks if Layering is enabled for this image
+// hasSnapshotFeature checks if Layering is enabled for this image.
 func (rv *rbdVolume) hasSnapshotFeature() bool {
 	return (uint64(rv.imageFeatureSet) & librbd.FeatureLayering) == librbd.FeatureLayering
 }
@@ -874,7 +874,7 @@ func (rv *rbdVolume) cloneRbdImageFromSnapshot(ctx context.Context, pSnapOpts *r
 	return nil
 }
 
-// imageInfo strongly typed JSON spec for image info
+// imageInfo strongly typed JSON spec for image info.
 type imageInfo struct {
 	ObjectUUID string     `json:"name"`
 	Size       int64      `json:"size"`
@@ -883,7 +883,7 @@ type imageInfo struct {
 	Parent     parentInfo `json:"parent"`
 }
 
-// parentInfo  spec for parent volume  info
+// parentInfo  spec for parent volume  info.
 type parentInfo struct {
 	Image    string `json:"image"`
 	Pool     string `json:"pool"`
@@ -891,7 +891,7 @@ type parentInfo struct {
 }
 
 // updateVolWithImageInfo updates provided rbdVolume with information from on-disk data
-// regarding the same
+// regarding the same.
 func (rv *rbdVolume) updateVolWithImageInfo(cr *util.Credentials) error {
 	// rbd --format=json info [image-spec | snap-spec]
 	var imgInfo imageInfo
@@ -931,7 +931,7 @@ func (rv *rbdVolume) updateVolWithImageInfo(cr *util.Credentials) error {
 }
 
 // getImageInfo queries rbd about the given image and returns its metadata, and returns
-// ErrImageNotFound if provided image is not found
+// ErrImageNotFound if provided image is not found.
 func (rv *rbdVolume) getImageInfo() error {
 	image, err := rv.open()
 	if err != nil {
@@ -962,7 +962,7 @@ func (rv *rbdVolume) getImageInfo() error {
 /*
 checkSnapExists queries rbd about the snapshots of the given image and returns
 ErrImageNotFound if provided image is not found, and ErrSnapNotFound if
-provided snap is not found in the images snapshot list
+provided snap is not found in the images snapshot list.
 */
 func (rv *rbdVolume) checkSnapExists(rbdSnap *rbdSnapshot) error {
 	image, err := rv.open()
@@ -985,7 +985,7 @@ func (rv *rbdVolume) checkSnapExists(rbdSnap *rbdSnapshot) error {
 	return ErrSnapNotFound{rbdSnap.RbdSnapName, fmt.Errorf("snap %s not found", rbdSnap.String())}
 }
 
-// rbdImageMetadataStash strongly typed JSON spec for stashed RBD image metadata
+// rbdImageMetadataStash strongly typed JSON spec for stashed RBD image metadata.
 type rbdImageMetadataStash struct {
 	Version   int    `json:"Version"`
 	Pool      string `json:"pool"`
@@ -994,16 +994,16 @@ type rbdImageMetadataStash struct {
 	Encrypted bool   `json:"encrypted"`
 }
 
-// file name in which image metadata is stashed
+// file name in which image metadata is stashed.
 const stashFileName = "image-meta.json"
 
-// spec returns the image-spec (pool/image) format of the image
+// spec returns the image-spec (pool/image) format of the image.
 func (ri *rbdImageMetadataStash) String() string {
 	return fmt.Sprintf("%s/%s", ri.Pool, ri.ImageName)
 }
 
 // stashRBDImageMetadata stashes required fields into the stashFileName at the passed in path, in
-// JSON format
+// JSON format.
 func stashRBDImageMetadata(volOptions *rbdVolume, path string) error {
 	var imgMeta = rbdImageMetadataStash{
 		Version:   2, // there are no checks for this at present
@@ -1031,7 +1031,7 @@ func stashRBDImageMetadata(volOptions *rbdVolume, path string) error {
 	return nil
 }
 
-// lookupRBDImageMetadataStash reads and returns stashed image metadata at passed in path
+// lookupRBDImageMetadataStash reads and returns stashed image metadata at passed in path.
 func lookupRBDImageMetadataStash(path string) (rbdImageMetadataStash, error) {
 	var imgMeta rbdImageMetadataStash
 
@@ -1053,7 +1053,7 @@ func lookupRBDImageMetadataStash(path string) (rbdImageMetadataStash, error) {
 	return imgMeta, nil
 }
 
-// cleanupRBDImageMetadataStash cleans up any stashed metadata at passed in path
+// cleanupRBDImageMetadataStash cleans up any stashed metadata at passed in path.
 func cleanupRBDImageMetadataStash(path string) error {
 	fPath := filepath.Join(path, stashFileName)
 	if err := os.Remove(fPath); err != nil {
@@ -1063,7 +1063,7 @@ func cleanupRBDImageMetadataStash(path string) error {
 	return nil
 }
 
-// resizeRBDImage resizes the given volume to new size
+// resizeRBDImage resizes the given volume to new size.
 func resizeRBDImage(rbdVol *rbdVolume, cr *util.Credentials) error {
 	var output []byte
 
@@ -1100,7 +1100,7 @@ func (rv *rbdVolume) SetMetadata(key, value string) error {
 	return image.SetMetadata(key, value)
 }
 
-// checkRbdImageEncrypted verifies if rbd image was encrypted when created
+// checkRbdImageEncrypted verifies if rbd image was encrypted when created.
 func (rv *rbdVolume) checkRbdImageEncrypted(ctx context.Context) (string, error) {
 	value, err := rv.GetMetadata(encryptionMetaKey)
 	if err != nil {
@@ -1122,7 +1122,7 @@ func (rv *rbdVolume) ensureEncryptionMetadataSet(status string) error {
 	return nil
 }
 
-// SnapshotInfo holds snapshots details
+// SnapshotInfo holds snapshots details.
 type snapshotInfo struct {
 	ID        int    `json:"id"`
 	Name      string `json:"name"`

internal/util/cephcmds.go

@@ -27,10 +27,10 @@ import (
 	klog "k8s.io/klog/v2"
 )
 
-// InvalidPoolID used to denote an invalid pool
+// InvalidPoolID used to denote an invalid pool.
 const InvalidPoolID int64 = -1
 
-// ExecCommand executes passed in program with args and returns separate stdout and stderr streams
+// ExecCommand executes passed in program with args and returns separate stdout and stderr streams.
 func ExecCommand(program string, args ...string) (stdout, stderr []byte, err error) {
 	var (
 		cmd           = exec.Command(program, args...) // nolint: gosec, #nosec
@@ -51,7 +51,7 @@ func ExecCommand(program string, args ...string) (stdout, stderr []byte, err err
 }
 
 // GetPoolID fetches the ID of the pool that matches the passed in poolName
-// parameter
+// parameter.
 func GetPoolID(monitors string, cr *Credentials, poolName string) (int64, error) {
 	conn, err := connPool.Get(monitors, cr.ID, cr.KeyFile)
 	if err != nil {
@@ -70,7 +70,7 @@ func GetPoolID(monitors string, cr *Credentials, poolName string) (int64, error)
 }
 
 // GetPoolName fetches the pool whose pool ID is equal to the requested poolID
-// parameter
+// parameter.
 func GetPoolName(monitors string, cr *Credentials, poolID int64) (string, error) {
 	conn, err := connPool.Get(monitors, cr.ID, cr.KeyFile)
 	if err != nil {
@@ -87,7 +87,7 @@ func GetPoolName(monitors string, cr *Credentials, poolID int64) (string, error)
 
 // GetPoolIDs searches a list of pools in a cluster and returns the IDs of the pools that matches
 // the passed in pools
-// TODO this should take in a list and return a map[string(poolname)]int64(poolID)
+// TODO this should take in a list and return a map[string(poolname)]int64(poolID).
 func GetPoolIDs(ctx context.Context, monitors, journalPool, imagePool string, cr *Credentials) (int64, int64, error) {
 	journalPoolID, err := GetPoolID(monitors, cr, journalPool)
 	if err != nil {
@@ -106,7 +106,7 @@ func GetPoolIDs(ctx context.Context, monitors, journalPool, imagePool string, cr
 }
 
 // CreateObject creates the object name passed in and returns ErrObjectExists if the provided object
-// is already present in rados
+// is already present in rados.
 func CreateObject(ctx context.Context, monitors string, cr *Credentials, poolName, namespace, objectName string) error {
 	conn := ClusterConnection{}
 	err := conn.Connect(monitors, cr)
@@ -141,7 +141,7 @@ func CreateObject(ctx context.Context, monitors string, cr *Credentials, poolNam
 }
 
 // RemoveObject removes the entire omap name passed in and returns ErrObjectNotFound is provided omap
-// is not found in rados
+// is not found in rados.
 func RemoveObject(ctx context.Context, monitors string, cr *Credentials, poolName, namespace, oMapName string) error {
 	conn := ClusterConnection{}
 	err := conn.Connect(monitors, cr)

internal/util/cephconf.go

@@ -43,7 +43,7 @@ func createCephConfigRoot() error {
 }
 
 // WriteCephConfig writes out a basic ceph.conf file, making it easy to use
-// ceph related CLIs
+// ceph related CLIs.
 func WriteCephConfig() error {
 	if err := createCephConfigRoot(); err != nil {
 		return err
@@ -64,7 +64,7 @@ if any ceph commands fails it will log below error message
 /etc/ceph/ceph.client.admin.keyring,/etc/ceph/ceph.keyring,/etc/ceph/keyring,
 /etc/ceph/keyring.bin,: (2) No such file or directory
 */
-// createKeyRingFile creates the keyring files to fix above error message logging
+// createKeyRingFile creates the keyring files to fix above error message logging.
 func createKeyRingFile() error {
 	_, err := os.Create(keyRing)
 	return err

internal/util/conn_pool.go

@@ -106,7 +106,7 @@ func (cp *ConnPool) generateUniqueKey(monitors, user, keyfile string) (string, e
 
 // getExisting returns the existing rados.Conn associated with the unique key.
 //
-// Requires: locked cp.lock because of ce.get()
+// Requires: locked cp.lock because of ce.get().
 func (cp *ConnPool) getConn(unique string) *rados.Conn {
 	ce, exists := cp.conns[unique]
 	if exists {

internal/util/conn_pool_test.go

@@ -33,7 +33,7 @@ const (
 // fakeGet is used as a replacement for ConnPool.Get and does not need a
 // working Ceph cluster to connect to.
 //
-// This is mostly a copy of ConnPool.Get()
+// This is mostly a copy of ConnPool.Get().
 func (cp *ConnPool) fakeGet(monitors, user, keyfile string) (*rados.Conn, string, error) {
 	unique, err := cp.generateUniqueKey(monitors, user, keyfile)
 	if err != nil {

internal/util/crypto.go

@@ -51,7 +51,7 @@ const (
 )
 
 // EncryptionKMS provides external Key Management System for encryption
-// passphrases storage
+// passphrases storage.
 type EncryptionKMS interface {
 	GetPassphrase(key string) (string, error)
 	SavePassphrase(key, value string) error
@@ -59,12 +59,12 @@ type EncryptionKMS interface {
 	GetID() string
 }
 
-// MissingPassphrase is an error instructing to generate new passphrase
+// MissingPassphrase is an error instructing to generate new passphrase.
 type MissingPassphrase struct {
 	error
 }
 
-// SecretsKMS is default KMS implementation that means no KMS is in use
+// SecretsKMS is default KMS implementation that means no KMS is in use.
 type SecretsKMS struct {
 	passphrase string
 }
@@ -77,28 +77,28 @@ func initSecretsKMS(secrets map[string]string) (EncryptionKMS, error) {
 	return SecretsKMS{passphrase: passphraseValue}, nil
 }
 
-// GetPassphrase returns passphrase from Kubernetes secrets
+// GetPassphrase returns passphrase from Kubernetes secrets.
 func (kms SecretsKMS) GetPassphrase(key string) (string, error) {
 	return kms.passphrase, nil
 }
 
-// SavePassphrase is not implemented
+// SavePassphrase is not implemented.
 func (kms SecretsKMS) SavePassphrase(key, value string) error {
 	return fmt.Errorf("save new passphrase is not implemented for Kubernetes secrets")
 }
 
 // DeletePassphrase is doing nothing as no new passphrases are saved with
-// SecretsKMS
+// SecretsKMS.
 func (kms SecretsKMS) DeletePassphrase(key string) error {
 	return nil
 }
 
-// GetID is returning ID representing default KMS `default`
+// GetID is returning ID representing default KMS `default`.
 func (kms SecretsKMS) GetID() string {
 	return defaultKMSType
 }
 
-// GetKMS returns an instance of Key Management System
+// GetKMS returns an instance of Key Management System.
 func GetKMS(kmsID string, secrets map[string]string) (EncryptionKMS, error) {
 	if kmsID == "" || kmsID == defaultKMSType {
 		return initSecretsKMS(secrets)
@@ -141,7 +141,7 @@ func GetKMS(kmsID string, secrets map[string]string) (EncryptionKMS, error) {
 	return nil, fmt.Errorf("unknown encryption KMS type %s", kmsType)
 }
 
-// GetCryptoPassphrase Retrieves passphrase to encrypt volume
+// GetCryptoPassphrase Retrieves passphrase to encrypt volume.
 func GetCryptoPassphrase(ctx context.Context, volumeID string, kms EncryptionKMS) (string, error) {
 	passphrase, err := kms.GetPassphrase(volumeID)
 	if err == nil {
@@ -164,7 +164,7 @@ func GetCryptoPassphrase(ctx context.Context, volumeID string, kms EncryptionKMS
 	return "", err
 }
 
-// generateNewEncryptionPassphrase generates a random passphrase for encryption
+// generateNewEncryptionPassphrase generates a random passphrase for encryption.
 func generateNewEncryptionPassphrase() (string, error) {
 	bytesPassphrase := make([]byte, encryptionPassphraseSize)
 	_, err := rand.Read(bytesPassphrase)
@@ -174,14 +174,14 @@ func generateNewEncryptionPassphrase() (string, error) {
 	return base64.URLEncoding.EncodeToString(bytesPassphrase), nil
 }
 
-// VolumeMapper returns file name and it's path to where encrypted device should be open
+// VolumeMapper returns file name and it's path to where encrypted device should be open.
 func VolumeMapper(volumeID string) (mapperFile, mapperFilePath string) {
 	mapperFile = mapperFilePrefix + volumeID
 	mapperFilePath = path.Join(mapperFilePathPrefix, mapperFile)
 	return mapperFile, mapperFilePath
 }
 
-// EncryptVolume encrypts provided device with LUKS
+// EncryptVolume encrypts provided device with LUKS.
 func EncryptVolume(ctx context.Context, devicePath, passphrase string) error {
 	DebugLog(ctx, "Encrypting device %s with LUKS", devicePath)
 	if _, _, err := LuksFormat(devicePath, passphrase); err != nil {
@@ -190,21 +190,21 @@ func EncryptVolume(ctx context.Context, devicePath, passphrase string) error {
 	return nil
 }
 
-// OpenEncryptedVolume opens volume so that it can be used by the client
+// OpenEncryptedVolume opens volume so that it can be used by the client.
 func OpenEncryptedVolume(ctx context.Context, devicePath, mapperFile, passphrase string) error {
 	DebugLog(ctx, "Opening device %s with LUKS on %s", devicePath, mapperFile)
 	_, _, err := LuksOpen(devicePath, mapperFile, passphrase)
 	return err
 }
 
-// CloseEncryptedVolume closes encrypted volume so it can be detached
+// CloseEncryptedVolume closes encrypted volume so it can be detached.
 func CloseEncryptedVolume(ctx context.Context, mapperFile string) error {
 	DebugLog(ctx, "Closing LUKS device %s", mapperFile)
 	_, _, err := LuksClose(mapperFile)
 	return err
 }
 
-// IsDeviceOpen determines if encrypted device is already open
+// IsDeviceOpen determines if encrypted device is already open.
 func IsDeviceOpen(ctx context.Context, device string) (bool, error) {
 	_, mappedFile, err := DeviceEncryptionStatus(ctx, device)
 	return (mappedFile != ""), err

internal/util/cryptsetup.go

@@ -23,22 +23,22 @@ import (
 	"strings"
 )
 
-// LuksFormat sets up volume as an encrypted LUKS partition
+// LuksFormat sets up volume as an encrypted LUKS partition.
 func LuksFormat(devicePath, passphrase string) (stdout, stderr []byte, err error) {
 	return execCryptsetupCommand(&passphrase, "-q", "luksFormat", "--hash", "sha256", devicePath, "-d", "/dev/stdin")
 }
 
-// LuksOpen opens LUKS encrypted partition and sets up a mapping
+// LuksOpen opens LUKS encrypted partition and sets up a mapping.
 func LuksOpen(devicePath, mapperFile, passphrase string) (stdout, stderr []byte, err error) {
 	return execCryptsetupCommand(&passphrase, "luksOpen", devicePath, mapperFile, "-d", "/dev/stdin")
 }
 
-// LuksClose removes existing mapping
+// LuksClose removes existing mapping.
 func LuksClose(mapperFile string) (stdout, stderr []byte, err error) {
 	return execCryptsetupCommand(nil, "luksClose", mapperFile)
 }
 
-// LuksStatus returns encryption status of a provided device
+// LuksStatus returns encryption status of a provided device.
 func LuksStatus(mapperFile string) (stdout, stderr []byte, err error) {
 	return execCryptsetupCommand(nil, "status", mapperFile)
 }

internal/util/csiconfig.go

@@ -29,7 +29,7 @@ const (
 	defaultCsiSubvolumeGroup = "csi"
 )
 
-// ClusterInfo strongly typed JSON spec for the below JSON structure
+// ClusterInfo strongly typed JSON spec for the below JSON structure.
 type ClusterInfo struct {
 	// ClusterID is used for unique identification
 	ClusterID string `json:"clusterID"`
@@ -57,7 +57,7 @@ type ClusterInfo struct {
 //         }
 // 	},
 // 	...
-// ]
+// ].
 func readClusterInfo(pathToConfig, clusterID string) (*ClusterInfo, error) {
 	var config []ClusterInfo
 
@@ -83,7 +83,7 @@ func readClusterInfo(pathToConfig, clusterID string) (*ClusterInfo, error) {
 	return nil, fmt.Errorf("missing configuration for cluster ID (%s)", clusterID)
 }
 
-// Mons returns a comma separated MON list from the csi config for the given clusterID
+// Mons returns a comma separated MON list from the csi config for the given clusterID.
 func Mons(pathToConfig, clusterID string) (string, error) {
 	cluster, err := readClusterInfo(pathToConfig, clusterID)
 	if err != nil {
@@ -96,7 +96,7 @@ func Mons(pathToConfig, clusterID string) (string, error) {
 	return strings.Join(cluster.Monitors, ","), nil
 }
 
-// CephFSSubvolumeGroup returns the subvolumeGroup for CephFS volumes. If not set, it returns the default value "csi"
+// CephFSSubvolumeGroup returns the subvolumeGroup for CephFS volumes. If not set, it returns the default value "csi".
 func CephFSSubvolumeGroup(pathToConfig, clusterID string) (string, error) {
 	cluster, err := readClusterInfo(pathToConfig, clusterID)
 	if err != nil {

internal/util/errors.go

@@ -16,7 +16,7 @@ limitations under the License.
 
 package util
 
-// ErrKeyNotFound is returned when requested key in omap is not found
+// ErrKeyNotFound is returned when requested key in omap is not found.
 type ErrKeyNotFound struct {
 	keyName string
 	err     error
@@ -37,7 +37,7 @@ func (e ErrKeyNotFound) Unwrap() error {
 	return e.err
 }
 
-// ErrObjectExists is returned when named omap is already present in rados
+// ErrObjectExists is returned when named omap is already present in rados.
 type ErrObjectExists struct {
 	objectName string
 	err        error
@@ -53,7 +53,7 @@ func (e ErrObjectExists) Unwrap() error {
 	return e.err
 }
 
-// ErrObjectNotFound is returned when named omap is not found in rados
+// ErrObjectNotFound is returned when named omap is not found in rados.
 type ErrObjectNotFound struct {
 	oMapName string
 	err      error
@@ -70,7 +70,7 @@ func (e ErrObjectNotFound) Unwrap() error {
 }
 
 // ErrSnapNameConflict is generated when a requested CSI snap name already exists on RBD but with
-// different properties, and hence is in conflict with the passed in CSI volume name
+// different properties, and hence is in conflict with the passed in CSI volume name.
 type ErrSnapNameConflict struct {
 	requestName string
 	err         error
@@ -91,7 +91,7 @@ func NewErrSnapNameConflict(name string, err error) ErrSnapNameConflict {
 	return ErrSnapNameConflict{name, err}
 }
 
-// ErrPoolNotFound is returned when pool is not found
+// ErrPoolNotFound is returned when pool is not found.
 type ErrPoolNotFound struct {
 	Pool string
 	Err  error

internal/util/httpserver.go

@@ -10,13 +10,13 @@ import (
 	klog "k8s.io/klog/v2"
 )
 
-// ValidateURL validates the url
+// ValidateURL validates the url.
 func ValidateURL(c *Config) error {
 	_, err := url.Parse(c.MetricsPath)
 	return err
 }
 
-// StartMetricsServer starts http server
+// StartMetricsServer starts http server.
 func StartMetricsServer(c *Config) {
 	addr := net.JoinHostPort(c.MetricsIP, strconv.Itoa(c.MetricsPort))
 	http.Handle(c.MetricsPath, promhttp.Handler())

internal/util/idlocker_test.go

@@ -20,7 +20,7 @@ import (
 	"testing"
 )
 
-// very basic tests for the moment
+// very basic tests for the moment.
 func TestIDLocker(t *testing.T) {
 	fakeID := "fake-id"
 	locks := NewVolumeLocks()

internal/util/k8s.go

@@ -25,7 +25,7 @@ import (
 	klog "k8s.io/klog/v2"
 )
 
-// NewK8sClient create kubernetes client
+// NewK8sClient create kubernetes client.
 func NewK8sClient() *k8s.Clientset {
 	var cfg *rest.Config
 	var err error

internal/util/log.go

@@ -20,13 +20,13 @@ import (
 
 type contextKey string
 
-// CtxKey for context based logging
+// CtxKey for context based logging.
 var CtxKey = contextKey("ID")
 
-// ReqID for logging request ID
+// ReqID for logging request ID.
 var ReqID = contextKey("Req-ID")
 
-// Log helps in context based logging
+// Log helps in context based logging.
 func Log(ctx context.Context, format string) string {
 	id := ctx.Value(CtxKey)
 	if id == nil {

internal/util/pidlimit.go

@@ -35,7 +35,7 @@ const (
 // find the line containing the pids group from the /proc/self/cgroup file
 // $ grep 'pids' /proc/self/cgroup
 // 7:pids:/kubepods.slice/kubepods-besteffort.slice/....scope
-// $ cat /sys/fs/cgroup/pids + *.scope + /pids.max
+// $ cat /sys/fs/cgroup/pids + *.scope + /pids.max.
 func getCgroupPidsFile() (string, error) {
 	cgroup, err := os.Open(procCgroup)
 	if err != nil {

internal/util/pidlimit_test.go

@@ -22,7 +22,7 @@ import (
 )
 
 // minimal test to check if GetPIDLimit() returns an int
-// changing the limit require root permissions, not tested
+// changing the limit require root permissions, not tested.
 func TestGetPIDLimit(t *testing.T) {
 	runTest := os.Getenv("CEPH_CSI_RUN_ALL_TESTS")
 	if runTest == "" {

internal/util/topology.go

@@ -44,7 +44,7 @@ func k8sGetNodeLabels(nodeName string) (map[string]string, error) {
 
 // GetTopologyFromDomainLabels returns the CSI topology map, determined from
 // the domain labels and their values from the CO system
-// Expects domainLabels in arg to be in the format "[prefix/]<name>,[prefix/]<name>,...",
+// Expects domainLabels in arg to be in the format "[prefix/]<name>,[prefix/]<name>,...",.
 func GetTopologyFromDomainLabels(domainLabels, nodeName, driverName string) (map[string]string, error) {
 	if domainLabels == "" {
 		return nil, nil
@@ -122,7 +122,7 @@ type topologySegment struct {
 	DomainValue string `json:"value"`
 }
 
-// TopologyConstrainedPool stores the pool name and a list of its associated topology domain values
+// TopologyConstrainedPool stores the pool name and a list of its associated topology domain values.
 type TopologyConstrainedPool struct {
 	PoolName       string            `json:"poolName"`
 	DataPoolName   string            `json:"dataPool"`
@@ -130,7 +130,7 @@ type TopologyConstrainedPool struct {
 }
 
 // GetTopologyFromRequest extracts TopologyConstrainedPools and passed in accessibility constraints
-// from a CSI CreateVolume request
+// from a CSI CreateVolume request.
 func GetTopologyFromRequest(req *csi.CreateVolumeRequest) (*[]TopologyConstrainedPool, *csi.TopologyRequirement, error) {
 	var (
 		topologyPools []TopologyConstrainedPool
@@ -158,7 +158,7 @@ func GetTopologyFromRequest(req *csi.CreateVolumeRequest) (*[]TopologyConstraine
 }
 
 // MatchTopologyForPool returns the topology map, if the passed in pool matches any
-// passed in accessibility constraints
+// passed in accessibility constraints.
 func MatchTopologyForPool(topologyPools *[]TopologyConstrainedPool,
 	accessibilityRequirements *csi.TopologyRequirement, poolName string) (map[string]string, error) {
 	var topologyPool []TopologyConstrainedPool
@@ -187,7 +187,7 @@ func MatchTopologyForPool(topologyPools *[]TopologyConstrainedPool,
 // FindPoolAndTopology loops through passed in "topologyPools" and also related
 // accessibility requirements, to determine which pool matches the requirement.
 // The return variables are, image poolname, data poolname, and topology map of
-// matched requirement
+// matched requirement.
 func FindPoolAndTopology(topologyPools *[]TopologyConstrainedPool,
 	accessibilityRequirements *csi.TopologyRequirement) (string, string, map[string]string, error) {
 	if topologyPools == nil || accessibilityRequirements == nil {
@@ -217,7 +217,7 @@ func FindPoolAndTopology(topologyPools *[]TopologyConstrainedPool,
 
 // matchPoolToTopology loops through passed in pools, and for each pool checks if all
 // requested topology segments are present and match the request, returning the first pool
-// that hence matches (or an empty string if none match)
+// that hence matches (or an empty string if none match).
 func matchPoolToTopology(topologyPools *[]TopologyConstrainedPool, topology *csi.Topology) TopologyConstrainedPool {
 	domainMap := extractDomainsFromlabels(topology)
 
@@ -243,7 +243,7 @@ func matchPoolToTopology(topologyPools *[]TopologyConstrainedPool, topology *csi
 }
 
 // extractDomainsFromlabels returns the domain name map, from passed in domain segments,
-// which is of the form [prefix/]<name>
+// which is of the form [prefix/]<name>.
 func extractDomainsFromlabels(topology *csi.Topology) map[string]string {
 	domainMap := make(map[string]string)
 	for domainKey, value := range topology.GetSegments() {

internal/util/topology_test.go

@@ -35,7 +35,7 @@ func checkAndReportError(t *testing.T, msg string, err error) {
 	}
 }
 
-// TestFindPoolAndTopology also tests MatchTopologyForPool
+// TestFindPoolAndTopology also tests MatchTopologyForPool.
 func TestFindPoolAndTopology(t *testing.T) {
 	var err error
 	var label1 = "region"

internal/util/util.go

@@ -44,7 +44,7 @@ const (
 	Trace
 )
 
-// RoundOffVolSize rounds up given quantity upto chunks of MiB/GiB
+// RoundOffVolSize rounds up given quantity upto chunks of MiB/GiB.
 func RoundOffVolSize(size int64) int64 {
 	size = RoundOffBytes(size)
 	// convert size back to MiB for rbd CLI
@@ -53,7 +53,7 @@ func RoundOffVolSize(size int64) int64 {
 
 // RoundOffBytes converts roundoff the size
 // 1.1Mib will be round off to 2Mib same for GiB
-// size less than 1MiB will be round off to 1MiB
+// size less than 1MiB will be round off to 1MiB.
 func RoundOffBytes(bytes int64) int64 {
 	var num int64
 	floatBytes := float64(bytes)
@@ -68,7 +68,7 @@ func RoundOffBytes(bytes int64) int64 {
 	return num
 }
 
-// variables which will be set during the build time
+// variables which will be set during the build time.
 var (
 	// GitCommit tell the latest git commit image is built from
 	GitCommit string
@@ -76,7 +76,7 @@ var (
 	DriverVersion string
 )
 
-// Config holds the parameters list which can be configured
+// Config holds the parameters list which can be configured.
 type Config struct {
 	Vtype        string // driver type [rbd|cephfs|liveness]
 	Endpoint     string // CSI endpoint
@@ -119,7 +119,7 @@ type Config struct {
 	MaxSnapshotsOnImage uint
 }
 
-// ValidateDriverName validates the driver name
+// ValidateDriverName validates the driver name.
 func ValidateDriverName(driverName string) error {
 	if driverName == "" {
 		return errors.New("driver name is empty")
@@ -150,7 +150,7 @@ func GetKernelVersion() (string, error) {
 	return strings.TrimRight(string(utsname.Release[:]), "\x00"), nil
 }
 
-// KernelVersion holds kernel related informations
+// KernelVersion holds kernel related informations.
 type KernelVersion struct {
 	Version      int
 	PatchLevel   int
@@ -235,7 +235,7 @@ func CheckKernelSupport(release string, supportedVersions []KernelVersion) bool
 }
 
 // GenerateVolID generates a volume ID based on passed in parameters and version, to be returned
-// to the CO system
+// to the CO system.
 func GenerateVolID(ctx context.Context, monitors string, cr *Credentials, locationID int64, pool, clusterID, objUUID string, volIDVersion uint16) (string, error) {
 	var err error
 
@@ -259,12 +259,12 @@ func GenerateVolID(ctx context.Context, monitors string, cr *Credentials, locati
 	return volID, err
 }
 
-// CreateMountPoint creates the directory with given path
+// CreateMountPoint creates the directory with given path.
 func CreateMountPoint(mountPath string) error {
 	return os.MkdirAll(mountPath, 0750)
 }
 
-// checkDirExists checks directory  exists or not
+// checkDirExists checks directory  exists or not.
 func checkDirExists(p string) bool {
 	if _, err := os.Stat(p); os.IsNotExist(err) {
 		return false
@@ -272,7 +272,7 @@ func checkDirExists(p string) bool {
 	return true
 }
 
-// IsMountPoint checks if the given path is mountpoint or not
+// IsMountPoint checks if the given path is mountpoint or not.
 func IsMountPoint(p string) (bool, error) {
 	dummyMount := mount.New("")
 	notMnt, err := dummyMount.IsLikelyNotMountPoint(p)
@@ -283,7 +283,7 @@ func IsMountPoint(p string) (bool, error) {
 	return !notMnt, nil
 }
 
-// Mount mounts the source to target path
+// Mount mounts the source to target path.
 func Mount(source, target, fstype string, options []string) error {
 	dummyMount := mount.New("")
 	return dummyMount.Mount(source, target, fstype, options)
@@ -321,7 +321,7 @@ func contains(s []string, key string) bool {
 	return false
 }
 
-// DefaultLog helps in logging with klog.level 1
+// DefaultLog helps in logging with klog.level 1.
 func DefaultLog(message string, args ...interface{}) {
 	logMessage := fmt.Sprintf(message, args...)
 	// If logging is disabled, don't evaluate the arguments
@@ -330,7 +330,7 @@ func DefaultLog(message string, args ...interface{}) {
 	}
 }
 
-// UsefulLog helps in logging with klog.level 2
+// UsefulLog helps in logging with klog.level 2.
 func UsefulLog(ctx context.Context, message string, args ...interface{}) {
 	logMessage := fmt.Sprintf(Log(ctx, message), args...)
 	// If logging is disabled, don't evaluate the arguments
@@ -339,7 +339,7 @@ func UsefulLog(ctx context.Context, message string, args ...interface{}) {
 	}
 }
 
-// ExtendedLogMsg helps in logging a message with klog.level 3
+// ExtendedLogMsg helps in logging a message with klog.level 3.
 func ExtendedLogMsg(message string, args ...interface{}) {
 	logMessage := fmt.Sprintf(message, args...)
 	// If logging is disabled, don't evaluate the arguments
@@ -348,7 +348,7 @@ func ExtendedLogMsg(message string, args ...interface{}) {
 	}
 }
 
-// ExtendedLog helps in logging with klog.level 3
+// ExtendedLog helps in logging with klog.level 3.
 func ExtendedLog(ctx context.Context, message string, args ...interface{}) {
 	logMessage := fmt.Sprintf(Log(ctx, message), args...)
 	// If logging is disabled, don't evaluate the arguments
@@ -357,7 +357,7 @@ func ExtendedLog(ctx context.Context, message string, args ...interface{}) {
 	}
 }
 
-// DebugLogMsg helps in logging a message with klog.level 4
+// DebugLogMsg helps in logging a message with klog.level 4.
 func DebugLogMsg(message string, args ...interface{}) {
 	logMessage := fmt.Sprintf(message, args...)
 	// If logging is disabled, don't evaluate the arguments
@@ -366,7 +366,7 @@ func DebugLogMsg(message string, args ...interface{}) {
 	}
 }
 
-// DebugLog helps in logging with klog.level 4
+// DebugLog helps in logging with klog.level 4.
 func DebugLog(ctx context.Context, message string, args ...interface{}) {
 	logMessage := fmt.Sprintf(Log(ctx, message), args...)
 	// If logging is disabled, don't evaluate the arguments
@@ -375,7 +375,7 @@ func DebugLog(ctx context.Context, message string, args ...interface{}) {
 	}
 }
 
-// TraceLogMsg helps in logging a message with klog.level 5
+// TraceLogMsg helps in logging a message with klog.level 5.
 func TraceLogMsg(message string, args ...interface{}) {
 	logMessage := fmt.Sprintf(message, args...)
 	// If logging is disabled, don't evaluate the arguments
@@ -384,7 +384,7 @@ func TraceLogMsg(message string, args ...interface{}) {
 	}
 }
 
-// TraceLog helps in logging with klog.level 5
+// TraceLog helps in logging with klog.level 5.
 func TraceLog(ctx context.Context, message string, args ...interface{}) {
 	logMessage := fmt.Sprintf(Log(ctx, message), args...)
 	// If logging is disabled, don't evaluate the arguments

internal/util/validate.go

@@ -6,7 +6,7 @@ import (
 	"google.golang.org/grpc/status"
 )
 
-// ValidateNodeStageVolumeRequest validates the node stage request
+// ValidateNodeStageVolumeRequest validates the node stage request.
 func ValidateNodeStageVolumeRequest(req *csi.NodeStageVolumeRequest) error {
 	if req.GetVolumeCapability() == nil {
 		return status.Error(codes.InvalidArgument, "volume capability missing in request")
@@ -32,7 +32,7 @@ func ValidateNodeStageVolumeRequest(req *csi.NodeStageVolumeRequest) error {
 	return nil
 }
 
-// ValidateNodeUnstageVolumeRequest validates the node unstage request
+// ValidateNodeUnstageVolumeRequest validates the node unstage request.
 func ValidateNodeUnstageVolumeRequest(req *csi.NodeUnstageVolumeRequest) error {
 	if req.GetVolumeId() == "" {
 		return status.Error(codes.InvalidArgument, "volume ID missing in request")
@@ -45,7 +45,7 @@ func ValidateNodeUnstageVolumeRequest(req *csi.NodeUnstageVolumeRequest) error {
 	return nil
 }
 
-// ValidateNodePublishVolumeRequest validates the node publish request
+// ValidateNodePublishVolumeRequest validates the node publish request.
 func ValidateNodePublishVolumeRequest(req *csi.NodePublishVolumeRequest) error {
 	if req.GetVolumeCapability() == nil {
 		return status.Error(codes.InvalidArgument, "volume capability missing in request")
@@ -66,7 +66,7 @@ func ValidateNodePublishVolumeRequest(req *csi.NodePublishVolumeRequest) error {
 	return nil
 }
 
-// ValidateNodeUnpublishVolumeRequest validates the node unpublish request
+// ValidateNodeUnpublishVolumeRequest validates the node unpublish request.
 func ValidateNodeUnpublishVolumeRequest(req *csi.NodeUnpublishVolumeRequest) error {
 	if req.GetVolumeId() == "" {
 		return status.Error(codes.InvalidArgument, "volume ID missing in request")

internal/util/vault.go

@@ -62,7 +62,7 @@ Example JSON structure in the KMS config is,
 		"vaultCAFromSecret": "vault-ca"
 	},
 	...
-}
+}.
 */
 type VaultKMS struct {
 	EncryptionKMSID     string
@@ -76,7 +76,7 @@ type VaultKMS struct {
 	vaultCA             *x509.CertPool
 }
 
-// InitVaultKMS returns an interface to HashiCorp Vault KMS
+// InitVaultKMS returns an interface to HashiCorp Vault KMS.
 func InitVaultKMS(kmsID string, config, secrets map[string]string) (EncryptionKMS, error) {
 	var (
 		ok  bool
@@ -136,12 +136,12 @@ func InitVaultKMS(kmsID string, config, secrets map[string]string) (EncryptionKM
 	return kms, nil
 }
 
-// GetID is returning correlation ID to KMS configuration
+// GetID is returning correlation ID to KMS configuration.
 func (kms *VaultKMS) GetID() string {
 	return kms.EncryptionKMSID
 }
 
-// GetPassphrase returns passphrase from Vault
+// GetPassphrase returns passphrase from Vault.
 func (kms *VaultKMS) GetPassphrase(key string) (string, error) {
 	var passphrase string
 	resp, err := kms.request("GET", kms.getKeyDataURI(key), nil)
@@ -182,7 +182,7 @@ func (kms *VaultKMS) GetPassphrase(key string) (string, error) {
 	return passphrase, nil
 }
 
-// SavePassphrase saves new passphrase in Vault
+// SavePassphrase saves new passphrase in Vault.
 func (kms *VaultKMS) SavePassphrase(key, value string) error {
 	data, err := json.Marshal(map[string]map[string]string{
 		"data": {
@@ -206,7 +206,7 @@ func (kms *VaultKMS) SavePassphrase(key, value string) error {
 	return nil
 }
 
-// DeletePassphrase deletes passphrase from Vault
+// DeletePassphrase deletes passphrase from Vault.
 func (kms *VaultKMS) DeletePassphrase(key string) error {
 	vaultToken, err := kms.getAccessToken()
 	if err != nil {
@@ -241,7 +241,7 @@ getVaultAccessToken retrieves vault token using kubernetes authentication:
  1. read jwt service account token from well known location
  2. request token from vault using service account jwt token
 Vault will verify service account jwt token with Kubernetes and return token
-if the requester is allowed
+if the requester is allowed.
 */
 func (kms *VaultKMS) getAccessToken() (string, error) {
 	saToken, err := ioutil.ReadFile(serviceAccountTokenPath)

internal/util/volid.go

@@ -49,7 +49,7 @@ type CSIIdentifier struct {
 	ObjectUUID      string
 }
 
-// This maximum comes from the CSI spec on max bytes allowed in the various CSI ID fields
+// This maximum comes from the CSI spec on max bytes allowed in the various CSI ID fields.
 const maxVolIDLen = 128
 
 const (
@@ -95,7 +95,7 @@ func (ci CSIIdentifier) ComposeCSIID() (string, error) {
 }
 
 /*
-DecomposeCSIID composes a CSIIdentifier from passed in string
+DecomposeCSIID composes a CSIIdentifier from passed in string.
 */
 func (ci *CSIIdentifier) DecomposeCSIID(composedCSIID string) (err error) {
 	bytesToProcess := uint16(len(composedCSIID))

internal/util/volid_test.go

@@ -29,7 +29,7 @@ type testTuple struct {
 	wantDecError  bool
 }
 
-// TODO: Add more test tuples to test out other edge conditions
+// TODO: Add more test tuples to test out other edge conditions.
 var testData = []testTuple{
 	{
 		vID: CSIIdentifier{