You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2020/02/11 08:10:31 UTC

[GitHub] [cloudstack] choleowsky opened a new issue #3878: Volume metrics doesn't update after volume migration

choleowsky opened a new issue #3878: Volume metrics doesn't update after volume migration
URL: https://github.com/apache/cloudstack/issues/3878
 
 
   <!--
   Verify first that your issue/request is not already reported on GitHub.
   Also test if the latest release and master branch are affected too.
   Always add information AFTER of these HTML comments, but no need to delete the comments.
   -->
   
   ##### ISSUE TYPE
    * Bug Report
   
   ##### COMPONENT NAME
   <!--
   Categorize the issue, e.g. API, VR, VPN, UI, etc.
   -->
   ~~~
   CloudRuntimeException
   ~~~
   
   ##### CLOUDSTACK VERSION
   <!--
   New line separated list of affected versions, commit ID for issues on master branch.
   -->
   
   ~~~
   4.11.2.0
   ~~~
   
   ##### CONFIGURATION
   <!--
   Information about the configuration if relevant, e.g. basic network, advanced networking, etc.  N/A otherwise
   -->
   Advanced network
   Qemu-KVM hypervisor
   NFS shared storage
   ##### OS / ENVIRONMENT
   <!--
   Information about the environment if relevant, N/A otherwise
   -->
   Ubuntu 16.04 / 18.04
   
   ##### SUMMARY
   <!-- Explain the problem/feature briefly -->
   Error in CloudRuntimeException: Can't find volume:
   ##### STEPS TO REPRODUCE
   <!--
   For bugs, show exactly how to reproduce the problem, using a minimal test-case. Use Screenshots if accurate.
   
   For new features, show how the feature would be used.
   -->
   
   <!-- Paste example playbooks or commands between quotes below -->
   
   * Install fresh cloudstack 4.11.2.0 whith Qemu-KVM hosts, NFS primary storage server and advanced network
   * Create two new instances
   * Add new primary storage
   * Migrate one instance on new primary storage server
   ~~~
   ~~~
   
   <!-- You can also paste gist.github.com links for larger files -->
   
   ##### EXPECTED RESULTS
   <!-- What did you expect to happen when running the steps above? -->
   Volume metrics are updated on all sotrage server
   ~~~
   
   ~~~
   
   ##### ACTUAL RESULTS
   <!-- What actually happened? -->
   The migration works but the volume metrics doesn't update in new storage server, on the first one are updated as aspected.
   
   <!-- Paste verbatim command output between quotes below -->
   ~~~
   
   ~~~
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] weizhouapache edited a comment on issue #3878: Volume metrics doesn't update after volume migration

Posted by GitBox <gi...@apache.org>.
weizhouapache edited a comment on issue #3878: Volume metrics doesn't update after volume migration
URL: https://github.com/apache/cloudstack/issues/3878#issuecomment-585418072
 
 
   @choleowsky 
   this is a regression issue by 4627fb2
   
   @skattoju3 can you please have a look at #3884 ? 
   for NFS, we should check by path not uuid as well, similar as #3181 
   
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] weizhouapache commented on issue #3878: Volume metrics doesn't update after volume migration

Posted by GitBox <gi...@apache.org>.
weizhouapache commented on issue #3878: Volume metrics doesn't update after volume migration
URL: https://github.com/apache/cloudstack/issues/3878#issuecomment-585418072
 
 
   @choleowsky 
   
   this is a regression issue by #3181
   
   @skattoju3 can you please have a look ? for NFS, we should check by path not uuid.
   
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] weizhouapache edited a comment on issue #3878: Volume metrics doesn't update after volume migration

Posted by GitBox <gi...@apache.org>.
weizhouapache edited a comment on issue #3878: Volume metrics doesn't update after volume migration
URL: https://github.com/apache/cloudstack/issues/3878#issuecomment-585418072
 
 
   @choleowsky 
   this is a regression issue by https://github.com/apache/cloudstack/commit/4627fb2cd7556173bd7e58bf1bec22c93a78f31d#diff-5a9d445f9ba381a581f4596d4887c8d5R904-R906
   
   @skattoju3 can you please have a look at #3884 ? 
   for NFS, we should check by path not uuid as well, similar as #3181 
   
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] weizhouapache edited a comment on issue #3878: Volume metrics doesn't update after volume migration

Posted by GitBox <gi...@apache.org>.
weizhouapache edited a comment on issue #3878: Volume metrics doesn't update after volume migration
URL: https://github.com/apache/cloudstack/issues/3878#issuecomment-585418072
 
 
   @choleowsky 
   this is a regression issue by 4627fb2
   
   @skattoju3 can you please have a look ? for NFS, we should check by path not uuid as well.
   similar as #3181 
   
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] choleowsky commented on issue #3878: Volume metrics doesn't update after volume migration

Posted by GitBox <gi...@apache.org>.
choleowsky commented on issue #3878: Volume metrics doesn't update after volume migration
URL: https://github.com/apache/cloudstack/issues/3878#issuecomment-584522200
 
 
   I found a wrkaround that seems to indicate where the problem lies.
   
   From what I have seen, the LibvirtStoragePool.getPhysicalDisk procedure goes to look for the files of the migrated volumes with the value of the "path" column of the "volumes" table.
   When a volume is migrated from one storage to another, the "path" field is changed but the LibvirtStoragePool.getPhysicalDisk procedure seems to go looking for the volume with its old "path" which now corresponds to the "uuid" field.
   
   If I make a symbolic link directly in the new NFS storage server's filesystem (ln -s <path> <uuid>), after some time I found that metrics were updated and the message in agent.log no longer appeared.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] weizhouapache edited a comment on issue #3878: Volume metrics doesn't update after volume migration

Posted by GitBox <gi...@apache.org>.
weizhouapache edited a comment on issue #3878: Volume metrics doesn't update after volume migration
URL: https://github.com/apache/cloudstack/issues/3878#issuecomment-585418072
 
 
   @choleowsky 
   this is a regression issue by 4627fb2
   
   @skattoju3 can you please have a look ? for NFS, we should check by path not uuid as well.
   
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services