#400 Fix resources requests for AWS Batch plugin
Pull request opened by
hamersaw
TL;DR
This PR uses the container resources as the base before merging with resource overrides and platform defaults. The logic involved in
this PR updated always setting the resource overrides even if there were none (if missing it would set overrides to the container resources), which introduced this regression.
Type
☑︎ Bug Fix
☐ Feature
☐ Plugin
Are all requirements met?
☑︎ Code completed
☑︎ Smoke tested
☐ Unit tests added
☑︎ Code documentation added
☐ Any pending items have an associated Issue
Complete description
^^^
Tracking Issue
flyteorg/flyte#4003
Follow-up issue
NA
flyteorg/flyteplugins
Codecov: 62.83% (-0.01%) compared to 552f145
✅ 6 other checks have passed
6/7 successful checks