Skip to content

update the worker to node 24#1357

Merged
josephjclark merged 2 commits intorelease/nextfrom
worker-node-24
Apr 10, 2026
Merged

update the worker to node 24#1357
josephjclark merged 2 commits intorelease/nextfrom
worker-node-24

Conversation

@josephjclark
Copy link
Copy Markdown
Collaborator

Didn't do this on the prior release - update the worker image to use node 24

We've tested the worker with the new loader on node 22 and it works great

This updates the worker to node 24

I need to test the image locally before putting onto staging because of permissions changes in node 24. It's probably fine - I don't think anything affects us? - but needs a sanity test before we do the update

@github-project-automation github-project-automation bot moved this to New Issues in Core Apr 7, 2026
@josephjclark josephjclark changed the base branch from main to release/next April 9, 2026 15:23
@josephjclark
Copy link
Copy Markdown
Collaborator Author

Working great:

RTE Memory limit: 500mb
RTE Timeout: 300s
RTE Payload limit: 10mb
VER Versions:
    ▸ node.js                    24.14.1
    ▸ worker                     1.23.3
    ▸ @openfn/language-common    3.3.1
R/T Compiling all workflow steps...
R/T Finished in 0.003s
R/T Compiled Transform data in 0.016s
R/T Workflow compilation complete in 0.063s
R/T Executing 162a8899-9fa3-40fd-9f8d-2c68229740fa
R/T Starting execution from step 6ef27774-3035-447c-8024-f18eaa62c830
R/T Starting step Transform data
R/T [linker] loading module @openfn/language-common
R/T [linker] Loading module @openfn/language-common from /root/.openfn/worker/repo/node_modules/@openfn/language-common_3.3.1/dist/index.js
R/T Resolved adaptor @openfn/language-common to version 3.3.1
R/T Executing expression (1 operations)
R/T Starting operation 1
R/T Operation 1 complete in 0ms
R/T Expression complete!
R/T Transform data completed in 238ms
R/T Cleaning up state. Removing keys: configuration
R/T Step memory usage: [step 49mb] [system 237mb]
R/T Run complete with status: success

@josephjclark josephjclark marked this pull request as ready for review April 10, 2026 08:40
@josephjclark josephjclark merged commit ec33504 into release/next Apr 10, 2026
1 of 2 checks passed
@github-project-automation github-project-automation bot moved this from New Issues to Done in Core Apr 10, 2026
@josephjclark josephjclark deleted the worker-node-24 branch April 10, 2026 08:41
josephjclark added a commit that referenced this pull request Apr 10, 2026
* update the worker to node 24 (#1357)

* update the worker to node 24

* update docker docs

* changeset

* Worker: fix an issue with batch logging (#1353)

* set min-release-age

* versions

* debugging flaky test

* fix an issue where the batch is never clear

* fix a timing issue when sending batch events

Big help from claude

* logging

* add a bunch of more controlled unit tests

* test on interrupt

* update and fix tests

I think this this fixes the actual issue - I just want more good focused tests now

* tidy logging

* more tests

* changeset

* types

* remove only

* run tests in serial

* worker: tweak event processor and be sure to reset timeout on batch

* remove comment

* remove more comments

* feat: update usage of getActiveProject to getCheckedOutProject (#1360)

* feat: update getActiveProject to getCheckedOutProject

* feat: back to active project

* chore: rename getActiveProject to getTrackedProject

* feat: deploy should use getTrackedProject

* chore: getCheckedout return undefined like getActiveprojects

* fix: types

* feat: update removed & renamed workflows on checkout (#1358)

* tests: remove unwanted fields

* feat: use currentProject

* tests: remove fields

* versions

---------

Co-authored-by: Farhan Y. <yahyafarhan48@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants