Skip to content

Commit 21a7e9d

Browse files
committed
Updated Kubernetes dependency to v1.15 and fixed all dep errors
1 parent 8156495 commit 21a7e9d

File tree

2,201 files changed

+329776
-629646
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

2,201 files changed

+329776
-629646
lines changed

Gopkg.lock

+165-595
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Gopkg.toml

+39-37
Original file line numberDiff line numberDiff line change
@@ -24,65 +24,67 @@
2424
# go-tests = true
2525
# unused-packages = true
2626

27-
[[constraint]]
28-
name = "cloud.google.com/go"
29-
version = "0.34.0"
27+
# WARNING: IF YOU ARE GETTING WEIRD DEP ERRORS MAKE SURE TO
28+
# rm -rf test/k8s-integration/src
29+
# AS IT MAY CONTAIN A PULLED IN KUBERNETES
3030

31-
[[constraint]]
32-
name = "github.com/container-storage-interface/spec"
33-
version = "1.1.0"
31+
[prune]
32+
go-tests = true
33+
unused-packages = true
34+
non-go = true
3435

35-
# Transitive dependency of CSI Spec
36-
[[override]]
37-
name = "github.com/golang/protobuf"
38-
version = "1.1.0"
36+
[[constraint]]
37+
name = "cloud.google.com/go"
38+
version = "0.40.0"
3939

4040
[[constraint]]
4141
branch = "master"
42-
name = "github.com/golang/glog"
43-
44-
#[[constraint]]
45-
# name = "github.com/stretchr/testify"
46-
# version = "1.2.1"
42+
name = "golang.org/x/oauth2"
4743

4844
[[constraint]]
4945
branch = "master"
50-
name = "golang.org/x/net"
46+
name = "google.golang.org/api"
5147

5248
[[constraint]]
53-
branch = "master"
54-
name = "golang.org/x/oauth2"
49+
name = "github.com/kubernetes-csi/csi-test"
50+
version = "v1.0.0"
5551

5652
[[constraint]]
5753
branch = "master"
58-
name = "google.golang.org/api"
54+
name = "k8s.io/test-infra"
55+
56+
###
57+
### Kubernetes Dependencies
58+
###
5959

6060
[[constraint]]
61-
name = "google.golang.org/grpc"
62-
version = "1.10.0"
61+
version = "v1.15.0"
62+
name = "k8s.io/kubernetes"
6363

6464
[[constraint]]
65-
branch = "release-1.11"
65+
version = "kubernetes-1.15.0"
6666
name = "k8s.io/apimachinery"
6767

68+
###
69+
### CSI Spec Dependencies
70+
###
71+
6872
[[constraint]]
69-
branch = "master"
70-
name = "k8s.io/utils"
73+
name = "github.com/container-storage-interface/spec"
74+
version = "1.1.0"
7175

72-
[prune]
73-
go-tests = true
74-
unused-packages = true
75-
non-go = true
76+
[[override]]
77+
name = "google.golang.org/grpc"
78+
version = "1.10.0"
7679

77-
[[constraint]]
78-
name = "github.com/kubernetes-csi/csi-test"
79-
version = "v1.0.0-rc2"
80+
[[override]]
81+
name = "github.com/golang/protobuf"
82+
version = "1.1.0"
8083

81-
[[constraint]]
82-
branch = "master"
83-
name = "k8s.io/test-infra"
84+
###
85+
### Crazy Stuff
86+
###
8487

85-
# Transitive dependency of k8s.io/kubernetes/pkg/util/parsers
8688
[[override]]
87-
branch = "master"
88-
name = "github.com/docker/distribution"
89+
name = "gopkg.in/fsnotify.v1"
90+
source = "https://github.com/fsnotify/fsnotify.git"

cmd/main.go

+6-6
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ import (
2020
"os"
2121
"time"
2222

23-
"github.com/golang/glog"
23+
"k8s.io/klog"
2424

2525
gce "sigs.k8s.io/gcp-compute-persistent-disk-csi-driver/pkg/gce-cloud-provider/compute"
2626
metadataservice "sigs.k8s.io/gcp-compute-persistent-disk-csi-driver/pkg/gce-cloud-provider/metadata"
@@ -51,29 +51,29 @@ func main() {
5151

5252
func handle() {
5353
if vendorVersion == "" {
54-
glog.Fatalf("vendorVersion must be set at compile time")
54+
klog.Fatalf("vendorVersion must be set at compile time")
5555
}
56-
glog.V(4).Infof("Driver vendor version %v", vendorVersion)
56+
klog.V(4).Infof("Driver vendor version %v", vendorVersion)
5757

5858
gceDriver := driver.GetGCEDriver()
5959

6060
//Initialize GCE Driver (Move setup to main?)
6161
cloudProvider, err := gce.CreateCloudProvider(vendorVersion, *gceConfigFilePath)
6262
if err != nil {
63-
glog.Fatalf("Failed to get cloud provider: %v", err)
63+
klog.Fatalf("Failed to get cloud provider: %v", err)
6464
}
6565

6666
mounter := mountmanager.NewSafeMounter()
6767
deviceUtils := mountmanager.NewDeviceUtils()
6868

6969
ms, err := metadataservice.NewMetadataService()
7070
if err != nil {
71-
glog.Fatalf("Failed to set up metadata service: %v", err)
71+
klog.Fatalf("Failed to set up metadata service: %v", err)
7272
}
7373

7474
err = gceDriver.SetupGCEDriver(cloudProvider, mounter, deviceUtils, ms, driverName, vendorVersion)
7575
if err != nil {
76-
glog.Fatalf("Failed to initialize GCE CSI Driver: %v", err)
76+
klog.Fatalf("Failed to initialize GCE CSI Driver: %v", err)
7777
}
7878

7979
gceDriver.Run(*endpoint)

pkg/common/utils.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@ import (
2020
"fmt"
2121
"strings"
2222

23+
"github.com/GoogleCloudPlatform/k8s-cloud-provider/pkg/cloud/meta"
2324
"k8s.io/apimachinery/pkg/util/sets"
24-
"k8s.io/kubernetes/pkg/cloudprovider/providers/gce/cloud/meta"
2525
)
2626

2727
const (

pkg/common/utils_test.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ import (
2121
"reflect"
2222
"testing"
2323

24-
"k8s.io/kubernetes/pkg/cloudprovider/providers/gce/cloud/meta"
24+
"github.com/GoogleCloudPlatform/k8s-cloud-provider/pkg/cloud/meta"
2525
)
2626

2727
const (

pkg/gce-cloud-provider/compute/fake-gce.go

+5-5
Original file line numberDiff line numberDiff line change
@@ -15,19 +15,19 @@ limitations under the License.
1515
package gcecloudprovider
1616

1717
import (
18+
"context"
1819
"fmt"
1920
"strconv"
2021
"strings"
2122

23+
"github.com/GoogleCloudPlatform/k8s-cloud-provider/pkg/cloud/meta"
2224
csi "github.com/container-storage-interface/spec/lib/go/csi"
23-
"github.com/golang/glog"
24-
"golang.org/x/net/context"
2525
computebeta "google.golang.org/api/compute/v0.beta"
2626
compute "google.golang.org/api/compute/v1"
2727
"google.golang.org/api/googleapi"
2828
"google.golang.org/grpc/codes"
2929
"google.golang.org/grpc/status"
30-
"k8s.io/kubernetes/pkg/cloudprovider/providers/gce/cloud/meta"
30+
"k8s.io/klog"
3131
"sigs.k8s.io/gcp-compute-persistent-disk-csi-driver/pkg/common"
3232
)
3333

@@ -184,7 +184,7 @@ func (cloud *FakeCloudProvider) ValidateExistingDisk(ctx context.Context, resp *
184184
return fmt.Errorf("disk already exists with incompatible type. Need %v. Got %v",
185185
diskType, respType[len(respType)-1])
186186
}
187-
glog.V(4).Infof("Compatible disk already exists")
187+
klog.V(4).Infof("Compatible disk already exists")
188188
return nil
189189
}
190190

@@ -384,7 +384,7 @@ func (cloud *FakeCloudProvider) ValidateExistingSnapshot(resp *compute.Snapshot,
384384
return status.Error(codes.AlreadyExists, fmt.Sprintf("snapshot already exists with same name but with a different disk source %s, expected disk source %s", diskSource, resp.SourceDisk))
385385
}
386386
// Snapshot exists with matching source disk.
387-
glog.V(4).Infof("Compatible snapshot already exists. Reusing existing.")
387+
klog.V(4).Infof("Compatible snapshot already exists. Reusing existing.")
388388
return nil
389389
}
390390

pkg/gce-cloud-provider/compute/gce-compute.go

+25-24
Original file line numberDiff line numberDiff line change
@@ -19,15 +19,16 @@ import (
1919
"strings"
2020
"time"
2121

22+
"context"
23+
24+
"github.com/GoogleCloudPlatform/k8s-cloud-provider/pkg/cloud/meta"
2225
csi "github.com/container-storage-interface/spec/lib/go/csi"
23-
"github.com/golang/glog"
24-
"golang.org/x/net/context"
2526
computebeta "google.golang.org/api/compute/v0.beta"
2627
compute "google.golang.org/api/compute/v1"
2728
"google.golang.org/grpc/codes"
2829
"google.golang.org/grpc/status"
2930
"k8s.io/apimachinery/pkg/util/wait"
30-
"k8s.io/kubernetes/pkg/cloudprovider/providers/gce/cloud/meta"
31+
"k8s.io/klog"
3132
"sigs.k8s.io/gcp-compute-persistent-disk-csi-driver/pkg/common"
3233
)
3334

@@ -143,23 +144,23 @@ func (cloud *CloudProvider) GetDisk(ctx context.Context, key *meta.Key) (*CloudD
143144
func (cloud *CloudProvider) getZonalDiskOrError(ctx context.Context, volumeZone, volumeName string) (*compute.Disk, error) {
144145
svc := cloud.service
145146
project := cloud.project
146-
glog.V(4).Infof("Getting disk %v from zone %v", volumeName, volumeZone)
147+
klog.V(4).Infof("Getting disk %v from zone %v", volumeName, volumeZone)
147148
disk, err := svc.Disks.Get(project, volumeZone, volumeName).Context(ctx).Do()
148149
if err != nil {
149150
return nil, err
150151
}
151-
glog.V(4).Infof("Got disk %v from zone %v", volumeName, volumeZone)
152+
klog.V(4).Infof("Got disk %v from zone %v", volumeName, volumeZone)
152153
return disk, nil
153154
}
154155

155156
func (cloud *CloudProvider) getRegionalDiskOrError(ctx context.Context, volumeRegion, volumeName string) (*computebeta.Disk, error) {
156157
project := cloud.project
157-
glog.V(4).Infof("Getting disk %v from region %v", volumeName, volumeRegion)
158+
klog.V(4).Infof("Getting disk %v from region %v", volumeName, volumeRegion)
158159
disk, err := cloud.betaService.RegionDisks.Get(project, volumeRegion, volumeName).Context(ctx).Do()
159160
if err != nil {
160161
return nil, err
161162
}
162-
glog.V(4).Infof("Got disk %v from region %v", volumeName, volumeRegion)
163+
klog.V(4).Infof("Got disk %v from region %v", volumeName, volumeRegion)
163164
return disk, nil
164165
}
165166

@@ -198,7 +199,7 @@ func (cloud *CloudProvider) ValidateExistingDisk(ctx context.Context, resp *Clou
198199
}
199200

200201
// Volume exists with matching name, capacity, type.
201-
glog.V(4).Infof("Compatible disk already exists. Reusing existing.")
202+
klog.V(4).Infof("Compatible disk already exists. Reusing existing.")
202203
return nil
203204
}
204205

@@ -245,7 +246,7 @@ func (cloud *CloudProvider) insertRegionalDisk(ctx context.Context, volKey *meta
245246
if err != nil {
246247
return err
247248
}
248-
glog.Warningf("GCE PD %s already exists, reusing", volKey.Name)
249+
klog.Warningf("GCE PD %s already exists, reusing", volKey.Name)
249250
return nil
250251
}
251252
return status.Error(codes.Internal, fmt.Sprintf("unkown Insert disk error: %v", err))
@@ -264,7 +265,7 @@ func (cloud *CloudProvider) insertRegionalDisk(ctx context.Context, volKey *meta
264265
if err != nil {
265266
return err
266267
}
267-
glog.Warningf("GCE PD %s already exists after wait, reusing", volKey.Name)
268+
klog.Warningf("GCE PD %s already exists after wait, reusing", volKey.Name)
268269
return nil
269270
}
270271
return fmt.Errorf("unkown Insert disk operation error: %v", err)
@@ -304,7 +305,7 @@ func (cloud *CloudProvider) insertZonalDisk(ctx context.Context, volKey *meta.Ke
304305
if err != nil {
305306
return err
306307
}
307-
glog.Warningf("GCE PD %s already exists, reusing", volKey.Name)
308+
klog.Warningf("GCE PD %s already exists, reusing", volKey.Name)
308309
return nil
309310
}
310311
return fmt.Errorf("unkown Insert disk error: %v", err)
@@ -324,7 +325,7 @@ func (cloud *CloudProvider) insertZonalDisk(ctx context.Context, volKey *meta.Ke
324325
if err != nil {
325326
return err
326327
}
327-
glog.Warningf("GCE PD %s already exists after wait, reusing", volKey.Name)
328+
klog.Warningf("GCE PD %s already exists after wait, reusing", volKey.Name)
328329
return nil
329330
}
330331
return fmt.Errorf("unkown Insert disk operation error: %v", err)
@@ -465,7 +466,7 @@ func (cloud *CloudProvider) waitForZonalOp(ctx context.Context, op *compute.Oper
465466
return wait.Poll(3*time.Second, 5*time.Minute, func() (bool, error) {
466467
pollOp, err := svc.ZoneOperations.Get(project, zone, op.Name).Context(ctx).Do()
467468
if err != nil {
468-
glog.Errorf("WaitForOp(op: %#v, zone: %#v) failed to poll the operation", op, zone)
469+
klog.Errorf("WaitForOp(op: %#v, zone: %#v) failed to poll the operation", op, zone)
469470
return false, err
470471
}
471472
done, err := opIsDone(pollOp)
@@ -477,7 +478,7 @@ func (cloud *CloudProvider) waitForRegionalOp(ctx context.Context, op *computebe
477478
return wait.Poll(3*time.Second, 5*time.Minute, func() (bool, error) {
478479
pollOp, err := cloud.betaService.RegionOperations.Get(cloud.project, region, op.Name).Context(ctx).Do()
479480
if err != nil {
480-
glog.Errorf("WaitForOp(op: %#v, region: %#v) failed to poll the operation", op, region)
481+
klog.Errorf("WaitForOp(op: %#v, region: %#v) failed to poll the operation", op, region)
481482
return false, err
482483
}
483484
done, err := regionalOpIsDone(pollOp)
@@ -491,7 +492,7 @@ func (cloud *CloudProvider) waitForGlobalOp(ctx context.Context, op *compute.Ope
491492
return wait.Poll(3*time.Second, 5*time.Minute, func() (bool, error) {
492493
pollOp, err := svc.GlobalOperations.Get(project, op.Name).Context(ctx).Do()
493494
if err != nil {
494-
glog.Errorf("waitForGlobalOp(op: %#v) failed to poll the operation", op)
495+
klog.Errorf("waitForGlobalOp(op: %#v) failed to poll the operation", op)
495496
return false, err
496497
}
497498
done, err := opIsDone(pollOp)
@@ -503,7 +504,7 @@ func (cloud *CloudProvider) WaitForAttach(ctx context.Context, volKey *meta.Key,
503504
return wait.Poll(5*time.Second, 2*time.Minute, func() (bool, error) {
504505
disk, err := cloud.GetDisk(ctx, volKey)
505506
if err != nil {
506-
glog.Errorf("GetDisk failed to get disk: %v", err)
507+
klog.Errorf("GetDisk failed to get disk: %v", err)
507508
return false, err
508509
}
509510

@@ -543,25 +544,25 @@ func regionalOpIsDone(op *computebeta.Operation) (bool, error) {
543544
func (cloud *CloudProvider) GetInstanceOrError(ctx context.Context, instanceZone, instanceName string) (*compute.Instance, error) {
544545
svc := cloud.service
545546
project := cloud.project
546-
glog.V(4).Infof("Getting instance %v from zone %v", instanceName, instanceZone)
547+
klog.V(4).Infof("Getting instance %v from zone %v", instanceName, instanceZone)
547548

548549
instance, err := svc.Instances.Get(project, instanceZone, instanceName).Do()
549550
if err != nil {
550551
return nil, err
551552
}
552-
glog.V(4).Infof("Got instance %v from zone %v", instanceName, instanceZone)
553+
klog.V(4).Infof("Got instance %v from zone %v", instanceName, instanceZone)
553554
return instance, nil
554555
}
555556

556557
func (cloud *CloudProvider) GetSnapshot(ctx context.Context, snapshotName string) (*compute.Snapshot, error) {
557558
svc := cloud.service
558559
project := cloud.project
559-
glog.V(4).Infof("Getting snapshot %v", snapshotName)
560+
klog.V(4).Infof("Getting snapshot %v", snapshotName)
560561
snapshot, err := svc.Snapshots.Get(project, snapshotName).Context(ctx).Do()
561562
if err != nil {
562563
return nil, err
563564
}
564-
glog.V(4).Infof("Got snapshot %v", snapshotName)
565+
klog.V(4).Infof("Got snapshot %v", snapshotName)
565566
return snapshot, nil
566567
}
567568

@@ -629,16 +630,16 @@ func (cloud *CloudProvider) waitForSnapshotCreation(ctx context.Context, snapsho
629630
for {
630631
select {
631632
case <-ticker.C:
632-
glog.V(5).Infof("Checking GCE Snapshot %s.", snapshotName)
633+
klog.V(5).Infof("Checking GCE Snapshot %s.", snapshotName)
633634
snapshot, err := cloud.GetSnapshot(ctx, snapshotName)
634635
if err != nil {
635-
glog.Warningf("Error in getting snapshot %s, %v", snapshotName, err)
636+
klog.Warningf("Error in getting snapshot %s, %v", snapshotName, err)
636637
} else if snapshot != nil {
637638
if snapshot.Status != "CREATING" {
638-
glog.V(5).Infof("Snapshot %s status is %s", snapshotName, snapshot.Status)
639+
klog.V(5).Infof("Snapshot %s status is %s", snapshotName, snapshot.Status)
639640
return snapshot, nil
640641
} else {
641-
glog.V(5).Infof("Snapshot %s is still creating ...", snapshotName)
642+
klog.V(5).Infof("Snapshot %s is still creating ...", snapshotName)
642643
}
643644
}
644645
case <-timer.C:

0 commit comments

Comments
 (0)