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/googlecloudmonitoringreceiver] Move googlecloudmonitoringreceiver component stability to alpha #35136

Conversation

abhishek-at-cloudwerx
Copy link
Contributor

Description: This PR moves the stability of the googlecloudmonitoringreceiver component to alpha

Link to tracking Issue: #33762

Testing: No additional test cases were added

Documentation: Previously updated

- Move googlecloudmonitoringreceiver component from development stage to alpha
- Add chloggen file entry
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.

needs a make gotidy

@abhishek-at-cloudwerx abhishek-at-cloudwerx changed the title Move googlecloudmonitoringreceiver component stability to alpha [receiver/googlecloudmonitoringreceiver] Move googlecloudmonitoringreceiver component stability to alpha Sep 12, 2024
It was listed under processors instead of receivers.

Signed-off-by: Alex Boten <[email protected]>
fix: receiver was in the incorrect config group
@datsabk
Copy link
Contributor

datsabk commented Sep 19, 2024

@TylerHelmuth - Could you please proceed with the merge

@codeboten
Copy link
Contributor

@datsabk it looks like there's still 16 failures in the PR, those will need to get resolved before it's merged

@abhishek-at-cloudwerx abhishek-at-cloudwerx requested a review from a team as a code owner September 20, 2024 05:02
@dashpole
Copy link
Contributor

Needs:

  • make gotidy
  • make genotelcontribcol

@abhishek-at-cloudwerx
Copy link
Contributor Author

@codeboten @TylerHelmuth - Good to proceed.
Sync with master would break the tests everytime so intentionally avoiding sync to master. Last synced a couple of days back.

go 1.22.0

toolchain go1.22.7
go 1.22.7
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
go 1.22.7
go 1.22.0
toolchain go1.22.7

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.

Will update the toolchain after this PR so we can get this merged

@codeboten codeboten merged commit 44c2db4 into open-telemetry:main Sep 25, 2024
156 checks passed
@github-actions github-actions bot added this to the next release milestone Sep 25, 2024
codeboten added a commit to codeboten/opentelemetry-collector-contrib that referenced this pull request Sep 25, 2024
codeboten added a commit to codeboten/opentelemetry-collector-contrib that referenced this pull request Sep 25, 2024
codeboten added a commit that referenced this pull request Sep 25, 2024
jriguera pushed a commit to springernature/opentelemetry-collector-contrib that referenced this pull request Oct 4, 2024
…ceiver component stability to alpha (open-telemetry#35136)

**Description:** This PR moves the stability of the
googlecloudmonitoringreceiver component to alpha

**Link to tracking Issue:**
open-telemetry#33762

**Testing:** No additional test cases were added

**Documentation:** Previously updated

---------

Signed-off-by: Alex Boten <[email protected]>
Co-authored-by: Alex Boten <[email protected]>
jriguera pushed a commit to springernature/opentelemetry-collector-contrib that referenced this pull request Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants