-
Notifications
You must be signed in to change notification settings - Fork 18
update the worker to node 24#1357
Merged
Merged
Conversation
@github-project-automation
github-project-automation
Bot
moved this to New Issues
in Core
Apr 7, 2026
josephjclark
commented
Apr 10, 2026
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
merged commit Apr 10, 2026
ec33504
into
release/next
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
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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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