Sanity: ListVolumes "should fail when an invalid starting_token is passed" what is invalid?

[davidz627]

This test makes the assumption that one can distinguish between a valid and invalid token and it also makes the assumption that invalid-token is actually an invalid token.

AFAIK there are no universal constraints on what an invalid token actually looks like, for some backends maybe invalid-token is a totally ok token and is functionally indistinguisable from this-is-a-valid-token.

/cc @darkowlzz @pohly @lpabon @msau42

[msau42]

We can do a similar thing with the volume-id and have a method where plugins can define their own valid/invalid syntaxes.

/kind bug
/help

[k8s-ci-robot]

@msau42:
This request has been marked as needing help from a contributor.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-help command.

In response to this:

We can do a similar thing with the volume-id and have a method where plugins can define their own valid/invalid syntaxes.

/kind bug
/help

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[fejta-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

[davidz627]

/remove-lifecycle stale
still relevant

[fejta-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

[msau42]

/lifecycle frozen

[NicolasT]

I failed to see this issue before bringing this to the mailing list. In any case, for reference: https://groups.google.com/forum/#!topic/container-storage-interface-community/01fWLJEIuRQ