Skip to content

Conversation

@mablr
Copy link
Contributor

@mablr mablr commented Nov 7, 2025

Description
Fix links to node troubleshooting & monitoring guides

Tests
N/A

Additional context
Follow-up for #1802 and #1811

Metadata
N/A

@mablr mablr requested a review from a team as a code owner November 7, 2025 13:15
Comment on lines 48 to 49
* If you've already got your node up and running, check out the [Node Metrics and Monitoring Guide](/operators/node-operators/management/metrics) to learn how to keep tabs on your node and make sure it keeps running smoothly.
* If you run into any problems, please visit the [Node Troubleshooting Guide](/operators/node-operators/management/troubleshooting) for help.
Copy link
Collaborator

Choose a reason for hiding this comment

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

these links are unfortunately also wrong

Copy link
Contributor Author

@mablr mablr Nov 7, 2025

Choose a reason for hiding this comment

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

Oh ok, thanks for the quick review!

I actually hesitated on these, and took as reference this page where the link is ok:
https://github.com/ethereum-optimism/docs/blob/103dd6c39a910b577a2c9a2dfff1e8e9be8f2fce/node-operators/reference/architecture/rollup-node.mdx?plain=1#L135-L136
https://docs.optimism.io/node-operators/reference/architecture/rollup-node#next-steps

I assume the correct ones are respectively:

  • "Node Metrics and Monitoring Guide": /node-operators/guides/monitoring/metrics
  • "Node Troubleshooting Guide": /node-operators/guides/troubleshooting

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yeah I think that's right. Everything got shuffled around recently because we moved to a new framework. I'm probably going to do some more sweeping changes soon. If you follow the filesystem paths and what's in the docs.json you should be good

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, fix applied 👍

/operators/node-operators/management/metrics -> /node-operators/guides/monitoring/metrics

/operators/node-operators/management/troubleshooting -> /node-operators/guides/troubleshooting
@sbvegan sbvegan merged commit 4787d1c into ethereum-optimism:mintlify Nov 7, 2025
4 checks passed
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