-
Notifications
You must be signed in to change notification settings - Fork 503
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
[DOC] Add fail processor documentation #5980
Conversation
Signed-off-by: Melissa Vagi <[email protected]>
@hdhalter We need a dev to take the first pass at drafting this content. I can support them in refining the content and getting it through the documentation process. The ingest processor template is provided with this PR, so it should help the dev get started. Thanks! |
@gaobinlong Would you have availability to help close the content gap on this PR? We don't have a set deadline, so at your availability is okay. The same process you've followed for the copy processor can be followed here. Thank you. |
OK, glad to do that, I'll open a PR for this issue. |
Thanks @gaobinlong. Please tag me in the PR when it's ready for doc review. |
@gaobinlong This template for the fail ingest processor is ready for you to add code examples at your availability. Thank you, Melissa |
Signed-off-by: Melissa Vagi <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
@gaobinlong Will you review this PR for technical accuracy? I added examples and new content. Thank you! |
Signed-off-by: Melissa Vagi <[email protected]>
@gaobinlong Thank you for your review and the code examples. I made your changes. Once. you approve, I'll send for editorial review and then merge the PR. |
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.
Overall looks good to me, some minor changes need to be done.
Signed-off-by: Melissa Vagi <[email protected]>
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.
A few suggestions, otherwise LGTM.
Co-authored-by: Naarcha-AWS <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Naarcha-AWS <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
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.
@vagimeli Just a few comments/changes. Thanks!
_ingest-pipelines/processors/fail.md
Outdated
|
||
### Step 4 (Optional): Retrieve the document | ||
|
||
Since the log event was not indexed due to the pipeline failure, attempting to retrieve it will result in a document not found error. |
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.
Should "document not found" be specially formatted as the name of the error?
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
* Add fail processor documentation Signed-off-by: Melissa Vagi <[email protected]> * Update fail.md Signed-off-by: Melissa Vagi <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Address tech review feedback and add SME's code examples Signed-off-by: Melissa Vagi <[email protected]> * Address tech review comments Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/fail.md Co-authored-by: Naarcha-AWS <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/fail.md Co-authored-by: Naarcha-AWS <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/fail.md Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/fail.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/fail.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/fail.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update fail.md Signed-off-by: Melissa Vagi <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> --------- Signed-off-by: Melissa Vagi <[email protected]> Co-authored-by: Naarcha-AWS <[email protected]> Co-authored-by: Nathan Bower <[email protected]> (cherry picked from commit 05710d0) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Description
Closes the content gap
Issues Resolved
##4647
Checklist
For more information on following Developer Certificate of Origin and signing off your commits, please check here.