-
Notifications
You must be signed in to change notification settings - Fork 27
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
Remove workers from ray-cluster #1397
Conversation
# Conflicts: # charts/qiskit-serverless/charts/gateway/templates/rayclustertemplate.yaml
@akihikokuroda reviewing the configuration of the cluster I had a question related with the values:
Would has sense to only do |
It means that it doesn't create the self-signed certificates. I think that's OK. If the cluster doesn't have the cert manager, the TLS may not be needed. |
And with Istio I suppose that they are not needed either, aren't they? |
@Tansito That's right. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would save updates to the TLS / certmanager options for another PR.
Sure! |
Summary
Due to some problems in the current configuration that we have in the cluster we cannot support workers right now. After some tests I could confirm that we can support the same features that we are supporting right now increasing the size of the head node.
So, this PR modifies the ray-cluster template removing the workers from the configuration.