fix: allocation csv: gpu_hours -> slot_hours, add resource_pool [DET-10408]

[jesse-amano-hpe]

Waiting on acceptance or response to https://hpe-aiatscale.atlassian.net/browse/DET-10408?focusedCommentId=32936

Ticket

DET-10408

Description

Renames the gpu_hours column in the allocation report CSV to slot_hours as this is a more accurate description. Adds a resource_pool column to help users identify what slot type(s) these slot-hours apply to.

Test Plan

Run any number of experiments >0 on a cluster that has at least one slot configured.
On cluster historical usage tab, make sure the end date is current, and click Download CSV.

Choose to group by Allocations and download the CSV

Observe slot_hours and resource_pool columns are present in the download

Checklist

• Changes have been manually QA'd
• New features have been approved by the corresponding PM
• User-facing API changes have the "User-facing API Change" label
• Release notes have been added as a separate file under docs/release-notes/
  See Release Note for details.
• Licenses have been included for new code which was copied and/or modified from any external code

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 8 lines in your changes missing coverage. Please review.

Project coverage is 52.88%. Comparing base (000c679) to head (06f0f61).
Report is 24 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #9616      +/-   ##
==========================================
- Coverage   52.88%   52.88%   -0.01%     
==========================================
  Files        1255     1255              
  Lines      153086   153090       +4     
  Branches     3230     3230              
==========================================
- Hits        80965    80964       -1     
- Misses      71970    71975       +5     
  Partials      151      151              

Flag	Coverage Δ
backend	44.00% <0.00%> (-0.01%)	⬇️
harness	72.76% <ø> (ø)
web	51.30% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
master/internal/core.go	6.48% <0.00%> (-0.04%)	⬇️

... and 3 files with indirect coverage changes

[netlify]

✅ Deploy Preview for determined-ui canceled.

Name	Link
🔨 Latest commit	06f0f61
🔍 Latest deploy log	https://app.netlify.com/sites/determined-ui/deploys/669172ab9825630007fbfdac

[hamidzr]

CE/SE question would be if this could break downstream integrations on that column name but since it's not accurate that's okay? alternative would be to make this a pure addition and avoid removing the "gpu_hours" column in the same release.

[jesse-amano-hpe]

CE/SE question would be if this could break downstream integrations on that column name but since it's not accurate that's okay? alternative would be to make this a pure addition and avoid removing the "gpu_hours" column in the same release.

We discussed on Slack with @hkumar92 ; I'll add mention of this to release notes but no further mitigation should be needed.