Skip to content

Conversation

@florence-crl
Copy link
Contributor

@florence-crl florence-crl commented Oct 29, 2025

Fixes DOC-14614

  • In available-metrics-in-metrics-list.csv, added 8 sql.routine.* metrics.
  • Updated metrics.yaml, ran ./cockroach gen metric-list > metrics.yaml with v25.4.0-rc.1 binary.
  • In v25.3/essential-metrics.md and v25.4/essential-metrics.md, fixed bug where it mentioned self-hosted on Advanced page.
  • In v25.4/essential-metrics.md, added sections for Physical Replication and Logical Replication.

Rendered preview

Updated metrics.yaml, ran `./cockroach gen metric-list > metrics.yaml` with v25.4.0-rc.1 binary.

In v25.3/essential-metrics.md and v25.4/essential-metrics.md, fixed bug where it mentioned self-hosted on Advanced page.

In v25.4/essential-metrics.md, added sections for Physical Replication and Logical Replication.
@netlify
Copy link

netlify bot commented Oct 29, 2025

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit 6ccbe81
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-interactivetutorials-docs/deploys/690507985215600008f88f91

@netlify
Copy link

netlify bot commented Oct 29, 2025

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit 6ccbe81
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-api-docs/deploys/69050798c9056a00077b529f

@github-actions
Copy link

Files changed:

@netlify
Copy link

netlify bot commented Oct 29, 2025

Netlify Preview

Name Link
🔨 Latest commit 6ccbe81
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-docs/deploys/69050798f66d530008183a5b
😎 Deploy Preview https://deploy-preview-20860--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@florence-crl
Copy link
Contributor Author

@ZhouXing19 please review the changes to these pages:

Available Metrics
Essential Metrics for CockroachDB Self-Hosted Deployments

Copy link
Contributor

@ZhouXing19 ZhouXing19 left a comment

Choose a reason for hiding this comment

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

Thanks! I reviewed the routine statement counters part and it looks good to me. Will defer the review for the rest of the change to the other reviewers.

Copy link
Contributor

@taroface taroface left a comment

Choose a reason for hiding this comment

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

LGTM, assuming all the copy associated with these metrics, and their inclusion on the Essential Metrics tables, results directly from generation and is not manual (i.e., the main thing I need to review is the inclusion of the 8 metrics in available-metrics-in-metrics-list.csv).

Copy link

@kevin-v-ngo kevin-v-ngo left a comment

Choose a reason for hiding this comment

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

LGTM!

@florence-crl florence-crl merged commit 3197791 into main Oct 31, 2025
5 checks passed
@florence-crl florence-crl deleted the DOC-14614 branch October 31, 2025 19:20
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.

4 participants