Fix(mesh): policy versioning and structure#5635
Conversation
✅ Deploy Preview for kongdeveloper ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
⚠️ Not ready to approve
There are a few concrete doc correctness issues introduced in the updated lines (notably MeshExtenalService typos and a new_in badge placement that can break list rendering) that should be fixed before merging.
Pull request overview
This PR updates Kong Mesh policy documentation to remove if_version gating (per #5364) and standardize structure/formatting, primarily by converting legacy {% tabs %}/Markdown tables and callout blocks into the site’s navtabs and {% table %} components.
Changes:
- Replaced
if_version-gated support matrices withnavtabs+{% table %}blocks across Mesh policy pages. - Converted
{% warning %}/{% tip %}blocks to the{:.warning}/{:.info}callout format. - Simplified examples by removing version-conditional YAML snippets and normalizing demo namespace usage in policy examples.
File summaries
| File | Description |
|---|---|
| app/_mesh_policies/meshtrafficpermission/index.md | Reworks callouts and replaces version-gated support matrix with navtabs + {% table %}. |
| app/_mesh_policies/meshtrace/index.md | Converts warning blocks and rebuilds support matrix with navtabs + {% table %}. |
| app/_mesh_policies/meshtls/index.md | Replaces version-gated tables with navtabs + {% table %}. |
| app/_mesh_policies/meshtimeout/index.md | Removes if_version gating in support matrix and related sections; converts callouts. |
| app/_mesh_policies/meshtcproute/index.md | Removes if_version blocks and rebuilds support matrix using navtabs + {% table %}. |
| app/_mesh_policies/meshretry/index.md | Replaces version-gated support matrix with navtabs + {% table %}. |
| app/_mesh_policies/meshratelimit/index.md | Converts support matrix to navtabs + {% table %} and updates delegated gateway note formatting. |
| app/_mesh_policies/meshproxypatch/index.md | Converts support matrix and multiple environment tabsets from {% tabs %} to navtabs. |
| app/_mesh_policies/meshpassthrough/index.md | Converts support matrix and warnings; replaces version gating with new_in for protocol list. |
| app/_mesh_policies/meshopa/index.md | Replaces version-gated support matrix with a unified navtabs + {% table %} and converts env var table to {% table %}. |
| app/_mesh_policies/meshmetric/index.md | Removes multiple version-conditional sections; standardizes support matrix + callouts; updates examples to consistent namespaces. |
| app/_mesh_policies/meshloadbalancingstrategy/index.md | Converts support matrix and warnings; removes older version-specific sectioning. |
| app/_mesh_policies/meshidentity/index.md | Minor cleanup (removes extra blank line). |
| app/_mesh_policies/meshhttproute/index.md | Converts support matrix and notes; simplifies version-gated enum wording. |
| app/_mesh_policies/meshhealthcheck/index.md | Replaces support matrix with navtabs + {% table %} and removes version gating around fields. |
| app/_mesh_policies/meshglobalratelimit/index.md | Rebuilds support matrix using navtabs + {% table %} and updates example namespaces. |
| app/_mesh_policies/meshfaultinjection/index.md | Converts warning + support matrix to navtabs + {% table %} and removes version-specific variants. |
| app/_mesh_policies/meshcircuitbreaker/index.md | Replaces large duplicated/versioned support matrix sections with a single navtabs + {% table %} and converts multiple nested tabsets. |
| app/_mesh_policies/meshaccesslog/index.md | Converts support matrix and replaces Markdown tables with {% table %} blocks; removes if_version variants for formats/backends. |
| app/_mesh_policies/external-services/index.md | Converts environment tabs to navtabs and removes version-gated locality-aware load balancing links. |
Copilot's findings
- Files reviewed: 20/20 changed files
- Comments generated: 5
Note
Your feedback helps us improve the quality of this feature.
Please use 👍 or 👎 to tell us whether this assessment is correct.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: Lucie Milan <32450552+lmilan@users.noreply.github.com>
There was a problem hiding this comment.
⚠️ Not ready to approve
A few doc issues introduced here (self-referential legacy-policy links, an unpaired <!-- vale off -->, and a ConfigMap name typo) should be fixed to avoid misleading readers and suppressing linting.
Copilot's findings
- Files reviewed: 24/24 changed files
- Comments generated: 6
Note
Your feedback helps us improve the quality of this feature.
Please use 👍 or 👎 to tell us whether this assessment is correct.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Description
Fixes #5364
Fixes #3737
Preview Links
Checklist
descriptionentry in frontmatter.