Github user anandsubbu commented on the issue:

    https://github.com/apache/metron/pull/829
  
    Hi @nickwallen , thank you for the fix. It is my bad, I should have added 
the line `commands = IndexingCommands(params)` before the `try` block in my 
previous fix :(.
    
    I am under way validating and will post my observation. However, given that 
the indexing templates are a must have for the alerts UI to work properly, 
should we remove the try/expect block and allow the install to fail in the 
event it was not successfully installed? Or do you think we should WARN and 
move on?


---

Reply via email to