[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-9251?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15238595#comment-15238595
 ] 

ASF GitHub Bot commented on CLOUDSTACK-9251:
--------------------------------------------

Github user jburwell commented on the pull request:

    https://github.com/apache/cloudstack/pull/1363#issuecomment-209222090
  
    I am confused as to the issue at this point.  Building the PR using ``mvn 
clean install findbugs:check`` passes.  It appears that commit reversions have 
addressed the issues.  Assuming I ran the appropriate, I don't see why static 
analysis should be blocking the progress of this PR.
    
    @koushik-das ``@SuppressWarnings`` will hide the issue forever because, in 
reality, we will never come back to it.  Therefore, it should never be used if 
we intend for static analysis to be useful/valuable practice
    
    @swill Speaking generally, my opinion is that the suggestion does not work. 
 The point of static analysis is to protect the integrity of the codebase.  If 
a patch introduces a new issue, it shouldn't be accepted until the issue is 
fixed.  The only exception to this rule is that we determine the rule is no 
longer valuable and stop checking it.  We only gain value from static analysis 
if we are disciplined about addressing the issues found rather than taking the 
most expedient path to get a pass.


> error while change instance offering to custom offering
> -------------------------------------------------------
>
>                 Key: CLOUDSTACK-9251
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9251
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>          Components: API
>    Affects Versions: 4.7.0, 4.7.1
>         Environment: ACS 4.7.1 -  
> https://dist.apache.org/repos/dist/dev/cloudstack/4.7.1/
>            Reporter: Tomasz Zieba
>
> Changing compute offering to custom offering end with:
> Step to reproduce:
> 1. Create and run new instance with offering eg. 1 core / 1GB
> 2. Create Custom Offering eg 2 core / 2 GB
> 3. Stop the instance
> 4. Try to change offering of the instance
> 5. Error: Invalid cpu cores value, specify a value between 1 and 2147483647



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to