Skip to content

Commit

Permalink
Merge branch 'epic/telemetry' of github.com:pullflow/vscode-pullflow …
Browse files Browse the repository at this point in the history
…into feature/quickpick-telemetry
  • Loading branch information
srzainab committed Jan 15, 2024
2 parents 9fde50f + 921ae27 commit 088f2b6
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .vscode/launch.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
// Hover to view descriptions of existing attributes.
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
{
"version": "1.2.2",
"version": "1.2.3",
"configurations": [
{
"name": "Run Extension",
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"publisher": "Pullflow",
"displayName": "Pullflow",
"description": "Code review collaboration across GitHub, Slack, and VS Code.",
"version": "1.2.2",
"version": "1.2.3",
"preview": true,
"license": "MIT",
"engines": {
Expand Down
1 change: 1 addition & 0 deletions src/utils/appConfig.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,5 +13,6 @@ export const AppConfig = {
pullflow: {
baseUrl: process.env['PULLFLOW_APP_URL'] as string,
graphqlUrl: `${process.env['PULLFLOW_APP_URL']}/api/graphql`,
telemetryUrl: process.env['PULLFLOW_TELEMETRY_URL'] as string,
},
}
3 changes: 2 additions & 1 deletion src/utils/trace.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ import { SemanticResourceAttributes } from '@opentelemetry/semantic-conventions'
import { ExtensionContext, extensions } from 'vscode'
import { TraceAttributes } from './types'
import { Store } from './store'
import { AppConfig } from './appConfig'

const extensionInfo = extensions.getExtension('Pullflow.pullflow')?.packageJSON

Expand All @@ -26,7 +27,7 @@ export class Trace {
})

const exporter = new OTLPTraceExporter({
url: 'http://localhost:4318/v1/traces',
url: AppConfig.pullflow.telemetryUrl + '/v1/traces',
compression: 'gzip' as any,
})
this.provider.addSpanProcessor(new BatchSpanProcessor(exporter))
Expand Down

0 comments on commit 088f2b6

Please sign in to comment.