Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add name suggestion to prometheus exporter option #3837

Conversation

codeboten
Copy link
Contributor

@codeboten codeboten commented Jan 19, 2024

Changes

Similarly to the other options listed in the Prometheus exporter specification, adding a suggested name for the option to apply resource attributes as metric attributes.

For non-trivial changes, follow the change proposal process.

@codeboten codeboten requested review from a team January 19, 2024 17:37
@codeboten codeboten force-pushed the codeboten/add-with_resource_as_constant_labels branch from d76d8ae to 258ae89 Compare January 19, 2024 17:38
Copy link

github-actions bot commented Feb 6, 2024

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 Feb 6, 2024
Copy link

Closed as inactive. Feel free to reopen if this PR is still being worked on.

@github-actions github-actions bot closed this Feb 27, 2024
@carlosalberto
Copy link
Contributor

@codeboten Should we re-open this?

@codeboten
Copy link
Contributor Author

@carlosalberto yes please

@codeboten
Copy link
Contributor Author

@jack-berg can you re-open this issue please

@jack-berg jack-berg reopened this Apr 15, 2024
Copy link

linux-foundation-easycla bot commented Apr 15, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: carlosalberto / name: Carlos Alberto Cortez (1ad35e2)
  • ✅ login: codeboten / name: Alex Boten (da07322)

@codeboten codeboten force-pushed the codeboten/add-with_resource_as_constant_labels branch from fa83641 to 0835e3c Compare April 15, 2024 16:21
Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
@carlosalberto
Copy link
Contributor

cc @dashpole

@carlosalberto carlosalberto merged commit 2bbcd2d into open-telemetry:main Apr 19, 2024
7 checks passed
@codeboten codeboten deleted the codeboten/add-with_resource_as_constant_labels branch April 24, 2024 19:56
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.

7 participants