Closed
Conversation
Contributor
Contributor
|
Added Jira card for editorial review: https://datadoghq.atlassian.net/browse/DOCS-13082 |
fmassot
force-pushed
the
fmassot/cloudprem-reorg
branch
from
7a0291b to
36fd4d6
Compare
Contributor
Author
|
Adding some remarks to improve the PR:
@estherk15 what do you think? |
estherk15
reviewed
Jan 15, 2026
Contributor
estherk15
left a comment
estherk15
left a comment
There was a problem hiding this comment.
Thanks for the reorg, I added an alias to the pages that were moved. I left some comments on links, let me know if you have any questions.
Co-authored-by: Esther Kim <esther.kim@datadoghq.com>
Co-authored-by: François Massot <francois.massot@gmail.com>
estherk15
reviewed
Jan 16, 2026
Comment on lines
+3
to
+4
| aliases: | ||
| - /cloudprem/introduction/network/ |
Contributor
There was a problem hiding this comment.
Suggested change
| aliases: | |
| - /cloudprem/introduction/network/ |
Comment on lines
+3
to
+4
| aliases: | ||
| - /cloudprem/configure/ingress/ |
Contributor
There was a problem hiding this comment.
Suggested change
| aliases: | |
| - /cloudprem/configure/ingress/ |
* move removal content from service override page into service override removal doc * add screenshots, copy refinements * tweak final secxtion * edit call-out to removal page on the service overrides guide page * Update content/en/tracing/guide/service_overrides.md Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com> * flesh out first paragraph * move service override removal page * title tweak for consistency with other pages * change title of page reference --------- Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>
…#32964) * Add troubleshooting case for workflows:write issue with GH * Consolidate sections Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com> * Copy improvements Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com> * Update GH troubleshooting steps with recommendation from SCI * Update content/en/bits_ai/bits_ai_dev_agent/setup.md * Apply suggestions from code review --------- Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com>
* firs tpush * update desc slightly * add aliases * Light copy update * Add screenshot * crop image * Update content/en/data_observability/_index.md Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com> --------- Co-authored-by: Kevin Hu <kevin.hu@datadoghq.com> Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>
* Remove Action Interface from the docs * Revert "Remove Action Interface from the docs" This reverts commit 767731a. * Add alias for Action Interface * Remove Action Interface file & folder * Remove link to Actions Interface from page * Remove Action Interface from sidebar
Co-authored-by: ci.datadog-api-spec <packages@datadoghq.com>
* restructure advanced notifications * typo * formatting
* Add documentation for GraphQL iOS * Fix: link * Fix: update v2 configuration * Fix: use info alert * Apply PR suggestions * Fix indentation * Update content/en/real_user_monitoring/application_monitoring/ios/advanced_configuration.md Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com> * Fix: datadog doesnt add headers --------- Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
* Translated file updates * fix corrupted syntax --------- Co-authored-by: webops-guacbot[bot] <214537265+webops-guacbot[bot]@users.noreply.github.com> Co-authored-by: Joe Peeples <joe.peeples@datadoghq.com>
…api/v2/metrics endpoint (#33918) Co-authored-by: ci.datadog-api-spec <packages@datadoghq.com>
* Sync LLM Observability API documentation with Go implementation Synced POST /api/intake/llm-obs/v1/trace/spans endpoint documentation with the Go source code implementation. ## Summary - **128 applicable changes** applied (138 total, 10 internal fields excluded) - **119 fields added** from Go implementation - **11 fields removed/restructured** (metrics conversion) - **4 type mismatches fixed** ## Changes by Section ### Span - Added `service` field for service identification - Added `ml_app` field (can override top-level value) - Updated `metrics` type from structured object to `Dict[key (string), float64]` ### Meta - Added `model_name`, `model_provider`, `model_version` for LLM model metadata - Added `embedding_for_prompt_idx` for embedding computation tracking - Added `span` field ([SpanField](#spanfield)) - Added `tool_definitions` array for available tools - Added `expected_output` for expected output structure - Added `intent` field for span intent ### IO (Input/Output/ExpectedOutput) - Fixed array notation for `messages` and `documents` (added brackets) - Added `embedding` field for embedding vectors - Added `parameters` field for additional parameters ### Message - Added `tool_calls` array ([ToolCall](#toolcall)) - Added `tool_results` array ([ToolResult](#toolresult)) ### Document - Added `ranking` field for document ranking - Added `metadata` field for additional metadata ### Prompt - Added `name` field for human-readable prompt name ### Metrics - Converted from structured object with specific fields to generic dictionary - Updated description to list common metric names as guidance - Added `Type: Dict[key (string), float64]` specification ### New Type Definitions - Added **ToolCall** section (4 fields: name, arguments, tool_id, type) - Added **ToolResult** section (4 fields: name, result, tool_id, type) - Added **ToolDefinition** section (3 fields: name, description, schema) - Added **SpanField** section (1 field: kind) ## Internal Fields Excluded The following internal fields were found in Go but excluded from documentation (10 fields): - `_dd` object and its children - `_dd_context_variable_keys` and `_dd_query_variable_keys` in Prompt ## Fields NOT Changed The following fields remain as-is per existing documentation standards: - Required markers (`[*required*]`) preserved on: data, data.type, data.attributes, meta.kind, messages.content - Prompt `variables` and `tags` remain as `Dict[key (string), string]` - `ml_app_version` intentionally not documented (deprecated field) 🤖 Generated with Claude Code Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> * Update content/en/llm_observability/instrumentation/api.md Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com> --------- Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com> Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com>
* [DOCS-12864] Add OCI billing guide * [DOCS-12864] Address Vale warnings * [DOCS-12864] Fix menu entry * [DOCS-12864] Update page title * Add log exclusion info * Clarify data exclusion options
github-actions
Bot
added
Github
github-actions
Bot
removed
Github
1 task
estherk15
approved these changes
Jan 16, 2026
Contributor
|
Sorry I messed up the PR with a rebase. I cherry picked the commits to a new PR: #33932 After the checks pass, I'll merge manually. This is good to go |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do? What is the motivation?
Reorganize cloudprem docs