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

ASF GitHub Bot commented on METRON-1344:
----------------------------------------

Github user cestella commented on a diff in the pull request:

    https://github.com/apache/metron/pull/858#discussion_r156182999
  
    --- Diff: 
metron-interface/metron-alerts/e2e/alerts-list/configure-table/configure-table.e2e-spec.ts
 ---
    @@ -47,7 +47,7 @@ describe('metron-alerts configure table', function() {
         let newColNamesColumnConfig = [ 'score', 'timestamp', 'source:type', 
'ip_src_addr', 'enrichments:geo:ip_dst_addr:country',
           'ip_dst_addr', 'host', 'alert_status', 'guid' ];
     
    -    page.clearLocalStorage();
    +    //page.clearLocalStorage();
    --- End diff --
    
    Can we remove this line instead of commenting it out?


> Externalize the infrastructural components using integration tests
> ------------------------------------------------------------------
>
>                 Key: METRON-1344
>                 URL: https://issues.apache.org/jira/browse/METRON-1344
>             Project: Metron
>          Issue Type: Improvement
>            Reporter: Ryan Merriman
>            Assignee: Ryan Merriman
>
> As we move towards adding the Alerts UI e2e test to our continuous build, we 
> need to remove the dependence on vagrant full dev and provide equivalent 
> infrastructure within Travis.  This will include all the services that the 
> e2e tests depend on including Kafka, Zookeeper, Elasticsearch (possibly 
> HBase), and Metron REST.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to