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

Failing ES Serverless Promotion: FTR Configs #3 / Reporting Generate CSV from SearchSource with unmapped fields includes an unmapped field to the report #180010

Closed
mistic opened this issue Apr 4, 2024 · 2 comments · Fixed by #180113
Labels
failed-es-promotion failed-test A test failure on a tracked branch, potentially flaky-test impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience)

Comments

@mistic
Copy link
Member

mistic commented Apr 4, 2024

Serverless Observability API Integration Tests - Common Group 1
x-pack/test_serverless/api_integration/test_suites/common/reporting/generate_csv_discover.ts

Reporting Generate CSV from SearchSource with unmapped fields includes an unmapped field to the report

This failure is preventing the promotion of the current Elasticsearch Serverless image.

Error: expect(received).toMatchSnapshot()

Snapshot name: `Reporting Generate CSV from SearchSource with unmapped fields includes an unmapped field to the report 1`

- Snapshot  - 2
+ Received  + 2

  "_id","_index","_score",text,unmapped
- 1,recipes,1,text1,unmapped1
- 2,recipes,1,text2,"-"
+ 2,recipes,1,text2,"-"
+ 1,recipes,1,text1,unmapped1
  ↵
    at Context.<anonymous> (generate_csv_discover.ts:199:51)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at Object.apply (wrap_function.js:73:16)
@mistic mistic added blocker failed-test A test failure on a tracked branch, potentially flaky-test skipped-test impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. failed-es-promotion Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) v8.14.0 labels Apr 4, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@mistic
Copy link
Member Author

mistic commented Apr 4, 2024

Skipped.

main: 1825fcb

pheyos added a commit that referenced this issue Apr 5, 2024
## Summary

This PR reenables the serverless Discover CSV tests.

### Details

* The tests were failing for a new serverless ES image due to a change
in default shards
* The tests have been actually fixed in #179901
* Due to the way how the ES pipeline picks up Kibana changes for their
verification, it took a while for this fix to take effect (actually
needed to wait for the next daily scheduled run)
* In the mean time, the test appeared to still be failing, so it was
skipped entirely, see #180010
* With the ES serverless promotion now done, the skipped (and already
fixed) test suite can be reenabled

Closes #180010
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
failed-es-promotion failed-test A test failure on a tracked branch, potentially flaky-test impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants