ferenc-csaky commented on PR #26723:
URL: https://github.com/apache/flink/pull/26723#issuecomment-3018583353

   > @ferenc-csaky I switched out the script the git action on a local repo and 
ran this script instead. Everything looked as expected when the github action 
ran.
   > 
   > ```
   > #!/usr/bin/env bash
   > echo "Got here"
   > USER_CACHE_FILENAME="user_cache.txt"
   > 
   > echo "foo bar" > $USER_CACHE_FILENAME
   > echo "wrote file"
   > cat $USER_CACHE_FILENAME
   > echo "finished cat"
   > ```
   > 
   > I think therefore we are safe to leave the cache where it is.
   > 
   > I wonder if the github action you saw issues with in the previous project 
might have been a "triggered" git action. which have permission restrictions 
from forked repos over those described in the yml. For the cron job should 
should be ok - I do not have write access to the Flink repo - so cannot 
actually test this on Flink , but have tested locally and on the getindata repo.
   
   Just replied in the original review comment to this, lets leave this as is, 
I am also thinking it should most probably work, but if not, we'll iterate on 
it in another PR. Thanks!


-- 
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.

To unsubscribe, e-mail: [email protected]

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

Reply via email to