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

Alpine docker image #3063

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

Alpine docker image #3063

wants to merge 17 commits into from

Conversation

roylez
Copy link

@roylez roylez commented Jan 9, 2022

I manage to build alpine image for huginn by removing mini_racer from Gemfile. Also there are other modifications just to suit my preference. Feel free to merge any part you think is useful.

Notable changes are:

  • mini_racer is removed, thus making it incapable of running javascript agents.
  • DATABASE_URL is the only way of configuring database connection. Default: sqlite3:/data/huginn.db
  • sqlite3 is the default database, but mysql2 and postgres are supported as well.
  • There is only one multi-process build.
  • Mysqld is removed, together with supervisord.
  • Database will not be seeded automatically, but migration will still be run each time container starts.

I tried building it on arm64, and while it did build, the image did not function well, workers run out of available database connections even if database is not sqlite.

An already built image is available at roylez/huginn.

@roylez
Copy link
Author

roylez commented Jan 24, 2022

I did not expect this got accepted. After all it has so many breaking changes.

Some updates here. I have been using arm64 build with sqlite3 for more than a week and there are no issues so far. The only problem with arm64 build is that there is no canned ruby-grpc binary for it. Building grpc from source takes hours of CI time to complete, but eventually it works. You may want to add the following to remove intermediate files generate when building grpc.

roylez@55eac40

Also, armv7 build still does not work for some reason that I cannot remember.

@yegle
Copy link

yegle commented Jun 20, 2022

Hmm, I'm very interested to use the part that uses sqlite3 as database. Would it be possible to split that part into a separate PR?

Thank you!

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.

None yet

3 participants