Skip to content
This repository was archived by the owner on Mar 22, 2018. It is now read-only.

Commit 8335505

Browse files
author
Kubernetes Submit Queue
authored
Merge pull request #47824 from mbohlool/revert2
Automatic merge from submit-queue (batch tested with PRs 47851, 47824, 47858, 46099) Revert 44714 manually #44714 broke backward compatibility for old swagger spec that kubectl still uses. The decision on #47448 was to revert this change but the change was not automatically revertible. Here I semi-manually remove all references to UnixUserID and UnixGroupID and updated generated files accordingly. Please wait for tests to pass then review that as there may still be tests that are failing. Fixes #47448 Adding release note just because the original PR has a release note. If possible, we should remove both release notes as they cancel each other. **Release note**: (removed by caesarxuchao) UnixUserID and UnixGroupID is reverted back as int64 to keep backward compatibility.
2 parents 9fc70f2 + 10bdf7f commit 8335505

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

pkg/volume/cinder/cinder.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -298,12 +298,12 @@ func (b *cinderVolumeMounter) CanMount() error {
298298
return nil
299299
}
300300

301-
func (b *cinderVolumeMounter) SetUp(fsGroup *types.UnixGroupID) error {
301+
func (b *cinderVolumeMounter) SetUp(fsGroup *int64) error {
302302
return b.SetUpAt(b.GetPath(), fsGroup)
303303
}
304304

305305
// SetUp bind mounts to the volume path.
306-
func (b *cinderVolumeMounter) SetUpAt(dir string, fsGroup *types.UnixGroupID) error {
306+
func (b *cinderVolumeMounter) SetUpAt(dir string, fsGroup *int64) error {
307307
glog.V(5).Infof("Cinder SetUp %s to %s", b.pdName, dir)
308308

309309
b.plugin.volumeLocks.LockKey(b.pdName)

0 commit comments

Comments
 (0)