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
build: bump traefik to 2.11 #5833
Conversation
e8ee541
to
04639c2
Compare
Thanks for tracking this @nilsandresen ! We can sure try this out. It does require pushing an image. I'll see if I can do that. I'm interested to know why you're paying such good attention. Is there something you're looking for in 2.11? (I saw the release go by, but didn't read carefully and actually thought we were tracking 2.x) |
04639c2
to
ffe2b50
Compare
Rebased, updated reference, pushed the image, tests are running, thanks. |
WalkthroughThe recent update involves upgrading the Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 2
Configuration used: CodeRabbit UI
Files selected for processing (2)
- containers/ddev-traefik-router/Dockerfile (1 hunks)
- pkg/versionconstants/versionconstants.go (1 hunks)
Additional comments: 3
containers/ddev-traefik-router/Dockerfile (2)
- 1-1: Ensure all dependent configurations and features are compatible with Traefik version 2.11.
- 2-2: Setting
TRAEFIK_MONITOR_PORT
to10999
is clear. Verify this port does not conflict with other services.pkg/versionconstants/versionconstants.go (1)
- 27-27: The update to
TraefikRouterImage
is consistent with the Traefik version upgrade. Verify its usage across the project.
@@ -1,4 +1,4 @@ | |||
FROM traefik:2.10 | |||
FROM traefik:2.11 | |||
|
|||
ENV TRAEFIK_MONITOR_PORT=10999 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Installing bash
, curl
, htop
, and vim
increases utility but also image size. Consider if all are necessary.
Download the artifacts for this pull request:
See Testing a PR |
@rfay We are using traefik as a router for some docker containers and I ran updates there today. Then I remembered the ddev router runs on traefik aswell and I thought why not open a PR. |
Thanks @nilsandresen - Do you do any custom config of traefik in DDEV? I always love to hear how people are using this feature. |
I also see that 3.0.0 is at rc1 status, and wonder what that will mean to us. We may want to get 3.0.0 in before DDEV v1.23.0, which is coming in the next couple of months. |
In DDEV not, I use traefik outside of DDEV aswell. I'm using it as reverse proxy with automatic SSL Certs. |
Traefik 3 hast been cooking for a long time and I don't think it will be released soon. |
One of the key things that hasn't been studied in DDEV with traefik is using Let's Encrypt, so the old nginx-proxy router recipe is still what I and other people are using. Would love to have you take a look at this if you're interested, https://ddev.readthedocs.io/en/latest/users/topics/hosting/ |
Yeah I might take a look. If I found out something interesting should I share it on discord, or a discussion? |
Just open a PR! Thanks! Happy to discuss it anywhere though. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, tests are good, thanks!
The Issue
Traefik got updated to version 2.11.
Summary by CodeRabbit
TRAEFIK_MONITOR_PORT
set to10999
.bash
,curl
,htop
,vim
) for better troubleshooting and debugging experience.