-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
failing deploys because of awk: write failure
#273
Comments
Thanks @Muffs! I deployed the awk change because These errors have started to appear recently; they tend to happen with the latest npm and with webpack, both of which yield verbose output. The quick-fix is to minimize output by configuring either npm or webpack to be quieter. Long-term we're looking into whether or not any platform changes could be limiting stdout. |
Good call, we'll figure out a way to minimize our webpack output. 😄 |
I am facing same error |
@FELS-Rohit @Muffs would you try this branch?
I'm interested to see if removing formatting altogether, with either |
@hunterloftis Thanks For Reply remote: readable-stream@1.0.33 /tmp/build_dbf888fac8fa0e62dbd7119f283cbd87/node_modules/loopback-component-stotee: write error remote: |
Thanks @FELS-Rohit, that's very helpful. It could be that the |
I'm also seeing this error:
Thankful for a fix or workaround! |
@antonholmquist you can probably lock your version to #v81:
It looks like Ubuntu's |
@FELS-Rohit okay, give it another go if you can. Also, if anyone is running into this and has an open-source project that consistently gives this issue, please let me know (so I can test it myself). So far I've been unable to reproduce the issue despite trying to simulate dumping a ton of output, and I hate asking you folks to test it repeatedly. |
Okay, I tried five times straight one hour ago and all failed. A few hours before that it failed the first time and passed right after. Now, when setting the buildback you suggested, it worked the first time. |
@hunterloftis I have removed the buildpacks stuff and test again |
@hunterloftis this is working fine with build pack |
Thanks for the help testing everyone. Because of your input I've been able to reproduce the issue and write a patch. The output function becomes somewhat more complex to handle all the cases; bash experts are welcome to help me simplify these buffering concerns: # format output on both OSX and Ubuntu, cedar-10 and cedar-14, while also splitting to a log file
output() {
local logfile="$1"
local c='s/^/ /'
if [ "${STACK:-cedar-14}" = "cedar" ]; then
tee -a "$logfile" | awk -W interactive '{ print " " $0 }'
else
case $(uname) in
Darwin) sed -l "$c" | tee -i -a "$logfile" 2> /dev/null;;
*) stdbuf -oL -eL sed -u "$c" | tee -i -a "$logfile" 2> /dev/null;;
esac
fi
} |
@hunterloftis Will review in morning its very late at my end |
This should be fixed in #v85; please let me know if you're still seeing it. |
i used #v85 and i'm still seeing the issue, but it doesn't even print
|
You're right @trinonsense, at a certain point (around 4000+ unbuffered stdout writes) it would still fail. I've just ripped out sed/awk/tee etc altogether and replaced them with a much simpler read/echo loop. |
Hey Heroku,
We began receiving
awk: write failure
errors this morning at ourwebpack -p
step. We are aware that we should be specifying a buildpack version and will make that change right now.Nonetheless, others may be encountering this issue. Hope this helps.
The text was updated successfully, but these errors were encountered: