Skip to content

Add mem alloc handling#199

Open
brandonmontijo wants to merge 6 commits into
mainfrom
Add-mem-alloc-handling
Open

Add mem alloc handling#199
brandonmontijo wants to merge 6 commits into
mainfrom
Add-mem-alloc-handling

Conversation

@brandonmontijo
Copy link
Copy Markdown
Contributor

Processing output as chunks instead of bytes.

Trying to fix this error (amplify-devops) in some pipeline and codebuild applies that have large outputs:

Traceback (most recent call last):
--
File "/usr/local/bin/tf", line 329, in <module>
handler()
File "/usr/local/bin/tf", line 318, in handler
exec_tf_command(
File "/usr/local/bin/tf", line 121, in exec_tf_command
exit_code, stdout = execute_command(
File "/usr/local/lib/python3.10/dist-packages/terrawrap/utils/cli.py", line 111, in execute_command
exit_code, stdout = _execute_command(
File "/usr/local/lib/python3.10/dist-packages/terrawrap/utils/cli.py", line 194, in _execute_command
output = stdout_read.read(1).decode(errors="replace")
OSError: [Errno 12] Cannot allocate memory

@brandonmontijo brandonmontijo requested a review from a team as a code owner October 21, 2025 17:55
Comment thread terrawrap/utils/cli.py Outdated
Comment thread terrawrap/utils/cli.py Outdated
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
2 Security Hotspots

See analysis details on SonarQube Cloud

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