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

Fix mapping error by replacing string with keyword #373

Merged
merged 1 commit into from
Jun 12, 2023

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
6 checks passed
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
saarikabhasi pushed a commit to saarikabhasi/kibana that referenced this pull request Jun 13, 2023
…tic#159484)

## 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:
elastic#72874
elastic#159450
elastic#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
saarikabhasi pushed a commit to saarikabhasi/kibana that referenced this pull request Jun 13, 2023
…tic#159484)

## 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:
elastic#72874
elastic#159450
elastic#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
saarikabhasi pushed a commit to saarikabhasi/kibana that referenced this pull request Jun 14, 2023
…tic#159484)

## 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:
elastic#72874
elastic#159450
elastic#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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants