ceph / ceph-iscsi

Ceph iSCSI tools
GNU General Public License v3.0
62 stars 59 forks source link

rbd-target-api: Disk limit 256 #270

Open lnsyyj opened 1 year ago

lnsyyj commented 1 year ago

Hello everyone During use, after creating more than 256 disks, I report an error that the disk limit of 256 reached.

            if len(config['disks']) >= 256:
                return "Disk limit of 256 reached."

why is the limit to 256? How do I configure my users who want to use more than 256 disks?

lxbsz commented 1 year ago

It's a hard code in ceph-iscsi and rtslib, which is compatible with the old kernels. For the new kernels this limit could be remove. And the code hasn't do that yet.

If you want to support more than 256 disks, you can setup multiple gateway pairs, which each pair will have its own dedicated configure pool, in the same ceph cluster.

lnsyyj commented 1 year ago

Thank you @lxbsz , is there any plan for the community to refactor the CEPH-iSCSI project in the near future? Or make the code more robust?

lxbsz commented 1 year ago

Thank you @lxbsz , is there any plan for the community to refactor the CEPH-iSCSI project in the near future? Or make the code more robust?

This is in my todo list, but not get a chance yet. If you'd like you can raise one PR to fix it. The rtslib has done that in https://github.com/open-iscsi/rtslib-fb/pull/191.

lnsyyj commented 1 year ago

Thank you @lxbsz , is there any plan for the community to refactor the CEPH-iSCSI project in the near future? Or make the code more robust?

This is in my todo list, but not get a chance yet. If you'd like you can raise one PR to fix it. The rtslib has done that in open-iscsi/rtslib-fb#191.

Ok thanks.