Skip to content

[7.x] Gracefully handle decryption errors during ESO migrations (#105968)#107051

Merged
ymao1 merged 2 commits intoelastic:7.xfrom
kibanamachine:backport/7.x/pr-105968
Jul 29, 2021
Merged

[7.x] Gracefully handle decryption errors during ESO migrations (#105968)#107051
ymao1 merged 2 commits intoelastic:7.xfrom
kibanamachine:backport/7.x/pr-105968

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

…5968)

* Updating unit tests

* Fixing types

* Updating readme and adding warning message

* Updating README

* PR fixes

* collapsing args to create migration fn

* Adding functional tests

* Adding comment to functional test

* Adding stripOrDecryptAttributesSync

* Using stripOrDecryptAttributesSync

* Fixing unit tests

* PR fixes

* PR fixes

* Moving validation of apikey existence in alerting task runner

* Cleanup

* Reverting changes to alerting task runner

* PR fixes

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jul 28, 2021
@kibanamachine kibanamachine enabled auto-merge (squash) July 28, 2021 17:07
@ymao1 ymao1 disabled auto-merge July 28, 2021 17:07
@ymao1
Copy link
Copy Markdown
Contributor

ymao1 commented Jul 29, 2021

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor Author

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

API count

id before after diff
encryptedSavedObjects 30 28 -2

API count missing comments

id before after diff
encryptedSavedObjects 28 26 -2

Non-exported public API item count

id before after diff
encryptedSavedObjects 3 4 +1

History

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

cc @ymao1

@ymao1 ymao1 merged commit 7b9a76d into elastic:7.x Jul 29, 2021
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