container-storage-interface / spec

Container Storage Interface (CSI) Specification.
Apache License 2.0
1.34k stars 373 forks source link

Relax node_id length limit to 256 #475

Closed Jiawei0227 closed 3 years ago

Jiawei0227 commented 3 years ago

node_id has been relaxed to 192 in https://github.com/container-storage-interface/spec/commit/396c3332ca1216dea620f64f5f2d60686ae9a0a5. But it is still not long enough for some plugins node_id. So relax the constraint to 256 to make sure this satisfy the requirements.

saad-ali commented 3 years ago

/lgtm /approve

ddebroy commented 3 years ago

/lgtm

saad-ali commented 3 years ago

Thanks. No objections to this in the CSI Community meeting either. Going to go ahead and merge this.