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.

Sign up for GitHub

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

Jump to bottom

Resource attributes copy processor #133

Merged
merged 7 commits into from
Nov 18, 2021
Merged

Resource attributes copy processor #133

merged 7 commits into from
Nov 18, 2021

Conversation

BinaryFissionGames
Copy link
Contributor

Proposed Change

  • Adds a new processor that copies resource-level attributes to individual metric data points
Checklist
  • Changes are tested
  • Changes are documented

@BinaryFissionGames BinaryFissionGames requested a review from a team as a code owner November 17, 2021 21:45
@BinaryFissionGames
Fix lint errors
1131bca
@BinaryFissionGames
testdata config should be a metrics pipeline, not a logs one
be3860e
@BinaryFissionGames
Rename to resourceattributetransposer
818316c
@BinaryFissionGames
Suffix package with processor, follow conventions
a096ecc 
I realized how it was redundant to have it under a processor package,
then have it suffixed with processor, then I realized not doing
that was breaking convention.
@BinaryFissionGames
Change header in README
f7439d3
@BinaryFissionGames BinaryFissionGames merged commit 47cc1a2 into master Nov 18, 2021
@BinaryFissionGames BinaryFissionGames deleted the resource_attrs_copy_processor branch November 18, 2021 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmwilliams89 jmwilliams89 jmwilliams89 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@BinaryFissionGames @jmwilliams89