GitHub user koushik-das opened a pull request:

    https://github.com/apache/cloudstack/pull/741

    Made following fixes in simulator

    - Support for ScaleVmCommand/NetworkRulesVmSecondaryIpCommand in resource 
layer
    - Added support for scaling up a running VM in simulator
    - Fixed some method names not following convention
    
    In order to test PR https://github.com/apache/cloudstack/pull/725 using 
simulator some of these changes are needed.
    
    Based on the way HV check is present in the scale VM API, had to explicitly 
put simulator related check to allow support. The ideal way would be to remove 
all these HV specific check from code and made them some configuration (by 
putting them in hypervisor_capabilities table in DB). But that will be a bigger 
effort outside the scope of this PR.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/koushik-das/cloudstack simulator_fixes

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/741.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #741
    
----
commit 86684cd4bc1966ffcc4b337641fc8bf814724eb1
Author: Koushik Das <kous...@apache.org>
Date:   2015-08-25T09:22:48Z

    Made following fixes in simulator
    - Support for ScaleVmCommand/NetworkRulesVmSecondaryIpCommand in resource 
layer
    - Added support for scaling up a running VM in simulator
    - Fixed some method names not following convention

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to