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

Add support in for signing and publishing RPM and Deb packages to GCP Artifact Registry #1246

Merged
merged 12 commits into from
Aug 13, 2024

Conversation

jdoss
Copy link
Contributor

@jdoss jdoss commented Jul 23, 2024

This adds support for signing RPM and Deb packages and uploading them to GCP Artifact Registry. It also changes the RPM and Deb file name format to use the ConventionalFileName macro in GoReleaser.

@jdoss jdoss requested review from tashian, maraino and dopey July 23, 2024 19:12
@github-actions github-actions bot added the needs triage Waiting for discussion / prioritization by team label Jul 23, 2024
.goreleaser.yml Outdated Show resolved Hide resolved
--package "${PACKAGE}" \
--filter "VERSION:${VER}" \
--format json 2> /dev/null \
| jq -re '.[].name?' >/dev/null 2>&1 \
Copy link
Contributor

Choose a reason for hiding this comment

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

might need to install jq on the VM

Copy link
Contributor Author

Choose a reason for hiding this comment

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

JQ comes preinstalled on the ubuntu action IIRC.

version: 2
project_name: step

variables:
packageName: step-cli
packageRelease: 1 # Manually update release: in the nfpm section to match this value if you change this
Copy link
Collaborator

@maraino maraino Jul 24, 2024

Choose a reason for hiding this comment

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

What's the reason for not using "{{ .Var.packageRelease }}" in the nfpm section?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It will default to -1 and you only need to set that if we need to manually change the release and the nfpm section doesn't support variables.

@jdoss jdoss force-pushed the jdoss/Package_Repos branch from ca2217e to 54d3e3b Compare August 8, 2024 19:38
@jdoss jdoss merged commit 35525cb into master Aug 13, 2024
16 checks passed
@jdoss jdoss deleted the jdoss/Package_Repos branch August 13, 2024 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs triage Waiting for discussion / prioritization by team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants