Error on install

The Fatal error: Uncaught Symfony\Component\DependencyInjection\Exception\ServiceCircularReferenceException: Circular reference detected for service is mentioned in the code at https://git.drupalcode.org/project/opentelemetry/-/blob/1.0.x/src/EventS...

Thanks for reporting! Will try to fix this.

The message:
[warning] Fallback OpenTelemetry endpoint http://localhost:4318 is not available. Please set the custom endpoint in the module settings. Parent exception: Export failure: Export retry limit exceeded
is expected, it just warns you that the endpoint is not available.

But about the exception - I can't reproduce the problem on my side to debug and investigate.

Please share the Drupal version and installed modules on your side, and any other non-standard environment settings.

I had https://www.drupal.org/project/webprofiler → installed and that relies upon https://www.drupal.org/project/tracer → . If I uninstall both then /admin/config/development/opentelemetry serves ok...

> is expected, it just warns you that the endpoint is not available.

This is similar to issues I opened, which changed it from an error to a warning, but I think that's still problematic.

I think there really needs to be a complete off switch without having to uninstall the modules, for example for non-production/local development environments, CI and also initial install.

There is the disable checkbox, but currently it's only respected for traces (without explaining that) and not logs.

Drupal modules can include a link to their configuration site which is shown after install now by default in drush, there could be a runtime requirements hook to show an error if the connection fails.

How about this idea as a compromise?

Require an explicit configuration of the endpoint, if the field is empty then skip tracing/logging/whatever, but ship it with localhost as default configuration?

if ($this->settings->get(OpentelemetryService::SETTING_ENDPOINT))  {
      $this->logger = $this->loggerProvider->getLogger(
        $this->settings->get(OpentelemetryService::SETTING_SERVICE_NAME) ?? OpentelemetryService::SERVICE_NAME_FALLBACK
      );
    }
    else {
      $this->logger = NoopLogger::getInstance();
    }

There might be better ways to set that up, but it wasn't obvious to me.

And make the disable checkbox in the UI clearer: "Disable tracing"

Or, make the disable checkbox apply to everything and then check for that instead of the endpoint. But I actually see use cases for having logging enabled but not tracing. Especially since I won't be able to have a local opentelemetry collector to batch data together, I'm still unsure how much data I really want to collect and process.

Created a merge requests, doesn't quite fit the actual issue, can also move it somewhere.