Skip to content

doc: xapi storage layer doc update #6351

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

Merged
merged 4 commits into from
Mar 13, 2025

Conversation

Vincent-lau
Copy link
Contributor

The storage layer code in xapi has undergone some changes: storage_impl.ml is no longer in the repo and is replaced by storage_smapiv1_wrapper.ml and storage_smapiv1.ml. Update the relevant docs to reflect this change.

Use a simple syntax so this is easier to preview for local environment.
No functional change.

Signed-off-by: Vincent Liu <shuntian.liu2@cloud.com>
Change all the link to the latest xapi version v25.11.0

Signed-off-by: Vincent Liu <shuntian.liu2@cloud.com>
@Vincent-lau
Copy link
Contributor Author

The new first diagram looks like this

image

@Vincent-lau Vincent-lau force-pushed the private/shul2/sxm-docs branch from e1e7520 to 06b6797 Compare March 12, 2025 13:25
The storage layer code in xapi has undergone some changes:
storage_impl.ml is no longer in the repo and is replaced by
`storage_smapiv1_wrapper.ml` and `storage_smapiv1.ml`. Update the
relevant docs to reflect this change.

Signed-off-by: Vincent Liu <shuntian.liu2@cloud.com>
Update the xen-api version info in the notice to the current version of
xen-api.

Signed-off-by: Vincent Liu <shuntian.liu2@cloud.com>
@Vincent-lau Vincent-lau force-pushed the private/shul2/sxm-docs branch from 06b6797 to 51b800b Compare March 12, 2025 15:42
@lindig lindig added this pull request to the merge queue Mar 13, 2025
Merged via the queue into xapi-project:master with commit 5ec5732 Mar 13, 2025
26 of 27 checks passed
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