|
| 1 | +# Node Foundation CTC Meeting 2016-09-28 |
| 2 | + |
| 3 | +## Links |
| 4 | + |
| 5 | +* **Audio Recording**: TBP |
| 6 | +* **GitHub Issue**: [#8802](https://github.com/nodejs/node/issues/8802) |
| 7 | +* **Minutes Google Doc**: <https://docs.google.com/document/d/1ZsLMM29guCBPGh7KhuhZEq6YQCKybiTfTUIXajTBalM/> |
| 8 | +* _Previous Minutes Google Doc_: <https://docs.google.com/document/d/1-PS20PGCJQIi4_quBHqVgrQNJeJP10NH21wUelcMhfw> |
| 9 | + |
| 10 | + |
| 11 | +## Present |
| 12 | + |
| 13 | +* Anna Henningsen @addaleax (CTC) |
| 14 | +* Сковорода Никита Андреевич @ChALkeR (CTC) |
| 15 | +* Colin Ihrig @cjihrig (CTC) |
| 16 | +* Evan Lucas @evanlucas (CTC) |
| 17 | +* Jeremiah Senkpiel @Fishrock123 (CTC) |
| 18 | +* Tracy Hinds @hackygolucky (observer/Node.js Foundation) |
| 19 | +* Josh Gavant @joshgav (observer/Microsoft) |
| 20 | +* Michael Dawson @mhdawson (CTC) |
| 21 | +* Ali Ijaz Sheikh @ofrobots (CTC) |
| 22 | +* Jenn Turner @renrutnnej (observer/Node.js Foundation) |
| 23 | +* Rod Vagg @rvagg (CTC) |
| 24 | +* Seth Thompson @s3ththompson (observer/Google) |
| 25 | +* Myles Borins @TheAlphaNerd (CTC) |
| 26 | +* Trevor Norris @trevnorris (CTC) |
| 27 | +* Rich Trott @Trott (CTC) |
| 28 | + |
| 29 | + |
| 30 | +## Standup |
| 31 | + |
| 32 | +* Anna Henningsen @addaleax (CTC) |
| 33 | + * The usual, issues and PR reviews |
| 34 | +* Сковорода Никита Андреевич @ChALkeR (CTC) |
| 35 | + * Some issue and PR comments and reviews as usual. |
| 36 | + * Some more work on docs linting. |
| 37 | +* Colin Ihrig @cjihrig (CTC) |
| 38 | + * Reviewing issues and PRs. |
| 39 | + * Evan Lucas @evanlucas (CTC) |
| 40 | + * v6.7.0 release |
| 41 | + * More work on types eps |
| 42 | +* Jeremiah Senkpiel @Fishrock123 (CTC) |
| 43 | + * Issue / PR Review … general stuff |
| 44 | + * Working towards ES Modules prototype implementations with Chris Dickinson |
| 45 | +* Tracy Hinds @hackygolucky (observer/Node.js Foundation) |
| 46 | + * getting the Outreachy info on website |
| 47 | +* Josh Gavant @joshgav (observer/Microsoft) |
| 48 | + * helping bring in some new MS contributors |
| 49 | + * scheduled diag meeting for next week |
| 50 | +* Michael Dawson @mhdawson (CTC) |
| 51 | + * Finishing off PPC migration |
| 52 | + * Fixing AIX issues when building from node-private |
| 53 | + * Some work on ABI-stable node |
| 54 | + * Misc PR review/lands |
| 55 | + * Keeping up with issues |
| 56 | + * Post-mortem nodereport review |
| 57 | +* Brian White @mscdex (CTC) |
| 58 | + * Worked on various performance improvements in node core |
| 59 | + * Reviewed PRs, commented on issues |
| 60 | +* Ali Ijaz Sheikh @ofrobots (CTC) |
| 61 | + * Looking at node+V8 (5.5) integration build failures that seem related to recent parser improvements |
| 62 | + * Investigating performance with the new interpreter |
| 63 | + * Working with @matthewloring on FFI |
| 64 | +* Jenn Turner @renrutnnej (observer/Node.js Foundation) |
| 65 | + * No update, just observing |
| 66 | +* Rod Vagg @rvagg (CTC) |
| 67 | + * Security releases, supposed to be on vacation |
| 68 | +* Seth Thompson @s3ththompson (observer/Google) |
| 69 | + * async/await landed in V8 Tip of Tree. on track to ship with V8 5.5 |
| 70 | + * expect a doc from V8 language team on promise hook API to allow microtask introspection in the near future |
| 71 | +* Myles Borins @TheAlphaNerd (CTC) |
| 72 | + * issue / pr review |
| 73 | + * helping with security release |
| 74 | + * backporting inspector |
| 75 | + * auditing v4 backlog |
| 76 | + * really have to get to that tap reporter |
| 77 | + * coming up with outreachy mentor project |
| 78 | +* Trevor Norris @trevnorris (CTC) |
| 79 | + * AsyncHooks |
| 80 | +* Rich Trott @Trott (CTC) |
| 81 | + * mentoring more first-time contributors (via Node Todo) |
| 82 | + * doc, test PRs |
| 83 | + * ramping up a tiny bit on Build WG stuff, but just a tiny bit |
| 84 | + |
| 85 | + |
| 86 | +## Agenda |
| 87 | + |
| 88 | +Extracted from **ctc-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting. |
| 89 | + |
| 90 | +### nodejs/CTC |
| 91 | + |
| 92 | +* Scheduling Meetings [#14](https://github.com/nodejs/CTC/issues/14) |
| 93 | + |
| 94 | +### nodejs/node |
| 95 | + |
| 96 | +* meta: update NODE_MODULE_VERSION to 51 [#8808](https://github.com/nodejs/node/pull/8808) |
| 97 | +* General v7.0.0 / v6 LTS Planning / Discussion |
| 98 | + |
| 99 | + |
| 100 | +## Previous Meeting Review |
| 101 | + |
| 102 | +* deps: update V8 to 5.4 [#8317](https://github.com/nodejs/node/pull/8317) |
| 103 | +* Scheduling Meetings [#14](https://github.com/nodejs/CTC/issues/14) |
| 104 | +* Decide on what problem points for ES Modules we care about the most. [#15](https://github.com/nodejs/CTC/issues/15) |
| 105 | + |
| 106 | + |
| 107 | +## Minutes |
| 108 | + |
| 109 | +### Scheduling Meetings [ctc#14](https://github.com/nodejs/CTC/issues/14) |
| 110 | + |
| 111 | +@trott: This is a status report. Initial proposal was to dive in and start rotating meetings. Some were on board, some were concerned. Nikita started Google spreadsheet to figure things out. |
| 112 | + |
| 113 | +One proposal is to move back one hour (12pm Pacific) which would be a mild improvement for Ben and Nikita. |
| 114 | + |
| 115 | +Input received from NA and EU but not Asia and Australia. Once we have that information we can figure out what might work. |
| 116 | + |
| 117 | +--- |
| 118 | + |
| 119 | +### meta: update NODE_MODULE_VERSION to 51 [#8808](https://github.com/nodejs/node/pull/8808) |
| 120 | + |
| 121 | +`process.versions.modules` == 48 for v6.7.0. |
| 122 | + |
| 123 | +Set in build script, we bump this number for each semver-minor. We’d update to 49, but Electron has been bumping in between, so we need to go to 51. |
| 124 | + |
| 125 | +@thealphanerd: Had a way to do this in the past, but never landed a version of V8 on master [before a release]. So those using master cannot rely on this check. |
| 126 | + |
| 127 | +Proposal is to add this to master now and v7.x when released. But should we wait to bump till the actual release? |
| 128 | + |
| 129 | +@rvagg: If you’re using master, it’s been a bit “buyer beware” in the past. |
| 130 | + |
| 131 | +@thealphanerd: Node-pre-gyp uses the module version number to determine whether to pull the pre-built binary or to rebuild. So that’s causing problems. |
| 132 | + |
| 133 | +@ofrobots: Is there a disadvantage to doing this now? |
| 134 | + |
| 135 | +@rvagg: Doesn’t seem to be. |
| 136 | + |
| 137 | +@joshgav: Any concern that we’d have to bump again at v7.x release? |
| 138 | + |
| 139 | +@rvagg: We’ll just bump again. |
| 140 | + |
| 141 | +@addaleax: We should watch what Electron is doing cause they pull in every V8 version. |
| 142 | + |
| 143 | +@trevnorris: Could it happen that newer version of V8 has a lower module version number? |
| 144 | + |
| 145 | +@trott: if module version mapped to V8 version we could always be in sync. |
| 146 | + |
| 147 | +@ofrobots: Problem is that ABI is more than just V8. Also, we’re moving to a VM-neutral API/ABI in the future and that will remove the relationship to a V8 version. |
| 148 | + |
| 149 | +@rvagg: We should coordinate with Electron and draw from the same pool of numbers. |
| 150 | + |
| 151 | +@ofrobots: A good point for a bump would be when we bring a new V8 into master. |
| 152 | + |
| 153 | +@rvagg: This would make testing those nightlies easier. |
| 154 | + |
| 155 | +@trevnorris: Sounds to me that we can’t map reliably map a version of Node to a version of V8. So pulling from the same pool as Electron might be misleading to developers. If the number in Electron doesn’t match a Node version there would be a conflict. |
| 156 | + |
| 157 | +@rvagg: People are tracking which module versions map to what, so they could follow this too. |
| 158 | + |
| 159 | +@trevnorris: Maybe we can give Electron the minor numbers. |
| 160 | + |
| 161 | +@rvagg: NW.js also had a similar issue. |
| 162 | + |
| 163 | +@thealphanerd: Electron bumped to 5.1 in an ABI-breaking way, so we have two versions of Node ABIs out there, cause they needed to stay closer to Chromium. |
| 164 | + |
| 165 | +@rvagg: They don’t need to keep up with the latest version of V8. |
| 166 | + |
| 167 | +@thealphanerd: Let’s talk offline. |
| 168 | + |
| 169 | +@rvagg: Back to GitHub? Or do we need to decide now? |
| 170 | + |
| 171 | +@thealphanerd : PR has a lot of LGTMs, would like to see this land today or tomorrow so we can unbreak master. |
| 172 | + |
| 173 | +@ofrobots: Two points - one, what to do now; two, what to do going forward? |
| 174 | + |
| 175 | +Ali and Myles will work on a policy going forward. |
| 176 | + |
| 177 | +@rvagg: Might belong in LTS repo as we’ve been doing a lot of versioning stuff there. |
| 178 | + |
| 179 | +**Next steps**: |
| 180 | + |
| 181 | +* If there are objections raise them in the issue, otherwise ready to merge. |
| 182 | + |
| 183 | +--- |
| 184 | + |
| 185 | +### General v7.0.0 / v6 LTS Planning / Discussion |
| 186 | + |
| 187 | +@Fishrock123: Make sure all are in the loop. |
| 188 | + |
| 189 | +Throw v0.10 in too since it’s end of life at end of October. |
| 190 | + |
| 191 | +@rvagg: LTS map says *first* of October. Some people expect that cause the docs say that. |
| 192 | + |
| 193 | +@Fishrock123: We discussed keeping it alive till end of December like v0.12, cause that’s when OpenSSL is EOL’ed. |
| 194 | + |
| 195 | +@mhdawson: If the doc says Oct 1 what’s the downside to sticking with that? |
| 196 | + |
| 197 | +@rvagg: We may have communicated Oct 31 through some channels, so some people may expect that. |
| 198 | + |
| 199 | +Or perhaps when v7.x is first released. |
| 200 | + |
| 201 | +Having said that, it’s been >2 years, so people have had time to migrate. |
| 202 | + |
| 203 | +@rvagg: Originally 0.12 was slated for EOL in April 2017, we moved it back because of the OpenSSL issue. |
| 204 | + |
| 205 | +@Fishrock123: Official LTS policy is target date is when the next release/LTS is cut. That’s usually midway through month. |
| 206 | + |
| 207 | +@rvagg: Push to LTS WG to resolve ASAP. |
| 208 | + |
| 209 | +@rvagg: James pushing another beta later this week or early next week. |
| 210 | + |
| 211 | +@Fishrock123: v7.x is now on semver-major freeze. |
| 212 | + |
| 213 | +@rvagg: There are some semver-major commits in master which aren’t in v7 release. |
| 214 | + |
| 215 | +@Fishrock123: Might still need to update? Might have been left out by James intentionally? |
| 216 | + |
| 217 | +@thealphanerd: Not a ton of semver-major things on master. There are the V8 upgrades (patches), and a move of a method to fs/internal. |
| 218 | + |
| 219 | +Big one is npm@4 coming through the pipeline 1-2 weeks before Node release, should we include that. |
| 220 | + |
| 221 | +@rvagg: npm@3 had problems originally so we delayed. Should we do the same for npm@4? |
| 222 | + |
| 223 | +@addaleax: I’d feel comfortable with landing it. Kat said they aren’t concerned if 4 is included now or not. |
| 224 | + |
| 225 | +@Fishrock123: Things which are deprecated in v4 will still be deprecated (not removed) in v5, so we could bump all the way to v5 in a later release. |
| 226 | + |
| 227 | +@rvagg: We don’t have to synchronize all these dates to one, we can be flexible if needed. |
| 228 | + |
| 229 | +@thealphanerd: If we have a date other than late October it might be a good idea to offer a date. |
| 230 | + |
| 231 | +@ofrobots: Tentatively Oct 18 is the target stable date for V8 5.4. As tentative as usual, not clear till the last moment. Low chance that V8 will be moving a lot around Oct 18. Haven’t seen this date slip by more than 1-2 days. Very low chance that V8 will destablizie us. |
| 232 | + |
| 233 | +@thealphanerd: Oct 25 as a tentative date for v7.x? |
| 234 | + |
| 235 | +@rvagg: Ali and Seth, what’s the risk of setting that date now? |
| 236 | + |
| 237 | +@ofrobots: Close to Oct 18 I can highlight any potential risk. |
| 238 | + |
| 239 | +@rvagg: Let’s say that - 25th is tentative date, we’ll communicate if there’s any change. Any objections? (No.) |
| 240 | + |
| 241 | +That will also be the day we switch v6 to LTS. |
| 242 | + |
| 243 | +@thealphanerd: Doing release of v6 LTS earlier might be helpful so we have that out of the way for potential v7 issues. |
| 244 | + |
| 245 | +@rvagg: Discussion on this will move to LTS WG. Join the LTS WG on Monday to discuss. |
| 246 | + |
| 247 | +@thealphanerd: Could use someone to be responsible for v6 LTS, please volunteer. |
| 248 | + |
| 249 | +@rvagg: It’s been helpful to have a single person for v4 LTS, but we need to find a model that scales in the future. |
| 250 | + |
| 251 | +@Fishrock123: Would be helpful to schedule LTS a week earlier to avoid problems. |
| 252 | + |
| 253 | +--- |
| 254 | + |
| 255 | +### Supported platforms proposal from Build WG [#488](https://github.com/nodejs/build/issues/488) |
| 256 | + |
| 257 | +Current proposal: <https://github.com/nodejs/build/issues/488#issuecomment-250155697> |
| 258 | + |
| 259 | +@trott should make a CTC agenda item next week? |
| 260 | + |
| 261 | +@rvagg: Give input on that issue before it comes to CTC. Build WG must review and sign off on as well. |
| 262 | + |
| 263 | +@rvagg: Some discussion about tiers, this affects OS vendors. |
| 264 | + |
| 265 | +--- |
| 266 | + |
| 267 | +### Other |
| 268 | + |
| 269 | +@thealphanerd: Node.js is going to be working with Outreachy project to help people from underrepresented groups get involved. |
| 270 | + |
| 271 | +We need projects for these people to work on in 3 months. If you can think of good parts of the project to assign… would love to hear your suggestions. |
| 272 | + |
| 273 | +@rvagg: GitHub thread? |
| 274 | + |
| 275 | +@hackygolucky: I’ll create a new one and ping @nodejs/collaborators. |
| 276 | + |
| 277 | +@rvagg: Are we getting a satisfactory response on the call for mentors? |
| 278 | + |
| 279 | +@hackygolucky: 5 primary mentors and a number of supplementals. 4 sponsors, which means we can accept 4 mentees. |
| 280 | + |
| 281 | +@rvagg: If someone wants to be a supplemental is that still open? |
| 282 | + |
| 283 | +@hackygolucky: Thread is still open: https://github.com/nodejs/education/issues/7 |
| 284 | + |
| 285 | +--- |
| 286 | + |
| 287 | +## Q/A on public channels |
| 288 | + |
| 289 | +None. |
| 290 | + |
| 291 | +--- |
| 292 | + |
| 293 | +## Upcoming Meetings |
| 294 | + |
| 295 | +* CTC: 2016-10-05 |
| 296 | +* TSC: 2016-10-06 |
| 297 | +* Build: 2016-10-11 |
| 298 | +* Diagnostics: 2016-10-05, 12pm Pacific |
| 299 | +* Benchmarking: |
| 300 | +* LTS: 2016-10-03 |
| 301 | +* Post-Mortem: |
| 302 | +* API: |
0 commit comments