Skip to content

[7.x] [ILM] Policy form should not throw away data (#83077)#83893

Merged
jloleysens merged 2 commits intoelastic:7.xfrom
jloleysens:backport/7.x/pr-83077
Nov 23, 2020
Merged

[7.x] [ILM] Policy form should not throw away data (#83077)#83893
jloleysens merged 2 commits intoelastic:7.xfrom
jloleysens:backport/7.x/pr-83077

Conversation

@jloleysens
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

* fix ilm policy deserialization

* reorder expected jest object to match actual

* fix removal of wait for snapshot if it is not on the form

* add client integration test for policy serialization of unknown fields

* save on a few chars

* added unit test for deserializer and serializer

* Implement feedback

- move serializer function around a little bit
- move serialize migrate and allocate function out of serializer
  file

* Updated serialization unit test coverage

- removed the "forcemergeEnabled" meta field that was not being
  used
- added test cases for deleting of values from policies

* fixed minor issue in how serialization tests are being set up

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@jloleysens jloleysens added the backport This PR is a backport of another PR label Nov 20, 2020
@jloleysens
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
indexLifecycleManagement 110 112 +2

Async chunks

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

id before after diff
indexLifecycleManagement 211.1KB 212.6KB +1.4KB

History

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

@jloleysens jloleysens merged commit 968b6e5 into elastic:7.x Nov 23, 2020
@jloleysens jloleysens deleted the backport/7.x/pr-83077 branch November 23, 2020 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants