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

NAS-127298 / 25.04 / Add extensive iSCSI ALUA unit test #14538

Merged
merged 2 commits into from
Sep 19, 2024
Merged

Conversation

bmeagherix
Copy link
Contributor

Add a more extensive iSCSI ALUA test.

  • Test that an ALUA target - without a LUN 0 - works correctly with only LUN 100. The previous test_33_no_lun_zero tests purely from client perspective, but this new test (test_alua_lun_100) will also exercise the "internal" target used in ALUA.
  • Test a non-trivial ALUA configuration. This includes targets with no LUNs, one LUN, two LUNs & five LUNs - 10 targets, 16 LUNs in total. With this config ...
  • Use KVM virsh command to power off the current ACTIVE node (i.e. not a clean shutdown) and ensure that the previous STANDBY node serves the ALUA targets, as soon as failover is complete.
  • Reset the current ACTIVE, and repower the previous ACTIVE and ensure that ALUA targets are served by both nodes. (Clean simultaneous boot of both nodes.)
  • Finally, if necessary reboot the ACTIVE so that the original ACTIVE node is restored. (i.e. clean reboot)

Clean CI run (wrt iSCSI) here.

@bugclerk
Copy link
Contributor

@bmeagherix bmeagherix requested a review from a team September 18, 2024 17:44
@bmeagherix bmeagherix merged commit 18b20f7 into master Sep 19, 2024
3 checks passed
@bmeagherix bmeagherix deleted the NAS-127298 branch September 19, 2024 20:24
@bugclerk
Copy link
Contributor

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Sep 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants