)]}'
{"tools/deployment/common/wait-for-pods.sh":[{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"35ebcd9d1b49801572f4642b571079cdc82d1343","unresolved":false,"context_lines":[{"line_number":15,"context_line":"#    under the License."},{"line_number":16,"context_line":"set -e"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"function get_logs() {"},{"line_number":19,"context_line":"    #Getting logs for the pods in state CrashLoopBackOff (unfortunately"},{"line_number":20,"context_line":"    # --field-selector status.phase\u003dCrashLoopBackOff does not work)"},{"line_number":21,"context_line":"    echo \"Printing logs for pods in CrashLoopBackOff state\""},{"line_number":22,"context_line":"    kubectl get pods --namespace $1 | grep CrashLoopBackOff | \\"},{"line_number":23,"context_line":"    cut -d\" \" -f1 | sed \u0027s/^/pods\\//\u0027 | \\"},{"line_number":24,"context_line":"    xargs -L1 -r kubectl --namespace $1 logs"},{"line_number":25,"context_line":"}"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"# From Kolla-Kubernetes, original authors Kevin Fox \u0026 Serguei Bezverkhi"},{"line_number":28,"context_line":"# Default wait timeout is 600 seconds"}],"source_content_type":"text/x-sh","patch_set":2,"id":"dfbec78f_3f38b13b","line":25,"range":{"start_line":18,"start_character":0,"end_line":25,"end_character":1},"updated":"2019-05-08 08:38:23.000000000","message":"I think there is already a log function in the tooling, and we should make use of it, instead of redefining one.\n\nWith a little refactor you could reuse https://github.com/openstack/openstack-helm-infra/blob/master/roles/gather-pod-logs/tasks/main.yaml in your case, or simply call the ansible playbook which calls gather-pod-logs.","commit_id":"0c8cb82b6fc8602c0c4937a428df24151df122e7"}]}