[chore] manage jmx receiver lifecycle (open-telemetry#17554)

Handle the shutdown of the jmx receiver properly, providing a valid config so it can be created

internal/components/receivers_test.go

@@ -40,6 +40,7 @@ import (
  "github.com/open-telemetry/opentelemetry-collector-contrib/receiver/carbonreceiver"
  "github.com/open-telemetry/opentelemetry-collector-contrib/receiver/chronyreceiver"
  "github.com/open-telemetry/opentelemetry-collector-contrib/receiver/filelogreceiver"
+ "github.com/open-telemetry/opentelemetry-collector-contrib/receiver/jmxreceiver"
  "github.com/open-telemetry/opentelemetry-collector-contrib/receiver/mongodbatlasreceiver"
  "github.com/open-telemetry/opentelemetry-collector-contrib/receiver/otlpjsonfilereceiver"
  "github.com/open-telemetry/opentelemetry-collector-contrib/receiver/prometheusreceiver"
@@ -198,6 +199,12 @@ func TestDefaultReceivers(t *testing.T) {
  {
  receiver: "jmx",
  skipLifecyle: true, // Requires a running instance with JMX
+ getConfigFn: func() component.Config {
+ cfg := jmxreceiver.NewFactory().CreateDefaultConfig().(*jmxreceiver.Config)
+ cfg.Endpoint = "localhost:1234"
+ cfg.TargetSystem = "jvm"
+ return cfg
+ },
  },
  {
  receiver: "journald",

receiver/jmxreceiver/receiver.go

@@ -119,6 +119,9 @@ func (jmx *jmxMetricReceiver) Start(ctx context.Context, host component.Host) er
 }
 
 func (jmx *jmxMetricReceiver) Shutdown(ctx context.Context) error {
+ if jmx.subprocess == nil {
+ return nil
+ }
  jmx.logger.Debug("Shutting down JMX Receiver")
  subprocessErr := jmx.subprocess.Shutdown(ctx)
  otlpErr := jmx.otlpReceiver.Shutdown(ctx)