cozystack
cozystack copied to clipboard
[Documentation] Some Storage-related suggestions
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 cdpcommands. However, applying aLinstorSatelliteConfigurationwould 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
failmodeon 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/DrbdOptionsthat 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
LinstorClusterresource 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.