Processes defined by empty string are broken (again!)

Yaeger · November 29, 2022, 8:49pm

A constraint has been added that requires an item in processes to not be an empty string. However, the Laravel configuration depends on this:

github.com

superfly/flyctl/blob/master/scanner/templates/laravel/.fly/entrypoint.sh

#!/usr/bin/env sh

# Run user scripts, if they exist
for f in /var/www/html/.fly/scripts/*.sh; do
    # Bail out this loop if any script exits with non-zero status code
    bash "$f" || break
done
chown -R webuser:webgroup /var/www/html

if [ $# -gt 0 ]; then
    # If we passed a command, run it as root
    exec "$@"
else
    exec /init
fi

It runs /init if no command is specified. Otherwise it runs the specified command.

Yaeger · November 29, 2022, 8:52pm

Btw, I said “again” because this update was also made in September:

Yaeger · November 29, 2022, 8:59pm

This update also makes this example not work anymore:

Yaeger · November 29, 2022, 9:00pm

I know the fix is simple, but it sucks a LOT to see a bunch of apps suddenly fail to deploy. Especially because it happened a few months ago as well.

jsierles · November 29, 2022, 9:33pm

Sorry about that. We just deployed a fix - can you try again?

Yaeger · November 29, 2022, 9:34pm

Going to try another app, but it worked fine on the first one I tried. Thanks!

Yaeger · November 29, 2022, 9:39pm

Yep, tried on two apps now. Seems fine, thank you!

Topic		Replies	Views
[Solved] Deploy fresh Laravel, page blank Questions / Help help-me-help-you	16	1765	September 28, 2022
Multiple processes: want to set process specific environment variables, but deployment fails because of missing command Questions / Help	4	609	September 11, 2023
Cronjobs that require env variables fail to execute Laravel	7	1534	October 4, 2022
Creating MySQl Volume - app = "--datadir /data/mysql error	2	147	September 28, 2023
Error we couldn't find a fly.toml nor an app specified by the -a flag. If you want to launch a new app, use 'flyctl launch' Questions / Help	2	537	March 26, 2023

Processes defined by empty string are broken (again!)

Related Topics