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

fine tune logic to filter disks #857

Merged
merged 1 commit into from
Oct 9, 2024

Conversation

ibrokethecloud
Copy link
Contributor

Problem:

Logic disks managed by the raid controller will have the same serial number. This results in disks being deduplicated and ignored.

Solution:

From the sample data available the wwn is unique despite the serial number being the same.

The PR adds a minor tweak to continue comparing disks even if serial number is same and check for wwn uniqueness.

Related Issue:
harvester/harvester#6694
Test plan:

Copy link
Contributor

@tserong tserong left a comment

Choose a reason for hiding this comment

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

This hurts my brain a little bit, but LGTM :-)

Copy link
Contributor

@Vicente-Cheng Vicente-Cheng left a comment

Choose a reason for hiding this comment

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

lgtm, but I am curious about the actual case.
Could we have the udev info for devices with the same serial but a different WWN?

@ibrokethecloud
Copy link
Contributor Author

seems to be easily reproducible when your disks are managed by a raid controller, and subsequent luns made available to the host. In this scenario as we have seen from our own testing the disk serial numbers are the same but wwn is different.

@Vicente-Cheng
Copy link
Contributor

seems to be easily reproducible when your disks are managed by a raid controller, and subsequent luns made available to the host. In this scenario as we have seen from our own testing the disk serial numbers are the same but wwn is different.

Sure, I thought I could ask for this information with @irishgordo.
Just curious. I did not notice that I had used a similar raid card before.

Feel free to merge it : ))

@ibrokethecloud ibrokethecloud merged commit b6a5418 into harvester:master Oct 9, 2024
7 checks passed
@ibrokethecloud
Copy link
Contributor Author

@Mergifyio backport v1.4

Copy link

mergify bot commented Oct 9, 2024

backport v1.4

✅ Backports have been created

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.

3 participants