Change:
This came up as a follow up to Also rather than opening 5 different issues - we are going to use this github issue to track all follow up items mentioned here -
/sig storage
/assign /assign
: PR
and the rest is fixed in

Discussion:
"description": "VolumeError captures an error encountered during a volume operation.", "properties": { "message": { <del> "description": "String detailing the error encountered during Attach or Detach operation. This string maybe logged, so it should not contain sensitive information.", </del> <ins> "description": "String detailing the error encountered during Attach or Detach operation. This string may be logged, so it should not contain sensitive information.", </ins> "type": "string" }, "time": {

}, "message": { "type": "string", <del> "description": "String detailing the error encountered during Attach or Detach operation. This string maybe logged, so it should not contain sensitive information." </del> <ins> "description": "String detailing the error encountered during Attach or Detach operation. This string may be logged, so it should not contain sensitive information." </ins> } } },

</tr> <tr> <td class="tableblock halign-left valign-top"><p class="tableblock">message</p></td> <del> <td class="tableblock halign-left valign-top"><p class="tableblock">String detailing the error encountered during Attach or Detach operation. This string maybe logged, so it should not contain sensitive information.</p></td> </del> <ins> <td class="tableblock halign-left valign-top"><p class="tableblock">String detailing the error encountered during Attach or Detach operation. This string may be logged, so it should not contain sensitive information.</p></td> </ins> <td class="tableblock halign-left valign-top"><p class="tableblock">false</p></td> <td class="tableblock halign-left valign-top"><p class="tableblock">string</p></td> <td class="tableblock halign-left valign-top"></td>

Time metav1.Time // String detailing the error encountered during Attach or Detach operation. <del> // This string maybe logged, so it should not contain sensitive </del> <ins> // This string may be logged, so it should not contain sensitive </ins> // information. // +optional Message string

store *genericregistry.Store } <ins> var _ = rest.Patcher(&StatusREST{}) </ins> // New creates a new VolumeAttachment resource func (r *StatusREST) New() runtime.Object { return &storageapi.VolumeAttachment{}

t.Errorf("unexpected objects differerence after modifying status: %v", diff.ObjectDiff(statusVolumeAttachment, expectedVolumeAttachment)) } <del> // modifying spec should be dropped </del> <ins> // spec and metadata modifications should be dropped </ins> newVolumeAttachment := volumeAttachment.DeepCopy() newVolumeAttachment.Spec.NodeName = "valid-node-2" <ins> newVolumeAttachment.Labels = map[string]string{"foo": "bar"} newVolumeAttachment.Annotations = map[string]string{"foo": "baz"} newVolumeAttachment.OwnerReferences = []metav1.OwnerReference{ { APIVersion: "v1", Kind:       "Pod", Name:       "Foo", }, } </ins> StatusStrategy.PrepareForUpdate(ctx, newVolumeAttachment, volumeAttachment) if !apiequality.Semantic.DeepEqual(newVolumeAttachment, volumeAttachment) {

optional k8s.io.apimachinery.pkg.apis.meta.v1.Time time = 1; // String detailing the error encountered during Attach or Detach operation. <del> // This string maybe logged, so it should not contain sensitive </del> <ins> // This string may be logged, so it should not contain sensitive </ins> // information. // +optional optional string message = 2;

Time metav1.Time `json:"time,omitempty" protobuf:"bytes,1,opt,name=time"` // String detailing the error encountered during Attach or Detach operation. <del> // This string maybe logged, so it should not contain sensitive </del> <ins> // This string may be logged, so it should not contain sensitive </ins> // information. // +optional Message string `json:"message,omitempty" protobuf:"bytes,2,opt,name=message"`

var map_VolumeError = map[string]string{ "":        "VolumeError captures an error encountered during a volume operation.", "time":    "Time the error was encountered.", <del> "message": "String detailing the error encountered during Attach or Detach operation. This string maybe logged, so it should not contain sensitive information.", </del> <ins> "message": "String detailing the error encountered during Attach or Detach operation. This string may be logged, so it should not contain sensitive information.", </ins> } func (VolumeError) SwaggerDoc() map[string]string {