use argv instead of env vars for redis log path

ansible/roles/redis/tasks/main.yml

@@ -45,6 +45,7 @@
   become: yes
   shell: docker inspect --format="{{ '{{' }}.LogPath {{ '}}' }}" redis
   register: redis_log_path
+
 - name: show redis_log_path
   debug:
     msg: "{{ redis_log_path.stdout }}"
@@ -55,7 +56,7 @@
     name: redis_bg_save_sanity_checker
     special_time: daily
     job: docker pull newsblur/newsblur_python3:latest;
-      docker run --rm -e redis_log_path={{redis_log_path.stdout}} -v {{redis_log_path.stdout}}:{{redis_log_path.stdout}} -v /srv/newsblur:/srv/newsblur --network=newsblurnet newsblur/newsblur_python3 /srv/newsblur/utils/monitor_redis_bgsave.py
+      docker run --rm -v {{redis_log_path.stdout}}:{{redis_log_path.stdout}} -v /srv/newsblur:/srv/newsblur --network=newsblurnet newsblur/newsblur_python3 /srv/newsblur/utils/monitor_redis_bgsave.py {{redis_log_path.stdout}}
   when: "'redis' in inventory_hostname"
   tags:
     - sanity-checker

utils/monitor_redis_bgsave.py

@@ -10,13 +10,14 @@ from newsblur_web import settings
 import socket
 
 def main():
+    redis_log_path = sys.argv[1]
     t = os.popen('stat -c%Y /srv/newsblur/docker/volumes/redis/')
     timestamp = t.read().split('\n')[0]
     modified = datetime.datetime.fromtimestamp(int(timestamp))
     ten_min_ago = datetime.datetime.now() - datetime.timedelta(minutes=10)
     hostname = socket.gethostname()
     modified_minutes = datetime.datetime.now() - modified
-    log_tail = os.popen("tail -n 100 $(docker inspect --format='\{\{.LogPath}}' redis)").read()
+    log_tail = os.popen(f"tail -n 100 {redis_log_path} redis)").read()
     if True:
     #if modified < ten_min_ago:
         requests.post(