Enhance caching logic with directory checks

[jagghoshal-blip]

• Add directory existence check during caching
• The ConstructCachePaths function now returns a Result and handles the creation of all required directories, including the build cache and the parent directories for target artifacts. This centralizes directory management within the path construction logic.
• Add cache hit/miss logging

Bug: 441779367

[cjreynol]

I think you meant to have the target be main. Also, was there a reason to not push this extra commit to #2040 and continue with that PR?

[jagghoshal-blip]

I think you meant to have the target be main. Also, was there a reason to not push this extra commit to #2040 and continue with that PR?

That was my initial attempt and it said I cannot publish the commit to this protected branch of mine, so I created cf_signed_1. 9bb0f4135 is the right HEAD commit for this change.

[Databean]

I think the confusion came from creating branches on the upstream github.com/google/android-cuttlefish repository. Normally the workflow is that every developer has their own fork (mine is github.com/Databean/android-cuttlefish) where they create new branches, and pull requests are made from the forked repository branches to the upstream main branch.

I've retargeted the pull request to reference main.

[jagghoshal-blip]

I think the confusion came from creating branches on the upstream github.com/google/android-cuttlefish repository. Normally the workflow is that every developer has their own fork (mine is github.com/Databean/android-cuttlefish) where they create new branches, and pull requests are made from the forked repository branches to the upstream main branch.

I've retargeted the pull request to reference main.

Thats it! This is the issue I created for myself. I should have used the jagghoshal-blip/android-cuttlefish repo. Thanks for the help!