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

make installation easier (goreleaser) #134

Closed
scottrigby opened this issue Dec 12, 2018 · 6 comments
Closed

make installation easier (goreleaser) #134

scottrigby opened this issue Dec 12, 2018 · 6 comments

Comments

@scottrigby
Copy link

scottrigby commented Dec 12, 2018

Problem

Installation is not straightforward enough for some users.

Suggestion

Use goreleaser (with a homebrew option for mac users, etc). The resulting installation section of your readme could then look something like this (you're not using dep, so the instructions would be even simpler, although… separate issue, but it may be fun to stop including the vendor dir directly - it'd make PRs a lot easier to review when deps need to change!).

It's pretty straightforward so I'm sure you don't need help, but 'm happy to pitch in if you think that'd be fun.

CC @anguslees & @jbeda 😄 👋

@anguslees
Copy link
Contributor

I agree we should switch to goreleaser.

In the meantime, there is now a homebrew recipe for kubeseal, thanks to @mellena1: https://github.com/Homebrew/homebrew-core/blob/master/Formula/kubeseal.rb

We should absolutely document that in the README installation section, because macOS users expect to be spoonfed :trollface:

@karlskewes
Copy link
Contributor

Had a go at a PR... don't mind github snitching on me.. 😅

bors bot added a commit that referenced this issue Mar 12, 2019
140: GoReleaser build and release support r=anguslees a=kskewes

GoReleaser build and release support

Simplified build (and release?) per #134.
Not automatically hooked into Travis or anything (yet).

**Binaries tested with `--version`:**
- linux (docker images as well)
- windows

**Binaries not tested:**
- darwin (don't have mac)

**Build and Publish tested:**
- Create [GitHub Token](https://goreleaser.com/environment/)
- In `.goreleaser.yml` comment out `release.disable: true`
- `git commit`
- `git tag v0.7.1 && git push`
- `make publish`
- single archive per platform contains both `controller` and `kubeseal` binaries, plus kubernetes yaml's
- locally docker images are created

**Not Tested:**
- docker image push - In `.goreleaser.yml` comment out `docker.skip_push: true`

**Build fork/dirty/etc:** (assuming merged, patch on upstream or blog link below)
```
make snapshot
```
This results in binaries, archives and docker images tagged with full git commit hash.

**To Do:**
- [x] TODO: comment added for brew to change to [published formula](https://github.com/Homebrew/homebrew-core/blob/master/Formula/kubeseal.rb)
- [x] confirm ok publishing archive instead of binaries
- [x] yamls added to archive
- [x] DOCS update and simplification

First time using goreleaser so... :)

Co-authored-by: Karl Skewes <karl.skewes@gmail.com>
Co-authored-by: Karl <karl.skewes@gmail.com>
@spencerolsen
Copy link

I apologize for the dumb question, but I am on Windows 10 and don't know how to install. Any documentation I can refer to? I have not used goreleaser before, and it seems like support for it was maybe reverted? Any help would be greatly appreciated.

@karlskewes
Copy link
Contributor

Yes the PR was reverted and replacement submitted.
I did a release on my fork here a couple of months back. The windows kubeseal should work but it's unofficial and ymmv as I haven't used extensively.

https://github.com/kskewes/sealed-secrets/releases

@mkmik mkmik added this to the v0.9.0 milestone Jul 19, 2019
@mkmik mkmik modified the milestones: v0.9.0, v0.10.0 Sep 2, 2019
@mkmik
Copy link
Collaborator

mkmik commented Sep 2, 2019

(sorry for the slowness on this I'm dealing with higher prio stuff first)

Now the installation instructions mention brew for macos, and we even have a badge that should indicate how much the homebrew release lags behind out latest github release:

@scottrigby, do you think it still makes sense to have a tap now that we have a formula in the main repo? I'm not very familiar with brew (despite being a macos user myself). Would a tap allow us to release (almost) atomically with the main release?

I noticed that the binary installed with brew install kubeseal is not the actual build artifact we release; it appears to be built by somebody else, likely just with go build since it doesn't have the version variable filled it by the linker flag (which would happen if you run make)

$ kubeseal --version
kubeseal version:
``

What are the other practical differences?

@mkmik mkmik modified the milestones: v0.10.0, v0.11.0 Mar 2, 2020
@mkmik mkmik removed this from the v0.11.0 milestone Mar 11, 2020
@juan131
Copy link
Collaborator

juan131 commented Dec 23, 2021

We adopted Goreleaser at #668

@juan131 juan131 closed this as completed Dec 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants