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 geode into open source packages testing #543

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

Add geode into open source packages testing #543

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')

Comments

@mamatha-jv
Copy link
Contributor

This issue is opened to track the process of adding geode 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
@Mesbah-Alam
Copy link
Contributor

@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 geocode 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.

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
None yet
Development

No branches or pull requests

3 participants