Skip to content

Conversation

@vepatel
Copy link
Contributor

@vepatel vepatel commented Dec 9, 2025

Proposed changes

  • Updates ingress-nginx to nginx-ingress (F5 NIC) migration doc with new annotations added in v5.3.0

Checklist

Before sharing this pull request, I completed the following checklist:

Footnotes

  1. Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content.

@github-actions github-actions bot added documentation Improvements or additions to documentation product/nic Issues related to NGINX Ingress Controller labels Dec 9, 2025
@github-actions
Copy link

github-actions bot commented Dec 9, 2025

Deploy Preview will be available once build job completes!

Name Link
😎 Deploy Preview https://frontdoor-test-docs.nginx.com/previews/docs/1553/

@vepatel vepatel marked this pull request as ready for review December 9, 2025 17:03
@vepatel vepatel requested a review from a team as a code owner December 9, 2025 17:03
Copy link
Member

@ADubhlaoich ADubhlaoich left a comment

Choose a reason for hiding this comment

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

Generally LGTM but the PR removes two annotations to the table on lines 476 and 486 you may want to re-add: the annotations are in the list underneath the table.

@vepatel vepatel merged commit 02c4046 into main Dec 9, 2025
9 checks passed
@vepatel vepatel deleted the nic/update-migration-doc branch December 9, 2025 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation product/nic Issues related to NGINX Ingress Controller

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants