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

Document GitHub apps/integrations #1845

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

austinlparker
Copy link
Member

@austinlparker austinlparker commented Dec 21, 2023

Related to #1844

This is incomplete as of now, we need help to fill it out and to perform appropriate security reviews.

Copy link
Member

@svrnm svrnm left a comment

Choose a reason for hiding this comment

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

My review comments add the links to the (hopefully) correct github apps, to make it easier to identify them. While doing so, it looks like ciMate is dead and stale bot states that it is deprecated and should be replaced by the stale gh action.

tools/github/README.md Outdated Show resolved Hide resolved
tools/github/README.md Outdated Show resolved Hide resolved
tools/github/README.md Outdated Show resolved Hide resolved
tools/github/README.md Outdated Show resolved Hide resolved
tools/github/README.md Outdated Show resolved Hide resolved
tools/github/README.md Outdated Show resolved Hide resolved
tools/github/README.md Outdated Show resolved Hide resolved
tools/github/README.md Outdated Show resolved Hide resolved
tools/github/README.md Outdated Show resolved Hide resolved
tools/github/README.md Outdated Show resolved Hide resolved
austinlparker and others added 10 commits December 22, 2023 09:12
Co-authored-by: Severin Neumann <severin.neumann@altmuehlnet.de>
Co-authored-by: Severin Neumann <severin.neumann@altmuehlnet.de>
Co-authored-by: Severin Neumann <severin.neumann@altmuehlnet.de>
Co-authored-by: Severin Neumann <severin.neumann@altmuehlnet.de>
Co-authored-by: Severin Neumann <severin.neumann@altmuehlnet.de>
Co-authored-by: Severin Neumann <severin.neumann@altmuehlnet.de>
Co-authored-by: Severin Neumann <severin.neumann@altmuehlnet.de>
Co-authored-by: Severin Neumann <severin.neumann@altmuehlnet.de>
Co-authored-by: Severin Neumann <severin.neumann@altmuehlnet.de>
Co-authored-by: Severin Neumann <severin.neumann@altmuehlnet.de>
Copy link
Member

@trask trask left a comment

Choose a reason for hiding this comment

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

@open-telemetry/technical-committee can you verify the list and let us know the scopes that are currently set for each?

@reyang
Copy link
Member

reyang commented Jan 2, 2024

@open-telemetry/technical-committee can you verify the list and let us know the scopes that are currently set for each?

  1. I just checked and the list is correct.
  2. I've updated the 3rd column in the following table based on the current configuration.
Name Description Scopes Notes
Azure Pipelines Support for Azure Pipelines open-telemetry/opentelemetry-dotnet, open-telemetry/opentelemetry-dotnet-contrib, open-telemetry/opentelemetry-dotnet-instrumentation Included by default
ciMate TBD open-telemetry/opentelemetry-java-instrumentation TBD
CircleCI Checks Integration with CircleCI all repositories TBD
Codecov Integration with Codecov for PR code coverage checks all repositories TBD
Forking Renovate Dependency updates via Renovate open-telemetry/opentelemetry-js, open-telemetry/opentelemetry-js-contrib TBD
EasyCLA Required CLA checking from LF all repositories This must be enabled for all repositories for PRs against main branch.
Mergify TBD open-telemetry/opentelemetry-operator TBD
Netlify Integration with Netlify for website deployments all repositories n/a
Octobox TBD all repositories TBD
OpenShift CI TBD open-telemetry/opentelemetry-operator TBD
Polls TBD all repositories TBD
project-bot TBD open-telemetry/opentelemetry-collector, open-telemetry/opentelemetry-collector-contrib TBD
Renovate Dependency updates via Renovate open-telemetry/opentelemetry-java, open-telemetry/opentelemetry-erlang, open-telemetry/opentelemetry-collector, open-telemetry/opentelemetry-collector-contrib, open-telemetry/opentelemetry-java-instrumentation, open-telemetry/opentelemetry-java-contrib, open-telemetry/opentelemetry-erlang-contrib, open-telemetry/opentelemetry-java-examples, open-telemetry/opentelemetry-demo, open-telemetry/opentelemetry-android TBD
Scope App TBD open-telemetry/opentelemetry-swift TBD
Slack Slack integration for GitHub all repositories TBD
Stale Stale issue bot all repositories n/a
Welcome TBD open-telemetry/opentelemetry-php, open-telemetry/opentelemetry-php-contrib TBD

@trask
Copy link
Member

trask commented Jan 3, 2024

Name Description Scopes Notes
ciMate TBD open-telemetry/opentelemetry-java-instrumentation TBD

sent #1849 to remove this as it's no longer being used

@svrnm
Copy link
Member

svrnm commented Jan 19, 2024

What are our next steps on this? Anything I can help with? Or is this something that can be handed over to SIG security?

@reyang
Copy link
Member

reyang commented Jan 19, 2024

FYI - most likely we don't need Azure Pipelines anymore. I'm confirming with the maintainers to see if we can remove it.

image

@reyang
Copy link
Member

reyang commented Jan 24, 2024

Confirmed with repo maintainers and removed the Azure Pipeline since it's not needed.

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.

None yet

4 participants