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

Add documentation for OpenSearch Flow OSD plugin #9101

Merged
merged 16 commits into from
Jan 31, 2025

Conversation

ohltyler
Copy link
Member

@ohltyler ohltyler commented Jan 21, 2025

Description

Describe what this change achieves.
Adds initial documentation for the new OpenSearch Flow OSD plugin, including an overview, examples, and general details.

Issues Resolved

Closes #9064

Version

List the OpenSearch version to which this PR applies, e.g. 2.14, 2.12--2.14, or all.
2.19+

Frontend features

If you're submitting documentation for an OpenSearch Dashboards feature, add a video that shows how a user will interact with the UI step by step. A voiceover is optional.

demo.webm

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Copy link

Thank you for submitting your PR. The PR states are In progress (or Draft) -> Tech review -> Doc review -> Editorial review -> Merged.

Before you submit your PR for doc review, make sure the content is technically accurate. If you need help finding a tech reviewer, tag a maintainer.

When you're ready for doc review, tag the assignee of this PR. The doc reviewer may push edits to the PR directly or leave comments and editorial suggestions for you to address (let us know in a comment if you have a preference). The doc reviewer will arrange for an editorial review.

@ohltyler
Copy link
Member Author

Setting PR to draft. Final doc location, wording, screenshots, & more details still in-progress.

@kolchfa-aws kolchfa-aws added release-notes PR: Include this PR in the automated release notes v2.19.0 labels Jan 21, 2025
@ohltyler
Copy link
Member Author

@opensearch-project/docs Hi team, I may need some assistance in determining the right location for the documentation for this new OSD plugin. At a high level, this plugin leverages the Flow Framework Plugin & ML Commons Plugin for building out ingest and search pipelines, so it doesn't cleanly fit as a "frontend" for particular backend/OpenSearch plugin, for example.

Right now, I've put it under the Flow Framework Plugin's Automating Configurations section of the documentation. Open to ideas/suggestions.

@ohltyler
Copy link
Member Author

@kamingleung & team is working on finalizing some wording and minor UX tuning, hence I'm holding off on adding screenshots or flipping this PR out of draft just yet.

@ohltyler ohltyler closed this Jan 27, 2025
@ohltyler ohltyler reopened this Jan 27, 2025
Signed-off-by: Tyler Ohlsen <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
@ohltyler
Copy link
Member Author

ohltyler commented Jan 28, 2025

@opensearch-project/docs, @kamingleung I've pushed changes based on the latest UX wording suggestions, and added corresponding screenshots. I will still need some assistance determining where this should live on the site; see above comment.

@ohltyler ohltyler marked this pull request as ready for review January 28, 2025 23:06
Signed-off-by: Tyler Ohlsen <[email protected]>
@kolchfa-aws
Copy link
Collaborator

Normally, we put dashboards features that are particular to a topic (say, ML), in that topic's section and not in the Dashboards section, so I think the logical section for this doc is Automating configurations.

Signed-off-by: Tyler Ohlsen <[email protected]>
Copy link

@minalsha minalsha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please review the feedback provided. Thanks

Signed-off-by: Tyler Ohlsen <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
@ohltyler ohltyler requested a review from minalsha January 29, 2025 01:07
@kolchfa-aws kolchfa-aws added the 4 - Doc review PR: Doc review in progress label Jan 29, 2025
Copy link

@minalsha minalsha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please address latest comments. Thanks

Signed-off-by: Fanit Kolchina <[email protected]>
Copy link
Collaborator

@natebower natebower left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kolchfa-aws @ohltyler Please see my comments and changes and let me know if you have any questions. Thanks!

_automating-configurations/workflow-builder.md Outdated Show resolved Hide resolved
_automating-configurations/workflow-builder.md Outdated Show resolved Hide resolved
_automating-configurations/workflow-builder.md Outdated Show resolved Hide resolved
_automating-configurations/workflow-builder.md Outdated Show resolved Hide resolved
_automating-configurations/workflow-builder.md Outdated Show resolved Hide resolved
_automating-configurations/workflow-builder.md Outdated Show resolved Hide resolved
_automating-configurations/workflow-builder.md Outdated Show resolved Hide resolved
_automating-configurations/workflow-builder.md Outdated Show resolved Hide resolved
_automating-configurations/workflow-builder.md Outdated Show resolved Hide resolved
- For suggested models and model interfaces to use within OpenSearch Flow, see [Models](https://github.com/opensearch-project/dashboards-flow-framework/blob/main/documentation/models.md).

- For additional tutorials on using OpenSearch Flow to build different AI/ML use cases, see [Tutorial](https://github.com/opensearch-project/dashboards-flow-framework/blob/main/documentation/tutorial.md).

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The note I deleted would be more appropriate to a blog post as opposed to the documentation.

@natebower natebower added 5 - Editorial review PR: Editorial review in progress and removed 4 - Doc review PR: Doc review in progress labels Jan 31, 2025
Copy link
Collaborator

@kolchfa-aws kolchfa-aws left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kolchfa-aws kolchfa-aws removed the 5 - Editorial review PR: Editorial review in progress label Jan 31, 2025
@kolchfa-aws kolchfa-aws merged commit 4b995b9 into opensearch-project:main Jan 31, 2025
6 checks passed
@ohltyler ohltyler deleted the add-flow-docs branch February 3, 2025 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-notes PR: Include this PR in the automated release notes v2.19.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOC] Add documentation for new OpenSearch Flow OSD plugin
6 participants