chore: default priority earlier to ensure constraints are satisfied [CM-553]

[stoksc]

Ticket

CM-553

Description

If we set priorities earlier during launch, then the constraint checks will apply to the default, too. This prevents jobs getting through when a limit conflicts with a default, which is strange but could be used to effectively force users to submit with a priority.

Test Plan

TBD.

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

[netlify]

✅ Deploy Preview for determined-ui canceled.

Name	Link
🔨 Latest commit	5e73489
🔍 Latest deploy log	https://app.netlify.com/sites/determined-ui/deploys/670d22b09c29ea0008e939d7

[codecov]

Codecov Report

Attention: Patch coverage is 46.15385% with 7 lines in your changes missing coverage. Please review.

Project coverage is 54.61%. Comparing base (2356f91) to head (5e73489).
Report is 3 commits behind head on main.

Files with missing lines	Patch %	Lines
master/internal/config/config.go	0.00%	4 Missing ⚠️
master/internal/api_generic_tasks.go	0.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10043      +/-   ##
==========================================
- Coverage   54.62%   54.61%   -0.02%     
==========================================
  Files        1260     1260              
  Lines      157621   157631      +10     
  Branches     3632     3631       -1     
==========================================
- Hits        86102    86083      -19     
- Misses      71385    71414      +29     
  Partials      134      134              

Flag	Coverage Δ
backend	45.38% <46.15%> (-0.05%)	⬇️
harness	72.74% <ø> (ø)
web	54.37% <ø> (ø)

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

Files with missing lines	Coverage Δ
master/internal/api_command.go	48.61% <100.00%> (+0.72%)	⬆️
master/internal/core_experiment.go	62.34% <100.00%> (+0.47%)	⬆️
master/internal/api_generic_tasks.go	8.88% <0.00%> (-0.06%)	⬇️
master/internal/config/config.go	58.63% <0.00%> (-0.20%)	⬇️

... and 9 files with indirect coverage changes

[kkunapuli]

Nice work!

[amandavialva01]

Love this helper!

[amandavialva01]

LGTM, awesome work!