Skip to content

feat: add capture-output input to prevent GHA memory limit failures#47

Merged
raphaeltm merged 1 commit into
mainfrom
fix/optional-stdout-capture
Jun 17, 2026
Merged

feat: add capture-output input to prevent GHA memory limit failures#47
raphaeltm merged 1 commit into
mainfrom
fix/optional-stdout-capture

Conversation

@simple-agent-manager

@simple-agent-manager simple-agent-manager Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds a capture-output input (default: true) that controls whether the command's stdout is captured as a step output
  • When disabled, the command runs directly without buffering stdout into $GITHUB_OUTPUT, avoiding the GitHub Actions memory limit error:
    The maximum allowed memory size was exceeded while evaluating the following expression: steps.command.outputs.stdout
    
  • Full output is still visible in the workflow logs regardless of this setting

Context

Large Pulumi deployments (e.g. Portal staging with 125+ events) produce enough output to exceed GHA's expression evaluation memory limit. The deploy succeeds but the workflow fails, which blocks downstream jobs like production deployment.

No existing consumers of this action use outputs.stdout, so this is a safe default-on addition.

Usage

- name: Deploy
  uses: DefangLabs/defang-github-action@v2
  with:
    capture-output: false

Test plan

  • CI passes on this branch (existing tests use default capture-output: true)
  • Verify Portal deploy workflow works with capture-output: false after merge

🤖 Generated with Claude Code

Large deployments (e.g. Pulumi with many resources) can exceed GitHub
Actions' memory limit when the full stdout is captured as a step output.
This adds a `capture-output` input (default: true) that lets callers
disable stdout capture when they don't need it.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@simple-agent-manager simple-agent-manager Bot requested a review from a team as a code owner June 17, 2026 11:34
@raphaeltm raphaeltm merged commit 727d6c4 into main Jun 17, 2026
4 checks passed
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.

1 participant