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.

Sign up for GitHub

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

Jump to bottom

Fix CSI volume list with prefix and * namespace #12184

Merged
merged 4 commits into from
Mar 3, 2022
Merged

Fix CSI volume list with prefix and * namespace #12184

merged 4 commits into from
Mar 3, 2022

Commits on Mar 3, 2022

  1. add falining test

    @lgfa29
    lgfa29 committed Mar 3, 2022
    Configuration menu
    Copy the full SHA
    d725970 View commit details
    Browse the repository at this point in the history

  2. api: fix Volume list endpoint with prefix and * ns 

    When using a prefix value and the * wildcard for namespace, the endpoint
would not take the prefix value into consideration due to the order in
which the checks were executed but also the logic for retrieving volumes
from the state store.

This commit changes the order to check for a prefix first and wraps the
result iterator of the state store query in a filter to apply the
prefix.
    @lgfa29
    lgfa29 committed Mar 3, 2022
    Configuration menu
    Copy the full SHA
    dfda28a View commit details
    Browse the repository at this point in the history

  3. changelog: add entry for #12184

    @lgfa29
    lgfa29 committed Mar 3, 2022
    Configuration menu
    Copy the full SHA
    e759063 View commit details
    Browse the repository at this point in the history

  4. fix comment description of CSIVolumesByIDPrefix

    @lgfa29
    lgfa29 committed Mar 3, 2022
    Configuration menu
    Copy the full SHA
    f7bd72b View commit details
    Browse the repository at this point in the history