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

PWX-36522: In case of devices use Exact match with mount and not a pr… #2431

Merged
merged 1 commit into from
Mar 21, 2024

Conversation

pnookala-px
Copy link
Contributor

@pnookala-px pnookala-px commented Mar 20, 2024

…efix match

PWX-36522: When the devices are of the format /dev/dm- and we do a prefix search, the search will lead incorrect match for ./dev/dm-. Instead when searching for Target Devices, check for exact match.
Testing:
On the system that had devices in the specified format installed binary and verified that filesystems/devices are rightly identified and pods come up properly.
Tested on a Aetos setup with a mix of RWO/RWX/FADA pods and kept restarting px randomly on all nodes. Pods came up cleanly.

Copy link
Contributor

@Pure-AdamuKaapan Pure-AdamuKaapan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, pending testing notes obviously

@pnookala-px pnookala-px merged commit 212d149 into master Mar 21, 2024
6 checks passed
pnookala-px added a commit that referenced this pull request Mar 21, 2024
pnookala-px added a commit that referenced this pull request Mar 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants