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

REQUEST: Add adrielp as a GitHub App manager #2581

Closed
adrielp opened this issue Feb 19, 2025 · 2 comments
Closed

REQUEST: Add adrielp as a GitHub App manager #2581

adrielp opened this issue Feb 19, 2025 · 2 comments
Labels
area/project-infra Non-GitHub project infra (DockerHub, etc.)

Comments

@adrielp
Copy link
Contributor

adrielp commented Feb 19, 2025

Request Overview

There are two GitHub Apps that have been transferred over to the OpenTelemetry organization. I'm requesting to be added as an App Manager to be able to update their configuration for the Project Infra work that is going on. These apps will behave in two ways:

OpenTelemetry VCS App Auth - This GitHub application provides read access to repositories, issues, commits, to be able to get VCS metrics for the repositories within OpenTelemetry. Authenticating through this application for REST & GraphQL API calls increase Rate Limit.

Permissions:

  • Read-only access to repositories within OpenTelemetry for contents, metadata, pull requests, and issues.

OpenTelemetry GitHub Events (I think that's what I named it) - This GitHub app will emit GitHub events from repositories within the organization over a webhook to the oracle cloud infrastructure. This will generate traces & semantically compliant events.

Permissions:

  • Read-only subscription for action events where workflow runs will get sent to a collector through a WAF for repositories it's enabled on (for now).
  • Future: Other event subscriptions (like push events, release events, attestation events, issue events, etc with approval*) that can be adjusted to match semantic conventions and can have indicators derived from.

I will need to be able to adjust the configuration of these Apps to be able to configure things on both ends.

Request Context

In collaboration between the CICD, Sem Conv, and Project Infra SIGs, we have been working to get some infrastructure stood up within OpenTelemetry that examines the VCS metrics from the GitHub receiver and action traces. The hope is to get most of this done by Kubecon EU, but enable the project has a whole to get some additional telemetry around the project, dogfooding in essence our own work. (OpenTelemetry inception 😄)

I've attached some example dashboards from other projects this receiver has been used against as examples.

Image

Image

cc. @austinlparker @trask

@danielgblanco danielgblanco added the area/project-infra Non-GitHub project infra (DockerHub, etc.) label Feb 20, 2025
@adrielp
Copy link
Contributor Author

adrielp commented Feb 20, 2025

Updated with permissions, context, and example screenshots.

Also, cc. @danielgblanco since we were talking about this earlier.

@adrielp
Copy link
Contributor Author

adrielp commented Feb 27, 2025

An update. I've been granted the app manager permissions for these two apps. After further discussion in slack we agreed that for any permissions changes I make, I'll open an issue here to document that change during the initial setup process. Additionally, once configuration and setup is complete, we can remove my app manager permissions.

This should be good to mark as resolved.

@trask trask closed this as completed Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/project-infra Non-GitHub project infra (DockerHub, etc.)
Projects
None yet
Development

No branches or pull requests

3 participants