[BUG] Normal container task resource should be not be counted as override resource on the node

[ByronHsu]

Describe the bug

Note: This bug does not affect the behavior of Flyte, but the logic contains a flaw.

In BuildRawContainer, we do not inject the container resource from the task template. Instead, we inject the resource from parameters.TaskExecMetadata.GetOverrides().GetResources().

This is because for both _with_override(resource=... and @task(requests=...), they are injected as the node resource override in buildNodeSpec

Expected behavior

For the case of @task(requests=...), the resource override on the node should be empty, and the resource is injected from tasktemplate at BuildRawContainer

Additional context to reproduce

No response

Screenshots

No response

Are you sure this issue hasn't been raised already?

• Yes

Have you read the Code of Conduct?

• Yes