Skip to content

Allow NodeJS cmd line options to be passed to NodeRED process#572

Merged
hardillb merged 6 commits intomainfrom
heap-size
Feb 13, 2026
Merged

Allow NodeJS cmd line options to be passed to NodeRED process#572
hardillb merged 6 commits intomainfrom
heap-size

Conversation

@hardillb
Copy link
Contributor

fixes #571

Description

Allows the Node-RED max_old_space_size to be set rather than hard coded

Adds

  • heapSize as device.yml config option
  • -s or --heap-size cmd line arguments

Related Issue(s)

#571

Checklist

  • I have read the contribution guidelines
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
    • Upgrade instructions
    • Configuration details
    • Concepts
  • Changes flowforge.yml?
    • Issue/PR raised on FlowFuse/helm to update ConfigMap Template
    • Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production
  • Link to Changelog Entry PR, or note why one is not needed.

Labels

  • Includes a DB migration? -> add the area:migration label

@hardillb hardillb self-assigned this Feb 10, 2026
@hardillb hardillb requested a review from knolleary February 10, 2026 11:44
@ppawlowski
Copy link
Contributor

Shouldn't we drop this approach in favour of using NODE_OPTIONS environmental variable? IMO, this is a more universal solution.
Additionally, with this change, we will also have to update the system's service templates created by the Device Agent Installer.

Copy link
Member

@knolleary knolleary left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't think we should assign a short-alias for this.

Also need to resolve the feedback fro @ppawlowski - not sure the right approach.

@hardillb hardillb changed the title Allow Node-RED Heap Size to be set Allow NodeJS cmd line options to be passed to NodeRED process Feb 13, 2026
@hardillb hardillb dismissed knolleary’s stale review February 13, 2026 10:35

Nick is now out on holiday for 10 days

@hardillb hardillb merged commit 4c2dcc9 into main Feb 13, 2026
8 checks passed
@hardillb hardillb deleted the heap-size branch February 13, 2026 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow max_old_space_size to be set for device agent NR instance

4 participants