fix(container): add default container annotation to generated manifests

This is useful when pod has multiple containers (e.g. injected as sidecars).
With this annotation, the right container is picked when exec
is run without specifying container name

core/src/plugins/kubernetes/container/deployment.ts

@@ -40,6 +40,7 @@ import {
 } from "../types"
 import { k8sGetContainerDeployStatus, ContainerServiceStatus } from "./status"
 import { emitNonRepeatableWarning } from "../../../warnings"
+import { K8_POD_DEFAULT_CONTAINER_ANNOTATION_KEY } from "../run"
 
 export const DEFAULT_CPU_REQUEST = "10m"
 export const DEFAULT_MEMORY_REQUEST = "90Mi" // This is the minimum in some clusters - or so they tell me
@@ -492,6 +493,9 @@ function workloadConfig({
   }
 
   const { annotations, daemon } = action.getSpec()
+  // Add default-container annotation in generated manifest
+  // so exec respects it in case of multiple containers in pod
+  annotations[K8_POD_DEFAULT_CONTAINER_ANNOTATION_KEY] = action.name
 
   return {
     kind: daemon ? "DaemonSet" : "Deployment",

core/test/integ/src/plugins/kubernetes/container/deployment.ts

@@ -51,6 +51,7 @@ import { ResolvedDeployAction } from "../../../../../../src/actions/deploy"
 import { ActionRouter } from "../../../../../../src/router/router"
 import { ActionMode } from "../../../../../../src/actions/types"
 import { createActionLog } from "../../../../../../src/logger/log-entry"
+import { K8_POD_DEFAULT_CONTAINER_ANNOTATION_KEY } from "../../../../../../src/plugins/kubernetes/run"
 
 describe("kubernetes container deployment handlers", () => {
   let garden: TestGarden
@@ -289,7 +290,9 @@ describe("kubernetes container deployment handlers", () => {
           selector: { matchLabels: { [gardenAnnotationKey("action")]: action.key() } },
           template: {
             metadata: {
-              annotations: {},
+              annotations: {
+                [K8_POD_DEFAULT_CONTAINER_ANNOTATION_KEY]: "simple-service",
+              },
               labels: getDeploymentLabels(action),
             },
             spec: {
@@ -331,7 +334,10 @@ describe("kubernetes container deployment handlers", () => {
       const action = await resolveDeployAction("simple-service")
       const namespace = provider.config.namespace!.name!
 
-      action["_config"].spec.annotations = { "annotation.key": "someValue" }
+      action["_config"].spec.annotations = {
+        "annotation.key": "someValue",
+        [K8_POD_DEFAULT_CONTAINER_ANNOTATION_KEY]: "simple-service",
+      }
 
       const resource = await createWorkloadManifest({
         ctx,