Changed implementation to just add to root of package

Signed-off-by: liamfallon <liam.fallon@est.tech>

Author: liamfallon

internal/kpt/builtins/pkg_context.go

@@ -95,7 +95,11 @@ func (pc *PackageContextGenerator) Process(resourceList *framework.ResourceList)
 				}
 				return resourceList.Results
 			}
-			contextResources = append(contextResources, pkgContext)
+
+			// Only add package context to the root of a kpt package
+			if pkgContext != nil {
+				contextResources = append(contextResources, pkgContext)
+			}
 		}
 	}
 
@@ -121,6 +125,12 @@ func pkgContextResource(kptfile *yaml.RNode, packageConfig *PackageConfig) (*yam
 	if err != nil {
 		return nil, err
 	}
+
+	// We only want one "package-context.yaml" in each kpt package
+	if kptfilePath != kptfilev1.KptFileGVK().Kind {
+		return nil, nil
+	}
+
 	annotations := map[string]string{
 		kioutil.PathAnnotation: path.Join(path.Dir(kptfilePath), PkgContextFile),
 	}

internal/kpt/builtins/testdata/pkg-with-nesting/out.yaml

@@ -42,21 +42,8 @@ items:
       internal.config.kubernetes.io/path: 'package-context.yaml'
   data:
     name: order-service
-- apiVersion: v1
-  kind: ConfigMap
-  metadata:
-    name: kptfile.kpt.dev
-    annotations:
-      config.kubernetes.io/local-config: "true"
-      internal.config.kubernetes.io/path: 'subpkg/package-context.yaml'
-  data:
-    name: subpkg
 results:
 - message: generated package context
   severity: info
   file:
     path: package-context.yaml
-- message: generated package context
-  severity: info
-  file:
-    path: subpkg/package-context.yaml

pkg/task/clone.go

@@ -80,6 +80,19 @@ func (m *clonePackageMutation) apply(ctx context.Context, resources repository.P
 		}
 	}
 
+	if m.isDeployment {
+		// TODO(droot): executing this as mutation is not really needed, but can be
+		// refactored once we finalize the task/mutation/commit model.
+		genPkgContextMutation, err := newPackageContextGeneratorMutation(m.packageConfig)
+		if err != nil {
+			return repository.PackageResources{}, nil, err
+		}
+		cloned, _, err = genPkgContextMutation.apply(ctx, cloned)
+		if err != nil {
+			return repository.PackageResources{}, nil, pkgerrors.Wrap(err, "failed to generate deployment context")
+		}
+	}
+
 	// ensure merge-key comment is added to newly added resources.
 	// this operation is done on best effort basis because if upstream contains
 	// valid YAML but invalid KRM resources, merge-key operation will fail