Show resolved semantic versions in the renderStatus

Signed-off-by: Aravindhan Ayyanathan <aravindhan.a@est.tech>

Author: aravindtga

e2e/testdata/fn-render/basicpipeline-semver/.expected/diff.patch

@@ -1,5 +1,5 @@
 diff --git a/Kptfile b/Kptfile
-index 2336da4..c1090e8 100644
+index 2336da4..ca2bcea 100644
 --- a/Kptfile
 +++ b/Kptfile
 @@ -2,13 +2,34 @@ apiVersion: kpt.dev/v1
@@ -27,14 +27,14 @@ index 2336da4..c1090e8 100644
 +      reason: RenderSuccess
 +  renderStatus:
 +    mutationSteps:
-+      - image: ghcr.io/kptdev/krm-functions-catalog/set-namespace
++      - image: ghcr.io/kptdev/krm-functions-catalog/set-namespace:v0.4.3
 +        exitCode: 0
 +        results:
 +          - message: namespace [default] updated to "staging", 1 value(s) changed
 +            severity: info
 +          - message: all `depends-on` annotations are up-to-date. no `namespace` changed
 +            severity: info
-+      - image: ghcr.io/kptdev/krm-functions-catalog/set-labels
++      - image: ghcr.io/kptdev/krm-functions-catalog/set-labels:v0.2.4
 +        exitCode: 0
 +        results:
 +          - message: set 4 labels in total

e2e/testdata/fn-render/krm-check-exclude-kustomize/.expected/diff.patch

@@ -1,5 +1,5 @@
 diff --git a/Kptfile b/Kptfile
-index 2985a1a..1cc880e 100644
+index 2985a1a..30b4376 100644
 --- a/Kptfile
 +++ b/Kptfile
 @@ -2,8 +2,19 @@ apiVersion: kpt.dev/v1
@@ -20,7 +20,7 @@ index 2985a1a..1cc880e 100644
 +      reason: RenderSuccess
 +  renderStatus:
 +    mutationSteps:
-+      - image: set-labels:v0.1.5
++      - image: ghcr.io/kptdev/krm-functions-catalog/set-labels:v0.1.5
 +        exitCode: 0
 diff --git a/kustomization.yaml b/kustomization.yaml
 index f3f0207..6c517af 100644

e2e/testdata/fn-render/short-image-path/.expected/diff.patch

@@ -1,5 +1,5 @@
 diff --git a/Kptfile b/Kptfile
-index d4e5935..0759cb0 100644
+index d4e5935..24022da 100644
 --- a/Kptfile
 +++ b/Kptfile
 @@ -2,6 +2,9 @@ apiVersion: kpt.dev/v1
@@ -23,9 +23,9 @@ index d4e5935..0759cb0 100644
 +      reason: RenderSuccess
 +  renderStatus:
 +    mutationSteps:
-+      - image: set-namespace:v0.2.0
++      - image: ghcr.io/kptdev/krm-functions-catalog/set-namespace:v0.2.0
 +        exitCode: 0
-+      - image: set-labels:v0.1.5
++      - image: ghcr.io/kptdev/krm-functions-catalog/set-labels:v0.1.5
 +        exitCode: 0
 diff --git a/resources.yaml b/resources.yaml
 index f2eec52..84cfb26 100644

internal/fnruntime/container.go

