microsoft / finops-toolkit

Tools and resources to help you adopt and implement FinOps capabilities that automate and extend the Microsoft Cloud.
https://aka.ms/finops/toolkit
MIT License
305 stars 106 forks source link

[Power BI] Eligible for AHB Windows Servers are Not Detected if licensetype is null or if the SQL server doesn't list Windows #938

Closed da5is closed 1 month ago

da5is commented 2 months ago

🐛 Problem

In cases where a Windows Virtual Machine is deployed and licenseType is set null or Windows SQL servers indicate SQL but not Windows (such as potential ws2012 deployments), the AHB report does not show these VMs as eligible but not covered by AHB.

_Verbatim customer comment: " I can see we are being charged for Windows Server Licenses, but they don’t have the License_Type = WindowsServer. I believe this is happening for two reasons, either we didn’t enter any “Source image details” or the details indicate “SQL” but the OS is “Windows”."

👣 Repro steps

Deploy VMs uncovered by AHB that meet these criteria - they will not show up as potential AHB candidates in the workbook.

ℹ️ Additional context

I believe this is related to lines 125 and 159 where we're checking for Windows in licenseType explicitly, which doesn't detect instances where licenseType is null or Windows isn't the explicit string (such as ws2012).

flanakin commented 2 months ago

Added this to the list for the September release for investigation.

@BenShy, if you have any context, that might help us determine the right fix.

flanakin commented 2 months ago

@allcontributors Please add da5is for bug

allcontributors[bot] commented 2 months ago

@flanakin

I've put up a pull request to add @da5is! :tada:

arthurclares commented 2 months ago

@flanakin This is for the workbook. Seif and I have been in touch with @da5is already.

arthurclares commented 2 months ago

This is fixed and will be shipped with the next release (0.6)