only read in inputs when rendering templates when input vars are set

[pvditt]

Why are the changes needed?

extra input read that's not needed

What changes were proposed in this pull request?

avoid un-needed io for reading in inputs when inputs aren't set in task commands/input templates

How was this patch tested?

has been running in Union clusters for a while

Labels

Please add one or more of the following labels to categorize your PR:

• added: For new features.
• changed: For changes in existing functionality.
• deprecated: For soon-to-be-removed features.
• removed: For features being removed.
• fixed: For any bug fixed.
• security: In case of vulnerabilities

This is important to improve the readability of release notes.

Setup process

Screenshots

Check all the applicable boxes

• [ ] I updated the documentation accordingly.
• [ ] All new and existing tests passed.
• [ ] All commits are signed-off.

Related PRs

Docs link

[flyte-bot]

Bito Automatic Review Skipped - Draft PR

Bito didn't auto-review because this pull request is in draft status.
No action is needed if you didn't intend for the agent to review it. Otherwise, to manually trigger a review, type /review in a comment and save.
You can change draft PR review settings here, or contact your Bito workspace admin at [email protected].

[codecov[bot]]

Codecov Report

:x: Patch coverage is 92.30769% with 1 line in your changes missing coverage. Please review. :white_check_mark: Project coverage is 59.71%. Comparing base (bca4499) to head (9318939).

Files with missing lines	Patch %	Lines
...go/tasks/pluginmachinery/core/template/template.go	92.30%	1 Missing :warning:

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #6755      +/-   ##
==========================================
- Coverage   59.71%   59.71%   -0.01%     
==========================================
  Files         929      929              
  Lines       58011    58012       +1     
==========================================
  Hits        34642    34642              
- Misses      20214    20215       +1     
  Partials     3155     3155              

Flag	Coverage Δ
unittests-datacatalog	60.30% <ø> (ø)
unittests-flyteadmin	57.86% <ø> (ø)
unittests-flytecopilot	43.16% <ø> (ø)
unittests-flytectl	65.31% <ø> (ø)
unittests-flyteidl	78.64% <ø> (ø)
unittests-flyteplugins	62.05% <92.30%> (+<0.01%)	:arrow_up:
unittests-flytepropeller	55.55% <ø> (ø)
unittests-flytestdlib	64.00% <ø> (-0.03%)	:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:

• :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.