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

adding minutes from 2017-11-15 wg meeting #123

Merged
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
74 changes: 74 additions & 0 deletions wg-meetings/2017-11-15.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,74 @@
# Node.js Foundation Diagnostics Working Group Meeting 2017-11-15

--------------------------------------

## Attendees

Ali Sheikh (@ofrobots)
Kelvin Jin (@kjin)
Dominic Kramer (@dominickramer)
Stephen Belanger (@qard)
Nikhil Komawar (@komawar)
Mark Marron (@mrkmarron)
Mike Kaufman (@mike-kaufman)
Zbyszek Tenerowicz (@naugtur)

## Links

* **Minutes Google Doc**:
- https://docs.google.com/document/d/1cqw00x3kWLYp7jmu6f6D4qmqiPpSwGdJaa3fQ4GHx0U/edit?usp=sharing
- google doc was innacessible for part of the meeting, so notes here are incomplete

* **Previous Minutes**:
- [md doc](https://github.com/nodejs/diagnostics/blob/master/wg-meetings/2017-11-01.md)
- [google doc]( https://docs.google.com/document/d/1YPysjpqnMdiXwAubWvN3AoMRGBEzk9cq2eLluj9hP6w/edit?usp=sharing)

## Agenda
- **Time zone shift:**
- In US, we shifted from Daylight Savings Time to Standard Time (moving from UTC-7 to UTC-8). This meeting remained at UTC 20:00 & did not shift to account for DST.
- Node.js calendar was not updated
- not clear we have quorum, but agreed to discuss what we could
- **Actions:**
- [x] fix calendar

- **Async-context formalization and diagnostics support** [#107](https://github.com/nodejs/diagnostics/issues/107)
- Interest for others to get involved in this, in particular @kjin.
- @mrkmarron, @mike-kaufman have a very rough draft of a document to "democratize" what's in the OOPSLA paper
- Plan was to get this concrete enough such that we can get useful feedback.
- Form will be a mark-down document in a PR that ppl can provide feedback on.
- Currently, document is on an microsoft-internal GH repo. Will try to find a way to either make public or grant access to others in the interim, in particular @kjin
- Need to move this forward, will shoot to have something public by 11/29 meeting
- Good discussion on "async context"s
- @ofrobots - we need to communicate why "async context" is a relevant problem. Thinks long stacks are a good motivating example.
- general high-level discussion about the DAG model about async context.
- analogy of thread/thread join brought up
- **ACTIONS**:
- [ ] @mike-kaufman - figure out how to grant access to early draft of md
- [ ] @mrkmarron, @mike-kaufman - get draft polished up enough to make public & make a PR to diagnostics GH repro

- **Integrate C++ AsyncHooks Embedder API with native abstraction** [#13254](https://github.com/nodejs/node/issues/13254)
- no update

- **Summary of Diag WG Discussion - 2017/10/05 (Node Interactive)** [#106](https://github.com/nodejs/diagnostics/issues/106)
- **ACTIONS:**
- [ ] @ofrobots to follow up on Chrome's plans for making their `trace-events` library public

- **Node.js Collaboration Summit diagnostics discussion minutes** [#95](https://github.com/nodejs/diagnostics/issues/95)
- no update

- **requesting comments on diagnostics-howtos project** [#92](https://github.com/nodejs/diagnostics/issues/92)
- @naugtur opened a PR on this

- **\[trace_event\] tracking issue** [#84](https://github.com/nodejs/diagnostics/issues/84)
- no updates

- **\[async_hooks\] tracking issue** [#29](https://github.com/nodejs/diagnostics/issues/29)
- no updates

- **@mike-kaufman can't tag issues**
- issue #120 opened to track

- **Documentation Discussion during diagnostics WG meeting** [163](https://github.com/nodejs/community-committee/issues/163)
- issue was opened based on discussion from 11/01/2017 meeting
- probably need to get more concrete w/ what needs to happen there, or just close it