TRY/FINALLY for unit tests + xml report artifact seems to cause constant cache misses about earthly

[brettdh]

Sorry for the delay; here you go:

Earthfile:

VERSION --try 0.8

test:
  FROM python:3.12

  RUN echo "asdf" > ./some-other-file
  TRY
    RUN python -c 'import time; time.sleep(5); print("jkl;")' > ./testfile
  FINALLY
    SAVE ARTIFACT ./testfile AS LOCAL ./testfile
  END

First run:

➜ earthly +test
<snip>
 Build 🔧
————————————————————————————————————————————————————————————————————————————————

         python:3.12 | --> Load metadata python:3.12 linux/arm64
               +test | --> FROM python:3.12
               +test | [----------] 100% FROM python:3.12
               +test | --> RUN echo "asdf" > ./some-other-file
               +test | --> RUN python -c 'import time; time.sleep(5); print("jkl;")' > ./testfile
(5 second pause)
               +test | --> SAVE ARTIFACT ./testfile +test/testfile AS LOCAL ./testfile
              output | --> exporting outputs
              output | sent 1 file stat)
              output | [----------] 100% exporting outputs
              output | --> exporting outputs
<snip>

Second run (immediately following first):

➜ earthly +test
<snip>
 Build 🔧
————————————————————————————————————————————————————————————————————————————————

         python:3.12 | --> Load metadata python:3.12 linux/arm64
               +test | --> FROM python:3.12
               +test | [----------] 100% FROM python:3.12
               +test | *cached* --> RUN echo "asdf" > ./some-other-file
               +test | --> RUN python -c 'import time; time.sleep(5); print("jkl;")' > ./testfile
(5 second pause)
               +test | *cached* --> SAVE ARTIFACT ./testfile +test/testfile AS LOCAL ./testfile
              output | --> exporting outputs
              output | sent 1 file stat)
              output | [----------] 100% exporting outputs
              output | --> exporting outputs
<snip>

On the second run, I'd have expected the RUN inside the TRY/FINALLY to have been a cache hit, but it was a cache miss, and accordingly it took 5 seconds to run.

[alexcb]

can you provide a minimal reproduction case that we can run?

Your example can't be run as-is (e.g. missing a VERSION command, missing the project directory, missing whatever dependencies are done in the base target).