Azure / avdaccelerator

AVD Accelerator deployment automation to simplify the setup of AVD (Azure Virtual Desktop) based on best practices
MIT License
305 stars 198 forks source link

JCore brownfield alerts - fix query for AVD Agent Health #586

Closed JCoreMS closed 5 months ago

JCoreMS commented 5 months ago

Overview/Summary

This fixes a noted issue with the alert "AVD-HP-VM-HlthChkFailure-(hostpoolname)" in which the Session Host Name was not populating due to the query having an extra space in the parse line for the Resource Id.

Noted space is after the quotes before '/providers' and removed for both the BICEP and ARM template.

| parse ....... SessionHostRG " /providers/Microsoft.Compute/virtualMachines/" SessionHostName vs | parse ....... SessionHostRG "/providers/Microsoft.Compute/virtualMachines/" SessionHostName

This PR fixes/adds/changes/removes

  1. Query for the AVD-HP-VM-HlthChkFailure alert to ensure the Session Host Name is returned in the results
  2. Adjusted threshold for Azure Files availability from 99 to 99.99 due to degradation noted prior to hitting the threshold which is too low. image

Breaking Changes

  1. N/A

Testing Evidence

Tested query in Log Analytics to verify issue was resolved and was noted while reviewing for another customer.

As part of this Pull Request I have

danycontre commented 5 months ago

@JCoreMS thanks for your contribution