Skip to content

Commit e375043

Browse files
authored
Merge pull request #18476 from mozilla/update-ci-image
revert(node): Revert to node 20.11.1 (WIP)
2 parents 7b78484 + 3103a5d commit e375043

13 files changed

Lines changed: 35 additions & 35 deletions

File tree

.nvmrc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
22.14
1+
20.11.1

_dev/docker/ci-lockfile-generator/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
# By running a yarn install inside the same base container that we use in our CI, we can
66
# ensure the same checksums are created correctly.
77
#
8-
FROM cimg/node:22.14
8+
FROM cimg/node:20.11
99

1010
COPY . .
1111
RUN sudo yarn cache clear --all

_dev/docker/ci/Dockerfile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
# Runs tests and common CI operations. Needs minimal install. Assumes
66
# workspace will be restored into the project folder.
7-
FROM cimg/node:22.14 AS test-runner
7+
FROM cimg/node:20.11 AS test-runner
88
RUN sudo apt-get update && sudo apt-get install -y \
99
python3-venv
1010
WORKDIR /home/circleci
@@ -29,15 +29,15 @@ RUN yarn install --immutable;
2929
# Acts as an intermediate stage for adding the firefox install. Note,
3030
# that a yarn install must happen first to ensure the correct version
3131
# of firefox is installed. Also note that the functional-test-runner
32-
# must based on cimg/node:22.14-browsers, which is why this stage
32+
# must based on cimg/node:20.11-browsers, which is why this stage
3333
# is necessary.
3434
FROM builder AS playwright-install
3535
RUN npx playwright install --with-deps firefox chromium webkit;
3636

3737

3838
# Runs functional tests in our CI. Needs minimal install. Assumes
3939
# workspace will be restored into the project folder.
40-
FROM cimg/node:22.14-browsers AS functional-test-runner
40+
FROM cimg/node:20.11-browsers AS functional-test-runner
4141
WORKDIR /home/circleci
4242
COPY --chown=circleci:circleci --from=playwright-install /home/circleci/.cache/ms-playwright .cache/ms-playwright/
4343
COPY --chown=circleci:circleci project project

_dev/docker/mono/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# Multi stage docker build for fxa-mono repo. Note that stages should be cached due to docker layer caching being
22
# turned on in CI.
33

4-
FROM node:22.14.0-bullseye-slim AS fxa-base
4+
FROM node:20.11.1-bullseye-slim AS fxa-base
55
RUN set -x \
66
&& addgroup --gid 10001 app \
77
&& adduser --disabled-password \

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@
136136
"winston": "^3.17.0"
137137
},
138138
"engines": {
139-
"node": "^22.14.0"
139+
"node": "^20.11.1"
140140
},
141141
"husky": {
142142
"hooks": {

packages/functional-tests/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
"lint": "eslint ."
1414
},
1515
"devDependencies": {
16-
"@playwright/test": "^1.43.1",
16+
"@playwright/test": "1.44.1",
1717
"@types/eslint": "^8",
1818
"@types/pdf-parse": "^1.1.4",
1919
"@types/upng-js": "^2",

packages/fxa-auth-server/test/local/email-cloud-tasks.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ const { EmailCloudTaskManager } = proxyquire('../../lib/email-cloud-tasks', {
2929
},
3030
},
3131
});
32-
const { EmailTypes } = require('@fxa/shared/cloud-tasks');
32+
import { EmailTypes } from '@fxa/shared/cloud-tasks';
3333

3434
describe('EmailCloudTaskManager', () => {
3535
const mockConfig = {

packages/fxa-auth-server/test/local/routes/cloud-scheduler.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@
44

55
const sinon = require('sinon');
66
const assert = { ...sinon.assert, ...require('chai').assert };
7-
const { ReasonForDeletion } = require('@fxa/shared/cloud-tasks');
8-
const proxyquire = require('proxyquire');
7+
import { ReasonForDeletion } from '@fxa/shared/cloud-tasks';
8+
import proxyquire from 'proxyquire';
99

1010
describe('CloudSchedulerHandler', function () {
1111
this.timeout(10000);

packages/fxa-auth-server/test/local/routes/recovery-phone.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
* License, v. 2.0. If a copy of the MPL was not distributed with this
33
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */
44

5-
const { AccountEventsManager } = require('../../../lib/account-events');
5+
import { AccountEventsManager } from '../../../lib/account-events';
66

77
const chai = require('chai');
88
const chaiAsPromised = require('chai-as-promised');
@@ -12,13 +12,13 @@ const sinon = require('sinon');
1212
const assert = { ...sinon.assert, ...chai.assert };
1313
const mocks = require('../../mocks');
1414
const { recoveryPhoneRoutes } = require('../../../lib/routes/recovery-phone');
15-
const {
15+
import {
1616
RecoveryNumberNotSupportedError,
1717
SmsSendRateLimitExceededError,
1818
RecoveryPhoneService,
1919
RecoveryNumberRemoveMissingBackupCodes,
20-
RecoveryPhoneRegistrationLimitReached
21-
} = require('@fxa/accounts/recovery-phone');
20+
RecoveryPhoneRegistrationLimitReached,
21+
} from '@fxa/accounts/recovery-phone';
2222

2323
const { getRoute } = require('../../routes_helpers');
2424
const { mockRequest } = require('../../mocks');

packages/fxa-auth-server/test/local/routes/subscriptions/mozilla.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
/* This Source Code Form is subject to the terms of the Mozilla Public
22
* License, v. 2.0. If a copy of the MPL was not distributed with this
33
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */
4-
const { MozillaSubscriptionTypes } = require('fxa-shared/subscriptions/types');
4+
import { MozillaSubscriptionTypes } from 'fxa-shared/subscriptions/types';
55

6-
const { ERRNO } = require('../../../../lib/error');
6+
import { ERRNO } from '../../../../lib/error';
77

88
('use strict');
99

0 commit comments

Comments
 (0)