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.

Sign up for GitHub

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

Jump to bottom

Enabling nodeport breaks the headless service #163

Closed
jonathon2nd opened this issue Dec 28, 2023 · 2 comments
Closed

Enabling nodeport breaks the headless service #163

jonathon2nd opened this issue Dec 28, 2023 · 2 comments

Comments

@jonathon2nd
Copy link
Contributor

jonathon2nd commented Dec 28, 2023 •
edited
Loading

Hello! I'm back :D

Enabling nodeport causes the deployment to fail, as the headless service has the nodeport if enabled. Which does not work IIRC.

values

...
service:
  type: NodePort
  nodePort: 30027
...

image

image
@jonathon2nd
Copy link
Contributor Author

What I have done for now is remove the nodeport from values, and manually set it up. That way I can go ahead and switch over to upstream helm and get everything tested.

bokysan added a commit that referenced this issue Dec 29, 2023
@bokysan
Fix for #163: Remove NodePort from headless service
b317e68
@bokysan
Copy link
Owner

bokysan commented Dec 29, 2023

You are correct -- Fixed in commit b317e68e28d449191b3d58dfbe00c66085f1986b.

@bokysan bokysan closed this as completed Dec 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bokysan @jonathon2nd