Skip to content

Commit 88b0067

Browse files
joshgavMylesBorins
authored andcommitted
doc: add CTC meeting minutes 2016-09-21
PR-URL: #8727 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]>
1 parent c543f4a commit 88b0067

File tree

1 file changed

+218
-0
lines changed

1 file changed

+218
-0
lines changed

doc/ctc-meetings/2016-09-21.md

+218
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,218 @@
1+
# Node Foundation CTC Meeting 2016-09-21
2+
3+
## Links
4+
5+
* **Audio Recording**: TBP
6+
* **GitHub Issue**: [#8656](https://github.com/nodejs/node/issues/8656)
7+
* **Minutes Google Doc**: <https://docs.google.com/document/d/1-PS20PGCJQIi4_quBHqVgrQNJeJP10NH21wUelcMhfw>
8+
* _Previous Minutes Google Doc_: <https://docs.google.com/document/d/1kW9wLdfojpIGbu3fQMrtr7S_RJFq8CWupoKnmtWw17Q>
9+
10+
## Present
11+
12+
* Anna Henningsen @addaleax (CTC)
13+
* Сковорода Никита Андреевич @ChALkeR (CTC)
14+
* Evan Lucas @evanlucas (CTC)
15+
* Jeremiah Senkpiel @Fishrock123 (CTC)
16+
* Fedor Indutny @indutny (CTC)
17+
* James M Snell @jasnell (CTC)
18+
* Michael Dawson @mhdawson (CTC)
19+
* Brian White @mscdex (CTC)
20+
* Ali Ijaz Sheikh @ofrobots (CTC)
21+
* Myles Borins @TheAlphaNerd (CTC)
22+
* Rich Trott @Trott (CTC)
23+
* Steven R Loomis @srl295 (observer/IBM/ICU)
24+
* Seth Thompson (observer/Google)
25+
* Jenn Turner (observer/Node Foundation)
26+
* Josh Gavant (observer/Microsoft)
27+
28+
29+
## Standup
30+
31+
* Anna Henningsen @addaleax (CTC)
32+
* Node Interactive EU
33+
* In particular, Code & Learn
34+
* Сковорода Никита Андреевич @ChALkeR (CTC)
35+
* Filed some PRs regarding doc linting
36+
* Some issue comments and PR reviews as usual
37+
* Evan Lucas @evanlucas (CTC)
38+
* Working on an EP for exposing is* methods from core
39+
* Jeremiah Senkpiel @Fishrock123 (CTC)
40+
* async_hooks review
41+
* v6.6.0 release
42+
* Fedor Indutny @indutny (CTC)
43+
* PRs, regular/security issues, Node.js Interactive!
44+
* James M Snell @jasnell (CTC)
45+
* Node Interactive EU/Collab summit
46+
* Reviewing PRs (primarily from the code n learn)
47+
* Preparing v7.0.0-beta1 (built now!)
48+
* Preparing for TC-39 meeting next week
49+
* Continued work on HTTP/2 plan
50+
* Michael Dawson @mhdawson (CTC)
51+
* Node Interactive EU/Collaborators summit
52+
* New PPC machines into CI
53+
* Addition of LinuxOne and AIX to nightlies
54+
* build WG meeting, benchmarking WG meeting
55+
* Investigating an AIX issues
56+
* Some work on ABI stable API
57+
* Catching up on issues
58+
* Brian White @mscdex (CTC)
59+
* Performance testing for various things
60+
* Submitted PRs to improve setImmediate() and setTimeout()/setInterval() performance
61+
* Submitted alternative PR to fix the malloc(0) compatibility issue introduced in node v6.6.0
62+
* Worked on other various potential optimizations in core
63+
* Reviewed PRs, commented on issues
64+
* Ali Ijaz Sheikh @ofrobots (CTC)
65+
* Mostly busy with internal stuff; work with some inspector PRs; finishing up V8 guide.
66+
* Myles Borins @TheAlphaNerd (CTC)
67+
* Node Interactive EU / Collab Summit
68+
* Work on new tap reporter
69+
* Reviewing @ofrobots work on V8 guide
70+
* LTS backporting work
71+
* Rich Trott @Trott (CTC)
72+
* Onboarded @ak239 and @eugeneo
73+
* assorted test, tools, and doc PRs
74+
* https://github.com/nodejs/node/pull/8572 (Malloc and Calloc with size 0) to fix https://github.com/nodejs/node/issues/8571 (node::PBKDF2() Out of Memory) to land imminently, please take a look
75+
* looking at the recent uptick in test failures, trying to figure out ways to fix it systemically and avoid it from recurring
76+
* CTC meeting time proposal: more work than you think, trust me
77+
* Seth Thompson @s3ththompson (observer/Google)
78+
* continued v8_inspector work
79+
80+
## Agenda
81+
82+
* deps: update V8 to 5.4 [node#8317](https://github.com/nodejs/node/pull/8317)
83+
* Scheduling Meetings [ctc#14](https://github.com/nodejs/CTC/issues/14)
84+
* Decide on what problem points for ES Modules we care about the most. [ctc#15](https://github.com/nodejs/CTC/issues/15)
85+
86+
## Previous Meeting Review
87+
88+
* AsyncWrap public API proposal [node-eps#18](https://github.com/nodejs/node-eps/pull/18)
89+
* Merge.
90+
91+
92+
## Minutes
93+
94+
@trott: removed the async_hooks EP from the agenda, if anybody has concerns, the place for that would be on Github, or if necessary the item can be put on the agenda again.
95+
96+
### deps: update V8 to 5.4 [#8317](https://github.com/nodejs/node/pull/8317)
97+
98+
@ofrobots: V8 5.4 is part of Chrome beta. Since we’re trying to release v7.x with V8 5.4 I think it would be best to land this in master now rather than close to release.
99+
100+
Traditionally we’ve only landed V8 once it goes stable, but in this particular instance I think it’s important cause of 7.0 and this beta lining up so closely.
101+
102+
@Fishrock123: Original reason why we only landed stable was because in io.js we were landing beta, but the V8 team warned us that things could change, and they did change.
103+
104+
Did these things change on the V8 end? How is this different than yesterday?
105+
106+
@ofrobots: Google is saying we shouldn’t *ship* a V8 beta. It’s in master so it can undergo testing, but we wouldn’t ship it. Things could change but it doesn’t matter cause it’s just in master, we’re not shipping it.
107+
108+
@Fishrock123: Do RC’s count as shipping this? Some people will expect not much breakage between RC and release.
109+
110+
@jasnell: We’ve had issues where things are broken in master and people complain cause it’s broken in master even if it doesn’t go to releases.
111+
112+
@ofrobots: Purpose of an RC is to say this is what we think we’re going to ship, intention is not to change. With V8, we’ll also have the intention not to change. If a bug is found which breaks the web and a change needs to happen then it would happen.
113+
114+
@Seth: We treat beta as an RC in that respect.
115+
116+
@myles: Re what @jasnell said about things on master, it’s about things landing on master which are experience breaking. We’ve already been testing with 5.4. We should just be ready to revert if there’s an experience-breaking problem.
117+
118+
@Fishrock123: Good arguments, but we’re changing what we’re doing where we took a particular stance because people had issues with it.
119+
120+
@jasnell: We took that stance before we had the improved release model, with LTS. Risk was much greater then than now.
121+
122+
@trott: Any other voices?
123+
124+
@jasnell: @Fishrock123 is this something you absolutely want to see?
125+
126+
@Fishrock123: If there’s a vote I’ll just abstain.
127+
128+
@trott: Seems we have consensus and don’t need to vote.
129+
130+
@Fishrock123: Might as well try it again and see if it works.
131+
132+
@jasnell: If anything comes up, V8 team please give us a heads up as early as possible.
133+
134+
@ofrobots: one change that’s going to happen, is that since Node is going to use V8 5.4, we’re going to get v8_inspector into v8 and out of separate deps folder. This will make build and maintenance easier for Node.
135+
136+
**Next steps**:
137+
138+
- Do this and keep a careful eye.
139+
- Evaluate after this cycle and decide what to do in the future.
140+
141+
---
142+
143+
### Scheduling Meetings [ctc#14](https://github.com/nodejs/CTC/issues/14)
144+
145+
@trott: Raising for CTC awareness.
146+
147+
---
148+
149+
### Decide on what problem points for ES Modules we care about the most. [ctc#15](https://github.com/nodejs/CTC/issues/15)
150+
151+
@Fishrock123: Majority of problems we can see so far are in EP #002, also in CTC#15.
152+
153+
Bradley’s been at this for almost a year trying to get better support for us and nothing’s really moved from the spec end. I kind of doubt it will but would like to push in that direction.
154+
155+
Would be good to find out what the CTC and Collaborators care about. Focus on CTC cause it’s easier to get overall decision with a smaller group. There’s a lot to talk about.
156+
157+
Hard to know what opinion I should have.
158+
159+
@trott: Is the question: Assuming the spec does not change, what should we do?
160+
161+
@Fishrock123: If we want to push the spec to changing for better compatibility, what do we want to push? Pushing on everything would be basically rewriting the entire spec…
162+
163+
@jasnell: Another aspect of this, conversations with TC39 at multiple levels, not just technical but also process. Try to help TC39 understand and care about the issues. If there’s something to change, it’s something they should care about also.
164+
165+
@Fishrock123: Still comes back to this list of issues. What do we want to tackle the most?
166+
167+
TC39 meeting next week, a couple people are going to that, context around what CTC cares about would be helpful.
168+
169+
@jasnell: TC39 is 9/27-29. We should have a call just for CTC to make sure all are on same page beforehand.
170+
171+
@Fishrock123: While I agree with that, we still need a consensus from the CTC on where to focus. Do we care about loading modules, REPLs, wrapping modules.
172+
173+
@trott: Anything in particular?
174+
175+
@jasnell: All are breaking issues.
176+
177+
@Fishrock123: We don’t break modules, but if a module moves to ESM they could break downstream modules.
178+
179+
@joshgav: In particular cannot change modules after import, breaks mocks and APMs.
180+
181+
@Fishrock123: For example, `graceful-fs` might not work at all.
182+
183+
@jasnell: What are our goals to come away from TC39 meeting?
184+
185+
@Fishrock123:
186+
* TC39 showing some amount of care for our problems.
187+
* Communication of those problems and some cooperation.
188+
* Look at delaying shipping dates so we don’t have a browser vendor war which exacerbates our problems.
189+
190+
@trott: Encourage everyone to read the issue carefully, ask a bunch of questions, know that date of TC39 meeting is coming and we need one clear set of priorities. Everyone that can have an informed opinion on this should educate themselves and participate.
191+
192+
@jasnell: Talked with some of the members of TC39. What they articulated is that not to expect sweeping changes be made just because we ask for them.
193+
194+
Instead describe why the current spec is a problem, and if there are aspects which are incorrect or can be improved, it’s on us to make those changes.
195+
196+
We can’t expect them to care. Ideally they’d care already. But we have to have a proposal - this is why the module spec is broken, in terms of “we can’t implement it and this is why.”
197+
198+
@thealphanerd: Conversations on Twitter, biggest sentiment from TC39 is that there are individuals who do care and want to work with us on it. If we approach with that narrative we’ll likely have more progress.
199+
200+
@jasnell: Definite sentiment that we should be able to make this work, just a matter of making the right case.
201+
202+
---
203+
204+
## Q/A on public channels
205+
206+
None.
207+
208+
209+
## Upcoming Meetings
210+
211+
* CTC: 2016-09-28
212+
* TSC: 2016-09-22
213+
* Build:
214+
* Diagnostics: 2016-09-28
215+
* Benchmarking: 2016-09-21
216+
* LTS:
217+
* Post-Mortem:
218+
* API:

0 commit comments

Comments
 (0)