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

Backport PR #15861 to 7.17: Allow running Java+Ruby tests on Windows separately #15887

Merged
merged 1 commit into from
Feb 1, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Feb 1, 2024

Backport PR #15861 to 7.17 branch, original message:


Release notes

[rn:skip]

What does this PR do?

This commit allows separate running of Java and Ruby tests on Windows i.e. the same way as we currently do on unix (unit_tests.sh) via a cli argument.
If no argument has been supplied, both tests are run (as it does now).

The wrapper script is also rewritten from old batch style script to powershell.

Why is it important/What is the impact to the user?

It allows us to split the existing Windows CI job in a subsequent PR to separate steps, as we currently do for Linux.

How to test this PR locally

Launch Windows VM using https://github.com/elastic/ingest-dev/blob/main/engineer-productivity/tools/launch_windows_vm.sh
launch powershell.
Git clone this PR and run:

.\ci\unit_tests.ps1 java (or skip the argument to run all)

To simulate a failure you can revert c33afd4

Example successful run on Buildkite:

Simulating a failure:

I temporarily reverted the muted Windows test to verify that a failure bubbles up as a test failure:

Related issues

This commit allows separate running of Java and Ruby tests on Windows i.e. the same way as we currently do on unix (unit_tests.sh) via a cli argument.
If no argument has been supplied, both tests are run (as it does now).

The wrapper script is also rewritten from old batch style script to Powershell.

This work allows us to split the existing Windows CI job in a subsequent PR to separate steps, as we currently do on Linux.

Relates: #15566
(cherry picked from commit 8ac5518)
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

@dliappis dliappis merged commit 3420d4c into 7.17 Feb 1, 2024
2 checks passed
@dliappis dliappis deleted the backport_15861_7.17 branch February 1, 2024 08:40
@dliappis dliappis added the ci label Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants