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 Simple Schema #2940

Merged
merged 11 commits into from
Feb 22, 2023
Merged

Add documentation for Simple Schema #2940

merged 11 commits into from
Feb 22, 2023

Conversation

ariamarble
Copy link
Contributor

@ariamarble ariamarble commented Feb 16, 2023

Description

Adds documentation for Simple Schema for Observability in OpenSearch.

Issues Resolved

fixes #2655

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.

Signed-off-by: ariamarble <[email protected]>
@ariamarble ariamarble requested a review from a team as a code owner February 16, 2023 19:51
@ariamarble ariamarble self-assigned this Feb 16, 2023
@ariamarble ariamarble added 2 - In progress Issue/PR: The issue or PR is in progress. observability labels Feb 16, 2023
@ariamarble
Copy link
Contributor Author

Creating placeholder PR with page framework. Content still in progress.

@hdhalter
Copy link
Contributor

Hi @ariamarble - the PR is not linking back to the issue. Can you please try adding one of these keywords before the issue number?
close
closes
closed
fix
fixes
fixed
resolve
resolves
resolved
Thanks!

Signed-off-by: ariamarble <[email protected]>
@ariamarble ariamarble added 3 - Tech review PR: Tech review in progress and removed 2 - In progress Issue/PR: The issue or PR is in progress. labels Feb 20, 2023
@ariamarble
Copy link
Contributor Author

@YANG-DB can you review this PR for technical accuracy?

@ariamarble ariamarble added the release-notes PR: Include this PR in the automated release notes label Feb 20, 2023
@YANG-DB
Copy link
Member

YANG-DB commented Feb 21, 2023

in addition I dont see how the image has anything todo with the text ?

_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
@ariamarble ariamarble added 2 - In progress Issue/PR: The issue or PR is in progress. and removed 3 - Tech review PR: Tech review in progress labels Feb 21, 2023
@ariamarble ariamarble added this to the v2.6 milestone Feb 21, 2023
Signed-off-by: ariamarble <[email protected]>
@ariamarble ariamarble added 4 - Doc review PR: Doc review in progress and removed 2 - In progress Issue/PR: The issue or PR is in progress. labels Feb 21, 2023
Copy link
Contributor

@vagimeli vagimeli left a comment

Choose a reason for hiding this comment

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

@ariamarble I made some comments that should be quick to address. Reach out if clarifications are needed.

_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
@carolxob carolxob self-requested a review February 21, 2023 19:54
Copy link
Contributor

@carolxob carolxob left a comment

Choose a reason for hiding this comment

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

LGTM with one minor suggestion in addition to @vagimeli 's comments.

@ariamarble ariamarble added 5 - Editorial review PR: Editorial review in progress and removed 4 - Doc review PR: Doc review in progress labels Feb 21, 2023
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.

@ariamarble Please see my comments and changes and let me know if you have any questions. Once you've addressed my comments, I'll take another look and approve. Thanks!

_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
@natebower natebower removed the 5 - Editorial review PR: Editorial review in progress label Feb 22, 2023
ariamarble and others added 2 commits February 22, 2023 08:49
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.

@ariamarble A few more comments and changes. Let me know once you've addressed this set and I'll review again.

_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
_observing-your-data/sso.md Outdated Show resolved Hide resolved
ariamarble and others added 4 commits February 22, 2023 10:24
Signed-off-by: ariamarble <[email protected]>
Signed-off-by: ariamarble <[email protected]>
Signed-off-by: ariamarble <[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.

LGTM

@ariamarble ariamarble merged commit 9052c45 into main Feb 22, 2023
@ariamarble ariamarble added the Closed - Complete Issue: Work is done and associated PRs closed label Feb 22, 2023
@hdhalter
Copy link
Contributor

@ariamarble - please make sure to change the PR name so it is meaningful in the documentation release notes. Thanks!

Introduced 2.6
{: .label .label-purple }

OpenSearch 2.6 introduced a standardization for conforming to a common and unified observability schema: Simple Schema for Observability (SSO). [Observability]({{site.url}}{{site.baseurl}}/observing-your-data/index/) is a collection of plugins and applications that let you visualize data-driven events by using Piped Processing Language (PPL) to explore and query data stored in OpenSearch.
Copy link
Member

Choose a reason for hiding this comment

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

data-driven events -> data-driven-signals

Introduced 2.6
{: .label .label-purple }

OpenSearch 2.6 introduced a standardization for conforming to a common and unified observability schema: Simple Schema for Observability (SSO). [Observability]({{site.url}}{{site.baseurl}}/observing-your-data/index/) is a collection of plugins and applications that let you visualize data-driven events by using Piped Processing Language (PPL) to explore and query data stored in OpenSearch.
Copy link
Member

Choose a reason for hiding this comment

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

Piped Processing Language (PPL) -> using PPL and DQK (for standard visualizations)


Data Prepper conforms to the SSO schema for metrics and will gradually support traces and logs.

Data Prepper's [trace mapping]({{site.url}}{{site.baseurl}}/data-prepper/common-use-cases/trace-analytics/) currently provides `service-map` data in a different way than SSO traces. To make the trace mapping compatible with Observability, it will be integrated with the SSO traces schema and will introduce `service-map` as an enriched field.
Copy link
Member

Choose a reason for hiding this comment

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

in a different way than SSO traces. -> in addition and thus enriching the standard trace signal with additional calculated field.


With the schema in place, Observability tools can ingest, automatically extract, and aggregate data and create custom dashboards, making it easier to understand the system at a higher level.

SSO is inspired by both [OpenTelemetry](https://opentelemetry.io/docs/) and the Elastic Common Schema (ECS) and uses Amazon Elastic Container Service ([Amazon ECS](https://docs.aws.amazon.com/AmazonECS/latest/developerguide/ecs_cwe_events.html)) event logs and OpenTelemetry metadata.

Choose a reason for hiding this comment

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

why are we talking about amazon ecs

@ariamarble ariamarble changed the title Add documentation for SSO Add documentation for Simple Schema Feb 28, 2023
@hdhalter hdhalter deleted the issue2655 branch March 28, 2024 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed - Complete Issue: Work is done and associated PRs closed observability release-notes PR: Include this PR in the automated release notes v2.6.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOC] Simple Schema for Observability
7 participants