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

SO Repository: Adding multi-NS support to remaining non-bulk operations #109996

Closed
pgayvallet opened this issue Aug 25, 2021 · 3 comments
Closed
Labels
discuss Feature:Saved Objects Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc

Comments

@pgayvallet
Copy link
Contributor

If most of the bulk APIs of the SOR now support handling either multiple namespaces (e.g find) or to specify the target namespace for each object (e.g bulkCreate), we're lacking the equivalent for most of the non-bulk operation (get or delete for example).

From my understanding, this state is mostly caused by two factors:

  • We decided to not alter each individual API until we have a real need to support specifying the namespace the for operation
  • Historically, we're not allowing to specify the namespace option of the SOR apis (when spaces is enabled), as this is a reserved option that should currently only be populated by the spaces wrapper, which introduces the question of how we should allow API consumers to specify the namespace when invoking such APIs.

After some discussions in #109197, (where we would have needed to specify a target namespace for delete) one option could be (see #109197 (comment)) to refactor the space wrapper and the base SOR options to rename the current namespace option to currentNamespace. This new property will preserve all the particularities of the old option (considered internal, only populated by the spaces wrapper), while the new-with-old-name namespace option would now be able to be specified by our API consumers.

@pgayvallet pgayvallet added discuss Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! Feature:Saved Objects labels Aug 25, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-security (Team:Security)

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core (Team:Core)

@exalate-issue-sync exalate-issue-sync bot added impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. loe:small Small Level of Effort labels Sep 10, 2021
@legrego legrego removed loe:small Small Level of Effort impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. labels Aug 22, 2022
@legrego legrego removed the Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! label Jan 13, 2023
@rayafratkina
Copy link
Contributor

We currently have no plans to improve these APIs

@rayafratkina rayafratkina closed this as not planned Won't fix, can't repro, duplicate, stale Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss Feature:Saved Objects Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc
Projects
None yet
Development

No branches or pull requests

4 participants