-
Notifications
You must be signed in to change notification settings - Fork 13
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
Update Docker Image to Ubuntu 22.04 #74
Conversation
@@ -36,7 +36,7 @@ RUN apt-get update -y && \ | |||
ln -fs /usr/share/zoneinfo/UTC /etc/localtime && \ | |||
dpkg-reconfigure -f noninteractive tzdata && \ | |||
apt-get install -y python3 python3-psycopg2 python3-pip \ | |||
python3-venv python3-dev python3-apt postgresql-12 \ | |||
python3-venv python3-dev python3-apt postgresql-14 \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this might be an issue since we have issue with using postgres14 on ARM (psycopg hasn't shipped an 14 compatible binary for ARM arch)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bryanculver I tested the build and a basic run on my M1 MBP this morning. Are there additional tests that I should perform?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just checked this not too long ago today and forgot to follow up. Since my PR many moons ago to psycopg it looks like they have updated. 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry I had forgotten to follow up to my original comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
closes #61
In my testing, bumping up to Ubuntu 22.04 in the docker image resolves the issue with building on the Apple ARM architecture.