moan0s
commented
1 year ago Yes, that's a serious issue. Did you have any success solving this?
Closed olivercoad closed 1 year ago
moan0s
commented
1 year ago Yes, that's a serious issue. Did you have any success solving this?
olivercoad
commented
1 year ago By manually building a previous version.
On Thu., 15 Dec. 2022, 04:04 Julian-Samuel Gebühr, @.***> wrote:
Yes, that's a serious issue. Did you have any success solving this?
— Reply to this email directly, view it on GitHub https://github.com/moan0s/matrix-registration-bot/issues/24#issuecomment-1351778768, or unsubscribe https://github.com/notifications/unsubscribe-auth/AF5P62VEYJFOP4XQIRAPKMDWNH4YZANCNFSM6AAAAAASEI5KGA . You are receiving this because you authored the thread.Message ID: @.***>
SapuSeven
commented
1 year ago If you still have problems, use the following patch to fix the current version:
diff --git a/Dockerfile b/Dockerfile
index 934616c..5a53d37 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -1,7 +1,7 @@
FROM python:3-slim AS compile-image
MAINTAINER Julian-Samuel Gebühr
-RUN apt-get update && apt-get install -y --no-install-recommends build-essential gcc
+RUN apt-get update && apt-get install -y --no-install-recommends build-essential gcc libolm-dev
RUN python -m venv /opt/venv
RUN /opt/venv/bin/pip install --no-cache-dir matrix-registration-botAs far as I can see, develop already has those changes, so if you are using matrix-docker-ansible-deploy you can just add the following line to your vars.yml for now:
matrix_bot_matrix_registration_bot_docker_repo_version: developI hope we can merge and release a new version soon to fix this issue.
moan0s
commented
1 year ago I think this is fixed with 3262c0d116e74bdc59d681d5e95bac940be39b01 if not feel free to reopen :)
Mawarii
commented
11 months ago I'm still getting an error while building the Dockerfile. Still a python-olm issue. I tried to build it on arm64 and amd64 but had no luck. So I tried to fix it. And it seems that only the python3 images have those problems. So I switched to a plain debian (maybe there is a better choice) and I was able to build it on arm64 and amd64 with no issue.
FROM debian:12-slim AS compile-image
MAINTAINER Julian-Samuel Gebühr
RUN apt-get update && apt-get install -y --no-install-recommends python3 python3-venv python3-dev build-essential gcc libolm-dev
RUN python3 -m venv /opt/venv
WORKDIR /app
COPY requirements.txt ./
RUN /opt/venv/bin/pip install -r requirements.txt
COPY . .
RUN /opt/venv/bin/pip install .
RUN /opt/venv/bin/pip install matrix-nio==0.20.2
FROM debian:12-slim
RUN apt-get update && apt-get install -y python3 python3-dev libolm-dev
COPY --from=compile-image /opt/venv /opt/venv
VOLUME ["/data"]
WORKDIR /data
CMD ["/opt/venv/bin/matrix-registration-bot"]OK the build works, but the image is running into issues. I don't know why. I will try to investigate more in the next days.
Hi,
This doesn't docker build on version 1.2.2. It has an error when trying to setup python-olm
This means it is broken with
matrix-docker-ansible-deployonarm64because the only image published is foramd64, so it builds the latest version instead of pulling the pre-build docker image.