Merge pull request #14 from IBM/anncy

Code improvements for internal pipeline

Author: ambiknai

.gosec.json

@@ -0,0 +1 @@
+{}

.nancy-ignore

@@ -1,10 +1,16 @@
+// Package v1alpha1 ...
 package v1alpha1
 
+// DriverPhase ...
 type DriverPhase string
 
 const (
-	DriverPhaseNone     DriverPhase = ""
+	// DriverPhaseNone ...
+	DriverPhaseNone DriverPhase = ""
+	// DriverPhaseCreating ...
 	DriverPhaseCreating DriverPhase = "Creating"
-	DriverPhaseRunning  DriverPhase = "Running"
-	DriverPhaseFailed   DriverPhase = "Failed"
+	// DriverPhaseRunning ...
+	DriverPhaseRunning DriverPhase = "Running"
+	// DriverPhaseFailed ...
+	DriverPhaseFailed DriverPhase = "Failed"
 )

api/v1alpha1/common.go

@@ -14,6 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
+// Package v1alpha1 ...
 package v1alpha1
 
 import (
@@ -118,6 +119,7 @@ type IBMObjectCSIList struct {
 	Items           []IBMObjectCSI `json:"items"`
 }
 
+// CSISidecar ...
 type CSISidecar struct {
 	// The name of the csi sidecar image
 	Name string `json:"name"`

api/v1alpha1/ibmobjectcsi_types.go

@@ -14,6 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
+// Package v1alpha1 ...
 package v1alpha1
 
 import (
@@ -24,12 +25,12 @@ import (
 // NOTE: json tags are required.  Any new fields you add must have json tags for the fields to be serialized.
 
 // RecoverStaleVolumeSpec defines the desired state of RecoverStaleVolume
-
 type RecoverStaleVolumeSpec struct {
 	NoOfLogLines int64            `json:"noOfLogLines,omitempty"`
 	Deployment   []DeploymentData `json:"deploymentData,omitempty"`
 }
 
+// DeploymentData ...
 type DeploymentData struct {
 	DeploymentName      string `json:"deploymentName,omitempty"`
 	DeploymentNamespace string `json:"deploymentNamespace,omitempty"`

api/v1alpha1/recoverstalevolume_types.go

@@ -14,6 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
+// Package fake ...
 package fake
 
 import (

controllers/fake/client_create/client.go

@@ -14,6 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
+// Package fake ...
 package fake
 
 import (

controllers/fake/client_delete/client.go

@@ -14,6 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
+// Package fake ...
 package fake
 
 import (

controllers/fake/client_get/client.go

@@ -14,6 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
+// Package fake ...
 package fake
 
 import (

controllers/fake/client_get/csidriver/client.go

@@ -14,6 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
+// Package fake ...
 package fake
 
 import (