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
OpenSDS installation using Helm charts with Ceph backend #109
OpenSDS installation using Helm charts with Ceph backend #109
Conversation
@leonwanghui PTAL. |
Generally it looks good, I’ll have test after the vacation |
Hi @Click2Cloud-Gamma, please rebase the target branch to |
Added tgt-conf-dir part
Now it will take input about the image from values.yml
@Click2Cloud-Gamma Please address the errors, thanks! |
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.
LGTM
Thanks @Click2Cloud-Gamma , I will take a test for this patch. |
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.
Please update the doc based on https://github.com/opensds/opensds/wiki/OpenSDS-Integration-with-Kubernetes-CSI
@@ -21,10 +21,14 @@ spec: | |||
hostNetwork: true | |||
containers: | |||
- name: node-driver-registrar | |||
image: quay.io/k8scsi/csi-node-driver-registrar:v1.0.1 |
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.
Same here
@Click2Cloud-Gamma I'm afraid you have deleted your folk, and I can't find the project in your code repo, PTAL |
Need rebase. |
@@ -39,7 +39,7 @@ spec: | |||
serviceAccount: csi-attacher | |||
containers: | |||
- name: csi-attacher | |||
image: quay.io/k8scsi/csi-attacher:v1.0.1 | |||
image: quay.io/k8scsi/csi-attacher:v0.4.0 |
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.
Please use v1.0.1 because it supports stable v1.0.0 csi spec and is supported by k8s v1.13.
No description provided.