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

[receiver/statsdreceiver] update to use generated status #21286

Merged
merged 4 commits into from
May 24, 2023

Conversation

my-git9
Copy link
Member

@my-git9 my-git9 commented May 1, 2023

Description:

Link to tracking Issue:
#21213
Testing:

Documentation:

@my-git9 my-git9 requested a review from a team May 1, 2023 15:38
@my-git9 my-git9 requested review from jmacd and dmitryax as code owners May 1, 2023 15:38
@github-actions github-actions bot added the receiver/statsd statsd related issues label May 1, 2023
@dmitryax dmitryax added the Skip Changelog PRs that do not require a CHANGELOG.md entry label May 3, 2023
receiver/statsdreceiver/factory.go Outdated Show resolved Hide resolved
@my-git9 my-git9 requested a review from codeboten as a code owner May 3, 2023 09:39
@github-actions github-actions bot requested a review from altuner May 3, 2023 09:40
@my-git9 my-git9 requested a review from dmitryax May 3, 2023 12:25
@atoulme
Copy link
Contributor

atoulme commented May 17, 2023

Please rebase, sorry, I added the splunk distro to this receiver.

Copy link
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

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

Please update the factory to use the generated Type and Stability consts here:

typeStr = "statsd"
stability = component.StabilityLevelBeta

class: receiver
stability:
beta: [metrics]
distributions: [contrib, splunk]
Copy link
Contributor

Choose a reason for hiding this comment

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

Needs another rebase, sumo was added. Sorry for all the churn @my-git9

Copy link
Contributor

Choose a reason for hiding this comment

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

I fixed the conflict with the github UI, lemme know if that works.

Copy link
Member

Choose a reason for hiding this comment

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

This is still unresolved. sumo is added in the doc only, not here

Copy link
Contributor

Choose a reason for hiding this comment

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

I pushed 2 more commits to get this synchronized

@codeboten codeboten closed this May 24, 2023
@codeboten codeboten reopened this May 24, 2023
@codeboten codeboten merged commit 0df5a2f into open-telemetry:main May 24, 2023
@github-actions github-actions bot added this to the next release milestone May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg/stanza receiver/azuremonitor receiver/statsd statsd related issues Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants