Skip to content

Conversation

@adity1raut
Copy link
Contributor

@adity1raut adity1raut commented Nov 9, 2025

Description

This PR Fixed #8353 refactors the Collector installation documentation by renaming, restructuring, and splitting the existing installation page into multiple focused sections. The goal is to improve clarity, organization, and navigation within the documentation.

Changes ...

Changes

Rename the page and update front matter:

  • Move and rename the file to index.md.

  • Add an alias in the front matter:

    aliases: [/docs/collector/installation/]
  • Create new child pages:

    • content/en/docs/collector/install/docker.mdInstall the Collector with Docker
    • content/en/docs/collector/install/kubernetes.mdInstall the Collector with Kubernetes
    • index.mdInstall from a Collector binary
    • content/en/docs/collector/install/binary/linux.mdInstall the Collector on Linux
    • content/en/docs/collector/install/binary/macos.mdInstall the Collector on macOS
    • content/en/docs/collector/install/binary/windows.mdInstall the Collector on Windows

Each new child page includes front matter with a redirect from the old location.

Content migration:

  • Keep the page title and first two paragraphs on /install/.
  • Move Docker, Docker Compose, Kubernetes, Linux, macOS, and Windows installation instructions to their respective child pages.
  • Retain the “Build from Source” section on /install/.
  • Remove the Nomad section.

Update cross-references:

  • Fix references to /docs/collector/installation/ in quick-start.md (line 131) and anywhere else within the English documentation.

Related Issues / References

  • N/A (Add issue link if applicable)

Additional Notes

This restructuring improves the logical flow and maintainability of Collector installation documentation.
Run npm run fix:all locally to ensure style and formatting compliance.


Preview: https://deploy-preview-8370--opentelemetry.netlify.app/docs/collector/install/
Redirect test: https://deploy-preview-8370--opentelemetry.netlify.app/docs/collector/installation/

Signed-off-by: adity1raut <araut7798@gmail.com>
@adity1raut adity1raut requested a review from a team as a code owner November 9, 2025 11:17
@otelbot-docs otelbot-docs bot requested review from a team and codeboten and removed request for a team November 9, 2025 11:17
Signed-off-by: adity1raut <araut7798@gmail.com>
@otelbot-docs otelbot-docs bot requested a review from a team November 9, 2025 11:19
@adity1raut
Copy link
Contributor Author

PTAL @tiffany76

@theletterf theletterf requested a review from tiffany76 November 11, 2025 08:43
Copy link
Member

@tiffany76 tiffany76 left a comment

Choose a reason for hiding this comment

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

This is looking pretty good, @adity1raut! I've left some comments based on my first-pass review. Once we address those, I'll take a closer look. Thanks!

