From 36178fe68a166ec69cb7ef88da2a1d91d620fd36 Mon Sep 17 00:00:00 2001 From: chrislusf Date: Thu, 23 Oct 2025 19:52:16 -0700 Subject: Fix filer_pb API signatures to include context parameter The filer_pb.Mkdir, filer_pb.Remove, and filer_pb.Exists functions now require context.Context as the first parameter. This commit updates all three function calls in controllerserver.go to match the updated API. Fixes #206 --- pkg/driver/controllerserver.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/pkg/driver/controllerserver.go b/pkg/driver/controllerserver.go index 3a2d9fc..987dc84 100644 --- a/pkg/driver/controllerserver.go +++ b/pkg/driver/controllerserver.go @@ -53,7 +53,7 @@ func (cs *ControllerServer) CreateVolume(ctx context.Context, req *csi.CreateVol glog.V(4).Infof("params:%v", params) capacity := req.GetCapacityRange().GetRequiredBytes() - if err := filer_pb.Mkdir(cs.Driver, "/buckets", volumeId, nil); err != nil { + if err := filer_pb.Mkdir(ctx, cs.Driver, "/buckets", volumeId, nil); err != nil { return nil, fmt.Errorf("error setting bucket metadata: %v", err) } @@ -84,7 +84,7 @@ func (cs *ControllerServer) DeleteVolume(ctx context.Context, req *csi.DeleteVol } glog.V(4).Infof("deleting volume %s", volumeId) - if err := filer_pb.Remove(cs.Driver, "/buckets", volumeId, true, true, true, false, nil); err != nil { + if err := filer_pb.Remove(ctx, cs.Driver, "/buckets", volumeId, true, true, true, false, nil); err != nil { return nil, fmt.Errorf("error setting bucket metadata: %v", err) } @@ -135,7 +135,7 @@ func (cs *ControllerServer) ValidateVolumeCapabilities(ctx context.Context, req return nil, status.Error(codes.InvalidArgument, "Volume capabilities missing in request") } - exists, err := filer_pb.Exists(cs.Driver, "/buckets", req.GetVolumeId(), true) + exists, err := filer_pb.Exists(ctx, cs.Driver, "/buckets", req.GetVolumeId(), true) if err != nil { return nil, fmt.Errorf("error checking bucket %s exists: %v", req.GetVolumeId(), err) } -- cgit v1.2.3