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

Deploy multiple csi drivers in single k8s deployment with soda-csi-provisioner #410

Closed
asifdxtreme opened this issue Apr 7, 2021 · 5 comments

Comments

@asifdxtreme
Copy link
Member

Issue/Feature Description:
Currently multiple CSI drivers cannot be deployed in a single k8s deployment using soda-csi-provisioner as only one csi driver can hold the lock for soda-csi

Why this issue to fixed / feature is needed(give scenarios or use cases):
This needs to be fixed to support heterogeneous support for soda-csi plug n play

How to reproduce, in case of a bug:
Deploy 2 CSI drivers with soda-csi-provisioner image

@asifdxtreme
Copy link
Member Author

/cc @Click2cloud-Ceres

@click2cloud-repo
Copy link

We have tried the above-mentioned issue with different CSI drivers and came to some conclusions which are listed below :

  1. If there are more than one CSI driver from the same storage vendor, it should be deployed in a different namespace to avoid the soda-csi lock conflicts.
    for example :
    we have used OPENEBS-LVM CSI driver in Kube-system namespace and OPENEBS-ZFS CSI driver in Kube-system-zfs namespace,
    both were working without holding any locks for soda-csi.

  2. CSI drivers from different storage vendors can be deployed in the same namespace without having any conflict.
    for example :
    we have used the CEPH-RBD CSI driver and IBM CSI driver both in the default namespace and didn't notice any soda-csi lock conflict issue.

@asifdxtreme
Copy link
Member Author

@Click2cloud-Ceres Thanks for digging into this issue, Can you please add this Note statement in the document over here https://docs.sodafoundation.io/guides/integration-guides/csi-integration/soda-csi-pnp/#prerequisite

@click2cloud-repo
Copy link

sure sir I have committed the changes in the below-mentioned PR. I request you to review it and suggest if any changes required.
sodafoundation/documentation#163
thanx.

@asifdxtreme asifdxtreme modified the milestones: Iter1_Jerba, Iter2_Jerba May 11, 2021
@asifdxtreme
Copy link
Member Author

Related Pr's have been merged in Documentation Repo, going ahead to close this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants