Enable caching in flyte-core helmchart

[cpaulik]

Why are the changes needed?

Without this caching was actually not enabled when deploying with the flyte-core helm chart

What changes were proposed in this pull request?

Adding storage to the cache.yaml file so that the configuration values for storage.cache.max_size_mbs and storage.cache.target_gc_percet are picked up correctly

How was this patch tested?

I've deployed flyte core without this change and caching was not enabled. After I added

configmap:
  core:
    storage:
      cache:
        max_size_mbs: 5000
        target_gc_percent: 70

caching started working so I assume that adding storage in the cache.yaml would also work.

[welcome]

Thank you for opening this pull request! 🙌

These tips will help get your PR across the finish line:

• Most of the repos have a PR template; if not, fill it out to the best of your knowledge.
• Sign off your commits (Reference: DCO Guide).

[cpaulik]

For DCO I'm waiting for our legal department so this might take a bit for the first contribution.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (master@312910d). Learn more about missing BASE report.
Report is 4 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##             master    #5739   +/-   ##
=========================================
  Coverage          ?   36.29%           
=========================================
  Files             ?     1305           
  Lines             ?   109997           
  Branches          ?        0           
=========================================
  Hits              ?    39927           
  Misses            ?    65914           
  Partials          ?     4156           

Flag	Coverage Δ
unittests-datacatalog	51.37% <ø> (?)
unittests-flyteadmin	55.62% <ø> (?)
unittests-flytecopilot	12.17% <ø> (?)
unittests-flytectl	62.21% <ø> (?)
unittests-flyteidl	7.12% <ø> (?)
unittests-flyteplugins	53.35% <ø> (?)
unittests-flytepropeller	41.88% <ø> (?)
unittests-flytestdlib	55.21% <ø> (?)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[cpaulik]

DCO should be fixed

[pingsutw]

LGTM, thank you for fixing it

[davidmirror-ops]

@cpaulik Thank so much! Also to fix the problem detected by generate_helm, you should be able to run make helm in your fork and push the changes

[cpaulik]

I ran make helm and updated the commit

[eapolinario]

Thank you.

[welcome]

Congrats on merging your first pull request! 🎉