Skip to content

Custom loadBalancerClass #285

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
Open
nightkr opened this issue Mar 17, 2025 · 1 comment · May be fixed by #288
Open

Custom loadBalancerClass #285

nightkr opened this issue Mar 17, 2025 · 1 comment · May be fixed by #288
Assignees
@nightkr @siegfriedweber
Labels
customer-request type/feature-improvement

Comments

@nightkr
Copy link
Member

nightkr commented Mar 17, 2025

See Service.spec.loadBalancerClass, this should ideally be configured on the ListenerClass.

This would be useful for both:

  • Clusters with multiple load balancer controllers
  • Manually configuring DNS hostnames for services

allocateLoadBalancerNodePorts could also be useful for minimizing the impact of the latter case...
nightkr added a commit to stackabletech/operator-rs that referenced this issue Mar 19, 2025
@nightkr
Add loadBalancerClass field to ListenerClass.spec
76b6190 
See stackabletech/listener-operator#285
@nightkr nightkr mentioned this issue Mar 19, 2025
Open
nightkr added a commit that referenced this issue Mar 19, 2025
@nightkr
Map ListenerClass.spec.loadBalancerClass into Service.spec
9fb4560 
Fixes #285, requires stackabletech/operator-rs#986
@nightkr nightkr linked a pull request Mar 19, 2025 that will close this issue
Draft
@nightkr nightkr self-assigned this Mar 19, 2025
@nightkr nightkr moved this to Development: Waiting for Review in Stackable Engineering Mar 19, 2025
@nightkr
Copy link
Member Author

nightkr commented Mar 19, 2025

CRD RFC is up: stackabletech/operator-rs#986

@sbernauer sbernauer moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Mar 24, 2025
github-merge-queue bot pushed a commit to stackabletech/operator-rs that referenced this issue May 13, 2025
@nightkr
Add loadBalancerClass field to ListenerClass.spec (#986)
17d9652 
* Add loadBalancerClass field to ListenerClass.spec

See stackabletech/listener-operator#285

* Changelog

* Mention loadBalancerAllocateNodePorts in the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nightkr nightkr

@siegfriedweber siegfriedweber

Labels
customer-request type/feature-improvement
Projects
Stackable End-to-End Coordination
Status: In Progress
Stackable Engineering
Status: Development: In Review
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Map ListenerClass.spec.loadBalancerClass into Service.spec stackabletech/listener-operator
2 participants
@nightkr @siegfriedweber