@@ -80,7 +80,6 @@ type ContainerFn struct {
 
 	// Image is the container image to run
 	Image string
-	Tag   string
 	// ImagePullPolicy controls the image pulling behavior.
 	ImagePullPolicy runneroptions.ImagePullPolicy
 	// Container function will be killed after this timeour.
@@ -152,21 +151,6 @@ func (f *ContainerFn) Run(reader io.Reader, writer io.Writer) error {
 		return err
 	}
 
-	if f.Tag != "" {
-		tagResolver := &TagResolver{
-			lister: &RegClientLister{
-				client: regclient.New(
-					regclient.WithUserAgent(UserAgent),
-					regclient.WithDockerCreds(),
-				),
-			},
-		}
-		f.Image, err = tagResolver.ResolveFunctionImage(f.Ctx, f.Image, f.Tag)
-		if err != nil {
-			return err
-		}
-	}
-
 	switch runtime {
 	case Podman:
 		return f.runCLI(reader, writer, podmanBin, filterPodmanCLIOutput)

internal/fnruntime/runner.go

@@ -35,6 +35,7 @@ import (
 	"github.com/kptdev/kpt/pkg/lib/errors"
 	"github.com/kptdev/kpt/pkg/lib/runneroptions"
 	"github.com/kptdev/kpt/pkg/printer"
+	"github.com/regclient/regclient"
 	"sigs.k8s.io/kustomize/kyaml/filesys"
 	"sigs.k8s.io/kustomize/kyaml/fn/framework"
 	"sigs.k8s.io/kustomize/kyaml/fn/runtime/runtimeutil"
@@ -63,6 +64,21 @@ func NewRunner(
 			return nil, err
 		}
 		f.Image = img
+
+		if f.Tag != "" {
+			tagResolver := &TagResolver{
+				lister: &RegClientLister{
+					client: regclient.New(
+						regclient.WithUserAgent(UserAgent),
+						regclient.WithDockerCreds(),
+					),
+				},
+			}
+			f.Image, err = tagResolver.ResolveFunctionImage(ctx, f.Image, f.Tag)
+			if err != nil {
+				return nil, err
+			}
+		}
 	}
 
 	fnResult := &fnresult.Result{
@@ -106,7 +122,6 @@ func NewRunner(
 				} else {
 					cfn := &ContainerFn{
 						Image:           f.Image,
-						Tag:             f.Tag,
 						ImagePullPolicy: opts.ImagePullPolicy,
 						Perm: ContainerFnPermission{
 							AllowNetwork: opts.AllowNetwork,

internal/util/render/executor.go

@@ -982,18 +982,17 @@ func fnChain(ctx context.Context, hctx *hydrationContext, pkgPath types.UniquePa
 	for i := range fns {
 		var err error
 		var runner *fnruntime.FunctionRunner
-		function := fns[i]
 		displayResourceCount := false
-		if len(function.Selectors) > 0 || len(function.Exclusions) > 0 {
+		if len(fns[i].Selectors) > 0 || len(fns[i].Exclusions) > 0 {
 			displayResourceCount = true
 		}
-		if function.Exec != "" && !hctx.runnerOptions.AllowExec {
+		if fns[i].Exec != "" && !hctx.runnerOptions.AllowExec {
 			return nil, i, errAllowedExecNotSpecified
 		}
 		opts := hctx.runnerOptions
 		opts.SetPkgPathAnnotation = true
 		opts.DisplayResourceCount = displayResourceCount
-		runner, err = fnruntime.NewRunner(ctx, hctx.fileSystem, &function, pkgPath, hctx.fnResults, opts, hctx.runtime)
+		runner, err = fnruntime.NewRunner(ctx, hctx.fileSystem, &fns[i], pkgPath, hctx.fnResults, opts, hctx.runtime)
 		if err != nil {
 			return nil, i, err
 		}

pkg/api/kptfile/v1/types.go

@@ -437,8 +437,8 @@ type PipelineStepResult struct {
 
 // ResultItem mirrors framework.Result with only the fields needed for Kptfile status.
 type ResultItem struct {
-	Message     string `yaml:"message,omitempty" json:"message,omitempty"`
-	Severity    string `yaml:"severity,omitempty" json:"severity,omitempty"`
+	Message     string       `yaml:"message,omitempty" json:"message,omitempty"`
+	Severity    string       `yaml:"severity,omitempty" json:"severity,omitempty"`
 	ResourceRef *ResourceRef `yaml:"resourceRef,omitempty" json:"resourceRef,omitempty"`
 	Field       *FieldRef    `yaml:"field,omitempty" json:"field,omitempty"`
 	File        *FileRef     `yaml:"file,omitempty" json:"file,omitempty"`

pkg/test/runner/runner.go

@@ -171,7 +171,6 @@ func (r *Runner) runFnEval() error {
 			return fmt.Errorf("failed to prepare package: %w", err)
 		}
 
-
 		err = r.runSetupScript(pkgPath)
 		if err != nil {
 			return err