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

Docs: update o11y section #884

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

robertjdominguez
Copy link
Collaborator

@robertjdominguez robertjdominguez commented Feb 6, 2025

Description 📝

This PR was mean to be more substantial and to include documentation for self-hosted users looking to attach their own APM. However, read the description here for why that's excluded from this PR.

Essentially, that means this is simply a reorganization of existing content with a bit of cleanup.

It should be rubber-stampable with more attention to be paid to it in the next sprint.

Quick Links 🚀

o11y

Creates a new directory structure largely based on the planning document. Notably, the majority of the existin content was brought into the `built-in` subdirectory with a new how-to page stubbed out
for exporting metrics (which I believe can only be done via self-hosting or Private DDN).
Updates the Basics page and audits existing pages. Much of what we have here is reference documentation and how-to guides or tutorials won't really fit.

Paritosh did a great job of providing some learn-by-examples, especially for the `explain` API.
Removes exporting sections until better internal docs are developed. Right now, there's too much
ambiguity around which otel image to use, how to configure Prometheus, and JSON snippets needed to
help users develop a dashboard immediately.

Ideally, the post-sales folks will turn their example — which is present only a branch of an internal repo —
into a standalone repoistory with more helpful documentation around how to run this side-by-side
to a DDN instance.
@hasura-bot
Copy link
Contributor

Redirects 🔀

Hey, @robertjdominguez & @seanparkross — the following redirects will be needed for deployment:

# TEST ME: https://stage.hasura.io/docs/3.0/observability/explain/_category_.json
location = /docs/3.0/observability/explain/_category_.json {
    return 301 https://$host/docs/3.0/observability/built-in/explain/_category_.json;
}
# TEST ME: https://stage.hasura.io/docs/3.0/observability/explain/api-reference/
location = /docs/3.0/observability/explain/api-reference/ {
    return 301 https://$host/docs/3.0/observability/built-in/explain/api-reference/;
}
# TEST ME: https://stage.hasura.io/docs/3.0/observability/explain/index/
location = /docs/3.0/observability/explain/index/ {
    return 301 https://$host/docs/3.0/observability/built-in/explain/index/;
}
# TEST ME: https://stage.hasura.io/docs/3.0/observability/metrics/
location = /docs/3.0/observability/metrics/ {
    return 301 https://$host/docs/3.0/observability/built-in/metrics/;
}
# TEST ME: https://stage.hasura.io/docs/3.0/observability/model-analytics/_category_.json
location = /docs/3.0/observability/model-analytics/_category_.json {
    return 301 https://$host/docs/3.0/observability/built-in/model-analytics/_category_.json;
}
# TEST ME: https://stage.hasura.io/docs/3.0/observability/model-analytics/field/
location = /docs/3.0/observability/model-analytics/field/ {
    return 301 https://$host/docs/3.0/observability/built-in/model-analytics/field/;
}
# TEST ME: https://stage.hasura.io/docs/3.0/observability/model-analytics/index/
location = /docs/3.0/observability/model-analytics/index/ {
    return 301 https://$host/docs/3.0/observability/built-in/model-analytics/index/;
}
# TEST ME: https://stage.hasura.io/docs/3.0/observability/model-analytics/operation/
location = /docs/3.0/observability/model-analytics/operation/ {
    return 301 https://$host/docs/3.0/observability/built-in/model-analytics/operation/;
}
# TEST ME: https://stage.hasura.io/docs/3.0/observability/platform-dashboard/
location = /docs/3.0/observability/platform-dashboard/ {
    return 301 https://$host/docs/3.0/observability/built-in/platform-dashboard/;
}
# TEST ME: https://stage.hasura.io/docs/3.0/observability/traces/
location = /docs/3.0/observability/traces/ {
    return 301 https://$host/docs/3.0/observability/built-in/traces/;
}

@hasura-bot
Copy link
Contributor

@robertjdominguez Thanks for your PR! I've assigned @robertjdominguez to review it.

@robertjdominguez robertjdominguez changed the title Robdominguez/doc 2261 update o11y section Docs: update o11y section Feb 6, 2025
Copy link

Deploying ddn-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9aeee1b
Status: ✅  Deploy successful!
Preview URL: https://d42fc3b4.v3-docs-eny.pages.dev
Branch Preview URL: https://robdominguez-doc-2261-update.v3-docs-eny.pages.dev

View logs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants