Skip to content

HTTP routing with Traefik: Update guide to include DHI - #23769

Merged
craig-osterhout merged 16 commits into
docker:mainfrom
jarqvi:chore/guides
Feb 20, 2026
Merged

craig-osterhout merged 16 commits into
docker:mainfrom
jarqvi:chore/guides

Conversation

@jarqvi

@jarqvi jarqvi commented Nov 24, 2025

Copy link
Copy Markdown
Contributor

Description

Related issues or tickets

Reviews

  • Technical review
  • Editorial review
  • Product review

@netlify

netlify Bot commented Nov 24, 2025 •

Copy link
Copy Markdown

✅ Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 0155444
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/698f46d404d74e00087dbbaa
😎 Deploy Preview https://deploy-preview-23769--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@craig-osterhout
craig-osterhout requested a review from a team November 25, 2025 20:00
@craig-osterhout craig-osterhout added the status/review Pull requests that are ready for review label Nov 25, 2025
@craig-osterhout

craig-osterhout commented Dec 18, 2025 •

Copy link
Copy Markdown
Contributor

Thanks @jarqvi. Sorry, we had to quietly pause all reviews/changes for the surprise DHI free updates. Can you help update this based on that?
Essentially, for DHI, mirroring is no longer required, and users can directly use the public images at dhi.io/<image>:<tag> as long as they authenticate.

@jarqvi

jarqvi commented Dec 28, 2025

Copy link
Copy Markdown
Contributor Author

It’s done; I made some changes in this regard.

@craig-osterhout

@craig-osterhout craig-osterhout left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @jarqvi.
For the linter, any content/lines in between ordered steps needs to be indented. Otherwise lgtm.

For example:

1. Run this command:

  `some command`
  
  Then xyz will happen.

2. Run this other command:

  `abc`

3. ...

Comment thread content/guides/traefik.md Outdated
Comment thread content/guides/traefik.md Outdated
Comment thread content/guides/traefik.md Outdated
Comment thread content/guides/traefik.md Outdated
Comment thread content/guides/traefik.md Outdated
Comment thread content/guides/traefik.md Outdated
Comment thread content/guides/traefik.md Outdated
Comment thread content/guides/traefik.md Outdated
Comment thread content/guides/traefik.md Outdated
Signed-off-by: MohammadHasan Akbari <jarqvi.jarqvi@gmail.com>
Signed-off-by: MohammadHasan Akbari <jarqvi.jarqvi@gmail.com>
Signed-off-by: MohammadHasan Akbari <jarqvi.jarqvi@gmail.com>
Signed-off-by: MohammadHasan Akbari <jarqvi.jarqvi@gmail.com>
@jarqvi

jarqvi commented Feb 13, 2026

Copy link
Copy Markdown
Contributor Author

I’m sorry that my review of this was delayed.
All the requested changes have been fixed.

@craig-osterhout

Signed-off-by: MohammadHasan Akbari <jarqvi.jarqvi@gmail.com>
Signed-off-by: MohammadHasan Akbari <jarqvi.jarqvi@gmail.com>
Signed-off-by: MohammadHasan Akbari <jarqvi.jarqvi@gmail.com>
Signed-off-by: MohammadHasan Akbari <jarqvi.jarqvi@gmail.com>
Signed-off-by: MohammadHasan Akbari <jarqvi.jarqvi@gmail.com>
Signed-off-by: MohammadHasan Akbari <jarqvi.jarqvi@gmail.com>
@craig-osterhout
craig-osterhout self-requested a review February 13, 2026 18:41

@craig-osterhout craig-osterhout left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you @jarqvi. LGTM

@craig-osterhout
craig-osterhout merged commit 7e2af24 into docker:main Feb 20, 2026
15 checks passed
@jarqvi
jarqvi deleted the chore/guides branch February 21, 2026 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/guides status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants