circleci-orb icon indicating copy to clipboard operation
circleci-orb copied to clipboard

CircleCI test-deploy workflow uses end-of-life Node.js 16

Open MikeMcC399 opened this issue 1 year ago • 7 comments

The CircleCI workflow .circleci/test-deploy.yml is configured to use the Docker container:

cypress/browsers:node-16.18.1-chrome-109.0.5414.74-1-ff-109.0-edge-109.0.1518.52-1

whereas the executor has been updated to node-version: "20.6.0".

Node.js 16 already reached end-of-life on Sep 11, 2023 and is no longer supported.

(Runs are logged to https://app.circleci.com/pipelines/github/cypress-io/circleci-orb.)

Suggestion

Use Node.js 20 for the Docker containers, for example, using the latest available cypress/browsers:node-20.x Docker image from :

cypress/browsers:node-20.9.0-chrome-118.0.5993.88-1-ff-118.0.2-edge-118.0.2088.46-1

MikeMcC399 avatar Dec 07 '23 16:12 MikeMcC399

It seems that https://app.circleci.com/pipelines/github/cypress-io/circleci-orb is not currently being triggered.

MikeMcC399 avatar Dec 10 '23 12:12 MikeMcC399

@MikeMcC399 Yes, I did notice this. I briefly looked at it but have no idea why.

  • Our circleci-orb CI runs are never running. https://github.com/cypress-io/circleci-orb/pull/449
  • Nothing has run since a commit 2 months ago that didn’t touch the Circle config: https://app.circleci.com/pipelines/github/cypress-io/circleci-orb
  • The CircleCI/GitHub webhook delivery looks fine: https://github.com/cypress-io/circleci-orb/settings/hooks/62000836?tab=deliveries

jennifer-shehane avatar Dec 13 '23 16:12 jennifer-shehane

@MikeMcC399 Figured it out with some help! https://github.com/cypress-io/circleci-orb/pull/449#issuecomment-1852262850

jennifer-shehane avatar Dec 13 '23 16:12 jennifer-shehane

@jennifer-shehane

Thank you for taking a look at this. I can't be much help as I'm inexperienced with CircleCI and I don't have visibility into the CircleCI settings https://github.com/cypress-io/circleci-orb/settings either.

MikeMcC399 avatar Dec 13 '23 16:12 MikeMcC399

Since https://github.com/cypress-io/circleci-orb/blob/aa59916d55c3e6b3bc9d6aba3b2d98971e7e9e6e/examples/angular-app/package.json#L27 is currently using Angular 15, this restricts Node.js to ^14.20.0 || ^16.13.0 || ^18.10.0 according to Angular > Version compatibility.

Angular 15 reaches end-of-life on May 18, 2024 so at some stage a migration to at least Angular 16 (supports Node.js ^16.14.0 || ^18.10.0) or preferably Angular 17 (supports Node.js ^18.13.0 || ^20.9.0) should be planned.

  • Edit: This comment has been transferred to https://github.com/cypress-io/circleci-orb/issues/470

MikeMcC399 avatar Dec 16 '23 14:12 MikeMcC399

Did https://github.com/cypress-io/circleci-orb/pull/449 close this issue @jennifer-shehane ?

jordanpowell88 avatar Dec 21 '23 21:12 jordanpowell88

@jordanpowell88

Did https://github.com/cypress-io/circleci-orb/pull/449 close this issue?

  • #449 is unrelated

https://github.com/cypress-io/circleci-orb/blob/master/.circleci/test-deploy.yml still contains the following usage of the now unsupported Node.js 16 version:

https://github.com/cypress-io/circleci-orb/blob/6fe76b416c2ef17123114400b3a8f337a1d4b452/.circleci/test-deploy.yml#L24-L26

https://github.com/cypress-io/circleci-orb/blob/6fe76b416c2ef17123114400b3a8f337a1d4b452/.circleci/test-deploy.yml#L35-L37

https://github.com/cypress-io/circleci-orb/blob/6fe76b416c2ef17123114400b3a8f337a1d4b452/.circleci/test-deploy.yml#L46-L48

This issue therefore remains unaddressed.

MikeMcC399 avatar Dec 22 '23 06:12 MikeMcC399