-
Notifications
You must be signed in to change notification settings - Fork 249
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
Add otelbot GitHub App to assets #2548
Conversation
c79ac6c
to
bb4f8b1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is really cool, looking forward to give it a try on the oteldotio repo!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM after addressing Severin's comment
Co-authored-by: Severin Neumann <severin.neumann@altmuehlnet.de>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very cool! See inline for copyedits.
Also, after the edits are made, could you run Prettier over this file (here or in a separate PR). |
Co-authored-by: Patrice Chalin <chalin@users.noreply.github.com>
Co-authored-by: Patrice Chalin <chalin@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Related to #1549 and #2127.
This is more secure than using @opentelemetrybot (a repo can't grant more permissions that the app requests), and can be managed directly in the org (instead of as a separate github account with separate credentials).
May need other GitHub App "bots" with more permissions that are isolated to specific repos.
Opening as draft while I test things out in the Java instrumentation repo...