Check return value from volume.SetVolumeOwnership() in downwardAPI

The function can fail, so we must check the return code.
lavalamp · Aug 17, 2016 · 5637569 · 5637569
1 parent 568f4c2
commit 5637569
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/pkg/volume/downwardapi/downwardapi.go b/pkg/volume/downwardapi/downwardapi.go
@@ -187,7 +187,11 @@ func (b *downwardAPIVolumeMounter) SetUpAt(dir string, fsGroup *int64) error {
 		return err
 	}
 
-	volume.SetVolumeOwnership(b, fsGroup)
+	err = volume.SetVolumeOwnership(b, fsGroup)
+	if err != nil {
+		glog.Errorf("Error applying volume ownership settings for group: %v", fsGroup)
+		return err
+	}
 
 	return nil
 }