Skip to content

Fix Managed inputs nav path_prefix (404 on staging)#3603

Merged
vishaangelova merged 1 commit into
mainfrom
7155-fix-managed-inputs-path-prefix
Jul 1, 2026
Merged

Fix Managed inputs nav path_prefix (404 on staging)#3603
vishaangelova merged 1 commit into
mainfrom
7155-fix-managed-inputs-path-prefix

Conversation

@vishaangelova

Copy link
Copy Markdown
Member

Summary

Hotfix for the 404s on the Managed inputs pages after #3601. The managed-inputs toc is built as part of opentelemetry://reference (path_prefix: reference/opentelemetry), so its pages are served under reference/opentelemetry/managed-inputs/.... The standalone nav entry pointed at reference/managed-inputs, which has no built content, so every Managed inputs page 404'd.

This restores the opentelemetry segment on the path_prefix, matching the previous motlp behavior and the actual (working) content location.

Effect

  • Nav links resolve to the built pages, e.g. /reference/opentelemetry/managed-inputs/managed-otlp-endpoint.
  • Existing motlp redirects remain valid (/reference/opentelemetry/motlp/*/reference/opentelemetry/managed-inputs/*).

Related to elastic/docs-content#7155

Made with Cursor

…-inputs

The managed-inputs toc is built as part of opentelemetry://reference
(path_prefix reference/opentelemetry), so its pages are served under
reference/opentelemetry/managed-inputs. The standalone nav entry pointed at
reference/managed-inputs, which had no built content and returned 404. Restore
the opentelemetry segment (matching the previous motlp behavior) so the nav
links resolve.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vishaangelova vishaangelova requested review from a team as code owners July 1, 2026 13:12
@vishaangelova vishaangelova added the documentation Improvements or additions to documentation label Jul 1, 2026
@vishaangelova vishaangelova requested a review from Mpdreamz July 1, 2026 13:12
@vishaangelova vishaangelova temporarily deployed to integration-tests July 1, 2026 13:12 — with GitHub Actions Inactive
@vishaangelova vishaangelova enabled auto-merge (squash) July 1, 2026 13:12
@coderabbitai

coderabbitai Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

This change updates a single navigation configuration entry in config/navigation.yml. The path_prefix for the OpenTelemetry "managed-inputs" table-of-contents entry is changed from reference/managed-inputs to reference/opentelemetry/managed-inputs, redirecting the navigation link target to the correct documentation path.

Changes

Compact metadata:

Cohort / File(s) Change Summary
config/navigation.yml Updated path_prefix for OpenTelemetry managed-inputs entry

Related issues: None provided.

Related PRs: None provided.

Suggested labels: documentation, navigation

Suggested reviewers: None specified.


Poem:

A rabbit hopped through nav so deep,
One path was wrong, but not asleep —
"opentelemetry" now leads the way,
To managed-inputs, correct today! 🐇📖

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the nav path_prefix fix and the 404 it addresses.
Description check ✅ Passed The description matches the change and explains the Managed inputs 404 fix in detail.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch 7155-fix-managed-inputs-path-prefix

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
config/navigation.yml (1)

478-479: 🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win

Align the EDOT Collector path with the rest of the section

The edot-collector entry still publishes under reference/edot-collector, while the surrounding EDOT docs and existing link point to reference/opentelemetry/edot-collector. That mismatch will leave the Collector URL broken.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@config/navigation.yml` around lines 478 - 479, The EDOT Collector navigation
entry is publishing under the wrong path and should match the rest of the EDOT
section. Update the `edot-collector` item in the navigation configuration so the
`path_prefix` aligns with the existing `reference/opentelemetry/edot-collector`
route, using the `toc: elastic-agent://reference/edot-collector` entry as the
locator.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Outside diff comments:
In `@config/navigation.yml`:
- Around line 478-479: The EDOT Collector navigation entry is publishing under
the wrong path and should match the rest of the EDOT section. Update the
`edot-collector` item in the navigation configuration so the `path_prefix`
aligns with the existing `reference/opentelemetry/edot-collector` route, using
the `toc: elastic-agent://reference/edot-collector` entry as the locator.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 6d48d40f-b027-4c23-861c-79e03c5a4668

📥 Commits

Reviewing files that changed from the base of the PR and between f1f0b76 and 547e99d.

📒 Files selected for processing (1)
  • config/navigation.yml

@vishaangelova vishaangelova merged commit d3f5fb6 into main Jul 1, 2026
26 checks passed
@vishaangelova vishaangelova deleted the 7155-fix-managed-inputs-path-prefix branch July 1, 2026 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants