Skip to content
This repository was archived by the owner on Aug 6, 2025. It is now read-only.

DOP-5490: Add trailing slashes to redirects for kubernetes-operator#1089

Merged
rayangler merged 2 commits intomainfrom
DOP-5490
Mar 13, 2025
Merged

DOP-5490: Add trailing slashes to redirects for kubernetes-operator#1089
rayangler merged 2 commits intomainfrom
DOP-5490

Conversation

@rayangler
Copy link
Copy Markdown
Contributor

@rayangler rayangler commented Mar 13, 2025

Stories/Links:

DOP-5490

Notes

  • Attempts to add trailing slashes to the redirects to avoid having examples like "v1.31" from being caught in redirects away from "v1.3".
  • We'll test in preprd first and roll back as needed.

README updates

    • This PR introduces changes that should be reflected in the README, and I have made those updates.
    • This PR does not introduce changes that should be reflected in the README

ReplaceKeyPrefixWith: ${self:custom.site.prefix.${self:provider.stage}}/kubernetes-operator/current
- RoutingRuleCondition:
KeyPrefixEquals: ${self:custom.site.prefix.${self:provider.stage}}/kubernetes-operator/v1.3
KeyPrefixEquals: ${self:custom.site.prefix.${self:provider.stage}}/kubernetes-operator/v1.3/
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Didn't see one for v1.2 but redirects seem to work fine due to existing object-level redirects?

@branberry branberry self-requested a review March 13, 2025 19:19
Copy link
Copy Markdown
Contributor

@mmeigs mmeigs left a comment

Choose a reason for hiding this comment

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

🌟

@rayangler rayangler merged commit 49efebb into main Mar 13, 2025
9 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants