Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-69815

Bump nodejs from v14.0.0 to current v14.15.0

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      A) Automated, just manually verify that:

      1. npm install
      2. grunt
      3. travis
      4. cibot
      5. phpunit and behat tests

      are all them passing for the target branches (35, 37, 38, 39, 310 & master).

      B) Verify that the NodeJSVersion and NodeJSExactVersion templates in the Docs have been updated to point to this issue and nodejs versions match the work done here (see issue's title).

      C) Quick manual exploratory test
      Optional. Really all these should be covered by behat tests, and some exploratory testing has been already performed so, for your consideration...

      1. Open a terminal and go to your moodle instance's root folder.
      2. Delete node_modules folder if it exists. "rm -rf node_modules"
      3. Run "nvm install". If you don't have nvm, check out the installation instructions.
      4. Confirm that it found the .nvmrc file with version v14.5.0
      5. Confirm that the installation was successful and is now using node v14.5.0
      6. Run "npm install"
      7. Confirm that the installation is successful.
      8. Run "grunt". If grunt is not found, run "npm install -g grunt-cli".
      9. Confirm that grunt completes without any errors.
      10. Check for modified files with "git status"
      11. Confirm that there are no modified files.
      12. With your browser's dev tools/JS console open, visit various pages having javascript modules (e.g. auto-completion, inline-editable, templates, ajax, menu-bar, etc.). The course homepage with editing mode on, the course participants page, and the messaging panel might be good candidates for this test.
      13. Confirm that these are all working ok.
      14. Confirm that there are no javascript console errors as well.
      Show
      A) Automated, just manually verify that: npm install grunt travis cibot phpunit and behat tests are all them passing for the target branches (35, 37, 38, 39, 310 & master). B) Verify that the NodeJSVersion  and NodeJSExactVersion templates in the Docs have been updated to point to this issue and nodejs versions match the work done here (see issue's title). C) Quick manual exploratory test Optional. Really all these should be covered by behat tests, and some exploratory testing has been already performed so, for your consideration... Open a terminal and go to your moodle instance's root folder. Delete node_modules folder if it exists. " rm -rf node_modules " Run " nvm install ". If you don't have nvm, check out the installation instructions . Confirm that it found the .nvmrc file with version v14.5.0 Confirm that the installation was successful and is now using node v14.5.0 Run " npm install " Confirm that the installation is successful. Run " grunt ". If grunt is not found, run " npm install -g grunt-cli ". Confirm that grunt completes without any errors. Check for modified files with " git status " Confirm that there are no modified files. With your browser's dev tools/JS console open, visit various pages having javascript modules (e.g. auto-completion, inline-editable, templates, ajax, menu-bar, etc.). The course homepage with editing mode on, the course participants page, and the messaging panel might be good candidates for this test. Confirm that these are all working ok. Confirm that there are no javascript console errors as well.
    • Affected Branches:
      MOODLE_310_STABLE, MOODLE_35_STABLE, MOODLE_37_STABLE, MOODLE_38_STABLE, MOODLE_39_STABLE, MOODLE_400_STABLE
    • Fixed Branches:
      MOODLE_35_STABLE, MOODLE_37_STABLE, MOODLE_38_STABLE, MOODLE_39_STABLE
    • Pull 3.9 Branch:
    • Pull 3.10 Branch:
      MDL-69815-310
    • Pull Master Branch:
      MDL-69815-master

      Description

      Policy: Keeping us on top of nodejs/npm LTS versions (including jumps to next LTS)

      Since September 2018 (MDL-63346):

      1. This policy will be applied to ALL supported branches, normal and security-only.
      2. This policy won't update any tool per se, unless strictly needed by nodejs/npm own dependencies/changes. It won't include npm audit changes either. Those are handled apart.
      3. TODO (MDLSITE-5536): A job @ CI servers will be, daily, using latest lts/current version to detect if there are npm-shrinkwrap.json changes. It will fail when that happens and inform integrators. Jumps to lts/next will be detected manually,.
      4. When that happens, a new issue, similar to this, will be created and these steps performed in order to get a renewed npm-shrinkwrap.json file:
        • remove node_modules & npm-shrinkwrap.json
        • npm cache clear --force
        • npm install
        • npm shrinkwrap
      5. The resulting branches will be sent to github (or similar), making a peer-review request here so both travis and cibot will verify that all them pass ok.
      6. Changes will be normally integrated and the NodeJSVersion template and NodeJSExactVersion templates will be updated with latest information in the Docs.

      As far as nodejs/npm v14 has been advancing over the last months (since MDL-66109) from v14.0.0 to current v14.12.0:

      https://nodejs.org/en/download/releases/

      This is about to consider to bump to it for all our supported branches (35 and up, right now). This requires:

      • with v14.12.0 installed, regenerate npm-shrinkwrap.json (see instructions above)
      • optionally (not part of the policy): look for deprecated and audit results, trying to update as many as possible.
      • apply all the changes to core.
      • review 3rd part systems: travis, CI servers, docs.moodle.org
      • share about it in all channels as possible

      This check should be performed some weeks before majors (part of the release process), hence creating it now (-6 weeks).

      Ciao

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              dobedobedoh Andrew Lyons
              Reporter:
              stronk7 Eloy Lafuente (stronk7)
              Peer reviewer:
              Simey Lameze Simey Lameze
              Integrator:
              Jun Pataleta Jun Pataleta
              Tester:
              Anna Carissa Sadia Anna Carissa Sadia
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan, Andrew Lyons, Dongsheng Cai, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze, Bas Brands
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                9/Nov/20

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 4 hours, 54 minutes
                  4h 54m