Fix vSAN type and disk count for Hybrid clusters - Githubissues

tpcarman / As-Built-Report

A collection of PowerShell scripts to generate as built reports on the configuration of datacentre infrastucture in Text, XML, HTML & MS Word formats.

MIT License

88 stars 28 forks source link

Fix vSAN type and disk count for Hybrid clusters #64

Closed danbarr closed 5 years ago

danbarr commented 5 years ago

Description

Corrects the vSAN Total Disk Count, which previously was only counting SSDs
Correct vSAN type detection; would return All-Flash for a Hybrid cluster

Related Issue

Fixes #63

How Has This Been Tested?

Tested with a Hybrid and All-Flash vSAN lab cluster to verify the report is showing the correct type and disk count.

Screenshots:

Correct information from a Hybrid vSAN: correct-hyb

Correct information from an All-Flash vSAN: correct-af

Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.