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 Dockerfile-setup #2773
Update Dockerfile-setup #2773
Conversation
RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y --no-install-recommends \ | ||
ca-certificates \ | ||
curl \ | ||
mysql-client \ | ||
mariadb-client \ |
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.
Why changing to mariadb
? 🤔
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.
Because mysql wasn't available
# RUN curl https://packages.microsoft.com/keys/microsoft.asc | apt-key add - && \ | ||
# RUN curl https://packages.microsoft.com/config/debian/9/prod.list > /etc/apt/sources.list.d/mssql-release.list && \ | ||
# RUN apt-get update && \ | ||
# ACCEPT_EULA=Y apt-get install -y msodbcsql mssql-tools |
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.
Why did you uncomment these lines? 🤔
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.
Consistency w/ the other file. If they really shouldn't be there, then why were they there?
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.
then why were they there?
No idea. That's always the issue with committed commented code 😕
MAINTAINER gustavo.amigo@gmail.com | ||
|
||
ARG CASSANDRA_URL_BASE=https://archive.apache.org/dist/cassandra/ |
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.
Why did you change from https://downloads.apache.org/cassandra/
to https://archive.apache.org/dist/cassandra/
? 🤔
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.
Because the requested version wasn't available in downloads.
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.
@jsoref Thanks for your contribution! I asked you a few questions to better understand your changes 🙂
* Upgrade debian from jessie to bullseye * Switch from mysql-client to mariadb-client * Change from msodbcsql to msodbcsql18 * Replace deprecated apt-key with a trusted.gpg.d entry * Update cassandra url (and use an arg to reduce duplication) * Upgrade cassandra to 3.11.13 * Mostly sync m1 and non-m1 Dockerfile
06b24b5
to
1348998
Compare
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.
LGTM
* Upgrade debian from jessie to bullseye * Switch from mysql-client to mariadb-client * Change from msodbcsql to msodbcsql18 * Replace deprecated apt-key with a trusted.gpg.d entry * Update cassandra url (and use an arg to reduce duplication) * Upgrade cassandra to 3.11.13 * Mostly sync m1 and non-m1 Dockerfile
Problem
Neither dockerfile works.
Solution
Notes
Additional notes.
Checklist
README.md
if applicable[WIP]
to the pull request title if it's work in progresssbt scalariformFormat test:scalariformFormat
to make sure that the source files are formatted@getquill/maintainers