Skip to content

Conversation

@MAVRICK-1
Copy link
Member

@MAVRICK-1 MAVRICK-1 commented Dec 27, 2025

This comprehensive migration replaces all NGINX Ingress resources with Gateway API equivalents:

Key Changes:

  • Replace NGINX Ingress Controller with NGINX Gateway Fabric v2.3.0
  • Implement Gateway API v1.4.1 with Gateway, GatewayClass, and HTTPRoute resources
  • Add Gateway API reconcilers in pkg/reconcilers/shared/gateway.go
  • Update all control plane reconcilers (k8s, k3s, ocm, vcluster) to use Gateway API
  • Replace ingress.go with gateway.go across reconciler packages
  • Update installation scripts to use NGINX Gateway Fabric
  • Add HTTPS support with both HTTP (80) and HTTPS (443) listeners
  • Remove all NGINX Ingress dependencies and resources
  • Update documentation and test scripts

Fixes #601

@kubestellar-prow kubestellar-prow bot added the dco-signoff: no Indicates the PR's author has not signed the DCO. label Dec 27, 2025
@kubestellar-prow
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign pdettori for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubestellar-prow kubestellar-prow bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Dec 27, 2025
@MAVRICK-1 MAVRICK-1 force-pushed the gateway-api-migration branch from 9121790 to cd8628d Compare December 27, 2025 15:55
@kubestellar-prow kubestellar-prow bot added dco-signoff: yes Indicates the PR's author has signed the DCO. and removed dco-signoff: no Indicates the PR's author has not signed the DCO. labels Dec 27, 2025
This comprehensive migration replaces all NGINX Ingress resources with Gateway API equivalents:

Key Changes:
- Replace NGINX Ingress Controller with NGINX Gateway Fabric v2.3.0
- Implement Gateway API v1.4.1 with Gateway, GatewayClass, and HTTPRoute resources
- Add Gateway API reconcilers in pkg/reconcilers/shared/gateway.go
- Update all control plane reconcilers (k8s, k3s, ocm, vcluster) to use Gateway API
- Replace ingress.go with gateway.go across reconciler packages
- Update installation scripts to use NGINX Gateway Fabric
- Add HTTPS support with both HTTP (80) and HTTPS (443) listeners
- Remove all NGINX Ingress dependencies and resources
- Update documentation and test scripts
- Add necessary Gateway API RBAC permissions

Signed-off-by: mavrick-1 <[email protected]>
@MAVRICK-1 MAVRICK-1 force-pushed the gateway-api-migration branch from cd8628d to b087216 Compare December 27, 2025 15:56
@MAVRICK-1 MAVRICK-1 changed the title Migrate from NGINX Ingress to Gateway API with NGINX Gateway Fabric ✨ Migrate from NGINX Ingress to Gateway API with NGINX Gateway Fabric Dec 27, 2025
@MAVRICK-1
Copy link
Member Author

cc @clubanderson @MikeSpreitzer

@clubanderson
Copy link
Contributor

clubanderson commented Jan 5, 2026

@francostellari @MikeSpreitzer can you ptal? Maybe we can avoid the nginx support issues sooner than expected.

@MikeSpreitzer
Copy link
Contributor

We need evidence that this works.

@MikeSpreitzer
Copy link
Contributor

MikeSpreitzer commented Jan 7, 2026

KubeFlex needs the routing into the hosting cluster to support TLS passthrough (https://github.com/kubestellar/kubeflex/blob/main/docs/users.md#install-kubeflex-on-an-existing-cluster). If I understand correctly, the Gateway API provides that only through a TLSRoute object --- which is not in the standard channel of the Gateway API.

@shivansh-source
Copy link
Contributor

@MAVRICK-1 status ?

@kubestellar-prow kubestellar-prow bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 12, 2026
@clubanderson
Copy link
Contributor

Hi @francostellari, this PR needs a rebase to resolve conflicts with the main branch. Could you please rebase and push?

git fetch origin
git rebase origin/main
git push --force-with-lease

Thanks!

@clubanderson
Copy link
Contributor

Hi @MAVRICK-1 👋

This PR needs a rebase to resolve conflicts with the main branch. You can do this with:

git fetch origin
git rebase origin/main
# Resolve any conflicts if needed
git push --force-with-lease

Once rebased, the CI checks will run again. Let us know if you need any help!

@MAVRICK-1
Copy link
Member Author

Hi @MAVRICK-1 👋

This PR needs a rebase to resolve conflicts with the main branch. You can do this with:

git fetch origin
git rebase origin/main
# Resolve any conflicts if needed
git push --force-with-lease

Once rebased, the CI checks will run again. Let us know if you need any help!

@clubanderson is that u , or AI generated ?

@clubanderson clubanderson added the kind/feature Categorizes issue or PR as related to a new feature. label Jan 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dco-signoff: yes Indicates the PR's author has signed the DCO. kind/feature Categorizes issue or PR as related to a new feature. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

update: Migrate from NGINX Ingress to Gateway API with NGINX Gateway Fabric

4 participants