-
-
Notifications
You must be signed in to change notification settings - Fork 101
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
Creating travis config to build both boxes with packer. #5
Conversation
Tzrlk
commented
Sep 14, 2015
- The ATLAS_TOKEN will need to be encrypted using travis
- I haven't tested the deployment script (I don't know what the packer outputs are).
Doesn't atlas support the github integration itself ? |
Just checked, atlas offers integration but wants full admin rights on public and private repos |
@Tzrlk sorry for the wait, I am still pondering on what the best approach is. Right now Atlas only works for the x86_64 arch for some reason so I still have to build the other box by hand anyways and running What would be nice is to have PR automatically validate some of the structure of the project like the .json format and make sure the Makefile has been run. |
Does the Atlas integration support building pull requests? That's the main On Tue, 20 Oct 2015 at 02:17 zimbatm notifications@github.com wrote:
|
Answering with another question: is it possible to run the builds without Atlas requires too many permissions that I feel comfortable to attach to my On Mon, 19 Oct 2015 23:50 Peter Cummuskey notifications@github.com wrote:
|
Yeah, that's the idea. It only deploys on tags. On Tue, 20 Oct 2015 at 23:01 zimbatm notifications@github.com wrote:
|
Hey, sorry for not keeping-up on that. I just setup the Atlas integration so Travis is not needed. I'll go ahead and close the issue but feel free to comment and I'll re-open it if you think this should still be tackled. |