Show resolved semantic versions in the renderStatus

[aravindtga]

Summary

The renderStatus.mutationSteps[].image field was recording short/unresolved image names instead of fully qualified ones. This PR fixes that by moving tag resolution from ContainerFn.Run() to NewRunner(), so f.Image is fully resolved before being captured in RenderStatus.

Changes

• internal/fnruntime/runner.go - Move tag resolution to NewRunner()
• internal/fnruntime/container.go - Remove Tag field and resolution logic
• internal/util/render/executor.go - Minor cleanup in fnChain()

Testing

• E2E tests are updated

[netlify]

✅ Deploy Preview for kptdocs ready!

Name	Link
🔨 Latest commit	f826c53
🔍 Latest deploy log	https://app.netlify.com/projects/kptdocs/deploys/69ce7e8b804d5d00081d081a
😎 Deploy Preview	https://deploy-preview-4461--kptdocs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[Copilot]

Pull request overview

This PR ensures renderStatus.mutationSteps[].image records fully-qualified/resolved function image references (including semantic-version tag resolution) by performing image/tag resolution earlier during runner construction.

Changes:

• Resolve function image tags in fnruntime.NewRunner() so the resolved image is available when RenderStatus is captured.
• Remove Tag handling from ContainerFn since resolution no longer happens at execution time.
• Minor cleanup in render executor fnChain() to pass the slice element pointer to NewRunner().

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
internal/fnruntime/runner.go	Moves tag resolution into NewRunner() so resolved image is available for status capture.
internal/fnruntime/container.go	Removes Tag field and execution-time tag resolution from ContainerFn.
internal/util/render/executor.go	Updates fnChain() to pass &fns[i], allowing resolved image to propagate to step capture.
pkg/api/kptfile/v1/types.go	Formatting-only alignment in ResultItem struct fields.
pkg/test/runner/runner.go	Removes stray whitespace line in test runner.
e2e/testdata/fn-render/short-image-path/.expected/diff.patch	Updates golden output to expect fully-qualified images in RenderStatus.
e2e/testdata/fn-render/krm-check-exclude-kustomize/.expected/diff.patch	Updates golden output to expect fully-qualified images in RenderStatus.
e2e/testdata/fn-render/basicpipeline-semver/.expected/diff.patch	Updates golden output to expect resolved semantic-version tags in RenderStatus.

Comments suppressed due to low confidence (1)

internal/fnruntime/container.go:87

• Typo in comment: "timeour" should be "timeout".

	// Container function will be killed after this timeour.
	// The default value is 5 minutes.
	Timeout time.Duration

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 9 out of 10 changed files in this pull request and generated no new comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.