Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

3.9.0: RPM package name changed again #1547

Open
felixfontein opened this issue Jun 28, 2024 · 2 comments
Open

3.9.0: RPM package name changed again #1547

felixfontein opened this issue Jun 28, 2024 · 2 comments

Comments

@felixfontein
Copy link
Contributor

The RPM packages had a -1 after the version before the 3.8.0 release (#1266 (comment)), which wasn't there anymore for 3.8.x. But now with 3.9.0 it's back:

sops-3.9.0-1.aarch64.rpm
sops-3.9.0-1.x86_64.rpm

(https://github.com/getsops/sops/releases/tag/v3.9.0)

This is extremely frustrating for automated consumers of the releases such as the Ansible community.sops release, since this again broke its SOPS installation step.

Is there a way to fix the filename format somehow so it won't change again in the future?

@felixfontein
Copy link
Contributor Author

My guess that this is related with updates to goreleaser, so I would suggest to add a test which runs regularly (maybe weekly, just to make sure it still works) and when the release workflow is updated (which happens if the goreleaser version used in the release process is updated) to run goreleaser and check the output filenames.

That way we find out ealier whether a goreleaser update breaks something, and we can investigate before releasing...

@felixfontein
Copy link
Contributor Author

The behavior changed in nFPM 2.33.1 (https://github.com/goreleaser/nfpm/releases/tag/v2.33.1; more precisely: goreleaser/nfpm#715), which was included in goreleaser 1.21.0 (https://github.com/goreleaser/goreleaser/releases/tag/v1.21.0), which we switched to in https://github.com/getsops/sops/pull/1427/files#diff-87db21a973eed4fef5f32b267aa60fcee5cbdf03c67fafdc2a9b553bb0b15f34R68 - the PR assumed that version: 1.20.x was the Go version, and not the goreleaser version, so this change happened unnoticed when #1427 got merged.

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

No branches or pull requests

1 participant