-
Notifications
You must be signed in to change notification settings - Fork 234
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
Enabled Snyk on collector, collector contrib, and collector releases #2228
Comments
Just a follow up on this, what's the requirements around the license check for snyk? We have the license check failing on MPL license in open-telemetry/opentelemetry-collector-contrib#34199 |
@austinlparker @jpkrohling Could the maintainers of each Collector repo the integration runs on be given access to the corresponding dashboard? Right now it runs on PRs, and it would be nice to know why a particular PR causes a failure. |
(we have a somewhat similar issue in Java instrumentation: open-telemetry/opentelemetry-java-instrumentation#10705 (comment)) |
thanks @trask, have you applied for an exception? |
hey @codeboten, no I haven't, it looks like we need to open an issue similar to these: https://github.com/cncf/foundation/issues?q=is%3Aissue+is%3Aopen+label%3Alicensing |
@evan-bradley , absolutely! To maintainers reading this: just ping me and tell me your email addressed and I'll add you to Snyk. |
Documenting that the Snyk integration (thru CNCF) has been enabled on collector repositories per @jpkrohling.
The text was updated successfully, but these errors were encountered: