Skip to content

Comments

Use traefik as a drop in replacement for ingress-nginx.#59

Open
paulbsch wants to merge 1 commit intomainfrom
FORGEOPS-6295-nginx-ingress
Open

Use traefik as a drop in replacement for ingress-nginx.#59
paulbsch wants to merge 1 commit intomainfrom
FORGEOPS-6295-nginx-ingress

Conversation

@paulbsch
Copy link
Collaborator

ref: FORGEOPS-6295

@paulbsch paulbsch requested review from damonconway, gandru, lee-baines and shankar-forgerock and removed request for damonconway February 18, 2026 19:25
@shankar-forgerock
Copy link
Collaborator

shankar-forgerock commented Feb 18, 2026

@lee-baines , @paulbsch ,
From the documentation perspective, I would like to know:

  1. Will traefik be considered the default ingress controller?
  2. Will it be provided by default in our Terraform manifests - similar previously to NGINX?
  3. Will forgeops prereq command include ways to install / enable it?
  4. Should it be in Tech Preview of the docs or just simply replace NGINX in the main doc?

In addition:
Would be using the latest (3.6) version?
Is there a Helm chart for this?

@lee-baines
Copy link
Collaborator

@lee-baines , @paulbsch , From the documentation perspective, I would like to know:

  1. Will traefik be considered the default ingress controller?
  2. Will it be provided by default in our Terraform manifests - similar previously to NGINX?
  3. Will forgeops prereq command include ways to install / enable it?
  4. Should it be in Tech Preview of the docs or just simply replace NGINX in the main doc?

In addition: Would be using the latest (3.6) version? Is there a Helm chart for this?

@shankar-forgerock Lets discuss this in Slack

dashboard:
enabled: false
providers:
kubernetesIngressNginx:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should this still be kubernetesIngressNginx?


resource "helm_release" "ingress_nginx" {
count = local.deploy_ingress_nginx ? 1 : 0
resource "kubernetes_ingress_class" "nginx" {
Copy link
Collaborator

Choose a reason for hiding this comment

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

"nginx"?

service.beta.kubernetes.io/aws-load-balancer-additional-resource-tags: cluster=${local.cluster_name}
annotations:
service.beta.kubernetes.io/aws-load-balancer-type: external
#service.beta.kubernetes.io/aws-load-balancer-type: nlb
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this commented out because it uses nlb by default? If not, then is the following line valid: service.beta.kubernetes.io/aws-load-balancer-nlb-target-type: ip ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants