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

declare expvarreceiver as alpha and add to components.go #10847

Merged

Conversation

jamesmoessis
Copy link
Contributor

@jamesmoessis jamesmoessis commented Jun 8, 2022

Description:

  • Add expvar receiver to components list.
  • Declare component as alpha

Link to tracking Issue: #9592

@jamesmoessis jamesmoessis changed the title add expvarreceiver to components.go declare expvarreceiver as alpha and add to components.go Jun 9, 2022
@jamesmoessis jamesmoessis marked this pull request as ready for review June 9, 2022 00:43
@jamesmoessis jamesmoessis requested review from a team and Aneurysm9 June 9, 2022 00:43
@jamesmoessis jamesmoessis force-pushed the expvar-in-components-list branch from 28acbca to 70a6e51 Compare June 9, 2022 00:45
Copy link
Member

@TylerHelmuth TylerHelmuth left a comment

Choose a reason for hiding this comment

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

Please create a status header like the one detailed in this issue: #10116

@TylerHelmuth
Copy link
Member

Also, if you haven't already, open a PR in the opentelemetry-collector-releases repo to add this component to the manifest so it is included in the contrib distribution.

@jamesmoessis
Copy link
Contributor Author

Thanks @TylerHelmuth for letting me know, I wasn't aware of these extra new steps. I've pushed up the status header, and I raised a PR in the releases repo: open-telemetry/opentelemetry-collector-releases#149

@codeboten codeboten merged commit a93e385 into open-telemetry:main Jun 9, 2022
kentquirk pushed a commit to McSick/opentelemetry-collector-contrib that referenced this pull request Jun 14, 2022
…try#10847)

* add expvarreceiver to components.go

* add to changelog

* add to go.mod

* mark as alpha

* add status header

* alpha not beta

* add links
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.

3 participants