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/githubreceiver] add githubreceiver alpha component to manifest #656

Closed
wants to merge 2 commits into from

Conversation

adrielp
Copy link

@adrielp adrielp commented Sep 4, 2024

No description provided.

@adrielp adrielp requested review from a team and codeboten September 4, 2024 15:17
@adrielp
Copy link
Author

adrielp commented Sep 4, 2024

Corresponding promotion of component to alpha status open-telemetry/opentelemetry-collector-contrib#34960

@adrielp
Copy link
Author

adrielp commented Sep 4, 2024

I preemptively set the version to v0.109.0 so that it could be merged in as part of the next release. The githubreceiver was renamed from the gitproviderreceiver and then promoted to alpha so if I had set as v0.108.0 the workflows would fail, and of course at time of writing, v0.109.0 hasn't been cut yet. If able, please merge in as part of v0.109.0 release.

@TylerHelmuth
Copy link
Member

@adrielp I reran CI after contrib's release, but things still failed.

@adrielp
Copy link
Author

adrielp commented Sep 10, 2024

@TylerHelmuth - the workflow looked odd, like a failure due to the strict flag on the builder (I didn't look too far in the log or action). Looks like it's being added in via this commit though #660 (comment) which may just wor.k. I'll close this PR if it's being added in the other and if it failed there too (i see the correct tags on contrib for go mod versioning to work) then can take a deeper look at open another pr.

@adrielp adrielp closed this Sep 10, 2024
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.

4 participants