Merge pull request #12 from travisghansen/master

resolve 120 char limit for NFS share comment
nmaupu · Oct 12, 2018 · dca7170 · dca7170
2 parents b86b58f + 5ac0d78
commit dca7170
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/README.md b/README.md
@@ -136,6 +136,8 @@ make fmt
  * https://github.com/kubernetes/community/blob/master/contributors/design-proposals/storage/volume-provisioning.md
  * https://kubernetes.io/docs/concepts/storage/storage-classes/
  * https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.11/#-strong-api-overview-strong-
+ * https://github.com/kubernetes/community/blob/master/contributors/design-proposals/storage/container-storage-interface.md
+ * https://github.com/kubernetes-csi/drivers/tree/master/pkg
 
 ## TODO
  * volume resizing - https://github.com/kubernetes/community/blob/master/contributors/design-proposals/storage/grow-volume-size.md

diff --git a/provisioner/provisioner.go b/provisioner/provisioner.go
@@ -311,7 +311,7 @@ func (p *freenasProvisioner) Provision(options controller.VolumeOptions) (*v1.Pe
 		MapallGroup:  config.ShareMapallGroup,
 		MaprootUser:  config.ShareMaprootUser,
 		MaprootGroup: config.ShareMaprootGroup,
-		Comment:      fmt.Sprintf("freenas-provisioner (%s): %s", p.Identifier, dsPath),
+		Comment:      fmt.Sprintf("freenas-provisioner (%s): %s", p.Identifier, dsPath)[0:120],
 	}
 
 	glog.Infof("Creating dataset: \"%s\", NFS share: \"%s\"", ds.Name, path)