Skip to content

Improve CI: add caching for Bookkeeper old release downloads and improve maven caching and error logs#4722

Merged
lhotari merged 14 commits intoapache:masterfrom
lhotari:lh-improve-ci-error-logging
Mar 10, 2026
Merged

Improve CI: add caching for Bookkeeper old release downloads and improve maven caching and error logs#4722
lhotari merged 14 commits intoapache:masterfrom
lhotari:lh-improve-ci-error-logging

Conversation

@lhotari
Copy link
Member

@lhotari lhotari commented Mar 10, 2026

Motivation

CI currently fails and it's hard to investigate the issue since surefire logs aren't uploaded and the threaddump isn't captured in backward compat test timeouts.
The problem appeared to be very slow progress of old Bookkeeper release downloads from archive.apache.org.
Maven caching wasn't usable for PR builds since the Maven cache wasn't updated as part of the daily builds. PR builds don't have write access to apache/bookkeeper cache.

Changes

  • fix condition for uploading surefire logs
  • add threaddump for backward compat tests
  • adds a cache for BK release downloads from archive.apache.org. In CI, GitHub Actions cache is used to persist the cache. In local development, a directory is used to cache the downloads.
  • Fix maven dependency caching by adding caching to the daily build

@lhotari lhotari changed the title Improve CI: upload logs on errors and capture threaddump when backward compat fails Improve CI: add caching for Bookkeeper old release downloads and improve maven caching and error logs Mar 10, 2026
Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

+1

@lhotari lhotari merged commit 5549045 into apache:master Mar 10, 2026
36 of 39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants