[GitHub] metron issue #803: Metron-1252: Build ui for grouping alerts into meta alert...

[iraghumitra <gi...@git.apache.org>]

Github user iraghumitra commented on the issue:

    https://github.com/apache/metron/pull/803
  
    @merrimanr Looks like the following rest-api's are not working as expected
     - When a meta alert is created and when we try to group by an IP address(ip_dst_addr/ip_src_addr) the IP address is shown as string instead of IP
     - When a meta alert is created the alert details of an alert under meta-alert are missing when we query using findone
    
    The spec that is failing is unable to find state field in the details view bcoz of findone issue  that i mentioned hence the failure. (The alert-detail-status.e2e-spec.ts set's the status of the alert that it is trying to test to [NEW](https://github.com/apache/metron/blob/383d798c7827e916068f89ff362a6ecccb550241/metron-interface/metron-alerts/e2e/alert-details/alert-status/alert-details-status.e2e-spec.ts#L55) before starting. So I feel the issue might not be bcoz of #796. Anyhow, I will check this again)
    
    I am trying to address the usage of sleep in specs by removing the 'fade' transition for dialogs. We don't need this since the dialog comes on top of the window and the css transition is not needed.
    
    Also, i noticed that sometimes when we delete a meta alert not all the alerts in the meta alert are returned back the alerts pool.Ex: If we have 100 alerts and create a meta-alert with 50 alerts and then delete the meta alert. The alert search shows only 60 alerts.


---