Moving from flyteplugins - updated maptask log links to use original index

[eapolinario]

TL;DR

Updated the maptask log links to use the subtask original index rather than the execution index.

NOTE: This change means that log links will not match the pod name in scenarios where the subtask execution and original indicies differ (ex. cache hits).

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

NA

Follow-up issue

NA

[codecov]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (d9586b0) 58.98% compared to head (f7db517) 59.37%.

❗ Current head f7db517 differs from pull request most recent head 950452f. Consider uploading reports for the commit 950452f to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4126      +/-   ##
==========================================
+ Coverage   58.98%   59.37%   +0.39%     
==========================================
  Files         619      516     -103     
  Lines       52804    37910   -14894     
==========================================
- Hits        31146    22510    -8636     
+ Misses      19174    13171    -6003     
+ Partials     2484     2229     -255     

Flag	Coverage Δ
unittests	?

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

Files	Coverage Δ
...lugins/go/tasks/plugins/array/awsbatch/executor.go	43.75% <100.00%> (+1.19%)	⬆️
...yteplugins/go/tasks/plugins/array/core/metadata.go	80.00% <100.00%> (-2.23%)	⬇️
...lyteplugins/go/tasks/plugins/array/k8s/executor.go	40.67% <100.00%> (+1.09%)	⬆️
...go/tasks/plugins/array/k8s/subtask_exec_context.go	83.33% <85.71%> (+1.51%)	⬆️

... and 555 files with indirect coverage changes

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