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

Cannot set valid loadBalancerSourceRanges on K8s cluster #27138

Closed
Setsushin opened this issue Jun 13, 2024 · 4 comments
Closed

Cannot set valid loadBalancerSourceRanges on K8s cluster #27138

Setsushin opened this issue Jun 13, 2024 · 4 comments
Assignees
Labels
solved spark tech-issues The user has a technical issue about an application triage Triage is needed

Comments

@Setsushin
Copy link

Setsushin commented Jun 13, 2024

Name and Version

bitnami/spark

What architecture are you using?

amd64

What steps will reproduce the bug?

  1. Set following configs in values.yaml (dummy example)
# In values.yaml
service:
  type: LoadBalancer
  annotations:
    cloud.google.com/load-balancer-type: External
  allocateLoadBalancerNodePorts: true
  loadBalancerSourceRanges:
  - 1.1.1.1/32
  - 2.2.2.2/32
  1. Run helm upgrade --install spark bitnami/spark -f values.yaml
  2. Got the error

Are you using any custom parameters or values?

service:
  type: LoadBalancer
  annotations:
    cloud.google.com/load-balancer-type: External
  allocateLoadBalancerNodePorts: true
  loadBalancerSourceRanges:
  - 1.1.1.1/32
  - 2.2.2.2/32

What is the expected behavior?

An external LoadBalancer service with corresponding IP source range should be created.

What do you see instead?

❯ helm upgrade --install spark bitnami/spark -f values.yaml
Error: UPGRADE FAILED: cannot patch "spark-master-svc" with kind Service: Service "spark-master-svc" is invalid: spec.LoadBalancerSourceRanges: Invalid value: "[1.1.1.1/32 2.2.2.2/32]": must be a list of IP ranges. For example, 10.240.0.0/24,10.250.0.0/24 

Additional information

I tried to use

loadBalancerSourceRanges: "[1.1.1.1/32, 2.2.2.2/32]"

as instead and it passed.

@Setsushin Setsushin added the tech-issues The user has a technical issue about an application label Jun 13, 2024
@github-actions github-actions bot added the triage Triage is needed label Jun 13, 2024
@carrodher
Copy link
Member

As the error says, you need to use a list of IP ranges in the following way: 10.240.0.0/24,10.250.0.0/24

@Setsushin
Copy link
Author

@carrodher Thanks for the reply! I converted it to a STRING then it passed the verification.

loadBalancerSourceRanges:
  - 10.240.0.0/24
  - 10.250.0.0/24

Btw, I have used the above method in many other Helm repos, and it is also considered correct. I wonder if there are any plans to improve usability in this regard?

@carrodher
Copy link
Member

We appreciate your involvement! If you're interested in contributing a solution, we welcome you to create a pull request. The Bitnami team is excited to review your submission and offer feedback. You can find the contributing guidelines here.

Your contribution will greatly benefit the community. Feel free to reach out if you have any questions or need assistance.

@Setsushin
Copy link
Author

Thanks for your kind reply.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
solved spark tech-issues The user has a technical issue about an application triage Triage is needed
Projects
None yet
Development

No branches or pull requests

2 participants