lightbend / kalix-jvm-sdk

Java and Scala SDKs for Kalix
https://docs.kalix.io/java/index.html
Other
58 stars 39 forks source link

usage of tmp dir in Kalix #2134

Closed max8github closed 5 months ago

max8github commented 5 months ago

From this thread, it is noted that setup-kalix-action is using /tmp/kalix-bin. See here The directory is not emptied on self-host runners and it makes it fail on a second run:

Run lightbend/setup-kalix-action@v1
Downloading install-cli script
Downloaded /opt/actions-runner/_work/_temp/bc8858ba-d943-4b96-95fb-5808ccfb756b
/usr/bin/chmod +x /opt/actions-runner/_work/_temp/bc8858ba-d943-4b96-95fb-5808ccfb756b
/usr/bin/mkdir /tmp/kalix-bin
/usr/bin/mkdir: cannot create directory '/tmp/kalix-bin': File exists
Error: The process '/usr/bin/mkdir' failed with exit code 1

Wouldn't it be better to use runner.temp from runner context ?