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

goreleaser based release build #136

Merged
merged 3 commits into from
Oct 24, 2024

Conversation

bavarianbidi
Copy link
Contributor

@bavarianbidi bavarianbidi commented Oct 17, 2024

closes #28

This will add a goreleaser based release process as described in the updated RELEASE.md

An example release with artifacts can be seen here: https://github.com/bavarianbidi/auger/releases/tag/v0.1.0

https://github.com/bavarianbidi/auger/actions/runs/11392186824/job/31697673348 will give you the github actions output of the release process, produced by goreleaser.

Copy link
Member

@ivanvc ivanvc left a comment

Choose a reason for hiding this comment

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

Hi @bavarianbidi, thanks for your pull request!

A question is whether we should go the gorelease way (with GitHub actions), which strives away from our effort of migrating the GitHub actions to the prow infrastructure.

I'm not sure if the workflow will work because, as we experienced before, we don't have write permission in the repositories.

I left some comments on the pull request (although I'm not familiar with auguer). One thing that I also noticed is that I suggest breaking this pull request into two. One addressing the changes to the Makefile, and another for introducing goreleaser. That way, we can squash commits, and we don't have multi-purpose commits in the same PR :)

Thoughts on this approach, @jmhbnz?

.github/workflows/release.yaml Outdated Show resolved Hide resolved
.github/workflows/release.yaml Show resolved Hide resolved
.goreleaser.yaml Outdated Show resolved Hide resolved
Copy link
Member

@jmhbnz jmhbnz left a comment

Choose a reason for hiding this comment

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

LGTM - Thanks @bavarianbidi

cc @ivanvc can you please also review once more before we merge.

Copy link
Member

@ivanvc ivanvc left a comment

Choose a reason for hiding this comment

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

LGTM. I'm leaving two open questions, but I'm fine if they're addressed in follow-up pull requests. Thanks, @bavarianbidi.

.github/workflows/release.yaml Outdated Show resolved Hide resolved
augerctl/command/version.go Outdated Show resolved Hide resolved
releases will be done via goreleaser. It will also add a version
subcommand to auger and augerctl

Signed-off-by: Mario Constanti <[email protected]>
Signed-off-by: Mario Constanti <[email protected]>
Copy link
Member

@ivanvc ivanvc left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks, @bavarianbidi 🎉

@k8s-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bavarianbidi, ivanvc, jmhbnz

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jmhbnz jmhbnz merged commit b7214d2 into etcd-io:main Oct 24, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create github workflow for release.yaml for auger project
4 participants