Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simple console tracing #1042

Merged
merged 14 commits into from Aug 24, 2022
Merged

Simple console tracing #1042

merged 14 commits into from Aug 24, 2022

Conversation

fetsorn
Copy link
Contributor

@fetsorn fetsorn commented Jul 13, 2022

Resolves #1001

Features

  • added an OpenTelemetry exporter for console.log - @fetsorn/opentelemetry-console-exporter
    • 1st column: unique and random icons represent trace hierarchy
    • 2nd column: duration of each span, currently in milliseconds
    • 3rd column: function names or detailed trace annotations
    • an object with arguments and output of each method, along with annotations
    • works both in browser and node
  • added detailed trace annotations to some of the main functions - invoke, load, resolveUri
  • added tracing level and detalization to client configuration
  • added missing tracing controls to React provider
  • updated openTelemetry dependencies

Changes to review

  • client configuration now has another two optional parameters. Haven't merged them into a TracingConfig type because it would be harder to set their values.
    • TracingLevel: only functions with equal or higher priority are traced
    • TracingDetailed: boolean switch between concise and detailed console export
  • removed inspect from the tracing library
  • added an external dependency to the exporter
  • tracing methods (traceMethod, startSpan, setAttribute etc.) now take an optional parameter tracingLevel that raises priority over the default TracingLevel.Low.
export enum TracingLevel {
 Low = 0,
 Medium,
 High,
}
There's screenshots under the thing Screen Shot 2022-07-13 at 11 23 17 Screen Shot 2022-07-14 at 14 50 12 Screen Shot 2022-07-13 at 11 22 46 Screen Shot 2022-07-13 at 11 23 38

@fetsorn fetsorn changed the base branch from origin to origin-dev July 13, 2022 07:18
@fetsorn fetsorn marked this pull request as ready for review July 13, 2022 07:20
@fetsorn fetsorn changed the title #1001 tracing #1001 Simple console tracing Jul 14, 2022
@fetsorn fetsorn changed the title #1001 Simple console tracing Simple console tracing Jul 14, 2022
dOrgJelli
dOrgJelli previously approved these changes Aug 24, 2022
@dOrgJelli dOrgJelli merged commit 0a1e6e8 into origin-dev Aug 24, 2022
@dOrgJelli dOrgJelli deleted the 1001-tracing branch April 10, 2023 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants