Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add hbase into open source packages testing #544

Closed
mamatha-jv opened this issue Sep 10, 2018 · 3 comments
Closed

Add hbase into open source packages testing #544

mamatha-jv opened this issue Sep 10, 2018 · 3 comments
Assignees
Labels
enhancement type:external 3rd party application tests (invoked by the make target 'external')
Projects

Comments

@mamatha-jv
Copy link
Contributor

This issue is opened to track the process of adding hbase into open source packages testing at https://github.com/AdoptOpenJDK/openjdk-tests/tree/master/thirdparty_containers

Refer issue #172 for more details.

@mamatha-jv mamatha-jv self-assigned this Sep 10, 2018
@mamatha-jv mamatha-jv added enhancement type:external 3rd party application tests (invoked by the make target 'external') labels Sep 10, 2018
@karianna karianna added this to TODO in aqa-tests via automation Sep 10, 2018
@Mesbah-Alam
Copy link
Contributor

Mesbah-Alam commented Sep 11, 2018

@mamatha-jv : Just an fyi, we should make the same change for this PR as was made in the Jenkins test -- which is adding a docker cp step in the command line to copy the junit xml reports out of the docker container & into the host machine's workspace. This allows the Jenkins reporting plugin to generate test reports at the end of the test build. Here's the PR for the Jenkins test change that can be taken as example: https://github.com/AdoptOpenJDK/openjdk-tests/pull/537/files.

So, When you run the hbase tests in your local, find out where it generates the xml junit reports and use that path in the docker cp command in playlist.xml. (Tip, when you create the Jenkins scripts for hbsase, start off by making a copy of the "jenkins" folder - which already has the change - and add your updates on top.)

@smlambert
Copy link
Contributor

Is this issue being worked? If not, I will suggest we close it (leaving the app in the list at #172), until someone is actually working on it.

@smlambert
Copy link
Contributor

I will close for now and keep geode in the #172 list.

aqa-tests automation moved this from TODO to Done Mar 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement type:external 3rd party application tests (invoked by the make target 'external')
Projects
No open projects
aqa-tests
  
Done
Development

No branches or pull requests

3 participants