Skip to content

MinIO update broke developer installation setup #8332

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

Closed
ruro opened this issue Jul 12, 2021 · 1 comment · Fixed by #8368
Closed

MinIO update broke developer installation setup #8332

ruro opened this issue Jul 12, 2021 · 1 comment · Fixed by #8368
Labels
Accepted Accepted issue on our roadmap Bug A bug

Comments

@ruro
Copy link

ruro commented Jul 12, 2021

Details

4 days ago, a new version of MinIO was released. It contains the following breaking change:

Embedded MinIO Browser removed replaced with MinIO Console project https://github.com/minio/console, this is a breaking change

MinIO redirects browser access requests to the configured server port (i.e. 127.0.0.1:9000) to the configured Console port. MinIO uses the hostname or IP address specified in the request when building the redirect URI. The URL and port must be accessible by the client for the redirection to work

This breaks the current development setup in 2 ways:

  1. http://localhost:9000/ now redirects to http://localhost:<some_port>/ where <some_port> is dynamically generated at MinIO startup. Since this port isn't forwarded, you can't configure MinIO:

    WARNING: MINIO_ACCESS_KEY and MINIO_SECRET_KEY are deprecated.
             Please use MINIO_ROOT_USER and MINIO_ROOT_PASSWORD
    API: http://10.10.0.2:9000  http://127.0.0.1:9000 
    
    Console: http://10.10.0.2:<some_port> http://127.0.0.1:<some_port> 
    
    Documentation: https://docs.min.io
    
    WARNING: Console endpoint is listening on a dynamic port (<some_port>), please use --console-address ":PORT" to choose a static port.
    
  2. Even if you manually add the --console-address setting and forward the appropriate port, the instructions in step 7 still don't make sense, since there doesn't seem to be a static bucket anywhere and the UI is different.

@astrojuanlu
Copy link
Contributor

Thanks @ruro , I can reproduce. A quick solution can be to pin the minio version in https://github.com/readthedocs/common/:

diff --git a/dockerfiles/docker-compose.yml b/dockerfiles/docker-compose.yml
index 95c4865..7eafcfd 100644
--- a/dockerfiles/docker-compose.yml
+++ b/dockerfiles/docker-compose.yml
@@ -144,7 +144,7 @@ services:
       readthedocs:
 
   storage:
-    image: minio/minio
+    image: minio/minio:RELEASE.2021-06-17T00-10-46Z
     ports:
       - "9000:9000"
     environment:

@astrojuanlu astrojuanlu added the Bug A bug label Jul 12, 2021
@humitos humitos added the Accepted Accepted issue on our roadmap label Jul 12, 2021
humitos added a commit to readthedocs/common that referenced this issue Jul 12, 2021
Latest MinIO version requires some changes to use `--console-address`. Also, the
new version of MinIO will require to update the documentation from our
installation guide.

Let's pin it for now to fix the immediate issue and come back with the proper solution.

Solution taken from @astrojuanlu's comment readthedocs/readthedocs.org#8332 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Accepted issue on our roadmap Bug A bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants