feat(otelcol.receiver): Add GitHub Receiver#5109
Closed
matt-gp wants to merge 9 commits intografana:mainfrom
Closed
feat(otelcol.receiver): Add GitHub Receiver#5109matt-gp wants to merge 9 commits intografana:mainfrom
matt-gp wants to merge 9 commits intografana:mainfrom
Conversation
matt-gp
force-pushed
the
otelcol-receiver-github
branch
from
0640c27 to
780e8d8
Compare
Contributor
clayton-cornell
left a comment
clayton-cornell
left a comment
There was a problem hiding this comment.
Docs look OK. A couple really minor passive voice vs active voice suggestions.
clayton-cornell
added
the
type/docs
dehaansa
reviewed
Jan 9, 2026
Contributor
dehaansa
left a comment
dehaansa
left a comment
There was a problem hiding this comment.
Couple questions, and then we'll also want a quick converter set up for the receiver, see the other examples in internal/converter/internal/otelcolconvert/
matt-gp
force-pushed
the
otelcol-receiver-github
branch
3 times, most recently
from
ea15b09 to
c03f704
Compare
matt-gp
changed the title
matt-gp
force-pushed
the
otelcol-receiver-github
branch
2 times, most recently
from
50f35d6 to
9a0122c
Compare
matt-gp
force-pushed
the
otelcol-receiver-github
branch
2 times, most recently
from
f9db2db to
bf1f490
Compare
Contributor
Author
|
Just to update this, I'm still working on it, having problems with dependencies as the Otel Github Receiver requires that the collector libs be updated from |
matt-gp
force-pushed
the
otelcol-receiver-github
branch
6 times, most recently
from
26c5d2b to
54ce204
Compare
This PR adds the opentelemetry github receiver.
matt-gp
force-pushed
the
otelcol-receiver-github
branch
from
54ce204 to
38c7d67
Compare
…#5528) ### Pull Request Details Noticed that we do not close file if we cannot get encoding. Review all other paths and we close file properly. I really wish go had something like [errdefer like zig does](https://ziglang.org/documentation/master/#errdefer). ### Issue(s) fixed by this Pull Request <!-- Uncomment the following line and fill in an issue number if you want a GitHub issue to be closed automatically when this PR gets merged. --> <!-- Fixes #issue_id --> ### Notes to the Reviewer <!-- Add any relevant notes for the reviewers and testers of this PR. --> ### PR Checklist <!-- Remove items that do not apply. For completed items, change [ ] to [x]. --> - [ ] Documentation added - [ ] Tests updated - [ ] Config converters updated
### Brief description of Pull Request I built a small tool with two different commands `pr-1` and `pr-2`, these should be triggered from our make targets `make update-go-version-pr-1 VERSION=1.25.7` and `make update-go-version-pr-2 VERSION=1.25.7`. There is also a diff for some modules that was missed in previous pr:s to update version ### Pull Request Details <!-- Add a more detailed descripion of the Pull Request here, if needed. --> ### Issue(s) fixed by this Pull Request <!-- Uncomment the following line and fill in an issue number if you want a GitHub issue to be closed automatically when this PR gets merged. --> <!-- Fixes #issue_id --> ### Notes to the Reviewer <!-- Add any relevant notes for the reviewers and testers of this PR. --> ### PR Checklist <!-- Remove items that do not apply. For completed items, change [ ] to [x]. --> - [ ] Documentation added - [ ] Tests updated - [ ] Config converters updated
…pose new collectors settings (grafana#4640) #### PR Description Update the `postgres_exporter` to version [v0.19.0](https://github.com/prometheus-community/postgres_exporter/releases/tag/v0.19.0) (but continue to use our [fork](https://github.com/grafana/postgres_exporter/tree/exporter-package-v0.19.0)). The update adds support for configuring the `stat_statements` collector flags, including `limit`, `exclude_databases`, and `exclude_users`. #### Which issue(s) this PR fixes Fixes grafana#5502 #### Notes to the Reviewer #### PR Checklist <!-- Remove items that do not apply. For completed items, change [ ] to [x]. --> - [ ] CHANGELOG.md updated - [x] Documentation added - [ ] Tests updated - [ ] Config converters updated --------- Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
### Brief description of Pull Request `prometheus.write.queue` is deprecated and will be removed in the future. We have decided to invest further in `prometheus.remote_write` over further investments in `prometheus.write.queue` as an outcome of grafana#5039. ### Issue(s) fixed by this Pull Request Resolves: grafana#5506 ### PR Checklist - [x] Documentation added --------- Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
matt-gp
force-pushed
the
otelcol-receiver-github
branch
from
f3d403d to
726354a
Compare
Contributor
Author
|
FYI I closed this as I rebased it incorrectly and raised #5542 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Description
This PR adds the opentelemetry github receiver https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/receiver/githubreceiver
This PR supersedes #4914 which got closed in error.
Which issue(s) this PR fixes
Notes to the Reviewer
PR Checklist