Skip to content

Conversation

@ywwg
Copy link

@ywwg ywwg commented Oct 7, 2025

DO NOT MERGE

This is a brute-force rename of all the metrics to their native otel equivalents. There may still be errors. For testing only

Signed-off-by: Owen Williams [email protected]# Changes

ywwg added 3 commits October 7, 2025 16:23
Signed-off-by: Owen Williams <[email protected]>
Signed-off-by: Owen Williams <[email protected]>
Signed-off-by: Owen Williams <[email protected]>
@github-actions github-actions bot added the helm-update-required Requires an update to the Helm chart when released label Oct 7, 2025
ywwg added 6 commits October 8, 2025 10:45
Signed-off-by: Owen Williams <[email protected]>
Signed-off-by: Owen Williams <[email protected]>
Signed-off-by: Owen Williams <[email protected]>
Signed-off-by: Owen Williams <[email protected]>
Signed-off-by: Owen Williams <[email protected]>
@github-actions
Copy link

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions
Copy link

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Oct 31, 2025
@julianocosta89
Copy link
Member

@ywwg do you want to keep this open?
Should we close it and re-open later?
Should we review it?

Let us know

@github-actions github-actions bot removed the Stale label Nov 8, 2025
@ywwg
Copy link
Author

ywwg commented Nov 10, 2025

I do not intend to merge this, so no review is needed. I think the right way to support native names would be to come up with a general way of generating these dashboard files with code, and allowing the user to select which translation scheme to use at code gen time. Do you think that is a practical approach for this repo?

@julianocosta89
Copy link
Member

I do not intend to merge this, so no review is needed. I think the right way to support native names would be to come up with a general way of generating these dashboard files with code, and allowing the user to select which translation scheme to use at code gen time. Do you think that is a practical approach for this repo?

I do not understand what you mean.

Actually, looking at the PR, I'm not quite sure I get what you are trying to solve here.
Is this a dashboard visualization issue?

AFAIK all dashboards are working, no?!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

helm-update-required Requires an update to the Helm chart when released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants