newrelic_rpm 6.6.0.358 → 6.11.0
Sign up to get free protection for your applications and to get access to all the features.
Potentially problematic release.
This version of newrelic_rpm might be problematic. Click here for more details.
- checksums.yaml +4 -4
- data/.gitignore +1 -0
- data/.travis.yml +65 -5
- data/CHANGELOG.md +222 -0
- data/Gemfile +5 -2
- data/Guardfile +18 -1
- data/LICENSE +1 -1
- data/Rakefile +2 -0
- data/lib/new_relic/agent.rb +89 -7
- data/lib/new_relic/agent/agent.rb +114 -55
- data/lib/new_relic/agent/agent_logger.rb +4 -0
- data/lib/new_relic/agent/attribute_filter.rb +7 -7
- data/lib/new_relic/agent/attributes.rb +150 -0
- data/lib/new_relic/agent/autostart.rb +19 -14
- data/lib/new_relic/agent/configuration/default_source.rb +127 -9
- data/lib/new_relic/agent/configuration/event_harvest_config.rb +11 -5
- data/lib/new_relic/agent/configuration/manager.rb +0 -8
- data/lib/new_relic/agent/configuration/server_source.rb +3 -2
- data/lib/new_relic/agent/configuration/yaml_source.rb +11 -6
- data/lib/new_relic/agent/connect/request_builder.rb +5 -13
- data/lib/new_relic/agent/database.rb +1 -2
- data/lib/new_relic/agent/datastores/mongo.rb +1 -1
- data/lib/new_relic/agent/datastores/mongo/event_formatter.rb +2 -2
- data/lib/new_relic/agent/datastores/mongo/obfuscator.rb +8 -8
- data/lib/new_relic/agent/distributed_tracing.rb +155 -6
- data/lib/new_relic/agent/{cross_app_payload.rb → distributed_tracing/cross_app_payload.rb} +0 -0
- data/lib/new_relic/agent/{cross_app_tracing.rb → distributed_tracing/cross_app_tracing.rb} +60 -45
- data/lib/new_relic/agent/distributed_tracing/distributed_trace_intrinsics.rb +80 -0
- data/lib/new_relic/agent/distributed_tracing/distributed_trace_metrics.rb +75 -0
- data/lib/new_relic/agent/{distributed_trace_payload.rb → distributed_tracing/distributed_trace_payload.rb} +23 -99
- data/lib/new_relic/agent/distributed_tracing/distributed_trace_transport_type.rb +39 -0
- data/lib/new_relic/agent/distributed_tracing/trace_context.rb +246 -0
- data/lib/new_relic/agent/distributed_tracing/trace_context_payload.rb +126 -0
- data/lib/new_relic/agent/error_collector.rb +33 -16
- data/lib/new_relic/agent/error_event_aggregator.rb +7 -5
- data/lib/new_relic/agent/external.rb +7 -7
- data/lib/new_relic/agent/guid_generator.rb +28 -0
- data/lib/new_relic/agent/hostname.rb +7 -1
- data/lib/new_relic/agent/http_clients/abstract.rb +82 -0
- data/lib/new_relic/agent/http_clients/curb_wrappers.rb +24 -19
- data/lib/new_relic/agent/http_clients/excon_wrappers.rb +28 -13
- data/lib/new_relic/agent/http_clients/http_rb_wrappers.rb +17 -21
- data/lib/new_relic/agent/http_clients/httpclient_wrappers.rb +10 -11
- data/lib/new_relic/agent/http_clients/net_http_wrappers.rb +16 -4
- data/lib/new_relic/agent/http_clients/typhoeus_wrappers.rb +4 -6
- data/lib/new_relic/agent/http_clients/uri_util.rb +3 -2
- data/lib/new_relic/agent/instrumentation/action_cable_subscriber.rb +5 -7
- data/lib/new_relic/agent/instrumentation/action_controller_subscriber.rb +4 -0
- data/lib/new_relic/agent/instrumentation/action_view_subscriber.rb +11 -2
- data/lib/new_relic/agent/instrumentation/active_record.rb +4 -2
- data/lib/new_relic/agent/instrumentation/active_record_subscriber.rb +7 -2
- data/lib/new_relic/agent/instrumentation/active_storage_subscriber.rb +8 -4
- data/lib/new_relic/agent/instrumentation/bunny.rb +45 -28
- data/lib/new_relic/agent/instrumentation/curb.rb +59 -18
- data/lib/new_relic/agent/instrumentation/data_mapper.rb +3 -1
- data/lib/new_relic/agent/instrumentation/excon.rb +1 -1
- data/lib/new_relic/agent/instrumentation/excon/connection.rb +6 -3
- data/lib/new_relic/agent/instrumentation/excon/middleware.rb +2 -1
- data/lib/new_relic/agent/instrumentation/grape.rb +5 -10
- data/lib/new_relic/agent/instrumentation/http.rb +6 -3
- data/lib/new_relic/agent/instrumentation/httpclient.rb +5 -3
- data/lib/new_relic/agent/instrumentation/memcache.rb +3 -1
- data/lib/new_relic/agent/instrumentation/memcache/dalli.rb +6 -2
- data/lib/new_relic/agent/instrumentation/mongo.rb +9 -3
- data/lib/new_relic/agent/instrumentation/mongodb_command_subscriber.rb +13 -0
- data/lib/new_relic/agent/instrumentation/net.rb +6 -3
- data/lib/new_relic/agent/instrumentation/notifications_subscriber.rb +25 -1
- data/lib/new_relic/agent/instrumentation/redis.rb +9 -3
- data/lib/new_relic/agent/instrumentation/resque.rb +3 -0
- data/lib/new_relic/agent/instrumentation/sidekiq.rb +0 -1
- data/lib/new_relic/agent/instrumentation/typhoeus.rb +23 -6
- data/lib/new_relic/agent/logging.rb +139 -0
- data/lib/new_relic/agent/messaging.rb +5 -73
- data/lib/new_relic/agent/method_tracer.rb +3 -2
- data/lib/new_relic/agent/method_tracer_helpers.rb +2 -2
- data/lib/new_relic/agent/monitors.rb +27 -0
- data/lib/new_relic/agent/monitors/cross_app_monitor.rb +110 -0
- data/lib/new_relic/agent/monitors/distributed_tracing_monitor.rb +27 -0
- data/lib/new_relic/agent/{inbound_request_monitor.rb → monitors/inbound_request_monitor.rb} +1 -1
- data/lib/new_relic/agent/{synthetics_monitor.rb → monitors/synthetics_monitor.rb} +2 -4
- data/lib/new_relic/agent/new_relic_service.rb +7 -5
- data/lib/new_relic/agent/noticible_error.rb +22 -0
- data/lib/new_relic/agent/span_event_aggregator.rb +1 -0
- data/lib/new_relic/agent/span_event_primitive.rb +82 -53
- data/lib/new_relic/agent/sql_sampler.rb +3 -3
- data/lib/new_relic/agent/supported_versions.rb +2 -2
- data/lib/new_relic/agent/tracer.rb +65 -18
- data/lib/new_relic/agent/transaction.rb +84 -79
- data/lib/new_relic/agent/transaction/abstract_segment.rb +28 -2
- data/lib/new_relic/agent/transaction/distributed_tracer.rb +171 -0
- data/lib/new_relic/agent/transaction/distributed_tracing.rb +57 -146
- data/lib/new_relic/agent/transaction/external_request_segment.rb +29 -36
- data/lib/new_relic/agent/transaction/message_broker_segment.rb +3 -11
- data/lib/new_relic/agent/transaction/segment.rb +7 -1
- data/lib/new_relic/agent/transaction/trace.rb +2 -4
- data/lib/new_relic/agent/transaction/trace_context.rb +168 -0
- data/lib/new_relic/agent/transaction/trace_node.rb +10 -8
- data/lib/new_relic/agent/transaction_error_primitive.rb +10 -15
- data/lib/new_relic/agent/transaction_event_primitive.rb +28 -39
- data/lib/new_relic/cli/commands/deployments.rb +1 -1
- data/lib/new_relic/cli/commands/install.rb +3 -2
- data/lib/new_relic/coerce.rb +31 -6
- data/lib/new_relic/constants.rb +34 -0
- data/lib/new_relic/control/instance_methods.rb +10 -1
- data/lib/new_relic/dependency_detection.rb +4 -4
- data/lib/new_relic/noticed_error.rb +38 -17
- data/lib/new_relic/rack/browser_monitoring.rb +5 -0
- data/lib/new_relic/supportability_helper.rb +14 -0
- data/lib/new_relic/version.rb +1 -1
- data/lib/tasks/multiverse.rb +25 -0
- data/lib/tasks/tests.rake +6 -1
- data/newrelic_rpm.gemspec +18 -7
- data/test/agent_helper.rb +322 -70
- metadata +98 -32
- data/lib/new_relic/agent/cross_app_monitor.rb +0 -110
- data/lib/new_relic/agent/distributed_trace_monitor.rb +0 -40
- data/lib/new_relic/agent/http_clients/abstract_request.rb +0 -31
- data/lib/new_relic/agent/transaction/attributes.rb +0 -154
- data/lib/tasks/versions.html.erb +0 -28
- data/lib/tasks/versions.postface.html +0 -8
- data/lib/tasks/versions.preface.html +0 -9
- data/lib/tasks/versions.rake +0 -65
- data/lib/tasks/versions.txt.erb +0 -14
@@ -14,7 +14,7 @@ DependencyDetection.defer do
|
|
14
14
|
|
15
15
|
executes do
|
16
16
|
::NewRelic::Agent.logger.info 'Installing Curb instrumentation'
|
17
|
-
require 'new_relic/agent/cross_app_tracing'
|
17
|
+
require 'new_relic/agent/distributed_tracing/cross_app_tracing'
|
18
18
|
require 'new_relic/agent/http_clients/curb_wrappers'
|
19
19
|
end
|
20
20
|
|
@@ -22,10 +22,12 @@ DependencyDetection.defer do
|
|
22
22
|
class Curl::Easy
|
23
23
|
|
24
24
|
attr_accessor :_nr_instrumented,
|
25
|
+
:_nr_failure_instrumented,
|
25
26
|
:_nr_http_verb,
|
26
27
|
:_nr_header_str,
|
27
28
|
:_nr_original_on_header,
|
28
29
|
:_nr_original_on_complete,
|
30
|
+
:_nr_original_on_failure,
|
29
31
|
:_nr_serial
|
30
32
|
|
31
33
|
# We have to hook these three methods separately, as they don't use
|
@@ -53,7 +55,7 @@ DependencyDetection.defer do
|
|
53
55
|
|
54
56
|
|
55
57
|
# Hook the #http method to set the verb.
|
56
|
-
def http_with_newrelic
|
58
|
+
def http_with_newrelic verb
|
57
59
|
self._nr_http_verb = verb.to_s.upcase
|
58
60
|
http_without_newrelic( verb )
|
59
61
|
end
|
@@ -72,9 +74,9 @@ DependencyDetection.defer do
|
|
72
74
|
alias_method :perform, :perform_with_newrelic
|
73
75
|
|
74
76
|
# Record the HTTP verb for future #perform calls
|
75
|
-
def method_with_newrelic
|
76
|
-
self._nr_http_verb =
|
77
|
-
method_without_newrelic(
|
77
|
+
def method_with_newrelic verb
|
78
|
+
self._nr_http_verb = verb.upcase
|
79
|
+
method_without_newrelic(verb)
|
78
80
|
end
|
79
81
|
|
80
82
|
alias_method :method_without_newrelic, :method
|
@@ -101,12 +103,12 @@ DependencyDetection.defer do
|
|
101
103
|
include NewRelic::Agent::MethodTracer
|
102
104
|
|
103
105
|
# Add CAT with callbacks if the request is serial
|
104
|
-
def add_with_newrelic(curl)
|
106
|
+
def add_with_newrelic(curl)
|
105
107
|
if curl.respond_to?(:_nr_serial) && curl._nr_serial
|
106
108
|
hook_pending_request(curl) if NewRelic::Agent::Tracer.tracing_enabled?
|
107
109
|
end
|
108
110
|
|
109
|
-
return add_without_newrelic
|
111
|
+
return add_without_newrelic curl
|
110
112
|
end
|
111
113
|
|
112
114
|
alias_method :add_without_newrelic, :add
|
@@ -125,9 +127,9 @@ DependencyDetection.defer do
|
|
125
127
|
alias_method :perform, :perform_with_newrelic
|
126
128
|
|
127
129
|
|
128
|
-
# Instrument the specified +request+ (a Curl::Easy object)
|
129
|
-
# tracing if it's enabled.
|
130
|
-
def hook_pending_request(request)
|
130
|
+
# Instrument the specified +request+ (a Curl::Easy object)
|
131
|
+
# and set up cross-application tracing if it's enabled.
|
132
|
+
def hook_pending_request(request)
|
131
133
|
wrapped_request, wrapped_response = wrap_request(request)
|
132
134
|
|
133
135
|
segment = NewRelic::Agent::Tracer.start_external_request_segment(
|
@@ -138,9 +140,11 @@ DependencyDetection.defer do
|
|
138
140
|
|
139
141
|
segment.add_request_headers wrapped_request
|
140
142
|
|
143
|
+
# install all callbacks
|
141
144
|
unless request._nr_instrumented
|
142
145
|
install_header_callback(request, wrapped_response)
|
143
146
|
install_completion_callback(request, wrapped_response, segment)
|
147
|
+
install_failure_callback(request, wrapped_response, segment)
|
144
148
|
request._nr_instrumented = true
|
145
149
|
end
|
146
150
|
rescue => err
|
@@ -149,35 +153,37 @@ DependencyDetection.defer do
|
|
149
153
|
|
150
154
|
|
151
155
|
# Create request and response adapter objects for the specified +request+
|
156
|
+
# NOTE: Although strange to wrap request and response at once, it works
|
157
|
+
# because curb's callback mechanism updates the instantiated wrappers
|
158
|
+
# during the life-cycle of external request
|
152
159
|
def wrap_request(request)
|
153
160
|
return NewRelic::Agent::HTTPClients::CurbRequest.new(request),
|
154
161
|
NewRelic::Agent::HTTPClients::CurbResponse.new(request)
|
155
162
|
end
|
156
163
|
|
157
|
-
|
158
|
-
#
|
159
|
-
|
160
|
-
def install_header_callback( request, wrapped_response )
|
164
|
+
# Install a callback that will record the response headers
|
165
|
+
# to enable CAT linking
|
166
|
+
def install_header_callback(request, wrapped_response)
|
161
167
|
original_callback = request.on_header
|
162
168
|
request._nr_original_on_header = original_callback
|
163
169
|
request._nr_header_str = nil
|
164
170
|
request.on_header do |header_data|
|
165
171
|
if original_callback
|
166
|
-
original_callback.call
|
172
|
+
original_callback.call header_data
|
167
173
|
else
|
168
|
-
wrapped_response.append_header_data
|
174
|
+
wrapped_response.append_header_data header_data
|
169
175
|
header_data.length
|
170
176
|
end
|
171
177
|
end
|
172
178
|
end
|
173
179
|
|
174
180
|
# Install a callback that will finish the trace.
|
175
|
-
def install_completion_callback(request, wrapped_response, segment)
|
181
|
+
def install_completion_callback(request, wrapped_response, segment)
|
176
182
|
original_callback = request.on_complete
|
177
183
|
request._nr_original_on_complete = original_callback
|
178
184
|
request.on_complete do |finished_request|
|
179
185
|
begin
|
180
|
-
segment.
|
186
|
+
segment.process_response_headers wrapped_response
|
181
187
|
ensure
|
182
188
|
segment.finish if segment
|
183
189
|
# Make sure the existing completion callback is run, and restore the
|
@@ -188,12 +194,47 @@ DependencyDetection.defer do
|
|
188
194
|
end
|
189
195
|
end
|
190
196
|
|
197
|
+
# Install a callback that will fire on failures
|
198
|
+
# NOTE: on_failure is not always called, so we're not always
|
199
|
+
# unhooking the callback. No harm/no foul in production, but
|
200
|
+
# definitely something to beware of if debugging callback issues
|
201
|
+
# _nr_failure_instrumented exists to prevent infinitely chaining
|
202
|
+
# our on_failure callback hook.
|
203
|
+
def install_failure_callback(request, wrapped_response, segment)
|
204
|
+
return if request._nr_failure_instrumented
|
205
|
+
original_callback = request.on_failure
|
206
|
+
request._nr_original_on_failure = original_callback
|
207
|
+
request.on_failure do |failed_request, error|
|
208
|
+
begin
|
209
|
+
if segment
|
210
|
+
noticible_error = NewRelic::Agent::NoticibleError.new error[0].name, error[-1]
|
211
|
+
segment.notice_error noticible_error
|
212
|
+
end
|
213
|
+
ensure
|
214
|
+
original_callback.call(failed_request, error) if original_callback
|
215
|
+
remove_failure_callback(failed_request)
|
216
|
+
end
|
217
|
+
request._nr_failure_instrumented = true
|
218
|
+
end
|
219
|
+
end
|
220
|
+
|
221
|
+
# on_failure callbacks cannot be removed in the on_complete
|
222
|
+
# callback where this method is invoked because on_complete
|
223
|
+
# fires before the on_failure!
|
191
224
|
def remove_instrumentation_callbacks(request)
|
192
225
|
request.on_complete(&request._nr_original_on_complete)
|
193
226
|
request.on_header(&request._nr_original_on_header)
|
194
227
|
request._nr_instrumented = false
|
195
228
|
end
|
196
229
|
|
230
|
+
# We execute customer's on_failure callback (if any) and
|
231
|
+
# uninstall our hook here since the on_complete callback
|
232
|
+
# fires before the on_failure callback.
|
233
|
+
def remove_failure_callback(request)
|
234
|
+
request.on_failure(&request._nr_original_on_failure)
|
235
|
+
request._nr_failure_instrumented = false
|
236
|
+
end
|
237
|
+
|
197
238
|
private
|
198
239
|
|
199
240
|
def first_request_is_serial?
|
@@ -140,7 +140,9 @@ module NewRelic
|
|
140
140
|
)
|
141
141
|
|
142
142
|
begin
|
143
|
-
|
143
|
+
NewRelic::Agent::Tracer.capture_segment_error segment do
|
144
|
+
self.send("#{method_name}_without_newrelic", *args, &blk)
|
145
|
+
end
|
144
146
|
rescue ::DataObjects::ConnectionError => e
|
145
147
|
raise
|
146
148
|
rescue ::DataObjects::SQLError => e
|
@@ -42,7 +42,7 @@ DependencyDetection.defer do
|
|
42
42
|
end
|
43
43
|
|
44
44
|
def install_excon_instrumentation(excon_version)
|
45
|
-
require 'new_relic/agent/cross_app_tracing'
|
45
|
+
require 'new_relic/agent/distributed_tracing/cross_app_tracing'
|
46
46
|
require 'new_relic/agent/http_clients/excon_wrappers'
|
47
47
|
|
48
48
|
if excon_version >= EXCON_MIDDLEWARE_MIN_VERSION
|
@@ -4,8 +4,9 @@
|
|
4
4
|
|
5
5
|
module ::Excon
|
6
6
|
class Connection
|
7
|
+
# @connection is deprecated in newer excon versions and replaced with @data
|
7
8
|
def newrelic_connection_params
|
8
|
-
(@
|
9
|
+
(@data || @connection)
|
9
10
|
end
|
10
11
|
|
11
12
|
def newrelic_resolved_request_params(request_params)
|
@@ -27,10 +28,12 @@ module ::Excon
|
|
27
28
|
response = nil
|
28
29
|
segment.add_request_headers wrapped_request
|
29
30
|
|
30
|
-
response =
|
31
|
+
response = NewRelic::Agent::Tracer.capture_segment_error segment do
|
32
|
+
request_without_newrelic_trace(resolved_params, &block)
|
33
|
+
end
|
31
34
|
|
32
35
|
wrapped_response = ::NewRelic::Agent::HTTPClients::ExconHTTPResponse.new(response)
|
33
|
-
segment.
|
36
|
+
segment.process_response_headers wrapped_response
|
34
37
|
|
35
38
|
response
|
36
39
|
ensure
|
@@ -49,11 +49,12 @@ module ::Excon
|
|
49
49
|
segment = datum[:connection] && datum[:connection].instance_variable_get(TRACE_DATA_IVAR)
|
50
50
|
if segment
|
51
51
|
begin
|
52
|
+
segment.notice_error(datum[:error]) if datum[:error]
|
52
53
|
datum[:connection].instance_variable_set(TRACE_DATA_IVAR, nil)
|
53
54
|
|
54
55
|
if datum[:response]
|
55
56
|
wrapped_response = ::NewRelic::Agent::HTTPClients::ExconHTTPResponse.new(datum[:response])
|
56
|
-
segment.
|
57
|
+
segment.process_response_headers wrapped_response
|
57
58
|
end
|
58
59
|
ensure
|
59
60
|
segment.finish if segment
|
@@ -14,7 +14,6 @@ module NewRelic
|
|
14
14
|
API_VERSION = 'api.version'.freeze
|
15
15
|
FORMAT_REGEX = /\(\/?\.[\:\w]*\)/.freeze # either :format (< 0.12.0) or .ext (>= 0.12.0)
|
16
16
|
VERSION_REGEX = /:version(\/|$)/.freeze
|
17
|
-
EMPTY_STRING = ''.freeze
|
18
17
|
MIN_VERSION = Gem::Version.new("0.2.0")
|
19
18
|
PIPE_STRING = '|'.freeze
|
20
19
|
|
@@ -33,7 +32,7 @@ module NewRelic
|
|
33
32
|
end
|
34
33
|
|
35
34
|
def name_for_transaction(route, class_name, version)
|
36
|
-
action_name = route.path.sub(FORMAT_REGEX,
|
35
|
+
action_name = route.path.sub(FORMAT_REGEX, NewRelic::EMPTY_STR)
|
37
36
|
method_name = route.request_method
|
38
37
|
version ||= route.version
|
39
38
|
|
@@ -42,7 +41,7 @@ module NewRelic
|
|
42
41
|
version = version.join(PIPE_STRING) if Array === version
|
43
42
|
|
44
43
|
if version
|
45
|
-
action_name = action_name.sub(VERSION_REGEX,
|
44
|
+
action_name = action_name.sub(VERSION_REGEX, NewRelic::EMPTY_STR)
|
46
45
|
"#{class_name}-#{version}#{action_name} (#{method_name})"
|
47
46
|
else
|
48
47
|
"#{class_name}#{action_name} (#{method_name})"
|
@@ -50,12 +49,12 @@ module NewRelic
|
|
50
49
|
end
|
51
50
|
|
52
51
|
def name_for_transaction_deprecated(route, class_name, version)
|
53
|
-
action_name = route.route_path.sub(FORMAT_REGEX,
|
52
|
+
action_name = route.route_path.sub(FORMAT_REGEX, NewRelic::EMPTY_STR)
|
54
53
|
method_name = route.route_method
|
55
54
|
version ||= route.route_version
|
56
55
|
|
57
56
|
if version
|
58
|
-
action_name = action_name.sub(VERSION_REGEX,
|
57
|
+
action_name = action_name.sub(VERSION_REGEX, NewRelic::EMPTY_STR)
|
59
58
|
"#{class_name}-#{version}#{action_name} (#{method_name})"
|
60
59
|
else
|
61
60
|
"#{class_name}#{action_name} (#{method_name})"
|
@@ -118,11 +117,7 @@ DependencyDetection.defer do
|
|
118
117
|
|
119
118
|
# Since 1.2.0, the class `Grape::API` no longer refers to an API instance, rather, what used to be `Grape::API` is `Grape::API::Instance`
|
120
119
|
# https://github.com/ruby-grape/grape/blob/c20a73ac1e3f3ba1082005ed61bf69452373ba87/UPGRADING.md#upgrading-to--120
|
121
|
-
grape_api_class =
|
122
|
-
::Grape::API::Instance
|
123
|
-
else
|
124
|
-
::Grape::API
|
125
|
-
end
|
120
|
+
grape_api_class = defined?(Grape::API::Instance) ? ::Grape::API::Instance : ::Grape::API
|
126
121
|
|
127
122
|
grape_api_class.class_eval do
|
128
123
|
def call_with_new_relic(env)
|
@@ -11,7 +11,7 @@ DependencyDetection.defer do
|
|
11
11
|
|
12
12
|
executes do
|
13
13
|
::NewRelic::Agent.logger.info 'Installing http.rb instrumentation'
|
14
|
-
require 'new_relic/agent/cross_app_tracing'
|
14
|
+
require 'new_relic/agent/distributed_tracing/cross_app_tracing'
|
15
15
|
require 'new_relic/agent/http_clients/http_rb_wrappers'
|
16
16
|
end
|
17
17
|
|
@@ -29,9 +29,12 @@ DependencyDetection.defer do
|
|
29
29
|
|
30
30
|
segment.add_request_headers wrapped_request
|
31
31
|
|
32
|
-
response =
|
32
|
+
response = NewRelic::Agent::Tracer.capture_segment_error segment do
|
33
|
+
perform_without_newrelic_trace(request, options)
|
34
|
+
end
|
35
|
+
|
33
36
|
wrapped_response = ::NewRelic::Agent::HTTPClients::HTTPResponse.new response
|
34
|
-
segment.
|
37
|
+
segment.process_response_headers wrapped_response
|
35
38
|
|
36
39
|
response
|
37
40
|
ensure
|
@@ -20,7 +20,7 @@ DependencyDetection.defer do
|
|
20
20
|
|
21
21
|
executes do
|
22
22
|
::NewRelic::Agent.logger.info 'Installing HTTPClient instrumentation'
|
23
|
-
require 'new_relic/agent/cross_app_tracing'
|
23
|
+
require 'new_relic/agent/distributed_tracing/cross_app_tracing'
|
24
24
|
require 'new_relic/agent/http_clients/httpclient_wrappers'
|
25
25
|
end
|
26
26
|
|
@@ -38,12 +38,14 @@ DependencyDetection.defer do
|
|
38
38
|
response = nil
|
39
39
|
segment.add_request_headers wrapped_request
|
40
40
|
|
41
|
-
|
41
|
+
NewRelic::Agent::Tracer.capture_segment_error segment do
|
42
|
+
do_get_block_without_newrelic(req, proxy, conn, &block)
|
43
|
+
end
|
42
44
|
response = conn.pop
|
43
45
|
conn.push response
|
44
46
|
|
45
47
|
wrapped_response = ::NewRelic::Agent::HTTPClients::HTTPClientResponse.new(response)
|
46
|
-
segment.
|
48
|
+
segment.process_response_headers wrapped_response
|
47
49
|
|
48
50
|
response
|
49
51
|
ensure
|
@@ -46,7 +46,9 @@ module NewRelic
|
|
46
46
|
operation: method_name
|
47
47
|
)
|
48
48
|
begin
|
49
|
-
|
49
|
+
NewRelic::Agent::Tracer.capture_segment_error segment do
|
50
|
+
send method_name_without, *args, &block
|
51
|
+
end
|
50
52
|
ensure
|
51
53
|
if NewRelic::Agent.config[:capture_memcache_keys]
|
52
54
|
segment.notice_nosql_statement "#{method_name} #{args.first.inspect}"
|
@@ -66,7 +66,9 @@ module NewRelic
|
|
66
66
|
::NewRelic::Agent::Instrumentation::Memcache::Dalli.assign_instance_to(segment, self)
|
67
67
|
|
68
68
|
begin
|
69
|
-
|
69
|
+
NewRelic::Agent::Tracer.capture_segment_error segment do
|
70
|
+
send_multiget_without_newrelic_trace(keys)
|
71
|
+
end
|
70
72
|
ensure
|
71
73
|
if ::NewRelic::Agent.config[:capture_memcache_keys]
|
72
74
|
segment.notice_nosql_statement "#{SEND_MULTIGET_METRIC_NAME} #{keys.inspect}"
|
@@ -87,7 +89,9 @@ module NewRelic
|
|
87
89
|
define_method method_name do |*args, &block|
|
88
90
|
segment = NewRelic::Agent::Tracer.start_segment name: "Ruby/Memcached/Dalli/#{method_name}"
|
89
91
|
begin
|
90
|
-
|
92
|
+
NewRelic::Agent::Tracer.capture_segment_error segment do
|
93
|
+
__send__ method_name_without, *args, &block
|
94
|
+
end
|
91
95
|
ensure
|
92
96
|
segment.finish if segment
|
93
97
|
end
|
@@ -87,7 +87,9 @@ DependencyDetection.defer do
|
|
87
87
|
|
88
88
|
begin
|
89
89
|
result = NewRelic::Agent.disable_all_tracing do
|
90
|
-
|
90
|
+
NewRelic::Agent::Tracer.capture_segment_error segment do
|
91
|
+
instrument_without_new_relic_trace(name, payload, &block)
|
92
|
+
end
|
91
93
|
end
|
92
94
|
|
93
95
|
new_relic_notice_statement(segment, payload, name) if segment
|
@@ -109,7 +111,9 @@ DependencyDetection.defer do
|
|
109
111
|
|
110
112
|
begin
|
111
113
|
result = NewRelic::Agent.disable_all_tracing do
|
112
|
-
|
114
|
+
NewRelic::Agent::Tracer.capture_segment_error segment do
|
115
|
+
save_without_new_relic_trace(doc, opts, &block)
|
116
|
+
end
|
113
117
|
end
|
114
118
|
|
115
119
|
new_relic_notice_statement(segment, doc, :save) if segment
|
@@ -131,7 +135,9 @@ DependencyDetection.defer do
|
|
131
135
|
|
132
136
|
begin
|
133
137
|
result = NewRelic::Agent.disable_all_tracing do
|
134
|
-
|
138
|
+
NewRelic::Agent::Tracer.capture_segment_error segment do
|
139
|
+
ensure_index_without_new_relic_trace(spec, opts, &block)
|
140
|
+
end
|
135
141
|
end
|
136
142
|
|
137
143
|
spec = case spec
|
@@ -19,10 +19,23 @@ module NewRelic
|
|
19
19
|
end
|
20
20
|
end
|
21
21
|
|
22
|
+
ERROR_KEYS = %w{ writeErrors writeConcernError writeConcernErrors }.freeze
|
23
|
+
|
24
|
+
def error_present?(event)
|
25
|
+
if reply = event.reply
|
26
|
+
ERROR_KEYS.detect{ |key| reply[key] }
|
27
|
+
end
|
28
|
+
end
|
29
|
+
|
22
30
|
def completed(event)
|
23
31
|
begin
|
24
32
|
return unless NewRelic::Agent::Tracer.tracing_enabled?
|
25
33
|
segment = segments.delete(event.operation_id)
|
34
|
+
if segment && (error_key = error_present?(event))
|
35
|
+
# taking the last error as there can potentially be many
|
36
|
+
attributes = event.reply[error_key][-1]
|
37
|
+
segment.notice_error Mongo::Error.new("%s (%s)" % [attributes["errmsg"], attributes["code"]])
|
38
|
+
end
|
26
39
|
segment.finish if segment
|
27
40
|
rescue Exception => e
|
28
41
|
log_notification_error('completed', e)
|
@@ -11,7 +11,7 @@ DependencyDetection.defer do
|
|
11
11
|
|
12
12
|
executes do
|
13
13
|
::NewRelic::Agent.logger.info 'Installing Net instrumentation'
|
14
|
-
require 'new_relic/agent/cross_app_tracing'
|
14
|
+
require 'new_relic/agent/distributed_tracing/cross_app_tracing'
|
15
15
|
require 'new_relic/agent/http_clients/net_http_wrappers'
|
16
16
|
end
|
17
17
|
|
@@ -33,10 +33,13 @@ DependencyDetection.defer do
|
|
33
33
|
# RUBY-1244 Disable further tracing in request to avoid double
|
34
34
|
# counting if connection wasn't started (which calls request again).
|
35
35
|
NewRelic::Agent.disable_all_tracing do
|
36
|
-
response =
|
36
|
+
response = NewRelic::Agent::Tracer.capture_segment_error segment do
|
37
|
+
request_without_newrelic_trace(request, *args, &block)
|
38
|
+
end
|
37
39
|
end
|
38
40
|
|
39
|
-
|
41
|
+
wrapped_response = NewRelic::Agent::HTTPClients::NetHTTPResponse.new response
|
42
|
+
segment.process_response_headers wrapped_response
|
40
43
|
response
|
41
44
|
ensure
|
42
45
|
segment.finish
|