-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Enable building snaps from source #969
Conversation
This would need to be hooked up with CI and all, but this should be first steps for allowing us to
|
In order to do that, we should integrate this as part of our release process, goreleaser supports snaps. I believe this simply implies we need make |
Thanks Ilya for the feedback. I'm not sure what the best way is to hook this up with goreleaser. 🤷♂️ |
My comment above was indeed to explain the direction, did you replied
before I added extra details to it?
…On Wed, 3 Jul 2019, 3:12 pm Daniel Holbach, ***@***.***> wrote:
Thanks Ilya for the feedback. I'm not sure what the best way is to hook
this up with goreleaser. 🤷♂
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#969>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAB5MS5Q4E42POG4MVASWBDP5SXVZANCNFSM4H5FLSKA>
.
|
Ah ok. Shall we do two separate PRs for this? |
@dholbach are you interested in finishing this? |
I was thinking we could land this piece of work first, then move on to |
Can we maybe just merge this and then do second PR with whatever's required to enable the build and publication of the snap? Goreleaser has support for |
Either we use build.snapcraft.io for this (the GH account to set this up will need to be admin unfortunately - to be able to set up the webhook). If that's not an option, we could try to roll our own and do a: # Prep
sudo apt install snapd
sudo snap install snapcraft --classic
sudo snap install multipass --beta --classic
# Actual build
snapcraft
snapcraft push eksctl_<version>_amd64.snap For the last step we'd need to login to the store, etc. |
@martina-if Thanks for assigning. On this one I think we're blocked because there was the idea to use What I'd suggest:
The last point we haven't figured out for
Let me know if it's unclear and if you have different thoughts. |
Updated. The PR is good to go AFAICS. Let's decouple the snap build bits from the publication in the store. |
Hi @dholbach thanks for updating this PR! If you think merging this is the best first step let's do that. Before that, can you please add those instructions here? https://github.com/weaveworks/eksctl/blob/master/CONTRIBUTING.md#release-process |
@martina-if I documented things TTBOMK. Please review. Our options for publishing (once we took over maintenance of the snap):
|
Thanks Michael Royal for your initial work on this. First attempts: #871 and mikeroyal#1
Build a snap from eksctl source. @mikeroyal and I worked on this (#871 and mikeroyal#1 are previous attempts at getting this in).
Description
To build the snap, run
The resulting snap works nicely for me.