From 7a8bb7058beb75a690a35e3d4998d1338aea9a58 Mon Sep 17 00:00:00 2001
From: Arun Kant <arun.kant@suse.com>
Date: Mon, 29 Jul 2019 16:27:37 -0700
Subject: [PATCH] Removing deprecated option usage in gatther pod logs logic

As per PR, https://github.com/kubernetes/kubernetes/pull/60210,
in kubectl get show-all option is deprecated and no longer needed.
Presumably now that's the default behavior.

Also in current logs gathering logic, we are interested in capturing
only pod names, so removing that option is harmless.

We are seeing related failures in local CI when kubectl version is
1.15.x. So removing this option.

Change-Id: I3886c792fe28bc8b80504d8c91e9524039131b15
---
 roles/gather-pod-logs/tasks/main.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/roles/gather-pod-logs/tasks/main.yaml b/roles/gather-pod-logs/tasks/main.yaml
index e05a8f11d..8f48b7da3 100644
--- a/roles/gather-pod-logs/tasks/main.yaml
+++ b/roles/gather-pod-logs/tasks/main.yaml
@@ -29,7 +29,7 @@
           }
           function get_pods () {
             NAMESPACE=$1
-            kubectl get pods -n ${NAMESPACE} -o name --show-all | awk -F '/' '{ print $NF }' | xargs -L1 -P 1 -I {} echo ${NAMESPACE} {}
+            kubectl get pods -n ${NAMESPACE} -o name | awk -F '/' '{ print $NF }' | xargs -L1 -P 1 -I {} echo ${NAMESPACE} {}
           }
           export -f get_pods
           function get_pod_logs () {