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

Support for ARM64 (Fixes #22) #25

Open
wants to merge 31 commits into
base: master
Choose a base branch
from
Open

Support for ARM64 (Fixes #22) #25

wants to merge 31 commits into from

Conversation

sahilph
Copy link
Member

@sahilph sahilph commented Jun 11, 2018

Overview

Support for ARM64

Testing recommendations

Multi-arch manifest pushed. Just use treehouses/couchdb on a arm64 device and it will run.
To access 64Bit specifically use:
treehouses/rpi-couchdb:arm64-2.1.1-0.0.1-add-arm64-support-5a2b63ca

GitHub issue number

(Fixes #22)

Related Pull Requests

N/A

Checklist

  • Code is written and works correctly;
  • Changes are covered by tests;
  • Documentation reflects the changes;

@dogi dogi requested review from empeje, dogi and akshayuncc June 12, 2018 16:26
Copy link
Member

@empeje empeje left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good enough, only small comment.

mv 2.1.1/Dockerfile-arm64 2.1.1/Dockerfile
if [ "$BRANCH" = "master" ]
then
sed -i -e "s|\(treehouses\/rpi-couchdb:2\.1\.1\)|$V211_DOCKER_NAME_LATEST|" 2.1.1/Dockerfile
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sed is faster, but for OLE you create YAML on the fly, any specific reason not doing that what in this repo?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no reason as such. This is how we used to do it before I found the method to create it on the fly. 😉

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🆗

@empeje
Copy link
Member

empeje commented Jul 23, 2018

Is this finally working @dogi and @sahilph?

I can merge this after you fix the conflicts.

@sahilph
Copy link
Member Author

sahilph commented Jul 24, 2018

@empeje
Oh. This one wasn't merged yet 😅 ? I'll fix the conflicts and then you can merge it

@sahilph
Copy link
Member Author

sahilph commented Jul 24, 2018

@empeje I have fix the conflicts and also removed the versions which can't be compiled. Since the version is updated to 2.1.2 and 1.7.2 we should first test them on arm64 after the build finishes.

@empeje empeje self-assigned this Sep 13, 2018
@empeje
Copy link
Member

empeje commented Sep 13, 2018

Step to test:

  • Install Ubuntu on Raspberry Pi 3
  • Test it on Raspberry Pi 3

@empeje
Copy link
Member

empeje commented Nov 15, 2020

Hi @dogi, hope you're doing good. Since I'm no longer working on the treehouses project, could you please unassign me from this MR?

@empeje empeje removed their assignment Nov 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Aarch64 support
2 participants