Skip to content

Commit 207c817

Browse files
helderpintoHélder Pinto
andauthored
[AOE] Fixes underutilized Premium disk recommendations in customers with Premium SSD V2 disks (#1832)
Co-authored-by: Hélder Pinto <[email protected]>
1 parent 5e97a9f commit 207c817

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

docs-mslearn/toolkit/changelog.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,11 @@ The following section lists features and enhancements that are currently in deve
2929
- **Added**
3030
- Document [how to remove private networking](hubs/private-networking.md#removing-private-networking) and switch back to public access to reduce costs ([#1342](https://github.com/microsoft/finops-toolkit/issues/1342)).
3131

32+
### [Optimization engine](optimization-engine/overview.md)
33+
34+
- **Fixed**
35+
- Underutilized disks recommendations were not being generated when customer environment has Premium SSD V2 disks ([#1831](https://github.com/microsoft/finops-toolkit/issues/1831)).
36+
3237
### Bicep Registry module pending updates
3338

3439
- Cost Management export modules for subscriptions and resource groups.

src/optimization-engine/runbooks/recommendations/Recommend-DiskOptimizationsToBlobStorage.ps1

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,7 @@ $baseQuery = @"
224224
| summarize MaxIOPSMetric = max(todouble(MetricValue_s)) by ResourceId
225225
| join kind=inner (
226226
$disksTableName
227-
| where TimeGenerated > ago(1d) and DiskState_s =~ 'Attached' and SKU_s startswith 'Premium'
227+
| where TimeGenerated > ago(1d) and DiskState_s =~ 'Attached' and SKU_s startswith 'Premium' and SKU_s !contains "V2"
228228
| extend DiskTier_s = strcat(DiskTier_s, ' ', tostring(split(SKU_s, '_')[1]))
229229
| project ResourceId=InstanceId_s, DiskName_s, ResourceGroup = ResourceGroupName_s, SubscriptionId = SubscriptionGuid_g, Cloud_s, TenantGuid_g, Tags_s, MaxIOPSDisk=toint(DiskIOPS_s), DiskSizeGB_s, SKU_s, DiskTier_s, DiskType_s
230230
) on ResourceId
@@ -237,7 +237,7 @@ $baseQuery = @"
237237
| summarize MaxMBsMetric = max(todouble(MetricValue_s)/1024/1024) by ResourceId
238238
| join kind=inner (
239239
$disksTableName
240-
| where TimeGenerated > ago(1d) and DiskState_s =~ 'Attached' and SKU_s startswith 'Premium'
240+
| where TimeGenerated > ago(1d) and DiskState_s =~ 'Attached' and SKU_s startswith 'Premium' and SKU_s !contains "V2"
241241
| extend DiskTier_s = strcat(DiskTier_s, ' ', tostring(split(SKU_s, '_')[1]))
242242
| project ResourceId=InstanceId_s, DiskName_s, ResourceGroup = ResourceGroupName_s, SubscriptionId = SubscriptionGuid_g, Cloud_s, TenantGuid_g, Tags_s, MaxMBsDisk=toint(DiskThroughput_s), DiskSizeGB_s, SKU_s, DiskTier_s, DiskType_s
243243
) on ResourceId

0 commit comments

Comments
 (0)