docs: automate release metrics generation#1151
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: AR21SM The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Hi @AR21SM. Thanks for your PR. I'm waiting for a etcd-io member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
/ok-to-test |
| etcd_disk_wal_write_duration_seconds_bucket{le="0.001"} | ||
| etcd_disk_wal_write_duration_seconds_bucket{le="0.002"} | ||
| etcd_disk_wal_write_duration_seconds_bucket{le="0.004"} | ||
| etcd_disk_wal_write_duration_seconds_bucket{le="0.008"} | ||
| etcd_disk_wal_write_duration_seconds_bucket{le="0.016"} | ||
| etcd_disk_wal_write_duration_seconds_bucket{le="0.032"} | ||
| etcd_disk_wal_write_duration_seconds_bucket{le="0.064"} | ||
| etcd_disk_wal_write_duration_seconds_bucket{le="0.128"} | ||
| etcd_disk_wal_write_duration_seconds_bucket{le="0.256"} | ||
| etcd_disk_wal_write_duration_seconds_bucket{le="0.512"} | ||
| etcd_disk_wal_write_duration_seconds_bucket{le="1.024"} | ||
| etcd_disk_wal_write_duration_seconds_bucket{le="2.048"} | ||
| etcd_disk_wal_write_duration_seconds_bucket{le="4.096"} | ||
| etcd_disk_wal_write_duration_seconds_bucket{le="8.192"} | ||
| etcd_disk_wal_write_duration_seconds_bucket{le="+Inf"} |
There was a problem hiding this comment.
Does it actually help users to output all of these bucket thresholds in this format?
There was a problem hiding this comment.
no, i dont think so
these bucket lines come from the existing tool tools/etcd-dump-metricsoutput https://github.com/etcd-io/etcd/tree/main/tools/etcd-dump-metrics. this PR was to automating the txt generation.
If the goal is to align more closely with the Kubernetes metrics reference,https://kubernetes.io/docs/reference/instrumentation/metrics/ I think the better fix would be to render one entry per metric with name, description, type, and label names, while hiding histogram/summary internals such as le and quantile.
alt, if u want i can filter those in the current tabular format if that is preferred.
There was a problem hiding this comment.
If the goal is to align more closely with the Kubernetes metrics reference,https://kubernetes.io/docs/reference/instrumentation/metrics/ I think the better fix would be to render one entry per metric with name, description, type, and label names, while hiding histogram/summary internals such as
leandquantile.
Sounds good to me.
There was a problem hiding this comment.
will implement that and update the pr
There was a problem hiding this comment.
@ahrtr does it looks good now than previously ??
https://deploy-preview-1151--etcd.netlify.app/docs/v3.6/metrics/etcd-metrics-latest/
|
Code-wise, this looks good, and it's updating the website. However, please note that it's not updating the page from the PR description. It's updating the |
it is updating html page too (maybe you can match https://deploy-preview-1151--etcd.netlify.app/docs/v3.6/metrics/etcd-metrics-latest/#:~:text=server_id%3D%227339c4e5e833c029%22), for reference here is the old main one https://etcd.io/docs/v3.6/metrics/etcd-metrics-latest/ old pr ref that does the html part: #1139 |
I may not be following. The automation from this pull request is only updating the text file:
And if we're okay with updating the text file only, then this is correct. I was just confused with the pull request description showing a screenshot of a page which is not being updated by the automation. |
yes, that's true
oh, i forget to mention in pr description that from txt -> html part was done in this pr #1139 which got merged (so i didn't changed that), just worked on automating the txt generation part. |
Oh, ok, now I understand it. I think we'd like to execute that gotemplate when this automation runs, too. We can do that in a follow-up PR :) |
Signed-off-by: AR21SM <mahajanashishar21sm@gmail.com>
087aee9 to
7d4dce8
Compare
i placed this here is that fine ?? |
What changed
This PR adds a script to update the generated metrics file from the etcd source.
It uses the existing
tools/etcd-dump-metricstool from etcd repo.I also added make targets for updating and verifying the metrics docs, and updated the release version script so future release updates can also refresh the metrics file.
This PR regenerates the v3.6 metrics file from
v3.6.0.closes #1133
Note
I did not update v3.7 metrics in this PR because v3.7 is still draft and uses
git_version_tag: main. I kept the script limited to released etcd tags to avoid committing metrics generated from a moving branch.If maintainers want draft docs to track metrics from
main, I can add that separately.