Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KubeVirt support added for CSM Modules #1399

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

khareRajshree
Copy link
Contributor

Description

Documentation updated for KubeVirt support with Replication Module

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
dell/csm#1563

Checklist:

  • Have you run a grammar and spell checks against your submission?
  • Have you tested the changes locally?
  • Have you tested whether the hyperlinks are working properly?
  • Did you add the examples wherever applicable?
  • Have you added high-resolution images?

Copy link

github-actions bot commented Jan 1, 2025

Test Results

77 tests  ±0   77 ✅ ±0   3s ⏱️ ±0s
 3 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 5efe9f5. ± Comparison against base commit 9c44beb.

♻️ This comment has been updated with latest results.

@shanmydell
Copy link
Collaborator

@khareRajshree : For consistency, have this PR to be worked for all Kubevirt related changes

## KubeVirt support with Replication

- Install the OpenShift Virtualization operator from the RedHat UI on both the
source and target clusters with OCP v4.18 and above.
Copy link
Contributor

Choose a reason for hiding this comment

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

This should change to v4.17

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Addressed.

source and target clusters with OCP v4.18 and above.
- The Replication modes supported are: SYNC and ASYNC.

- Make sure the driver is installed with the Replication Module enabled and a
Copy link
Contributor

Choose a reason for hiding this comment

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

Link to where the instructions are for enabling the replication module

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Addressed.

the source will be replicated to the target cluster, as there is an active
replication session between the two clusters.

- sample _create-vm.yaml_ manifest:
Copy link
Contributor

Choose a reason for hiding this comment

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

Point to RedHat documentation here instead

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Addressed.

@khareRajshree khareRajshree changed the title KubeVirt support added for Replication Module KubeVirt support added for CSM Modules Jan 3, 2025
PowerFlex, PowerMax, PowerStore, PowerScale.
OpenShift Virtualization 4.17 <b> supports [storage profile](https://github.com/kiagnose/kubevirt-storage-checkup) operations only </b> for the following:</br>
- Storage Systems: PowerFlex, PowerMax, PowerStore, PowerScale.
- CSM Module: Replication Module
Copy link

@AronAtDell AronAtDell Jan 3, 2025

Choose a reason for hiding this comment

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

Could this line just be: "Modules: Replication"? If yes then I would suggest we simplify. If you can avoid the use of the acronym CSM when you produce documentation that would be ideal. Thanks!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Addressed.

the source will be replicated to the target cluster, as there is an active
replication session between the two clusters.

- Refer this documentation

Choose a reason for hiding this comment

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

"Refer to this document ..."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Addressed.

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.

6 participants