Skip to content

Annotation conversion guidance for SSL and gRPC backend (upstream) services #886

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

jnewfield
Copy link
Contributor

@jnewfield jnewfield commented Jul 23, 2025

Proposed changes

Problem: Add annotation conversion guidance for SSL and gRPC backend (upstream) services from Kubernetes Ingress-NGINX Controller to F5 NGINX Ingress Controller

Solution: Document conversions

Testing: I implemented F5 NGINX Ingress Controller annotations that are being documented in lab NIC environment to confirm they function as expected

Checklist

Before merging a pull request, run through this checklist and mark each as complete.

  • I have read the contributing guidelines
  • I have signed the F5 Contributor License Agreement (CLA)
  • I have rebased my branch onto main
  • I have ensured my PR is targeting the main branch and pulling from my branch from my own fork
  • I have ensured that the commit messages adhere to Conventional Commits
  • I have ensured that documentation content adheres to the style guide
  • If the change involves potentially sensitive changes1, I have assessed the possible impact
  • If applicable, I have added tests that prove my fix is effective or that my feature works
  • I have ensured that existing tests pass after adding my changes
  • If applicable, I have updated README.md

Footnotes

  1. Potentially sensitive changes include anything involving code, personally identify information (PII), live URLs or significant amounts of new or revised documentation. Please refer to our style guide for guidance about placeholder content.

@jnewfield jnewfield requested a review from a team as a code owner July 23, 2025 22:15
@github-actions github-actions bot added documentation Improvements or additions to documentation product/nic NGINX Ingress Controller related issues labels Jul 23, 2025
@ADubhlaoich ADubhlaoich requested a review from a team July 24, 2025 12:10
Copy link
Contributor

@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.

I have removed the internal resource link from the PR description.

Could you reformat these snippets into a table?

There are some other small style conventions I can address as suggestions afterwards.

I would also like a different PR title, as the current one had no indication of what the context of the PR would be.

You might find our Commit messages guidelines useful.

@ADubhlaoich
Copy link
Contributor

On further investigation, I'm curious what the intended use case for this specific detail is.

We have a Migrate from Ingress-NGINX Controller document that already include a table mapping the Ingress-NGINX resources: is there a reason these could not be added to the existing table?

@jnewfield jnewfield changed the title Changes have been made Annotation conversion guidance for SSL and gRPC backend (upstream) services Jul 24, 2025
@jnewfield
Copy link
Contributor Author

I thought about putting these annotations in the existing table. However, the proposed annotation conversions are predicated on corresponding values. The tables contain just annotations without values while configuration snippets contain annotations and values. For example, implementing just nginx.ingress.kubernetes.io/backend-protocol to nginx.org/ssl-services would not provide the full configuration requirements for a successful conversion. This is what we saw with customer in internal case 00898038.

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 NGINX Ingress Controller related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants