deargopinath opened a new pull request #3982:
URL: https://github.com/apache/cloudstack/pull/3982


   ## Description
   I provided a fix for the error reported in issue # 3642
   Details of the error can be found at: 
       https://github.com/apache/cloudstack/issues/3642
   
   **Root cause of the problem:**
   The incorrect error messages were found in the lines 431, 434 and 437 of the 
file - 
       server/src/main/java/org/apache/cloudstack/vm/VmImportManagerImpl.java
   
   _**Line # 431:**_
           ` throw new ServerApiException ... CPU cores does not matches ... `
   
   _**Line # 434:**_
           ` throw new ServerApiException ... memory does not matches ...`
   
   _**Line # 437:**_
            `throw new ServerApiException ... CPU speed does not matches ...`
   
   **Changes done to fix the problem:**
   
   I replaced the word 'matches' with 'match' inside the 3 error messages.
   
   ## Types of changes
   <!--- What types of changes does your code introduce? Put an `x` in all the 
boxes that apply: -->
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   - [ ] New feature (non-breaking change which adds functionality)
   - [x] Bug fix (non-breaking change which fixes an issue)
   - [ ] Enhancement (improves an existing feature and functionality)
   - [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
   
   ## Screenshots (if appropriate):
   
   ## How Has This Been Tested?
   I compiled the code after the changes and tested whether the error messages 
with the updated text are shown for these 3 scenarios of Server API exceptions:
   1. CPU cores not matching
   2. Memory not matching
   3. CPU speed not matching
   
   <!-- Please read the 
[CONTRIBUTING](https://github.com/apache/cloudstack/blob/master/CONTRIBUTING.md)
 document -->
   


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to