@@ -0,0 +1,140 @@
---
title: Install the Collector on Linux
aliases: [../installation/#linux]
Copy link
Member

Choose a reason for hiding this comment

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

@chalin I know this alias won't work because the source is a fragment. How do we keep the fragment's target and point it to an alert about refactoring?

Signed-off-by: adity1raut <araut7798@gmail.com>
@adity1raut adity1raut requested a review from tiffany76 November 13, 2025 06:09
Signed-off-by: adity1raut <araut7798@gmail.com>
@adity1raut
Copy link
Contributor Author

@tiffany76 Can you please Give me review

Copy link
Contributor

@chalin chalin left a comment

Choose a reason for hiding this comment

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

@tiffany76 - is there a reason to prefer naming the section install vs the original installation? If we keep the original name then we won't need any aliases.

(And your right, there should be a single alias in this refactoring.)

@tiffany76
Copy link
Member

@chalin All of the top level pages that instruct people how to do things will be verbs: Install, Configure, Deploy, Manage, Extend. I'd like the iA to be uniform.

@chalin
Copy link
Contributor

chalin commented Nov 17, 2025

@tiffany76 - that works for me. Then this PR should have a single alias in the install/_index.md file. As you know, we can't redirect fragments/hashes (at least not using just Netlify's redirect facility ... which is what we want to stick to -- we're just trying to prevent 404s). Folks will figure out what subpage to visit.

Copy link
Contributor

@chalin chalin left a comment

Choose a reason for hiding this comment

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

If @tiffany76 agrees with the linkTitles, add appropriate linkTitle entries to each page based on the examples I have given in the comments below.

@chalin
Copy link
Contributor

chalin commented Nov 17, 2025

/fix:refcache

@otelbot
Copy link
Contributor

otelbot bot commented Nov 17, 2025

ℹ️ fix:refcache made no changes. Nothing to commit.

@tiffany76
Copy link
Member

If @tiffany76 agrees with the linkTitles, add appropriate linkTitle entries to each page based on the examples I have given in the comments below.

I think they're probably good, but I'd like to get the page weights sorted out so I can see how the nav looks in the preview before settling for sure on the linkTitles.

Copy link
Member

@tiffany76 tiffany76 left a comment

Choose a reason for hiding this comment

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

Hi @adity1raut! I've made some suggested changes to address the outstanding issues. If you commit them, I'll do a final review and we can hopefully get this merged. Thanks!

@tiffany76
Copy link
Member

The link checker also found that a 2024 blog post is referring to the old installation page. If you can update the reference in blog/2024/llm-observability/index.html to point to the new URL, that should resolve the current CI failure. Thanks.

@tiffany76 tiffany76 added this to the otelcol-phase-1 milestone Nov 21, 2025
@tiffany76 tiffany76 moved this from Backlog to In progress in SIG Comms + SIG Collector: Doc refactoring Nov 21, 2025
@tiffany76 tiffany76 added the sig:collector:refactor These issues are part of the Collector documentation refactoring project. label Nov 21, 2025
Signed-off-by: adity1raut <araut7798@gmail.com>
Co-authored-by: Patrice Chalin <chalin@users.noreply.github.com>
@tiffany76
Copy link
Member

@adity1raut I'm repasting my earlier comment. This PR is very close, but you'll need to fix the following issue before we can move forward. Thanks!

The link checker also found that a 2024 blog post is referring to the old installation page. If you can update the reference in blog/2024/llm-observability/index.html to point to the new URL, that should resolve the current CI failure. Thanks.

@tiffany76 tiffany76 moved this from In progress to In review in SIG Comms + SIG Collector: Doc refactoring Nov 26, 2025
@github-actions github-actions bot added the blog label Dec 3, 2025
@otelbot-docs otelbot-docs bot requested a review from a team December 3, 2025 18:51
Copy link
Member

@tiffany76 tiffany76 left a comment

Choose a reason for hiding this comment

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

Hi @adity1raut - I updated the 2024 blog post on your branch because we need to get this merged.

Thanks for your contribution!

@tiffany76 tiffany76 added this pull request to the merge queue Dec 3, 2025
Merged via the queue into open-telemetry:main with commit ec1657d Dec 3, 2025
22 checks passed
@otelbot
Copy link
Contributor

otelbot bot commented Dec 3, 2025

Thank you for your contribution @adity1raut! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey.

jaehanbyun pushed a commit to jaehanbyun/opentelemetry.io that referenced this pull request Dec 8, 2025
…metry#8370)

Signed-off-by: adity1raut <araut7798@gmail.com>
Signed-off-by: Tiffany Hrabusa <tiffany.hrabusa@grafana.com>
Co-authored-by: Tiffany Hrabusa <30397949+tiffany76@users.noreply.github.com>
Co-authored-by: Patrice Chalin <chalin@users.noreply.github.com>
Co-authored-by: Tiffany Hrabusa <tiffany.hrabusa@grafana.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blog sig:collector:refactor These issues are part of the Collector documentation refactoring project. sig:collector

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[otelcol docs] Rename and split up Collector installation page

3 participants