Update integration tests to account for nonpointerstructs

Author: JoelSpeed

tests/integration/testdata/default_configurations/affinity.go

@@ -93,7 +93,7 @@ type WeightedPodAffinityTerm struct {
 	// in the range 1-100.
 	Weight int32 `json:"weight" protobuf:"varint,1,opt,name=weight"` // want "optionalorrequired: field WeightedPodAffinityTerm.Weight must be marked as optional or required"
 	// Required. A pod affinity term, associated with the corresponding weight. // want "commentstart: godoc for field WeightedPodAffinityTerm.PodAffinityTerm should start with 'podAffinityTerm ...'"
-	PodAffinityTerm PodAffinityTerm `json:"podAffinityTerm" protobuf:"bytes,2,opt,name=podAffinityTerm"` // want "optionalorrequired: field WeightedPodAffinityTerm.PodAffinityTerm must be marked as optional or required"
+	PodAffinityTerm PodAffinityTerm `json:"podAffinityTerm" protobuf:"bytes,2,opt,name=podAffinityTerm"` // want "optionalorrequired: field WeightedPodAffinityTerm.PodAffinityTerm must be marked as optional or required" "nonpointerstructs: field WeightedPodAffinityTerm.PodAffinityTerm is a non-pointer struct with no required fields. It must be marked as optional."
 }
 
 // Defines a set of pods (namely those matching the labelSelector
@@ -193,7 +193,7 @@ type PreferredSchedulingTerm struct {
 	// Weight associated with matching the corresponding nodeSelectorTerm, in the range 1-100. // want "commentstart: godoc for field PreferredSchedulingTerm.Weight should start with 'weight ...'"
 	Weight int32 `json:"weight" protobuf:"varint,1,opt,name=weight"` // want "optionalorrequired: field PreferredSchedulingTerm.Weight must be marked as optional or required"
 	// A node selector term, associated with the corresponding weight. // want "commentstart: godoc for field PreferredSchedulingTerm.Preference should start with 'preference ...'"
-	Preference NodeSelectorTerm `json:"preference" protobuf:"bytes,2,opt,name=preference"` // want "optionalorrequired: field PreferredSchedulingTerm.Preference must be marked as optional or required"
+	Preference NodeSelectorTerm `json:"preference" protobuf:"bytes,2,opt,name=preference"` // want "optionalorrequired: field PreferredSchedulingTerm.Preference must be marked as optional or required" "nonpointerstructs: field PreferredSchedulingTerm.Preference is a non-pointer struct with no required fields. It must be marked as optional."
 }
 
 // The node this Taint is attached to has the "effect" on

tests/integration/testdata/default_configurations/object_references.go

@@ -1,7 +1,5 @@
 package defaultconfigurations
 
-import "k8s.io/apimachinery/pkg/types"
-
 // ObjectReference contains enough information to let you inspect or modify the referred object.
 // ---
 // New uses of this type are discouraged because of difficulty describing its usage when embedded in APIs.
@@ -33,10 +31,11 @@ type ObjectReference struct {
 	// More info: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
 	// +optional
 	Name string `json:"name,omitempty" protobuf:"bytes,3,opt,name=name"` // want "optionalfields: field ObjectReference.Name should be a pointer."
-	// UID of the referent. // want "commentstart: godoc for field ObjectReference.UID should start with 'uid ...'"
+	// UID of the referent.
 	// More info: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#uids
 	// +optional
-	UID types.UID `json:"uid,omitempty" protobuf:"bytes,4,opt,name=uid,casttype=k8s.io/apimachinery/pkg/types.UID"` // want "optionalfields: field ObjectReference.UID should be a pointer."
+	// UID types.UID `json:"uid,omitempty" protobuf:"bytes,4,opt,name=uid,casttype=k8s.io/apimachinery/pkg/types.UID"`
+
 	// API version of the referent. // want "commentstart: godoc for field ObjectReference.APIVersion should start with 'apiVersion ...'"
 	// +optional
 	APIVersion string `json:"apiVersion,omitempty" protobuf:"bytes,5,opt,name=apiVersion"` // want "optionalfields: field ObjectReference.APIVersion should be a pointer."

tests/integration/testdata/default_configurations/volume.go

@@ -1634,7 +1634,7 @@ type PersistentVolumeClaimTemplate struct {
 	// copied unchanged into the PVC that gets created from this
 	// template. The same fields as in a PersistentVolumeClaim
 	// are also valid here.
-	Spec PersistentVolumeClaimSpec `json:"spec" protobuf:"bytes,2,name=spec"` // want "optionalorrequired: field PersistentVolumeClaimTemplate.Spec must be marked as optional or required"
+	Spec PersistentVolumeClaimSpec `json:"spec" protobuf:"bytes,2,name=spec"` // want "optionalorrequired: field PersistentVolumeClaimTemplate.Spec must be marked as optional or required" "nonpointerstructs: field PersistentVolumeClaimTemplate.Spec is a non-pointer struct with no required fields. It must be marked as optional."
 }
 
 // Adapts a ConfigMap into a projected volume.