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

Run tests after building java tools. #8205

Closed

Conversation

iirina
Copy link
Contributor

@iirina iirina commented Apr 30, 2019

This PR adds a shell integration test for the java rules that is run with the java toolchain of the java tools build from head. The test is created and run by src/upload_java_tools.sh which is used by the Buildkite pipeline to generate the java tools zip.

The generated java tools zip is uploaded to GCS only if the test succeeded. This provides testing before creating release candidates for the java tools.

@iirina iirina force-pushed the run_tests_after_building_java_tools branch 2 times, most recently from 5b7da11 to 5392b4e Compare May 6, 2019 07:52
@irengrig irengrig added the WIP label May 7, 2019
@iirina iirina force-pushed the run_tests_after_building_java_tools branch 3 times, most recently from 8dd23d0 to cf87b44 Compare May 8, 2019 14:36
@iirina iirina marked this pull request as ready for review May 8, 2019 14:48
@iirina iirina requested a review from lberki May 8, 2019 14:48
@iirina iirina removed the WIP label May 8, 2019
src/test/shell/bazel/bazel_java_test.sh Outdated Show resolved Hide resolved
src/upload_java_tools.sh Show resolved Hide resolved
src/upload_all_java_tools.sh Outdated Show resolved Hide resolved
@iirina iirina force-pushed the run_tests_after_building_java_tools branch from cf87b44 to 71799e4 Compare May 9, 2019 08:50
@iirina
Copy link
Contributor Author

iirina commented May 9, 2019

@lberki comments addressed!

src/test/shell/bazel/BUILD Outdated Show resolved Hide resolved
src/test/shell/bazel/BUILD Outdated Show resolved Hide resolved
src/upload_java_tools.sh Show resolved Hide resolved
@bazel-io bazel-io closed this in 1df1635 May 9, 2019
irengrig pushed a commit to irengrig/bazel that referenced this pull request Jun 18, 2019
This PR adds a shell integration test for the java rules that is run with the java toolchain of the java tools build from head. The test is created and run by `src/upload_java_tools.sh` which is used by the Buildkite pipeline to generate the java tools zip.

The generated java tools zip is uploaded to GCS only if the test succeeded. This provides testing before creating release candidates for the java tools.

Closes bazelbuild#8205.

PiperOrigin-RevId: 247408063
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants