ashb commented on a change in pull request #9220:
URL: https://github.com/apache/airflow/pull/9220#discussion_r438468236



##########
File path: scripts/ci/libraries/_kind.sh
##########
@@ -365,19 +365,23 @@ function apply_kubernetes_resources() {
 function dump_kubernetes_logs() {
     POD=$(kubectl get pods -o go-template --template '{{range 
.items}}{{.metadata.name}}{{"\n"}}{{end}}' \
         --cluster "${KUBECTL_CLUSTER_NAME}" | grep airflow | head -1)
+    kubectl get all
+    kubectl get all --namespace kube-system
+    echo "------coredns logs-------"
+    DNS_POD=$(kubectl get pods -o go-template -n kube-system --template 
'{{range .items}}{{.metadata.name}}{{"\n"}}{{end}}' \
+        --cluster "${KUBECTL_CLUSTER_NAME}" | grep dns | head -1)

Review comment:
       You forgot the logs command :dagger: 
   
   Also an easier way for this
   
   ```
   ```suggestion
     kubectl -n kube-system logs -l k8s-app=kube-dns --all-containers --cluster 
"${KUBECTL_CLUSTER_NAME}"
   ```
   
   (Even with coredns, the k8s-app label is still kube-dns)




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to