feat: Read log signal from DB

[gt2345]

Ticket

MD-495 MD-496

Description

Updates the /api/v1/runs and /api/v1/trials/{id} to return the log signal.
Display the log signal at run details page

Display of the log signal at runs table.

This PR uses the same DB migration as the one from this PR

Test Plan

Update DB, set tasks -> log_signal to values like CUDA OOM, at the run details page, there should be a badge that displays the log signal.

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 ready!

Name	Link
🔨 Latest commit	cb49904
🔍 Latest deploy log	https://app.netlify.com/sites/determined-ui/deploys/66f4466067388e0008cae771
😎 Deploy Preview	https://deploy-preview-9959--determined-ui.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[codecov]

Codecov Report

Attention: Patch coverage is 47.82609% with 12 lines in your changes missing coverage. Please review.

Project coverage is 54.53%. Comparing base (bc3b2a6) to head (cb49904).
Report is 4 commits behind head on main.

Files with missing lines	Patch %	Lines
.../src/pages/TrialDetails/Header/TrialHeaderLeft.tsx	41.17%	10 Missing ⚠️
webui/react/src/pages/FlatRuns/columns.ts	0.00%	1 Missing ⚠️
webui/react/src/services/decoder.ts	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #9959      +/-   ##
==========================================
- Coverage   54.53%   54.53%   -0.01%     
==========================================
  Files        1257     1257              
  Lines      156933   156956      +23     
  Branches     3614     3614              
==========================================
+ Hits        85587    85593       +6     
- Misses      71213    71230      +17     
  Partials      133      133              

Flag	Coverage Δ
backend	45.15% <100.00%> (-0.01%)	⬇️
harness	72.74% <ø> (ø)
web	54.34% <42.85%> (-0.01%)	⬇️

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

Files with missing lines	Coverage Δ
master/internal/api_experiment.go	56.82% <100.00%> (+0.02%)	⬆️
master/internal/api_runs.go	66.22% <100.00%> (+0.04%)	⬆️
webui/react/src/types.ts	99.70% <100.00%> (+<0.01%)	⬆️
webui/react/src/pages/FlatRuns/columns.ts	18.60% <0.00%> (-0.04%)	⬇️
webui/react/src/services/decoder.ts	21.34% <0.00%> (-0.03%)	⬇️
.../src/pages/TrialDetails/Header/TrialHeaderLeft.tsx	80.39% <41.17%> (-19.61%)	⬇️

... and 5 files with indirect coverage changes

[salonig23]

Is it possible to add some test cases? Also could you check the performance dashboard to make sure that sql script changes do not cause a regression before merging? Everything else looks great!

[jgongd]

It works as expected. Nice work!

[gt2345]

Is it possible to add some test cases? Also could you check the performance dashboard to make sure that sql script changes do not cause a regression before merging? Everything else looks great!

@salonig23 Thanks for your feedbacks! As discussed offline, we plan to add test cases to the "write" side of log signal. Also I do see a increase of query time at the perf dashboard, but I doubt that's related to the changes from this PR..

[keita-determined]

added few comments, otherwise lgtm

[salonig23]

LGTM! Great work and thanks for looking into the perf dashboard