-
Notifications
You must be signed in to change notification settings - Fork 24.4k
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
[buildkite] Add Windows packaging and platform support tests to periodic pipeline #98072
Conversation
Pinging @elastic/es-delivery (Team:Delivery) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple comments but otherwise LGTM.
|
||
SET JAVA_HOME=%USERPROFILE%\.java\%ES_BUILD_JAVA% | ||
SET JAVA11_HOME=%USERPROFILE%\.java\java11 | ||
SET JAVA16_HOME=%USERPROFILE%\.java\openjdk16 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, I'm wondering why we'd need Java 16. I think we can ditch this.
@@ -121,12 +121,12 @@ buildScan { | |||
buildScanPublished { scan -> | |||
// Attach build scan link as build metadata | |||
// See: https://buildkite.com/docs/pipelines/build-meta-data | |||
['buildkite-agent', 'meta-data', 'set', "build-scan-${System.getenv('BUILDKITE_JOB_ID')}", "${scan.buildScanUri}"].execute() | |||
new ProcessBuilder('buildkite-agent', 'meta-data', 'set', "build-scan-${System.getenv('BUILDKITE_JOB_ID')}", "${scan.buildScanUri}").start() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we also call waitFor()
here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, yep, thanks.
…dic pipeline (elastic#98072) (cherry picked from commit dfadca8)
💚 All backports created successfully
Questions ?Please refer to the Backport tool documentation |
This PR adds the platform/packaging tests that are run on Windows to the new Buildkite periodic pipeline.
Linux jobs will be added in a separate PR.
See here for test builds: https://buildkite.com/elastic/brianseeders-elasticsearch-sandbox/builds?branch=buildkite-migration-periodic-windows
See also: https://elasticco.atlassian.net/jira/software/c/projects/ES/boards/297?modal=detail&selectedIssue=ES-6358
run-gradle.sh
also work with Windows (previously unix only)ProcessBuilder.start()