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

Fix potential version mismatch in v2.10 update instructions #10712

Closed
wants to merge 1 commit into from

Conversation

sreeram-venkitesh
Copy link

What does this PR do?

Fixes a potential error in the docs for instructions to upgrade to Traefik v2.10.

Motivation

It is mentioned that traefik.io/v1alpha1 API group will not be supported in Traefik v3. I believe this was supposed to be traefik.containo.us/v1alpha1 since that API group is getting deprecated in v2.10 and later removed in v3.0. The upgradation instruction can be misleading for users trying to upgrade to v2.10 before upgrading to v3.

More

  • Added/updated tests
  • Added/updated documentation

Additional Notes

Please let me know if I should change the base branch of the PR to v2.11 branch. When I had tried that as per the instruction in the PR, I saw a lot of changes show up in my PR.

@ldez
Copy link
Member

ldez commented May 11, 2024

Hello,

Can you rebase your branch against the branch v2.11?

@sreeram-venkitesh
Copy link
Author

While rebasing the branch I noticed that this is fixed in #10658. But for some reason this has not reflected in the documentation website. That PR was made directly to the v2.11 branch and when I rebase my PR, the change is already there. I think we can close this PR and look into why the v2.11 branch is not deployed yet.

CC: @stemar94

@ldez
Copy link
Member

ldez commented May 12, 2024

Duplicate of #10658

It will backported to v3.0 and master when we will do the merge back of the branch v2.11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants