[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.