search

open-telemetry / opentelemetry-collector-releases

OpenTelemetry Collector Official Releases
https://opentelemetry.io
Apache License 2.0
240 stars 153 forks source link

Include all licenses in the resulting containers #45

Open jpkrohling opened 2 years ago

jpkrohling commented 2 years ago

From https://github.com/open-telemetry/opentelemetry-collector/pull/2604

bogdandrutu commented 2 years ago

I would generate this file as part of the "builder" and maybe include in the container if we want.

shree007 commented 2 years ago

@jpkrohling i may work

shree007 commented 2 years ago

I was out of station. I started to look into it.

jpkrohling commented 2 years ago

Welcome back!

shree007 commented 2 years ago

@jpkrohling I went throgh the https://github.com/open-telemetry/opentelemetry-collector/pull/2604 So all licences will be copied into third-party folder in build phase and that folder will be copied in docker container, for which i need to write docker insturctions. right ?

jpkrohling commented 2 years ago

That sounds right.

shree007 commented 2 years ago

@jpkrohling From https://github.com/open-telemetry/opentelemetry-collector/pull/2604 here, go-licenses has been used to collect the licenses which is this https://github.com/google/go-licenses. Its first prerequisites that targeted folder should have .go file, Earlier when I started to work, There was internal/tools folder which contains .go files, Now I am not seeing any .go files in main branch.

May you guide me little bit here to proceed ? Thanks

jpkrohling commented 2 years ago

The sources are generated. Run 'make build' and you should end up with the source code for each one of the distributions. This source code isn't checked in, though.