Identify uninstrumented services

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>
jaegertracing · Nov 11, 2020 · ab0fa50 · ab0fa50
1 parent b07aab8
commit ab0fa50
Show file tree

Hide file tree

Showing 3 changed files with 34 additions and 0 deletions.
diff --git a/packages/jaeger-ui/src/components/TracePage/TraceTimelineViewer/SpanBarRow.tsx b/packages/jaeger-ui/src/components/TracePage/TraceTimelineViewer/SpanBarRow.tsx
@@ -48,6 +48,12 @@ type SpanBarRowProps = {
         serviceName: string;
       }
     | TNil;
+  noInstrumentedServer?:
+    | {
+        color: string;
+        serviceName: string;
+      }
+    | TNil;
   showErrorIcon: boolean;
   getViewedBounds: ViewedBoundsFunctionType;
   traceStartTime: number;
@@ -87,6 +93,7 @@ export default class SpanBarRow extends React.PureComponent<SpanBarRowProps> {
       isMatchingFilter,
       numTicks,
       rpc,
+      noInstrumentedServer,
       showErrorIcon,
       getViewedBounds,
       traceStartTime,
@@ -151,6 +158,16 @@ export default class SpanBarRow extends React.PureComponent<SpanBarRowProps> {
                     {rpc.serviceName}
                   </span>
                 )}
+                {noInstrumentedServer && (
+                  <span>
+                    <IoArrowRightA />{' '}
+                    <i
+                      className="SpanBarRow--rpcColorMarker"
+                      style={{ background: noInstrumentedServer.color }}
+                    />
+                    {noInstrumentedServer.serviceName}
+                  </span>
+                )}
               </span>
               <small className="endpoint-name">{rpc ? rpc.operationName : operationName}</small>
             </a>

diff --git a/packages/jaeger-ui/src/components/TracePage/TraceTimelineViewer/VirtualizedTraceView.tsx b/packages/jaeger-ui/src/components/TracePage/TraceTimelineViewer/VirtualizedTraceView.tsx
@@ -31,6 +31,7 @@ import {
   createViewedBoundsFunc,
   findServerChildSpan,
   isErrorSpan,
+  isKindClient,
   spanContainsErredSpan,
   ViewedBoundsFunctionType,
 } from './utils';
@@ -44,6 +45,7 @@ import TTraceTimeline from '../../../types/TTraceTimeline';
 
 import './VirtualizedTraceView.css';
 import updateUiFind from '../../../utils/update-ui-find';
+import { PEER_SERVICE } from '../../../constants/tag-keys';
 
 type RowState = {
   isDetail: boolean;
@@ -360,6 +362,17 @@ export class VirtualizedTraceViewImpl extends React.Component<VirtualizedTraceVi
         };
       }
     }
+    const peerServiceKV = span.tags.find(kv => kv.key === PEER_SERVICE);
+    // Leaf, kind == client and has peer.service tag, is likely a client span that does a request
+    // to an uninstrumented/external service
+    let noInstrumentedServer = null;
+    if (!span.hasChildren && peerServiceKV && isKindClient(span)) {
+      noInstrumentedServer = {
+        serviceName: peerServiceKV.value,
+        color: colorGenerator.getColorByKey(peerServiceKV.value),
+      };
+    }
+
     return (
       <div className="VirtualizedTraceView--row" key={key} style={style} {...attrs}>
         <SpanBarRow
@@ -373,6 +386,7 @@ export class VirtualizedTraceViewImpl extends React.Component<VirtualizedTraceVi
           onDetailToggled={detailToggle}
           onChildrenToggled={childrenToggle}
           rpc={rpc}
+          noInstrumentedServer={noInstrumentedServer}
           showErrorIcon={showErrorIcon}
           getViewedBounds={this.getViewedBounds()}
           traceStartTime={trace.startTime}

diff --git a/packages/jaeger-ui/src/components/TracePage/TraceTimelineViewer/utils.tsx b/packages/jaeger-ui/src/components/TracePage/TraceTimelineViewer/utils.tsx
@@ -117,4 +117,7 @@ export function findServerChildSpan(spans: Span[]) {
   return null;
 }
 
+export const isKindClient = (span: Span) =>
+  span.tags.find(({ key, value }) => key === 'span.kind' && value === 'client');
+
 export { formatDuration } from '../../../utils/date';