Skip to content

Conversation

@lukasmalkmus
Copy link
Collaborator

@lukasmalkmus lukasmalkmus commented Aug 20, 2025

Had a bad archive config. Fixed it and also changed CI to run goreleaser --snapshot to mitigate this in the future.

Also includes a fix for #290.

@lukasmalkmus lukasmalkmus requested a review from thesollyz August 20, 2025 14:45
@lukasmalkmus lukasmalkmus self-assigned this Aug 20, 2025
@lukasmalkmus lukasmalkmus force-pushed the lukasmalkmus/lvqsrvtutptp branch from 35dd22b to 4d9be49 Compare August 20, 2025 15:01
@lukasmalkmus lukasmalkmus force-pushed the lukasmalkmus/lvqsrvtutptp branch from 4d9be49 to 49618ee Compare August 20, 2025 15:22
@lukasmalkmus
Copy link
Collaborator Author

Fixed the CI failures! 🎉

The issue was that the workflow changed from goreleaser build --snapshot to goreleaser --snapshot, which creates archives instead of raw binaries. The test jobs were looking for raw binaries at paths like dist/linux_linux_amd64_v1/axiom, but those didn't exist anymore.

The fix updates the test jobs to:

  1. Upload the archives created by goreleaser --snapshot (e.g., dist/axiom_*_linux_amd64.tar.gz)
  2. Extract the binaries from these archives before testing
  3. Handle the different archive formats (tar.gz for Unix, zip for Windows)

The CI should now pass with the goreleaser running in full snapshot mode, which will help catch release issues early as intended.

@lukasmalkmus lukasmalkmus enabled auto-merge (rebase) August 20, 2025 15:39
@lukasmalkmus lukasmalkmus merged commit c6e205c into main Aug 20, 2025
8 checks passed
@lukasmalkmus lukasmalkmus deleted the lukasmalkmus/lvqsrvtutptp branch August 20, 2025 15:43
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