Merge pull request #38655 from abrarshivani/fsGroupforvSphere

Automatic merge from submit-queue (batch tested with PRs 39059, 39175, 35676, 38655) Fix fsGroup to vSphere **What this PR does / why we need it**: Fixes #34039 by adding support for fsGroup to vSphere Volume. **Special notes for your reviewer**: Tested with example from http://stackoverflow.com/questions/35213589/docker-container-with-non-root-user-deployed-in-google-container-engine-can-not Before this fix got error ```Permission Denied```. **Release note**: `NONE` cc @pdhamdhere @kerneltime @BaluDontu

Merge pull request #38655 from abrarshivani/fsGroupforvSphere
f1aa0258 · Kubernetes Submit Queue · GitHub · af54124f · 5cb7faac · f1aa0258
Commit f1aa0258 authored Dec 22, 2016 by Kubernetes Submit Queue Committed by GitHub Dec 22, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 0 deletions

vsphere_volume.go pkg/volume/vsphere_volume/vsphere_volume.go +1 -0

No files found.
--- a/pkg/volume/vsphere_volume/vsphere_volume.go
+++ b/pkg/volume/vsphere_volume/vsphere_volume.go
@@ -243,6 +243,7 @@ func (b *vsphereVolumeMounter) SetUpAt(dir string, fsGroup *int64) error {
 		os.Remove(dir)
 		return err
 	}
+	volume.SetVolumeOwnership(b, fsGroup)
 	glog.V(3).Infof("vSphere volume %s mounted to %s", b.volPath, dir)

 	return nil