[docker] Load fixtures only on dev environment

[Ferror]

#813

Turning off relading the fixtures is just one line, and I decided not to overcomplicate the default config and just keep the fixtures reloading on the dev environment, but not on production and testing.

[vvasiloi]

Now run docker-compose run --rm php bin/console any:command and see what happens.

[Ferror]

Now run docker-compose run --rm php bin/console any:command and see what happens.

Why would you do docker compose run instead of docker compose exec? The container is up and running. Btw. Even without this change the service would start and execute everything that is in the docker-entrypoint.sh file. So I don't understand the point 😢

If you would like to use docker compose run... we should get rid of one of the if statements, but I don't believe that's the way the author wanted to use the container.

[vvasiloi]

The point is that the dev should be able to use docker-compose run php, docker-compose restart php, docker-compose up -d --scale php=3 10 times a day without resetting the database every time.
Let's say I set up a context for reproducing a bug or testing a feature, I might add tons of data, and then for some reason I have to restart the container. All my work is gone now.
I always made my own Docker setups for Sylius, so I don't care about it for myself., but as more people will start using it I'm sure more will complain, it's just a matter of time.

[Ferror]

The point is that the dev should be able to use docker-compose run php, docker-compose restart php, docker-compose up -d --scale php=3 10 times a day without resetting the database every time. Let's say I set up a context for reproducing a bug or testing a feature, I might add tons of data, and then for some reason I have to restart the container. All my work is gone now. I always made my own Docker setups for Sylius, so I don't care about it for myself., but as more people will start using it I'm sure more will complain, it's just a matter of time.

I think you may want to convert that message to the issue, as there is a specific thing you want to achieve and it's more like feedback for the whole PHP container execution than this particular change.

[Ferror]

It will be changed by other PR. #859