OCPVE-620: docs: add guidance on Multi-Node Topology Snapshotting

[jakobmoellerdev]

Creates a guidance documentation on Snapshotting and Cloning in Multi-Node Topologies. It specifically recommends to use Stateful Sets alongside preprovisioned and bound source PVCs to avoid running into scheduler limitations that may (or may not) occur in TopoLVM CSI scheduling (I opened a discussion in topolvm/topolvm#737 to clarify the exact conditions, also I am trying to independently verify this issue).

[openshift-ci]

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

[openshift-ci-robot]

@jakobmoellerdev: This pull request references OCPVE-620 which is a valid jira issue.

In response to this:

Creates a guidance documentation on Snapshotting and Cloning in Multi-Node Topologies. It specifically recommends to use Stateful Sets alongside preprovisioned and bound source PVCs to avoid running into scheduler limitations that may (or may not) occur in TopoLVM CSI scheduling (I opened a discussion in topolvm/topolvm#737 to clarify the exact conditions, also I am trying to independently verify this issue).

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[codecov-commenter]

Codecov Report

Merging #384 (1812657) into main (a962b90) will increase coverage by 43.34%.
Report is 37 commits behind head on main.
The diff coverage is 73.39%.

Additional details and impacted files

@@             Coverage Diff             @@
##             main     #384       +/-   ##
===========================================
+ Coverage   16.59%   59.93%   +43.34%     
===========================================
  Files          24       26        +2     
  Lines        2061     2631      +570     
===========================================
+ Hits          342     1577     +1235     
+ Misses       1693      950      -743     
- Partials       26      104       +78     

Files Changed	Coverage Δ
controllers/lvmcluster_controller_watches.go	90.32% <ø> (+90.32%)	⬆️
pkg/vgmanager/vgmanager_controller.go	11.57% <0.00%> (+11.57%)	⬆️
controllers/topolvm_snapshotclass.go	68.65% <10.00%> (+68.65%)	⬆️
controllers/lvmcluster_controller.go	65.41% <66.66%> (+65.41%)	⬆️
pkg/cluster/leaderelection.go	66.66% <66.66%> (ø)
pkg/cluster/sno.go	72.72% <72.72%> (ø)
controllers/topolvm_controller.go	95.74% <100.00%> (+95.74%)	⬆️
pkg/vgmanager/devices.go	76.80% <100.00%> (+2.60%)	⬆️

... and 10 files with indirect coverage changes

[suleymanakbas91]

Great explanation!
/lgtm
/approve
/hold
for docs team to review.

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jakobmoellerdev, suleymanakbas91

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [suleymanakbas91]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[jakobmoellerdev]

/unhold @johnwilkins made a pass over this

[jakobmoellerdev]

/unhold

[openshift-ci]

@jakobmoellerdev: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.