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

DOCS-910: Updating proxy and console configurations #921

Merged
merged 3 commits into from
Jul 21, 2023
Merged

Conversation

ravindk89
Copy link
Collaborator

@ravindk89 ravindk89 commented Jul 17, 2023

Summary

Closes #910

So NGINX is not my strong suit, but enough of our users use it, so here we go.

minio/console#2818 <- added better support to subpath behaviors

I also recall there being an issue related to how the upstream blocks were configured, but I can't find it now. Fixed that while I was here.

Finally, documented MINIO_DOMAINS, but did not fully test it as I don't have the stretch to set up apps and poke around that behavior.

Local environment

  • Latest Stable NGINX on CentOS Stream
  • Disable SELinux, it only makes your life harder
  • MinIO latest stable

Set /etc/hosts to have the preferred main hostname for the server on both the VM and local machine

Set nginx to proxy to hostname/ and hostname/minio/ui for the S3 API and Console API respectively

Set MINIO_SERVER_URL and MINIO_BROWSER_REDIRECT_URL based on the new proxy URLs

Tested both Console and mc, was able to connect, create a bucket, copy an object, generate a presigned URL, and wget the object using that URL.

Staged

http://192.241.195.202:9000/staging/DOCS-910/linux/reference/minio-server/minio-server.html#envvar.MINIO_DOMAIN

http://192.241.195.202:9000/staging/DOCS-910/linux/reference/minio-server/minio-server.html#envvar.MINIO_BROWSER_REDIRECT_URL

http://192.241.195.202:9000/staging/DOCS-910/linux/reference/minio-server/minio-server.html#envvar.MINIO_SERVER_URL

http://192.241.195.202:9000/staging/DOCS-910/linux/integrations/setup-nginx-proxy-with-minio.html

@ravindk89 ravindk89 added the Review Assigned Reviewers Must LGTM To Close label Jul 17, 2023
Copy link
Collaborator

@feorlen feorlen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple suggestions for your consideration, looks good 👍🏻

@ravindk89
Copy link
Collaborator Author

Ping to engineers, I'm going to merge this EoD absent any other commentary.

@ravindk89 ravindk89 merged commit 975cbeb into main Jul 21, 2023
@ravindk89 ravindk89 deleted the DOCS-910 branch July 21, 2023 21:41
@ravindk89 ravindk89 mentioned this pull request Aug 2, 2023
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Review Assigned Reviewers Must LGTM To Close
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve NGINX integration doc, clarify hostname/URL envvars
2 participants