-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Ingest] Make all saved objects namespace agnostic, move to NP plugin #64360
Conversation
Pinging @elastic/ingest-management (Feature:EPM) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm going to pull this down and try it out, but it reads clearly to me. I like the changes to the names and appreciate swapping out the strings with variables 👏
}, | ||
}, | ||
}, | ||
}, | ||
}; | ||
|
||
export function registerSavedObjects(savedObjects: SavedObjectsServiceSetup) { | ||
Object.entries(savedObjectTypes).forEach(([, type]) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we don't care about the key, Object.values
is also an option.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I checked it out and did several operations (install/uninstall packages, datasources, agents, and agent configs) and everything worked well.
I spot checked a few of the saved objects using the _find
API and everything looked as expected.
@jfsiii Thanks for testing! |
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
…elastic#64360) * Move saved object type registration & mapping to NP plugin; normalize and prefix saved object type names * Replace hardcoded saved object type names with constants * Adjust output type name * Review and import fixes * Update tests, fixtures, and instructions
…#64360) (#64465) * Move saved object type registration & mapping to NP plugin; normalize and prefix saved object type names * Replace hardcoded saved object type names with constants * Adjust output type name * Review and import fixes * Update tests, fixtures, and instructions Co-authored-by: Elastic Machine <[email protected]>
Summary
Note: To test this PR, you will need to clear your ES data and do
yarn kbn bootstrap
.1) Commit b6b658e
.kibana
index which has a large number of other types registered to it.datasources
andagents
as names for example can be ambiguous and may conflict in the future with other apps.2) Commit 0a8d0d3