fixed deployment issue with xloader version 1.2.0 #802
Conversation
Dockerfile
Outdated
| @@ -1,4 +1,4 @@ | |||
| FROM ckan/ckan-dev:2.10.5-py3.10 | |||
| FROM --platform=linux/amd64 ckan/ckan-dev:2.10.5-py3.10 | |||
There was a problem hiding this comment.
Does this build on M-series Macs? I've been struggling with this separately and I don't know the best way to support multiple platform architectures in code.
There was a problem hiding this comment.
Yes, it works on M-series Macs. This flag specifies the target platform of the container (Linux in this case), not the host OS where Docker is running.
There was a problem hiding this comment.
I removed this setting to avoid complexity, as individual users can configure it in their local environment.
Dockerfile
Outdated
| @@ -1,4 +1,4 @@ | |||
| FROM ckan/ckan-dev:2.10.5-py3.10 | |||
| FROM --platform=linux/amd64 ckan/ckan-dev:2.10.5-py3.10 | |||
There was a problem hiding this comment.
can you move this to an env var instead? DOCKER_DEFAULT_PLATFORM
There was a problem hiding this comment.
I removed this setting to avoid complexity, as individual users can configure it in their local environment.
fixed deployment issue with xloader version 1.2.0
Pull Request