GitHub user iraghumitra reopened a pull request:
https://github.com/apache/metron/pull/526
Metron-846: Add E2E tests for metron management ui
## Contributor Comments
This PR adds e2e test support for management ui. We have few e2e tests in
management ui but they do not work on quick-dev-platform. The PR makes
significant changes to the existing e2e code so I would request to review the
entire e2e tests in total.
- The e2e tests are written to run on quick-dev-platform
- Prerequisites for running e2e test are documented in metron-config README
- Steps to run the e2e tests are documented in metron-config README
- Implemented promises wherever applicable in .po files
## Pull Request Checklist
Thank you for submitting a contribution to Apache Metron (Incubating).
Please refer to our [Development
Guidelines](https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=61332235)
for the complete guide to follow for contributions.
Please refer also to our [Build Verification
Guidelines](https://cwiki.apache.org/confluence/display/METRON/Verifying+Builds?show-miniview)
for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow
these guidelines and ask you to double check the following:
### For all changes:
- [x] Is there a JIRA ticket associated with this PR? If not one needs to
be created at [Metron
Jira](https://issues.apache.org/jira/browse/METRON/?selectedTab=com.atlassian.jira.jira-projects-plugin:summary-panel).
- [x] Does your PR title start with METRON-XXXX where XXXX is the JIRA
number you are trying to resolve? Pay particular attention to the hyphen "-"
character.
- [x] Has your PR been rebased against the latest commit within the target
branch (typically master)?
### For code changes:
- [x] Have you included steps to reproduce the behavior or problem that is
being changed or addressed?
- [x] Have you included steps or a guide to how the change may be verified
and tested manually?
- [x] Have you ensured that the full suite of tests and checks have been
executed in the root incubating-metron folder via:
```
mvn -q clean integration-test install && build_utils/verify_licenses.sh
```
- [x] Have you written or updated unit tests and or integration tests to
verify your changes?
- [x] If adding new dependencies to the code, are these dependencies
licensed in a way that is compatible for inclusion under [ASF
2.0](http://www.apache.org/legal/resolved.html#category-a)?
- [x] Have you verified the basic functionality of the build by building
and running locally with Vagrant full-dev environment or the equivalent?
### For documentation related changes:
- [x] Have you ensured that format looks appropriate for the output in
which it is rendered by building and verifying the site-book? If not then run
the following commands and the verify changes via
`site-book/target/site/index.html`:
```
cd site-book
bin/generate-md.sh
mvn site:site
```
#### Note:
Please ensure that once the PR is submitted, you check travis-ci for build
issues and submit an update to your PR as soon as possible.
It is also recommened that [travis-ci](https://travis-ci.org) is set up for
your personal repository such that your branches are built there before
submitting a pull request.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/iraghumitra/incubator-metron METRON-846
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/metron/pull/526.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #526
----
commit 21300e21c99db32e2c4921f389db3bab6f6271da
Author: rmerriman <[email protected]>
Date: 2016-12-06T00:10:54Z
Merge branch 'middleware' into webapp
commit a7dea157fb9b02739ed9d4f5a647d97b086d4822
Author: rmerriman <[email protected]>
Date: 2016-12-06T00:11:57Z
Resolved licensing issues
commit d38b6265b4c0806826db6f77c7c036f6115db53e
Author: rmerriman <[email protected]>
Date: 2016-12-06T00:12:38Z
Resolved licensing issues
commit f1ce2d0ab740b9c67f4abe00e8e88d81b43dc8cd
Author: rmerriman <[email protected]>
Date: 2016-12-06T00:12:54Z
Merge branch 'docker' into middleware
commit a7b9e7e96aa339e922ec6d2098eeee563f726ced
Author: rmerriman <[email protected]>
Date: 2016-12-06T00:13:32Z
Merge branch 'middleware' into webapp
# Conflicts:
# pom.xml
commit f7bf12e12c979f541b4ac16174db39d1bf739bae
Author: rmerriman <[email protected]>
Date: 2016-12-06T00:15:28Z
Resolved licensing issues
commit 25bc44e3f50f21b447d0217a2eccb4184f743c24
Author: rmerriman <[email protected]>
Date: 2016-12-06T17:10:37Z
Updated scripts to add Hibernate and MySQL client to classpath
commit c03e83c012b6a9c007a3d584ce1a7a99e6a26dd3
Author: rmerriman <[email protected]>
Date: 2016-12-06T17:11:52Z
Merge branch 'middleware' into webapp
# Conflicts:
# metron-interface/metron-rest/src/main/scripts/start.sh
commit a91ce504ed0bd5bcc0fa98e77c6953f44b58bc23
Author: rmerriman <[email protected]>
Date: 2016-12-09T20:38:23Z
Created LICENSE/NOTICE files and made small adjustment to the README.
commit c91680007c2dd71ca590b615a1dc131efb6402ae
Author: rmerriman <[email protected]>
Date: 2016-12-09T21:43:52Z
Added license header to docker-compose.yml
commit b8ef856384a258b8a5674275526ffc56ae856938
Author: rmerriman <[email protected]>
Date: 2016-12-09T21:50:42Z
Merge branch 'METRON-622' into middleware
commit 6be6a0ac79931c3e5e55443646db5e0234818450
Author: rmerriman <[email protected]>
Date: 2016-12-09T22:34:50Z
Updated context paths and added assembly.xml file
commit 0a896491f248e9a587734180798bbe340bf9f284
Author: rmerriman <[email protected]>
Date: 2016-12-09T23:10:13Z
Merge branch 'METRON-503' into webapp
# Conflicts:
# metron-docker/docker-compose.yml
commit 6f39322ff0ad446599f6f3703afed1a0342e011c
Author: rmerriman <[email protected]>
Date: 2016-12-09T23:19:38Z
Small documentation updates
commit 786aa97664fb5e2b5ab6b8ffba9ce79436c212da
Author: rmerriman <[email protected]>
Date: 2016-12-12T19:20:24Z
Changes based on PR feedback:
- hbase config is now read from Storm classpath instead of hbase jar
- removed global config push from kafkazk init script
- added license to hbase/bin/init-commands.txt
- improved data generator script and updated documentation
commit 385bcb0821ea1f536c6dc48881eb0eff2c7f7df4
Author: rmerriman <[email protected]>
Date: 2016-12-12T20:06:19Z
Updated flatfile_loader.sh to only depend on HBase
commit 5d86575f46b1e2b75b140a19fb088e29b55b7fd8
Author: rmerriman <[email protected]>
Date: 2016-12-12T20:15:56Z
Fixed rest database name
commit edd4b872489fce58211add6c3baa96e0e9d2c008
Author: rmerriman <[email protected]>
Date: 2016-12-16T18:32:48Z
Changes based on PR feedback
commit 11fa4ae1634d683f545b7d75a46a131ae8295f0f
Author: rmerriman <[email protected]>
Date: 2016-12-16T18:42:20Z
Fixed README API links
commit 07f1a5fe209d8a29d835557a0e9a1836ed4a29ab
Author: rmerriman <[email protected]>
Date: 2016-12-19T21:05:28Z
Changes from PR feedback. Replaced install script with maven pom. Also
added more documentation.
commit bd4b81bfe459ac28324000b419d9328f82d57b87
Author: rmerriman <[email protected]>
Date: 2016-12-20T14:35:30Z
Added commons-lang:2.4 to dependencies_with_url.csv
commit 6b8babc20df2e93df9777104e050bd749a9d1966
Author: rmerriman <[email protected]>
Date: 2016-12-21T16:29:13Z
Merge branch 'METRON-622' into METRON-503
# Conflicts:
# pom.xml
commit adce9216629611208f496be291ab7f839b3a9d9c
Author: rmerriman <[email protected]>
Date: 2016-12-21T17:16:55Z
Removed config directory
commit 8040ab3faafcf80d549ff5166a0eb7392bd22f8e
Author: rmerriman <[email protected]>
Date: 2016-12-21T17:17:37Z
Merge branch 'METRON-622' into METRON-503
commit 583ff934e4127ebdcf85822e1d9724d98c730f64
Author: rmerriman <[email protected]>
Date: 2016-12-21T17:33:04Z
Merge branch 'METRON-503' into METRON-623
# Conflicts:
# metron-interface/metron-rest/README.md
commit 2861756c9c7601eca96c0e46148a24e37634a07d
Author: rmerriman <[email protected]>
Date: 2016-12-21T17:46:11Z
Updated docker compose path in application-docker.yml
commit 972475e379fdb7320316a57c2c36f12e83ef03c9
Author: rmerriman <[email protected]>
Date: 2016-12-21T17:47:22Z
Merge branch 'METRON-503' into METRON-623
commit 31d32612fb7fdee3b93fd95a8eb8ba881a480656
Author: iraghumitra <[email protected]>
Date: 2017-01-01T10:00:21Z
BUG-70448: Extra Storm topology status call is unnecessarily added for each
new sensor created
Removed setTimeout and using Observable.interval for polling
commit 9552ce96ccf989154eaec4d23bdb1e522cabd2af
Author: iraghumitra <[email protected]>
Date: 2017-01-03T14:54:17Z
BUG-71134: Improve Grok Statement Interface
-Integrated ACE editor
-Added ACE editor for JSON config
-Config panel to have extra width in case screen width is greater than
2020px
-Added help text for sample data component
-Removed plus icon for multiple input component
commit 07c46bb78fe879a0caa12e39e9304c2f3de57682
Author: iraghumitra <[email protected]>
Date: 2017-01-04T14:42:33Z
BUG-71134: Improve Grok Statement Interface
Fixed the test cases
Removeed the language tools will add it back once auto complete is
implemented
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
