Skip to content

Conversation

@catundercar
Copy link

Proposed changes

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.

@catundercar catundercar requested a review from a team as a code owner December 24, 2025 15:27
@github-actions
Copy link

github-actions bot commented Dec 24, 2025

✅ All required contributors have signed the F5 CLA for this PR. Thank you!
Posted by the CLA Assistant Lite bot.

@github-actions github-actions bot added documentation Improvements or additions to documentation product/ngf Issues related to NGINX Gateway Fabric labels Dec 24, 2025
@catundercar
Copy link
Author

I have hereby read the F5 CLA and agree to its terms

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, though it's the engineering team's call on correctness acceptance.

@sjberman
Copy link
Contributor

The original ports work for me when going through the guide. The updated ports do not.

The purpose of the kind config at the start is to forward the 31437 port to your 8080 host port, so you can access locally on 8080.

@catundercar
Copy link
Author

sorry, I don't run through kind, so I can't access 8080.

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/ngf Issues related to NGINX Gateway Fabric

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants