Skip to content

Conversation

@datacore-chiranthjs
Copy link
Collaborator

@datacore-chiranthjs datacore-chiranthjs commented Aug 4, 2025

Custom Proxmox Storage Plugin for SANsymphony

A Proxmox VE storage plugin to integrate SANsymphony storage using iSCSI, with native multipath support and custom CLI management.

This plugin leverages the SANsymphony REST API for all provisioning and management operations.

  • Adds a Proxmox storage plugin for DataCore SANsymphony (iSCSI, multipath support).
  • Includes CLI tool (ssy-plugin) for interactive and scripted storage management.
  • Supports provisioning, LVM, and shared storage.
  • Installation via APT or .deb; config via CLI, pvesm add, or manual edit.

can refer README.md for usage, config, and troubleshooting.

NOTE: I will be updating the README file once the Public Documentaion is ready.

Copy link
Contributor

@RainerBr RainerBr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I couldn't figure out where ‘udev/rules.d/99-datacore.rules’ is created.
Please let me know where this happens.

@datacore-chiranthjs
Copy link
Collaborator Author

I couldn't figure out where ‘udev/rules.d/99-datacore.rules’ is created. Please let me know where this happens.

we are configuring it in the .deb package its self which is not published here but can be seen here in the provate repo
https://github.com/DataCoreSoftware/datacore-proxmox-storage-plugin/blob/main/SANsymphony-plugin_1.0.0_amd64/etc/udev/rules.d/99-datacore.rules

Let me know if we need to publish entire Debian package or few files here.

@datacore-chiranthjs
Copy link
Collaborator Author

datacore-chiranthjs commented Aug 29, 2025

I couldn't figure out where ‘udev/rules.d/99-datacore.rules’ is created. Please let me know where this happens.

Now I am adding the Debian Repo al well here so that you can refer to Udev rules and similar files in the Debian package

https://github.com/DataCoreSoftware/Scripts/pull/12/files#diff-bfa4a17ab461d71023d97a3199fef9bee25332c9cee8c954b97ef2d20642fcc8

cc: @RainerBr

@datacore-chiranthjs datacore-chiranthjs marked this pull request as ready for review August 29, 2025 08:15
@datacore-chiranthjs datacore-chiranthjs requested a review from a team as a code owner August 29, 2025 08:15
Copy link
Contributor

@RainerBr RainerBr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for including that in the Debian package.

Copy link
Contributor

@RainerBr RainerBr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am fine with the implementation of the changes.

Copy link
Contributor

@RainerBr RainerBr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm fine

Copy link
Collaborator

@Alex130469 Alex130469 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved after conversation with Rainer as he was not able to see any remaining issues.

@datacore-chiranthjs datacore-chiranthjs merged commit 053e20a into main Sep 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants