This is an automated email from the ASF dual-hosted git repository.

jhg03a pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficcontrol.git


The following commit(s) were added to refs/heads/master by this push:
     new e07cdaee55 Ansible (ATS role): improve resiliancy around 
storage.config (#6949)
e07cdaee55 is described below

commit e07cdaee55439c772002b46f729e4a6be5d267c4
Author: Zach Hoffman <[email protected]>
AuthorDate: Fri Jul 8 19:26:52 2022 -0600

    Ansible (ATS role): improve resiliancy around storage.config (#6949)
    
    Added an additional guard to the Ansible ATS role when defining devices 
that should be part of the storage.config to not accidentally include the root 
disk when blank lines were included in the device list.
---
 infrastructure/ansible/roles/ats/tasks/ats.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/infrastructure/ansible/roles/ats/tasks/ats.yml 
b/infrastructure/ansible/roles/ats/tasks/ats.yml
index e454552a87..46e7714118 100644
--- a/infrastructure/ansible/roles/ats/tasks/ats.yml
+++ b/infrastructure/ansible/roles/ats/tasks/ats.yml
@@ -102,7 +102,7 @@
       command: /opt/trafficserver/bin/traffic_server -Cclear
 
     - name: Get list of targeted ATS cache disks
-      shell: cat /opt/trafficserver/etc/trafficserver/storage.config | grep -v 
'#' | cut -d' ' -f1
+      shell: cat /opt/trafficserver/etc/trafficserver/storage.config | grep 
-ve '#' -ve '^$' | cut -d' ' -f1
       register: cachedisk_results
       changed_when: false
 

Reply via email to