Skip to content

Release proposal: v1.6.3 #1252

Description

@rvagg

I'd really like to see some work on timers get merged, specificially #1152 (@Fishrock123) and maybe also #1231 (@silverwind). Anecdotally I'm hearing challenges with io.js adoption because of "timer bugs".

A release tomorrow is doable.

  • [7dd5e824be] - assert: simplify logic of testing buffer equality (Alex Yursha) #1171
  • [a2ea16838f] - debugger: don't spawn child process in remote mode (Jackson Tian) #1282
  • [2752da4b64] - deps: make node-gyp work with io.js (cjihrig) #990
  • [f166cdecf1] - deps: upgrade npm to 2.7.4 (Forrest L Norvell)
  • [318d9d8fd7] - deps: upgrade v8 to 4.1.0.27 (Ben Noordhuis) #1289
  • [269e46be37] - deps: make node-gyp work with io.js (cjihrig) #990
  • [b542fb94a4] - deps: upgrade npm to 2.7.3 (Forrest L Norvell) #1219
  • [73de13511d] - doc: add WG links in WORKING_GROUPS.md & fix nits (Farrin Reid) #1113
  • [19641b17be] - doc: decouple sidebar scrolling (Roman Reiss) #1274
  • [dbccf8d3ed] - doc: fix spelling error in feature flags (Phillip Lamplugh) #1286
  • [5e609e9324] - _Revert_ "doc: clarify real name requirement" (Jeremiah Senkpiel) #1276
  • [45814216ee] - doc: fix format docs discrepancy (Brendan Ashworth) #1255
  • [4e9bf93e9c] - doc: clarify real name requirement (Roman Reiss) #1250
  • [e84dd5f651] - doc: document repl on-demand module loading (Roman Reiss) #1249
  • [c9207f7fc2] - fs: fix corruption in writeFile and writeFileSync (Olov Lassus) #1063
  • [2db758c562] - iojs: introduce internal modules (Vladimir Kurchatkin) #848
  • [36f017afaf] - js2c: fix module id generation on windows (Ben Noordhuis) #1281
  • [1832743e18] - lib: add missing new for errors lib/*.js (Mayhem) #1246
  • [ea37ac04f4] - src: ignore ENOTCONN on shutdown race with child (Ben Noordhuis) #1214
  • [f06b16f2e9] - src: fix minor memleak in preload-modules (Ali Ijaz Sheikh) #1265
  • [2903410aa8] - src: don't lazy-load timer globals (Ben Noordhuis) #1280
  • [2e5b87a147] - src: remove unnecessary environment lookups (Ben Noordhuis) #1238
  • [7e88a9322c] - src: make accessors immune to context confusion (Ben Noordhuis) #1238
  • [c8fa8ccdbc] - streams: use strict on _stream_wrap (Brendan Ashworth) #1279
  • [8a945814dd] - string_decoder: optimize write() (Brian White) #1209
  • [8d1c87ea0a] - test: fix race in parallel/test-vm-debug-context (Ben Noordhuis) #1294
  • [955c1508da] - test: reduce sequential/test-fs-watch flakiness (Roman Reiss) #1275
  • [77c2da10fd] - timers: make Timer.close idempotent (Petka Antonov) #1288
  • [776b73b243] - timers: cleanup interval handling (Jeremiah Senkpiel) #1272
  • [caf0b36de3] - timers: assure setTimeout callback only runs once (Roman Reiss) #1231
  • [2ccc8f3970] - tls_wrap: fix this incredibly stupid leak (Fedor Indutny) #1244
  • [e74b5d278c] - tls_wrap: fix BIO leak on SSL error (Fedor Indutny) #1244
  • [ba93c583bc] - win,node-gyp: optionally allow node.exe/iojs.exe to be renamed (Bert Belder) #1266
  • [08acf1352c] - win,node-gyp: make delay-load hook optional (Bert Belder) #1266
  • [3d46fefe0c] - win,node-gyp: allow node.exe/iojs.exe to be renamed (Bert Belder) #1251

Activity

  1. piscisaureus commented on Mar 25, 2015

    @piscisaureus
    Contributor

    I'd like to get #1251 in, because the .exe naming issue causes so many issues that it forced me to actually fix it myself :). But don't hold up the release for it.

  2. added
    metaIssues and PRs related to the general management of the project.
    on Mar 25, 2015
  3. silverwind commented on Mar 25, 2015

    @silverwind
    Contributor

    I'll try to finish #1231 today, just have to make sure it causes no side effects.

  4. alanpurple commented on Mar 25, 2015

    @alanpurple

    I'm wating for this, thanks again 👍 :)

  5. Fishrock123 commented on Mar 25, 2015

    @Fishrock123
    Contributor

    #1152 will likely not be ready for this.

  6. silverwind commented on Mar 25, 2015

    @silverwind
    Contributor

    #1231 is ready to merge from my point. Edit: It has landed.

  7. Fishrock123 commented on Mar 26, 2015

    @Fishrock123
    Contributor

    #1075 (comment) -- we should get this out tomorrow #1244 is making a huge difference for the leak.

  8. Fishrock123 commented on Mar 26, 2015

    @Fishrock123
    Contributor

    Blocked by the entire windows test suite failing due to a bug in internal modules: #848 (comment)

  9. rvagg commented on Mar 26, 2015

    @rvagg
    MemberAuthor

    Ouch, we should revert that unless a fix gets in within the next day or so.

    /cc @vkurchatkin

  10. Fishrock123 commented on Mar 27, 2015

    @Fishrock123
    Contributor
  11. Fishrock123 commented on Mar 27, 2015

    @Fishrock123
    Contributor

    7 specific tests have been timing out for the last several runs on win2012r2 only. Investigating.
    Update: maybe just jenkins. I had an out of hdd/ram failure earlier.

    https://gist.github.com/Fishrock123/be271ab4b9bbf393b23a

  12. Fishrock123 commented on Mar 27, 2015

    @Fishrock123
    Contributor

    #1152 is broken and should not go in. #1152 (comment)

    @rvagg I think we should be mostly ok for a release if jenkins looks fine after a reboot.

    Maybe just get #1285 (npm@2.7.4 in? I'll probably merge before the release if it's not in yet anyways.)

  13. rvagg commented on Mar 28, 2015

    @rvagg
    MemberAuthor
  14. Fishrock123 commented on Mar 28, 2015

    @Fishrock123
    Contributor

    Same 6 (7 - 1) sort of timeouts we've been seeing on win2012r2 recently..

  15. rvagg commented on Mar 29, 2015

    @rvagg
    MemberAuthor

    https://jenkins-iojs.nodesource.com/job/iojs+any-pr+multi/396/ let's see if a reboot of those machines does anything, it has fixed windows timeouts in the past

  16. 14 remaining items

  17. bnoordhuis commented on Mar 30, 2015

    @bnoordhuis
    Member

    ARMv8 is currently compiled without openssl (i.e. crypto and tls) support pending an upgrade to 1.0.2 and not all tests handle that as graciously as they should. I also suspect that there are a few libuv bugs that need to be addressed in order to fix the failing child process tests.

  18. Fishrock123 commented on Mar 30, 2015

    @Fishrock123
    Contributor

    ARMv* and *-wheezy are currently not considered blockers, afaik.

  19. jbergstroem commented on Mar 30, 2015

    @jbergstroem
    Member

    @bnoordhuis I can't reproduce any failing tests with --without-ssl.

  20. rvagg commented on Mar 31, 2015

    @rvagg
    MemberAuthor

    nightly again because I had armv6 instead of armv6l in ARCH

  21. rvagg commented on Mar 31, 2015

    @rvagg
    MemberAuthor

    This is new, on OSX:

    === release test-http-destroyed-socket-write2 ===
    Path: parallel/test-http-destroyed-socket-write2
    assert.js:88
      throw new assert.AssertionError({
            ^
    AssertionError: false == true
        at test (/Users/iojs/build/workspace/iojs+any-pr+multi/nodes/osx1010/test/parallel/test-http-destroyed-socket-write2.js:89:5)
        at Immediate.write [as _onImmediate] (/Users/iojs/build/workspace/iojs+any-pr+multi/nodes/osx1010/test/parallel/test-http-destroyed-socket-write2.js:29:7)
        at processImmediate [as _immediateCallback] (timers.js:361:17)
    Command: out/Release/iojs /Users/iojs/build/workspace/iojs+any-pr+multi/nodes/osx1010/test/parallel/test-http-destroyed-socket-write2.js
    

    Running again to see if it's persistent: https://jenkins-iojs.nodesource.com/job/iojs+any-pr+multi/417/

  22. rvagg commented on Mar 31, 2015

    @rvagg
    MemberAuthor

    on OSX, second attempt:

    === release test-fs-readfilesync-pipe-large ===
    Path: parallel/test-fs-readfilesync-pipe-large
    assert.js:88
      throw new assert.AssertionError({
            ^
    AssertionError: it reads the file and outputs it
        at /Users/iojs/build/workspace/iojs+any-pr+multi/nodes/osx1010/test/parallel/test-fs-readfilesync-pipe-large.js:31:3
        at ChildProcess.exithandler (child_process.js:707:7)
        at emitTwo (events.js:87:13)
        at ChildProcess.emit (events.js:169:7)
        at maybeClose (child_process.js:984:16)
        at Process.ChildProcess._handle.onexit (child_process.js:1057:5)
    Command: out/Release/iojs /Users/iojs/build/workspace/iojs+any-pr+multi/nodes/osx1010/test/parallel/test-fs-readfilesync-pipe-large.js
    

    Not great but different errors on both runs so I'm not going to let this hold up release.

  23. rvagg commented on Mar 31, 2015

    @rvagg
    MemberAuthor
  24. rvagg commented on Mar 31, 2015

    @rvagg
    MemberAuthor

    done & landed @ https://iojs.org/dist/latest/

    @iojs/website no need to act on this release, I hooked up the build-on-release thing we were testing so it's already rebuilt for you

    @iojs/collaborators:

    Thanks for the work in this release, unfortunately there's a bunch of intermittent failures in our CI that keep on popping up, and we have the strange TIMEOUTs on Windows that have yet to be figured out (happen when running in Jenkins). It'd be great if we could have a bit of focus on sorting out the tests across all platforms so the CI is more useful and informative than it is now. I know a lot of you don't have access to it yet and that's mostly my fault, I'll try and get emails out to everyone this week so you can start using it.

    I'm personally a little concerned about the leak report (#1075) still not being sorted out, and the serious timers problems (#1151 and #1264) causing a lack of trust in our ability to deliver a stable and quality platform because I've heard feedback about both of these things from outsiders and users alike. We need to all try hard to make sure that detractors have no technical reasons to detract and those on the fence don't have technical reasons not to start deploying io.js.

  25. silverwind commented on Mar 31, 2015

    @silverwind
    Contributor

    My RPi2 should arrive tomorrow, and then I'll look into adding platform-specific test timeout values for ARMv6 and ARMv7 so we get meaningful results on these CI runs.

  26. indutny commented on Mar 31, 2015

    @indutny
    Member

    @rvagg we resolved most of the aspects of the leak. It is only a matter of reproducing it/gathering more information until we'll fix it once and for all.

  27. Fishrock123 commented on Mar 31, 2015

    @Fishrock123
    Contributor

    Most of #1075 was fixed in 2db758c...2ccc8f3 (see: #1075 (comment))

    Timer issues are probably just going to be solved by a re-write from @bnoordhuis. (all of which (and more) also exist in node 0.12, and some possibly worse now.)

  28. rvagg commented on Apr 1, 2015

    @rvagg
    MemberAuthor

    Timer issues are probably just going to be solved by a re-write from @bnoordhuis.

    Yes, I've heard this but it makes me afraid that we're punting on fixing stuff with the dream of a rewrite, I know Ben's probably the most productive of us here but let's not defer simpler fixes with the hope that a rewrite will drop some day soon because we may end up waiting for a while because we have to add together coding + reviewing + testing which adds up for a large amount of code (think how long the Promises PR took to get in!)

    (all of which (and more) also exist in node 0.12, and some possibly worse now.)

    Yes, but primarily we're competing against 0.10 right now, it's where most significant deployments are stuck while there's a sense of limbo around 0.12 and io.js and 0.12 has a natural momentum in most people's minds (as strange as that may sound to us here!) so we just need to do 10x better.

  29. Fishrock123 commented on Apr 1, 2015

    @Fishrock123
    Contributor

    but let's not defer simpler fixes with the hope that a rewrite will drop some day soon

    if only timers were that simple :P

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    metaIssues and PRs related to the general management of the project.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions