Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Several tests require CreateVolume capability #564

Open
joejulian opened this issue Nov 1, 2024 · 0 comments
Open

Several tests require CreateVolume capability #564

joejulian opened this issue Nov 1, 2024 · 0 comments

Comments

@joejulian
Copy link

joejulian commented Nov 1, 2024

I'm writing a CSI driver that does not have the CreateVolume capability. This test fails because it tries to CreateVolume before testing anything.

  [FAILED] in [It] - /home/jjulian/dev/go/src/github.com/kubernetes-csi/csi-test/pkg/sanity/controller.go:761 @ 11/01/24 16:39:00.443
  << Timeline

  [FAILED] volume create failed
  Unexpected error:
      <*status.Error | 0xc0005a00e8>: 
      rpc error: code = Unimplemented desc = CreateVolume is not implemented
      {
          s: {
              s: {
                  state: {
                      NoUnkeyedLiterals: {},
                      DoNotCompare: [],
                      DoNotCopy: [],
                      atomicMessageInfo: nil,
                  },
                  sizeCache: 0,
                  unknownFields: nil,
                  Code: 12,
                  Message: "CreateVolume is not implemented",
                  Details: nil,
              },
          },
      }
  occurred
  In [It] at: /home/jjulian/dev/go/src/github.com/kubernetes-csi/csi-test/pkg/sanity/controller.go:761 @ 11/01/24 16:39:00.443

The list is at least:
"should fail when no volume capabilities are provided",
"should return appropriate values",
"should fail when the node does not exist",
"should remove target path",
"should fail when no volume capability is provided",
"should be idempotent",
"should work",

There may be more, but I also have to skip all the "volume does not exist" because the volume may always exist in this driver, so I might not have a complete set.

@joejulian joejulian changed the title ValidateVolumeCapabilities [It] should fail when no volume capabilities are provided requires CreateVolume capability Several tests require CreateVolume capability Nov 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant