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

[CI] Mandatory windows support for all the versions #23615

Merged
merged 4 commits into from
Jan 26, 2021

Conversation

v1v
Copy link
Member

@v1v v1v commented Jan 21, 2021

What does this PR do?

Mandatory stages in the CI for older windows versions on a PR basis, similar to what it runs on master/7.x/7.[0-9]+

Why is it important?

Discover incompatibility issues earlier rather than when merged.

Further details

Jenkinsfile.yml supports stages with multiple platforms, if we use that approach then the test output format will change and might affect the Test Dashboards regarding the top-10 failures or the flakiness analysis.

If we do so, we might need to run some migration.

@v1v v1v requested review from a team as code owners January 21, 2021 14:49
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label Team:Automation Label for the Observability productivity team Team:Ingest Management labels Jan 21, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ingest-management (Team:Ingest Management)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Jan 21, 2021
@v1v v1v self-assigned this Jan 21, 2021
@elasticmachine
Copy link
Collaborator

elasticmachine commented Jan 21, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: v1v commented: /test

    • Start Time: 2021-01-26T09:42:15.285+0000
  • Duration: 47 min 25 sec

  • Commit: 69b66fd

Test stats 🧪

Test Results
Failed 0
Passed 45149
Skipped 4737
Total 49886

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 45149
Skipped 4737
Total 49886

@v1v
Copy link
Member Author

v1v commented Jan 21, 2021

/test

@v1v
Copy link
Member Author

v1v commented Jan 25, 2021

Linting caused by another PR that did not change the x-pack, and test failure unrelated to this changes.

.ci/scripts/generate_build_table.py Outdated Show resolved Hide resolved
@v1v
Copy link
Member Author

v1v commented Jan 25, 2021

/test

1 similar comment
@v1v
Copy link
Member Author

v1v commented Jan 26, 2021

/test

@v1v v1v merged commit 3894f0c into elastic:master Jan 26, 2021
@v1v v1v deleted the feature/mandatory-windows branch January 26, 2021 10:41
v1v added a commit to v1v/beats that referenced this pull request Jan 26, 2021
v1v added a commit to v1v/beats that referenced this pull request Jan 26, 2021
v1v added a commit to v1v/beats that referenced this pull request Jan 26, 2021
…pack-when-oss-changes

* upstream/master:
  [DOCS] Add setup content to Kubernetes and Cloud Foundry docs (elastic#23580)
  [CI] Mandatory windows support for all the versions (elastic#23615)
  Add check when retrieving the worker process id using performance counters  (elastic#23647)
  Remove 4912 evtx from testing (elastic#23669)
  Add missing SSL settings (elastic#23632)
  Update X-Pack Packetbeat config (elastic#23666)
  Use hostname check from verify.go to handle patterns in TLS certs (elastic#23661)
  Fix: Dissect Cisco ASA 302013 message usernames (elastic#21196)
  Add FAQ entry for MADV settings in older versions (elastic#23429)
  Sync fixes from Integration Package Testing (elastic#23424)
  [Filebeat] Add Cisco ASA message '302023' parsing (elastic#23092)
  [Elastic Log Driver] Change hosts config flag (elastic#23628)
  Audit and Authentication Policy Change Events (elastic#20684)
v1v added a commit to v1v/beats that referenced this pull request Jan 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Automation Label for the Observability productivity team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants