add USDT DTrace support via node-dtrace-provider #48

Closed
bcantrill opened this Issue Oct 22, 2012 · 1 comment

Comments

Projects
None yet
2 participants
Contributor

bcantrill commented Oct 22, 2012

No description provided.

@bcantrill bcantrill added a commit to bcantrill/node-bunyan that referenced this issue Oct 31, 2012

@bcantrill bcantrill issue #48: add USDT DTrace support via node-dtrace-provider af5a3d4

@trentm trentm added a commit that referenced this issue Nov 1, 2012

@trentm trentm issue #48: fix a couple issues with dtrace support
- before this, if there were no non-raw streams on the logger the
  `str` for the dtrace probe.fire would not be created
- ensure that with no non-raw streams we only render a record's
  JSON string if the dtrace probe is enabled
2438b0b

@trentm trentm added a commit that referenced this issue Nov 1, 2012

@trentm trentm issue #48: make dtrace-provider presence optional (disabled dtrace pr…
…obes)

dtrace-provider is still in *dependencies* to strongly encourage
installing it. However I want bunyan.js to be useable for basic
usage standalone.
2c4d471

@trentm trentm added a commit that referenced this issue Nov 1, 2012

@trentm trentm issue #48: changelog, add Bryan to AUTHORS, basic dtrace test suite
The test suite for this is a bit finnicky because root rights are
required to run dtrace. By default we run the dtrace.test.js file
with 'sudo'. Use this to skip the dtrace tests:
    SKIP_DTRACE=1 make test

Note taht 'make test09' is busted because dtrace-provider seems
busted with node 0.9.1 at least.
8732764
Owner

trentm commented Nov 2, 2012

This is in bunyan 0.15.0

trentm closed this Nov 2, 2012

trentm removed this from the sdc7 milestone Nov 8, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment