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

[ML] Fix Transforms not retaining _meta on clone #116206

Merged
merged 6 commits into from
Oct 28, 2021

Conversation

qn895
Copy link
Member

@qn895 qn895 commented Oct 25, 2021

Summary

This PR fixes #115546 and retains the _meta field on clone.

Checklist

Delete any items that are not applicable to this PR.

@qn895 qn895 added :ml v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Feature:Transforms ML transforms v7.16.0 labels Oct 25, 2021
@qn895 qn895 self-assigned this Oct 25, 2021
@qn895 qn895 marked this pull request as ready for review October 26, 2021 14:13
@qn895 qn895 requested a review from a team as a code owner October 26, 2021 14:13
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

Copy link
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested latest edits and LGTM

Copy link
Contributor

@darnautov darnautov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just one suggestion regarding types

@qn895 qn895 enabled auto-merge (squash) October 27, 2021 16:08
@darnautov
Copy link
Contributor

@elasticmachine merge upstream

@qn895 qn895 merged commit 21cd11a into elastic:master Oct 28, 2021
@kibanamachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Test Failures

  • [job] [logs] Default CI Group #12 / detection engine api security and spaces enabled Detection exceptions data types and operators Rule exception operators for data type ip "is in list" operator will return 0 results if we have a list that includes all ips

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
transform 327.2KB 327.3KB +71.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @qn895

@darnautov darnautov added the auto-backport Deprecated - use backport:version if exact versions are needed label Oct 28, 2021
@kibanamachine
Copy link
Contributor

The following labels were identified as gaps in your version labels and will be added automatically:

  • v8.1.0

If any of these should not be on your pull request, please manually remove them.

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 28, 2021
* [ML] Retain _meta on clone

* [ML] Fix validation on schema to only check it it's defined/not null

* [ML] Remove validation because es should handle the validation already

* Change type to unknown
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 28, 2021
* [ML] Retain _meta on clone

* [ML] Fix validation on schema to only check it it's defined/not null

* [ML] Remove validation because es should handle the validation already

* Change type to unknown
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
8.0
7.16

The backport PRs will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Oct 28, 2021
* [ML] Retain _meta on clone

* [ML] Fix validation on schema to only check it it's defined/not null

* [ML] Remove validation because es should handle the validation already

* Change type to unknown

Co-authored-by: Quynh Nguyen <[email protected]>
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Oct 29, 2021
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Nov 1, 2021
qn895 added a commit that referenced this pull request Nov 1, 2021
* [ML] Retain _meta on clone

* [ML] Fix validation on schema to only check it it's defined/not null

* [ML] Remove validation because es should handle the validation already

* Change type to unknown

Co-authored-by: Quynh Nguyen <[email protected]>
@qn895 qn895 deleted the transform-retain-meta-on-clone branch November 2, 2021 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed Feature:Transforms ML transforms :ml release_note:skip Skip the PR/issue when compiling release notes v7.16.0 v8.0.0 v8.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ML] Transforms: _meta field not retained on clone
5 participants