Docker Image Change #1207
Docker Image Change #1207
Conversation
|
Please find the ci env pod logs here |
|
Please find the ci env pod logs here |
|
Please find the ci env pod logs here |
Dockerfile
Outdated
| libxcrypt-compat-4.4.33 \ | ||
| libpq-15.0 && \ | ||
| echo "Installing RPM"; \ | ||
| rpm -i https://ccrypt.sourceforge.net/download/1.11/ccrypt-1.11-1.x86_64.rpm; \ |
There was a problem hiding this comment.
I noticed we are using an x86 binary for ARM. Will this be emulated for ARM during use? Has any testing been done to confirm its functionality?
pyproject.toml
Outdated
| @@ -22,7 +22,7 @@ cached_property = "^1.5.1" | |||
| cdiserrors = "<2.0.0" | |||
| cdislogging = "^1.0.0" | |||
| cdispyutils = "^2.0.1" | |||
| flask = ">=3.0.0" | |||
| flask = "^2.2.3" | |||
There was a problem hiding this comment.
Could you clarify the reason for downgrading Flask?
|
Please find the ci env pod logs here |
|
Please find the ci env pod logs here |
|
Please find the ci env pod logs here |
BinamB
changed the title
|
Please find the ci env pod logs here |
|
Please find the ci env pod logs here |
Dockerfile
Outdated
| RUN poetry lock -vv --no-update \ | ||
| && poetry install -vv --only main --no-interaction | ||
|
|
||
| ENV PATH="$(poetry env info --path)/bin:$PATH" |
There was a problem hiding this comment.
Docker does not support SHELL expansion of variables in ENV clause. The value of $(poetry env info --path) is taken as a literal string and is prepended to $PATH.
bash-5.2# echo $PATH
$(poetry env info --path)/bin:/home/gen3/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/root/.local/bin
|
Please find the ci env pod logs here |
|
Please find the ci env pod logs here |
|
Please find the ci env pod logs here |
|
Please find the ci env pod logs here |
|
Please find the ci env pod logs here |
|
Please find the ci env pod logs here |
Related PR: https://github.com/uc-cdis/cloud-automation/pull/2682/files
Without the cloud-auto PR, we're not able to successfully deploy fence.
Improvements