-
Notifications
You must be signed in to change notification settings - Fork 24
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
feat: e2e Docker OS support #3341
Conversation
9ee1897
to
e6ff453
Compare
0e67040
to
a241abc
Compare
- OpenSSL location: `/lib/libssl.so.1.1` | ||
- Binary target: `linux-musl` | ||
|
||
Although Linux Alpine 3.17+ upgraded to OpenSSL 3, this example uses the `openssl1.1-compat` package to install OpenSSL 1.1. |
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.
Is the folder name then not incorrect/misleading?
# Update system dependencies. | ||
# OpenSSL is missing in the base image, so we install it | ||
RUN apt-get update -y \ | ||
&& apt-get install -y openssl |
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.
Does this reliably pin to OpenSSL 1.1?
@@ -0,0 +1,17 @@ | |||
{ | |||
"name": "@prisma/debian-buster-amd64-openssl-undefined", |
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.
Long term Prisma should throw an installation when no OpenSSL is found I assume instead of failing later?
Follow-up of #3340, pointing to the
dev
branch.Contributes to prisma/prisma#14017, #3332, prisma/prisma#16962, prisma/docs#4303
This PR adds Dockerised OS Support tests for in the
./docker
folder. In particular, it tries to run Prisma Client (both binary and library) in the following systems:Additionally, for each Dockerised system, we assert the
binaryTarget
selected by Prisma (i.e., theCurrent platform
value inprisma -v
). If a major Docker image suddenly introduces a new OpenSSL version we don't support, or switches to another architecture by default, we will receive a Slack notification.Multiarchitecture support (
amd64
,arm64
) works via QEMU (docker/setup-qemu-action@v2
) and Docker Buildx (docker/setup-buildx-action@v2
).Review Helpers
.yaml
,.sh
No extension
.md
.md
,.yaml
,.sh
,No extension
TODOs
From these tests, I have opened the following issues:
We should:
debian-latest-arm64-openssl-1.1.x
to the Docker test pipeline once https://github.com/prisma/prisma-private/issues/205 is solved. Done in: ci: add debian-latest-arm64-openssl-1.1.x to docker workflow #3342docker
workflow only once per day_fail-debian-buster-amd64-openssl-1.1.x