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

[Feature/extensions] Run AD backend plugin as an Extension to create a detector #3635

Closed
owaiskazi19 opened this issue Jun 20, 2022 · 0 comments
Assignees
Labels
Build Libraries & Interfaces enhancement Enhancement or improvement to existing feature or request Meta Meta issue, not directly linked to a PR

Comments

@owaiskazi19
Copy link
Member

owaiskazi19 commented Jun 20, 2022

Is your feature request related to a problem? Please describe.
To move forward with Anomaly Detection backend plugin as an extension, we plan to start with creating a detector as the first feature to pick up. Currently, we are targeting the detector without scheduling the job for it (removes the dependencies of Job Scheduler) and without auth user(removes the dependencies of Common Utils).To integrate the detector from AD boils down to the following dependencies from OpenSearch

  1. AnomalyDetectorPlugin -> getActions() -> IndexAnomalyDetectorTransportAction
    Dependencies from OpenSearch:
    a. client
    b. transportService
    c. clusterService
    d. NamedXContentRegistry

  2. doExecute
    Dependencies from OpenSearch:
    a. ActionListener actionListener
    b. User from Common Utils

  3. initAnomalyDetectorIndex
    Dependencies from OpenSearch:
    a. CreateIndexRequest API

The action items to achieve the above:

  1. [FEATURE] Utilize the codebase required to create a detector from Anomaly Detector Plugin opensearch-sdk-java#20.
  2. [META] Create Transport APIs for the dependencies of OpenSearch for creating detector opensearch-sdk-java#21
  3. [META] APIs for Extension Point for creating a detector of AD opensearch-sdk-java#100
  4. [FEATURE] Remove dependent code associated with the Client thread pool/context in AD for creating detector opensearch-sdk-java#22
  5. [FEATURE] Remove the ThreadContext dependency on the User from Common Utils for creating detector opensearch-sdk-java#23
  6. [META] Migrate Anomaly Detector plugin to work as an Extension opensearch-sdk-java#24
  7. Meta: [FEATURE/Extensions] Interaction of AD Extension with OpenSearch opensearch-sdk-java#132

Next Steps:

  1. Feature to update the detector
  2. Deleting the detector
  3. Validating the detector

Describe the solution you'd like
A clear and concise description of what you want to happen.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

@owaiskazi19 owaiskazi19 added enhancement Enhancement or improvement to existing feature or request untriaged Build Libraries & Interfaces and removed untriaged labels Jun 20, 2022
@saratvemulapalli saratvemulapalli changed the title [META] Run AD plugin as an Extension to create a detector [Feature/extensions] [META] Run AD plugin as an Extension to create a detector Jun 20, 2022
@saratvemulapalli saratvemulapalli changed the title [Feature/extensions] [META] Run AD plugin as an Extension to create a detector [Feature/extensions] Run AD plugin as an Extension to create a detector Jun 20, 2022
@saratvemulapalli saratvemulapalli added the Meta Meta issue, not directly linked to a PR label Jun 20, 2022
@minalsha minalsha changed the title [Feature/extensions] Run AD plugin as an Extension to create a detector [Feature/extensions] Run AD backend plugin as an Extension to create a detector Jun 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Libraries & Interfaces enhancement Enhancement or improvement to existing feature or request Meta Meta issue, not directly linked to a PR
Projects
None yet
Development

No branches or pull requests

2 participants