refactor: remove agentID type from agentrm

[jesse-amano-hpe]

Description

DET-9976

Removes the agentID type in favor of using aproto.ID throughout package agentrm.

Test Plan

N/A

Commentary (optional)

A little surprising that there's a net gain of SLOC, but it's because of all the new import...aproto lines.

Also, this refactor touches a ton of untested code, but I promise it's purely a static change >.<

Checklist

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

[codecov]

Codecov Report

Attention: Patch coverage is 54.16667% with 11 lines in your changes are missing coverage. Please review.

Project coverage is 47.80%. Comparing base (e4bc377) to head (b35e765).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #9040      +/-   ##
==========================================
- Coverage   47.80%   47.80%   -0.01%     
==========================================
  Files        1161     1161              
  Lines      143646   143646              
  Branches     2373     2373              
==========================================
- Hits        68673    68672       -1     
- Misses      74820    74821       +1     
  Partials      153      153              

Flag	Coverage Δ
backend	42.94% <54.16%> (-0.01%)	⬇️
harness	63.84% <ø> (ø)
web	40.90% <ø> (ø)

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

Files	Coverage Δ
master/internal/rm/agentrm/agent_model.go	100.00% <ø> (ø)
master/internal/rm/agentrm/agent_state.go	83.84% <100.00%> (ø)
master/internal/rm/agentrm/agents.go	63.30% <100.00%> (ø)
master/internal/rm/agentrm/fair_share.go	95.85% <ø> (ø)
master/internal/rm/agentrm/fitting.go	95.68% <ø> (ø)
master/internal/rm/agentrm/priority.go	87.50% <100.00%> (ø)
master/internal/rm/agentrm/round_robin.go	66.66% <ø> (ø)
master/internal/rm/agentrm/summaries.go	60.86% <ø> (ø)
master/internal/rm/agentrm/resource_pool.go	34.13% <0.00%> (ø)
master/internal/rm/agentrm/resources.go	0.00% <0.00%> (ø)
... and 2 more

... and 1 file with indirect coverage changes

[netlify]

✅ Deploy Preview for determined-ui canceled.

Name	Link
🔨 Latest commit	b35e765
🔍 Latest deploy log	https://app.netlify.com/sites/determined-ui/deploys/65fd2045804edc0008d07d73

[amandavialva01]

This is a resource management ticket but was a pretty straight-forward refactor, so I reviewed it...LGTM!
Thanks for helping get rid of some unnecessary back-and-forth casting, and for removing the duplicate type.

[stoksc]

lgtm (didn't review a ton, if the compiler is happy i assume this is fine)