[CI] Add skeleton for integrations on Windows #1713
Conversation
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
🤖 GitHub commentsTo re-run your PR in the CI, just comment with:
|
marc-gr
force-pushed
the
feature/refactor-to-support-windows-integrations
branch
from
b636a52 to
62b5b5a
Compare
marc-gr
force-pushed
the
feature/refactor-to-support-windows-integrations
branch
from
62b5b5a to
30610b4
Compare
…thub.com:v1v/integrations into feature/refactor-to-support-windows-integrations * 'feature/refactor-to-support-windows-integrations' of github.com:v1v/integrations: (200 commits) [CI] Draft for windows system tests [journald] Create custom journald input package (elastic#1739) [Microsoft_DHCP] New package replacing old RSA2ELK DHCP package (elastic#1793) Fix contains check for forwarded tag (elastic#1861) [google_workspace] Fix contains check for forwarded tag (elastic#1819) Fix contains check for forwarded tag (elastic#1805) Fix contains check for forwarded tag (elastic#1815) Fix contains check for forwarded tag (elastic#1816) Fix contains check for forwarded tag (elastic#1817) Fix contains check for forwarded tag (elastic#1818) Fix contains check for forwarded tag (elastic#1820) Fix contains check for forwarded tag (elastic#1821) Fix contains check for forwarded tag (elastic#1822) Fix contains check for forwarded tag (elastic#1823) Fix contains check for forwarded tag (elastic#1824) Fix contains check for forwarded tag (elastic#1825) Fix contains check for forwarded tag (elastic#1826) Fix contains check for forwarded tag (elastic#1827) Fix contains check for forwarded tag (elastic#1828) Fix contains check for forwarded tag (elastic#1829) ...
.ci/Jenkinsfile
Outdated
|
|
||
| def runWindowsSystemTests(name) { | ||
| return { | ||
| def linuxIp = grabWorkerIP() |
There was a problem hiding this comment.
Look for the ip in the current linux worker.
.ci/Jenkinsfile
Outdated
| // TODO: run the stack that's needed | ||
|
|
There was a problem hiding this comment.
prepare the context in the linux worker (services and so on)
.ci/Jenkinsfile
Outdated
| def envContext = [] | ||
| envContext.add("KIBANA_URL=https://${linuxIp}:5601") | ||
| envContext.add("ELASTICSEARCH_URL=https://${linuxIp}:9200") |
There was a problem hiding this comment.
Setup the environment variables fro the services that run in the linux worker then the windows worker will consume them
.ci/Jenkinsfile
Outdated
| // TOOD use the batch syntax in the pipelien and in the script | ||
| sh(label: "Test integration: ${name}", script: ''' | ||
| eval "$(../../build/elastic-package stack shellinit)" | ||
| ../../build/elastic-package test -v --report-format xUnit --report-output file --test-coverage | ||
| ''') |
There was a problem hiding this comment.
This is needed to be changed to be Windows based
| envContext.add("ELASTICSEARCH_URL=https://${linuxIp}:9200") | ||
|
|
||
| withNode(labels: 'windows-2019 && docker && immutable', sleepMin: 10, sleepMax: 100) { | ||
| withEnv(envContext) { |
There was a problem hiding this comment.
Load the above defined environment variables that contain the IP of the linux worker
🌐 Coverage report
|
|
Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as |
|
I'll close this PR, you can freely reopen it if this integration is something needed in the future |
What does this PR do?
Refactor pipeline to support windows integrations. It requires to implement
isIntegrationForWindowsChecklist
changelog.ymlfile.manifest.ymlfile to point to the latest Elastic stack release (e.g.^7.13.0).Author's Checklist
How to test this PR locally
Related issues
Screenshots