Skip to content

Pin snapshot reproducibility checks to the deployed artifacts - #4215

Merged
ppkarwasz merged 1 commit into
2.xfrom
fix/2.x/verify-reproducibility-race
Jul 26, 2026
Merged

Pin snapshot reproducibility checks to the deployed artifacts#4215
ppkarwasz merged 1 commit into
2.xfrom
fix/2.x/verify-reproducibility-race

Conversation

@ppkarwasz

Copy link
Copy Markdown
Member

Caller-side counterpart of apache/logging-parent#505: the snapshot reproducibility check now verifies against the repository artifact uploaded by the current run instead of the shared Nexus snapshot repository, which a concurrent run could overwrite. The release path keeps using the per-release staging repository URL.

Warning

Should be merged only after apache/logging-parent#505 lands and the gha/v0 tag is updated.

Split `verify-reproducibility` into per-deployment jobs. The snapshot
variant now verifies against the repository artifact uploaded by
`deploy-snapshot-reusable.yaml` instead of the shared Nexus snapshot
repository, which could be overwritten by a concurrent run. The release
variant keeps using the per-release staging repository URL.

Requires apache/logging-parent#505.

Assisted-By: Claude Fable 5 <noreply@anthropic.com>
@ramanathan1504
ramanathan1504 marked this pull request as ready for review July 26, 2026 01:07

@ramanathan1504 ramanathan1504 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

apache/logging-parent#505

Successfully merged and all CI Pass

@ppkarwasz

Copy link
Copy Markdown
Member Author

Let's merge this and see if the check succeeds.

@ppkarwasz
ppkarwasz merged commit 9588841 into 2.x Jul 26, 2026
12 checks passed
@ppkarwasz
ppkarwasz deleted the fix/2.x/verify-reproducibility-race branch July 26, 2026 05:12
@github-project-automation github-project-automation Bot moved this from Approved to Merged in Log4j pull request tracker Jul 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants