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

Use heimdalljs for structured instrumentation & structured logging #6086

Merged
merged 10 commits into from Aug 10, 2016

Conversation

hjdivad
Copy link
Contributor

@hjdivad hjdivad commented Jul 16, 2016

@stefanpenner
Copy link
Contributor

stefanpenner commented Jul 18, 2016

  • remove broccoli-viz dep
  • serialize data automatically based on w/e heimdall collected
  • restore FS monitor

@stefanpenner
Copy link
Contributor

stefanpenner commented Aug 2, 2016

supersedes #6135 & #6141

@hjdivad hjdivad force-pushed the use-structured-logging branch 3 times, most recently from ae42331 to 47d2682 Compare August 4, 2016 22:44
@hjdivad hjdivad changed the title [WIP] Upgrade sane-watcher to use heimdall [WIP] Use heimdalljs for structured instrumentation & structured logging Aug 4, 2016
@homu
Copy link
Contributor

homu commented Aug 8, 2016

☔ The latest upstream changes (presumably #6103) made this pull request unmergeable. Please resolve the merge conflicts.

@hjdivad hjdivad force-pushed the use-structured-logging branch 2 times, most recently from f28fa16 to 28dcb10 Compare August 9, 2016 19:14
@hjdivad hjdivad changed the title [WIP] Use heimdalljs for structured instrumentation & structured logging Use heimdalljs for structured instrumentation & structured logging Aug 9, 2016
@stefanpenner
Copy link
Contributor

@hjdivad looks like we are green.

@hjdivad
Copy link
Contributor Author

hjdivad commented Aug 10, 2016

@stefanpenner see any lingering issues or seem good to merge?

@stefanpenner stefanpenner merged commit 2bce175 into master Aug 10, 2016
@stefanpenner stefanpenner deleted the use-structured-logging branch August 10, 2016 01:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants