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

Docker Integration #284

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TheLoneRonin
Copy link

@TheLoneRonin TheLoneRonin commented Dec 31, 2020

Description

Notes

  • It may be a good idea to add other badges to the top of the repository. However, we may want other badges too ie: license, AR version etc.

  • We should investigate a way to resolve: G++ library linker issue with Travis CI #285

@TheLoneRonin TheLoneRonin force-pushed the @ronin/docker+travis branch 13 times, most recently from 9b79b31 to 5cc5515 Compare January 3, 2021 20:56
@TheLoneRonin TheLoneRonin changed the title Docker + Travis-CI Integration Docker Integration Jan 3, 2021
@ldmberman
Copy link
Member

Hi @TheLoneRonin, thank you for the update.

We are using Travis CI in fact. It does not work for you because you specified OTP 23, which is not compatible with the current versions of the libraries. OTP 22 should work.

As per the setup, I believe it's important to configure external storage (a volume/bind mount). Only using the container's writable layer has very limited use, even for tests.

@tudorfil9 do you have any thoughts or developments to share?

@tudorfil9
Copy link

I agree, there should be a separate storage option. The container layer is very small.

Have a look at this Arweave Docker Image from DockerHub.

https://hub.docker.com/layers/116646671/tudorfil9/arweave-node/latest/images/sha256-9df66b9b4f705eda5a1f27908ca21c4a4e08d13d33e8b52d252d77896ce19901?context=explore

@TheLoneRonin
Copy link
Author

TheLoneRonin commented Jan 4, 2021

@ldmberman, @tudorfil9,

OTP 23.0 compiles Arweave on a fresh install of Ubuntu 20.04.

I will update this PR with 22.2 and see if Travis works.


In terms of the storage option. Agree absolutely I should document how to mount volumes.

I think there should be two documentation options for volumes.

  1. How to use a volume it using the --mount flag.

  2. How to mount an existing volume.

Furthermore, I should add snippets on how to mount it to a different drive... ie: using lsblk to find the right drive you want to mount to.

And of course, link to existing Docker documentation for reference purposes.

@TheLoneRonin
Copy link
Author

@ldmberman, @tudorfil9, now I remember why I changed it from OTP 22.2...

This bizarre error came up: https://travis-ci.org/github/TheLoneRonin/arweave/jobs/752828446

@tudorfil9
Copy link

tudorfil9 commented Jan 4, 2021

Hey @TheLoneRonin it looks like your key for otp version might not be right.

Can you check your available versions and change the otp_release key to 22.2.4 for example ?

https://docs.travis-ci.com/user/languages/erlang/#choosing-otp-releases-to-test-against

@TheLoneRonin
Copy link
Author

@tudorfil9, yea, Travis is acting very weird. Not sure the best approach to resolving this.

Then ontop of that we'd have to tackle the -lerl_interface issue too once this is resolved.

https://travis-ci.org/github/ArweaveTeam/arweave/jobs/752840013

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.

G++ library linker issue with Travis CI
3 participants