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

logstream-leader ingress error #92

Closed
bdalpe opened this issue Jan 9, 2023 · 5 comments · Fixed by #169
Closed

logstream-leader ingress error #92

bdalpe opened this issue Jan 9, 2023 · 5 comments · Fixed by #169
Assignees
Labels
bug Something isn't working

Comments

@bdalpe
Copy link
Contributor

bdalpe commented Jan 9, 2023

error retrieving endpoint for logstream-leader. no endpoints for target port 9000 in server logstream-leader
@bdalpe bdalpe added the bug Something isn't working label Jan 9, 2023
@bdalpe bdalpe self-assigned this Jan 9, 2023
@bdalpe bdalpe closed this as completed in 9368af0 Jan 11, 2023
@Kavinraja-G
Copy link

@bdalpe -- With /* as prefix I'm facing the below error when deploying with AWS ALB ingress controller.
Failed build model due to ingress: test/test-leader: prefix path should not contain wildcards: /*

Looks like the ingress defaults to pathType as Prefix which will not work with wildcards. So we need to override it with ImplementationSpecific

Refer issue here

Can we include that field customizable?

@bdalpe
Copy link
Contributor Author

bdalpe commented Jan 21, 2023

Hi @Kavinraja-G, I addressed this in #82. You can set the prefix desired in your Helm values file. This chart will continue to use /* as the default for backwards compatibility.

ingress:
  prefix: /

@brittonhayes
Copy link

Hey @bdalpe! 👋 I believe that @Kavinraja-G was hoping that the setting pathType could be provided as a helm value so users could specify other pathTypes such as ImplementationSpecific.

I don't believe PR #82 solved that request.

Would you be able to add support for customizing pathType?

Reference

@bdalpe
Copy link
Contributor Author

bdalpe commented Aug 16, 2023

I see. Happy to surface this setting into the values.

@bdalpe bdalpe reopened this Aug 16, 2023
@brittonhayes
Copy link

Thanks so much! Appreciate your fast reply and all your work to maintain this repository.

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.

3 participants