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 the packaging metadata to build the gobot snap #353

Merged
merged 2 commits into from
Jan 5, 2017

Conversation

come-maiz
Copy link
Contributor

No description provided.

@come-maiz come-maiz changed the base branch from master to dev January 5, 2017 03:12
@coveralls
Copy link

coveralls commented Jan 5, 2017

Coverage Status

Coverage increased (+0.04%) to 70.315% when pulling 2658ac1 on elopio:snapcraft into e0ccab8 on hybridgroup:dev.

@come-maiz
Copy link
Contributor Author

Today I'm back at work, and I love everything I learned with gobot and my bbb over the holidays. I think it will be an awesome component of our snappy systems with Ubuntu Core. If you haven't heard about snaps yet, you can take a look at http://snapcraft.io/

As soon as we improve a little our bbb kernel snap I will be writing a few simple demos using gobot to make snaps that interact with the board. But in the mean time, I thought that it wouldn't hurt to make a snap for the gobot code generator. It's not terribly useful because you would get the binary with a go get + go build, but it might serve to expose it a little more to all the ubuntu users. And maybe you can start looking at the security, isolation, automatic updates, unstable channels and the store, which I think would be really useful for gort. A gort snap would be amazing! And making sure that it works in strict isolation would be quite fun.

So please take a look at this, and let me know it seems interesting for your projects. For us any kind of feedback would be great, to help us assign the priorities on our backlog.

To give it a try in any Ubuntu 16.04 or later:

$ sudo apt install git snapcraft
$ git clone https://github.com/elopio/gobot
$ cd gobot
$ git checkout snapcraft
$ snapcraft
$ sudo snap install *.snap --dangerous
$ gobot

@deadprogram
Copy link
Member

This is excellent work @ElOpio I've been wanting to see this happen for a while. I'd love to help out however needed, can't wait to see your BBB snaps!

@deadprogram deadprogram merged commit 29651fb into hybridgroup:dev Jan 5, 2017
@deadprogram
Copy link
Member

Also, can we get your assistance to add a snapcraft file to Gort? https://github.com/hybridgroup/gort :)

@come-maiz come-maiz deleted the snapcraft branch January 6, 2017 16:35
@come-maiz
Copy link
Contributor Author

Thanks @deadprogram. It would be nice if you register the snap and push it once to the edge channel, to get a feel of what the store offers: https://myapps.developer.ubuntu.com/dev/click-apps/register-name/
And I will be happy to assist with gort. Would you like to meet next week to talk about it?

@popey
Copy link

popey commented Jan 25, 2017

Is there a blocker getting gobot in the store? I'd also like to help get this in the store!

@deadprogram
Copy link
Member

The snap I built locally used golang 1.6. I want to build using 1.7.4 at the very least. How can I do this?

@deadprogram
Copy link
Member

In the meantime, here is https://myapps.developer.ubuntu.com/dev/click-apps/6777/

@come-maiz
Copy link
Contributor Author

Thanks a lot for the release @deadprogram!

We have an open bug for go 1.7: https://bugs.launchpad.net/snapcraft/+bug/1616985
It would be nice to get your comment in there about your use case, so my other teammates take it into account when discussing about priorities.

go 1.7 is not super simple to support, because in xenial we have go 1.6. This bug made us discuss about having snapcraft parts [1] that can provide binaries to build other parts. We all loved the idea, and now that you requested it I hope we'll get to work on it soon. In the meantime, I have a solution that I can propose for gobot: https://github.com/elopio/hashicorp-snaps/blob/master/consul/parts/plugins/x-go.py#L3 I want to try to clean it up a little using the most recent features of snapcraft, so I'll send you a PR tomorrow to see if you like it.

pura vida.

[1] http://snapcraft.io/docs/build-snaps/parts

@deadprogram
Copy link
Member

Hi @ElOpio looks like a good plan to me. I've already posted a comment on the bug tracker.

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

Successfully merging this pull request may close these issues.

4 participants