Skip to content

Make llvm-cache.sh log more useful and verbose information#614

Merged
bors-ferrocene[bot] merged 1 commit intomainfrom
hoverbear/improve-llvm-cache-logs
May 7, 2024
Merged

Make llvm-cache.sh log more useful and verbose information#614
bors-ferrocene[bot] merged 1 commit intomainfrom
hoverbear/improve-llvm-cache-logs

Conversation

@Hoverbear
Copy link
Member

We store our LLVM caches in a specific S3 bucket location. This PR ensures we log the location, it also ensures we log both when we start, and when we complete, both actions. (Previously we only logged starting storing the cache, and after restoring)

This fell mostly out of #565.

@Hoverbear Hoverbear self-assigned this May 7, 2024
@Hoverbear Hoverbear marked this pull request as ready for review May 7, 2024 17:49
Copy link
Member

@tshepang tshepang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors merge

@bors-ferrocene
Copy link
Contributor

bors-ferrocene bot commented May 7, 2024

Build succeeded:

  • full

@bors-ferrocene bors-ferrocene bot merged commit a423db6 into main May 7, 2024
@bors-ferrocene bors-ferrocene bot deleted the hoverbear/improve-llvm-cache-logs branch May 7, 2024 19:46
@github-actions github-actions bot added the backport:maybe PR candidate for backport label May 7, 2024
@Hoverbear Hoverbear added backport:manual PR requiring manual intervention to backport backport:1.79 and removed backport:maybe PR candidate for backport labels Jul 2, 2024
Hoverbear added a commit that referenced this pull request Jul 2, 2024
@Hoverbear Hoverbear removed backport:manual PR requiring manual intervention to backport backport:1.79 labels Jul 2, 2024
ferrocene-automations bot pushed a commit that referenced this pull request Apr 23, 2025
ferrocene-automations bot pushed a commit that referenced this pull request Dec 16, 2025
fix freestanding `let` in a module and wrong brackets
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants