[GitHub] [cloudstack] DaanHoogland opened a new issue #3652: improve logging in vmware storage processor

[GitBox <gi...@apache.org>]

DaanHoogland opened a new issue #3652: improve logging in vmware storage processor
URL: https://github.com/apache/cloudstack/issues/3652
 
 
   <!--
   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.
   -->
   VolumeOrchestrator gives a cryptic message when copy from primary to secondary failes on Vmware during migrate volume.
   
   ```
   Failed to migrate volumecom.cloud.exception.StorageUnavailableException: Resource [StoragePool:1446] is unreachable: Migrate volume failed: copy volume from primary to secondary failed due to exception: Exception: java.lang.NullPointerExceptionMessage: null
   ```
   there is a lot of information in the message about where things went wrong but not about what is wrong. The NPE is not locatable from the message or the accompanyting stacktrace.
   
   in ```com.cloud.storage.resource.VmwareStorageProcessor#copyVolumeFromPrimaryToSecondary``` , diagnostic output should be improved.
   
   ##### ISSUE TYPE
   <!-- Pick one below and delete the rest -->
    * Improvement Request
   
   ##### COMPONENT NAME
   <!--
   Categorize the issue, e.g. API, VR, VPN, UI, etc.
   -->
   ~~~
   Vmware Storage
   ~~~
   
   ##### CLOUDSTACK VERSION
   <!--
   New line separated list of affected versions, commit ID for issues on master branch.
   -->
   
   ~~~
   
   ~~~
   
   ##### CONFIGURATION
   <!--
   Information about the configuration if relevant, e.g. basic network, advanced networking, etc.  N/A otherwise
   -->
   
   
   ##### OS / ENVIRONMENT
   <!--
   Information about the environment if relevant, N/A otherwise
   -->
   
   
   ##### SUMMARY
   <!-- Explain the problem/feature briefly -->
   
   
   ##### 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 -->
   ~~~
   
   ~~~
   
   <!-- You can also paste gist.github.com links for larger files -->
   
   ##### EXPECTED RESULTS
   <!-- What did you expect to happen when running the steps above? -->
   
   ~~~
   
   ~~~
   
   ##### ACTUAL RESULTS
   <!-- What actually happened? -->
   
   <!-- 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