Skip to content

Conversation

@kevinlog
Copy link
Contributor

Change Summary

Fixes a mapping error due to the inclusion of type: string in a recent change. The proposed change is to use type: keyword in place.

Failing test where this was caught: elastic/kibana#147640

Release Target

8.9

For mapping changes:

  • I ran make after making the schema changes, and committed all changes
  • If these field(s) are "exception"-able, I made a companion PR to Kibana adding it (see Readme)
  • If this is a metadata change, I also updated both transform destination schemas to match
@kevinlog kevinlog requested a review from a team as a code owner June 12, 2023 12:45
@elasticmachine
Copy link
Contributor

💚 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 preview

Expand to view the summary

Build stats

  • Start Time: 2023-06-12T12:45:33.059+0000

  • Duration: 9 min 45 sec

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@kevinlog
Copy link
Contributor Author

Going to merge this as it's blocking downstream work, we can update the type again in a followup

@kevinlog kevinlog merged commit 2355af2 into main Jun 12, 2023
ashokaditya added a commit to ashokaditya/kibana that referenced this pull request Jun 12, 2023
kevinlog added a commit to elastic/kibana that referenced this pull request Jun 12, 2023
) ## Summary Unskip Endpoint API tests after package fix: elastic/endpoint-package#373 Flaky runner to check that tests are passing: https://ci-stats.kibana.dev/trigger_flaky_test_runner/3 resolves: #72874 #159450 #147640 ### Checklist - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios
@kevinlog kevinlog deleted the task/fix-mapping-error branch June 19, 2023 13:43
@elasticmachine
Copy link
Contributor

Package endpoint - 8.9.0 containing this change is available at https://epr.elastic.co/search?package=endpoint

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

5 participants