opentelemetry-instrumentation-rails 0.17.0 → 0.19.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 689a1c682f6de7f70663955bdc99a1de4f758ff9d54cc14f9d18d1b5c3896b54
-  data.tar.gz: 43387df2aea22327d040971a3c500db575fcf100114835e8f2707f89e5289230
+  metadata.gz: 9271273081a8a5583bb56e4f4ee781c55bed1ad2b4be4253ab1c0ccc215bddb7
+  data.tar.gz: cfd66986f1e528027900893af155d781a86ff2be5afc2367126dc73cd3d121c2
 SHA512:
-  metadata.gz: 9491e565317e410bfbb0c1dbab9b2b95a0320b68d5018b7aaf4d852f8dcf90a8a3e07e60c36d9fb83378673f14701e644b74456a7b4a0a866335eb1d7424cfd4
-  data.tar.gz: eba631a571aea18efa7aab46a4f39e06466be945ea3e28040fa0961f14265ab4aa80772b49a6cce75d38ec1414b83ede2d70b893fd4d9f545415bcb9b3edce1c
+  metadata.gz: 74c5ded807e64df1319c892772c0f5f83a91c1568a7e0c9f9da6e52e88ab07b9b97e070d7076a461bbc11f8b6ea2d28ac2d8847b60ba2a7c0aa4937189182bea
+  data.tar.gz: 68f3f69014b5b07023970fc5caff33a41011dbbab3ce2b83e4b5b6385383d58e9b702c4c9fc65b6b803e1c0b6353ea52abf72ca82e6b8a436a9b362aa2fd22a8

data/CHANGELOG.md

@@ -1,15 +1,38 @@
 # Release History: opentelemetry-instrumentation-rails
 
-### v0.17.0 / 2021-04-22
+### v0.19.1 / 2021-09-09
+
+* (No significant changes)
+
+### v0.19.0 / 2021-08-12
+
+* ADDED: Instrument active record
+* ADDED: Add ActionView instrumentation via ActiveSupport::Notifications
+* FIXED: Rails instrumentation to not explicitly install sub gems
+* DOCS: Update docs to rely more on environment variable configuration
+
+* This release adds support for Active Record and Action View.
+* The `enable_recognize_route` configuration option has been moved to the ActionPack gem.
+* See readme for details on how to configure the sub instrumentation gems.
+
+### v0.18.1 / 2021-06-23
 
-* ADDED: Added http.route in rails instrumentation to match the spec 
-* FIXED: Rails example by not using `rails` from git 
 * FIXED: Updated rack middleware position to zero 
 
+### v0.18.0 / 2021-05-21
+
+* ADDED: Updated API depedency for 1.0.0.rc1
+
+### v0.17.0 / 2021-04-22
+
+* ADDED: Added http.route in rails instrumentation to match the spec
+* FIXED: Rails example by not using `rails` from git
+* FIXED: Updated rack middleware position to zero
+
 ### v0.16.0 / 2021-03-17
 
-* FIXED: Example scripts now reference local common lib 
-* DOCS: Replace Gitter with GitHub Discussions 
+* FIXED: Example scripts now reference local common lib
+* DOCS: Replace Gitter with GitHub Discussions
 
 ### v0.15.0 / 2021-02-18
 
@@ -29,12 +52,12 @@
 
 ### v0.11.0 / 2020-12-11
 
-* FIXED: Rails tests 
-* FIXED: Copyright comments to not reference year 
+* FIXED: Rails tests
+* FIXED: Copyright comments to not reference year
 
 ### v0.10.0 / 2020-12-03
 
-* FIXED: Otel-instrumentation-all not installing all 
+* FIXED: Otel-instrumentation-all not installing all
 
 ### v0.9.0 / 2020-11-27
 

data/README.md

@@ -14,15 +14,7 @@ Or, if you use [bundler][bundler-home], include `opentelemetry-instrumentation-r
 
 ## Usage
 
-To use the instrumentation, call `use` with the name of the instrumentation:
-
-```ruby
-OpenTelemetry::SDK.configure do |c|
-  c.use 'OpenTelemetry::Instrumentation::Rails'
-end
-```
-
-Alternatively, you can also call `use_all` to install all the available instrumentation.
+To use the Rails instrumentation, call `use_all` so it installs all the instrumentation gems.
 
 ```ruby
 OpenTelemetry::SDK.configure do |c|
@@ -30,15 +22,14 @@ OpenTelemetry::SDK.configure do |c|
 end
 ```
 
-
 ### Configuration options
 
-The `http.route` attribute is disabled by default because we use [.recognize](https://github.com/rails/rails/blob/v6.1.3/actionpack/lib/action_dispatch/journey/router.rb#L65)
+The Rails instrumentation attempts to mirror the structure of the Ruby on Rails.  It is a collection of instrumentation gems for components of Rails such as Action View, Active Record, Action Pack, etc...
+
+You may want to include all of the Rails instrumentation but disable a single instrumentation gem that it includes.  Here is an example of how you can disable Active Record when using this instrumentation gem.
 ```ruby
 OpenTelemetry::SDK.configure do |c|
-  c.use 'OpenTelemetry::Instrumentation::Rails', {
-    enable_recognize_route: true
-  }
+  c.use_all({ 'OpenTelemetry::Instrumentation::ActiveRecord' => { enabled: false } })
 end
 ```
 

data/lib/opentelemetry/instrumentation/rails/instrumentation.rb

@@ -12,30 +12,19 @@ module OpenTelemetry
       # The Instrumentation class contains logic to detect and install the Rails
       # instrumentation
       class Instrumentation < OpenTelemetry::Instrumentation::Base
-        install do |_config|
-          require_dependencies
-          require_railtie
-          patch_metal
-        end
-
-        present do
-          defined?(::Rails)
-        end
+        MINIMUM_VERSION = Gem::Version.new('5.2.0')
 
-        option :enable_recognize_route, default: false, validate: :boolean
+        # This gem requires the instrumentantion gems for the different
+        # components of Rails, as a result it does not have any explicit
+        # work to do in the install step.
+        install { true }
+        present { defined?(::Rails) }
+        compatible { gem_version >= MINIMUM_VERSION }
 
         private
 
-        def require_dependencies
-          require_relative 'patches/action_controller/metal'
-        end
-
-        def require_railtie
-          require_relative 'railtie'
-        end
-
-        def patch_metal
-          ::ActionController::Metal.prepend(Patches::ActionController::Metal)
+        def gem_version
+          Gem.loaded_specs['actionpack'].version
         end
       end
     end

data/lib/opentelemetry/instrumentation/rails/version.rb

@@ -7,7 +7,7 @@
 module OpenTelemetry
   module Instrumentation
     module Rails
-      VERSION = '0.17.0'
+      VERSION = '0.19.1'
     end
   end
 end

data/lib/opentelemetry/instrumentation/rails.rb

@@ -15,6 +15,8 @@ module OpenTelemetry
   end
 end
 
-require 'opentelemetry-instrumentation-rack'
+require 'opentelemetry-instrumentation-action_pack'
+require 'opentelemetry-instrumentation-action_view'
+require 'opentelemetry-instrumentation-active_record'
 require_relative './rails/instrumentation'
 require_relative './rails/version'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: opentelemetry-instrumentation-rails
 version: !ruby/object:Gem::Version
-  version: 0.17.0
+  version: 0.19.1
 platform: ruby
 authors:
 - OpenTelemetry Authors
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-04-23 00:00:00.000000000 Z
+date: 2021-09-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: opentelemetry-api
@@ -16,42 +16,70 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.17.0
+        version: 1.0.0.rc3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.17.0
+        version: 1.0.0.rc3
 - !ruby/object:Gem::Dependency
-  name: opentelemetry-instrumentation-base
+  name: opentelemetry-instrumentation-action_pack
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.1.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.1.0
+- !ruby/object:Gem::Dependency
+  name: opentelemetry-instrumentation-action_view
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.17.0
+        version: 0.1.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.17.0
+        version: 0.1.1
 - !ruby/object:Gem::Dependency
-  name: opentelemetry-instrumentation-rack
+  name: opentelemetry-instrumentation-active_record
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.1.1
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.1.1
+- !ruby/object:Gem::Dependency
+  name: opentelemetry-instrumentation-base
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.17.0
+        version: 0.18.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.17.0
+        version: 0.18.2
 - !ruby/object:Gem::Dependency
   name: appraisal
   requirement: !ruby/object:Gem::Requirement
@@ -100,14 +128,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.0'
+        version: 1.0.0.rc1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.0'
+        version: 1.0.0.rc1
 - !ruby/object:Gem::Dependency
   name: rack-test
   requirement: !ruby/object:Gem::Requirement
@@ -235,8 +263,6 @@ files:
 - lib/opentelemetry/instrumentation.rb
 - lib/opentelemetry/instrumentation/rails.rb
 - lib/opentelemetry/instrumentation/rails/instrumentation.rb
-- lib/opentelemetry/instrumentation/rails/patches/action_controller/metal.rb
-- lib/opentelemetry/instrumentation/rails/railtie.rb
 - lib/opentelemetry/instrumentation/rails/version.rb
 homepage: https://github.com/open-telemetry/opentelemetry-ruby
 licenses:

data/lib/opentelemetry/instrumentation/rails/patches/action_controller/metal.rb

@@ -1,38 +0,0 @@
-# frozen_string_literal: true
-
-# Copyright The OpenTelemetry Authors
-#
-# SPDX-License-Identifier: Apache-2.0
-
-module OpenTelemetry
-  module Instrumentation
-    module Rails
-      module Patches
-        module ActionController
-          # Module to prepend to ActionController::Metal for instrumentation
-          module Metal
-            def dispatch(name, request, response)
-              rack_span = OpenTelemetry::Instrumentation::Rack.current_span
-              rack_span.name = "#{self.class.name}##{name}" if rack_span.context.valid? && !request.env['action_dispatch.exception']
-
-              add_rails_route(rack_span, request) if instrumentation_config[:enable_recognize_route]
-              super(name, request, response)
-            end
-
-            private
-
-            def add_rails_route(rack_span, request)
-              ::Rails.application.routes.router.recognize(request) do |route, _params|
-                rack_span.set_attribute('http.route', route.path.spec.to_s)
-              end
-            end
-
-            def instrumentation_config
-              Rails::Instrumentation.instance.config
-            end
-          end
-        end
-      end
-    end
-  end
-end

data/lib/opentelemetry/instrumentation/rails/railtie.rb

@@ -1,25 +0,0 @@
-# frozen_string_literal: true
-
-# Copyright The OpenTelemetry Authors
-#
-# SPDX-License-Identifier: Apache-2.0
-
-module OpenTelemetry
-  module Instrumentation
-    module Rails
-      # The Instrumentation class contains logic to detect and install the Rails
-      # instrumentation, while this Railtie is used to conventionally instrument
-      # the Rails application through its initialization hooks
-      class Railtie < ::Rails::Railtie
-        config.before_initialize do |app|
-          OpenTelemetry::Instrumentation::Rack::Instrumentation.instance.install({})
-
-          app.middleware.insert_after(
-            0,
-            OpenTelemetry::Instrumentation::Rack::Middlewares::TracerMiddleware
-          )
-        end
-      end
-    end
-  end
-end