Merge pull request #14 from worksome/feature/otel-config

feat: update to use Laravel Telemetry v0.5
worksome · Jul 25, 2024 · 127628e · 127628e
2 parents 122c738 + 551ac2d
commit 127628e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/composer.json b/composer.json
@@ -20,7 +20,7 @@
  "illuminate/contracts": "^10.0 || ^11.0",
  "laravel/horizon": "^5.22.1",
  "open-telemetry/api": "^1.0",
- "worksome/laravel-telemetry": "^0.4"
+ "worksome/laravel-telemetry": "^0.5"
  },
  "require-dev": {
  "nunomaduro/collision": "^7.10 || ^8.1",

diff --git a/src/HorizonTelemetryServiceProvider.php b/src/HorizonTelemetryServiceProvider.php
@@ -30,8 +30,8 @@ public function boot(): void
  /** @var Repository $config */
  $config = $this->app->make(Repository::class);
 
- if (! $config->get('telemetry.enabled', true)) {
- return null;
+ if ($config->get('telemetry.sdk.disabled')) {
+ return;
  }
 
  if ($config->get($this->configKey(MeterName::FailedJobs), true)) {
@@ -52,8 +52,8 @@ public function boot(): void
  /** @var Repository $config */
  $config = $this->app->make(Repository::class);
 
- if (! $config->get('telemetry.enabled', true)) {
- return null;
+ if ($config->get('telemetry.sdk.disabled')) {
+ return;
  }
 
  if ($currentMasterSupervisorsSchedule = $config->get(