Set kube-apiserver as default logs container

Apply suggestions from code review

Co-authored-by: Peter Rifel <rifelpet@users.noreply.github.com>

nodeup/pkg/model/kube_apiserver.go

@@ -732,6 +732,7 @@ func (b *KubeAPIServerBuilder) buildPod(kubeAPIServer *kops.KubeAPIServerConfig)
 
 func (b *KubeAPIServerBuilder) buildAnnotations() map[string]string {
 	annotations := make(map[string]string)
+	annotations["kubectl.kubernetes.io/default-container"] = "kube-apiserver"
 
 	if b.Cluster.Spec.API != nil {
 		if b.Cluster.Spec.API.LoadBalancer == nil || !b.Cluster.Spec.API.LoadBalancer.UseForInternalApi {

nodeup/pkg/model/tests/golden/awsiam/tasks-kube-apiserver.yaml

@@ -26,6 +26,7 @@ contents: |
     annotations:
       dns.alpha.kubernetes.io/external: api.minimal.example.com
       dns.alpha.kubernetes.io/internal: api.internal.minimal.example.com
+      kubectl.kubernetes.io/default-container: kube-apiserver
       scheduler.alpha.kubernetes.io/critical-pod: ""
     creationTimestamp: null
     labels:

nodeup/pkg/model/tests/golden/dedicated-apiserver/tasks-kube-apiserver.yaml

@@ -5,6 +5,7 @@ contents: |
     annotations:
       dns.alpha.kubernetes.io/external: api.minimal.example.com
       dns.alpha.kubernetes.io/internal: api.internal.minimal.example.com
+      kubectl.kubernetes.io/default-container: kube-apiserver
       scheduler.alpha.kubernetes.io/critical-pod: ""
     creationTimestamp: null
     labels:

nodeup/pkg/model/tests/golden/minimal/tasks-kube-apiserver.yaml

@@ -5,6 +5,7 @@ contents: |
     annotations:
       dns.alpha.kubernetes.io/external: api.minimal.example.com
       dns.alpha.kubernetes.io/internal: api.internal.minimal.example.com
+      kubectl.kubernetes.io/default-container: kube-apiserver
       scheduler.alpha.kubernetes.io/critical-pod: ""
     creationTimestamp: null
     labels:

nodeup/pkg/model/tests/golden/side-loading/tasks-kube-apiserver-amd64.yaml

@@ -5,6 +5,7 @@ contents: |
     annotations:
       dns.alpha.kubernetes.io/external: api.minimal.example.com
       dns.alpha.kubernetes.io/internal: api.internal.minimal.example.com
+      kubectl.kubernetes.io/default-container: kube-apiserver
       scheduler.alpha.kubernetes.io/critical-pod: ""
     creationTimestamp: null
     labels:

nodeup/pkg/model/tests/golden/side-loading/tasks-kube-apiserver-arm64.yaml

@@ -5,6 +5,7 @@ contents: |
     annotations:
       dns.alpha.kubernetes.io/external: api.minimal.example.com
       dns.alpha.kubernetes.io/internal: api.internal.minimal.example.com
+      kubectl.kubernetes.io/default-container: kube-apiserver
       scheduler.alpha.kubernetes.io/critical-pod: ""
     creationTimestamp: null
     labels:

nodeup/pkg/model/tests/golden/without-etcd-events/tasks-kube-apiserver.yaml

@@ -5,6 +5,7 @@ contents: |
     annotations:
       dns.alpha.kubernetes.io/external: api.minimal.example.com
       dns.alpha.kubernetes.io/internal: api.internal.minimal.example.com
+      kubectl.kubernetes.io/default-container: kube-apiserver
       scheduler.alpha.kubernetes.io/critical-pod: ""
     creationTimestamp: null
     labels: