Skip to content
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

Litmus does not honor service.type field when ingress is enabled #256

Closed
dchourasia opened this issue Aug 1, 2022 · 1 comment · Fixed by #257
Closed

Litmus does not honor service.type field when ingress is enabled #256

dchourasia opened this issue Aug 1, 2022 · 1 comment · Fixed by #257
Assignees
Labels
bug Something isn't working

Comments

@dchourasia
Copy link

dchourasia commented Aug 1, 2022

We are hosting litmus on an kubernetes cluster on AWS (EKS) and we want to expose an external dns for litmus in order for us to access it from outside.

We are using aws ingress controller for this (aws load balancer controller), but it expects services to be NodePort as a prerequisite for creating the application load balancer (alb) and external dns

Here is the doc mentioning this : https://kubernetes-sigs.github.io/aws-load-balancer-controller/v2.4/guide/ingress/spec/
The service, service-2048, must be of type NodePort in order for the provisioned ALB to route to it.

But we observed that litmus forces services to be ClusterIP if ingress is enabled, it does not honor service.type fields provided in values.yaml while installing helm release when ingress is enabled.

here is link and snippet from one of the service chart:

  1. https://github.com/litmuschaos/litmus-helm/blob/master/charts/litmus/templates/frontend-svc.yaml
  2. https://github.com/litmuschaos/litmus-helm/blob/master/charts/litmus/templates/server-svc.yaml
    image
@Jasstkn
Copy link
Member

Jasstkn commented Aug 1, 2022

Nice catch. I'll fix it asap

@Jasstkn Jasstkn self-assigned this Aug 1, 2022
@Jasstkn Jasstkn added the bug Something isn't working label Aug 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants