-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[junit] make sure that report paths are unique #81255
[junit] make sure that report paths are unique #81255
Conversation
💛 Build succeeded, but was flaky
Test FailuresChrome X-Pack UI Functional Tests.x-pack/test/functional/apps/monitoring/logstash/pipelines·js.Monitoring app Logstash pipelines "before all" hook for "should have Logstash Cluster Summary Status showing correct info"Standard Out
Stack Trace
Metrics [docs]
To update your PR or re-run it, just comment with: |
Pinging @elastic/kibana-operations (Team:Operations) |
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.
LGTM - also verified tests are being reported in https://kibana-ci.elastic.co/job/elastic+kibana+pipeline-pull-request/82942/testReport/
Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: spalger <spalger@users.noreply.github.com> # Conflicts: # packages/kbn-test/src/junit_report_path.ts # src/dev/jest/integration_tests/junit_reporter.test.js # src/dev/jest/junit_reporter.js
Co-authored-by: spalger <spalger@users.noreply.github.com> # Conflicts: # packages/kbn-test/src/jest/integration_tests/junit_reporter.test.ts # packages/kbn-test/src/jest/junit_reporter.ts # packages/kbn-test/src/jest/report_path.ts # packages/kbn-test/src/mocha/junit_report_generation.js
Co-authored-by: spalger <spalger@users.noreply.github.com> # Conflicts: # packages/kbn-test/src/jest/integration_tests/junit_reporter.test.ts # packages/kbn-test/src/jest/junit_reporter.ts # packages/kbn-test/src/jest/report_path.ts # packages/kbn-test/src/mocha/junit_report_generation.js
* master: (63 commits) [KP] Fix Headers timeout issue (elastic#81140) [ML] Functional tests - stabilize typing with checks service method (elastic#81338) tabify - support docs (elastic#80351) [Security Solution][Detections] Look-back time logic fix (elastic#81383) [Workplace Search] Add top-level tests for Groups (elastic#81215) [Fleet] Fix agent action observable for long polling (elastic#81376) [Maps] fix feature tooltip remains open when zoom level change hides layer (elastic#81373) skip flaky suite (elastic#78689) chore(NA): add spec-to-console and plugin-helpers as devOnly dependencies (elastic#81357) Ensure some data is returned (elastic#81375) Change dumb-init to tini (elastic#81126) [Reporting/Tech Debt] Convert PdfMaker class to TypeScript (elastic#81242) Use Storybook Controls instead of Knobs (elastic#80705) [junit] make sure that report paths are unique (elastic#81255) bump elastic/elasticsearch-js version to 7.10.0-rc1 (elastic#81288) run ssl tests on CI (elastic#81320) Fix alert defaults (elastic#81207) [ML] DF Analytics wizard: ensure user can set mml manually or select to use given estimate (elastic#81078) Add UI notifier to indicate secret fields and to remember / reenter values (elastic#80657) [Monitoring] Use async/await (elastic#81200) ...
Fixes #81253
Instead of overwriting report files when using the same reportName as another FTR config, or in TeamCity when we will (probably) just rerun the FTR config with the same arguments to retry, we want to avoid overwriting the existing junit output.