Elegant way to check gitdiff files on PR for arm-estimator?

[riosengineer]

Hey,

Wondering if anyone has gone through the pain of figuring out a clean and sleek way to only run the tooling on commits present in a PR within Azure DevOps?

Would be awesome to run an estimation of costs for any files edited on PR, and even post the table in the ADO comments.

For now, it looks like I will need to create a ton of custom logic to get this done using gitdiff, and then figure out the paired param file based on repo structure.

[kamil-mrzyglod]

Hey @riosengineer, do you need to run estimation for all the Bicep / ARM Templates files during PR, or only for the ones, which have changed? I don't know how the structure of your files looks like, but ACE is intended to run estimations against selected scope (resource group, subscription, etc.). In other words - it produces both total estimation and diff as long as ARM is able to provide what has changed. If this is your case, then it means, you don't need git diff - you will always run estimation for all the files and get the cost difference.

If the cost difference is not reported, then it's either ARM not returning the proper value, or there's a bug in ACE :)

[kamil-mrzyglod]

Sure, if you find a smart way to do that, happy to discuss that further :)

[riosengineer]

Getting somewhere :) On PR, gitdiff to find all changes bicep files and their corresponding param to run ACE with the output.

Posts the markdown output, formatting is a bit off, I am not sure if ADO comments expect certain markdown formatting. The markdown looks fine in PS terminal though. Will keep playing!

[kamil-mrzyglod]

Not sure if ADO accepts markdown as format for the comments. The markdown may be accepted only as input when writing a comment and transformed to HTML on the fly. Just a blind shot :)

[riosengineer]

I ended up using PowerShell Get-Content -Path $markdownFile -Raw and it looks pretty good I think!

[kamil-mrzyglod]

Superb! Feel free to request changes to the output if you feel something's missing.