-
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
fix certificate contents #911
Conversation
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
@@ -303,17 +306,6 @@ data: | |||
cpu: {{ .Values.application.ray.cpu }} | |||
memory: {{ .Values.application.ray.memory }}Gi | |||
securityContext: | |||
volumeMounts: |
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.
we do not need mounts for worker anymore?
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.
There were 2 volumeMounts. The latter was used. This PR merges them into one.
BASE_DIR=$1 | ||
CLUSTER_NAME=$2 | ||
IP_ADDRESS=$3 | ||
NAMESPACE=$4 |
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'm trying to understand what the pull request is doing Aki, so correct me if I'm wrong. NAMESPACE
argument is not needed in this case, isn't it? At least by the script.
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.
You are right. The same script was used for head and worker nodes before. I separated it for the head and worker in this PR. The namespace is not used in the worker node script. I take it out. Thanks!
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
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.
Checked on rancher, everything works! Thank you!
Summary
Fix: #910
Details and comments