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

Always add the traefik label to map the port #26

Closed
YuukanOO opened this issue Jun 29, 2023 · 1 comment
Closed

Always add the traefik label to map the port #26

YuukanOO opened this issue Jun 29, 2023 · 1 comment
Labels
enhancement New feature or request released

Comments

@YuukanOO
Copy link
Owner

Some images do not use the EXPOSE instruction in a Dockerfile and without it, Traefik do not know how to forward the traffic to the container.

Since seelf rely on the port being defined in a compose file, we could easily add the needed label "traefik.http.services.service_name.loadbalancer.server.port=80" to prevent unreachable services.

@YuukanOO YuukanOO added the enhancement New feature or request label Jun 29, 2023
@YuukanOO YuukanOO added this to the Next milestone Jun 29, 2023
github-actions bot pushed a commit that referenced this issue Jul 4, 2023
## [1.0.3](v1.0.2...v1.0.3) (2023-07-04)

### Bug Fixes

* closes [#26](#26), always add the container target port using a traefik label ([e9cfabc](e9cfabc))
@YuukanOO
Copy link
Owner Author

YuukanOO commented Jul 4, 2023

🎉 This issue has been resolved in version 1.0.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@YuukanOO YuukanOO removed this from the Next milestone Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request released
Projects
None yet
Development

No branches or pull requests

1 participant