Bump SMB api from v1beta2 to v1

Author: mauriciopoppe

client/api/smb/v1/api.pb.go

@@ -0,0 +1,57 @@
+syntax = "proto3";
+
+package v1;
+
+option go_package = "github.com/kubernetes-csi/csi-proxy/client/api/smb/v1";
+
+service Smb {
+    // NewSmbGlobalMapping creates an SMB mapping on the SMB client to an SMB share.
+    rpc NewSmbGlobalMapping(NewSmbGlobalMappingRequest) returns (NewSmbGlobalMappingResponse) {}
+
+    // RemoveSmbGlobalMapping removes the SMB mapping to an SMB share.
+    rpc RemoveSmbGlobalMapping(RemoveSmbGlobalMappingRequest) returns (RemoveSmbGlobalMappingResponse) {}
+}
+
+
+message NewSmbGlobalMappingRequest {
+    // A remote SMB share to mount
+    // All unicode characters allowed in SMB server name specifications are
+    // permitted except for restrictions below
+    //
+    // Restrictions:
+    // SMB remote path specified in the format: \\server-name\sharename, \\server.fqdn\sharename or \\a.b.c.d\sharename
+    // If not an IP address, share name has to be a valid DNS name.
+    // UNC specifications to local paths or prefix: \\?\ is not allowed.
+    // Characters: + [ ] " / : ; | < > , ? * = $ are not allowed.
+    string remote_path = 1;
+    // Optional local path to mount the smb on
+    string local_path = 2;
+
+    // Username credential associated with the share
+    string username = 3;
+
+    // Password credential associated with the share
+    string password = 4;
+}
+
+message NewSmbGlobalMappingResponse {
+    // Intentionally empty.
+}
+
+
+message RemoveSmbGlobalMappingRequest {
+    // A remote SMB share mapping to remove
+    // All unicode characters allowed in SMB server name specifications are
+    // permitted except for restrictions below
+    //
+    // Restrictions:
+    // SMB share specified in the format: \\server-name\sharename, \\server.fqdn\sharename or \\a.b.c.d\sharename
+    // If not an IP address, share name has to be a valid DNS name.
+    // UNC specifications to local paths or prefix: \\?\ is not allowed.
+    // Characters: + [ ] " / : ; | < > , ? * = $ are not allowed.
+    string remote_path = 1;
+}
+
+message RemoveSmbGlobalMappingResponse {
+    // Intentionally empty.
+}

client/api/smb/v1/api.proto

@@ -132,4 +132,7 @@ func TestSmbAPIGroup(t *testing.T) {
 	t.Run("v1beta2SmbTests", func(t *testing.T) {
 		v1beta2SmbTests(t)
 	})
+	t.Run("v1SmbTests", func(t *testing.T) {
+		v1SmbTests(t)
+	})
 }

client/groups/smb/v1/client_generated.go

@@ -0,0 +1,73 @@
+package integrationtests
+
+import (
+	"context"
+	"fmt"
+	"os"
+	"testing"
+
+	"github.com/kubernetes-csi/csi-proxy/client/api/smb/v1"
+	client "github.com/kubernetes-csi/csi-proxy/client/groups/smb/v1"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func v1SmbTests(t *testing.T) {
+	client, err := client.NewClient()
+	if err != nil {
+		t.Fatalf("Fail to get smb API group client %v", err)
+	}
+	defer client.Close()
+
+	username := randomString(5)
+	password := randomString(10) + "!"
+	sharePath := fmt.Sprintf("C:\\smbshare%s", randomString(5))
+	smbShare := randomString(6)
+
+	localPath := fmt.Sprintf("C:\\localpath%s", randomString(5))
+
+	if err = setupUser(username, password); err != nil {
+		t.Fatalf("TestSmbAPIGroup %v", err)
+	}
+	defer removeUser(t, username)
+
+	if err = setupSmbShare(smbShare, sharePath, username); err != nil {
+		t.Fatalf("TestSmbAPIGroup %v", err)
+	}
+	defer removeSmbShare(t, smbShare)
+
+	hostname, err := os.Hostname()
+	assert.Nil(t, err)
+
+	username = "domain\\" + username
+	remotePath := "\\\\" + hostname + "\\" + smbShare
+	// simulate Mount SMB operations around staging a volume on a node
+	mountSmbShareReq := &v1.NewSmbGlobalMappingRequest{
+		RemotePath: remotePath,
+		Username:   username,
+		Password:   password,
+	}
+	_, err = client.NewSmbGlobalMapping(context.Background(), mountSmbShareReq)
+	if err != nil {
+		t.Fatalf("TestSmbAPIGroup %v", err)
+	}
+
+	err = getSmbGlobalMapping(remotePath)
+	assert.Nil(t, err)
+
+	err = writeReadFile(remotePath)
+	assert.Nil(t, err)
+
+	unmountSmbShareReq := &v1.RemoveSmbGlobalMappingRequest{
+		RemotePath: remotePath,
+	}
+	_, err = client.RemoveSmbGlobalMapping(context.Background(), unmountSmbShareReq)
+	if err != nil {
+		t.Fatalf("TestSmbAPIGroup %v", err)
+	}
+	err = getSmbGlobalMapping(remotePath)
+	assert.NotNil(t, err)
+	err = writeReadFile(localPath)
+	assert.NotNil(t, err)
+
+}

integrationtests/smb_test.go

@@ -0,0 +1,3 @@
+package v1
+
+// Add manual conversion functions here to override automatic conversion functions