[ https://issues.apache.org/jira/browse/CLOUDSTACK-8897?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14936591#comment-14936591 ]
ASF GitHub Bot commented on CLOUDSTACK-8897: -------------------------------------------- Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/874#issuecomment-144332355 We won't be able to test this on simulator. @harikrishna-patnala can you answer @borisroman and then we can merge. LGTM +1 on code review alone > baremetal:addHost:make host tag info mandtory in baremetal addhost Api call > --------------------------------------------------------------------------- > > Key: CLOUDSTACK-8897 > URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8897 > Project: CloudStack > Issue Type: Bug > Security Level: Public(Anyone can view this level - this is the > default.) > Components: Baremetal, Management Server > Reporter: Harikrishna Patnala > Assignee: Harikrishna Patnala > Fix For: 4.6.0 > > > Right now in baremetal, addhost api is successful with out providing the host > tag info and we recommend host tag is mandatory for bare-metal. > in the current implementation host tag check is happening at vm deployment > stage but it will be good to have host tag field as mandatory field during > adding of the host it self. -- This message was sent by Atlassian JIRA (v6.3.4#6332)