chore: undo default log retention in values.yaml

[salonig23]

Ticket

Description

Test Plan

make sure tests pass

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.

[netlify]

✅ Deploy Preview for determined-ui canceled.

Name	Link
🔨 Latest commit	1780c6e
🔍 Latest deploy log	https://app.netlify.com/sites/determined-ui/deploys/662aaaccb8e1ab0008390e5f

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 44.72%. Comparing base (4d20fae) to head (1780c6e).
Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #9245   +/-   ##
=======================================
  Coverage   44.71%   44.72%           
=======================================
  Files        1270     1270           
  Lines      155328   155328           
  Branches     2436     2436           
=======================================
+ Hits        69461    69467    +6     
+ Misses      85631    85625    -6     
  Partials      236      236           

Flag	Coverage Δ
backend	41.66% <ø> (+0.01%)	⬆️
harness	64.28% <ø> (ø)
web	35.22% <ø> (ø)

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

see 2 files with indirect coverage changes

[NicholasBlaskey]

LGTM

Personally I would make this a chore: so we don't need to test this in the release party