You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by "harikrishna-patnala (via GitHub)" <gi...@apache.org> on 2024/02/05 08:19:08 UTC
[PR] Documentation for checkVolume API [cloudstack-documentation]
harikrishna-patnala opened a new pull request, #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380
This is a document PR for the PR https://github.com/apache/cloudstack/pull/8577
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380#issuecomment-1926464631
QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/380. (QA-JID 205)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "harikrishna-patnala (via GitHub)" <gi...@apache.org>.
harikrishna-patnala commented on PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380#issuecomment-1953606780
> @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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "rohityadavcloud (via GitHub)" <gi...@apache.org>.
rohityadavcloud merged PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "harikrishna-patnala (via GitHub)" <gi...@apache.org>.
harikrishna-patnala commented on PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380#issuecomment-1953617289
Addressed them @sureshanaparti, please review
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "sureshanaparti (via GitHub)" <gi...@apache.org>.
sureshanaparti commented on code in PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380#discussion_r1495337661
##########
source/adminguide/storage.rst:
##########
@@ -962,6 +962,54 @@ Bytes read/write, as well as the total IO/s, are exposed via UI, as shown in the
These statistics are obtained from the hypervisor directly and they represent
"current" bytes/s and IO/s values at the time of collection.
+Check and repair Volume
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+When there are any leaks or any inconsistencies in the volume, then the checkVolume API can be used to
+check any such errors in the volume and helps in repairing. This feature is currently available only
Review Comment:
```suggestion
check for any such errors in the volume and helps in repairing them. This feature is currently available only
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "harikrishna-patnala (via GitHub)" <gi...@apache.org>.
harikrishna-patnala commented on PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380#issuecomment-1953617425
@blueorangutan docbuild
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380#issuecomment-1931740451
@harikrishna-patnala a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "sureshanaparti (via GitHub)" <gi...@apache.org>.
sureshanaparti commented on code in PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380#discussion_r1495338830
##########
source/adminguide/storage.rst:
##########
@@ -962,6 +962,54 @@ Bytes read/write, as well as the total IO/s, are exposed via UI, as shown in the
These statistics are obtained from the hypervisor directly and they represent
"current" bytes/s and IO/s values at the time of collection.
+Check and repair Volume
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+When there are any leaks or any inconsistencies in the volume, then the checkVolume API can be used to
+check any such errors in the volume and helps in repairing. This feature is currently available only
+for KVM and volumes with QCOW2 format. This API uses "qemu-img check" command on the KVM host.
+
+Also a global or storage pool level setting "volume.check.and.repair.leaks.before.use" is available which allows
+to do a check and repair leaks operation for the volume during instance start and volume attach operations.
Review Comment:
```suggestion
to check and repair any leaks of the volume during instance start and volume attach operations.
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "harikrishna-patnala (via GitHub)" <gi...@apache.org>.
harikrishna-patnala commented on PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380#issuecomment-1926451960
@blueorangutan docbuild
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "sureshanaparti (via GitHub)" <gi...@apache.org>.
sureshanaparti commented on code in PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380#discussion_r1495339756
##########
source/adminguide/storage.rst:
##########
@@ -962,6 +962,54 @@ Bytes read/write, as well as the total IO/s, are exposed via UI, as shown in the
These statistics are obtained from the hypervisor directly and they represent
"current" bytes/s and IO/s values at the time of collection.
+Check and repair Volume
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+When there are any leaks or any inconsistencies in the volume, then the checkVolume API can be used to
+check any such errors in the volume and helps in repairing. This feature is currently available only
+for KVM and volumes with QCOW2 format. This API uses "qemu-img check" command on the KVM host.
+
+Also a global or storage pool level setting "volume.check.and.repair.leaks.before.use" is available which allows
+to do a check and repair leaks operation for the volume during instance start and volume attach operations.
+This will help in repairing any leaks for the volume before using it. This is a blocking operations, meaning
Review Comment:
```suggestion
This will help in repairing any leaks of the volume before using it. This is a blocking operation, meaning
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "sureshanaparti (via GitHub)" <gi...@apache.org>.
sureshanaparti commented on code in PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380#discussion_r1495338395
##########
source/adminguide/storage.rst:
##########
@@ -962,6 +962,54 @@ Bytes read/write, as well as the total IO/s, are exposed via UI, as shown in the
These statistics are obtained from the hypervisor directly and they represent
"current" bytes/s and IO/s values at the time of collection.
+Check and repair Volume
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+When there are any leaks or any inconsistencies in the volume, then the checkVolume API can be used to
+check any such errors in the volume and helps in repairing. This feature is currently available only
+for KVM and volumes with QCOW2 format. This API uses "qemu-img check" command on the KVM host.
+
+Also a global or storage pool level setting "volume.check.and.repair.leaks.before.use" is available which allows
Review Comment:
```suggestion
Also, a global or storage pool level setting "volume.check.and.repair.leaks.before.use" is available which allows
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "harikrishna-patnala (via GitHub)" <gi...@apache.org>.
harikrishna-patnala commented on PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380#issuecomment-1931737376
@blueorangutan docbuild
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380#issuecomment-1931742259
QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/380. (QA-JID 211)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380#issuecomment-1953618382
@harikrishna-patnala a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380#issuecomment-1953619383
QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/380. (QA-JID 214)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380#issuecomment-1926463119
@harikrishna-patnala a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Documentation for checkVolume API [cloudstack-documentation]
Posted by "harikrishna-patnala (via GitHub)" <gi...@apache.org>.
harikrishna-patnala commented on PR #380:
URL: https://github.com/apache/cloudstack-documentation/pull/380#issuecomment-1926519575
Here it is https://qa.cloudstack.cloud/builds/docs-build/pr/380/adminguide/storage.html#check-and-repair-volume
![image](https://github.com/apache/cloudstack-documentation/assets/3348673/6944d7bf-ffe7-4701-baee-7bb2bdf585d6)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org