[jira] [Commented] (CLOUDSTACK-8379) add support to marvin to enable deployed zone based on the value provided in config file

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

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

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

Github user nitt10prashant commented on the pull request:

    https://github.com/apache/cloudstack/pull/158#issuecomment-127891937
  
     enabled = enabled if enabled else 'true' 
     will set enable to true if  enable is false or None  which is unexpected,
     ideally  enable should be false if it is false in config file else true 


> add support to marvin to enable deployed zone based on the value provided in config file
> ----------------------------------------------------------------------------------------
>
>                 Key: CLOUDSTACK-8379
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8379
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the default.) 
>          Components: marvin
>    Affects Versions: 4.6.0
>            Reporter: Srikanteswararao Talluri
>            Assignee: Srikanteswararao Talluri
>             Fix For: 4.6.0
>
>
> add support to marvin to enable deployed zone based on the value provided in config file
> if under zone section, if the 'enabled' element is not mentioned, then zone will be enabled otherwise zone will be enabled/disabled based on value provided ('false' or 'true') for 'enabled'



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