Skip to content

Default NXF_FUSION_TRACE to false#7071

Merged
pditommaso merged 1 commit intomasterfrom
fusion-trace-default-false
Apr 24, 2026
Merged

Default NXF_FUSION_TRACE to false#7071
pditommaso merged 1 commit intomasterfrom
fusion-trace-default-false

Conversation

@pditommaso
Copy link
Copy Markdown
Member

Summary

  • Change the default for NXF_FUSION_TRACE from true to false so that Fusion trace-based resource metrics (CPU, memory, I/O) are opt-in rather than on by default.
  • Document the default in docs/reference/env-vars.md.

Test plan

  • Existing TaskHandlerTest / BashWrapperBuilderTest suites (which set the variable explicitly) still pass.
  • With the variable unset, the bash command-trace wrapper is used as before.

Change the default for the NXF_FUSION_TRACE environment variable from
true to false so that Fusion trace-based resource metrics are opt-in.
Update the docs accordingly.

Signed-off-by: Paolo Di Tommaso <paolo@seqera.io>
Signed-off-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com>
@pditommaso pditommaso requested a review from a team as a code owner April 24, 2026 13:05
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 24, 2026

Deploy Preview for nextflow-docs-staging ready!

Name Link
🔨 Latest commit 567e338
🔍 Latest deploy log https://app.netlify.com/projects/nextflow-docs-staging/deploys/69eb6a88a12b930008159c63
😎 Deploy Preview https://deploy-preview-7071--nextflow-docs-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@pditommaso pditommaso requested a review from bentsherman April 24, 2026 13:05
Copy link
Copy Markdown
Member

@bentsherman bentsherman left a comment

Choose a reason for hiding this comment

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

I don't have a strong a opinion on this, so I'm fine with either default

@pditommaso
Copy link
Copy Markdown
Member Author

Still not 100% sure with new metrics, safer to keep opt-in

@pditommaso pditommaso merged commit 5b4c8f0 into master Apr 24, 2026
13 checks passed
@pditommaso pditommaso deleted the fusion-trace-default-false branch April 24, 2026 14:48
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.

2 participants