Add divide by zero safety

Azure · Nov 7, 2024 · effc6eb · effc6eb
1 parent 38fc808
commit effc6eb
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/eng/common/pipelines/templates/steps/emit-rate-limit-metrics.yml b/eng/common/pipelines/templates/steps/emit-rate-limit-metrics.yml
@@ -16,16 +16,19 @@ steps:
     foreach ($property in $response.resources.PSObject.Properties)
     { 
         $labels = @{ user= $env:GITHUB_USER; resource= $property.Name }
-        
+
         $remaining = $property.Value.remaining
         $limit = $property.Value.limit
         $used = $property.Value.used
-        $percent = $used / $limit * 100
 
         Write-Host "logmetric: $( [ordered]@{ name= "github_ratelimit_remaining_total"; value= $remaining; timestamp= $timestamp; labels= $labels } | ConvertTo-Json -Compress)"
         Write-Host "logmetric: $( [ordered]@{ name= "github_ratelimit_limit_total"; value= $limit; timestamp= $timestamp; labels= $labels } | ConvertTo-Json -Compress)"
         Write-Host "logmetric: $( [ordered]@{ name= "github_ratelimit_used_total"; value= $used; timestamp= $timestamp; labels= $labels } | ConvertTo-Json -Compress)"
-        Write-Host "logmetric: $( [ordered]@{ name= "github_ratelimit_used_percent"; value= $percent; timestamp= $timestamp; labels= $labels } | ConvertTo-Json -Compress)"
+
+        if ($limit -ne 0) {
+          $percent = $used / $limit * 100
+          Write-Host "logmetric: $( [ordered]@{ name= "github_ratelimit_used_percent"; value= $percent; timestamp= $timestamp; labels= $labels } | ConvertTo-Json -Compress)"
+        }
     }          
   displayName: Check GitHub Rate Limit
   env: