Skip to content

Add nodejs to dependencies in construct.yaml#182

Closed
ryuwd wants to merge 1 commit intoDIRACGrid:mainfrom
ryuwd:patch-1
Closed

Add nodejs to dependencies in construct.yaml#182
ryuwd wants to merge 1 commit intoDIRACGrid:mainfrom
ryuwd:patch-1

Conversation

@ryuwd
Copy link
Copy Markdown

@ryuwd ryuwd commented May 5, 2026

Needed for javascript substitution features in CWL workflows. Found this missing dependency while testing in certification...

ERROR I'm sorry, I couldn't load this CWL file, try again with --debug for more information.
The error was: NodeJSEngine requires Node.js engine to evaluate and validate Javascript expressions, but couldn't find it.  Tried nodejs, node, docker run node:alpine
Workflow failed with exit code 1 after 0:00:00

BEGINRELEASENOTES

NEW: Add nodejs to dependencies in construct.yaml

ENDRELEASENOTES

@ryuwd ryuwd requested a review from aldbr May 5, 2026 14:22
Copy link
Copy Markdown
Member

@chrisburr chrisburr left a comment

Choose a reason for hiding this comment

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

This is an extremely heavy dependency, so I think we should evaluate a little bit if we want to carry it

@chaen
Copy link
Copy Markdown
Collaborator

chaen commented May 5, 2026

This is an extremely heavy dependency, so I think we should evaluate a little bit if we want to carry it

I agree !

@ryuwd
Copy link
Copy Markdown
Author

ryuwd commented May 5, 2026

I will pull it in for now through singularity/apptainer, that way it only exists in the job payload in the specific context of JS expression evaluation, not in pods or voboxes

@ryuwd ryuwd closed this May 5, 2026
@ryuwd ryuwd deleted the patch-1 branch May 5, 2026 19:50
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.

4 participants