Skip to content

[7.x] add isExportable SO export API (#101860)#102698

Merged
pgayvallet merged 2 commits intoelastic:7.xfrom
pgayvallet:backport/7.x/pr-101860
Jun 21, 2021
Merged

[7.x] add isExportable SO export API (#101860)#102698
pgayvallet merged 2 commits intoelastic:7.xfrom
pgayvallet:backport/7.x/pr-101860

Conversation

@pgayvallet
Copy link
Contributor

Backports the following commits to 7.x:

* add `isExportable` SO export API

* add warning when export contains excluded objects

* add FTR test

* fix API integration assertions

* lint

* fix assertions again

* doc

* update generated doc

* fix esarchiver paths

* use maps instead of objects

* SavedObjectsExportablePredicate is no longer async

* more docs

* generated doc

* use info instead of warning when export contains excluded objects

* try/catch on isExportable call and add exclusion reason

* add FTR test for errored objects

* log error if isExportable throws
@pgayvallet pgayvallet added the backport This PR is a backport of another PR label Jun 21, 2021
@pgayvallet pgayvallet enabled auto-merge (squash) June 21, 2021 08:08
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
core 1053 1054 +1

Async chunks

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

id before after diff
savedObjectsManagement 138.4KB 138.8KB +430.0B

Public APIs missing exports

Total count of every type that is part of your API that should be exported but is not. This will cause broken links in the API documentation system. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats exports for more detailed information.

id before after diff
core 26 27 +1
Unknown metric groups

API count

id before after diff
core 2276 2283 +7

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

@pgayvallet pgayvallet merged commit 9cf01f1 into elastic:7.x Jun 21, 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