Skip to content

Conversation

@ranchodeluxe
Copy link
Contributor

@ranchodeluxe ranchodeluxe commented Jun 20, 2023

What I am changing

Nginx Ingress Controller with single NLB and path rewrites which addreses #4

How I did it

It took awhile but all the magic was (of course) in annotations and applying them during helm install:

helm upgrade --cleanup-on-fail \
  -i nginx \
  ingress-nginx/ingress-nginx \
  --set controller.service.type=LoadBalancer \
  --set controller.service.annotations."service\.beta\.kubernetes\.io/aws-load-balancer-type"="nlb"  \
  --set controller.service.annotations."service\.beta\.kubernetes\.io/aws-load-balancer-scheme"="internet-facing" \
  --namespace gcorradini

How you can test it

I can give you the url for my dev cluster to test on

@ranchodeluxe ranchodeluxe requested a review from ividito June 20, 2023 00:47
@ranchodeluxe ranchodeluxe added this to the AWS EKS IAC milestone Jun 20, 2023
@ranchodeluxe
Copy link
Contributor Author

Screen Shot 2023-06-20 at 12 19 57 PM

@ranchodeluxe
Copy link
Contributor Author

Note that the links are now prepended with /stac root path:

Screen Shot 2023-06-20 at 1 16 05 PM

@ranchodeluxe
Copy link
Contributor Author

to keep on track with deadlines gonna merge this. if there are any issues we can talk about them async and loop back around to fix things

@ranchodeluxe ranchodeluxe merged commit c256b8f into main Jun 26, 2023
@j08lue j08lue deleted the feature/nginx-controller branch October 7, 2024 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants