[GitHub] [cloudstack] deargopinath commented on a change in pull request #3982: Updated 3 error messages to replace the word 'matches' with 'match'

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

deargopinath commented on a change in pull request #3982: Updated 3 error messages to replace the word 'matches' with 'match'
URL: https://github.com/apache/cloudstack/pull/3982#discussion_r398002330
 
 

 ##########
 File path: server/src/main/java/org/apache/cloudstack/vm/VmImportManagerImpl.java
 ##########
 @@ -428,13 +428,13 @@ private ServiceOfferingVO getUnmanagedInstanceServiceOffering(final UnmanagedIns
             serviceOffering = serviceOfferingDao.getComputeOffering(serviceOffering, parameters);
         } else {
             if (!cpu.equals(serviceOffering.getCpu()) && !instance.getPowerState().equals(UnmanagedInstanceTO.PowerState.PowerOff)) {
-                throw new ServerApiException(ApiErrorCode.INTERNAL_ERROR, String.format("Service offering (%s) %d CPU cores does not matches VM CPU cores %d  and VM is not in powered off state (Power state: %s)", serviceOffering.getUuid(), serviceOffering.getCpu(), cpu, instance.getPowerState()));
+                throw new ServerApiException(ApiErrorCode.INTERNAL_ERROR, String.format("Service offering (%s) %d CPU cores does not match VM CPU cores %d  and VM is not in powered off state (Power state: %s)", serviceOffering.getUuid(), serviceOffering.getCpu(), cpu, instance.getPowerState()));
 
 Review comment:
   Hi Gabriel,  Thank you very much for the review. I made the 2 changes as per the review. I will submit the pull request shortly.

----------------------------------------------------------------
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