Skip to content

Documentation for checkVolume API - #380

Merged
yadvr merged 4 commits into
apache:4.19from
shapeblue:CheckVolumeAPI
Feb 29, 2024
Merged

Documentation for checkVolume API#380
yadvr merged 4 commits into
apache:4.19from
shapeblue:CheckVolumeAPI

Conversation

@harikrishna-patnala

@harikrishna-patnalaharikrishna-patnala commented Feb 5, 2024

Copy link
Copy Markdown
Member

This is a document PR for the PR apache/cloudstack#8577


📚 Documentation preview 📚: https://cloudstack-documentation--380.org.readthedocs.build/en/380/

@harikrishna-patnala

Copy link
Copy Markdown
MemberAuthor

@blueorangutan docbuild

@blueorangutan

Copy link
Copy Markdown

@harikrishna-patnala a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.

@blueorangutan

Copy link
Copy Markdown

QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/380. (QA-JID 205)

@harikrishna-patnala

Copy link
Copy Markdown
MemberAuthor

@harikrishna-patnala

Copy link
Copy Markdown
MemberAuthor

@blueorangutan docbuild

@blueorangutan

Copy link
Copy Markdown

@harikrishna-patnala a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.

@blueorangutan

Copy link
Copy Markdown

QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/380. (QA-JID 211)

@kiranchavalakiranchavala left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@harikrishna-patnala

Could you please mention the check volume api will work only if disk formats is in qcow2.

It doesn't work with Raw disk format

Also for Powerflex, the volumes need to be in encrypted format for it to work

@harikrishna-patnala

Copy link
Copy Markdown
MemberAuthor

@harikrishna-patnala

Could you please mention the check volume api will work only if disk formats is in qcow2.

It doesn't work with Raw disk format

Also for Powerflex, the volumes need to be in encrypted format for it to work

Added the volume format detail @kiranchavala that implicitly means about powerflex volumes as well. Please review.

Comment threadsource/adminguide/storage.rst Outdated
Comment threadsource/adminguide/storage.rst Outdated
Comment threadsource/adminguide/storage.rst Outdated
Comment threadsource/adminguide/storage.rst Outdated
@harikrishna-patnala

Copy link
Copy Markdown
MemberAuthor

Addressed them @sureshanaparti, please review

@harikrishna-patnala

Copy link
Copy Markdown
MemberAuthor

@blueorangutan docbuild

@blueorangutan

Copy link
Copy Markdown

@harikrishna-patnala a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.

@sureshanapartisureshanaparti left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

doc updates lgtm

@blueorangutan

Copy link
Copy Markdown

QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/380. (QA-JID 214)

@kiranchavalakiranchavala left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, doc contains the necessary information

@harikrishna-patnala
harikrishna-patnala changed the base branch from main to 4.19February 21, 2024 04:53
@harikrishna-patnalaharikrishna-patnala added this to the 4.19.1 milestone Feb 21, 2024
@yadvr
yadvr merged commit 5a47bef into apache:4.19Feb 29, 2024
@yadvr
yadvr deleted the CheckVolumeAPI branch February 29, 2024 09:10
DaanHoogland added a commit that referenced this pull request Apr 8, 2024
* 4.19:
notes on POD-subnets (#391)
Documentation for checkVolume API (#380)
Document advance DRS settings (#374)
veeam: update API commands and versions in doc (#357)
Update virtual_machines.rst (#353)
Sign up for freeto 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

@harikrishna-patnala@blueorangutan@kiranchavala@sureshanaparti@yadvr