There are no unittests for target discovery. I had a long discussion
with Rob about this test gap and offered a solution that Harold
suggested a few months ago as a possible idea. We'll have to follow up
with Rob later once he's had a chance to investigate it.
-Drew
On 4/11/12 8:28 PM, Martin Widjaja wrote:
Jesse - Thanks for taking this up.
One question: any additional unit test or test case we need to
introduce to ensure this works fine? If it's a test case, might it be
worth mentioning it to QE so they can include this in their list?
Other than that, LGTM!
BTW, for the other bug, 7079889, I was going to remove the potential
stopper status today but didn't get around to it. I will do this
tomorrow and keep you and Son in the loop.
Thanks,
Martin
On 4/11/2012 7:02 PM, Drew Fisher wrote:
LGTM!
Thanks for working on the fix to this!
-Drew
On 4/11/12 7:58 PM, Jesse Butler wrote:
Could I get a review for:
7150578 install discovery ignores device path lun numbers
This CR is a small piece of the overall multipath target selection
issue that the discovery code has. When we encounter multipath
targets with MPxIO disabled, as is the default on SPARC, everything
kind of falls apart. This is being worked separately.
During that work, this bug was discovered. For each drive we
discover, we retrieve a WWN. If the WWN name is found, we use it. We
throw out subsequent drives with the same WWN.
This is a problem in a few scenarios, most commonly when a RAID
array is used. Most RAID arrays use the WWN of the HBA and a LUN
number to identify a LUN. So, the current code will only discover
one LUN per WWN, throwing away all the other LUNs using the same
WWN. This could be the entire array on some configurations. Further,
the ones that are put on the list are missing their LUN number.
This changeset tacks the LUN number onto the end of the WWN string
if it's in use, thus getting around the bad duplication avoidance,
and setting a full WWN+LUN descriptor for each Disk object.
https://cr.opensolaris.org/action/browse/caiman/jesseb/7150578/webrev/
Thanks
Jesse
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
