Skip to content
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

New Docker development environment #7

Open
wants to merge 34 commits into
base: master
Choose a base branch
from
Open

New Docker development environment #7

wants to merge 34 commits into from

Conversation

JoostSFS
Copy link

@JoostSFS JoostSFS commented Aug 5, 2022

No description provided.

sahithyaravi and others added 30 commits December 23, 2021 00:40
…into portml

* 'portml' of https://github.com/openml/openml-docker-dev:
  Removed quotation marks in http.cors.allow-headers: which are preventing the correct CORS policy to be set
…ations and data volumes are created for docker and can be used in AWS
New website is now copied from external folder in order to quickly change front-end using docker-compose args variable -sourceDirectory
…be erased by Slack also updated the AWS template and Dockerfile
@prabhant
Copy link

prabhant commented Oct 4, 2022

Are you planning to include MinIO in this docker image as well?

@joaquinvanschoren
Copy link
Contributor

For context: when requesting datasets, the API gives links to parquet files which (currently) depend on a minio service.
How is this handled in the docker environment? Having a simple minio service in the docker environment would be great since it would most closely mimic what we use in production.

@MatsPortML
Copy link

Comments regarding MinIO: This is something we did not consider at the moment of this PR. But have been looking at it more recently. It would be great to have a small meeting to discuss how MinIO is used in the production environment for OpenML, based on that we can then implement it for the docker env and for PortML.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants