[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

2018-02-22 Thread merrimanr
Github user merrimanr closed the pull request at: https://github.com/apache/metron/pull/858 ---

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

2017-12-14 Thread merrimanr
Github user merrimanr closed the pull request at: https://github.com/apache/metron/pull/858 ---

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

2017-12-14 Thread merrimanr
GitHub user merrimanr reopened a pull request: https://github.com/apache/metron/pull/858 METRON-1344: Externalize the infrastructural components using integration tests ## Contributor Comments This PR will add infrastructure to our Travis build that will allow the Alerts UI

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

2017-12-12 Thread merrimanr
GitHub user merrimanr reopened a pull request: https://github.com/apache/metron/pull/858 METRON-1344: Externalize the infrastructural components using integration tests ## Contributor Comments This PR will add infrastructure to our Travis build that will allow the Alerts UI

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

2017-12-12 Thread merrimanr
Github user merrimanr closed the pull request at: https://github.com/apache/metron/pull/858 ---

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

2017-12-11 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/858#discussion_r156215449 --- Diff: metron-interface/metron-alerts/protractor.conf.js --- @@ -25,27 +25,28 @@ var SpecReporter = require('jasmine-spec-reporter').SpecReporter;

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

2017-12-11 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/858#discussion_r156215392 --- Diff: metron-interface/metron-alerts/e2e/alerts-list/configure-table/configure-table.e2e-spec.ts --- @@ -58,26 +58,26 @@ describe('metron-alerts

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

2017-12-11 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/858#discussion_r156215355 --- Diff: metron-interface/metron-alerts/e2e/alerts-list/configure-table/configure-table.e2e-spec.ts --- @@ -47,7 +47,7 @@ describe('metron-alerts

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

2017-12-11 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/858#discussion_r156183357 --- Diff: metron-interface/metron-alerts/protractor.conf.js --- @@ -25,27 +25,28 @@ var SpecReporter = require('jasmine-spec-reporter').SpecReporter;

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

2017-12-11 Thread cestella
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

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

2017-12-11 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/858#discussion_r156183048 --- Diff: metron-interface/metron-alerts/e2e/alerts-list/configure-table/configure-table.e2e-spec.ts --- @@ -58,26 +58,26 @@ describe('metron-alerts

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

2017-12-06 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/858#discussion_r155339646 --- Diff: metron-interface/metron-alerts/pom.xml --- @@ -130,4 +146,44 @@ + + + + e2e

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

2017-12-06 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/858#discussion_r155339627 --- Diff: metron-interface/metron-alerts/e2e/utils/e2e_util.ts --- @@ -46,25 +48,53 @@ export function waitForStalenessOf (_element ) { }

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

2017-12-06 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/858#discussion_r155288566 --- Diff: metron-interface/metron-alerts/pom.xml --- @@ -130,4 +146,44 @@ + + + +

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

2017-12-06 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/858#discussion_r155288298 --- Diff: metron-interface/metron-alerts/e2e/utils/e2e_util.ts --- @@ -46,25 +48,53 @@ export function waitForStalenessOf (_element ) { }

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

2017-12-06 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/metron/pull/858 METRON-1344: Externalize the infrastructural components using integration tests ## Contributor Comments This PR will add infrastructure to our Travis build that will allow the Alerts UI e2e