[cmd/opampsupervisor]: Refactor NewSupervisor to take a config instead of a filepath

[BinaryFissionGames]

Component(s)

cmd/opampsupervisor

Describe the issue you're reporting

The supervisor currently takes in a path to a config file.

opentelemetry-collector-contrib/cmd/opampsupervisor/supervisor/supervisor.go

Line 141 in 20d6bae

func NewSupervisor(logger *zap.Logger, configFile string) (*Supervisor, error) {

Instead, it should take in a config struct (this config can be read from disk in main). This makes it easier to unit test and easier to use as a library, since the config would not need to be written to disk in order to run the supervisor.

[github-actions]

Pinging code owners:

• cmd/opampsupervisor: @evan-bradley @atoulme @tigrannajaryan @BinaryFissionGames

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[crobert-1]

Removed needs triage as this makes sense to me, was filed by a code owner, and another code owner responded with +1.

[github-actions]

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

• cmd/opampsupervisor: @evan-bradley @atoulme @tigrannajaryan @BinaryFissionGames

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[crobert-1]

Resolved by #34597