search

openflighthpc / flight-control

0 stars 0 forks source link

Updated azure data out check #66

Closed timalces closed 1 year ago

timalces commented 1 year ago

The azure costs API now has multiple possible meter names containing 'Data Transfer Out' (see https://alces.slack.com/archives/C01A4K27B37/p1656000576384299). It's unclear, but this may be part of the recent change to 'modern' subscriptions.

This PR therefore updates the data out check to check for any meter name containing Data Transfer Out, instead of an exact match. As data out is usually the least common cost type, logic is also updated so this is the last check when recording cost logs (to reduce the number of unnecessary checks).

timalces commented 1 year ago

Merging this, as appears to be as intended.