Feat: added env support to Dokploy stack compose

[xenonwellz]

No description provided.

[Siumauricio]

Hey, thanks for the PR!

This works for stack, but is breaking the compose, I researched a little bit more about this problem and there is a option called docker stack config https://docs.docker.com/reference/cli/docker/stack/config/ and this command takes the provided stack file and it perform any neccesary mergind and variable sustitution(such enviroment variables) and outputs the result

I think is better this approach since we don't need to apply any regex validation to replace the values

[xenonwellz]

I thought of this too and initially looked to do this if it was possible.

The issue with it was, we'd have to export the env variables to the current shell which will not work with the spawnAsync function, but could be solved by parsing the .env file and passing it as the SpawnOption.env argument, then pass a command to generate the config file, this also means we will need to call another spawnAsync function. and for some reasons, it keeps throwing ENOENT error.

Also, I felt it would make more sense to structure the functions to match what we had before, writing files directly with writeAsync, and I decided to use regex to avoid installing a template rendering package.

I will look at the bug that breaks the docker-compose, and will push a fix.

Thanks.

[xenonwellz]

Apparently, the issue was, I did not generate a processed file when building without stack so it fails with an error indicating it could not find docker-config.processed.yml

[xenonwellz]

I have also added stop functionality for stack.

[Siumauricio]

Thanks, I will take a closer look at the docker stack config, because with this solution it does not work using this syntax

env_file:
  - .env

However using docker stack config it resolve the variables correctly

[xenonwellz]

Okay, I will try the docker stack config once more.

[xenonwellz]

I have updated it to use docker config. I was able to find my way around the bug i had earlier.

[Siumauricio]

It works great, thanks, I left you a few extra comments, let me know what you think!

[Siumauricio]

Check the request

[xenonwellz]

Log removed as requested

[Siumauricio]

I have tried with source type from github and it doesn't seem to work, neither with multi server feature it breaks it doesn't run docker compose correctly,

it only works when you use raw, I think this needs to be tested more, as we are definitely forgetting certain cases and this makes them break, and I'm afraid of that.

I think this might take a little longer, would you mind creating tests that are in the test folder, as I see that many cases fail and to prevent that it would be better to do that.