[jira] [Commented] (CLOUDSTACK-8959) Option to remove mapping of the config drive when deploying VM

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

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

Github user pedro-martins commented on the pull request:

    https://github.com/apache/cloudstack/pull/938#issuecomment-172103324
  
    Hi.
    There is no problem to set vmData to null?  and what about replace this short if to a method with documentation and a little test.
    And if you extract all that you do to little methods, it will be easier to create test cases and confirm that you do works correctly
    
    Ty =) .


> Option to remove mapping of the config drive when deploying VM
> --------------------------------------------------------------
>
>                 Key: CLOUDSTACK-8959
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8959
>             Project: CloudStack
>          Issue Type: Improvement
>      Security Level: Public(Anyone can view this level - this is the default.) 
>            Reporter: Kshitij Kansal
>




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