Add metadata information in tag on PD

[davidz627]

info like namespace and pvc name (check with feature parity for in-tree plugin)

[davidz627]

In-tree plugin provisions with Description field example:

Description
{"kubernetes.io/created-for/pv/name":"pvc-6a10b78e-4cae-4c9e-8dd2-1cb2c60211e1","kubernetes.io/created-for/pvc/name":"test-disk","kubernetes.io/created-for/pvc/namespace":"default"}

No such thing as a "tag" AFAIK. Do people really rely on json description text? I guess we should continue this for backwards compatibility reasons. Considering adding some more description fields to specify the driver name that provisioned to volume.

/cc @msau42

[msau42]

What about "labels"? https://cloud.google.com/kubernetes-engine/docs/how-to/creating-managing-labels

[davidz627]

IIRK labels are new and basically what we were trying to approximate with the description field before. Labels are not used by in-tree currently. They are empty on my test PD.

IIUC description should stay the same because of backwards compatibility (who is actually depending on this functionality...?). Do we even have any expectations for backwards compatibility for a freeform text field meant for human-readable descriptions?

We could duplicate whats in the description into the labels but that's almost uglier than only having them in the description. However, I would strongly prefer putting this information only in the labels and break the descriptions field.

[msau42]

It has the possibility to break users once migration is on though.

[davidz627]

Depends on: kubernetes-csi/external-provisioner#370

[fejta-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

[davidz627]

/remove-lifecycle stale

[davidz627]

depends on: kubernetes-csi/external-provisioner#399

[verult]

kubernetes-csi/external-provisioner#399 now available in v1.6.0

[fejta-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

[verult]

/remove-lifecycle stale

[shay-berman]

what is the status of this ticket?
I would like to know when CSI driver will be GA in GKE?

[saad-ali]

/assign