cozystack icon indicating copy to clipboard operation
cozystack copied to clipboard

[Documentation] Some Storage-related suggestions

Open WanzenBug opened this issue 1 month ago • 0 comments

Hi! Piraeus Maintainer here. Thanks for using Piraeus in you project!

I worked through your documentation, and everything worked as described 😄

I do have some suggestions related to the Storage sections. If that is fine by you, I can also create a PR later addressing these points:

  • [ ] "Getting Started: Configure Storage": You are currently manually issuing linstor ps cdp commands. However, applying a LinstorSatelliteConfiguration would work just as well, and is what we would recommend.
    ...
    spec:
      storagePools:
      - name: data
        zfsPool: {}
        source:
          hostDevices:
          - /dev/sdb
    
  • [ ] "Getting Started: Configure Storage": You are manually setting the failmode on ZFS. This is still required for now, but can be removed with next LINSTOR release. See https://github.com/LINBIT/linstor-server/issues/463
  • [ ] "Getting Started: Configure Storage": The replicated storage class is using some property.linstor.csi.linbit.com/DrbdOptions that are already the default on the LINSTOR Controller level (as applied by the Piraeus Operator), and as such are not required.
  • [ ] "Operations Guide: Multi Datacenter: LINSTOR DRBD": We generally recommend that these options should be set on the LinstorCluster resource instead of ad-hoc commands. This makes them more discoverable within the K8s system, and also means people don't forget to run that one command in automated deployments so easily.
  • [ ] "Storage: Configuring DRBD": Same as above. Can be set on LinstorCluster.
  • [ ] "Storage: Encrypted Storage": You should create a kubernetes secret and pass the name of the secret to LinstorCluster.spec.linstorPassphraseSecret: with that, the passphrase is always unlocked automatically.

WanzenBug avatar Nov 28 '25 08:11 WanzenBug